Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753500AbbFHSHu (ORCPT ); Mon, 8 Jun 2015 14:07:50 -0400 Received: from mga03.intel.com ([134.134.136.65]:9027 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752988AbbFHSHn (ORCPT ); Mon, 8 Jun 2015 14:07:43 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,574,1427785200"; d="scan'208";a="707405845" Date: Mon, 8 Jun 2015 11:19:37 -0700 From: Bin Gao To: Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Greg Kroah-Hartman Cc: One Thousand Gnomes , Peter Hurley , Jiri Slaby , Borislav Petkov , "Stuart R. Anderson" , linux-kernel@vger.kernel.org, bin.gao@linux.intel.com Subject: [PATCH v6 2/2] arch/x86: remove pci uart early console from early_prink.c Message-ID: <20150608181936.GB181146@worksta> References: <20150529184123.GB13090@worksta> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150529184123.GB13090@worksta> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 9034 Lines: 306 The arch independent uart8250 early console driver has good support for memory mapped and io port based 8250 uarts. Since pci is arch independent so it's natural to extend uart8250 to support mem, io and pci. Hence pci uart early console in arch/x86/kernel_printk.c by the following commit: 'commit 5140fda16051 ("Specify PCI based UART for earlyprintk")' is removed. And its equivalent function will be available from uart8250 early console driver. Signed-off-by: Bin Gao --- Changes in v6: - limited the early parameter 'earlyprintk' in drivers/tty/serial/earlycon.c to uart8250,pci[32][,options] only Changes in v5: - moved earlyprintk= (an alias to earlycon=) from patch 1/2 to patch 2/2 Changes in v2-v4: None arch/x86/kernel/early_printk.c | 180 ++++------------------------------------- drivers/tty/serial/earlycon.c | 15 ++++ 2 files changed, 30 insertions(+), 165 deletions(-) diff --git a/arch/x86/kernel/early_printk.c b/arch/x86/kernel/early_printk.c index 89427d8..00c2e2a 100644 --- a/arch/x86/kernel/early_printk.c +++ b/arch/x86/kernel/early_printk.c @@ -19,7 +19,6 @@ #include #include #include -#include /* Simple VGA output */ #define VGABASE (__ISA_IO_base + 0xb8000) @@ -77,7 +76,7 @@ static struct console early_vga_console = { /* Serial functions loosely based on a similar package from Klaus P. Gerlicher */ -static unsigned long early_serial_base = 0x3f8; /* ttyS0 */ +static int early_serial_base = 0x3f8; /* ttyS0 */ #define XMTRDY 0x20 @@ -95,26 +94,13 @@ static unsigned long early_serial_base = 0x3f8; /* ttyS0 */ #define DLL 0 /* Divisor Latch Low */ #define DLH 1 /* Divisor latch High */ -static unsigned int io_serial_in(unsigned long addr, int offset) -{ - return inb(addr + offset); -} - -static void io_serial_out(unsigned long addr, int offset, int value) -{ - outb(value, addr + offset); -} - -static unsigned int (*serial_in)(unsigned long addr, int offset) = io_serial_in; -static void (*serial_out)(unsigned long addr, int offset, int value) = io_serial_out; - static int early_serial_putc(unsigned char ch) { unsigned timeout = 0xffff; - while ((serial_in(early_serial_base, LSR) & XMTRDY) == 0 && --timeout) + while ((inb(early_serial_base + LSR) & XMTRDY) == 0 && --timeout) cpu_relax(); - serial_out(early_serial_base, TXR, ch); + outb(ch, early_serial_base + TXR); return timeout ? 0 : -1; } @@ -128,28 +114,13 @@ static void early_serial_write(struct console *con, const char *s, unsigned n) } } -static __init void early_serial_hw_init(unsigned divisor) -{ - unsigned char c; - - serial_out(early_serial_base, LCR, 0x3); /* 8n1 */ - serial_out(early_serial_base, IER, 0); /* no interrupt */ - serial_out(early_serial_base, FCR, 0); /* no fifo */ - serial_out(early_serial_base, MCR, 0x3); /* DTR + RTS */ - - c = serial_in(early_serial_base, LCR); - serial_out(early_serial_base, LCR, c | DLAB); - serial_out(early_serial_base, DLL, divisor & 0xff); - serial_out(early_serial_base, DLH, (divisor >> 8) & 0xff); - serial_out(early_serial_base, LCR, c & ~DLAB); -} - #define DEFAULT_BAUD 9600 static __init void early_serial_init(char *s) { + unsigned char c; unsigned divisor; - unsigned long baud = DEFAULT_BAUD; + unsigned baud = DEFAULT_BAUD; char *e; if (*s == ',') @@ -174,138 +145,23 @@ static __init void early_serial_init(char *s) s++; } - if (*s) { - if (kstrtoul(s, 0, &baud) < 0 || baud == 0) - baud = DEFAULT_BAUD; - } - - /* Convert from baud to divisor value */ - divisor = 115200 / baud; - - /* These will always be IO based ports */ - serial_in = io_serial_in; - serial_out = io_serial_out; - - /* Set up the HW */ - early_serial_hw_init(divisor); -} - -#ifdef CONFIG_PCI -static void mem32_serial_out(unsigned long addr, int offset, int value) -{ - u32 *vaddr = (u32 *)addr; - /* shift implied by pointer type */ - writel(value, vaddr + offset); -} - -static unsigned int mem32_serial_in(unsigned long addr, int offset) -{ - u32 *vaddr = (u32 *)addr; - /* shift implied by pointer type */ - return readl(vaddr + offset); -} - -/* - * early_pci_serial_init() - * - * This function is invoked when the early_printk param starts with "pciserial" - * The rest of the param should be ",B:D.F,baud" where B, D & F describe the - * location of a PCI device that must be a UART device. - */ -static __init void early_pci_serial_init(char *s) -{ - unsigned divisor; - unsigned long baud = DEFAULT_BAUD; - u8 bus, slot, func; - u32 classcode, bar0; - u16 cmdreg; - char *e; - - - /* - * First, part the param to get the BDF values - */ - if (*s == ',') - ++s; - - if (*s == 0) - return; - - bus = (u8)simple_strtoul(s, &e, 16); - s = e; - if (*s != ':') - return; - ++s; - slot = (u8)simple_strtoul(s, &e, 16); - s = e; - if (*s != '.') - return; - ++s; - func = (u8)simple_strtoul(s, &e, 16); - s = e; - - /* A baud might be following */ - if (*s == ',') - s++; - - /* - * Second, find the device from the BDF - */ - cmdreg = read_pci_config(bus, slot, func, PCI_COMMAND); - classcode = read_pci_config(bus, slot, func, PCI_CLASS_REVISION); - bar0 = read_pci_config(bus, slot, func, PCI_BASE_ADDRESS_0); - - /* - * Verify it is a UART type device - */ - if (((classcode >> 16 != PCI_CLASS_COMMUNICATION_MODEM) && - (classcode >> 16 != PCI_CLASS_COMMUNICATION_SERIAL)) || - (((classcode >> 8) & 0xff) != 0x02)) /* 16550 I/F at BAR0 */ - return; - - /* - * Determine if it is IO or memory mapped - */ - if (bar0 & 0x01) { - /* it is IO mapped */ - serial_in = io_serial_in; - serial_out = io_serial_out; - early_serial_base = bar0&0xfffffffc; - write_pci_config(bus, slot, func, PCI_COMMAND, - cmdreg|PCI_COMMAND_IO); - } else { - /* It is memory mapped - assume 32-bit alignment */ - serial_in = mem32_serial_in; - serial_out = mem32_serial_out; - /* WARNING! assuming the address is always in the first 4G */ - early_serial_base = - (unsigned long)early_ioremap(bar0 & 0xfffffff0, 0x10); - write_pci_config(bus, slot, func, PCI_COMMAND, - cmdreg|PCI_COMMAND_MEMORY); - } + outb(0x3, early_serial_base + LCR); /* 8n1 */ + outb(0, early_serial_base + IER); /* no interrupt */ + outb(0, early_serial_base + FCR); /* no fifo */ + outb(0x3, early_serial_base + MCR); /* DTR + RTS */ - /* - * Lastly, initalize the hardware - */ if (*s) { - if (strcmp(s, "nocfg") == 0) - /* Sometimes, we want to leave the UART alone - * and assume the BIOS has set it up correctly. - * "nocfg" tells us this is the case, and we - * should do no more setup. - */ - return; if (kstrtoul(s, 0, &baud) < 0 || baud == 0) baud = DEFAULT_BAUD; } - /* Convert from baud to divisor value */ divisor = 115200 / baud; - - /* Set up the HW */ - early_serial_hw_init(divisor); + c = inb(early_serial_base + LCR); + outb(c | DLAB, early_serial_base + LCR); + outb(divisor & 0xff, early_serial_base + DLL); + outb((divisor >> 8) & 0xff, early_serial_base + DLH); + outb(c & ~DLAB, early_serial_base + LCR); } -#endif static struct console early_serial_console = { .name = "earlyser", @@ -326,6 +182,7 @@ static inline void early_console_register(struct console *con, int keep_early) early_console->flags &= ~CON_BOOT; else early_console->flags |= CON_BOOT; + register_console(early_console); } @@ -353,13 +210,6 @@ static int __init setup_early_printk(char *buf) early_serial_init(buf + 4); early_console_register(&early_serial_console, keep); } -#ifdef CONFIG_PCI - if (!strncmp(buf, "pciserial", 9)) { - early_pci_serial_init(buf + 9); - early_console_register(&early_serial_console, keep); - buf += 9; /* Keep from match the above "serial" */ - } -#endif if (!strncmp(buf, "vga", 3) && boot_params.screen_info.orig_video_isVGA == 1) { max_xpos = boot_params.screen_info.orig_video_cols; diff --git a/drivers/tty/serial/earlycon.c b/drivers/tty/serial/earlycon.c index 6dc471e..4ab822c 100644 --- a/drivers/tty/serial/earlycon.c +++ b/drivers/tty/serial/earlycon.c @@ -193,6 +193,21 @@ static int __init param_setup_earlycon(char *buf) } early_param("earlycon", param_setup_earlycon); +/* + * X86 uses "earlyprintk=xxx", so we keep the compatibility here. + * But we only handle the earlyprintk=uart8250,pci[32][,options] case. + */ +#ifdef CONFIG_X86 +static int __init param_setup_earlycon_x86(char *buf) +{ + if (strncmp(buf, "uart8250,pci", 12)) + return -EINVAL; + + return param_setup_earlycon(buf); +} +early_param("earlyprintk", param_setup_earlycon_x86); +#endif + int __init of_setup_earlycon(unsigned long addr, int (*setup)(struct earlycon_device *, const char *)) { -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/