On Sat, Oct 07, 2017 at 05:16:31PM +1100, Ian W MORRISON wrote: > The current Kconfig for serdev is not compatible when adding ACPI support as it does not work when built as a module as it requires config SERIAL_DEV_CTRL_TTYPORT to be set. This patch makes serdev compiled into the kernel if selected so that config SERIAL_DEV_CTRL_TTYPORT can be correctly set if requiring ACPI support. > --- > drivers/tty/serdev/Kconfig | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/tty/serdev/Kconfig b/drivers/tty/serdev/Kconfig > index cdc6b820cf93..a9fb09a9c105 100644 > --- a/drivers/tty/serdev/Kconfig > +++ b/drivers/tty/serdev/Kconfig > @@ -2,7 +2,8 @@ > # Serial bus device driver configuration > # > menuconfig SERIAL_DEV_BUS > - tristate "Serial device bus" > + bool "Serial device bus" > + default y I understand why you want this (to prevent hci_bcm from breaking), but we should generally not have new entries default to y. > help > Core support for devices connected via a serial port. > > @@ -11,6 +12,6 @@ if SERIAL_DEV_BUS > config SERIAL_DEV_CTRL_TTYPORT > bool "Serial device TTY port controller" > depends on TTY > - depends on SERIAL_DEV_BUS != m > + default y Same here. It may be better to have BT_HCIUART_BCM depend on (or select?) SERIAL_DEV_CTRL_TTYPORT instead. Johan -- 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
