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 spec

[no subject]

2013-08-29 Thread gcb
My wife and I won the Euro Millions Lottery of 41 Million British Pounds and we have decided to donate 1.5 million British Pounds to 6 individuals worldwide as our own charity project. Your email address was among the emails which were submitted to us by the Google, Inc as a web user, which was

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

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

2013-08-29 Thread Alexander Shishkin
Peter Chen 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 look at 2/5 and queue 5/5 for 3.13 unless you have objections. Regards, -- Alex -- To unsubscri

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 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 >> wrote: >> Still noone answered me why "drivers/usb/host/xhci-ring.c" does NOT >> include (dev_info_ratelimited() and other d

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 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 samples per packet ident

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

2013-08-29 Thread Bjørn Mork
David Miller writes: > From: Rob Gardner > 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. So maybe we just have a nome

[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 --- drivers/usb/host/ehci-hub.c | 6 - dr

[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 --- drivers/usb/host/ehci-dbg.c | 8 drivers/usb/host/ehci-fsl.c | 2 +- dr

[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 --- drivers/usb/host/ehci-dbg.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/usb/host/ehci-dbg.c b/drivers/usb/host/ehci-

[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 --- drivers/usb/host/ehci-hcd.c | 8 1 fil

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

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" fo

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

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

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 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, and they will

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 US

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 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 look at 2/5

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 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 running, like i

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 -> extcon-d

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 fixe

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 n

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 the

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 contr

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

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

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 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 scheduled for the f

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

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 Ragia

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

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

2013-08-29 Thread Alan Stern
On Thu, 29 Aug 2013, Ming Lei wrote: > > In the end, this comes down to a decision about priorities. Do you > > want to give higher priority to new events or to givebacks? Overall, I > > don't think it matters. > > If possible, the events should be put higher priority, like events > handling in

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 lsp

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 co

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

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 read

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 m

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 exi

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

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

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 writes: > > > > > Changed for v2: > > > - Fixed the build error for patch 5/5 > > > > > > Below are un-queued chipidea patches, some of them were review

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 th

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 $i

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

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: https://git.kernel.org/cgit/linux/kernel/git/gregkh/usb

"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] u

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 exp

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 wrot

Re: 3.4.4: disabling irq

2013-08-29 Thread Udo van den Heuvel
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. > > Maybe without activity, the OHCI cont

[PATCH] 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 original

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

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 wrote: > On Thu, 29 Aug 2013, James Stone wrote: > >> On Wed, Aug 28, 2013 at 7:46 PM, Alan Stern >> wrote: >> > On Wed, 28

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 ha

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 se

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

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 f

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

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 w

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 09:16:56PM +0300, Xenia Ragiadakou wrote: > 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 d

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

2013-08-29 Thread Greg Kroah-Hartman
On Thu, Aug 29, 2013 at 11:18:21AM -0700, 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.9

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-usb&m=136269803207465&w=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 wro

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 11:22:46AM -0700, Sarah Sharp wrote: > 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 valu

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 210

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

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 er

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

2013-08-29 Thread Martin MOKREJŠ
Xenia Ragiadakou wrote: > 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 sho

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 us

[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 original

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

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 wan

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

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"; > +

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

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

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 wrote: Subject: Re: Bug 60810 - Kernel oops

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

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

2013-08-29 Thread Francois Romieu
- use all columns - the driver uses both 'netdev' and 'net'. Let's use 'netdev' ('net' is shorter but it tastes of network namespaces) - don't dereference dev->net when there is a local netdev at hand - use some existing #define (please check those) - skb_set_tail_pointer to avoid compiler cast w

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 +++ b/drivers/net/usb/s

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 i

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 0/2 v2] Various fixes for the dummy_hcd driver

2013-08-29 Thread Philippe De Swert
These two patches fix some minor issues in the dummy_hcd driver. Both errors were detected with the help of coverity analysis. These patches have been updated based on Alan Sterns comments. For the first patch I have harmonized both occurences of the different clean-up routine as suggested. The

[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 sugges

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 defau

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

[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 --- drivers/usb/phy/phy-tegra-usb.c |2 +- 1 files changed, 1 insertions

[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 --- drivers/usb/host/r8a66597-hcd.c |4 ++-- 1 files changed, 2 insertio

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

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)

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/30/2013 5:41 AM, Chanwoo Choi wrote: 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

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/30/2013 03:15 PM, George Cherian wrote: > Hi Chanwoo, > > On 8/30/2013 5:41 AM, Chanwoo Choi wrote: >> 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 deve