Messages in this thread | | | Date | Wed, 3 Mar 2010 21:14:02 +0300 | From | Anton Vorontsov <> | Subject | Re: [PATCH 10/11] ahci: Export generic AHCI symbols, turn libahci into a separate module |
| |
On Wed, Mar 03, 2010 at 09:08:34PM +0300, Anton Vorontsov wrote: > On Wed, Mar 03, 2010 at 08:53:30PM +0300, Sergei Shtylyov wrote: > [...] > > >diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig > > >index 01c52c4..365a9da 100644 > > >--- a/drivers/ata/Kconfig > > >+++ b/drivers/ata/Kconfig > > >@@ -59,12 +59,18 @@ config SATA_PMP > > > config SATA_AHCI > > > tristate "AHCI SATA support" > > >- depends on PCI > > > help > > > This option enables support for AHCI Serial ATA. > > > If unsure, say N. > > >+config SATA_AHCI_PCI > > >+ tristate "PCI AHCI SATA support" > > >+ depends on SATA_AHCI && PCI > > > > Perhaps it's worth to turn that into select SATA_AHCI and make > > SATA_AHCI invisible to users instead, so that each particular AHCI > > implementation could select it. > > No, we must maintain old .config's semantics. SATA_AHCI is already > visible to the users, and if we remove it, we'll break > 'make oldconfig'. > > > >+ default y > > > > Why force the default "y" if the predecessor option SATA_AHCI > > didn't do it? > > For the same reason, to keep old .configs working.
OTOH, we can introduce invisible SATA_LIBAHCI, which is better, I think.
| |