Commit b6efa1ea authored by Sonic Zhang's avatar Sonic Zhang Committed by Linus Torvalds

Blackfin Serial Driver: Clean serial console and early prink code.

Signed-off-by: default avatarSonic Zhang <sonic.zhang@analog.com>
Signed-off-by: default avatarBryan Wu <cooloney@kernel.org>
Signed-off-by: default avatarAlan Cox <alan@redhat.com>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 80d5c474
...@@ -986,7 +986,7 @@ static void __init bfin_serial_init_ports(void) ...@@ -986,7 +986,7 @@ static void __init bfin_serial_init_ports(void)
} }
#ifdef CONFIG_SERIAL_BFIN_CONSOLE #if defined(CONFIG_SERIAL_BFIN_CONSOLE) || defined(CONFIG_EARLY_PRINTK)
/* /*
* If the port was already initialised (eg, by a boot loader), * If the port was already initialised (eg, by a boot loader),
* try to determine the current setup. * try to determine the current setup.
...@@ -1030,16 +1030,13 @@ bfin_serial_console_get_options(struct bfin_serial_port *uart, int *baud, ...@@ -1030,16 +1030,13 @@ bfin_serial_console_get_options(struct bfin_serial_port *uart, int *baud,
} }
pr_debug("%s:baud = %d, parity = %c, bits= %d\n", __func__, *baud, *parity, *bits); pr_debug("%s:baud = %d, parity = %c, bits= %d\n", __func__, *baud, *parity, *bits);
} }
#endif
#if defined(CONFIG_SERIAL_BFIN_CONSOLE) || defined(CONFIG_EARLY_PRINTK)
static struct uart_driver bfin_serial_reg; static struct uart_driver bfin_serial_reg;
static int __init static int __init
bfin_serial_console_setup(struct console *co, char *options) bfin_serial_console_setup(struct console *co, char *options)
{ {
struct bfin_serial_port *uart; struct bfin_serial_port *uart;
# ifdef CONFIG_SERIAL_BFIN_CONSOLE
int baud = 57600; int baud = 57600;
int bits = 8; int bits = 8;
int parity = 'n'; int parity = 'n';
...@@ -1047,7 +1044,6 @@ bfin_serial_console_setup(struct console *co, char *options) ...@@ -1047,7 +1044,6 @@ bfin_serial_console_setup(struct console *co, char *options)
int flow = 'r'; int flow = 'r';
# else # else
int flow = 'n'; int flow = 'n';
# endif
# endif # endif
/* /*
...@@ -1059,16 +1055,12 @@ bfin_serial_console_setup(struct console *co, char *options) ...@@ -1059,16 +1055,12 @@ bfin_serial_console_setup(struct console *co, char *options)
co->index = 0; co->index = 0;
uart = &bfin_serial_ports[co->index]; uart = &bfin_serial_ports[co->index];
# ifdef CONFIG_SERIAL_BFIN_CONSOLE
if (options) if (options)
uart_parse_options(options, &baud, &parity, &bits, &flow); uart_parse_options(options, &baud, &parity, &bits, &flow);
else else
bfin_serial_console_get_options(uart, &baud, &parity, &bits); bfin_serial_console_get_options(uart, &baud, &parity, &bits);
return uart_set_options(&uart->port, co, baud, parity, bits, flow); return uart_set_options(&uart->port, co, baud, parity, bits, flow);
# else
return 0;
# endif
} }
#endif /* defined (CONFIG_SERIAL_BFIN_CONSOLE) || #endif /* defined (CONFIG_SERIAL_BFIN_CONSOLE) ||
defined (CONFIG_EARLY_PRINTK) */ defined (CONFIG_EARLY_PRINTK) */
...@@ -1177,7 +1169,7 @@ struct console __init *bfin_earlyserial_init(unsigned int port, ...@@ -1177,7 +1169,7 @@ struct console __init *bfin_earlyserial_init(unsigned int port,
return &bfin_early_serial_console; return &bfin_early_serial_console;
} }
#endif /* CONFIG_SERIAL_BFIN_CONSOLE */ #endif /* CONFIG_EARLY_PRINTK */
static struct uart_driver bfin_serial_reg = { static struct uart_driver bfin_serial_reg = {
.owner = THIS_MODULE, .owner = THIS_MODULE,
......
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