Support for new USB Modem 4G drive

2014-03-19 Thread Julio Araujo
Hello, My name is Julio and I'm trying to use a USB Modem 4G model OLICARD500 from the manufacture Olivetti and I got the follow message from syslog: Mar 19 05:23:29 mynote kernel: [ 265.072844] usbserial_generic 1-1.3:1.0: Tell linux-usb@vger.kernel.org to add your device to a proper

Re: questions about give back urb in tasklet

2014-03-19 Thread vichy
hi Ming.lei: 2014-03-17 22:01 GMT+08:00 Alan Stern st...@rowland.harvard.edu: On Sun, 16 Mar 2014, vichy wrote: hi all: recently we bump to system performance issue when usb irq take quite long. I found below link for discussing how to short in

[PATCH v4 00/11] Add USB OTG HNP and SRP support on Chipidea usb driver

2014-03-19 Thread Li Jun
From: Li Jun b47...@freescale.com This patchset adds USB OTG HNP and SRP support on chipidea usb driver, existing OTG port role swtich function by ID pin status kept unchanged, based on that, if select CONFIG_USB_OTG_FSM, OTG HNP and SRP will be supported. Reference to: On-The-Go and Embedded

[PATCH v4 02/11] usb: chipidea: host: vbus control change for OTG HNP.

2014-03-19 Thread Li Jun
Leave vbus on/off hanlded by OTG fsm if in OTG mode. Signed-off-by: Li Jun b47...@freescale.com --- drivers/usb/chipidea/host.c |6 +++--- drivers/usb/chipidea/host.h |9 + 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/drivers/usb/chipidea/host.c

[PATCH v4 05/11] usb: chipidea: add OTG fsm operation functions implemenation.

2014-03-19 Thread Li Jun
Add OTG HNP and SRP operation functions implementation: - charge vbus - drive vbus - connection signaling - drive sof - start data pulse - add fsm timer - delete fsm timer - start host - start gadget Signed-off-by: Li Jun b47...@freescale.com --- drivers/usb/chipidea/bits.h| 10 +-

[PATCH v4 10/11] Documentation: ABI: usb: sysfs Description for chipidea USB OTG HNP and SRP

2014-03-19 Thread Li Jun
From: Li Jun b47...@freescale.com This patch adds sysfs interface description for chipidea USB OTG HNP and SRP. Signed-off-by: Li Jun b47...@freescale.com --- .../ABI/testing/sysfs-platform-chipidea-usb-otg| 56 1 file changed, 56 insertions(+) diff --git

[PATCH v4 08/11] usb: chipidea: add sys inputs for OTG fsm input.

2014-03-19 Thread Li Jun
From: Li Jun b47...@freescale.com This patch adds sys input to control and show OTG fsm inputs by application, user can do host and preipheral role switch by change these inputs. Signed-off-by: Li Jun b47...@freescale.com --- drivers/usb/chipidea/otg.c |1 +

[PATCH v4 03/11] usb: chipidea: host: init otg port number.

2014-03-19 Thread Li Jun
Init otg_port number of otg capable host to be 1 at host start. Signed-off-by: Li Jun b47...@freescale.com --- drivers/usb/chipidea/host.c |7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/usb/chipidea/host.c b/drivers/usb/chipidea/host.c index 4b609bc..5ec8ccd

[PATCH v4 06/11] usb: chipidea: OTG fsm timers initialization.

2014-03-19 Thread Li Jun
From: Li Jun b47...@freescale.com This patch adds OTG fsm timers initialization, which use controller's 1ms interrupt as timeout counter, also adds some local timers which are not in otg_fsm_timer list. Signed-off-by: Li Jun b47...@freescale.com --- drivers/usb/chipidea/bits.h|1 +

[PATCH v4 01/11] usb: chipidea: usb OTG fsm initialization.

2014-03-19 Thread Li Jun
This patch adds OTG fsm related initialization when do otg init, add a seperate file for OTG fsm related utilities. Signed-off-by: Li Jun b47...@freescale.com --- drivers/usb/chipidea/Makefile |1 + drivers/usb/chipidea/ci.h |2 ++ drivers/usb/chipidea/otg.c |7 +

[PATCH v4 11/11] Documentation: usb: add chipidea.txt for how to demo usb OTG HNP and SRP

2014-03-19 Thread Li Jun
From: Li Jun b47...@freescale.com This patch adds a file chipidea.txt for how to demo chipidea usb OTG HNP and SRP functions via sysfs input files, any other possible information should be documented for chipidea usb driver in future can be added into this file. Signed-off-by: Li Jun

[PATCH v4 07/11] usb: chipidea: OTG HNP and SRP fsm implementation.

2014-03-19 Thread Li Jun
From: Li Jun b47...@freescale.com USB OTG interrupt handling and fsm transitions according to USB OTG and EH 2.0. Signed-off-by: Li Jun b47...@freescale.com --- drivers/usb/chipidea/core.c| 10 +- drivers/usb/chipidea/otg.c |9 +- drivers/usb/chipidea/otg_fsm.c | 235

[PATCH v4 04/11] usb: chipidea: udc: driver update for OTG HNP.

2014-03-19 Thread Li Jun
Add b_hnp_enable request handling and enable gadget-is_otg Signed-off-by: Li Jun b47...@freescale.com --- drivers/usb/chipidea/udc.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/drivers/usb/chipidea/udc.c b/drivers/usb/chipidea/udc.c index f58857d..022dd18

[PATCH v4 09/11] usb: chipidea: debug: add debug file for OTG variables

2014-03-19 Thread Li Jun
From: Li Jun b47...@freescale.com This patch adds a debug file for OTG vairables show. Signed-off-by: Li Jun b47...@freescale.com --- drivers/usb/chipidea/debug.c | 84 ++ 1 file changed, 84 insertions(+) diff --git a/drivers/usb/chipidea/debug.c

[PATCH 0/6] Fix line over 80 characters warning in bcm.

2014-03-19 Thread Heena Sirwani
This patchset consists of patches that fix line over 80 characters warning in bcm module in Adapter.h.Please review the patches that follow. Heena Sirwani (6): staging: bcm: Fix line over 80 characters. staging: bcm: Fix line over 80 characters. staging: bcm: Fix line over 80 characters.

Re: [xhci_hcd] reset SuperSpeed, xhci_drop_endpoint called with disabled ep, Error in queuecommand_lck: task blocked

2014-03-19 Thread Andreas Reis
I've uploaded a dmesg with the new debugging patch to bugzilla: https://bugzilla.kernel.org/attachment.cgi?id=130041 Andreas Reis On 18.03.2014 15:57, Alan Stern wrote: The debugging information didn't go far enough. Try the patch below instead, which has some additional messages. There

[PATCH 1/6] staging: bcm: Fix line over 80 characters.

2014-03-19 Thread Heena Sirwani
Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani heenasirw...@gmail.com --- drivers/staging/bcm/Adapter.h |6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h

[PATCH 2/6] staging: bcm: Fix line over 80 characters.

2014-03-19 Thread Heena Sirwani
Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani heenasirw...@gmail.com --- drivers/staging/bcm/Adapter.h | 23 ++- 1 file changed, 14 insertions(+), 9 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h

[PATCH 3/6] staging: bcm: Fix line over 80 characters.

2014-03-19 Thread Heena Sirwani
Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani heenasirw...@gmail.com --- drivers/staging/bcm/Adapter.h |9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h

[PATCH 4/6] staging: bcm: Fix line over 80 characters.

2014-03-19 Thread Heena Sirwani
Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani heenasirw...@gmail.com --- drivers/staging/bcm/Adapter.h |6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h

[PATCH 5/6] staging: bcm: Fix line over 80 characters.

2014-03-19 Thread Heena Sirwani
Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani heenasirw...@gmail.com --- drivers/staging/bcm/Adapter.h |7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h

[PATCH 6/6] staging: bcm: Fix line over 80 characters.

2014-03-19 Thread Heena Sirwani
Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani heenasirw...@gmail.com --- drivers/staging/bcm/Adapter.h | 20 ++-- 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h

Re: [PATCH 1/2] extcon: of: Remove unnecessary function call by using the name of device_node

2014-03-19 Thread Kishon Vijay Abraham I
Hi, On Tuesday 18 March 2014 05:34 PM, Chanwoo Choi wrote: This patch remove unnecessary function call in of_extcon_get_extcon_dev() by using the name of device_node structure. Signed-off-by: Chanwoo Choi cw00.c...@samsung.com --- drivers/extcon/of_extcon.c | 12 ++-- 1 file

Re: [PATCH] usb:hub set hub-change_bits when over-current happens

2014-03-19 Thread Yuvaraj Cd
With this patch,I can see usb 3.0 device detection is failing on exynos5250-smdk5250 boards. CONFIG_USB_SUSPEND=n / $ [ 11.486922] hub 2-0:1.0: connect-debounce failed, port 1 disabled [ 13.891919] hub 2-0:1.0: connect-debounce failed, port 1 disabled [ 16.296919] hub 2-0:1.0:

Re: questions about give back urb in tasklet

2014-03-19 Thread vichy
hi all: 2014-03-19 17:57 GMT+08:00 vichy vichy@gmail.com: hi Ming.lei: 2014-03-17 22:01 GMT+08:00 Alan Stern st...@rowland.harvard.edu: On Sun, 16 Mar 2014, vichy wrote: hi all: recently we bump to system performance issue when usb irq take quite long. I found below link for

Re: Support for new USB Modem 4G drive

2014-03-19 Thread Dan Williams
On Wed, 2014-03-19 at 06:25 -0300, Julio Araujo wrote: Hello, My name is Julio and I'm trying to use a USB Modem 4G model OLICARD500 from the manufacture Olivetti and I got the follow message from syslog: Mar 19 05:23:29 mynote kernel: [ 265.072844] usbserial_generic 1-1.3:1.0: Tell

Re: questions about give back urb in tasklet

2014-03-19 Thread Ming Lei
On Wed, Mar 19, 2014 at 5:57 PM, vichy vichy@gmail.com wrote: hi Ming.lei: 2014-03-17 22:01 GMT+08:00 Alan Stern st...@rowland.harvard.edu: On Sun, 16 Mar 2014, vichy wrote: hi all: recently we bump to system performance issue when usb irq take quite long. I found below link for

Re: questions about give back urb in tasklet

2014-03-19 Thread vichy
hi Ming in http://permalink.gmane.org/gmane.linux.usb.general/89363 I have some questions about this patch. 1. is there patch or kernel config I can use to measure man/avage usb irq time consuming like the above link show One approach I like to use is trace event. Would you please let us

Re: questions about give back urb in tasklet

2014-03-19 Thread Ming Lei
On Wed, Mar 19, 2014 at 9:34 PM, vichy vichy@gmail.com wrote: hi Ming in http://permalink.gmane.org/gmane.linux.usb.general/89363 I have some questions about this patch. 1. is there patch or kernel config I can use to measure man/avage usb irq time consuming like the above link show

Re: questions about give back urb in tasklet

2014-03-19 Thread Alan Stern
On Wed, 19 Mar 2014, Ming Lei wrote: But actually the local interrupt is still disabled during completion because we need to convert spin_lock in all current usb drivers to spin_lock_irqsave() first. I posted lot of patches to do the conversion, but unfortunately most of them aren't

RE: [PATCH] usb: gadget: fsl: Add FSL USB Gadget entry in platform device id

2014-03-19 Thread suresh.gu...@freescale.com
Hi -Original Message- From: Felipe Balbi [mailto:ba...@ti.com] Sent: Saturday, March 15, 2014 7:10 AM To: Gupta Suresh-B42813 Cc: ba...@ti.com; gre...@linuxfoundation.org; linux-usb@vger.kernel.org; linux-ker...@vger.kernel.org Subject: Re: [PATCH] usb: gadget: fsl: Add FSL USB

RE: [PATCH] USB: Gadget: fsl driver pullup fix

2014-03-19 Thread suresh.gu...@freescale.com
-Original Message- From: Felipe Balbi [mailto:ba...@ti.com] Sent: Saturday, March 15, 2014 7:05 AM To: Gupta Suresh-B42813 Cc: ba...@ti.com; gre...@linuxfoundation.org; linux-usb@vger.kernel.org; linux-ker...@vger.kernel.org; Stefani Seibold Subject: Re: [PATCH] USB: Gadget: fsl

Re: questions about give back urb in tasklet

2014-03-19 Thread vichy
hi Ming: One approach I like to use is trace event. Would you please let us how to to use trace event for calculate max/average usb irq time? You can enable irq_handler_entry and irq_handler_exit event, then write a script easily to figure out the time consumed in ehci irq handler, see

Re: problem with resume after s2ram

2014-03-19 Thread Alan Stern
On Wed, 19 Mar 2014, Peter Münster wrote: On Tue, Mar 18 2014, Alan Stern wrote: Commit 0aa2832dd0d9d8609fd8f15139bc7572541a1215 introduces a problem for my system: You should include the name of the commit along with the hash ID; otherwise nobody will know what it is unless they

Re: [PATCH net,stable] net: cdc_ncm: fix control message ordering

2014-03-19 Thread Thomas Schäfer
I will test also a further device(k5150), which was not working since 3.13, later this week. The test was ok. The K5150 from HUAWEI/Vodafone (mbim) works again. At least with IPv4. No statement about IPv6. Regards, Thomas -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH v2 01/16] staging: wlags49_h2: Use tabs for indentation

2014-03-19 Thread Greg KH
On Tue, Mar 18, 2014 at 10:13:45PM +0100, Jelena Bjelja wrote: Fix the following checkpatch.pl issues in wl_netdev.c: WARNING: please, no spaces at the start of a line ERROR: code indent should use tabs where possible Signed-off-by: Jelena Bjelja jelena.bjelja@gmail.com ---

Re: [PATCH] USB: Gadget: fsl driver pullup fix

2014-03-19 Thread Felipe Balbi
On Wed, Mar 19, 2014 at 02:23:59PM +, suresh.gu...@freescale.com wrote: -Original Message- From: Felipe Balbi [mailto:ba...@ti.com] Sent: Saturday, March 15, 2014 7:05 AM To: Gupta Suresh-B42813 Cc: ba...@ti.com; gre...@linuxfoundation.org; linux-usb@vger.kernel.org;

Re: [OPW kernel] [PATCH 2/6] staging: bcm: Fix line over 80 characters.

2014-03-19 Thread Greg KH
On Wed, Mar 19, 2014 at 05:08:58PM +0530, Heena Sirwani wrote: Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani heenasirw...@gmail.com --- drivers/staging/bcm/Adapter.h | 23 ++- 1 file changed, 14 insertions(+), 9

Re: [PATCH 4/6] staging: bcm: Fix line over 80 characters.

2014-03-19 Thread Greg KH
On Wed, Mar 19, 2014 at 05:12:13PM +0530, Heena Sirwani wrote: Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani heenasirw...@gmail.com You have 5 patches here with the same Subject:, which I can't take. Please uniquely name the subject and

Re: [PATCH 3/6] staging: bcm: Fix line over 80 characters.

2014-03-19 Thread Greg KH
On Wed, Mar 19, 2014 at 05:10:34PM +0530, Heena Sirwani wrote: Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani heenasirw...@gmail.com --- drivers/staging/bcm/Adapter.h |9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff

Re: [PATCH] usb: gadget: fsl: Add FSL USB Gadget entry in platform device id

2014-03-19 Thread gre...@linuxfoundation.org
On Wed, Mar 19, 2014 at 02:25:25PM +, suresh.gu...@freescale.com wrote: Hi -Original Message- From: Felipe Balbi [mailto:ba...@ti.com] Sent: Saturday, March 15, 2014 7:10 AM To: Gupta Suresh-B42813 Cc: ba...@ti.com; gre...@linuxfoundation.org; linux-usb@vger.kernel.org;

Re: Support for new USB Modem 4G drive

2014-03-19 Thread Dan Williams
On Wed, 2014-03-19 at 13:22 -0300, Julio Araujo wrote: Hi Dan, Thanks for your answer. I would like to inform you that I'm using an Ubuntu 12.04 TLS julio@mynote:~$ uname -a Linux mynote 3.8.0-36-generic #52~precise1-Ubuntu SMP Mon Feb 3 21:54:46 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux

Re: Support for new USB Modem 4G drive

2014-03-19 Thread Julio Araujo
Hi Dan, Thanks for your answer. I would like to inform you that I'm using an Ubuntu 12.04 TLS julio@mynote:~$ uname -a Linux mynote 3.8.0-36-generic #52~precise1-Ubuntu SMP Mon Feb 3 21:54:46 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Another thing that I've been done was a search in the google

RE: [PATCH] usb: gadget: fsl: Add FSL USB Gadget entry in platform device id

2014-03-19 Thread suresh.gu...@freescale.com
Hi Ramneek, Do you understand, what Greg want to communicate. Thanks SuresH -Original Message- From: gre...@linuxfoundation.org [mailto:gre...@linuxfoundation.org] Sent: Wednesday, March 19, 2014 10:22 PM To: Gupta Suresh-B42813 Cc: ba...@ti.com; linux-usb@vger.kernel.org;

Re: [PATCH v4 03/11] usb: chipidea: host: init otg port number.

2014-03-19 Thread Sergei Shtylyov
Hello. On 03/19/2014 01:01 PM, Li Jun wrote: Init otg_port number of otg capable host to be 1 at host start. Signed-off-by: Li Jun b47...@freescale.com --- drivers/usb/chipidea/host.c |7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git

[PATCH 1/5] staging: wlags49_h2: Enclose macro in a do-while loop

2014-03-19 Thread Jelena Bjelja
Fix the following checkpatch.pl issues in wl_netdev.c: ERROR: Macros with multiple statements should be enclosed in a do - while loop Signed-off-by: Jelena Bjelja jelena.bjelja@gmail.com --- drivers/staging/wlags49_h2/wl_netdev.c |8 +--- 1 file changed, 5 insertions(+), 3

[PATCH 0/5] staging: wlags49_h2: Fix checkpatch issues in wl_netdev.c

2014-03-19 Thread Jelena Bjelja
This patchset fixes most of the checkpatch.pl issues in wl_netdev.c Jelena Bjelja (5): staging: wlags49_h2: Enclose macro in a do-while loop Delete unnecessary spaces Remove trailing whitespace staging: wlags49_h2: Do not use assignment in if condition staging: wlags49_h2: Delete

[PATCH 3/5] Remove trailing whitespace

2014-03-19 Thread Jelena Bjelja
Remove trailing whitespace in wl_netdev.c Signed-off-by: Jelena Bjelja jelena.bjelja@gmail.com --- drivers/staging/wlags49_h2/wl_netdev.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wlags49_h2/wl_netdev.c b/drivers/staging/wlags49_h2/wl_netdev.c

[PATCH 2/5] Delete unnecessary spaces

2014-03-19 Thread Jelena Bjelja
Fix the following checkpatch.pl issues in wl_netdev.c: ERROR: space prohibited after that open parenthesis '(' #116: FILE: drivers/staging/wlags49_h2/wl_netdev.c:116: ERROR: space prohibited before that close parenthesis ')' #116: FILE: drivers/staging/wlags49_h2/wl_netdev.c:116: Signed-off-by:

[PATCH 4/5] staging: wlags49_h2: Do not use assignment in if condition

2014-03-19 Thread Jelena Bjelja
Fix the following checkpatch.pl issue in wl_netdev.c: ERROR: do not use assignment in if condition Signed-off-by: Jelena Bjelja jelena.bjelja@gmail.com --- drivers/staging/wlags49_h2/wl_netdev.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 5/5] staging: wlags49_h2: Delete unnecessary braces

2014-03-19 Thread Jelena Bjelja
Fix the following checkpatch.pl warnings in wl_netdev.c: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Jelena Bjelja jelena.bjelja@gmail.com --- drivers/staging/wlags49_h2/wl_netdev.c | 24 1 file changed, 8 insertions(+), 16

Failure to recognize device for 45 minutes and Device Disconnects

2014-03-19 Thread Mike Mitchell
Please help me narrow down this perplexing mystery: 1. USB Mic device takes ~45 minutes to recognize after warm start. 2. Same device disconnect sporadically, usually about an hour after it was last used. 3. Otherwise device works fine for days on end. Host: Intel NUC dccp847dye Device: CAD U7

Re: Failure to recognize device for 45 minutes and Device Disconnects

2014-03-19 Thread Alan Stern
On Wed, 19 Mar 2014, Mike Mitchell wrote: Please help me narrow down this perplexing mystery: 1. USB Mic device takes ~45 minutes to recognize after warm start. 2. Same device disconnect sporadically, usually about an hour after it was last used. 3. Otherwise device works fine for days on

Re: [PATCH 2/2] net: cdc_ncm: respect operator preferred MTU reported by MBIM

2014-03-19 Thread David Miller
From: Ben Chan benc...@chromium.org Date: Mon, 17 Mar 2014 21:00:06 -0700 On Mon, Mar 17, 2014 at 6:41 PM, David Miller da...@davemloft.net wrote: From: Ben Chan benc...@chromium.org Date: Mon, 17 Mar 2014 17:46:27 -0700 Thanks again for the review and tip. I've submitted patch v2 to

Re: [PATCH v3 1/2] USB: cdc: add MBIM extended functional descriptor structure

2014-03-19 Thread David Miller
From: Bjørn Mork bj...@mork.no Date: Tue, 18 Mar 2014 09:35:57 +0100 Or will patchwork keep track of tags across different patch versions? It doesn't, and should not. It has no idea if the patch changed or not, the ACK'er might not still agree with the patch if any changes were made. -- To

Re: [PATCH 2/5] Delete unnecessary spaces

2014-03-19 Thread Greg KH
On Wed, Mar 19, 2014 at 07:45:52PM +0100, Jelena Bjelja wrote: Fix the following checkpatch.pl issues in wl_netdev.c: Your subject is missing some prefixes :( -- To unsubscribe from this list: send the line unsubscribe linux-usb in the body of a message to majord...@vger.kernel.org More

Deadlock in usb-storage error handling

2014-03-19 Thread Alan Stern
On Wed, 19 Mar 2014, Andreas Reis wrote: I've uploaded a dmesg with the new debugging patch to bugzilla: https://bugzilla.kernel.org/attachment.cgi?id=130041 Thanks. I have now managed to reproduce many of the features of this problem on my own computer. James, I will need your help (or help

[PATCH net-next v4 0/2] adjust MTU as indicated by MBIM extended functional descriptor

2014-03-19 Thread Ben Chan
The MBIM extended functional descriptor, defined in Universal Serial Bus Communications Class Subclass Specification for Mobile Broadband Interface Model, Revision 1.0, Errata-1 by USB-IF, indicates the operator preferred MTU value via a wMTU field. This patch set ensures that the initial MTU

Re: Deadlock in usb-storage error handling

2014-03-19 Thread Dan Williams
On Wed, Mar 19, 2014 at 1:31 PM, Alan Stern st...@rowland.harvard.edu wrote: On Wed, 19 Mar 2014, Andreas Reis wrote: I've uploaded a dmesg with the new debugging patch to bugzilla: https://bugzilla.kernel.org/attachment.cgi?id=130041 Thanks. I have now managed to reproduce many of the

[PATCH net-next v4 2/2] net: cdc_ncm: respect operator preferred MTU reported by MBIM

2014-03-19 Thread Ben Chan
According to Universal Serial Bus Communications Class Subclass Specification for Mobile Broadband Interface Model, Revision 1.0, Errata-1 published by USB-IF, the wMTU field of the MBIM extended functional descriptor indicates the operator preferred MTU for IP data streams. This patch modifies

[PATCH net-next v4 1/2] USB: cdc: add MBIM extended functional descriptor structure

2014-03-19 Thread Ben Chan
This patch adds the MBIM extended functional descriptor structure defined in Universal Serial Bus Communications Class Subclass Specification for Mobile Broadband Interface Model, Revision 1.0, Errata-1 published by USB-IF. Signed-off-by: Ben Chan benc...@chromium.org Acked-by: Greg Kroah-Hartman

Re: Deadlock in usb-storage error handling

2014-03-19 Thread James Bottomley
On Wed, 2014-03-19 at 16:31 -0400, Alan Stern wrote: On Wed, 19 Mar 2014, Andreas Reis wrote: I've uploaded a dmesg with the new debugging patch to bugzilla: https://bugzilla.kernel.org/attachment.cgi?id=130041 Thanks. I have now managed to reproduce many of the features of this

[PATCH v7 16/16] usb: documentation for usb port power off mechanisms

2014-03-19 Thread Dan Williams
From: Lan Tianyu tianyu@intel.com describe the mechanisms for controlling port power policy and discovering the port power state. Cc: Oliver Neukum oneu...@suse.de [oliver]: fixes, clarification of wakeup vs port-power-control Signed-off-by: Lan Tianyu tianyu@intel.com [sarah]:

[PATCH v7 03/16] usb: cleanup setting udev-removable from port_dev-connect_type

2014-03-19 Thread Dan Williams
Once usb-acpi has set the port's connect type the usb_device's -removable attribute can be set in the standard location set_usb_port_removable(). This also changes behavior in the case where the firmware says that the port connect type is unknown. In that case just use the default setting

[PATCH v7 10/16] usb: usb3 ports do not support FEAT_C_ENABLE

2014-03-19 Thread Dan Williams
The port pm_runtime implementation unconditionally clears FEAT_C_ENABLE after clearing PORT_POWER, but the bit is reserved on usb3 hub ports. We expect khubd to be prevented from running because the port state is not RPM_ACTIVE, so we need to clear any errors for usb2 ports. Acked-by: Alan Stern

[PATCH v7 07/16] usb: sysfs link peer ports

2014-03-19 Thread Dan Williams
The usb topology after this change will have symlinks between usb3 ports and their usb2 peers, for example: usb2/2-1/2-1:1.0/2-1-port1/peer = ../../../../usb3/3-1/3-1:1.0/3-1-port1 usb2/2-1/2-1:1.0/2-1-port2/peer = ../../../../usb3/3-1/3-1:1.0/3-1-port2 usb2/2-1/2-1:1.0/2-1-port3/peer =

[PATCH v7 13/16] usb: introduce port status lock

2014-03-19 Thread Dan Williams
In general we do not want khubd to act on port status changes that are the result of in progress resets or USB runtime PM operations. Specifically port power control testing has been able to trigger an unintended disconnect in hub_port_connect_change(), paraphrasing: if ((portstatus

[PATCH v7 01/16] usb: disable port power control if not supported in wHubCharacteristics

2014-03-19 Thread Dan Williams
A hub indicates whether it supports per-port power control via the wHubCharacteristics field in its descriptor. If it is not supported a hub will still emulate ClearPortPower(PORT_POWER) requests by stopping the link state machine. However, since this does not save power do not bother

[PATCH v7 02/16] usb: rename usb_port device objects

2014-03-19 Thread Dan Williams
The current port name portX is ambiguous. Before adding more port messages rename ports to hub-device-name-portX This is an ABI change, but the suspicion is that it will go unnoticed as the port power control implementation has been broken since its introduction. If however, someone was relying

[PATCH v7 05/16] usb: assign usb3 external hub port peers

2014-03-19 Thread Dan Williams
Given that root hub port peers are already established, external hub peer ports can be determined by traversing the device topology: 1/ ascend to the parent hub and find the upstream port_dev 2/ walk -peer to find the peer port 3/ descend to the peer hub via -child 4/ find the port with the

[PATCH v7 12/16] usb: synchronize port poweroff and khubd

2014-03-19 Thread Dan Williams
If a port is powered-off, or in the process of being powered-off, prevent khubd from operating on it. Otherwise, the following sequence of events leading to an unintended disconnect may occur: Events: (0) set pm_qos_no_poweroff to '0' for port1 (1) hub 2-2:1.0: hub_resume (2) hub 2-2:1.0: port

[PATCH v7 04/16] usb: assign default peer ports for root hubs

2014-03-19 Thread Dan Williams
Assume that the peer of a superspeed port is the port with the same id on the shared_hcd root hub. This identification scheme is required of external hubs by the USB3 spec [1]. However, for root hubs, tier mismatch may be in effect [2]. Tier mismatch can only be enumerated via platform

[PATCH v7 00/16] port power control reworks

2014-03-19 Thread Dan Williams
Status: To date Alan has acked 9 of 16 patches (patch 1, 2, and 4 - 10), thanks Alan! Of course, if others want to jump in please feel free. Changes since v6 [1] [2] are simply rebasing/testing on top of current usb-next (commit 6aec044cc2f5 USB: unbind all interfaces before rebinding any),

[PATCH v7 11/16] usb: refactor port handling in hub_events()

2014-03-19 Thread Dan Williams
In preparation for synchronizing port handling with pm_runtime transitions refactor port handling into its own subroutine. We expect that clearing some status flags will be required regardless of the port state, so handle those first and group all non-trivial actions at the bottom of the routine.

[PATCH v7 14/16] usb: resume (wakeup) child device when port is powered on

2014-03-19 Thread Dan Williams
Unconditionally wake up the child device when the power session is recovered. This address the following scenarios: 1/ The device may need a reset on power-session loss, without this change port power-on recovery exposes khubd to scenarios that usb_port_resume() is set to handle. Prior to

[PATCH v7 15/16] usb: force warm reset to break link re-connect livelock

2014-03-19 Thread Dan Williams
Resuming a powered down port sometimes results in the port state being stuck in the training sequence. hub 3-0:1.0: debounce: port 1: total 2000ms stable 0ms status 0x2e0 port1: can't get reconnection after setting port power on, status -110 hub 3-0:1.0: port 1 status .02e0 after resume,

[PATCH v7 09/16] usb: don't clear FEAT_C_ENABLE on usb_port_runtime_resume failure

2014-03-19 Thread Dan Williams
Three reasons: 1/ It's an invalid operation on usb3 ports 2/ There's no guarantee of when / if a usb2 port has entered an error state relative to PORT_POWER request 3/ The port is active / powered at this point, so khubd will clear it as a matter of course Acked-by: Alan Stern

[PATCH v7 08/16] usb: block suspension of superspeed port while hispeed peer is active

2014-03-19 Thread Dan Williams
ClearPortFeature(PORT_POWER) on a usb3 port places the port in either a DSPORT.Powered-off-detect / DSPORT.Powered-off-reset loop, or the DSPORT.Powered-off state. There is no way to ensure that RX terminations will persist in this state, so it is possible a device will degrade to its usb2

[PATCH 1/2] coding style: fix quoted string split across lines

2014-03-19 Thread Cédric Cabessa
Signed-off-by: Cédric Cabessa c...@ryick.net --- drivers/staging/usbip/stub_tx.c | 16 drivers/staging/usbip/usbip_common.c | 10 -- drivers/staging/usbip/vhci_sysfs.c | 4 ++-- 3 files changed, 14 insertions(+), 16 deletions(-) diff --git

[PATCH 0/2] coding style for drivers/staging/usbip

2014-03-19 Thread Cédric Cabessa
Various coding style fixes for drivers/staging/usbip Cédric Cabessa (2): coding style: fix quoted string split across lines coding style: fix line over 80 characters drivers/staging/usbip/stub_tx.c | 16 drivers/staging/usbip/usbip_common.c | 10 --

[PATCH 2/2] coding style: fix line over 80 characters

2014-03-19 Thread Cédric Cabessa
Signed-off-by: Cédric Cabessa c...@ryick.net --- drivers/staging/usbip/vhci_hcd.c | 33 ++--- 1 file changed, 22 insertions(+), 11 deletions(-) diff --git a/drivers/staging/usbip/vhci_hcd.c b/drivers/staging/usbip/vhci_hcd.c index f690668..1e84577 100644 ---

[PATCH] staging: octeon-usb: prevent memory corruption

2014-03-19 Thread Aaro Koskinen
octeon-hcd will crash the kernel when SLOB is used. This usually happens after the 18-byte control transfer when a device descriptor is read. The DMA engine is always transfering full 32-bit words and if the transfer is shorter, some random garbage appears after the buffer. The problem is not

Re: usbnet: driver_info-stop required to stop USB interrupts?

2014-03-19 Thread Grant Grundler
On Tue, Mar 18, 2014 at 6:40 PM, Grant Grundler grund...@google.com wrote: On Tue, Mar 18, 2014 at 6:09 PM, Julius Werner jwer...@chromium.org wrote: I think a better place for this would be in usbnet_probe() (together with all the other dev-xxx initialization). Definitely better. @@

Re: [PATCH 1/2] extcon: of: Remove unnecessary function call by using the name of device_node

2014-03-19 Thread Chanwoo Choi
Hi, On 03/19/2014 09:08 PM, Kishon Vijay Abraham I wrote: Hi, On Tuesday 18 March 2014 05:34 PM, Chanwoo Choi wrote: This patch remove unnecessary function call in of_extcon_get_extcon_dev() by using the name of device_node structure. Signed-off-by: Chanwoo Choi cw00.c...@samsung.com ---

Re: [PATCH 1/2] extcon: of: Remove unnecessary function call by using the name of device_node

2014-03-19 Thread Kishon Vijay Abraham I
Hi, On Thursday 20 March 2014 07:52 AM, Chanwoo Choi wrote: Hi, On 03/19/2014 09:08 PM, Kishon Vijay Abraham I wrote: Hi, On Tuesday 18 March 2014 05:34 PM, Chanwoo Choi wrote: This patch remove unnecessary function call in of_extcon_get_extcon_dev() by using the name of device_node

RE: [PATCH] usb: gadget: fsl: Add FSL USB Gadget entry in platform device id

2014-03-19 Thread suresh.gu...@freescale.com
Ignore last mail, was not intended for you. -Original Message- From: Gupta Suresh-B42813 Sent: Wednesday, March 19, 2014 11:33 PM To: 'gre...@linuxfoundation.org' Cc: ba...@ti.com; linux-usb@vger.kernel.org; linux-ker...@vger.kernel.org Subject: RE: [PATCH] usb: gadget: fsl: Add