Re: REGRESSION: usbdevfs: Use-scatter-gather-lists-for-large-bulk-transfers

Xiaofan Chen wrote: > On Thu, Oct 11, 2012 at 2:50 PM, Peter Stuge wrote: > > But it isn't certain that is is involved at all. If your impact > > version uses libusb.so like in ISE 11.1 then it will not be. One way > > to test is by trying to generate a libusb debug log. See > > https://libusb.org

Re: [PATCH 1/3] USB: EHCI: move logging macros to ehci.h

On Wed, Oct 10, 2012 at 03:07:30PM -0400, Alan Stern wrote: > In preparation for splitting the ehci-hcd driver into a core library > and separate platform-specific driver modules, this patch (as1616) > moves the console logging macros from ehci-dbg.c to ehci.h, where they > will be available to the

Re: [PATCH 2/3] USB: EHCI: make ehci_read_frame_index platform independent

On Wed, Oct 10, 2012 at 03:07:39PM -0400, Alan Stern wrote: > In preparation for splitting the ehci-hcd driver into a core library > and separate platform-specific driver modules, this patch (as1617) > changes the way ehci_read_frame_index() is handled. > > Since the same core library will have to

Re: [PATCH 3/3] USB: EHCI: move ehci_update_device() to ehci-lpm.c

On Wed, Oct 10, 2012 at 03:07:46PM -0400, Alan Stern wrote: > In preparation for splitting the ehci-hcd driver into a core library > and separate platform-specific driver modules, this patch (as1618) > moves ehci_update_device() from a couple of platform-specific source > files into ehci-lpm.c. Th

Re: [PATCH 2/3] usb: xhci: Add the suspend/resume functionality

On Wed, Oct 10, 2012 at 10:58:16AM -0700, Sarah Sharp wrote: > On Wed, Oct 10, 2012 at 07:35:48PM +0530, Vikas C Sajjan wrote: > > From: Vikas Sajjan > > > > Adding the suspend and resume functionality for the XHCI driver > > > > Signed-off-by: Abhilash Kesavan > > Signed-off-by: Vikas C Sajjan

Re: [PATCH 01/12] usbnet: introduce usbnet 3 command helpers

On Thursday 11 October 2012 11:18:22 Ming Lei wrote: > On Wed, Oct 10, 2012 at 1:51 PM, Oliver Neukum wrote: > > > > No, the problem is autoresume. > > > > Suppose we have a device with two interface. Interface A be usbnet; > > interface B > > something you page on. Now consider that you can only

Re: [PATCH 3/3] exynos5: usb: dwc3: Add the suspend/resume functionality

Hi, On Wed, Oct 10, 2012 at 07:35:49PM +0530, Vikas C Sajjan wrote: > From: Vikas Sajjan > > Adding the suspend and resume functionality to exynos dwc3 driver > > Signed-off-by: Abhilash Kesavan > Signed-off-by: Vikas C Sajjan > CC: Doug Anderson > --- > drivers/usb/dwc3/dwc3-exynos.c | 6

Re: [PATCH 0/3] USB: dwc3: Add suspend/resume support

On Wed, Oct 10, 2012 at 07:35:46PM +0530, Vikas C Sajjan wrote: > From: Vikas Sajjan > > This patchset adds suspend/resume functionality to dwc3-core layer > and xhci-platform driver. It also adds S2R support to dwc3-exynos > glue layer. how has this been tested ? For how long ? > Based on 'usb

Re: [PATCH 1/3] usb: dwc3: Add the suspend/resume functionality

Hi, On Wed, Oct 10, 2012 at 07:35:47PM +0530, Vikas C Sajjan wrote: > From: Vikas Sajjan > > Adding the suspend and resume funtionality to DWC3 core. > > Signed-off-by: Abhilash Kesavan > Signed-off-by: Vikas C Sajjan > CC: Doug Anderson > --- > drivers/usb/dwc3/core.c | 268 ++

Re: [PATCH 2/3] usb: xhci: Add the suspend/resume functionality

On Wed, Oct 10, 2012 at 07:35:48PM +0530, Vikas C Sajjan wrote: > From: Vikas Sajjan > > Adding the suspend and resume functionality for the XHCI driver > > Signed-off-by: Abhilash Kesavan > Signed-off-by: Vikas C Sajjan > CC: Doug Anderson > --- > drivers/usb/host/xhci-plat.c | 44 >

Re: [PATCH 01/12] usbnet: introduce usbnet 3 command helpers

On Thu, Oct 11, 2012 at 12:11 PM, Oliver Neukum wrote: > On Thursday 11 October 2012 11:18:22 Ming Lei wrote: >> Currently scsi disk can only be runtime suspended when the device is not >> opened, so are you sure that the paging out above can cause IO on a suspend >> usb mass storage disk which i

Re: REGRESSION: usbdevfs: Use-scatter-gather-lists-for-large-bulk-transfers

Hi, On 10/10/2012 10:31 PM, Henrik Rydberg wrote: Hi Hans, Alan, Greg, commit 3d97ff63f8997761f12c8fbe8082996c6eeaba1a Author: Hans de Goede Date: Wed Jul 4 09:18:03 2012 +0200 usbdevfs: Use scatter-gather lists for large bulk transfers breaks an usb programming cable over here. The p

usb audio device can't work when it is attached to host via usb 2.0 hub

Hi all, I recently found a strange issue on a embedded platform and don't know how to fix it. Please give me some suggestions. The platform is ar9331. The issue is if we attach a usb audio device to the host on ar9331 directly or via a usb 1.1 hub the usb audio works fine, but if we attach it via

Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc

On 10/11/2012 12:05 PM, Heiko Schocher wrote: > Hello Manjunathappa > > On 11.10.2012 07:42, Manjunathappa, Prakash wrote: >> Hi, >> On Mon, Oct 08, 2012 at 18:47:07, Constantine Shulyupin wrote: >>> From: Constantine Shulyupin >>> >>> Remove USB configuration in arch/arm/mach-davinci/usb.c accord

Re: [RFC] net: usbnet: add pre and post_reset support

Bjørn Mork writes: > Oliver Neukum writes: > >> On Wednesday 10 October 2012 18:34:52 Bjørn Mork wrote: >>> Signed-off-by: Bjørn Mork >>> --- >>> What do you think about something like this? Does it have to >>> be more complicated? Somewhat tested, and seems to do the >>> job for me. >> >> Look

Re: [PATCH 2/3] usb: xhci: Add the suspend/resume functionality

Hi, On Thu, Oct 11, 2012 at 02:22:42PM +0530, Vikas Sajjan wrote: > Hi Felipe, > > Thanks for the comments, will get back with modifications. cool, but please avoid top posting ;-) -- balbi signature.asc Description: Digital signature

Re: [PATCH 01/12] usbnet: introduce usbnet 3 command helpers

On Thursday 11 October 2012 16:14:02 Ming Lei wrote: > On Thu, Oct 11, 2012 at 12:11 PM, Oliver Neukum wrote: > > On Thursday 11 October 2012 11:18:22 Ming Lei wrote: > > >> Currently scsi disk can only be runtime suspended when the device is not > >> opened, so are you sure that the paging out a

Re: FTDI USB-to-UART converters and tcdrain()

On Thu, 2012-10-11 at 04:57 +0900, Greg KH wrote: > The patch looks great, thanks so much for doing this. I'll queue it up > after 3.7-rc1 is released by Linus (which should be in a few days). No problem. It's pretty cool to be able to contribute. -- To unsubscribe from this list: send the line

Re: usb audio device can't work when it is attached to host via usb 2.0 hub

Elric Fu wrote: > It is a usb 1.1 device and works at full speed. It is a frequent misunderstanding that a device must be 1.1 only because it does not support high speed. This is not the case. > But the interesting thing is the bcdUSB field of device descriptor > is 0x0200. It is perfectly lega

Re: [RFC] net: usbnet: add pre and post_reset support

On Thursday 11 October 2012 11:05:05 Bjørn Mork wrote: > Bjørn Mork writes: > This is harmless because the driver ignores the sequence numbers anyway. > But if the disconnect on reset is typical for NCM devices, then I guess > it does not help much adding pre and post_reset in cdc_ncm. Nor does

Re: [RFC] net: usbnet: add pre and post_reset support

On Thursday 11 October 2012 11:05:05 Bjørn Mork wrote: > So I guess that demonstrates the issue with pre and post_reset on > devices keeping internal state pretty well: The effect is entirely > device dependent. The device reset behaviour is not known, and we risk > that the device changes some of

Re: [PATCH] usb: remove CONFIG_USB_MUSB_HOST etc

I've tested gadget mode on DM365 without HW changes. On Thu, Oct 11, 2012 at 10:50 AM, Sekhar Nori wrote: > On 10/11/2012 12:05 PM, Heiko Schocher wrote: >> Hello Manjunathappa >> >> On 11.10.2012 07:42, Manjunathappa, Prakash wrote: >>> Hi, >>> On Mon, Oct 08, 2012 at 18:47:07, Constantine Shuly

Re: removing the timer from cdc-ncm

Oliver Neukum writes: > On Tuesday 25 September 2012 13:18:10 Alexey ORISHKO wrote: > >> Linux host got panic right after data path has been established >> (i.e. connected to mobile network). > > Thank you. Worse than I hoped, but not unexpected. I'll stare at the > code a bit. I had the same ex

Re: [RFC] net: usbnet: add pre and post_reset support

Oliver Neukum writes: > On Thursday 11 October 2012 11:05:05 Bjørn Mork wrote: >> So I guess that demonstrates the issue with pre and post_reset on >> devices keeping internal state pretty well: The effect is entirely >> device dependent. The device reset behaviour is not known, and we risk >> tha

Re: cannot submit urb 0, error -22: internal error followed by USB hung tasks

On 21/09/12 22:44, Alan Stern wrote: > What is the Razer DeathAdder? A mouse. (With some extra buttons) > I can't tell much from these logs. A usbmon trace might help more, > although I don't know whether it would be better to trace the bus for > the headset or the bus for the camera. See the

Re: [PATCH 01/12] usbnet: introduce usbnet 3 command helpers

On Thu, Oct 11, 2012 at 5:05 PM, Oliver Neukum wrote: > That depends on whether an API change would be necessary. > Changing the code only when necessary is no problem. But the > API I want to do right from the beginning if that is possible. For the auto-resume situation, the current API is OK ev

USB errors, ehci testing modes

Hi I have some stange errors on a arm pcm043 with internal usb phy. The log is attached at the end of the mail. It would be nice if someone could give me a pointer whats going wrong?. It would also be nice to know where the status flags which are output by the usb debug messages are documented?

Re: removing the timer from cdc-ncm

Bjørn Mork writes: > [48880.048494] BUG: unable to handle kernel NULL pointer dereference at > 0068 > [48880.048573] IP: [] cdc_ncm_tx_bundle+0x168/0x43b > [cdc_ncm] This one is because you removed the "if (skb == NULL)" from the for loop, but left the "skb = NULL;" at the end: @

[PATCH v1 00/11] usbnet: usb_control_msg cleanup

Hi, This patch set introduces 3 helpers for handling usb read, write and write_async command, and replaces the low level's implemention with the generic ones. This patchset is a cleanup and about 300 lines code can be saved. Also, the patchset fixes problem of DMA on buffer embedded inside one d

[PATCH v1 01/11] usbnet: introduce usbnet 3 command helpers

This patch introduces the below 3 usb command helpers: usbnet_read_cmd / usbnet_write_cmd / usbnet_write_cmd_async so that each low level driver doesn't need to implement them by itself, and the dma buffer allocation for usb transfer and runtime PM things can be handled just in one place.

[PATCH v1 02/11] usbnet: asix: apply introduced usb command APIs

Signed-off-by: Ming Lei --- drivers/net/usb/asix_common.c | 117 + 1 file changed, 13 insertions(+), 104 deletions(-) diff --git a/drivers/net/usb/asix_common.c b/drivers/net/usb/asix_common.c index 774d9ce..50d1673 100644 --- a/drivers/net/usb/asix_comm

[PATCH v1 03/11] usbnet: cdc-ncm: apply introduced usb command APIs

Signed-off-by: Ming Lei --- drivers/net/usb/cdc_ncm.c | 73 ++--- 1 file changed, 23 insertions(+), 50 deletions(-) diff --git a/drivers/net/usb/cdc_ncm.c b/drivers/net/usb/cdc_ncm.c index 4cd582a..429a2ad 100644 --- a/drivers/net/usb/cdc_ncm.c +++ b/dr

[PATCH v1 04/11] usbnet: dm9601: apply introduced usb command APIs

Signed-off-by: Ming Lei --- drivers/net/usb/dm9601.c | 107 +++--- 1 file changed, 15 insertions(+), 92 deletions(-) diff --git a/drivers/net/usb/dm9601.c b/drivers/net/usb/dm9601.c index e0433ce..3f554c1 100644 --- a/drivers/net/usb/dm9601.c +++ b/drive

[PATCH v1 05/11] usbnet: int51x1: apply introduced usb command APIs

Signed-off-by: Ming Lei --- drivers/net/usb/int51x1.c | 52 +++-- 1 file changed, 3 insertions(+), 49 deletions(-) diff --git a/drivers/net/usb/int51x1.c b/drivers/net/usb/int51x1.c index 8de6417..ace9e74 100644 --- a/drivers/net/usb/int51x1.c +++ b/dri

[PATCH v1 06/11] usbnet: mcs7830: apply introduced usb command APIs

Signed-off-by: Ming Lei --- drivers/net/usb/mcs7830.c | 85 - 1 file changed, 6 insertions(+), 79 deletions(-) diff --git a/drivers/net/usb/mcs7830.c b/drivers/net/usb/mcs7830.c index 03c2d8d..db46a68 100644 --- a/drivers/net/usb/mcs7830.c +++ b/dri

[PATCH v1 07/11] usbnet: net1080: apply introduced usb command APIs

Signed-off-by: Ming Lei --- drivers/net/usb/net1080.c | 110 + 1 file changed, 30 insertions(+), 80 deletions(-) diff --git a/drivers/net/usb/net1080.c b/drivers/net/usb/net1080.c index c062a3e..93e0716 100644 --- a/drivers/net/usb/net1080.c +++ b/dr

[PATCH v1 08/11] usbnet: plusb: apply introduced usb command APIs

Signed-off-by: Ming Lei --- drivers/net/usb/plusb.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/net/usb/plusb.c b/drivers/net/usb/plusb.c index 4584b9a..0fcc8e6 100644 --- a/drivers/net/usb/plusb.c +++ b/drivers/net/usb/plusb.c @@ -71,13 +71,10 @@ s

[PATCH v1 09/11] usbnet: sierra_net: apply introduced usb command APIs

Signed-off-by: Ming Lei --- drivers/net/usb/sierra_net.c | 45 -- 1 file changed, 17 insertions(+), 28 deletions(-) diff --git a/drivers/net/usb/sierra_net.c b/drivers/net/usb/sierra_net.c index c27d277..eb5c7a8 100644 --- a/drivers/net/usb/sierra_net.c

[PATCH v1 10/11] usbnet: smsc75xx: apply introduced usb command APIs

Signed-off-by: Ming Lei --- drivers/net/usb/smsc75xx.c | 39 ++- 1 file changed, 14 insertions(+), 25 deletions(-) diff --git a/drivers/net/usb/smsc75xx.c b/drivers/net/usb/smsc75xx.c index b77ae76..1baa53a 100644 --- a/drivers/net/usb/smsc75xx.c +++ b/drive

[PATCH v1 11/11] usbnet: smsc95xx: apply introduced usb command APIs

Signed-off-by: Ming Lei --- drivers/net/usb/smsc95xx.c | 115 +++- 1 file changed, 27 insertions(+), 88 deletions(-) diff --git a/drivers/net/usb/smsc95xx.c b/drivers/net/usb/smsc95xx.c index 7479a57..1730f75 100644 --- a/drivers/net/usb/smsc95xx.c +++ b/

Re: [PATCH 01/12] usbnet: introduce usbnet 3 command helpers

On Thu, 11 Oct 2012, Oliver Neukum wrote: > On Thursday 11 October 2012 11:18:22 Ming Lei wrote: > > On Wed, Oct 10, 2012 at 1:51 PM, Oliver Neukum wrote: > > > > > > No, the problem is autoresume. > > > > > > Suppose we have a device with two interface. Interface A be usbnet; > > > interface B

Re: Linux Kernel: NP dereference on removing usb device

On Fri, 12 Oct 2012, L N Wirz wrote: > Dear Alan, > > I found your address on the lkml when I looked for someone who recently > worked on the usb subsystem. Messages like this should always be CC'ed to the linux-usb mailing list. > I've encountered a reproducible kernel oops, when I remove a (

Re: usb audio device can't work when it is attached to host via usb 2.0 hub

On Thu, 11 Oct 2012, Elric Fu wrote: > Hi all, > > I recently found a strange issue on a embedded platform and don't know > how to fix it. Please give me some suggestions. > > The platform is ar9331. The issue is if we attach a usb audio device to > the host on ar9331 directly or via a usb 1.1 h

usb audio race at disconnect time

Hi, while doing some monkey tests on a product we found races in usb audio code when the device in unplugged from usb (on linus master tree). This can be reproduced with usb_audio_show_race.diff and CONFIG_DEBUG_SLAB. With this patch, start a stream : # arecord -D hw:0 -r 44100 -c 2 -f S16_LE >

Re: cannot submit urb 0, error -22: internal error followed by USB hung tasks

On Thu, 11 Oct 2012, Gian-Carlo Pascutto wrote: > I noticed that sometimes when booting the USB subsystem already behaves > erratically. For example this morning I'm greeted by: > > [ 234.455604] ohci_hcd :00:04.0: urb 8801296fac80 path 4 ep1in > 9312 cc 9 --> status -121 > [ 234.46

Re: USB errors, ehci testing modes

On Thu, 11 Oct 2012, Tim Sander wrote: > Hi > > I have some stange errors on a arm pcm043 with internal usb phy. The log is > attached at the end of the mail. It would be nice if someone could give me a > pointer whats going wrong?. It would also be nice to know where the status > flags which

Re: usb audio race at disconnect time

On Thu, 11 Oct 2012, Matthieu CASTET wrote: > Hi, > > while doing some monkey tests on a product we found races in usb audio code > when > the device in unplugged from usb (on linus master tree). > > This can be reproduced with usb_audio_show_race.diff and CONFIG_DEBUG_SLAB. > With this patch,

Re: cannot submit urb 0, error -22: internal error followed by USB hung tasks

On 11/10/12 17:19, Alan Stern wrote: >> Maybe my hardware/chipset is just crap? > > I don't know, maybe. Maybe that's just a bad cable connection (this > includes the possibility of bad connections inside the PC case). Ouch. I can check a few things, like the USB card reader. It should be poss

[PATCH] kaweth: print correct debug ptr

From: Alan Cox We nowdays copy the buffer and free fw->data, so make the debug printk use the right thing. Signed-off-by: Alan Cox --- drivers/net/usb/kaweth.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/usb/kaweth.c b/drivers/net/usb/kaweth.c index c75

[PATCH v7] Enable USB peripheral mode on dm365 EVM

From: Constantine Shulyupin Sets USB PHY clock source to 24 MHz clock. Tested with OTG configuration, usb gadget g_zero on DM365 EVM connected to PC. To active the patch need to call davinci_setup_usb from dm365_evm_init References: Definition of USB_PHY_CTRL and PHYCLKFREQ: - http://www.make

Re: usb audio race at disconnect time

[Added Daniel and Clemens in the loop] At Thu, 11 Oct 2012 17:17:59 +0200, Matthieu CASTET wrote: > > Hi, > > while doing some monkey tests on a product we found races in usb audio code > when > the device in unplugged from usb (on linus master tree). > > This can be reproduced with usb_audio_

[PATCH v7] Initialize USB on dm365 EVM

From: Constantine Shulyupin Call USB initialization davinci_setup_usb from board initialization dm365_evm_init. Tested with OTG configuration, usb gadget g_zero on DM365 EVM connected to PC. Note: register USB_PHY_CTRL must have flag USBPHY_CLKFREQ_24MHZ References: Original patch by miguel.

Re: cannot submit urb 0, error -22: internal error followed by USB hung tasks

On Thu, 11 Oct 2012, Gian-Carlo Pascutto wrote: > On 11/10/12 17:19, Alan Stern wrote: > > >> Maybe my hardware/chipset is just crap? > > > > I don't know, maybe. Maybe that's just a bad cable connection (this > > includes the possibility of bad connections inside the PC case). > > Ouch. I ca

Hibernation with LPM, was: Re: xhci: LPM issues using Western Digital harddrive

On Mon, Oct 01, 2012 at 03:53:03PM -0400, Alan Stern wrote: > On Mon, 1 Oct 2012, Sarah Sharp wrote: > > > > I don't remember the details. Doesn't usb_unlocked_disable_lpm() force > > > the device back into U0? You call that routine before suspending the > > > port. > > > > It sends the contr

Re: Link PM policy advice?

On Tue, Oct 09, 2012 at 03:36:59PM -0400, Alan Stern wrote: > On Tue, 9 Oct 2012, Sarah Sharp wrote: > > > Hmm, so I've tested the VIA hubs under a different Intel chipset (Lynx > > Point), and they exhibit the same issues as when a bus analyzer is > > connected on Panther Point. Only the issues

Re: Etron USB 3.0 xHCI

On Wed, Oct 10, 2012 at 02:15:33PM -0700, Matthew Hall wrote: > On Mon, Oct 08, 2012 at 02:34:24PM -0700, Gary E. Miller wrote: > > > The error you're getting seems to me to indicate a > > > hardware issue, and I'm really not sure how to solve it. > > > > Could be, I find many people have problems

Re: Hibernation with LPM, was: Re: xhci: LPM issues using Western Digital harddrive

On Thu, 11 Oct 2012, Sarah Sharp wrote: > Thinking about this further, USB 2.1 devices need to be brought out of > their low power link state (L1) before they are suspended. Some xHCI > host controllers have hardware-driven USB 2.1 LPM, so the device could > have been put in L1 before hibernation

Re: Link PM policy advice?

On Thu, 11 Oct 2012, Sarah Sharp wrote: > When LPM fails, the device often goes into the Inactive state, which > also marks the port as not connected. The USB core will issue a warm > reset to the device and then do a logical disconnect. VIA hubs come > back fine after the warm reset, but that d

Re: [PATCH] kaweth: print correct debug ptr

From: Alan Cox Date: Thu, 11 Oct 2012 17:22:03 +0100 > From: Alan Cox > > We nowdays copy the buffer and free fw->data, so make the debug printk use > the right thing. > > Signed-off-by: Alan Cox Applied and queued up for -stable. -- To unsubscribe from this list: send the line "unsubscribe

Re: About resume time optimization for bus resume routine

On Thu, 11 Oct 2012, Peter Chen wrote: > > You could test all the port status registers. If any of them have the > > PORT_PE bit set and the PORT_SUSPEND bit clear then the delay is > > needed; otherwise it can be skipped. > > I think the condition should like below, as we need to consider remot

[PATCH 1/4] USB: Enable LPM after a failed probe.

Before a driver is probed, we want to disable USB 3.0 Link Power Management (LPM), in case the driver needs hub-initiated LPM disabled. After the probe finishes, we want to attempt to re-enable LPM, order to balance the LPM ref count. When a probe fails (such as when libusual doesn't want to bind

[PATCH 2/4] usb: Don't enable LPM if the exit latency is zero.

Some USB 3.0 devices signal that they don't implement Link PM by having all zeroes in the U1/U2 exit latencies in their SuperSpeed BOS descriptor. Don found that a Western Digital device he has experiences transfer errors when LPM is enabled. The lsusb shows the U1/U2 exit latencies are set to ze

[PATCH 4/4] usb: trival: Fix debugging units mistake.

SEL and PEL are in microseconds, not milliseconds. Also, fix a split string that will trigger checkpatch warnings. Signed-off-by: Sarah Sharp --- drivers/usb/core/hub.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c ind

[Pull Request] Link PM patches for 3.7

The following changes since commit ecefbd94b834fa32559d854646d777c56749ef1c: Merge tag 'kvm-3.7-1' of git://git.kernel.org/pub/scm/virt/kvm/kvm (2012-10-04 09:30:33 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/sarah/xhci.git for-usb-linus-20

[PATCH 3/4] usb: Send Set SEL before enabling parent U1/U2 timeout.

The Set SEL control transfer tells a device the exit latencies associated with a device-initated U1 or U2 exit. Since a parent hub may initiate a transition to U1 soon after a downstream port's U1 timeout is set, we need to make sure the device receives the Set SEL transfer before the parent hub t

Re: REGRESSION: usbdevfs: Use-scatter-gather-lists-for-large-bulk-transfers

Hi Hans, > Oh what fun (not). The best way to figure out what really is going > on is to get some usb level traces. Note my first hunch is that what > you're seeing is a device firmware bug, as this patch together with > a new libusb (which you seem to also have) will make bulk transfers > run sli

Re: REGRESSION: usbdevfs: Use-scatter-gather-lists-for-large-bulk-transfers

Henrik Rydberg wrote: > Hi Hans, > > > Oh what fun (not). The best way to figure out what really is going > > on is to get some usb level traces. Note my first hunch is that what > > you're seeing is a device firmware bug, as this patch together with > > a new libusb (which you seem to also have)

Re: REGRESSION: usbdevfs: Use-scatter-gather-lists-for-large-bulk-transfers

On Thu, Oct 11, 2012 at 08:50:33AM +0200, Peter Stuge wrote: > Henrik Rydberg wrote: > > > What is the programming cable and software that uses it? > > > > The programmer is impact, using libusbx-1.0.14-1. > > Do you know for a fact that your version calls libusb-1.0? Did you > establish this wit

Re: REGRESSION: usbdevfs: Use-scatter-gather-lists-for-large-bulk-transfers

On Thu, Oct 11, 2012 at 11:37:07PM +0200, Henrik Rydberg wrote: > Hi Hans, > > > Oh what fun (not). The best way to figure out what really is going > > on is to get some usb level traces. Note my first hunch is that what > > you're seeing is a device firmware bug, as this patch together with > > a

Re: [Pull Request] Link PM patches for 3.7

On Thu, Oct 11, 2012 at 02:28:06PM -0700, Sarah Sharp wrote: > The following changes since commit ecefbd94b834fa32559d854646d777c56749ef1c: > > Merge tag 'kvm-3.7-1' of git://git.kernel.org/pub/scm/virt/kvm/kvm > (2012-10-04 09:30:33 -0700) > > are available in the git repository at: > > >

Re: [PATCH] usbhid: use GFP_NOIO in reset code path

On Thu, 11 Oct 2012, Ming Lei wrote: > Keeping allowed gfp_flag inside task_struct should be one solution, and > let that teach mm to allocate memory, see the draft idea below: Hmm, I actually like that idea. I guess you are going to propose this to a wider audience, right? So I am now putting

Re: locking, reset, memory allocations and deadlocks

On Wed, Oct 10, 2012 at 11:08:11AM -0400, Alan Stern wrote: > On Wed, 10 Oct 2012, Oliver Neukum wrote: > > > Hi, > > > > Ming Lei made me look. I found a scenario I don't like. > > Suppose we have a device with two interfaces #1 and #0 > > Let #1 be storage and #0 be vendor specific and the skel

Re: [PATCH] usbhid: use GFP_NOIO in reset code path

On Fri, Oct 12, 2012 at 5:55 AM, Jiri Kosina wrote: > On Thu, 11 Oct 2012, Ming Lei wrote: > >> Keeping allowed gfp_flag inside task_struct should be one solution, and >> let that teach mm to allocate memory, see the draft idea below: > > Hmm, I actually like that idea. > > I guess you are going t

Re: USB errors, ehci testing modes

Hi Alan Thanks for your reply! > > I have some stange errors on a arm pcm043 with internal usb phy. The log > > is attached at the end of the mail. It would be nice if someone could > > give me a pointer whats going wrong?. It would also be nice to know > > where the status flags which are output

Re: [PATCH] usbhid: use GFP_NOIO in reset code path

On Thu, Oct 11, 2012 at 12:00 AM, Ming Lei wrote: > > Keeping allowed gfp_flag inside task_struct should be one solution, and > let that teach mm to allocate memory, see the draft idea below: > > diff --git a/drivers/usb/storage/transport.c b/drivers/usb/storage/transport.c > index c0543c8..781447

Re: usb audio device can't work when it is attached to host via usb 2.0 hub

2012/10/11 Alan Stern : > On Thu, 11 Oct 2012, Elric Fu wrote: > >> Hi all, >> >> I recently found a strange issue on a embedded platform and don't know >> how to fix it. Please give me some suggestions. >> >> The platform is ar9331. The issue is if we attach a usb audio device to >> the host on ar

Re: usb audio device can't work when it is attached to host via usb 2.0 hub

2012/10/11 Peter Stuge : > Elric Fu wrote: >> It is a usb 1.1 device and works at full speed. > > It is a frequent misunderstanding that a device must be 1.1 only > because it does not support high speed. This is not the case. > > >> But the interesting thing is the bcdUSB field of device descripto

Re: [PATCH 01/12] usbnet: introduce usbnet 3 command helpers

On Thu, Oct 11, 2012 at 10:36 PM, Alan Stern wrote: > > It's worse than you may realize. When a SCSI disk is suspended, all of > its ancestor devices may be suspended too. Pages can't be read in from > the drive until all those ancestors are resumed. This means that all > runtime resume code p

[PATCH] usb: renesas_usbhs: fixup __usbhs_for_each_pipe 1st pos

1st pos of __usbhs_for_each_pipe() was wrong. Each pipe were pipe0, pipe2, pipe3 ... This patch modifies it. Signed-off-by: Kuninori Morimoto --- drivers/usb/renesas_usbhs/pipe.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/renesas_usbhs/pipe.h b/drivers/usb

Re: About resume time optimization for bus resume routine

On Thu, Oct 11, 2012 at 03:20:51PM -0400, Alan Stern wrote: > On Thu, 11 Oct 2012, Peter Chen wrote: > > > > > There is a port on the root hub with suspend status clear > > > and enabled status set (which implies there must be a device > > > attached to the port, because disconnected ports

[PATCH] usb: renesas_usbhs: gadget: add usb_gadget_ops :: pullup support

Signed-off-by: Kuninori Morimoto --- drivers/usb/renesas_usbhs/common.c |5 + drivers/usb/renesas_usbhs/common.h |1 + drivers/usb/renesas_usbhs/mod_gadget.c | 11 +++ 3 files changed, 17 insertions(+) diff --git a/drivers/usb/renesas_usbhs/common.c b/drivers/usb/r

[PATCH] usb: renesas_usbhs: fixup interrupt status clear method

When interrupt happened, renesas_usbhs driver gets irq status by usbhs_status_get_each_irq(), and cleared all status by using 0. But, this method is incorrect, since extra interrupt might occur between them. This patch cleared corresponding bits only Signed-off-by: Kuninori Morimoto --- drivers/