Re: 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)

2013-03-05 Thread g...@kroah.com
On Tue, Mar 05, 2013 at 01:23:33PM +0100, Bjørn Mork wrote: > As you are well aware of, we parse descriptors as needed on Linux and > can just as easy support the Windows look'n'feel if required. There is > absolutely no need to create special Linux descriptors to adapt to > existing drivers.

RE: 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)

2013-03-05 Thread Fangxiaozhi (Franko)
Dear Oliver: > -Original Message- > From: Oliver Neukum [mailto:oneu...@suse.de] > Sent: Tuesday, March 05, 2013 4:49 PM > To: Fangxiaozhi (Franko) > Cc: Bjørn Mork; linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; > Xueguiying (Zihan); Linlei (Lei Lin); g...@kroah.com; Yili

Re: 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)

2013-03-05 Thread Oliver Neukum
On Tuesday 05 March 2013 02:28:07 Fangxiaozhi wrote: Hi, > 1. As far as I know, usb_modeswitch is now only integrated in the PC > Linux. It isn't integrated to other system with linux kernel, such as > Android, Chrome OS, etc. On these system, how can we switch the device? Then those

Re: 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)

2013-03-05 Thread Oliver Neukum
On Tuesday 05 March 2013 02:28:07 Fangxiaozhi wrote: Hi, 1. As far as I know, usb_modeswitch is now only integrated in the PC Linux. It isn't integrated to other system with linux kernel, such as Android, Chrome OS, etc. On these system, how can we switch the device? Then those

RE: 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)

2013-03-05 Thread Fangxiaozhi (Franko)
Dear Oliver: -Original Message- From: Oliver Neukum [mailto:oneu...@suse.de] Sent: Tuesday, March 05, 2013 4:49 PM To: Fangxiaozhi (Franko) Cc: Bjørn Mork; linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; Xueguiying (Zihan); Linlei (Lei Lin); g...@kroah.com; Yili (Neil);

Re: 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)

2013-03-05 Thread g...@kroah.com
On Tue, Mar 05, 2013 at 01:23:33PM +0100, Bjørn Mork wrote: As you are well aware of, we parse descriptors as needed on Linux and can just as easy support the Windows look'n'feel if required. There is absolutely no need to create special Linux descriptors to adapt to existing drivers. We'll

RE: 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)

2013-03-04 Thread Fangxiaozhi (Franko)
Dear Mork: Thank you very much for your test. > -Original Message- > From: Bjørn Mork [mailto:bj...@mork.no] > Sent: Monday, March 04, 2013 7:41 PM > To: Fangxiaozhi (Franko) > Cc: linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; Xueguiying > (Zihan); Linlei (Lei Lin);

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)

2013-03-04 Thread Bjørn Mork
Hello Franko, This patch causes a number of regressions for both the Huawei devices I have available for testing. One of them is completely unusable in v3.8 (unable to switch to modem mode) unless the usb-storage driver is disabled. I realize that some devices are historically handled by the

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)

2013-03-04 Thread Bjørn Mork
Hello Franko, This patch causes a number of regressions for both the Huawei devices I have available for testing. One of them is completely unusable in v3.8 (unable to switch to modem mode) unless the usb-storage driver is disabled. I realize that some devices are historically handled by the

RE: 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)

2013-03-04 Thread Fangxiaozhi (Franko)
Dear Mork: Thank you very much for your test. -Original Message- From: Bjørn Mork [mailto:bj...@mork.no] Sent: Monday, March 04, 2013 7:41 PM To: Fangxiaozhi (Franko) Cc: linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; Xueguiying (Zihan); Linlei (Lei Lin);

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-27 Thread Sebastian Andrzej Siewior
On Fri, Jan 25, 2013 at 10:46:13AM +0800, fangxiaozhi 00110321 wrote: > diff -uprN linux-3.8-rc4_orig/drivers/usb/storage/initializers.c > linux-3.8-rc4/drivers/usb/storage/initializers.c > --- linux-3.8-rc4_orig/drivers/usb/storage/initializers.c 2013-01-22 > 14:12:42.594238726 +0800 > +++

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-27 Thread Sebastian Andrzej Siewior
On Fri, Jan 25, 2013 at 10:46:13AM +0800, fangxiaozhi 00110321 wrote: diff -uprN linux-3.8-rc4_orig/drivers/usb/storage/initializers.c linux-3.8-rc4/drivers/usb/storage/initializers.c --- linux-3.8-rc4_orig/drivers/usb/storage/initializers.c 2013-01-22 14:12:42.594238726 +0800 +++

RE: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-13 Thread Fangxiaozhi (Franko)
Dear Greg: > -Original Message- > From: Greg KH [mailto:g...@kroah.com] > Sent: Saturday, January 12, 2013 8:22 AM > To: Fangxiaozhi (Franko) > Cc: linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; Xueguiying > (Zihan); > Linlei (Lei Lin); Yili (Neil); Wangyuhua (Roger, Credit);

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-13 Thread Sebastian Andrzej Siewior
On Fri, Jan 11, 2013 at 05:57:44PM +0800, fangxiaozhi 00110321 wrote: > diff -uprN linux-3.8-rc3_orig/drivers/usb/storage/initializers.c > linux-3.8-rc3/drivers/usb/storage/initializers.c > --- linux-3.8-rc3_orig/drivers/usb/storage/initializers.c 2013-01-11 > 17:53:19.757842845 +0800 > +++

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-13 Thread Sebastian Andrzej Siewior
On Fri, Jan 11, 2013 at 05:57:44PM +0800, fangxiaozhi 00110321 wrote: diff -uprN linux-3.8-rc3_orig/drivers/usb/storage/initializers.c linux-3.8-rc3/drivers/usb/storage/initializers.c --- linux-3.8-rc3_orig/drivers/usb/storage/initializers.c 2013-01-11 17:53:19.757842845 +0800 +++

RE: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-13 Thread Fangxiaozhi (Franko)
Dear Greg: -Original Message- From: Greg KH [mailto:g...@kroah.com] Sent: Saturday, January 12, 2013 8:22 AM To: Fangxiaozhi (Franko) Cc: linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; Xueguiying (Zihan); Linlei (Lei Lin); Yili (Neil); Wangyuhua (Roger, Credit); Huqiao

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-11 Thread Greg KH
On Fri, Jan 11, 2013 at 05:57:44PM +0800, fangxiaozhi 00110321 wrote: > From: fangxiaozhi > > 1. Optimize the match rules with new macro for Huawei USB storage devices, >to avoid to load USB storage driver for the modem interface >with Huawei devices. > 2. Add to support new switch

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-11 Thread Greg KH
On Fri, Jan 11, 2013 at 05:57:44PM +0800, fangxiaozhi 00110321 wrote: From: fangxiaozhi huana...@huawei.com 1. Optimize the match rules with new macro for Huawei USB storage devices, to avoid to load USB storage driver for the modem interface with Huawei devices. 2. Add to support

RE: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-03 Thread Fangxiaozhi (Franko)
Dear Matthew: > -Original Message- > From: Matthew Dharm [mailto:mdharm-...@one-eyed-alien.net] > Sent: Wednesday, December 19, 2012 11:41 PM > To: Sebastian Andrzej Siewior > Cc: Fangxiaozhi (Franko); linux-...@vger.kernel.org; > linux-kernel@vger.kernel.org; Xueguiying (Zihan); Linlei

RE: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-03 Thread Fangxiaozhi (Franko)
Dear Matthew: -Original Message- From: Matthew Dharm [mailto:mdharm-...@one-eyed-alien.net] Sent: Wednesday, December 19, 2012 11:41 PM To: Sebastian Andrzej Siewior Cc: Fangxiaozhi (Franko); linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; Xueguiying (Zihan); Linlei (Lei

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2012-12-19 Thread Matthew Dharm
On Wed, Dec 19, 2012 at 12:34 AM, Sebastian Andrzej Siewior wrote: > On Wed, Dec 19, 2012 at 03:13:32AM +, Fangxiaozhi (Franko) wrote: >> > And shouldn't you read something from the us->recv_bulk_pipe after >> > that? >> Well, because our device will re-connect to switch the ports if it

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2012-12-19 Thread Sebastian Andrzej Siewior
On Wed, Dec 19, 2012 at 03:13:32AM +, Fangxiaozhi (Franko) wrote: > By the way, I found the kernel is updated to 3.7.1 today. So I have to > update my patch based on 3.7.1, and resubmit it? > Right? You should rebase your patch on top of Greg's usb-next branch of his usb tree.

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2012-12-19 Thread Sebastian Andrzej Siewior
On Wed, Dec 19, 2012 at 03:13:32AM +, Fangxiaozhi (Franko) wrote: By the way, I found the kernel is updated to 3.7.1 today. So I have to update my patch based on 3.7.1, and resubmit it? Right? You should rebase your patch on top of Greg's usb-next branch of his usb tree.

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2012-12-19 Thread Matthew Dharm
On Wed, Dec 19, 2012 at 12:34 AM, Sebastian Andrzej Siewior sebast...@breakpoint.cc wrote: On Wed, Dec 19, 2012 at 03:13:32AM +, Fangxiaozhi (Franko) wrote: And shouldn't you read something from the us-recv_bulk_pipe after that? Well, because our device will re-connect to switch

RE: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2012-12-18 Thread Fangxiaozhi (Franko)
Dear Sebastian: Please see the comments follows yours. By the way, I found the kernel is updated to 3.7.1 today. So I have to update my patch based on 3.7.1, and resubmit it? Right? Best Regards, Franko Fang > -Original Message- > From: Sebastian Andrzej

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2012-12-18 Thread Sebastian Andrzej Siewior
On Tue, Dec 18, 2012 at 10:44:19AM +0800, fangxiaozhi 00110321 wrote: > diff -uprN linux-3.7_bak/drivers/usb/storage/initializers.c > linux-3.7/drivers/usb/storage/initializers.c > --- linux-3.7_bak/drivers/usb/storage/initializers.c 2012-12-11 > 09:56:11.0 +0800 > +++

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2012-12-18 Thread Sebastian Andrzej Siewior
On Tue, Dec 18, 2012 at 10:44:19AM +0800, fangxiaozhi 00110321 wrote: diff -uprN linux-3.7_bak/drivers/usb/storage/initializers.c linux-3.7/drivers/usb/storage/initializers.c --- linux-3.7_bak/drivers/usb/storage/initializers.c 2012-12-11 09:56:11.0 +0800 +++

RE: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2012-12-18 Thread Fangxiaozhi (Franko)
Dear Sebastian: Please see the comments follows yours. By the way, I found the kernel is updated to 3.7.1 today. So I have to update my patch based on 3.7.1, and resubmit it? Right? Best Regards, Franko Fang -Original Message- From: Sebastian Andrzej