diff options
| author | Jean Delvare | 2016-12-19 10:58:29 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2017-01-12 11:51:25 +0100 |
| commit | f1e8c710e20c6e5d4546396c08bb7ac28a06ed18 (patch) | |
| tree | 8eac6ad7f6921cfbcf494afd6a3a2fc8b49f8dc4 /drivers/tty | |
| parent | serial: 8250_port: Remove dangerous pr_debug() (diff) | |
| download | kernel-qcow2-linux-f1e8c710e20c6e5d4546396c08bb7ac28a06ed18.tar.gz kernel-qcow2-linux-f1e8c710e20c6e5d4546396c08bb7ac28a06ed18.tar.xz kernel-qcow2-linux-f1e8c710e20c6e5d4546396c08bb7ac28a06ed18.zip | |
serial: 8250_pci: make option visible
Hiding tristate options with "if EXPERT" is usually not a good idea.
You can decide that the driver should be included by default, but you
don't know if the user wants it built-in or as a module. Hiding the
option prevents the user from making that decision.
In this specific case, driver 8250_pci ends up being built-in as soon
as SERIAL_8250=y. It is very common for distribution kernels to build
the subsystem core code into the kernel, because almost everybody
will need it, but build all the device drivers as modules. This
should be made possible.
So drop the "if EXPERT" and make SERIAL_8250_PCI visible.
Signed-off-by: Jean Delvare <jdelvare@suse.de>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Jiri Slaby <jslaby@suse.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty')
| -rw-r--r-- | drivers/tty/serial/8250/Kconfig | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/tty/serial/8250/Kconfig b/drivers/tty/serial/8250/Kconfig index 0b8b6740ba43..a20f52590e4d 100644 --- a/drivers/tty/serial/8250/Kconfig +++ b/drivers/tty/serial/8250/Kconfig @@ -117,7 +117,7 @@ config SERIAL_8250_DMA compatible UART controllers that support DMA signaling. config SERIAL_8250_PCI - tristate "8250/16550 PCI device support" if EXPERT + tristate "8250/16550 PCI device support" depends on SERIAL_8250 && PCI default SERIAL_8250 help |