public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Russell King <rmk+lkml@arm.linux.org.uk>
To: Olaf Zaplinski <o.zaplinski@broadnet-mediascape.de>,
	linux-kernel@vger.kernel.org, Greg KH <greg@kroah.com>
Subject: Re: 2.6.4: disabling SCSI support not possible
Date: Fri, 2 Apr 2004 14:42:17 +0100	[thread overview]
Message-ID: <20040402144216.A12306@flint.arm.linux.org.uk> (raw)
In-Reply-To: <6uad1uv7kr.fsf@zork.zork.net>; from sneakums@zork.net on Fri, Apr 02, 2004 at 02:21:40PM +0100

On Fri, Apr 02, 2004 at 02:21:40PM +0100, Sean Neakums wrote:
> Olaf Zaplinski <o.zaplinski@broadnet-mediascape.de> writes:
> 
> > I cannot disable SCSI completely in 2.6.4's 'menuconfig'.
> 
> I believe that some kernel components require SCSI to be useful and so
> force SCSI to be activated.  One example that springs to mind is
> usb-storage.

usb-storage should depend on SCSI rather than forcing SCSI to be
enabled.

Using 'select' is all very well for the case where the target
configuration symbol is not user selectable, but in the case that
it is, it leads to the confusion shown above.

Maybe USB_STORAGE help text should say that it needs SCSI support?

Therefore, I propose this patch:

--- orig/drivers/usb/storage/Kconfig	Sat Mar 20 09:22:45 2004
+++ linux/drivers/usb/storage/Kconfig	Fri Apr  2 14:41:05 2004
@@ -4,8 +4,7 @@
 
 config USB_STORAGE
 	tristate "USB Mass Storage support"
-	depends on USB
-	select SCSI
+	depends on USB && SCSI
 	---help---
 	  Say Y here if you want to connect USB mass storage devices to your
 	  computer's USB port. This is the driver you need for USB floppy drives,
@@ -13,6 +12,9 @@ config USB_STORAGE
 	  similar devices. This driver may also be used for some cameras and
 	  card readers.
 
+	  Please select SCSI support before enabling USB Mass Storage
+	  support.
+
 	  To compile this driver as a module, choose M here: the
 	  module will be called usb-storage.
 

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:  2.6 PCMCIA      - http://pcmcia.arm.linux.org.uk/
                 2.6 Serial core

  reply	other threads:[~2004-04-02 13:42 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-04-02 13:09 2.6.4: disabling SCSI support not possible Olaf Zaplinski
2004-04-02 13:18 ` Grzegorz Kulewski
2004-04-02 13:21 ` Sean Neakums
2004-04-02 13:42   ` Russell King [this message]
2004-04-02 14:03     ` Marc-Christian Petersen
2004-04-02 16:59     ` Greg KH
2004-04-02 17:16       ` Russell King
2004-04-02 17:56         ` Kevin P. Fleming
2004-04-02 18:14         ` Greg KH
2004-04-05 22:17         ` Bill Davidsen
2004-04-06  8:42           ` Russell King
2004-04-06 10:46             ` Marc-Christian Petersen
2004-04-06 11:03               ` Marc-Christian Petersen
2004-04-02 19:29     ` Ricky Beam

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20040402144216.A12306@flint.arm.linux.org.uk \
    --to=rmk+lkml@arm.linux.org.uk \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=o.zaplinski@broadnet-mediascape.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox