From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King Date: Sat, 26 Jun 2004 10:15:27 +0000 Subject: Re: [PATCH] Add PCDP console detection support Message-Id: <20040626111527.B21799@flint.arm.linux.org.uk> List-Id: References: <200406251504.50579.bjorn.helgaas@hp.com> In-Reply-To: <200406251504.50579.bjorn.helgaas@hp.com>; from bjorn.helgaas@hp.com on Fri, Jun 25, 2004 at 03:04:50PM -0600 MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Bjorn Helgaas Cc: Andrew Morton , linux-ia64@vger.kernel.org, linux-kernel@vger.kernel.org, Matt Tolentino , David Mosberger On Fri, Jun 25, 2004 at 03:04:50PM -0600, Bjorn Helgaas wrote: > This moves the code from drivers/serial/8250_hcdp.[ch] to > drivers/firmware/pcdp.[ch], since it's no longer 8250-specific. > It also obsoletes CONFIG_SERIAL_8250_HCDP, replacing it with > CONFIG_EFI_PCDP (which defaults to Y for ia64). I'm happy with this. The only thing I spotted on a quick read through was: > +static void __init > +setup_serial_console(int rev, struct pcdp_uart *uart) > +{ > +#ifdef CONFIG_SERIAL_8250_CONSOLE >... > + if (uart->addr.address_space_id = ACPI_ADR_SPACE_SYSTEM_MEMORY) { > + port.mapbase = uart->addr.address; > + port.membase = ioremap(port.mapbase, 64); What if ioremap fails (can it fail here?) > + port.iotype = SERIAL_IO_MEM; Should be UPIO_MEM please. Seems I missed this in the HCDP code... > + } else if (uart->addr.address_space_id = ACPI_ADR_SPACE_SYSTEM_IO) { > + port.iobase = uart->addr.address; > + port.iotype = SERIAL_IO_PORT; Likewise should be UPIO_PORT please. -- Russell King Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/ maintainer of: 2.6 PCMCIA - http://pcmcia.arm.linux.org.uk/ 2.6 Serial core