serial: make FSL errata depend on 8250_CONSOLE, not just 8250
authorPaul Gortmaker <paul.gortmaker@windriver.com>
Tue, 13 Dec 2011 02:47:38 +0000 (21:47 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 13 Dec 2011 17:50:13 +0000 (09:50 -0800)
The recent commit "serial: add irq handler for Freescale 16550 errata"
would allow Kconfig choices that had 8250 support as a module and
yet still try and build in the errata fix non-modular, resulting
in build failures for some non-embedded PPC targets.

Since we hook in the errata fix from legacy_serial.c, which is
built only for PPC_UDBG_16550, and since the errata is only really
relevant for SysRQ on serial console, tighten up the dependencies
to be exactly that.

We'll get coverage on the relevant Freescale boards because the
Kconfig for their CPU types all select the PPC_UDBG_16550 option,
and the defconfigs also all select the 8250_CONSOLE option.  Also,
the 8250_CONSOLE option has a strict dependency on "SERIAL_8250=y"
which resolves the reported problem for non Freescale targets.

Reported-by: Michael Neuling <mikey@neuling.org>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Tested-by: Michael Neuling <mikey@neuling.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/tty/serial/Kconfig

index fee9e04f42e729fd79e0c522cf648ab61cf76e25..d1e4f203ae619ff63b1b327ed36da34f1ef422cf 100644 (file)
@@ -99,7 +99,7 @@ config SERIAL_8250_PNP
 
 config SERIAL_8250_FSL
        bool
-       depends on SERIAL_8250 && PPC
+       depends on SERIAL_8250_CONSOLE && PPC_UDBG_16550
        default PPC
 
 config SERIAL_8250_HP300