[PATCH v3] staging: usbip: userspace: add support for viewing imported devices

2013-12-24 Thread Valentina Manea
As of Matt Mooney's major refactoring in 2011, usbip port option was left out. Add support for this option in a manner similar to the old implementation. Sample output: Imported USB devices Port 00: at Full Speed(12Mbps) unknown vendor : unknown product (1687:6211)

[PATCH v5 1/1] usb: gadget: f_fs: Add support for SuperSpeed Mode

2013-12-24 Thread Manu Gautam
Allow userspace to pass SuperSpeed descriptors and handle them in the driver accordingly. This change doesn't modify existing desc_header and thereby keeps the ABI changes backward compatible i.e. existing userspace drivers compiled with old header (functionfs.h) would continue to work with the upd

Re: [PATCH v4 1/1] usb: gadget: f_fs: Add support for SuperSpeed Mode

2013-12-24 Thread Manu Gautam
On 12/24/2013 8:20 AM, Michal Nazarewicz wrote: > On Mon, Dec 23 2013, Manu Gautam wrote: >> Allow userspace to pass SuperSpeed descriptors and >> handle them in the driver accordingly. >> This change doesn't modify existing desc_header and thereby >> keeps the ABI changes backward compatible i.e.

RE: gadgetfs USB2.0 Chapter 9 Tests: Test after "Addressed State" fails

2013-12-24 Thread roshan.jhalani
>> Do you have a patch that does that? It would help in analysing the issue. >> Something quick and dirty will do just to see. Please refer patch below. I am not so good in gadgetFs, so just tried to hack this issue as per the following: Signed-off-by: Roshan Jhalani --- drivers/usb/gadget/c

Re: gadgetfs USB2.0 Chapter 9 Tests: Test after "Addressed State" fails

2013-12-24 Thread Michal Nazarewicz
On Tue, Dec 24 2013, roshan.jhal...@broadcom.com wrote: >> Do you have a patch that does that? It would help in analysing the >> issue. Something quick and dirty will do just to see. > Please refer patch below. I am not so good in gadgetFs, so just tried > to hack this issue as per the following

[PATCH -next] usb: phy-keystone: remove redundant return value check of platform_get_resource()

2013-12-24 Thread Wei Yongjun
From: Wei Yongjun Remove unneeded error handling on the result of a call to platform_get_resource() when the value is passed to devm_ioremap_resource(). Signed-off-by: Wei Yongjun --- drivers/usb/phy/phy-keystone.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/usb/phy/phy-key

RE: gadgetfs USB2.0 Chapter 9 Tests: Test after "Addressed State" fails

2013-12-24 Thread roshan.jhalani
>> This function is called whenever an ep file is released by user space (be it >> ep0 file or the other “regular” ep files). It is complementary to the >> ffs_data_opened function which in tandem track whether >> at least one >> epfile is open. >> The way it works is that if user space closes

RE: [PATCH] usb/core: fix NULL pointer dereference in recursively_mark_NOTATTACHED

2013-12-24 Thread Du, ChangbinX
> From: Alan Stern [mailto:st...@rowland.harvard.edu] > Sent: Monday, December 23, 2013 11:13 PM > To: Du, ChangbinX > Cc: gre...@linuxfoundation.org; sarah.a.sh...@linux.intel.com; Lan, Tianyu; > burzalod...@gmail.com; linux-usb@vger.kernel.org; > linux-ker...@vger.kernel.org > Subject: Re: [PATCH

Using both usbmon and ftrace?

2013-12-24 Thread Smilen Dimitrov
Hi all, Apologies for somewhat of a cross-post; I have already tried to ask about this here: Reliable Linux kernel timestamps (or adjustment thereof) with both usbmon and ftrace? http://stackoverflow.com/questions/20417897/reliable-linux-kernel-timestamps-or-adjustment-thereof-with-both-u

Re: gadgetfs USB2.0 Chapter 9 Tests: Test after "Addressed State" fails

2013-12-24 Thread Michal Nazarewicz
On Tue, Dec 24 2013, roshan.jhal...@broadcom.com wrote: > I agree with your point. But this doesn’t work when used this driver > in Android platform. Kernel driver sends USB_DISCONNECT event even in > BUS reset case and by that time USB disconnect is not expected from > FunctionFs driver. > > More

Re: [PATCH v3] staging: usbip: userspace: add support for viewing imported devices

2013-12-24 Thread Ilija Hadzic
Reviewed-by: Ilija Hadzic On Tue, 24 Dec 2013, Valentina Manea wrote: As of Matt Mooney's major refactoring in 2011, usbip port option was left out. Add support for this option in a manner similar to the old implementation. Sample output: Imported USB devices Port 00:

RE: gadgetfs USB2.0 Chapter 9 Tests: Test after "Addressed State" fails

2013-12-24 Thread roshan.jhalani
>>This is a bug in user space. FUNCTIONFS_DISABLE event means the host has >>chosen a configuration with given function disabled. User-space should clean >>up it's state, as any pending requests of any >> kind are now invalid. It >>should not close all of the ep files. Then, I would say this

Re: gadgetfs USB2.0 Chapter 9 Tests: Test after "Addressed State" fails

2013-12-24 Thread Michal Nazarewicz
>>This is a bug in user space. FUNCTIONFS_DISABLE event means the host >>has chosen a configuration with given function disabled. User-space >>should clean up it's state, as any pending requests of any >> kind are >>now invalid. It should not close all of the ep files. On Tue, Dec 24 2013, rosh

Re: [PATCH RFC 1/4] phy: Add new Exynos5 USB 3.0 PHY driver

2013-12-24 Thread Kishon Vijay Abraham I
Hi, On Thursday 05 December 2013 01:44 PM, Vivek Gautam wrote: Hi Kishon, On Wed, Dec 4, 2013 at 7:58 PM, Kishon Vijay Abraham I wrote: Hi Vivek, On Wednesday 20 November 2013 09:14 PM, Kishon Vijay Abraham I wrote: Hi, On Wednesday 20 November 2013 03:02 PM, Vivek Gautam wrote: On Wed,

RE: [PATCH] usb/core: fix NULL pointer dereference in recursively_mark_NOTATTACHED

2013-12-24 Thread Alan Stern
On Tue, 24 Dec 2013, Du, ChangbinX wrote: > > > usb_hub_to_struct_hub() can return NULL if the hub without active > > > configuration. So the result must be checked. > > > > > > BUG: unable to handle kernel NULL pointer dereference at 015c > > > How did you manage to trigger this BUG? If hub

[PATCH 0/2] Fixes for CY7C67x00 host controller

2013-12-24 Thread Max Filippov
Hello, there are two fixes for the CY7C67x00 host controller: one fixes memory allocation with non-atomic flags in the atomic context, the other adds proper delays to the register access methods which makes possible to use the chip with fast host CPU. Max Filippov (2): USB: c67x00: move URB pri

[PATCH 1/2] USB: c67x00: move URB private data allocation from under spinlock

2013-12-24 Thread Max Filippov
This fixes the following warning: BUG: sleeping function called from invalid context at mm/slub.c:940 in_atomic(): 1, irqs_disabled(): 1, pid: 17, name: khubd CPU: 0 PID: 17 Comm: khubd Not tainted 3.12.0-4-g938dd60-dirty #1 __might_sleep+0xbe/0xc0 kmem_cache_alloc_trace+0x36/0x17

[PATCH 2/2] USB: c67x00: add proper delays to HPI read/write

2013-12-24 Thread Max Filippov
According to CY7C67300 specification HPI read and write cycle duration Tcyc must be at least 6T long, where T is 1/48MHz, which is 125ns. Without this delay fast host processor cannot write to chip registers. Add proper ndelay to hpi_{read,write}_reg. Signed-off-by: Max Filippov --- drivers/usb/

Re: Using both usbmon and ftrace?

2013-12-24 Thread Alan Stern
On Tue, 24 Dec 2013, Smilen Dimitrov wrote: > Hi all, > > Apologies for somewhat of a cross-post; I have already tried to ask about > this here: > > Reliable Linux kernel timestamps (or adjustment thereof) with both usbmon > and ftrace? > > http://stackoverflow.com/questions/20417897/rel

[BUGREPORT] Linux USB 3.0

2013-12-24 Thread Markus Rechberger
A customer using a device with USBDEVFS is reporting following backtrace (it seems to be a rather generic issue related to linux usb 3.0 in general): According to him this problem is reproducible as soon as he starts the data transfer, is there anything known about that? He is using 3.12.0-031200-