From: "Randy.Dunlap" <rddunlap@osdl.org>
To: Stelian Pop <stelian@popies.net>
Cc: Matthew Dharm <mdharm-kernel@one-eyed-alien.net>,
Christoph Hellwig <hch@infradead.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Andrew Morton <akpm@osdl.org>,
greg@kroah.com
Subject: Re: [PATCH] usb-storage should enable scsi disk in Kconfig
Date: Mon, 22 Nov 2004 08:01:30 -0800 [thread overview]
Message-ID: <41A20D5A.6040004@osdl.org> (raw)
In-Reply-To: <20041122103520.GA3550@crusoe.alcove-fr>
Stelian Pop wrote:
> On Sat, Nov 20, 2004 at 03:21:54PM -0800, Randy.Dunlap wrote:
>
>
>>>>>On Fri, Nov 19, 2004 at 08:33:50PM +0100, Stelian Pop wrote:
>>>>
>>>>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 ?
>
>
>>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.
>
>
> Isn't my above suggestion even better ? A separate config option
> is much more visible IMHO...
Sounds good, as long as we don't mind the same option being
settable in multiple places. Did you submit a patch?
--
~Randy
next prev parent reply other threads:[~2004-11-22 16:42 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-11-19 19:33 [PATCH] usb-storage should enable scsi disk in Kconfig Stelian Pop
2004-11-19 19:57 ` Christoph Hellwig
2004-11-19 21:39 ` Stelian Pop
2004-11-19 23:08 ` Matthew Dharm
2004-11-20 23:21 ` Randy.Dunlap
2004-11-22 10:35 ` Stelian Pop
2004-11-22 16:01 ` Randy.Dunlap [this message]
2004-11-22 16:30 ` Stelian Pop
2004-11-22 14:52 ` Martin Waitz
2004-11-29 19:08 ` Greg KH
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=41A20D5A.6040004@osdl.org \
--to=rddunlap@osdl.org \
--cc=akpm@osdl.org \
--cc=greg@kroah.com \
--cc=hch@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mdharm-kernel@one-eyed-alien.net \
--cc=stelian@popies.net \
/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