Re: [PATCH 10/11] ahci: Export generic AHCI symbols, turn libahci into a separate module

[Anton Vorontsov <avorontsov@ru.mvista.com>]

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.

Thanks,

-- 
Anton Vorontsov
email: cbouatmailru@gmail.com
irc://irc.freenode.net/bd2