Re: [RESEND PATCH v5 4/7] usb: chipidea: consolidate ci_role_driver's API for both roles

2013-01-27 Thread Peter Chen
On Fri, Jan 25, 2013 at 02:12:20PM +0200, Alexander Shishkin wrote: > Peter Chen writes: > > > On Thu, Jan 24, 2013 at 04:35:30PM +0200, Alexander Shishkin wrote: > >> Peter Chen writes: > >> > >> > - Create init/destroy API for probe and remove > >> > - start/stop API are only used otg id swit

Re: [RESEND PATCH v5 3/7] usb: chipidea: add otg id switch and vbus connect/disconnect detect

2013-01-27 Thread Peter Chen
On Fri, Jan 25, 2013 at 11:40:10AM +0200, Alexander Shishkin wrote: > Peter Chen writes: > > > On Thu, Jan 24, 2013 at 05:25:17PM +0200, Alexander Shishkin wrote: > > What doesn't happen is the reset into device mode (unless you have > _REGS_SHARED set, which by the way seems a bit misleading) a

[PATCH] chipidea: ci13xxx_imx: Remove sparse warning

2013-01-27 Thread Fabio Estevam
From: Fabio Estevam Remove the following sparse warning: drivers/usb/chipidea/ci13xxx_imx.h:22:25: error: dubious one-bit signed bitfield drivers/usb/chipidea/ci13xxx_imx.h:22:25: error: dubious one-bit signed bitfield Signed-off-by: Fabio Estevam --- drivers/usb/chipidea/ci13xxx_imx.h |2

Re: ehci with full speed audio device leaking sitds

2013-01-27 Thread Alan Stern
On Sun, 27 Jan 2013, Andy Leiserson wrote: > Please CC me on replies, as I'm not on the list. > > My system is an AR9132 mips SoC, kernel 3.6.11 with OpenWrt patches. > The audio device is a C-Media usb audio adapter, IDs 0d8c/000e. It is a > full speed device, and is connected directly to the on

Re: [PATCH] net: cdc_ncm: use IAD provided by the USB core

2013-01-27 Thread David Miller
From: Bjørn Mork Date: Sat, 26 Jan 2013 10:36:59 +0100 > commit 9992c2e (net: cdc_ncm: workaround for missing CDC Union) > added code to lookup an IAD for the interface we are probing. > This is redundant. The USB core has already done the lookup > and saved the result in the USB interface struc

Re: [PATCH 1/3] usb: otg: nop: add dt support

2013-01-27 Thread Michael Grzeschik
Hi, On Thu, Sep 27, 2012 at 11:15:05AM +0530, Afzal Mohammed wrote: > From: "B, Ravi" > > Added device tree support for nop transceiver driver and updated the > Documentation with device tree binding information for am33xx platform. > > Signed-off-by: Ravi Babu > Signed-off-by: Ajay Kumar Gupt

Re: Problem with loopback configuration in testusb.

2013-01-27 Thread Sebastian Andrzej Siewior
On Fri, Jan 25, 2013 at 03:52:43PM +0100, humanshu bhatia wrote: > Hello, Hi, > I am new to linux device driver I am performing usb test on Netchip PLX > 3380 card by using testusb. 3380 is currently out of tree. > All test are running fine so far with sudo ./testusb -D > /proc/bus/usb/001/010 -a

Re: UAS questions

2013-01-27 Thread Felipe Balbi
Hi, On Sun, Jan 27, 2013 at 07:21:01PM +0100, Sebastian Andrzej Siewior wrote: > >> commands support by UAS device mandatory? > > > > Can't answer this. Sebastian, do you know if those are mandatory ? > > I would say so. The specification does not explicit say this but it > also does not say tha

Re: [PATCH v1] usb: musb: Make dma_controller_create __devinit

2013-01-27 Thread Sergei Shtylyov
Hello. On 09-08-2012 18:38, Shubhrajyoti D wrote: dma_controller_create is called only from musb_init_controller which is __devint so annotate dma_controller_create also with __devint. fixes the warn WARNING: vmlinux.o(.devinit.text+0x6fa8): Section mismatch in reference from the functio

Re: [PATCH 2/2]linux-usb:optimize to match the Huawei USB storage devices and support new switch command

2013-01-27 Thread Sebastian Andrzej Siewior
On Fri, Jan 25, 2013 at 10:46:13AM +0800, fangxiaozhi 00110321 wrote: > diff -uprN linux-3.8-rc4_orig/drivers/usb/storage/initializers.c > linux-3.8-rc4/drivers/usb/storage/initializers.c > --- linux-3.8-rc4_orig/drivers/usb/storage/initializers.c 2013-01-22 > 14:12:42.594238726 +0800 > +++ linux

Re: [PATCH 1/2]linux-usb:Define a new macro for USB storage match rules

2013-01-27 Thread Sebastian Andrzej Siewior
On Fri, Jan 25, 2013 at 10:44:50AM +0800, fangxiaozhi 00110321 wrote: > diff -uprN linux-3.8-rc4_orig/drivers/usb/storage/usual-tables.c > linux-3.8-rc4/drivers/usb/storage/usual-tables.c > --- linux-3.8-rc4_orig/drivers/usb/storage/usual-tables.c 2013-01-22 > 14:12:42.594238726 +0800 > +++ linux

Re: [PATCH 06/30] usb/gadget: add some infracture to register/unregister functions

2013-01-27 Thread Sebastian Andrzej Siewior
* Andrzej Pietrasiewicz | 2013-01-15 16:26:20 [+0100]: >On Tuesday, January 15, 2013 1:03 PM Sebastian Andrzej Siewior wrote: > > > >> >usb_function_driver. >> > >> >I would call the structure in question "usb_function_module". >> >Or even better: "usb_function_pool" - you get usb_functions from i

Re: [PATCH 17/30] usb/gadget: nokia: use function framework for ACM

2013-01-27 Thread Sebastian Andrzej Siewior
On 01/17/2013 12:22 PM, Felipe Balbi wrote: > On Sun, Dec 23, 2012 at 09:10:11PM +0100, Sebastian Andrzej Siewior > wrote: >> This patch converts the acm_ms gadget to make use of the >> function framework to request the ACM function. >> >> The "old" include interface for acm is now removed since n

Re: UAS questions

2013-01-27 Thread Sebastian Andrzej Siewior
On 01/27/2013 04:55 PM, Felipe Balbi wrote: > On Sun, Jan 27, 2013 at 02:54:53PM +0200, Vlad Silman wrote: >> I have a few questions regarding Linux UAS host-side and >> device-side drivers. I've seen that Linux UAS host driver >> supports the task management commands as defined by T10 UAS spec, >>

ehci with full speed audio device leaking sitds

2013-01-27 Thread Andy Leiserson
Please CC me on replies, as I'm not on the list. My system is an AR9132 mips SoC, kernel 3.6.11 with OpenWrt patches. The audio device is a C-Media usb audio adapter, IDs 0d8c/000e. It is a full speed device, and is connected directly to the only USB port. I'm using only the output, but it does h

Re: UAS questions

2013-01-27 Thread Felipe Balbi
Hi, On Sun, Jan 27, 2013 at 02:54:53PM +0200, Vlad Silman wrote: > I have a few questions regarding Linux UAS host-side and device-side drivers. > I've seen that Linux UAS host driver supports the task management > commands as defined by T10 UAS spec, such as ABORT TASK and LOGICAL > UNIT RESET. >

[PATCH] usb/isp1760: declare schedule_ptds() and errata2_function() static

2013-01-27 Thread Sebastian Andrzej Siewior
From: Arvid Brodin Fix two problems detected by the sparse code analyser: |drivers/usb/host/isp1760-hcd.c:935:6: warning: symbol 'schedule_ptds' was not declared. Should it be static? |drivers/usb/host/isp1760-hcd.c:1288:6: warning: symbol 'errata2_function' was not declared. Should it be stat