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

Hi, On 03/20/2014 02:20 PM, Kishon Vijay Abraham I wrote: > 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

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

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: A

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

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 de

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

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 >> --- >> driv

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

On Wed, Mar 19, 2014 at 11:04:56PM +0100, Cédric Cabessa wrote: > Signed-off-by: Cédric Cabessa In the future, please pick a subject line that says what part of the kernel you are changing. For example, this patch would be: staging: usbip: fix quoted string split across lines I'll fix t

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

On Tue, Mar 18, 2014 at 6:40 PM, Grant Grundler wrote: > On Tue, Mar 18, 2014 at 6:09 PM, Julius Werner wrote: >> I think a better place for this would be in usbnet_probe() (together >> with all the other dev->xxx initialization). > > Definitely better. > > @@ -1536,6 +1536,7 @@ usbnet_probe (str

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

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 visib

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

Signed-off-by: Cédric Cabessa --- 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 --- a/drivers/staging/usbip/

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

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 -- drivers/stag

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

Signed-off-by: Cédric Cabessa --- 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 a/drivers/staging/usbip/stub_tx.c b/dr

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

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 08/16] usb: block suspension of superspeed port while hispeed peer is active

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 connect

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

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 m

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

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 firmware.

[PATCH v7 06/16] usb: find internal hub tier mismatch via acpi

ACPI identifies peer ports by setting their 'group_token' and 'group_position' _PLD data to the same value. If a platform has tier mismatch [1] , ACPI can override the default (USB3 defined) peer port association for internal hubs. External hubs follow the default peer association scheme. Locati

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

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 02/16] usb: rename usb_port device objects

The current port name "portX" is ambiguous. Before adding more port messages rename ports to "-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 on the old

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

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 09/16] usb: don't clear FEAT_C_ENABLE on usb_port_runtime_resume failure

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 Signed-off-by

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

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) (1) hub 2-2:1.0: hub_resume (2) hub 2-2:1.0: port 1: status 0301 change (3) hub 2-2:1.

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

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

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

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 & USB_

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

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 suspending.

[PATCH v7 00/16] port power control reworks

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"), fix

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

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 determ

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

From: Lan Tianyu describe the mechanisms for controlling port power policy and discovering the port power state. Cc: Oliver Neukum [oliver]: fixes, clarification of wakeup vs port-power-control Signed-off-by: Lan Tianyu [sarah]: wordsmithing [djbw]: updates for peer port changes Signed-off-by:

Re: Deadlock in usb-storage error handling

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 net-next v4 2/2] net: cdc_ncm: respect operator preferred MTU reported by MBIM

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

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 Acked-by: Greg Kroah-Hartman --- include/uapi

Re: Deadlock in usb-storage error handling

On Wed, Mar 19, 2014 at 1:31 PM, 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 > pro

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

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 va

Deadlock in usb-storage error handling

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 hel

Re: [PATCH 2/5] Delete unnecessary spaces

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 major

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

From: Bjørn Mork 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 unsubscribe fr

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

From: Ben Chan Date: Mon, 17 Mar 2014 21:00:06 -0700 > On Mon, Mar 17, 2014 at 6:41 PM, David Miller wrote: >> From: Ben Chan >> Date: Mon, 17 Mar 2014 17:46:27 -0700 >> >>> Thanks again for the review and tip. I've submitted patch v2 to >>> address the le16_to_cpu conversion. >> >> When you up

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

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 o

Failure to recognize device for 45 minutes and Device Disconnects

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 Mic

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

Fix the following checkpatch.pl warnings in wl_netdev.c: WARNING: braces {} are not necessary for single statement blocks Signed-off-by: Jelena Bjelja --- drivers/staging/wlags49_h2/wl_netdev.c | 24 1 file changed, 8 insertions(+), 16 deletions(-) diff --git a/driver

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

Fix the following checkpatch.pl issue in wl_netdev.c: ERROR: do not use assignment in if condition Signed-off-by: Jelena Bjelja --- drivers/staging/wlags49_h2/wl_netdev.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/wlags49_h2/wl_netdev.c b/drivers/st

[PATCH 2/5] Delete unnecessary spaces

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 3/5] Remove trailing whitespace

Remove trailing whitespace in wl_netdev.c Signed-off-by: Jelena Bjelja --- 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 index 8e571b2..eb428ae 100644

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

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 unneces

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

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 --- drivers/staging/wlags49_h2/wl_netdev.c |8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/sta

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

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 --- drivers/usb/chipidea/host.c |7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/usb/chipidea/host.c b/drivers/usb/chip

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

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; li

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

Hello. On 03/19/2014 02:32 PM, Heena Sirwani wrote: This patchset consists of patches that fix line over 80 characters warning in bcm module in Adapter.h.Please review the patches that follow. It's very strange that you broke this change up into 6 patches. You're fixing the same issue in

Re: Support for new USB Modem 4G drive

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/Li

Re: Support for new USB Modem 4G drive

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 f

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

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.ker

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

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 > --- > drivers/staging/bcm/Adapter.h |9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/drivers

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

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 You have 5 patches here with the same Subject:, which I can't take. Please uniquely name the subject and resend. thanks, greg k-

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

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 > --- > drivers/staging/bcm/Adapter.h | 23 ++- > 1 file changed, 14 insertions(+), 9 deletions(-) > > diff

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

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.kerne

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

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 > --- > drivers/staging/wlags49_h2/wl_n

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

> > 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 lin

Re: problem with resume after s2ram

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

Re: questions about give back urb in tasklet

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,

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

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

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

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 U

Re: questions about give back urb in tasklet

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 m

Re: questions about give back urb in tasklet

On Wed, Mar 19, 2014 at 9:34 PM, vichy 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

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 pl

Re: questions about give back urb in tasklet

On Wed, Mar 19, 2014 at 5:57 PM, vichy wrote: > hi Ming.lei: > 2014-03-17 22:01 GMT+08:00 Alan Stern : >> 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 h

Re: Support for new USB Modem 4G drive

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

hi all: 2014-03-19 17:57 GMT+08:00 vichy : > hi Ming.lei: > 2014-03-17 22:01 GMT+08:00 Alan Stern : >> 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 http

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

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: connect-deboun

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

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 > --- > drivers/extcon/of_extcon.c | 12 ++-- > 1 file changed, 2 insertio

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

Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani --- drivers/staging/bcm/Adapter.h | 20 ++-- 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h index be959b

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

Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani --- 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 index d0c8cec..be959b9 100

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

Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani --- 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 index d241164..d0c8cec 1006

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

Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani --- 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 index a387234..d241164 1

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

Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani --- drivers/staging/bcm/Adapter.h | 23 ++- 1 file changed, 14 insertions(+), 9 deletions(-) diff --git a/drivers/staging/bcm/Adapter.h b/drivers/staging/bcm/Adapter.h index 1b2

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

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. > > Ther

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

Fix checkpatch.pl issues with line over 80 characters in Adapter.h Signed-off-by: Heena Sirwani --- 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 index f0d6f0c..1b2d9f3 1006

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

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. s

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

From: Li Jun 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 --- drivers/usb/chipidea/otg.c |1 + drivers/usb/chipidea/otg_fsm.c | 174 +++

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

Init otg_port number of otg capable host to be 1 at host start. Signed-off-by: Li Jun --- 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 100644 --- a/drivers

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

From: Li Jun 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 --- drivers/usb/chipidea/bits.h|1 + drivers/usb/chipidea/otg_fsm.c | 188 +++

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

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 --- drivers/usb/chipidea/Makefile |1 + drivers/usb/chipidea/ci.h |2 ++ drivers/usb/chipidea/otg.c |7 + drivers/usb/chipidea/otg_f

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

From: Li Jun 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 --- Documentation/usb/chipidea

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

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

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

Add b_hnp_enable request handling and enable gadget->is_otg Signed-off-by: Li Jun --- 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 100644 --- a/drivers/u

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

From: Li Jun This patch adds a debug file for OTG vairables show. Signed-off-by: Li Jun --- drivers/usb/chipidea/debug.c | 84 ++ 1 file changed, 84 insertions(+) diff --git a/drivers/usb/chipidea/debug.c b/drivers/usb/chipidea/debug.c index 5b890c1..

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

Leave vbus on/off hanlded by OTG fsm if in OTG mode. Signed-off-by: Li Jun --- 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 b/drivers/usb/chipidea/host.c index

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

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 --- drivers/usb/chipidea/bits.h| 10 +- drivers/usb/chipidea/ci.h

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

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

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

From: Li Jun 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 Host Supplement to the

Re: questions about give back urb in tasklet

hi Ming.lei: 2014-03-17 22:01 GMT+08:00 Alan Stern : > 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 http://permalink.gmane.org/gmane.linux.usb.general/89363 I h

Support for new USB Modem 4G drive

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 driver.