Re: [RFC PATCH 0/7] usb: musb: add driver for control module

2013-01-18 Thread Felipe Balbi
Hi, On Tue, Jan 15, 2013 at 02:12:51PM +0530, Kishon Vijay Abraham I wrote: > Added a new driver for the usb part of control module. This has an API > to power on the USB2 phy and an API to write to the mailbox depending on > whether MUSB has to act in host mode or in device mode. > > Writing to

Re: [RFC PATCH 1/6] usb: otg: Add an API to bind the USB controller and PHY

2013-01-18 Thread Felipe Balbi
On Thu, Jan 17, 2013 at 04:44:52PM +0530, kishon wrote: > >>@@ -171,6 +188,11 @@ static inline void devm_usb_put_phy(struct device > >>*dev, struct usb_phy *x) > >> { > >> } > >> > >>+static inline struct usb_phy_bind *usb_bind_phy(const char *dev_name, u8 > >>index, > >>+

Re: [RFC PATCH 0/6] USB: Add multiple PHYs of same type

2013-01-18 Thread Felipe Balbi
On Wed, Jan 16, 2013 at 08:30:56PM +0530, Kishon Vijay Abraham I wrote: > New platforms are being added which has multiple PHY's (of same type) and > which has multiple USB controllers. The binding information has to be > present in the PHY library (otg.c) in order for it to return the > appropriat

Re: [PATCH 1/4] drivers: usb: phy: add a new driver for usb part of control module

2013-01-18 Thread Felipe Balbi
Hi, On Fri, Jan 18, 2013 at 03:10:42PM +0530, Kishon Vijay Abraham I wrote: > Added a new driver for the usb part of control module. This has an API > to power on the USB2 phy and an API to write to the mailbox depending on > whether MUSB has to act in host mode or in device mode. > > Writing to

Re: [PATCH 2/4] ARM: OMAP: devices: create device for usb part of control module

2013-01-18 Thread Felipe Balbi
On Fri, Jan 18, 2013 at 03:10:43PM +0530, Kishon Vijay Abraham I wrote: > A seperate driver has been added to handle the usb part of control > module. A device for the above driver is created here, using the register > address information to be used by the driver for powering on the PHY and > for w

Re: [PATCH 4/4] drivers: usb: start using the control module driver

2013-01-18 Thread Felipe Balbi
Hi, On Fri, Jan 18, 2013 at 03:10:45PM +0530, Kishon Vijay Abraham I wrote: > Start using the control module driver for powering on the PHY and for > writing to the mailbox instead of writing to the control module > registers on their own. > > Signed-off-by: Kishon Vijay Abraham I > --- > Docum

Re: [balbi-usb:gadget 19/61] drivers/usb/gadget/mv_udc_core.c:1124:16: error: 'struct mv_usb_platform_data' has no member named 'phy_init'

2013-01-18 Thread Felipe Balbi
On Fri, Jan 18, 2013 at 07:48:14PM +0800, kbuild test robot wrote: > tree: git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git gadget > head: 7620f5f22e9317ab26c0fdbdd15c5e9ed972ef89 > commit: 84b140524c107fb4f4fe618509bc067245c9d878 [19/61] usb: phy: mv_usb2: > add PHY driver for marv

Re: [PATCH 1/4] drivers: usb: phy: add a new driver for usb part of control module

2013-01-18 Thread Felipe Balbi
Hi, On Fri, Jan 18, 2013 at 05:40:04PM +0530, kishon wrote: > >>+void omap_control_usb_host_mode(struct device *dev) > >>+{ > >>+ u32 val; > >>+ struct omap_control_usb *control_usb = dev_get_drvdata(dev); > >>+ > >>+ val = AVALID | VBUSVALID; > >>+ > >>+ writel(val, control_usb->otghs_con

Re: [PATCH 0/2] ARM: Exynos5250: Enabling samsung usb phy

2013-01-18 Thread Felipe Balbi
Hi, On Tue, Dec 18, 2012 at 09:09:40PM +0530, Vivek Gautam wrote: > This patch-set enables the samsung-usbphy driver on exynos5250, > which enables the support for USB2 type and USB3 type phys. > The corresponding phy driver patches are available at: > 1) https://lkml.org/lkml/2012/12/18/201 > 2

Re: [PATCH 0/2] ARM: Exynos5250: Enabling samsung usb phy

2013-01-18 Thread Felipe Balbi
On Fri, Jan 18, 2013 at 03:10:13PM +0200, Felipe Balbi wrote: > Hi, > > On Tue, Dec 18, 2012 at 09:09:40PM +0530, Vivek Gautam wrote: > > This patch-set enables the samsung-usbphy driver on exynos5250, > > which enables the support for USB2 type and USB3 type phys. >

Re: [PATCH v6 3/4] USB: ehci-s5p: Add phy driver support

2013-01-18 Thread Felipe Balbi
On Fri, Jan 11, 2013 at 08:32:29PM +0530, Vivek Gautam wrote: > Adding the phy driver to ehci-s5p. Keeping the platform data > for continuing the smooth operation for boards which still uses it > > Signed-off-by: Vivek Gautam > Acked-by: Jingoo Han Missing Alan's Acked-by here. -- balbi sig

Re: [PATCH v7 2/4] usb: phy: samsung: Add host phy support to samsung-phy driver

2013-01-18 Thread Felipe Balbi
Hi, On Mon, Jan 14, 2013 at 05:52:15PM +0530, Vivek Gautam wrote: > This patch adds host phy support to samsung-usbphy driver and > further adds support for samsung's exynos5250 usb-phy. > > Signed-off-by: Praveen Paneri > Signed-off-by: Vivek Gautam Missing Kukjim's Acked-by here -- balbi

Re: [PATCH v7 2/4] usb: phy: samsung: Add host phy support to samsung-phy driver

2013-01-18 Thread Felipe Balbi
On Fri, Jan 18, 2013 at 07:51:08PM +0530, Vivek Gautam wrote: > Hi Felipe, > > > On Fri, Jan 18, 2013 at 6:46 PM, Felipe Balbi wrote: > > Hi, > > > > On Mon, Jan 14, 2013 at 05:52:15PM +0530, Vivek Gautam wrote: > >> This patch adds host phy support to

Re: [PATCH v7 2/4] usb: phy: samsung: Add host phy support to samsung-phy driver

2013-01-18 Thread Felipe Balbi
On Fri, Jan 18, 2013 at 07:59:52PM +0530, Vivek Gautam wrote: > HI Balbi, > > > On Fri, Jan 18, 2013 at 7:52 PM, Felipe Balbi wrote: > > On Fri, Jan 18, 2013 at 07:51:08PM +0530, Vivek Gautam wrote: > >> Hi Felipe, > >> > >> > >> On Fri, J

[GIT PULL] USB: fixes for v3.8-rc5

2013-01-18 Thread Felipe Balbi
Hi Greg, Here are my latest fixes for this -rc cycle. Unless something extremely important shows up, we should be ready for a final v3.8 ;-) Hopefully no surprises will come. ps: You will notice a few changes on arch/arm/mach-imx/ but they're needed to make fsl build and work again. We _do_ have

Re: [PATCH v3 4/4] usb: Add APIs to access host registers from Tegra PHY

2013-01-18 Thread Felipe Balbi
On Fri, Jan 18, 2013 at 10:28:38AM -0700, Stephen Warren wrote: > On 01/18/2013 08:30 AM, Alan Stern wrote: > > On Fri, 18 Jan 2013, Venu Byravarasu wrote: > > > >> As Tegra PHY driver needs to access one of the Host registers, > >> added few APIs. > >> > >> Signed-off-by: Venu Byravarasu > >> --

Re: [PATCH 2/4] USB: PHY: Get rid of instance number to differentiate legacy controller

2013-01-18 Thread Felipe Balbi
ram to get this info and > processed accordingly. > > Signed-off-by: Venu Byravarasu Acked-by: Felipe Balbi > --- > drivers/usb/phy/tegra_usb_phy.c | 32 +++- > include/linux/usb/tegra_usb_phy.h |1 + > 2 files changed, 16 insertions(+

Re: [PATCH 3/4] USB: PHY: Tegra: Get rid of instance number to differentiate PHY type

2013-01-18 Thread Felipe Balbi
; > Signed-off-by: Venu Byravarasu Acked-by: Felipe Balbi > --- > drivers/usb/phy/tegra_usb_phy.c | 23 +-- > include/linux/usb/tegra_usb_phy.h |1 + > 2 files changed, 10 insertions(+), 14 deletions(-) > > diff --git a/drivers/usb/phy/tegra_usb_

Re: [RESEND PATCH v9 0/2] usb: phy: samsung: Introducing usb phy driver for samsung SoCs

2013-01-18 Thread Felipe Balbi
Hi, On Fri, Jan 18, 2013 at 02:30:21PM +0530, Praveen Paneri wrote: > Changes from v8: > Resending this patch series after rebasing to the latest usb-next branch. > Rewording inline comments for better readability. > Removed IS_ENABLED(CONFIG_OF) as pdev->dev.of_node is enough to check for dt > s

Re: [PATCH V3 0/5] UVC webcam gadget related changes

2013-01-18 Thread Felipe Balbi
Hi, On Thu, Jan 17, 2013 at 04:23:48PM +0530, Bhupesh Sharma wrote: > This patchset tries to enhance the UVC webcam gadget driver and > is based on Laurent's git tree available here (head uvc-gadget): > git://linuxtv.org/pinchartl/uvcvideo.git > > Note that to ease review and integration of these

Re: Unable to reinsert the am35x MUSB module

2013-01-18 Thread Felipe Balbi
Hi, On Sun, Dec 02, 2012 at 05:33:36PM +0200, Dmitry Lifshitz wrote: > Hi, > > We are running kernel v3.5.7 on am3517 based hardware (CM-T3517 module). > Recently we faced with the following issue: am35x MUSB module fails > when trying to reinsert it. > It fails with the message "musb-hdrc.0: fai

Re: [PATCH] usb: dwc3: debugfs: convert our regdump to use regsets

2013-01-18 Thread Felipe Balbi
On Fri, Jan 18, 2013 at 11:23:08AM -0800, Greg KH wrote: > On Fri, Jan 18, 2013 at 10:24:39AM +0200, Felipe Balbi wrote: > > regset is a generic implementation of regdump > > utility through debugfs. > > > > Signed-off-by: Felipe Balbi > > --- > > > &g

Re: MUSB testing failures

2013-01-18 Thread Felipe Balbi
Hi, On Fri, Jan 18, 2013 at 03:56:12PM -0500, Shane Whalen wrote: > I am having problems with the 3.6 musb kernel driver. I am running > testusb.c and test.sh to try and diagnose where the problem may live. > Here are the outputs from the 'unlink' and 'control' tests. > > Host setup: 3.6.5-rt14-d

Re: [PATCH] usb: gadget: nokia: Add mass storage driver to g_nokia

2013-01-20 Thread Felipe Balbi
On Sun, Jan 20, 2013 at 03:58:13AM +0100, Pali Rohár wrote: > Signed-off-by: Pali Rohár NAK for two reasons: a) the original Nokia kernel used a separate g_file_storage gadget to use Mass Storage mode, use that b) there is no commit log -- balbi signature.asc Description: Digital si

Re: USB: storage: optimize the matching rules and support new switch command for Huawei USB storage devices

2013-01-20 Thread Felipe Balbi
Hi, On Fri, Jan 18, 2013 at 03:43:59PM -0800, Greg KH wrote: > On Mon, Jan 14, 2013 at 10:55:48AM +0800, fangxiaozhi 00110321 wrote: > > > > From: fangxiaozhi > > > > 1. Optimize the matching rules with new macro for Huawei USB storage > >devices, to avoid to load USB storage driver for the

Re: [PATCH] usb: gadget: nokia: Add mass storage driver to g_nokia

2013-01-21 Thread Felipe Balbi
Hi, On Sun, Jan 20, 2013 at 11:17:31AM +0100, Pali Rohár wrote: > On Sunday 20 January 2013 10:25:37 Felipe Balbi wrote: > > On Sun, Jan 20, 2013 at 03:58:13AM +0100, Pali Rohár wrote: > > > Signed-off-by: Pali Rohár > > > > NAK for two reasons: > > >

Re: Linux USB file storage gadget with new UDC

2013-01-21 Thread Felipe Balbi
Hi, On Mon, Jan 21, 2013 at 01:32:09PM +0800, victor yeo wrote: > >> > This log looks oddly incomplete. A lot of debugging messages are > >> > missing. > >> > > >> >> g_file_storage gadget: ep0-setup, length 8: > >> >> : 80 06 00 01 00 00 40 00 > >> >> g_file_storage gadget: get device de

Re: [balbi-usb:gadget 19/61] drivers/usb/gadget/mv_udc_core.c:1124:16: error: 'struct mv_usb_platform_data' has no member named 'phy_init'

2013-01-21 Thread Felipe Balbi
Hi, On Sun, Jan 20, 2013 at 05:24:05PM -0800, Chao Xie wrote: > -Original Message- > From: Felipe Balbi [mailto:ba...@ti.com] > Sent: Friday, January 18, 2013 8:05 PM > To: kbuild test robot > Cc: Chao Xie; linux-usb@vger.kernel.org; Felipe Balbi > Subject: Re: [balb

Re: Linux USB file storage gadget with new UDC

2013-01-21 Thread Felipe Balbi
Hi, On Mon, Jan 21, 2013 at 04:15:32PM +0800, victor yeo wrote: > Hi, > > >> >> >> g_file_storage gadget: ep0-setup, length 8: > >> >> >> : 80 06 00 01 00 00 40 00 > >> >> >> g_file_storage gadget: get device descriptor > >> >> > > >> >> > For example, right here we should see the device

Re: USB subsystem stops working

2013-01-21 Thread Felipe Balbi
Hi, On Mon, Jan 21, 2013 at 04:51:28PM +0900, Norbert Preining wrote: > Hi everyone, > (please Cc) > > I am currently running 3.8.0-rc4 and see the following behaviour: > When I plug/eject my kindle a few times, suddenly the kernel does > not see any USB activity any more. Nothing. PLugging in th

Re: [PATCH v3 0/2] Adding USB 3.0 DRD-phy support for exynos5250

2013-01-21 Thread Felipe Balbi
Hi, On Mon, Jan 21, 2013 at 12:15:10PM +0530, Vivek Gautam wrote: > Hi Felipe, > > > On Mon, Jan 14, 2013 at 6:29 PM, Vivek Gautam > wrote: > > Changes from v2: > > - Renaming 'samsung-usbphy.c' driver to 'samsung-usb2.c' indicating > >usb 2.0 phy controller's driver for Samsung's SoCs. >

Re: [balbi-usb:gadget 19/61] drivers/usb/gadget/mv_udc_core.c:1124:16: error: 'struct mv_usb_platform_data' has no member named 'phy_init'

2013-01-21 Thread Felipe Balbi
Hi, On Mon, Jan 21, 2013 at 04:38:32PM +0800, Fengguang Wu wrote: > On Mon, Jan 21, 2013 at 10:13:43AM +0200, Felipe Balbi wrote: > > On Sun, Jan 20, 2013 at 05:24:05PM -0800, Chao Xie wrote: > [snip] > > > I am formatting and re-test the patches. I will send ou

Re: Linux USB file storage gadget with new UDC

2013-01-21 Thread Felipe Balbi
On Mon, Jan 21, 2013 at 04:45:03PM +0800, victor yeo wrote: > Hi, > > >> > ep0_complete() should be called by your UDC driver. Seriously dude, read > >> > the documentation and read other drivers to figure out how things should > >> > be called. A quick look in the storage drivers would have shown

Re: Linux USB file storage gadget with new UDC

2013-01-21 Thread Felipe Balbi
Hi, On Mon, Jan 21, 2013 at 07:13:03PM +0800, victor yeo wrote: > >> >> Yes, ep0_complete() is called in UDC driver queue function, i just > >> >> added it in and ep0_complete() is called now. The dmesg output is > >> >> below. > >> > > >> > not in queue function, you should call it when completin

Re: [PATCH 13/15] drivers/usb/musb/omap2430.c: adjust duplicate test

2013-01-21 Thread Felipe Balbi
esult of devm_kzalloc() call to the 'config' variable but then checks for NULL the 'data' variable (already checked after previous call). Thus we risk a kernel oops further when data pointed by 'config' is written to by subsequent of_property_read_u32() call

Re: Linux USB file storage gadget with new UDC

2013-01-21 Thread Felipe Balbi
On Mon, Jan 21, 2013 at 11:14:19PM +0800, victor yeo wrote: > Hi, > > >> >> >> Yes, ep0_complete() is called in UDC driver queue function, i just > >> >> >> added it in and ep0_complete() is called now. The dmesg output is > >> >> >> below. > >> >> > > >> >> > not in queue function, you should cal

Re: [PATCH 12/15] USB: gadget/freescale: disable non-multiplatform drivers

2013-01-21 Thread Felipe Balbi
ination > with CONFIG_ARCH_MULTIPLATFORM, and fix them properly in v3.9 unless > someone has an better solution. > > Without this patch, building allyesconfig results in: > > drivers/usb/gadget/fsl_mxc_udc.c:21:27: fatal error: mach/hardware.h: No such > file or directory > > S

Re: [PATCH 13/15] USB: ehci: make orion and mxc bus glues coexist

2013-01-21 Thread Felipe Balbi
On Mon, Jan 21, 2013 at 05:16:06PM +, Arnd Bergmann wrote: > In linux-3.8-rc1 it became possible to build the imx and > mvebu platforms together in a single kernel, which clashes > in the ehci driver. > > Manjunath Goudar is already working on a patch to convert > both the imx and the mvebu gl

Re: [PATCH 29/33] usb: Convert to devm_ioremap_resource()

2013-01-21 Thread Felipe Balbi
icit > error messages can be removed from the failure code paths. > > Signed-off-by: Thierry Reding > Cc: Greg Kroah-Hartman > Cc: Alan Stern > Cc: Felipe Balbi > Cc: linux-usb@vger.kernel.org For drivers/usb/renenas, drivers/usb/otg, drivers/usb/phy, drivers/usb/musb and driv

Re: [PATCH 12/15] USB: gadget/freescale: disable non-multiplatform drivers

2013-01-21 Thread Felipe Balbi
On Mon, Jan 21, 2013 at 10:57:16AM -0800, Greg Kroah-Hartman wrote: > On Mon, Jan 21, 2013 at 08:41:38PM +0200, Felipe Balbi wrote: > > Hi, > > > > On Mon, Jan 21, 2013 at 05:16:05PM +, Arnd Bergmann wrote: > > > Both the fsl_mxc gadget and the imx_ud

Re: Linux USB file storage gadget with new UDC

2013-01-22 Thread Felipe Balbi
Hi, On Tue, Jan 22, 2013 at 11:47:09AM +0800, victor yeo wrote: > >> > >> Some progress, now i am able to see the dump_msg from ep0_complete(). > >> > >> It looks like the enumeration is ok, but the SCSI Inquiry command is > >> > >> not received by the UDC driver. The do_scsi_command() in > >> > >

Re: [PATCH 0/2] usb: exynos: Fix compatible strings used for device

2013-01-22 Thread Felipe Balbi
Hi, On Tue, Jan 22, 2013 at 11:18:55AM +0530, Vivek Gautam wrote: > On Tue, Jan 22, 2013 at 11:05 AM, Kukjin Kim wrote: > > Sylwester Nawrocki wrote: > >> > >> On 12/24/2012 09:13 AM, Vivek Gautam wrote: > >> These two changes look good to me. For both of them: > >> > >> Reviewed-

Re: [PATCH v6 0/4] Adding usb2.0 host-phy support for exynos5250

2013-01-22 Thread Felipe Balbi
HI, On Mon, Jan 21, 2013 at 09:23:28PM -0800, Kukjin Kim wrote: > Vivek Gautam wrote: > > > > Changes from v5: > > - Rebased on top of latest patches: > > usb: phy: samsung: Introducing usb phy driver for hsotg (v9) > > usb: phy: samsung: Add support to set pmu isolation (v6) > >

Re: USB subsystem stops working

2013-01-22 Thread Felipe Balbi
Hi, On Tue, Jan 22, 2013 at 10:52:41AM +0900, Norbert Preining wrote: > Hi Felipe, hi all, > > On Mo, 21 Jan 2013, Felipe Balbi wrote: > > Can you try rebuilding your kernel with CONFIG_USB_DEBUG=y and run your > > test again ? Maybe it gives us more information of what&#

Re: USB subsystem stops working

2013-01-22 Thread Felipe Balbi
HI, On Tue, Jan 22, 2013 at 06:04:53PM +0900, Norbert Preining wrote: > Hi > > > I need to see what happens before the problem shows up. Your log starts > > after the problem has already triggered. > > It is a cycle, it repeats after the lsusb -v when it is back to > normal. So reading from *aft

Re: Linux USB file storage gadget with new UDC

2013-01-22 Thread Felipe Balbi
Hi, On Tue, Jan 22, 2013 at 05:22:58PM +0800, victor yeo wrote: > >> >> > Is there any way to to further debug into why do_scsi_command() is > >> >> > not called? > >> >> > >> >> usbmon, printk(), trace_printk(), kernel function tracer, there are many > >> >> ways choose one and spend some time d

Re: [PATCH v6 0/4] Adding usb2.0 host-phy support for exynos5250

2013-01-22 Thread Felipe Balbi
On Tue, Jan 22, 2013 at 02:45:26PM +0530, Vivek Gautam wrote: > Hi Felipe, > > > On Tue, Jan 22, 2013 at 2:20 PM, Felipe Balbi wrote: > > HI, > > > > On Mon, Jan 21, 2013 at 09:23:28PM -0800, Kukjin Kim wrote: > >> Vivek Gautam wrote: > >> >

Re: [PATCH v6 0/4] Adding usb2.0 host-phy support for exynos5250

2013-01-22 Thread Felipe Balbi
HI, On Tue, Jan 22, 2013 at 11:29:47AM +0200, Felipe Balbi wrote: > On Tue, Jan 22, 2013 at 02:45:26PM +0530, Vivek Gautam wrote: > > Hi Felipe, > > > > > > On Tue, Jan 22, 2013 at 2:20 PM, Felipe Balbi wrote: > > > HI, > > > > > > On M

Re: [PATCH] usb: phy: moving all PHY API definitions to usb/phy directory

2013-01-22 Thread Felipe Balbi
HI, On Tue, Jan 22, 2013 at 04:26:50PM +0530, Venu Byravarasu wrote: > As drivers/usb/otg/otg.c contains most of the PHY related APIs > which are not OTG specific, moving them to more logical place > under driver/usb/phy. > > Signed-off-by: Venu Byravarasu > --- > drivers/usb/otg/otg.c| 18

Re: [PATCH v2] usb: phy: moving all PHY API definitions to usb/phy directory

2013-01-22 Thread Felipe Balbi
Hi, On Tue, Jan 22, 2013 at 04:31:11PM +0530, Venu Byravarasu wrote: > As drivers/usb/otg/otg.c contains most of the PHY related APIs > which are not OTG specific, moving them to more logical place > under driver/usb/phy. > > Signed-off-by: Venu Byravarasu > --- > delta from v1: > Missed adding

Re: Linux USB file storage gadget with new UDC

2013-01-22 Thread Felipe Balbi
On Tue, Jan 22, 2013 at 10:49:50PM +0800, victor yeo wrote: > Hi, > > >> >> > It sounds like the UDC driver now works okay for control transfers but > >> >> > not for bulk transfers. > >> >> > > >> >> > >> >> Some code in file_storage.c that i cannot understand. In > >> >> ep0_complete(), the code

Re: [PATCH] usb: gadget: nokia: Add mass storage driver to g_nokia

2013-01-22 Thread Felipe Balbi
Hi, On Tue, Jan 22, 2013 at 12:03:09PM +0100, Pali Rohár wrote: > On Monday 21 January 2013 09:05:06 Felipe Balbi wrote: > > Hi, > > > > On Sun, Jan 20, 2013 at 11:17:31AM +0100, Pali Rohár wrote: > > > On Sunday 20 January 2013 10:25:37 Felipe Balbi wrote: > &g

Re: [PATCH 0/2] usb: exynos: Fix compatible strings used for device

2013-01-23 Thread Felipe Balbi
Hi, On Tue, Jan 22, 2013 at 02:04:56PM -0800, Kukjin Kim wrote: > Felipe Balbi wrote: > > Hi, > > > Hi Felipe, > > [...] > > > > Right, DWC has version number, but that being the kind of USB controller > > > (USB 2.0 and USB 3.0) > > > &g

Re: Linux USB file storage gadget with new UDC

2013-01-23 Thread Felipe Balbi
Hi, On Wed, Jan 23, 2013 at 03:10:50PM +0800, victor yeo wrote: > >> >> Ok, rephrase the question, is this the flow for bulk transfer? > >> >> > >> >> 1) data is received by Usb mass storage HW, the UDC driver ISR is > >> >> called to read the data to usb_request buffer > >> >> 2) bulk_out_complet

Re: Linux USB file storage gadget with new UDC

2013-01-23 Thread Felipe Balbi
Hi, On Wed, Jan 23, 2013 at 06:04:55PM +0800, victor yeo wrote: > Hi, > > >> >> >> Ok, rephrase the question, is this the flow for bulk transfer? > >> >> >> > >> >> >> 1) data is received by Usb mass storage HW, the UDC driver ISR is > >> >> >> called to read the data to usb_request buffer > >> >

Re: Linux USB file storage gadget with new UDC

2013-01-23 Thread Felipe Balbi
Hi, On Wed, Jan 23, 2013 at 07:17:44PM +0800, victor yeo wrote: > >> >> In fsg_dev structure, there are 3 usb_ep: bulk_in, bulk_out, and > >> >> intr_in. Why is the intr_out endpoint not defined? > >> > > >> > because it's not needed. Read the USB Mass Storage Class specification > >> > from usb.o

Re: [PATCH v2 2/4] ARM: OMAP: devices: create device for usb part of control module

2013-01-23 Thread Felipe Balbi
Hi, On Mon, Jan 21, 2013 at 07:38:26PM +0530, Kishon Vijay Abraham I wrote: > A seperate driver has been added to handle the usb part of control > module. A device for the above driver is created here, using the register > address information to be used by the driver for powering on the PHY and >

Re: Linux USB file storage gadget with new UDC

2013-01-23 Thread Felipe Balbi
Hi, On Wed, Jan 23, 2013 at 07:35:41PM +0800, victor yeo wrote: > Hi, > > >> >> Ok. In the gadget driver, it keeps on receiving the same bulk_out > >> >> data, maybe because the data is not processed. The get_next_command is > >> >> not called. > >> >> > >> >> g_file_storage gadget: bulk-out, len

Re: [V4 PATCH 18/26] usb: phy: mv_usb2_phy: add externel chip support

2013-01-23 Thread Felipe Balbi
Hi, On Tue, Jan 22, 2013 at 10:51:32AM +0800, Chao Xie wrote: > On Mon, Jan 21, 2013 at 11:51 PM, Russell King - ARM Linux > wrote: > > On Mon, Jan 21, 2013 at 05:07:36AM -0500, Chao Xie wrote: > >> + mv_phy->extern_chip.head = devm_kzalloc(&pdev->dev, > >> +

Re: [V4 PATCH 00/26] mv-usb fix and enhancement patches

2013-01-23 Thread Felipe Balbi
Hi, On Mon, Jan 21, 2013 at 05:07:18AM -0500, Chao Xie wrote: > The patches are divied into 4 parts > 1. bug fixes > usb: gadget: mv_udc: use udc_start and udc_stop functions > usb: gadget: mv_udc: use devm_xxx for probe > usb: gadget: mv_udc: fix the warning of mv_udc_remove > usb: otg: m

Re: Linux USB file storage gadget with new UDC

2013-01-23 Thread Felipe Balbi
Hi, On Wed, Jan 23, 2013 at 08:24:36PM +0800, victor yeo wrote: > >> >> In my udc driver, i set the req->length to the number of bytes i > >> >> received from HW, which is 31 bytes. Is it necessary to do that? How > >> > > >> > you shouldn't touch req->lenght, you should only update req->actual. >

Re: Linux USB file storage gadget with new UDC

2013-01-23 Thread Felipe Balbi
Hi, On Wed, Jan 23, 2013 at 11:20:56PM +0800, victor yeo wrote: > Hi, > > >> Here are the last two setup data and CBW data received. the > >> get_next_command() is not called when CBW data is received. the > >> bulk_out_complete() wakes up the thread, however, get_next_command() > >> still sleeps

Re: Linux USB file storage gadget with new UDC

2013-01-23 Thread Felipe Balbi
On Thu, Jan 24, 2013 at 02:37:32PM +0800, victor yeo wrote: > Hi, > > >> >> Here are the last two setup data and CBW data received. the > >> >> get_next_command() is not called when CBW data is received. the > >> >> bulk_out_complete() wakes up the thread, however, get_next_command() > >> >> still

[PATCH 1/8] usb: gadget: adm5536udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/amd5536udc.c | 59 ++--- drivers/usb/gadget/amd5536udc.h | 2 ++ 2 files changed, 16 insertions

[PATCH 2/8] usb: gadget: fusb300_udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/fusb300_udc.c | 65 +++- drivers/usb/gadget/fusb300_udc.h | 2 ++ 2 files changed, 20 insertions

[PATCH 3/8] usb: gadget: goku_udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/goku_udc.c | 70 +-- drivers/usb/gadget/goku_udc.h | 1 + 2 files changed, 15 insertions(+), 56

[PATCH 4/8] usb: gadget: fsl_udc_core: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/fsl_udc_core.c | 58 --- 1 file changed, 11 insertions(+), 47 deletions(-) diff --git a/drivers/usb

[PATCH 5/8] usb: gadget: m66592-udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/m66592-udc.c | 70 +++-- drivers/usb/gadget/m66592-udc.h | 1 + 2 files changed, 20 insertions

[PATCH 7/8] usb: gadget: pch_udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/pch_udc.c | 67 +--- 1 file changed, 14 insertions(+), 53 deletions(-) diff --git a/drivers/usb

[PATCH 8/8] usb: gadget: completely remove ->start/->stop

2013-01-24 Thread Felipe Balbi
Those have been deprecated for a long time and previous patches just converted all remaining users of those. Since there are no in-tree users and we don't want any new users for them, let's obliterate every piece of code related to those calls. Signed-off-by: Felipe Balbi --- d

[PATCH 6/8] usb: gadget: omap_udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/omap_udc.c | 49 +++ 1 file changed, 12 insertions(+), 37 deletions(-) diff --git a/drivers/usb

Re: Linux USB file storage gadget with new UDC

2013-01-24 Thread Felipe Balbi
Hi, On Thu, Jan 24, 2013 at 04:25:24PM +0800, victor yeo wrote: > Hi, > > >> > you should an interrupt handler to handle interrupts from your device. > >> > Also, there are way too many mistakes on your driver, run checkpatch.pl, > >> > compile it with sparse, don't hardcode addresses, don't reim

Re: [V5 PATCH 07/26] usb: gadget: mv_udc: make mv_udc depends on ARCH_MMP or ARCH_PXA

2013-01-24 Thread Felipe Balbi
Hi, On Thu, Jan 24, 2013 at 01:38:31AM -0500, Chao Xie wrote: > Only ARCH_PXA and ARCH_MMP will use mv_udc. > > Signed-off-by: Chao Xie NAK, you should fix your transceiver dependency. I want these gadget drivers to compile cleanly on all arches so we make proper use of linux-next. I have appli

[PATCH] usb: dwc3: gadget: change HIRD threshold to 12

2013-01-24 Thread Felipe Balbi
code has been wrong forever, so this should be backported as far back as possible. Cc: Signed-off-by: Felipe Balbi --- It's getting late for v3.8-rc cycle, so I'll send this on v3.9 merge window with backport for stable tree. drivers/usb/dwc3/gadget.c | 7 +-- 1 file change

Re: Linux USB file storage gadget with new UDC

2013-01-24 Thread Felipe Balbi
Hi, On Thu, Jan 24, 2013 at 06:23:08PM +0800, victor yeo wrote: > Hi, > > >> >> > Also, you shouldn't requeue the request yourself, gadget driver owns > >> >> > the > >> >> > request. > >> >> which request do i requeue? please kindly point that out. Thanks! > >> > > >> > look at your code. Look

Re: Linux USB file storage gadget with new UDC

2013-01-24 Thread Felipe Balbi
Hi, On Thu, Jan 24, 2013 at 08:14:25PM +0800, victor yeo wrote: > Hi, > > >> >> is it this code? i will remove the redundant code after complete. > >> >> > >> >> ka_ep_p->req.complete(&ka_ep_p->ep, &req->req); > >> >> if (num == 0) { > >> >> ka_ep_p->req.length = 0

Re: [PATCH v2] usb: host: tegra: make use of PHY pointer of HCD

2013-01-24 Thread Felipe Balbi
Hi, On Thu, Jan 24, 2013 at 03:57:03PM +0530, Venu Byravarasu wrote: > As pointer to PHY structure can be stored in struct usb_hcd > making use of it, to call Tegra PHY APIs. > > Call to usb_phy_shutdown() is moved up in tegra_ehci_remove(), > so that to avoid dereferencing of hcd after its freed

Re: [RFC PATCH] uvc_v4l2: reset connection on uvc open

2013-01-24 Thread Felipe Balbi
On Thu, Jan 24, 2013 at 03:35:58PM +0100, Michael Grzeschik wrote: > On Thu, Jan 24, 2013 at 03:21:37PM +0100, Laurent Pinchart wrote: > > Hi Michael, > > > > On Thursday 24 January 2013 14:54:01 Michael Grzeschik wrote: > > > On Thu, Jan 24, 2013 at 12:02:12PM +0100, Laurent Pinchart wrote: > > >

Re: [PATCH 1/8] usb: gadget: adm5536udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
HI, On Thu, Jan 24, 2013 at 11:01:24AM +0200, Felipe Balbi wrote: > Mechanical change making use of the new (can we > still call it new ?) interface for registering > UDC drivers. > > Signed-off-by: Felipe Balbi I'll resend this series, my grep missed pxa folks. -- b

[PATCH 01/11] usb: gadget: adm5536udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/amd5536udc.c | 59 ++--- drivers/usb/gadget/amd5536udc.h | 2 ++ 2 files changed, 16 insertions

[PATCH 02/11] usb: gadget: fusb300_udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/fusb300_udc.c | 65 +++- drivers/usb/gadget/fusb300_udc.h | 2 ++ 2 files changed, 20 insertions

[PATCH 03/11] usb: gadget: goku_udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/goku_udc.c | 70 +-- drivers/usb/gadget/goku_udc.h | 1 + 2 files changed, 15 insertions(+), 56

[PATCH 04/11] usb: gadget: fsl_udc_core: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/fsl_udc_core.c | 58 --- 1 file changed, 11 insertions(+), 47 deletions(-) diff --git a/drivers/usb

[PATCH 05/11] usb: gadget: m66592-udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/m66592-udc.c | 70 +++-- drivers/usb/gadget/m66592-udc.h | 1 + 2 files changed, 20 insertions

[PATCH 06/11] usb: gadget: omap_udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/omap_udc.c | 49 +++ 1 file changed, 12 insertions(+), 37 deletions(-) diff --git a/drivers/usb

[PATCH 07/11] usb: gadget: pch_udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/pch_udc.c | 67 +--- 1 file changed, 14 insertions(+), 53 deletions(-) diff --git a/drivers/usb

[PATCH 08/11] usb: gadget: pxa25x_udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/pxa25x_udc.c | 62 ++--- drivers/usb/gadget/pxa25x_udc.h | 1 + 2 files changed, 15 insertions

[PATCH 09/11] usb: gadget: pxa27x_udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/pxa27x_udc.c | 58 +++-- drivers/usb/gadget/pxa27x_udc.h | 1 + 2 files changed, 16 insertions

[PATCH 10/11] usb: gadget: s3c2410: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/s3c2410_udc.c | 65 drivers/usb/gadget/s3c2410_udc.h | 1 + 2 files changed, 13 insertions

[PATCH 11/11] usb: gadget: completely remove ->start/->stop

2013-01-24 Thread Felipe Balbi
Those have been deprecated for a long time and previous patches just converted all remaining users of those. Since there are no in-tree users and we don't want any new users for them, let's obliterate every piece of code related to those calls. Signed-off-by: Felipe Balbi --- d

[PATCH v2] usb: gadget: pxa27x_udc: convert to udc_start/udc_stop

2013-01-24 Thread Felipe Balbi
Mechanical change making use of the new (can we still call it new ?) interface for registering UDC drivers. Signed-off-by: Felipe Balbi --- Had missed a trailing gadget->disconnect() call. drivers/usb/gadget/pxa27x_udc.c | 61 ++--- drivers/usb/gad

Re: [RFC PATCH] uvc_v4l2: reset connection on uvc open

2013-01-24 Thread Felipe Balbi
On Thu, Jan 24, 2013 at 04:05:16PM +0100, Laurent Pinchart wrote: > Hi Felipe, > > On Thursday 24 January 2013 16:42:50 Felipe Balbi wrote: > > On Thu, Jan 24, 2013 at 03:35:58PM +0100, Michael Grzeschik wrote: > > > On Thu, Jan 24, 2013 at 03:21:37PM +0100, Laurent Pi

[RFC/PATCH 01/32] usb: gadget: udc-core: allow udc class register gadget device

2013-01-24 Thread Felipe Balbi
, we're adding a flag which will be set by UDC driver once it removes the code for registering the gadget device. Once all are converted, the new flag will be removed. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/udc-core.c | 23 +++ include/linux/usb/gadget.h

[RFC/PATCH 03/32] usb: musb: gadget: let udc-core manage gadget-dev

2013-01-24 Thread Felipe Balbi
By simply setting a flag, we can delete a little boilerplate code. Signed-off-by: Felipe Balbi --- drivers/usb/musb/musb_gadget.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/usb/musb/musb_gadget.c b/drivers/usb/musb/musb_gadget.c index 8767874..1c4a8e8

[RFC/PATCH 04/32] usb: gadget: omap_udc: let udc-core manage gadget->dev

2013-01-24 Thread Felipe Balbi
By simply setting a flag, we drop some boilerplate code. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/omap_udc.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/drivers/usb/gadget/omap_udc.c b/drivers/usb/gadget/omap_udc.c index d0c87b1..234e82b 100644

[RFC/PATCH 08/32] usb: gadget: bcm63xx_udc: let udc-core manage gadget->dev

2013-01-24 Thread Felipe Balbi
By simply setting a flag, we can drop some boilerplate code. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/bcm63xx_udc.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/usb/gadget/bcm63xx_udc.c b/drivers/usb/gadget/bcm63xx_udc.c index 47a4993..10b0475

[RFC/PATCH 06/32] usb: gadget: at91_udc: let udc-core manage gadget->dev

2013-01-24 Thread Felipe Balbi
By simply setting a flag, we can remove some boilerplate code. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/at91_udc.c | 13 ++--- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/drivers/usb/gadget/at91_udc.c b/drivers/usb/gadget/at91_udc.c index 10f45fa..177b56d

[RFC/PATCH 02/32] usb: dwc3: gadget: let udc-core manage gadget->dev

2013-01-24 Thread Felipe Balbi
We don't need to register that device ourselves if we simply set gadget->register_my_device. Signed-off-by: Felipe Balbi --- drivers/usb/dwc3/gadget.c | 17 ++--- 1 file changed, 2 insertions(+), 15 deletions(-) diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/

[RFC/PATCH 00/32] usb: refactor gadget->dev registration

2013-01-24 Thread Felipe Balbi
Hi folks, Please comment on this series as I need help testing it on all UDC controllers so we can send it for v3.10 merge window. NOTE: I will queue this series for v3.10 merge window, so make sure to test and comment. cheers Felipe Balbi (32): usb: gadget: udc-core: allow udc class

[RFC/PATCH 05/32] usb: gadget: amd5536udc: let udc-core manage gadget->dev

2013-01-24 Thread Felipe Balbi
By simply setting a flag, we drop some boilerplate code. Signed-off-by: Felipe Balbi --- drivers/usb/gadget/amd5536udc.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/drivers/usb/gadget/amd5536udc.c b/drivers/usb/gadget/amd5536udc.c index 3dac001..1827d4c 100644

<    1   2   3   4   5   6   7   8   9   10   >