All of lore.kernel.org
 help / color / mirror / Atom feed
From: Josua Dietze <digidietze@draisberghof.de>
To: "Bjørn Mork" <bjorn@mork.no>
Cc: Matthew Dharm <mdharm-usb@one-eyed-alien.net>,
	Ben Hutchings <ben@decadent.org.uk>,
	"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Fangxiaozhi (Franko)" <fangxiaozhi@huawei.com>,
	zihan@huawei.com, Lin.Lei@huawei.com,
	Greg KH <gregkh@linuxfoundation.org>,
	"Yili (Neil)" <neil.yi@huawei.com>, "Wangyuhua \(Roger\,
	Credit\)" <wangyuhua@huawei.com>, Huqiao <huqiao36@huawei.com>,
	Felipe Balbi <balbi@ti.com>,
	Sebastian Andrzej Siewior <sebastian@breakpoint.cc>,
	stable <stable@vger.kernel.org>
Subject: Re: [PATCH] USB: storage: fix Huawei mode switching regression
Date: Mon, 04 Mar 2013 23:28:47 +0100	[thread overview]
Message-ID: <5135201F.4030200@draisberghof.de> (raw)
In-Reply-To: <87wqtnq8bb.fsf@nemi.mork.no>

Am 04.03.2013 20:22, schrieb Bjørn Mork:
> Matthew Dharm <mdharm-usb@one-eyed-alien.net> writes:
>> The question is not one of reminding me of what I said earlier....
>> it's one of pointing people in the right direction.  Frankly, some of
>> the fault for this patch lies with Greg and myself for letting it
>> through.  I had just assumed that the Huawei guys had already been in
>> touch with usb-modeswitch for some reason, and that this was just an
>> optimization of existing logic (not an expansion).

I was contacted at one point by a Huawei engineer who convinced me to change the default mode-switching 'message' for all Huawei devices.
The reason was the introduction of an 'advanced' Linux driver by Huawei which requires a specific target mode. This was in October 2010.
No contact attempt since then.

>> Who is maintaining usb-modeswitch these days, anyway?  The comment in
>> the file should point people directly there....

I never ceased work on it and intend to do so for years to come. I would certainly welcome any pointer to the usb_modeswitch main site in the code or the documentation. Although modem developers or engineers should not have a problem finding the site and providing new device information.

>> And, as of now, I would really like to see as many of these devices
>> migrated (albeit slowly) to using usb-modeswitch wherever possible.  I
>> know there are a few devices for which that might not be possible, but
>> I am DONE dealing with this same issue over and over and over again.
>> It will certainly be work to migrate support; maybe we should wrap all
>> the relevant unusual_devs.h entires with
>> CONFIG_UPDATED_MODESWITCH_INSTALLED_SO_MAKE_THESE_GO_AWAY during a
>> transition period?

I think it's safe to say that usb_modeswitch is included in all distributions now. Usually, no user interaction is necessary.

> I guess the real problem will be verifying that all of the entries *can*
> go away. This type of hardware tends to get old very fast, but there is
> always someone having a really ancient device.

I will check this and add any missing USB IDs to usb_modeswitch, but I can't shake the feeling that not *all* Huawei entries in "unusual_devs.h" did actually materialize as devices ...

Anyway, as Bjørn said, putting that initialization into the storage driver takes away quite some possibilities to handle these modems in a flexible way.


Josua Dietze


WARNING: multiple messages have this Message-ID (diff)
From: Josua Dietze <digidietze@draisberghof.de>
To: "Bjørn Mork" <bjorn@mork.no>
Cc: Matthew Dharm <mdharm-usb@one-eyed-alien.net>,
	Ben Hutchings <ben@decadent.org.uk>,
	"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Fangxiaozhi (Franko)" <fangxiaozhi@huawei.com>,
	zihan@huawei.com, Lin.Lei@huawei.com,
	Greg KH <gregkh@linuxfoundation.org>,
	"Yili (Neil)" <neil.yi@huawei.com>, "Wangyuhua \(Roger\,
	Credit\)" <wangyuhua@huawei.com>, Huqiao <huqiao36@huawei.com>,
	Felipe Balbi <balbi@ti.com>,
	Sebastian Andrzej Siewior <sebastian@breakpoint.cc>,
	stable <stable@vger.kernel.org>
Subject: Re: [PATCH] USB: storage: fix Huawei mode switching regression
Date: Mon, 04 Mar 2013 23:28:47 +0100	[thread overview]
Message-ID: <5135201F.4030200@draisberghof.de> (raw)
In-Reply-To: <87wqtnq8bb.fsf@nemi.mork.no>

Am 04.03.2013 20:22, schrieb Bjᅵrn Mork:
> Matthew Dharm <mdharm-usb@one-eyed-alien.net> writes:
>> The question is not one of reminding me of what I said earlier....
>> it's one of pointing people in the right direction.  Frankly, some of
>> the fault for this patch lies with Greg and myself for letting it
>> through.  I had just assumed that the Huawei guys had already been in
>> touch with usb-modeswitch for some reason, and that this was just an
>> optimization of existing logic (not an expansion).

I was contacted at one point by a Huawei engineer who convinced me to change the default mode-switching 'message' for all Huawei devices.
The reason was the introduction of an 'advanced' Linux driver by Huawei which requires a specific target mode. This was in October 2010.
No contact attempt since then.

>> Who is maintaining usb-modeswitch these days, anyway?  The comment in
>> the file should point people directly there....

I never ceased work on it and intend to do so for years to come. I would certainly welcome any pointer to the usb_modeswitch main site in the code or the documentation. Although modem developers or engineers should not have a problem finding the site and providing new device information.

>> And, as of now, I would really like to see as many of these devices
>> migrated (albeit slowly) to using usb-modeswitch wherever possible.  I
>> know there are a few devices for which that might not be possible, but
>> I am DONE dealing with this same issue over and over and over again.
>> It will certainly be work to migrate support; maybe we should wrap all
>> the relevant unusual_devs.h entires with
>> CONFIG_UPDATED_MODESWITCH_INSTALLED_SO_MAKE_THESE_GO_AWAY during a
>> transition period?

I think it's safe to say that usb_modeswitch is included in all distributions now. Usually, no user interaction is necessary.

> I guess the real problem will be verifying that all of the entries *can*
> go away. This type of hardware tends to get old very fast, but there is
> always someone having a really ancient device.

I will check this and add any missing USB IDs to usb_modeswitch, but I can't shake the feeling that not *all* Huawei entries in "unusual_devs.h" did actually materialize as devices ...

Anyway, as Bjᅵrn said, putting that initialization into the storage driver takes away quite some possibilities to handle these modems in a flexible way.


Josua Dietze


  reply	other threads:[~2013-03-04 23:02 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-04  7:16 [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command fangxiaozhi 00110321
2013-03-04 11:41 ` v3.8 regression: Huawei mode switching fails (was Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command) Bjørn Mork
2013-03-04 13:19   ` [PATCH] USB: storage: fix Huawei mode switching regression Bjørn Mork
2013-03-04 14:29     ` Ben Hutchings
2013-03-04 16:47       ` Bjørn Mork
2013-03-04 16:59         ` Matthew Dharm
2013-03-04 16:59           ` Matthew Dharm
2013-03-04 19:22           ` Bjørn Mork
2013-03-04 22:28             ` Josua Dietze [this message]
2013-03-04 22:28               ` Josua Dietze
2013-03-05  8:32               ` Oliver Neukum
2013-03-05  8:32                 ` Oliver Neukum
2013-03-05 11:35                 ` Bjørn Mork
2013-03-05  2:15     ` Fangxiaozhi (Franko)
2013-03-05  2:15       ` Fangxiaozhi (Franko)
2013-03-05 10:07       ` Bjørn Mork
2013-03-05 11:52         ` Oliver Neukum
2013-03-05 11:52           ` Oliver Neukum
2013-03-05 14:08           ` Bjørn Mork
2013-03-06  1:34         ` 答复: " Linlei (Lei Lin)
2013-03-06  1:34           ` Linlei (Lei Lin)
2013-03-06  1:44           ` Greg KH
2013-03-06 15:45             ` Dan Williams
2013-03-07  2:54               ` Fangxiaozhi (Franko)
2013-03-07  2:54                 ` Fangxiaozhi (Franko)
2013-03-07  3:18                 ` Greg KH
2013-03-07 12:19                 ` Bjørn Mork
2013-03-07 14:11                   ` Josua Dietze
2013-03-05  2:28   ` v3.8 regression: Huawei mode switching fails (was Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command) Fangxiaozhi (Franko)
2013-03-05  8:48     ` Oliver Neukum
2013-03-05  9:05       ` Fangxiaozhi (Franko)
2013-03-05 12:23         ` Bjørn Mork
2013-03-06  1:40           ` greg

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=5135201F.4030200@draisberghof.de \
    --to=digidietze@draisberghof.de \
    --cc=Lin.Lei@huawei.com \
    --cc=balbi@ti.com \
    --cc=ben@decadent.org.uk \
    --cc=bjorn@mork.no \
    --cc=fangxiaozhi@huawei.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=huqiao36@huawei.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=mdharm-usb@one-eyed-alien.net \
    --cc=neil.yi@huawei.com \
    --cc=sebastian@breakpoint.cc \
    --cc=stable@vger.kernel.org \
    --cc=wangyuhua@huawei.com \
    --cc=zihan@huawei.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.