[PATCH] PCDP: handle tables that don't supply baud rate

The HCDP specs (i.e., PCDP revision < 3) allow zero as a default value for
baud rate and data bits.  So if firmware doesn't supply them, let
early_serial_console_init() probe for them rather than telling it the baud
rate is zero.

Also, update the URL for the PCDP spec.

Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

1 files changed, 7 insertions, 4 deletions

diff --git a/drivers/firmware/pcdp.c b/drivers/firmware/pcdp.c
index df1b721154d2..839b44a7e08b 100644
--- a/drivers/firmware/pcdp.c
+++ b/drivers/firmware/pcdp.c
@@ -23,12 +23,15 @@ setup_serial_console(struct pcdp_uart *uart)
 {
 #ifdef CONFIG_SERIAL_8250_CONSOLE
         int mmio;
-        static char options[64];
+        static char options[64], *p = options;
 
         mmio = (uart->addr.address_space_id == ACPI_ADR_SPACE_SYSTEM_MEMORY);
-        snprintf(options, sizeof(options), "console=uart,%s,0x%lx,%lun%d",
-                mmio ? "mmio" : "io", uart->addr.address, uart->baud,
-                uart->bits ? uart->bits : 8);
+        p += sprintf(p, "console=uart,%s,0x%lx",
+                mmio ? "mmio" : "io", uart->addr.address);
+        if (uart->baud)
+                p += sprintf(p, ",%lu", uart->baud);
+        if (uart->bits)
+                p += sprintf(p, "n%d", uart->bits);
 
         return early_serial_console_init(options);
 #else