Re: [PATCH] move eject code from zd1211rw to usb-storage

[Matthew Dharm <mdharm-kernel@one-eyed-alien.net>]

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

On Wed, Dec 16, 2009 at 08:47:17AM -0800, Dan Williams wrote:
> On Wed, 2009-12-16 at 13:14 +0100, Stefan Seyfried wrote:
> > On Wed, 16 Dec 2009 12:22:31 +0100
> > Josua Dietze <digidietze@draisberghof.de> wrote:
> > 
> > > Stefan Seyfried schrieb:
> > > > Preferably some code that can be built actually?
> > > 
> > > If the "eject" from SCSI tools does not work, "usb_modeswitch" can 
> > > send customized bulk messages to devices.
> > 
> > for my device, eject seems to be enough, so I'll be trying to get
> > this into udev.
> > 
> > > http://www.draisberghof.de/usb_modeswitch/
> > 
> > Unfortunately it cannot be built with current libusb and once you
> > compile it without warnings turned off (which is the default!), you
> > probably don't want to run it anymore ;)
> 
> At this point, however, usb_modeswitch is the correct place to put eject
> code for all devices.  I wouldn't put this code into udev; I'd put it
> into usb_modeswitch instead, since usb_modeswitch is (a) the de-facto
> standard, (b) has the most users, and (c) has the most devices.  Yes, it
> has problems, but at this point we should fix those problems instead of
> creating 5 different eject tools.

udev just calls either usb_modeswitch or eject at the hotplug event.  So,
you always need an update to the udev database, regardless.

The choice between using usb_modeswitch or eject is really one of "does the
standard eject work properly" or not.  If not, add the code to
usb_modeswitch.  Tho, if the usb_modeswitch maintainers want to carry
around a duplicate set of 'eject' code, that's really up to them.

Matt

-- 
Matthew Dharm                              Home: mdharm-usb@one-eyed-alien.net 
Maintainer, Linux USB Mass Storage Driver

Would you mind not using our Web server? We're trying to have a game of 
Quake here.
					-- Greg
User Friendly, 5/11/1998

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]