Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756448AbYJUWdg (ORCPT ); Tue, 21 Oct 2008 18:33:36 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751783AbYJUWdY (ORCPT ); Tue, 21 Oct 2008 18:33:24 -0400 Received: from willie.n0ano.com ([64.66.241.76]:56210 "EHLO xolotl.n0ano.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751969AbYJUWdX (ORCPT ); Tue, 21 Oct 2008 18:33:23 -0400 From: n0ano@n0ano.com Date: Tue, 21 Oct 2008 16:33:04 -0600 To: Bjorn Helgaas Cc: n0ano@n0ano.com, Lennart Sorensen , Alan Cox , donald.d.dugger@intel.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Enable console on PCI serial devices Message-ID: <20081021223304.GM14490@xolotl.n0ano.com> References: <200810142231.m9EMV95O002974@los-vmm.sc.intel.com> <200810201311.06602.bjorn.helgaas@hp.com> <20081021200841.GL14490@xolotl.n0ano.com> <200810211608.04121.bjorn.helgaas@hp.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200810211608.04121.bjorn.helgaas@hp.com> User-Agent: Mutt/1.4.2.1i Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3060 Lines: 76 On Tue, Oct 21, 2008 at 04:08:03PM -0600, Bjorn Helgaas wrote: >... > Down to nits now :-) That last sentence would be clearer to me if it > said '(typically "base" is the fastest baud rate supported by the UART).' > I capitalized UART to match the one earlier. NP, consistency is a good thing. Signed-off-by: Don Dugger -- Don Dugger "Censeo Toto nos in Kansa esse decisse." - D. Gale n0ano@n0ano.com Ph: 303/443-3786 Documentation/kernel-parameters.txt | 14 +++++++++----- drivers/serial/8250_early.c | 5 +++++ 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt index 82c561f..f07e77c 100644 --- a/Documentation/kernel-parameters.txt +++ b/Documentation/kernel-parameters.txt @@ -505,12 +505,16 @@ and is between 256 and 4096 characters. It is defined in the file Documentation/networking/netconsole.txt for an alternative. - uart[8250],io,[,options] - uart[8250],mmio,[,options] + uart[8250],io,[,options[,base]] + uart[8250],mmio,[,options[,base]] Start an early, polled-mode console on the 8250/16550 UART at the specified I/O port or MMIO address, switching to the matching ttyS device later. The - options are the same as for ttyS, above. + "options" are the same as for ttyS, above. "base" + is base baud, defaults to 115200, which is used to + calculate the divisor for the crystal frequency + (typically "base" is the fastest baud rate supported + by the UART). If the device connected to the port is not a TTY but a braille device, prepend "brl," before the device type, for instance @@ -518,8 +522,8 @@ and is between 256 and 4096 characters. It is defined in the file For now, only VisioBraille is supported. earlycon= [KNL] Output early console device and options. - uart[8250],io,[,options] - uart[8250],mmio,[,options] + uart[8250],io,[,options,[base]] + uart[8250],mmio,[,options,[base]] Start an early, polled-mode console on the 8250/16550 UART at the specified I/O port or MMIO address. The options are the same as for ttyS, above. diff --git a/drivers/serial/8250_early.c b/drivers/serial/8250_early.c index f279745..edd5803 100644 --- a/drivers/serial/8250_early.c +++ b/drivers/serial/8250_early.c @@ -180,6 +180,11 @@ static int __init parse_options(struct early_serial8250_device *device, snprintf(device->options, sizeof(device->options), "%u", device->baud); } + options = strchr(options, ','); + if (options) { + options++; + port->uartclk = simple_strtoul(options, NULL, 0) * 16; + } printk(KERN_INFO "Early serial console at %s 0x%llx (options '%s')\n", mmio ? "MMIO" : "I/O port", -- 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/