Commit f1e8c710 authored by Jean Delvare's avatar Jean Delvare Committed by Greg Kroah-Hartman

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: default avatarJean Delvare <jdelvare@suse.de>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Jiri Slaby <jslaby@suse.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 699a11ba
...@@ -117,7 +117,7 @@ config SERIAL_8250_DMA ...@@ -117,7 +117,7 @@ config SERIAL_8250_DMA
compatible UART controllers that support DMA signaling. compatible UART controllers that support DMA signaling.
config SERIAL_8250_PCI config SERIAL_8250_PCI
tristate "8250/16550 PCI device support" if EXPERT tristate "8250/16550 PCI device support"
depends on SERIAL_8250 && PCI depends on SERIAL_8250 && PCI
default SERIAL_8250 default SERIAL_8250
help help
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment