Re: URB completion order, normal behavior or bug?

2013-06-24 Thread Daniel Santos
On 06/23/2013 09:08 PM, Alan Stern wrote: On Sun, 23 Jun 2013, Daniel Santos wrote: So I'm working on this MCP2210 driver which talks in 64-byte request/response pairs using interrupt URBs (it can be used with hid-generic, but I don't want to do that). I'm testing on a Raspberry Pi Model B us

Re: [PATCH V2] USB: initialize or shutdown PHY when add or remove host controller

2013-06-24 Thread Chao Xie
On Tue, Jun 25, 2013 at 3:45 AM, Felipe Balbi wrote: > Hi, > > On Fri, Jun 21, 2013 at 09:07:59AM +0800, Chao Xie wrote: >> On Fri, Jun 21, 2013 at 1:25 AM, Alan Stern >> wrote: >> > On Thu, 20 Jun 2013, Felipe Balbi wrote: >> > >> >> > In fact, the PHY setting and handling is related to platfor

RE: Chipidea usb otg support for IMX/MXS (device functionality)

2013-06-24 Thread Chen Peter-B29397
> > No, I don't have this one available right now. I tried both MX23 > Olinuxino maxi > (I just mutilated the board so that I cut traces to the USB devices on > the board > and routed out USB gadget connector) and a custom MX23 board. > > Note that both have working USB peripheral mode in U-Bo

Re: [PATCH] chipidea: ci13xxx_imx: Access phy via private data

2013-06-24 Thread Peter Chen
On Tue, Jun 25, 2013 at 2:58 AM, Fabio Estevam wrote: > commit ea1418b5f1a (usb: chipidea: i.MX: use devm_usb_get_phy_by_phandle to > get > phy) causes the USB host to miss the disconnect/connect events. > > In order to reproduce this problem: > > - Insert a USB thumb into the USB host port (conn

Re: [PATCH V2] USB: initialize or shutdown PHY when add or remove host controller

2013-06-24 Thread Felipe Balbi
Hi, On Tue, Jun 25, 2013 at 09:25:30AM +0800, Chao Xie wrote: > >> It is same as clk, irq requested by ehci-xxx driver. > > > > clocks could be handled generically in some cases, we have pm_clk_add() > > for a reason ;-) > > > > Also, clock handling can be hidden under pm_runtime callbacks (say, >

[PATCH v8 0/8] Generic PHY Framework

2013-06-24 Thread Kishon Vijay Abraham I
Added a generic PHY framework that provides a set of APIs for the PHY drivers to create/destroy a PHY and APIs for the PHY users to obtain a reference to the PHY with or without using phandle. This framework will be of use only to devices that uses external PHY (PHY functionality is not embedded w

[PATCH v8 8/8] usb: phy: twl4030-usb: remove *set_suspend* and *phy_init* ops

2013-06-24 Thread Kishon Vijay Abraham I
Now that twl4030-usb is adapted to the new generic PHY framework, *set_suspend* and *phy_init* ops can be removed from twl4030-usb driver. Signed-off-by: Kishon Vijay Abraham I Reviewed-by: Felipe Balbi Reviewed-by: Sylwester Nawrocki --- drivers/usb/phy/phy-twl4030-usb.c | 57 +-

[PATCH v8 6/8] usb: musb: omap2430: use the new generic PHY framework

2013-06-24 Thread Kishon Vijay Abraham I
Use the generic PHY framework API to get the PHY. The usb_phy_set_resume and usb_phy_set_suspend is replaced with power_on and power_off to align with the new PHY framework. musb->xceiv can't be removed as of now because musb core uses xceiv.state and xceiv.otg. Once there is a separate state mach

[PATCH v8 7/8] usb: phy: omap-usb2: remove *set_suspend* callback from omap-usb2

2013-06-24 Thread Kishon Vijay Abraham I
Now that omap-usb2 is adapted to the new generic PHY framework, *set_suspend* ops can be removed from omap-usb2 driver. Signed-off-by: Kishon Vijay Abraham I Reviewed-by: Felipe Balbi Reviewed-by: Sylwester Nawrocki --- drivers/usb/phy/phy-omap-usb2.c | 25 - 1 file c

[PATCH v8 2/8] usb: phy: omap-usb2: use the new generic PHY framework

2013-06-24 Thread Kishon Vijay Abraham I
Used the generic PHY framework API to create the PHY. Now the power off and power on are done in omap_usb_power_off and omap_usb_power_on respectively. However using the old USB PHY library cannot be completely removed because OTG is intertwined with PHY and moving to the new framework will break

[PATCH v8 3/8] usb: phy: twl4030: use the new generic PHY framework

2013-06-24 Thread Kishon Vijay Abraham I
Used the generic PHY framework API to create the PHY. For powering on and powering off the PHY, power_on and power_off ops are used. Once the MUSB OMAP glue is adapted to the new framework, the suspend and resume ops of usb phy library will be removed. However using the old usb phy library cannot

[PATCH v8 5/8] ARM: dts: omap: update usb_otg_hs data

2013-06-24 Thread Kishon Vijay Abraham I
Updated the usb_otg_hs dt data to include the *phy* and *phy-names* binding in order for the driver to use the new generic PHY framework. Also updated the Documentation to include the binding information. The PHY binding information can be found at Documentation/devicetree/bindings/phy/phy-bindings

[PATCH v8 4/8] ARM: OMAP: USB: Add phy binding information

2013-06-24 Thread Kishon Vijay Abraham I
In order for controllers to get PHY in case of non dt boot, the phy binding information (phy device name) should be added in the platform data of the controller. Signed-off-by: Kishon Vijay Abraham I Reviewed-by: Sylwester Nawrocki --- arch/arm/mach-omap2/usb-musb.c |6 +- include/linux

[PATCH v8 1/8] drivers: phy: add generic PHY framework

2013-06-24 Thread Kishon Vijay Abraham I
The PHY framework provides a set of APIs for the PHY drivers to create/destroy a PHY and APIs for the PHY users to obtain a reference to the PHY with or without using phandle. For dt-boot, the PHY drivers should also register *PHY provider* with the framework. PHY drivers should create the PHY by

[PATCH V3 4/6] USB: OHCI: make ohci-spear a separate driver

2013-06-24 Thread Manjunath Goudar
Separate the ST OHCI SPEAr host controller driver from ohci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; it would be nice to have in 3.11. Signed-off-by: Manjunath Goudar Acked-by: Alan Stern Cc: Viresh Kumar Cc:

[PATCH V3 2/6] USB: OHCI: make ohci-omap a separate driver

2013-06-24 Thread Manjunath Goudar
Separate the TI OHCI OMAP1/2 host controller driver from ohci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; it would be nice to have in 3.11. Signed-off-by: Manjunath Goudar Cc: Felipe Balbi Cc: Arnd Bergmann Cc:

[PATCH V3 5/6] USB: OHCI: make ohci-at91 a separate driver

2013-06-24 Thread Manjunath Goudar
Separate the TI OHCI Atmel host controller driver from ohci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; it would be nice to have in 3.11. Signed-off-by: Manjunath Goudar Cc: Arnd Bergmann Cc: Alan Stern Cc: Gre

[PATCH V3 6/6] USB: OHCI: make ohci-s3c2410 a separate driver

2013-06-24 Thread Manjunath Goudar
Separate the Samsung OHCI S3C host controller driver from ohci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; it would be nice to have in 3.11. Signed-off-by: Manjunath Goudar Cc: Arnd Bergmann Cc: Alan Stern C

[PATCH V3 3/6] USB: OHCI: make ohci-omap3 a separate driver

2013-06-24 Thread Manjunath Goudar
Separate the TI OHCI OMAP3 host controller driver from ohci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; it would be nice to have in 3.11. Signed-off-by: Manjunath Goudar Cc: Anand Gadiyar Cc: Felipe Balbi Cc: A

[PATCH V3 0/6] USB: OHCI: more bus glues as separate modules

2013-06-24 Thread Manjunath Goudar
These patches are for separating the SOC On-Chip ohci host controller from ohci-hcd host code into its own driver module. This work is part of enabling multi-platform kernels on ARM; it would be nice to have in 3.11. V2: In patch 5/6 and 6/6: -Set non-standard fields in hc_driver manually, rat

[PATCH V3 1/6] USB: OHCI: make ohci-exynos a separate driver

2013-06-24 Thread Manjunath Goudar
Separate the Samsung OHCI EXYNOS host controller driver from ohci-hcd host code so that it can be built as a separate driver module. This work is part of enabling multi-platform kernels on ARM; it would be nice to have in 3.11. Signed-off-by: Manjunath Goudar Acked-by: Alan Stern Acked-by: Jing

[PATCH V4 05/11] USB: OHCI: Properly handle ohci-ep93xx suspend

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of ohci-ep93xx glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend() routine in ohci_hcd_ep93xx_drv_suspend() will ensure proper handling of suspend scenario. Signed-off-by: Manjunath Goudar Acked-by:

[PATCH V4 02/11] USB: OHCI: Properly handle ohci-at91 suspend

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of ohci-at91 glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend() routine in ohci_hcd_at91_drv_suspend() will ensure proper handling of suspend scenario. Signed-off-by: Manjunath Goudar Acked-by: Ala

[PATCH V4 01/11] USB: OHCI: Properly handle OHCI controller suspend

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of OHCI was not properly handled in ochi_suspend()routine. This does proper handling of suspend scenario. Signed-off-by: Manjunath Goudar Acked-by: Alan Stern Cc: Arnd Bergmann Cc: Greg KH Cc: linux-usb@vger.kernel.org V3: New patch. V4: No change. --- drivers/usb/h

[PATCH V4 04/11] USB: OHCI: Properly handle ohci-da8xx suspend

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of ohci-da8xx glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend() routine in ohci_da8xx_suspend() will ensure proper handling of suspend scenario. Signed-off-by: Manjunath Goudar Acked-by: Alan Ster

[PATCH V4 00/11] USB: OHCI:Properly handle ohci_suspend()routine in bus glue

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of ohci bus glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend()routine will ensure proper handling of suspend scenario. V2: -Incase ohci_suspend() fails, return right away without executing further.

[PATCH V4 03/11] USB: OHCI: Properly handle ohci-s3c2410 suspend

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of ohci-s3c2410 glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend() routine in ohci_hcd_s3c2410_drv_suspend() will ensure proper handling of suspend scenario. Signed-off-by: Manjunath Goudar Acked-b

[PATCH V4 07/11] USB: OHCI: Properly handle ohci-omap suspend

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of ohci-omap glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend() routine in ohci_omap_suspend() will ensure proper handling of suspend scenario. Signed-off-by: Manjunath Goudar Acked-by: Alan Stern

[PATCH V4 06/11] USB: OHCI: Properly handle ohci-exynos suspend

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of ohci-exynos glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend() routine in exynos_ohci_suspend() will ensure proper handling of suspend scenario. Signed-off-by: Manjunath Goudar Acked-by: Alan St

[PATCH V4 09/11] USB: OHCI: Properly handle ohci-pxa27x suspend

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of ohci-pxa27x glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend() routine in ohci_hcd_pxa27x_drv_suspend() will ensure proper handling of suspend scenario. Signed-off-by: Manjunath Goudar Acked-by:

[PATCH V4 11/11] USB: OHCI: Properly handle ohci-spear suspend

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of ohci-spear glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend() routine in spear_ohci_hcd_drv_suspend() will ensure proper handling of suspend scenario. Signed-off-by: Manjunath Goudar Acked-by: A

[PATCH V4 10/11] USB: OHCI: Properly handle ohci-sm501 suspend

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of ohci-sm501 glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend() routine in ohci_sm501_suspend() will ensure proper handling of suspend scenario. Signed-off-by: Manjunath Goudar Acked-by: Alan Ster

[PATCH V4 08/11] USB: OHCI: Properly handle ohci-platform suspend

2013-06-24 Thread Manjunath Goudar
Suspend scenario in case of ohci-platform glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend() routine in ohci_platform_suspend() will ensure proper handling of suspend scenario. Signed-off-by: Manjunath Goudar Acked-by: Ala

<    1   2