Commit 3ae988d9 authored by Laurent Pinchart's avatar Laurent Pinchart Committed by Simon Horman

serial: sh-sci: Move overrun_bit and error_mask fields out of pdata

None of the fields is ever set by board code, and both of them are set
in the driver at probe time. Move them out of struct plat_sci_port to
struct sci_port.
Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Acked-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarSimon Horman <horms+renesas@verge.net.au>
parent 1fcc91a6
...@@ -64,6 +64,9 @@ struct sci_port { ...@@ -64,6 +64,9 @@ struct sci_port {
/* Platform configuration */ /* Platform configuration */
struct plat_sci_port *cfg; struct plat_sci_port *cfg;
int overrun_bit;
unsigned int error_mask;
/* Break timer */ /* Break timer */
struct timer_list break_timer; struct timer_list break_timer;
...@@ -760,19 +763,15 @@ static int sci_handle_errors(struct uart_port *port) ...@@ -760,19 +763,15 @@ static int sci_handle_errors(struct uart_port *port)
struct tty_port *tport = &port->state->port; struct tty_port *tport = &port->state->port;
struct sci_port *s = to_sci_port(port); struct sci_port *s = to_sci_port(port);
/* /* Handle overruns */
* Handle overruns, if supported. if (status & (1 << s->overrun_bit)) {
*/ port->icount.overrun++;
if (s->cfg->overrun_bit != SCIx_NOT_SUPPORTED) {
if (status & (1 << s->cfg->overrun_bit)) {
port->icount.overrun++;
/* overrun error */ /* overrun error */
if (tty_insert_flip_char(tport, 0, TTY_OVERRUN)) if (tty_insert_flip_char(tport, 0, TTY_OVERRUN))
copied++; copied++;
dev_notice(port->dev, "overrun error"); dev_notice(port->dev, "overrun error");
}
} }
if (status & SCxSR_FER(port)) { if (status & SCxSR_FER(port)) {
...@@ -834,7 +833,7 @@ static int sci_handle_fifo_overrun(struct uart_port *port) ...@@ -834,7 +833,7 @@ static int sci_handle_fifo_overrun(struct uart_port *port)
if (!reg->size) if (!reg->size)
return 0; return 0;
if ((serial_port_in(port, SCLSR) & (1 << s->cfg->overrun_bit))) { if ((serial_port_in(port, SCLSR) & (1 << s->overrun_bit))) {
serial_port_out(port, SCLSR, 0); serial_port_out(port, SCLSR, 0);
port->icount.overrun++; port->icount.overrun++;
...@@ -2253,28 +2252,25 @@ static int sci_init_single(struct platform_device *dev, ...@@ -2253,28 +2252,25 @@ static int sci_init_single(struct platform_device *dev,
/* /*
* Establish some sensible defaults for the error detection. * Establish some sensible defaults for the error detection.
*/ */
if (!p->error_mask) sci_port->error_mask = (p->type == PORT_SCI) ?
p->error_mask = (p->type == PORT_SCI) ?
SCI_DEFAULT_ERROR_MASK : SCIF_DEFAULT_ERROR_MASK; SCI_DEFAULT_ERROR_MASK : SCIF_DEFAULT_ERROR_MASK;
/* /*
* Establish sensible defaults for the overrun detection, unless * Establish sensible defaults for the overrun detection, unless
* the part has explicitly disabled support for it. * the part has explicitly disabled support for it.
*/ */
if (p->overrun_bit != SCIx_NOT_SUPPORTED) { if (p->type == PORT_SCI)
if (p->type == PORT_SCI) sci_port->overrun_bit = 5;
p->overrun_bit = 5; else if (p->scbrr_algo_id == SCBRR_ALGO_4)
else if (p->scbrr_algo_id == SCBRR_ALGO_4) sci_port->overrun_bit = 9;
p->overrun_bit = 9; else
else sci_port->overrun_bit = 0;
p->overrun_bit = 0;
/* /*
* Make the error mask inclusive of overrun detection, if * Make the error mask inclusive of overrun detection, if
* supported. * supported.
*/ */
p->error_mask |= (1 << p->overrun_bit); sci_port->error_mask |= 1 << sci_port->overrun_bit;
}
port->type = p->type; port->type = p->type;
port->flags = UPF_FIXED_PORT | p->flags; port->flags = UPF_FIXED_PORT | p->flags;
......
...@@ -9,7 +9,7 @@ ...@@ -9,7 +9,7 @@
#define SCxSR_PER(port) (((port)->type == PORT_SCI) ? SCI_PER : SCIF_PER) #define SCxSR_PER(port) (((port)->type == PORT_SCI) ? SCI_PER : SCIF_PER)
#define SCxSR_BRK(port) (((port)->type == PORT_SCI) ? 0x00 : SCIF_BRK) #define SCxSR_BRK(port) (((port)->type == PORT_SCI) ? 0x00 : SCIF_BRK)
#define SCxSR_ERRORS(port) (to_sci_port(port)->cfg->error_mask) #define SCxSR_ERRORS(port) (to_sci_port(port)->error_mask)
#if defined(CONFIG_CPU_SUBTYPE_SH7705) || \ #if defined(CONFIG_CPU_SUBTYPE_SH7705) || \
defined(CONFIG_CPU_SUBTYPE_SH7720) || \ defined(CONFIG_CPU_SUBTYPE_SH7720) || \
......
...@@ -152,9 +152,6 @@ struct plat_sci_port { ...@@ -152,9 +152,6 @@ struct plat_sci_port {
/* /*
* Platform overrides if necessary, defaults otherwise. * Platform overrides if necessary, defaults otherwise.
*/ */
int overrun_bit;
unsigned int error_mask;
int port_reg; int port_reg;
unsigned char regshift; unsigned char regshift;
unsigned char regtype; unsigned char regtype;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment