[PATCH] USB: usb-skeleton.c: fix blocked forever in skel_read

2013-03-03 Thread duxing2007
From: Du Xing duxing2...@gmail.com In skel_read,the reader blocked in wait_for_completion before submit bulk in urb. Using processed_urb is totally unnecessary, remove it can fix the bug. --- Signed-off-by: Du Xing duxing2...@gmail.com --- diff --git a/drivers/usb/usb-skeleton.c

Re: Testing for hardware bug in EHCI controllers

2013-03-03 Thread Alan Stern
On Sat, 2 Mar 2013, Clemens Ladisch wrote: At any rate, if you're interested in finding out what the problem might be, the best place to start would be with a usbmon trace. ehci-test: 8801e96c06c0 2335244662 S Bo:1:018:1 -115 31 = 55534243 6500 0400 0a28

Re: [PATCH] drivers/usb/gadget: using strlcpy instead of strncpy

2013-03-03 Thread Laurent Pinchart
Hi Felipe, On Sunday 03 March 2013 01:23:46 Felipe Balbi wrote: On Fri, Mar 01, 2013 at 08:47:34PM +0100, Laurent Pinchart wrote: On Wednesday 27 February 2013 10:26:23 Felipe Balbi wrote: On Sat, Feb 02, 2013 at 03:48:54PM +0800, Chen Gang wrote: for NUL terminated string, better

Re: [PATCH 3/3] usb: Make sure each c67x00 TD has been executed

2013-03-03 Thread Peter Korsgaard
Dave == Dave Tubbs dave.tu...@portalislc.com writes: Dave From: Dave Tubbs dave.tu...@portalislc.com Dave Make sure each c67x00 TD has been executed or retry using the existing Dave retry mechanism. Reference Cypress Semiconductor BIOS User's Manual 1.2, Dave page 3-16 Dave Signed-off-by:

Re: [PATCH 0/5] usb: musb: am335x support

2013-03-03 Thread Daniel Mack
Hi Peter, On 03.03.2013 23:24, Peter Korsgaard wrote: Daniel == Daniel Mack zon...@gmail.com writes: Hi, Daniel On my board, the USB is purely used as host interface, with a Daniel type B plug soldered. In the DT, I'm using the following Daniel sniplet in accordance to the

RE: Not enough resource for old configuration after USB bus reset

2013-03-03 Thread Soar Hung
Hi, I try two experiment, 1. Test it on general desktop, the device will enum as CDC_ECM( 1 control, 2 bulk, 1 interrupt), and CDC_ECM driver will use interupt to polling link status. 2. Test it on embedded system, the device will controlled by a custom driver ( 1 control, 2 bulk, 1

Re: [PATCH v2 1/5] usb: chipidea: udc: move _ep_queue into an unlocked function

2013-03-03 Thread Peter Chen
On Fri, Mar 01, 2013 at 03:42:23PM +0100, Michael Grzeschik wrote: There is no need to call ep_queue unlocked inside the own driver. We move its functionionality into an unlocked version. This patch removes potential unlocked timeslots inside isr_setup_status_phase and

Re: [PATCH v2 2/5] usb: chipidea: udc: configure iso endpoints

2013-03-03 Thread Peter Chen
On Fri, Mar 01, 2013 at 03:42:24PM +0100, Michael Grzeschik wrote: The implementation is derived from the fsl_udc_core code in fsl_ep_enable and makes basic iso handling possible. Signed-off-by: Michael Grzeschik m.grzesc...@pengutronix.de Signed-off-by: Marc Kleine-Budde m...@pengutronix.de

Re: Testing for hardware bug in EHCI controllers

2013-03-03 Thread Ming Lei
On Tue, Feb 26, 2013 at 4:54 AM, Alan Stern st...@rowland.harvard.edu wrote: I'd be interested to hear the results of testing on a variety of controllers. (This computer also has an NEC EHCI controller, and that one does not have the bug.) Do the EHCI controllers on current Intel chipsets

Re: [PATCH 3/5] usb: chipidea: udc: add the define TD_COUNT and fix all users

2013-03-03 Thread Peter Chen
On Fri, Mar 01, 2013 at 03:42:25PM +0100, Michael Grzeschik wrote: A static count of transfer descriptors was used everywhere in the driver with the fixed number 4. This patch adds a define, named TD_COUNT, and replaces all users of this value. This way its possible to have only one parameter

Re: Testing for hardware bug in EHCI controllers

2013-03-03 Thread Bo Shen
Hi Alan, On 02/26/2013 04:54 AM, Alan Stern wrote: Sarah (and anyone else who's interested): A while ago I wrote about a hardware bug in my Intel ICH5 and ICH8 EHCI controllers. You pointed out that these are rather old components, not being used in current systems, which is quite true. I

RE: [PATCH] usb: host: tegra: Reset Tegra USB controller before init

2013-03-03 Thread Venu Byravarasu
-Original Message- From: Stephen Warren [mailto:swar...@wwwdotorg.org] Sent: Thursday, February 28, 2013 11:47 PM To: Venu Byravarasu Cc: gre...@linuxfoundation.org; st...@rowland.harvard.edu; linux- u...@vger.kernel.org; linux-ker...@vger.kernel.org Subject: Re: [PATCH] usb: host: