Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751065AbVJQRLX (ORCPT ); Mon, 17 Oct 2005 13:11:23 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751053AbVJQRLW (ORCPT ); Mon, 17 Oct 2005 13:11:22 -0400 Received: from mail.dvmed.net ([216.237.124.58]:58766 "EHLO mail.dvmed.net") by vger.kernel.org with ESMTP id S1751059AbVJQRLL (ORCPT ); Mon, 17 Oct 2005 13:11:11 -0400 Message-ID: <4353DB2C.10905@pobox.com> Date: Mon, 17 Oct 2005 13:11:08 -0400 From: Jeff Garzik User-Agent: Mozilla Thunderbird 1.0.7-1.1.fc4 (X11/20050929) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Linus Torvalds CC: Andrew Morton , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, davej@redhat.com Subject: Re: [PATCH] libata: fix broken Kconfig setup References: <20051017044606.GA1266@havoc.gtf.org> <4353C42A.3000005@pobox.com> <4353CF7E.1090404@pobox.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Score: 0.0 (/) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1565 Lines: 53 Linus Torvalds wrote: > > On Mon, 17 Oct 2005, Linus Torvalds wrote: > >>> And then with the quirk issue out of >>>the way, CONFIG_SCSI_SATA becomes purely a boolean enable/disable-this-menu >>>switch. >> >>No it does not. You continue to ignore the fact that it's not an >>enable/disable thing. It's a "can we enable SATA drivers" vs "can we >>enable SATA drivers as modules" vs "do we do any SATA drivers at all?" >>thing. >> >>A tristate. > > > Btw, if you want to have the _question_ always be y/n only, that's easy > enough to do, just make that one do > > config SATA_MENU > bool "Want to see SATA drivers" > depends on SCSI != n > > config SCSI_SATA > tristate > depends on SCSI && SATA_MENU > default y > > and now you have a totally sensible setup, where the low-level drivers can > depend on something sane. > > I don't think it _buys_ you anything, but hey, at least it's logical. That's a reasonable solution. I think it does buy you reduced user confusion. > Btw, wouldn't it be much nicer to also have this all in a totally separate > Kconfig file? That SCSI Kconfig file is one of the biggest ones (yeah, > drivers/net/Kconfig is bigger, but hey, that's not a surprise, is it ;) Honestly, I've been pondering moving all libata drivers to drivers/ata anyway... Jeff - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/