From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Williams Date: Thu, 07 Jan 2010 02:20:14 +0000 Subject: Re: Question about your reversion of the udev Huawei modem rule Message-Id: <1262830814.3608.24.camel@localhost.localdomain> List-Id: References: <20100105213602.GC2822@piware.de> In-Reply-To: <20100105213602.GC2822@piware.de> MIME-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: quoted-printable To: linux-hotplug@vger.kernel.org On Tue, 2010-01-05 at 22:36 +0100, Martin Pitt wrote: > Hey Dan, >=20 > in [1] you reverted a previous commit [2] to fix the Huawei E1550 GSM > modem in udev (using modem-modeswitch) with the rationale that "other > manufacturers have completely different methods for killing their fake > driver CD". However, the rule was tested on that actual hardware, and > it worked fine. Lets use the *correct* method for ejecting a device instead of using one manufacturer's method with another manufacturer's device. 'rezero' is *NOT* the correct method for ejecting Huawei devices. It doesn't really matter whether it works or not. usb_modeswitch has working methods for ejecting the E1550 (at least they certainly work for me). Rezero is Option NV specific and can/will cause undesirable results on other manufacturer's equipment. ZTE devices specifically can be affected by this and do not react well, confusing both the device and the kernel USB stack in the process. I should think it would be clear that we shouldn't be using 'rezero' with devices for which its not intended, but instead to use the method that all other Huawei devices use (usb_stor_huawei_e220_init from drivers/usb/storage/initializers.c). The E1550 has a new USB ID that's not found in unusual_devs.h; just add the ID there, or start using usb_modeswitch. Dan > I'm interested in why this rule was bad, and what should be done > instead? >=20 > Thanks, >=20 > Martin >=20 > [1] http://git.kernel.org/?p=3Dlinux/hotplug/udev.git;a=3Dcommitdiff;h=8A= 993fab2f218234f06743bcc127dda61a45711b > [2] http://git.kernel.org/?p=3Dlinux/hotplug/udev.git;a=3Dcommit;hS842b53= 820a0f0b2b0ff5b28caf01e3f7e26f22