Re: [PATCH 2/2] Add UAS driver

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

 



On Tue, Sep 28, 2010 at 11:11:34AM -0700, Greg KH wrote:
> On Tue, Sep 28, 2010 at 10:51:22AM -0700, Matthew Dharm wrote:
> > On Tue, Sep 28, 2010 at 06:14:56AM -0400, Matthew Wilcox wrote:
> > > From: Matthew Wilcox <matthew.r.wilcox@xxxxxxxxx>
> > > 
> > > USB Attached SCSI is a new protocol specified jointly by the SCSI T10
> > > committee and the USB Implementors Forum.
> > > 
> > > Signed-off-by: Matthew Wilcox <willy@xxxxxxxxxxxxxxx>
> > > ---
> > >  MAINTAINERS                  |    8 +
> > >  drivers/usb/storage/Kconfig  |   13 +
> > >  drivers/usb/storage/Makefile |    1 +
> > >  drivers/usb/storage/uas.c    |  751 ++++++++++++++++++++++++++++++++++++++++++
> > >  4 files changed, 773 insertions(+), 0 deletions(-)
> > >  create mode 100644 drivers/usb/storage/uas.c
> > 
> > Given that this is distinctly not usb-storage, and it is entirely contained
> > within a single file, does it really belong in the drivers/usb/storage
> > directory?
> > 
> > That just seems like a plan for confusion.  The fact that there is a big
> > "this is not usb-storage" message in the comments of uas.c would seem to
> > support this position.
> > 
> > Given that it is a single file, I would put it in drivers/usb directly.  If
> > you wanted your own directory for possible future refactoring into multiple
> > files or addid other files (like usb-storage did to support oddball
> > devices), then maybe create a drivers/usb/uas directory.
> 
> What about drivers/usb/class/ where a number of other USB class drivers
> live.  That would make more sense, right?

No argument from me.  Heck, maybe someone wants to move usb-storage into
that directory also?

My main point is that it doesn't belong in the same directory as
usb-storage.  There are lots of good alternatives.

Matt

-- 
Matthew Dharm                              Home: mdharm-usb@xxxxxxxxxxxxxxxxxx 
Maintainer, Linux USB Mass Storage Driver

It's not that hard.  No matter what the problem is, tell the customer 
to reinstall Windows.
					-- Nurse
User Friendly, 3/22/1998

Attachment: pgptzks22Ybx_.pgp
Description: PGP signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux