On Wed, Jan 28, 2015 at 11:37 AM, Peter Hurley <peter@xxxxxxxxxxxxxxxxxx> wrote: > On 01/27/2015 11:44 AM, Rob Herring wrote: >> On Tue, Jan 27, 2015 at 9:09 AM, Peter Hurley <peter@xxxxxxxxxxxxxxxxxx> wrote: > > [...] > >>> Maybe Kconfig should warn if they're both built-in or both modules? >> >> Is there a way to do that? > > Well, it's not what I had in mind originally, but the diff below > generates a broken dependencies warning without causing build problems: > $ scripts/kconfig/mconf Kconfig > warning: (SERIAL_PXA) selects TTYS_DRIVER_PXA which has unmet direct dependencies (TTY && HAS_IOMEM && !TTYS_DRIVER) > > (My original idea was thwarted by the requirement that choice/endchoice > requires prompts). Okay, but this should be a separate patch. The problem exists with or without my patch. > That said, for PXA, I think we agree splitting out a standalone 8250 platform > driver is the solution. So after all the discussion, you are okay with the original patch? Rob > > Regards, > Peter Hurley > > -- > diff --git a/drivers/tty/serial/8250/Kconfig b/drivers/tty/serial/8250/Kconfig > index 0fcbcd2..c367542 100644 > --- a/drivers/tty/serial/8250/Kconfig > +++ b/drivers/tty/serial/8250/Kconfig > @@ -6,6 +6,7 @@ > config SERIAL_8250 > tristate "8250/16550 and compatible serial support" > select SERIAL_CORE > + select TTYS_DRIVER > ---help--- > This selects whether you want to include the driver for the standard > serial ports. The standard answer is Y. People who might say N > diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig > index c79b43c..10c3e27 100644 > --- a/drivers/tty/serial/Kconfig > +++ b/drivers/tty/serial/Kconfig > @@ -7,6 +7,13 @@ if TTY > menu "Serial drivers" > depends on HAS_IOMEM > > +config TTYS_DRIVER > + tristate > + > +config TTYS_DRIVER_PXA > + tristate > + depends on !TTYS_DRIVER > + > config SERIAL_EARLYCON > bool > help > @@ -438,6 +445,7 @@ config SERIAL_PXA > bool "PXA serial port support" > depends on ARCH_PXA || ARCH_MMP > select SERIAL_CORE > + select TTYS_DRIVER_PXA > help > If you have a machine based on an Intel XScale PXA2xx CPU you > can enable its onboard serial ports by enabling this option. > > -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
