Re: [PATCH] usb-storage should enable scsi disk in Kconfig

["Randy.Dunlap" <rddunlap@osdl.org>]

[-- Attachment #1: Type: text/plain, Size: 1721 bytes --]

Matthew Dharm wrote:
> On Fri, Nov 19, 2004 at 10:39:42PM +0100, Stelian Pop wrote:
> 
>>On Fri, Nov 19, 2004 at 07:57:36PM +0000, Christoph Hellwig wrote:
>>
>>
>>>On Fri, Nov 19, 2004 at 08:33:50PM +0100, Stelian Pop wrote:
>>>
>>>>As $subject says, usb-storage storage should automatically enable
>>>>scsi disk support in Kconfig.
>>>>
>>>>Please apply.
>>>
>>>No, it shouldn't.  There's lots of usb storage devices that don't use
>>>sd, as there are lots of SPI/FC/etc.. devices.
>>
>>Indeed, I should have checked more carefully. My bad.
>>
>>Still, it is not obvious that one should go into a completly different
>>config section and manually enable sd support, and I have been bitten
>>by this more than one time.
>>
>>Maybe we should add, just below the 'USB storage' Kconfig option another
>>one, let's say 'SCSI disk based USB storage support', which documentation
>>would talk about 'usb keys, memory stick readers, USB floppy drives etc',
>>which should just be a dummy option selecting  BLK_DEV_SD ?
>>
>>Or perhaps we should add something along the Debian's dpkg 'suggests' rule
>>to Kconfig ? :)
> 
> 
> I get enough e-mail on this topic that we should do something about it.
> We need some sort of 'suggests' rule, or at least some sort of message to
> the user to tell them to enable the high-level drivers.

Until 'suggests' is available, does this help any?
It's tough getting people to read Help messages though.



Add comment/NOTE that USB_STORAGE probably needs BLK_DEV_SD also.
Add a few device types to help text and reformat it.

diffstat:=
  drivers/usb/storage/Kconfig |   15 +++++++++++----
  1 files changed, 11 insertions(+), 4 deletions(-)

Signed-off-by: Randy Dunlap <rddunlap@osdl.org>

[-- Attachment #2: usb_sd_comment.patch --]
[-- Type: text/x-patch, Size: 1371 bytes --]

diff -Naurp ./drivers/usb/storage/Kconfig~usb_comment ./drivers/usb/storage/Kconfig
--- ./drivers/usb/storage/Kconfig~usb_comment	2004-10-18 14:55:28.000000000 -0700
+++ ./drivers/usb/storage/Kconfig	2004-11-20 15:15:10.640398000 -0800
@@ -2,16 +2,23 @@
 # USB Storage driver configuration
 #
 
+comment "NOTE: USB_STORAGE enables SCSI, and 'SCSI disk support' may also be needed; see USB_STORAGE Help for more information"
+
 config USB_STORAGE
 	tristate "USB Mass Storage support"
 	depends on USB
 	select 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,
-	  USB hard disks, USB tape drives and USB CD-ROMs, along with
-	  similar devices. This driver may also be used for some cameras and
-	  card readers.
+	  computer's USB port. This is the driver you need for USB
+	  floppy drives, USB hard disks, USB tape drives, USB CD-ROMs,
+	  USB flash devices, and memory sticks, along with
+	  similar devices. This driver may also be used for some cameras
+	  and card readers.
+
+	  This option 'selects' (turns on, enables) 'SCSI', but you
+	  probably also need 'SCSI device support: SCSI disk support'
+	  (BLK_DEV_SD) for most USB storage devices.
 
 	  To compile this driver as a module, choose M here: the
 	  module will be called usb-storage.