Return-Path: Sender: Johan Hovold Date: Sat, 7 Oct 2017 17:24:14 +0200 From: Johan Hovold To: Ian W MORRISON Cc: marcel@holtmann.org, gustavo@padovan.org, johan.hedberg@gmail.com, "bluez mailin list (linux-bluetooth@vger.kernel.org)" , Hans de Goede , =?iso-8859-1?Q?Fr=E9d=E9ric?= Danis , Rob Herring , Sebastian Reichel , Loic Poulain , Johan Hovold , Lukas Wunner , linux-serial@vger.kernel.org, linux-acpi@vger.kernel.org, Greg Kroah-Hartman , "Rafael J. Wysocki" Subject: Re: [PATCH] serdev: Update drivers/tty/serdev/Kconfig for ACPI support Message-ID: <20171007152414.GK2618@localhost> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: List-ID: 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