Re: Linux 2.6.22-rc1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi David,

On 5/14/07, David Miller <[email protected]> wrote:
From: Tilman Schmidt <[email protected]>
Date: Sun, 13 May 2007 20:19:09 +0200

> Would it be asking too much to have help texts on the following
> new (wrt 2.6.21) configuration options?
>
> ESP Scsi Driver Core (SCSI_ESP_CORE) [N/m] (NEW)

This one is a case where the option makes no sense by itself,
it provides the core common code for other front-end drivers.

The documentation exists in those front end drivers, which
in turn auto-matically select and enable this config option.

It would be nice if there was a Kconfig way to not provide
this thing at all unless one of the front-ends got selected

Yeah, so the Kconfig-blessed way that you're looking for is:

config SCSI_ESP_CORE
	tristate
	depends on SCSI && SCSI_SPI_ATTRS
	default y if SCSI_SUNESP=y
	default m if SCSI_SUNESP=m

Add other front-ends (other than SCSI_SUNESP) to the last
two lines, as applicable.

Important: And then *remove* the "select SCSI_ESP_CORE"
from the Kconfig options of the front-ends themselves.

[ You could do something similar with SCSI_SPI_ATTRS
also, it looks like to be something that probably never makes
sense alone either, and needs only to be automatically pulled
in by other options (?) ]

[ Note that "select"ing stuff that is not library-like and itself
depends on other stuff is a Bad Thing. ]

but on the otherhand I like how anyone can select it and thus
test the build of it :-)

Satyam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux