Re: [PATCH v3 1/3] extcon: extcon-gpio-usbvid: Generic USB VBUS/ID detection via GPIO

2013-08-29 Thread Chanwoo Choi
Hi George, On 08/29/2013 11:21 AM, George Cherian wrote: Hi Chanwoo, Thanks for the review and sorry for all the trivial mistakes. On 8/29/2013 7:05 AM, Chanwoo Choi wrote: Hi George, You didn't modify this patchset about my comment on v1 patchset. Please pay attention to comment. On

Re: [PATCH 2/4] usb: r8a66597-hcd: use platform_{get,set}_drvdata()

2013-08-29 Thread Greg KH
On Thu, Aug 29, 2013 at 10:54:51AM +0800, Libo Chen wrote: On 2013/8/28 12:36, Greg KH wrote: On Tue, Aug 27, 2013 at 04:10:22PM +0800, Libo Chen wrote: Use the wrapper functions for getting and setting the driver data using platform_device instead of using dev_{get,set}_drvdata() with

Re: [PATCH v3 1/3] extcon: extcon-gpio-usbvid: Generic USB VBUS/ID detection via GPIO

2013-08-29 Thread George Cherian
Hi Chanwoo, On 8/29/2013 11:53 AM, Chanwoo Choi wrote: [snip] You should keep following naming stlye. extcon-gpio-usbvid.c is wrong naming style. - extcon-[device name].c - extcon-gpio-usbvid.c - extcon-dra7xx.c or etc. Actually dra7xx is the SoC name and the USB VBUS/ID detection is not

Re: Bug 60810 - Kernel oops with controller XHCI while wait usb packet

2013-08-29 Thread Huang Rui
Hi, On Wed, Aug 28, 2013 at 09:46:42PM -0700, Giovanni wrote: https://bugzilla.kernel.org/show_bug.cgi?id=60810 Bug ID: 60810 Summary: Kernel oops with controller XHCI while wait usb packet Can you give us a detail kernel top commit and config which you built, in

Re: linux-next: Tree for Aug 28 [ xhci build breakage ]

2013-08-29 Thread Sedat Dilek
On Wed, Aug 28, 2013 at 11:15 PM, Sarah Sharp sarah.a.sh...@linux.intel.com wrote: On Wed, Aug 28, 2013 at 07:39:14PM +0200, Sedat Dilek wrote: On Wed, Aug 28, 2013 at 7:24 PM, Dmitry Kasatkin d.kasat...@samsung.com wrote: Still noone answered me why drivers/usb/host/xhci-ring.c does NOT

Re: [alsa-devel] Buffer size for ALSA USB PCM audio

2013-08-29 Thread James Stone
On Wed, Aug 28, 2013 at 7:46 PM, Alan Stern st...@rowland.harvard.edu wrote: On Wed, 28 Aug 2013, Clemens Ladisch wrote: Sorry, what I said applies more to explicit sync endpoints. When using implicit sync, a playback URB is submitted for each completed capture URB, with the number of

Re: [PATCH net,stable] net: usb: Add HP hs2434 device to ZLP exception table

2013-08-29 Thread Bjørn Mork
David Miller da...@davemloft.net writes: From: Rob Gardner robma...@gmail.com Date: Wed, 28 Aug 2013 18:40:22 -0600 The exception list means usb_device_id entries for specific devices that are known to need the workaround. There are just two such entries. There isn't even a separate list.

[PATCH 1/4] ehci: remove ehci_vdbg() verbose debugging statements

2013-08-29 Thread Xenia Ragiadakou
This patch removes ehci_vdbg debugging statements from EHCI host controller driver because they produce too much information, lowering the signal to noise ratio when debugging, and because they are not used anymore. Signed-off-by: Xenia Ragiadakou burzalod...@gmail.com ---

[PATCH 4/4] ehci: enable debugging code when CONFIG_DYNAMIC_DEBUG is set

2013-08-29 Thread Xenia Ragiadakou
The debugging code for ehci is enabled to run if the DEBUG flag is defined. This patch enables the debugging code also when the kernel is configured with dynamic debugging on. Signed-off-by: Xenia Ragiadakou burzalod...@gmail.com --- drivers/usb/host/ehci-dbg.c | 8

[PATCH 3/4] ehci: remove duplicate debug_async_open() prototype in ehci-dbg.c

2013-08-29 Thread Xenia Ragiadakou
This patch removes the duplicate of debug_async_open() prototype following three lines below the debug_async_open() declaration. Signed-off-by: Xenia Ragiadakou burzalod...@gmail.com --- drivers/usb/host/ehci-dbg.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/usb/host/ehci-dbg.c

[PATCH 2/4] ehci: remove debugging statement with ehci statistics in ehci_stop()

2013-08-29 Thread Xenia Ragiadakou
This patch removes the ehci statictics information output in ehci_stop() because they do not provide interesting info. At any case, the current statistics can be viewed by reading the 'registers' file in debugfs. Signed-off-by: Xenia Ragiadakou burzalod...@gmail.com ---

Re: [PATCH v2 1/3] usb: add is_usb_mouse routine and remote wakeup quirk

2013-08-29 Thread Huang Rui
On Thu, Aug 29, 2013 at 01:07:36AM +0800, Greg Kroah-Hartman wrote: On Wed, Aug 28, 2013 at 04:09:32PM +0800, Huang Rui wrote: On Wed, Aug 28, 2013 at 11:38:28AM +0800, Greg Kroah-Hartman wrote: On Wed, Aug 28, 2013 at 11:13:12AM +0800, Huang Rui wrote: Yes, you're right. This issue is

Re: [PATCH v2 1/3] usb: add is_usb_mouse routine and remote wakeup quirk

2013-08-29 Thread Huang Rui
On Thu, Aug 29, 2013 at 01:07:36AM +0800, Greg Kroah-Hartman wrote: On Wed, Aug 28, 2013 at 04:09:32PM +0800, Huang Rui wrote: On Wed, Aug 28, 2013 at 11:38:28AM +0800, Greg Kroah-Hartman wrote: On Wed, Aug 28, 2013 at 11:13:12AM +0800, Huang Rui wrote: Then just do this quirk for Pixart

Re: [PATCH v2 1/3] usb: add is_usb_mouse routine and remote wakeup quirk

2013-08-29 Thread Huang Rui
On Thu, Aug 29, 2013 at 02:59:50AM +0800, Alan Stern wrote: On Wed, 28 Aug 2013, Greg Kroah-Hartman wrote: On Wed, Aug 28, 2013 at 01:13:46PM -0400, Alan Stern wrote: On Wed, 28 Aug 2013, Greg Kroah-Hartman wrote: [Q] Why the special devices are only mice? Would high speed devices

Re: [PATCH v3 1/3] extcon: extcon-gpio-usbvid: Generic USB VBUS/ID detection via GPIO

2013-08-29 Thread Chanwoo Choi
On 08/29/2013 04:30 PM, George Cherian wrote: Hi Chanwoo, On 8/29/2013 11:53 AM, Chanwoo Choi wrote: [snip] You should keep following naming stlye. extcon-gpio-usbvid.c is wrong naming style. - extcon-[device name].c - extcon-gpio-usbvid.c - extcon-dra7xx.c or etc. Actually dra7xx is

Re: [PATCH v1] USB: EHCI: disable IAA_WATCHDOG and START_UNLINK_INTR if they needn't to be handled

2013-08-29 Thread Ming Lei
On Thu, Aug 29, 2013 at 12:05 AM, Alan Stern st...@rowland.harvard.edu wrote: On Wed, 28 Aug 2013, Ming Lei wrote: Think about it this way: Why did you write the USB: EHCI: improve interrupt qh unlink patch in the first place? Essentially the same reason applies to uhci-hcd and ohci-hcd,

Re: [RFC v2] usbcore: compare and release one bos descriptor in usb_reset_and_verify_device()

2013-08-29 Thread Martin MOKREJŠ
Hi Xenia, thank you for the patch. I tried to reproduce the error with patched 3.10.9 kernel but it seems the kmemleak is indeed gone. Provided I get only these lines logged which used to be followed by kmemleak findings I believe the original fixed: [15885.206032] usb 4-2.1: reset SuperSpeed

Re: [PATCH v2 0/5] Chipidea Misc patchset

2013-08-29 Thread Peter Chen
On Thu, Aug 29, 2013 at 10:53:50AM +0300, Alexander Shishkin wrote: Peter Chen peter.c...@freescale.com writes: Changed for v2: - Fixed the build error for patch 5/5 Below are un-queued chipidea patches, some of them were reviewed. I'd like to send 1/5, 3/5 and 4/5 now, have a closer

Re: [RFC 2/3] EHCI: convert the IRQ handler to a tasklet

2013-08-29 Thread Ming Lei
On Tue, Aug 27, 2013 at 10:39 PM, Alan Stern st...@rowland.harvard.edu wrote: On Tue, 27 Aug 2013, Ming Lei wrote: Yes. A new spinlock would be needed to synchronize the top half and the bottom half. The same spinlock would also be used to avoid scheduling the tasklet when it is already

Re: [PATCH v3 1/3] extcon: extcon-gpio-usbvid: Generic USB VBUS/ID detection via GPIO

2013-08-29 Thread George Cherian
On 8/29/2013 4:07 PM, Chanwoo Choi wrote: On 08/29/2013 04:30 PM, George Cherian wrote: Hi Chanwoo, On 8/29/2013 11:53 AM, Chanwoo Choi wrote: [snip] You should keep following naming stlye. extcon-gpio-usbvid.c is wrong naming style. - extcon-[device name].c - extcon-gpio-usbvid.c -

Re: [RFC v2] usbcore: compare and release one bos descriptor in usb_reset_and_verify_device()

2013-08-29 Thread Xenia Ragiadakou
On 08/29/2013 02:21 PM, Martin MOKREJŠ wrote: Hi Xenia, thank you for the patch. I tried to reproduce the error with patched 3.10.9 kernel but it seems the kmemleak is indeed gone. Provided I get only these lines logged which used to be followed by kmemleak findings I believe the original

Re: [PATCH v3 1/3] extcon: extcon-gpio-usbvid: Generic USB VBUS/ID detection via GPIO

2013-08-29 Thread Chanwoo Choi
On 08/29/2013 08:48 PM, George Cherian wrote: On 8/29/2013 4:07 PM, Chanwoo Choi wrote: On 08/29/2013 04:30 PM, George Cherian wrote: Hi Chanwoo, On 8/29/2013 11:53 AM, Chanwoo Choi wrote: [snip] You should keep following naming stlye. extcon-gpio-usbvid.c is wrong naming style. -

Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Martin MOKREJŠ
Hi Xenia, thank you, how about inclusion of the parent hub number in the following message (as of now): Parent hub missing LPM exit latency info. Power management will be impacted. I find it awkward to later on run manually lspci/lsusb to find what is the parent. I think I do NOT get

Re: Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Martin MOKREJŠ
Actually, there is some new bug I haven't seen before (this is 3.10.9 kernel). First of all, I see my TI XHCI controller does not use MSI-X anymore, will have to check my .config why is it so. Second, it should have IRQ 45 and 46 according to dmesg. But lspci reports IRQ 16 is used by TI XHCI

Re: [PATCH v3 1/3] extcon: extcon-gpio-usbvid: Generic USB VBUS/ID detection via GPIO

2013-08-29 Thread George Cherian
Hi Chanwoo, On 8/29/2013 5:42 PM, Chanwoo Choi wrote: [big snip ] I tested various development board based on Samsung Exynos series SoC. Although some gpio of Exynos series SoC set high state(non zero, 1) as default value, this gpio state could mean off state, disconnected or un-powered state

Re: Memory synchronization vs. interrupt handlers

2013-08-29 Thread Alan Stern
On Wed, 28 Aug 2013, H. Peter Anvin wrote: On 08/28/2013 12:16 PM, Alan Stern wrote: Russell, Peter, and Ingo: Can you folks enlighten us regarding this issue for some common architectures? On x86, IRET is a serializing instruction; it guarantees hard serialization of absolutely

USB-HID

2013-08-29 Thread kasi viswanathan
Hi, I have a custom device, which is usb-hid. But, when i plug it in the system, it is not recognized as a usb-hid device. I have put prints on the usbhid_probe() function, to confirm that, but there is no prints in the dmesg, which confirms that this is not recognized as a usb-hid device.

Re: [RFC 3/3] EHCI: handle late isochronous submissions

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, Ming Lei wrote: On Wed, 28 Aug 2013 12:23:31 -0400 (EDT) Alan Stern st...@rowland.harvard.edu wrote: You must never alter ehci-last_iso_frame like this. It violates the driver's invariants for time to run backward. After all, there may already be other TDs

Re: [alsa-devel] Buffer size for ALSA USB PCM audio

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, James Stone wrote: On Wed, Aug 28, 2013 at 7:46 PM, Alan Stern st...@rowland.harvard.edu wrote: On Wed, 28 Aug 2013, Clemens Ladisch wrote: Sorry, what I said applies more to explicit sync endpoints. When using implicit sync, a playback URB is submitted for each

Re: [PATCH 1/4] ehci: remove ehci_vdbg() verbose debugging statements

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, Xenia Ragiadakou wrote: This patch removes ehci_vdbg debugging statements from EHCI host controller driver because they produce too much information, lowering the signal to noise ratio when debugging, and because they are not used anymore. Signed-off-by: Xenia

Re: [PATCH v1] USB: EHCI: disable IAA_WATCHDOG and START_UNLINK_INTR if they needn't to be handled

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, Ming Lei wrote: You have said the same problem exists on these drivers already without giveback in tasklet patch, so for the problem and solution, there is no difference. What do you mean? We can't fix those other drivers by changing ehci-hcd. It's their fault

Re: Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, Martin MOKREJŠ wrote: Hi Xenia, thank you, how about inclusion of the parent hub number in the following message (as of now): Parent hub missing LPM exit latency info. Power management will be impacted. I find it awkward to later on run manually lspci/lsusb

Re: USB-HID

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, kasi viswanathan wrote: Hi, I have a custom device, which is usb-hid. But, when i plug it in the system, it is not recognized as a usb-hid device. I have put prints on the usbhid_probe() function, to confirm that, but there is no prints in the dmesg, which confirms

RE: stv090x vs stv0900 support

2013-08-29 Thread Krishna Kishore
Hi, Can someone help me, please? Regards, Kishore. -Original Message- From: Krishna Kishore Sent: Wednesday, August 28, 2013 6:05 PM To: Chris Lee; Mariusz Bialonczyk Cc: Linux Media Mailing List; linux-usb@vger.kernel.org Subject: RE: stv090x vs stv0900 support Hi, When

Re: Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, Martin MOKREJŠ wrote: Alan Stern wrote: On Thu, 29 Aug 2013, Martin MOKREJŠ wrote: Hi Xenia, thank you, how about inclusion of the parent hub number in the following message (as of now): Parent hub missing LPM exit latency info. Power management will be

Re: Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Martin MOKREJŠ
Alan Stern wrote: On Thu, 29 Aug 2013, Martin MOKREJŠ wrote: Alan Stern wrote: On Thu, 29 Aug 2013, Martin MOKREJŠ wrote: Hi Xenia, thank you, how about inclusion of the parent hub number in the following message (as of now): Parent hub missing LPM exit latency info. Power

Re: [Bug] [Sony VAIO SVT15117CXS] USB 2.0 ports not working with any USB device

2013-08-29 Thread Alan Stern
On Wed, 28 Aug 2013, Kevin Archer wrote: unbind yielded nothing using 3.11.0-031100rc5-generic lsusb -v -s 1:2 Bus 001 Device 002: ID 8087:0024 Intel Corp. Integrated Rate Matching Hub ... Hub Port Status: Port 1: .0103 power enable connect Port 2: .0100 power Port

Re: [PATCH v2 0/5] Chipidea Misc patchset

2013-08-29 Thread Greg KH
On Thu, Aug 29, 2013 at 05:25:55PM +0800, Peter Chen wrote: On Thu, Aug 29, 2013 at 10:53:50AM +0300, Alexander Shishkin wrote: Peter Chen peter.c...@freescale.com writes: Changed for v2: - Fixed the build error for patch 5/5 Below are un-queued chipidea patches, some of them

Re: stv090x vs stv0900 support

2013-08-29 Thread Greg KH
On Thu, Aug 29, 2013 at 03:20:32PM +, Krishna Kishore wrote: Hi, Can someone help me, please? Personally, I'm not allowed to do so because of: SASKEN BUSINESS DISCLAIMER: This message may contain confidential, proprietary or legally privileged information. In case you are not the

Re: Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, Martin MOKREJŠ wrote: Since this device is 4-2, the parent hub is usb4. Actually, even if that would be a another USB HUB and not a PCI device (root hub), I would be happy if it extracted something like: Bus 004 Device 006: ID 2109:0810 $iManufacturer and $iProduct

Re: Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Xenia Ragiadakou
On 08/29/2013 04:31 PM, Martin MOKREJŠ wrote: Actually, there is some new bug I haven't seen before (this is 3.10.9 kernel). First of all, I see my TI XHCI controller does not use MSI-X anymore, will have to check my .config why is it so. Second, it should have IRQ 45 and 46 according to dmesg.

Re: linux-next: Tree for Aug 28 [ xhci build breakage ]

2013-08-29 Thread Sarah Sharp
On Thu, Aug 29, 2013 at 10:02:13AM +0200, Sedat Dilek wrote: P.S.: The forgotten patch is now in usb-next, but I don't see any credits, coins, gold, platin... Thank you for reporting this. Your name was mentioned in the tag Greg pulled:

Set SEL for device-initiated U1 failed. errors

2013-08-29 Thread Greg Kroah-Hartman
Hi Sarah, I'm getting the following warnings from the 3.10.9 kernel all the time when I unplug a USB 3 storage device from my laptop: [203282.987687] usb 4-1: USB disconnect, device number 21 [203282.992904] usb 4-1: Set SEL for device-initiated U1 failed. [203282.992909]

Re: [PATCH] usb: xhci-plat: Enable USB 2.0 hardware LPM support for platform xHCs

2013-08-29 Thread Sarah Sharp
On Wed, Aug 28, 2013 at 04:08:12PM -0700, Julius Werner wrote: So the 2.41a has BESL support, but may not set the BLC flag. What happens if we use the HIRD encoding instead? Will things break? It seems like we would need to disable USB 2.0 LPM on that host all together, if it expects

Re: [PATCH v4 2/3] usb: phy: Add Qualcomm SS-USB and HS-USB drivers for DWC3 core

2013-08-29 Thread Ivan T. Ivanov
On Thu, 2013-08-22 at 15:41 -0500, Felipe Balbi wrote: On Wed, Aug 21, 2013 at 10:13:28AM -0500, Kumar Gala wrote: On Aug 21, 2013, at 8:06 AM, Ivan T. Ivanov wrote: On Tue, 2013-08-20 at 18:01 +0100, Pawel Moll wrote: On Tue, 2013-08-20 at 16:06 +0100, Pawel Moll wrote: On Tue,

Re: [PATCH] usb: xhci-plat: Enable USB 2.0 hardware LPM support for platform xHCs

2013-08-29 Thread Sarah Sharp
On Wed, Aug 28, 2013 at 10:15:34PM +, Paul Zimmerman wrote: From: Sarah Sharp [mailto:sarah.a.sh...@linux.intel.com] Sent: Wednesday, August 28, 2013 2:52 PM On Mon, Aug 26, 2013 at 07:37:56PM +, Paul Zimmerman wrote: From: Sarah Sharp [mailto:sarah.a.sh...@linux.intel.com]

Re: [alsa-devel] Buffer size for ALSA USB PCM audio

2013-08-29 Thread James Stone
At 32 frames/period (reported round-trip latency 1.33ms), it starts up but there are too many xruns for it to be usable. James On Thu, Aug 29, 2013 at 4:00 PM, Alan Stern st...@rowland.harvard.edu wrote: On Thu, 29 Aug 2013, James Stone wrote: On Wed, Aug 28, 2013 at 7:46 PM, Alan Stern

Re: [PATCH] usbcore: compare and release one bos descriptor in usb_reset_and_verify_device()

2013-08-29 Thread Greg KH
On Thu, Aug 29, 2013 at 08:37:57PM +0300, Xenia Ragiadakou wrote: In usb_reset_and_verify_device(), hub_port_init() allocates a new bos descriptor to hold the value read by the device. The new bos descriptor has to be compared with the old one in order to figure out if device 's firmware has

Re: [PATCH] usb: xhci-plat: Enable USB 2.0 hardware LPM support for platform xHCs

2013-08-29 Thread Julius Werner
If you take a look at Table 13: BESL/HIRD Encoding from the xHCI spec version including errata to 08/14/2012 Could you please provide a link to that errata? I still cannot find it... but from your explanation, that design decision sounds pretty horrible. Why didn't they just choose not to set

RE: [PATCH] usb: xhci-plat: Enable USB 2.0 hardware LPM support for platform xHCs

2013-08-29 Thread Paul Zimmerman
From: Sarah Sharp [mailto:sarah.a.sh...@linux.intel.com] Sent: Thursday, August 29, 2013 10:42 AM On Wed, Aug 28, 2013 at 10:15:34PM +, Paul Zimmerman wrote: From: Sarah Sharp [mailto:sarah.a.sh...@linux.intel.com] Sent: Wednesday, August 28, 2013 2:52 PM On Mon, Aug 26, 2013

Re: [PATCH] usbcore: compare and release one bos descriptor in usb_reset_and_verify_device()

2013-08-29 Thread Xenia Ragiadakou
On 08/29/2013 08:53 PM, Greg KH wrote: On Thu, Aug 29, 2013 at 08:37:57PM +0300, Xenia Ragiadakou wrote: In usb_reset_and_verify_device(), hub_port_init() allocates a new bos descriptor to hold the value read by the device. The new bos descriptor has to be compared with the old one in order to

Re: Set SEL for device-initiated U1 failed. errors

2013-08-29 Thread Sarah Sharp
On Thu, Aug 29, 2013 at 10:06:16AM -0700, Greg Kroah-Hartman wrote: Hi Sarah, I'm getting the following warnings from the 3.10.9 kernel all the time when I unplug a USB 3 storage device from my laptop: [203282.987687] usb 4-1: USB disconnect, device number 21 [203282.992904] usb

Re: [PATCH] usbcore: compare and release one bos descriptor in usb_reset_and_verify_device()

2013-08-29 Thread Sarah Sharp
On Thu, Aug 29, 2013 at 10:53:13AM -0700, Greg KH wrote: On Thu, Aug 29, 2013 at 08:37:57PM +0300, Xenia Ragiadakou wrote: In usb_reset_and_verify_device(), hub_port_init() allocates a new bos descriptor to hold the value read by the device. The new bos descriptor has to be compared with

Re: Bug 60810 - Kernel oops with controller XHCI while wait usb packet

2013-08-29 Thread Sarah Sharp
Hi Giovanni, Mathias mentioned that this patch might fix your issue: http://marc.info/?l=linux-usbm=136269803207465w=2 Can you please compile a custom 3.10 stable kernel with that patch applied and see if it fixes your issue? Sarah Sharp On Wed, Aug 28, 2013 at 09:46:42PM -0700, Giovanni

Re: Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Martin MOKREJŠ
Alan Stern wrote: On Thu, 29 Aug 2013, Martin MOKREJŠ wrote: Since this device is 4-2, the parent hub is usb4. Actually, even if that would be a another USB HUB and not a PCI device (root hub), I would be happy if it extracted something like: Bus 004 Device 006: ID 2109:0810

Re: 3.4.4: disabling irq

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, Udo van den Heuvel wrote: On 2013-08-28 21:37, Alan Stern wrote: No, if you unload the ohci-hcd driver then the webcam won't be used. Are you certain that merely stopping the daemon program will prevent the problem? Quite certain but retesting can confirm that.

RE: [PATCH] usb: xhci-plat: Enable USB 2.0 hardware LPM support for platform xHCs

2013-08-29 Thread Paul Zimmerman
From: jwer...@google.com [mailto:jwer...@google.com] On Behalf Of Julius Werner Sent: Thursday, August 29, 2013 11:07 AM If you take a look at Table 13: BESL/HIRD Encoding from the xHCI spec version including errata to 08/14/2012 Could you please provide a link to that errata? I

Re: [alsa-devel] Buffer size for ALSA USB PCM audio

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, James Stone wrote: At 32 frames/period (reported round-trip latency 1.33ms), it starts up but there are too many xruns for it to be usable. Interesting. Can you try doing the same thing with just playback, no recording? If that still gets underruns, please collect a

[PATCH v2] usbcore: compare and release one bos descriptor in usb_reset_and_verify_device()

2013-08-29 Thread Xenia Ragiadakou
In usb_reset_and_verify_device(), hub_port_init() allocates a new bos descriptor to hold the value read by the device. The new bos descriptor has to be compared with the old one in order to figure out if device 's firmware has changed in which case the device has to be reenumerated. In the

Re: Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Greg KH
On Thu, Aug 29, 2013 at 08:36:51PM +0200, Martin MOKREJŠ wrote: Alan Stern wrote: On Thu, 29 Aug 2013, Martin MOKREJŠ wrote: Since this device is 4-2, the parent hub is usb4. Actually, even if that would be a another USB HUB and not a PCI device (root hub), I would be happy if

Re: Set SEL for device-initiated U1 failed. errors

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, Sarah Sharp wrote: On Thu, Aug 29, 2013 at 10:06:16AM -0700, Greg Kroah-Hartman wrote: Hi Sarah, I'm getting the following warnings from the 3.10.9 kernel all the time when I unplug a USB 3 storage device from my laptop: [203282.987687] usb 4-1: USB

Re: Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, Martin MOKREJŠ wrote: This really is asking too much of the kernel. That's why we have userspace utilities like lsusb. For example, it wouldn't be hard to write a shell script that would take a device name like 4-2 and print out the information you want. The

Re: [PATCH v3 1/3] extcon: extcon-gpio-usbvid: Generic USB VBUS/ID detection via GPIO

2013-08-29 Thread Stephen Warren
On 08/29/2013 05:48 AM, George Cherian wrote: On 8/29/2013 4:07 PM, Chanwoo Choi wrote: ... I tested various development board based on Samsung Exynos series SoC. Although some gpio of Exynos series SoC set high state(non zero, 1) as default value, this gpio state could mean off state,

Re: [PATCH v3 1/3] extcon: extcon-gpio-usbvid: Generic USB VBUS/ID detection via GPIO

2013-08-29 Thread Stephen Warren
On 08/28/2013 11:33 AM, George Cherian wrote: Add a generic USB VBUS/ID detection EXTCON driver. This driver expects the ID/VBUS pin are connected via GPIOs. This driver is tested on DRA7x board were the ID pin is routed via GPIOs. The driver supports both VBUS and ID pin configuration and ID

Re: [PATCH v3 3/3] ARM: dts: dra7-evm: Add extcon nodes for USB ID pin detection

2013-08-29 Thread Stephen Warren
On 08/28/2013 11:33 AM, George Cherian wrote: Add -extcon nodes for USB ID pin detection. -i2c nodes. -pcf nodes to which USB ID pin is connected. diff --git a/arch/arm/boot/dts/dra7-evm.dts b/arch/arm/boot/dts/dra7-evm.dts dwc3_1 { - dr_mode = otg; + dr_mode =

Re: Set SEL for device-initiated U1 failed. errors

2013-08-29 Thread Sarah Sharp
On Thu, Aug 29, 2013 at 02:58:25PM -0400, Alan Stern wrote: On Thu, 29 Aug 2013, Sarah Sharp wrote: On Thu, Aug 29, 2013 at 10:06:16AM -0700, Greg Kroah-Hartman wrote: Hi Sarah, I'm getting the following warnings from the 3.10.9 kernel all the time when I unplug a USB 3 storage

Re: Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, Martin MOKREJŠ wrote: This really isn't necessary. All the information you want is already in the system log. It's merely a matter of locating it. I can show you how. If you post a complete dmesg log, starting from boot and running up to one of these errors,

Re: Bug 60810 - Kernel oops with controller XHCI while wait usb packet

2013-08-29 Thread Giovanni
Hi Sarah, thank you for your patch, but not fix my issue. I attached new kernel oops log into bugzilla. https://bugzilla.kernel.org/attachment.cgi?id=107363 Regards, Giovanni On Thu, 8/29/13, Sarah Sharp sarah.a.sh...@linux.intel.com wrote:

Re: Include parent hub number in current warning message Parent hub missing LPM exit latency info

2013-08-29 Thread Xenia Ragiadakou
On 08/29/2013 11:32 PM, Alan Stern wrote: On Thu, 29 Aug 2013, Martin MOKREJŠ wrote: [snip] I understand your point but don't think this is the case. OK, please show me why is LPM not available. If USB is complaining about that being disabled it could tell me based on what attribute at least.

Re: [PATCH] USB2NET : SR9700 : One Chip USB 1.1 USB2NET SR9700 Device Driver Support

2013-08-29 Thread Francois Romieu
Keep it small though literate. --- drivers/net/usb/sr9700.c | 30 -- 1 file changed, 12 insertions(+), 18 deletions(-) diff --git a/drivers/net/usb/sr9700.c b/drivers/net/usb/sr9700.c index f7f46e6..3f05b35 100644 --- a/drivers/net/usb/sr9700.c +++

Re: [PATCH] USB2NET : SR9700 : One Chip USB 1.1 USB2NET SR9700 Device Driver Support

2013-08-29 Thread Francois Romieu
Liu, please check those. --- drivers/net/usb/sr9700.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/net/usb/sr9700.c b/drivers/net/usb/sr9700.c index 3f05b35..3ae3caf 100644 --- a/drivers/net/usb/sr9700.c +++ b/drivers/net/usb/sr9700.c @@ -99,8 +99,9 @@ static

Re: Memory synchronization vs. interrupt handlers

2013-08-29 Thread Paul E. McKenney
On Wed, Aug 28, 2013 at 01:28:08PM -0700, H. Peter Anvin wrote: On 08/28/2013 12:16 PM, Alan Stern wrote: Russell, Peter, and Ingo: Can you folks enlighten us regarding this issue for some common architectures? On x86, IRET is a serializing instruction; it guarantees hard

[PATCH 1/2 v2] usb:gadget:dummy_hcd : Avoid infinite loop

2013-08-29 Thread Philippe De Swert
When an error occurs adding a udc platform device there is a risk of an infinite loop. If more than one platform device was added i will remain = than 0. The intention seems to clean up all the different already added platform devices before the failure occurs, so fixed the code to actually do

[PATCH 2/2 v2] usb:gadget:dummy_hcd : Detect port and link state correctly to avoid unreachable code

2013-08-29 Thread Philippe De Swert
Since USB_SS_PORT_LS_U0 is 0x the operation with the port state would always be 0. Thus the if would never be true. Moreover USB_PORT_STAT_ENABLE is 0x0002 and as such would never equal to 1. What we actually look for is a port that is in U0/link active state. Thanks to Alan Stern for

Re: [PATCH v3 1/3] extcon: extcon-gpio-usbvid: Generic USB VBUS/ID detection via GPIO

2013-08-29 Thread Chanwoo Choi
Hi George, On 08/29/2013 10:45 PM, George Cherian wrote: Hi Chanwoo, On 8/29/2013 5:42 PM, Chanwoo Choi wrote: [big snip ] I tested various development board based on Samsung Exynos series SoC. Although some gpio of Exynos series SoC set high state(non zero, 1) as default value, this

Re: Memory synchronization vs. interrupt handlers

2013-08-29 Thread Benjamin Herrenschmidt
On Thu, 2013-08-29 at 16:51 -0700, Paul E. McKenney wrote: On Wed, Aug 28, 2013 at 01:28:08PM -0700, H. Peter Anvin wrote: On 08/28/2013 12:16 PM, Alan Stern wrote: Russell, Peter, and Ingo: Can you folks enlighten us regarding this issue for some common architectures? On

[PATCH RESEND 3/4] usb: phy-tegra-usb: use platform_{get,set}_drvdata()

2013-08-29 Thread Libo Chen
Use the wrapper functions for getting and setting the driver data using platform_device instead of using dev_{get,set}_drvdata() with of-dev, so we can directly pass a struct platform_device. Signed-off-by: Libo Chen libo.c...@huawei.com --- drivers/usb/phy/phy-tegra-usb.c |2 +- 1 files

[PATCH RESEND 2/4] usb: r8a66597-hcd: use platform_{get,set}_drvdata()

2013-08-29 Thread Libo Chen
Use the wrapper functions for getting and setting the driver data using platform_device instead of using dev_{get,set}_drvdata() with of-dev, so we can directly pass a struct platform_device. Signed-off-by: Libo Chen libo.c...@huawei.com --- drivers/usb/host/r8a66597-hcd.c |4 ++-- 1 files

Re: Memory synchronization vs. interrupt handlers

2013-08-29 Thread H. Peter Anvin
On 08/29/2013 04:51 PM, Paul E. McKenney wrote: On Wed, Aug 28, 2013 at 01:28:08PM -0700, H. Peter Anvin wrote: On 08/28/2013 12:16 PM, Alan Stern wrote: Russell, Peter, and Ingo: Can you folks enlighten us regarding this issue for some common architectures? On x86, IRET is a serializing

Re: [PATCH] USB2NET : SR9700 : One Chip USB 1.1 USB2NET SR9700 Device Driver Support

2013-08-29 Thread Joe Perches
On Fri, 2013-08-30 at 01:06 +0200, Francois Romieu wrote: just trivia: diff --git a/drivers/net/usb/sr9700.c b/drivers/net/usb/sr9700.c [] @@ -288,19 +291,19 @@ static void sr9700_set_multicast(struct net_device *net) [] +static int sr9700_set_mac_address(struct net_device *netdev, void *p)