Re: [PATCH 1/1] usb: fix build error without CONFIG_USB_PHY

2013-06-19 Thread Felipe Balbi
On Wed, Jun 19, 2013 at 10:11:05AM +0800, Peter Chen wrote: on i386: drivers/built-in.o: In function `ci_hdrc_probe': core.c:(.text+0x20446b): undefined reference to `of_usb_get_phy_mode' Signed-off-by: Peter Chen peter.c...@freescale.com --- include/linux/usb/of.h | 14 +-

Re: [PATCH] build some drivers only when compile-testing

2013-06-19 Thread Jiri Slaby
On 06/18/2013 06:04 PM, Greg Kroah-Hartman wrote: So currently I have what is attached... Comments? Looks good to me, want me to queue it up through my char/misc driver tree for 3.11? If there are no objections... Whoever picks that up, I would be happy 8-). -- js suse labs -- To

Re: [PATCH 1/1] usb: fix build error without CONFIG_USB_PHY

2013-06-19 Thread Peter Chen
On Wed, Jun 19, 2013 at 09:25:27AM +0300, Felipe Balbi wrote: On Wed, Jun 19, 2013 at 10:11:05AM +0800, Peter Chen wrote: on i386: drivers/built-in.o: In function `ci_hdrc_probe': core.c:(.text+0x20446b): undefined reference to `of_usb_get_phy_mode' Signed-off-by: Peter Chen

Re: [PATCH] build some drivers only when compile-testing

2013-06-19 Thread Tomi Valkeinen
On 17/06/13 23:05, Jiri Slaby wrote: The last point I inclined to the Greg's argument to remove the EXPERT dependency. So currently I have what is attached... Comments? The patch looks a bit odd with the USB_CHIPIDEA_IMX parts. You're not adding COMPILE_TEST there, but you're adding a

Re: [PATCH] build some drivers only when compile-testing

2013-06-19 Thread Jiri Slaby
On 06/19/2013 09:10 AM, Tomi Valkeinen wrote: On 17/06/13 23:05, Jiri Slaby wrote: The last point I inclined to the Greg's argument to remove the EXPERT dependency. So currently I have what is attached... Comments? The patch looks a bit odd with the USB_CHIPIDEA_IMX parts. You're not

Re: Linux USB file storage gadget with new UDC

2013-06-19 Thread victor yeo
Hi, There is a mistake in the previous log file, because the fifo size is still set to 512 byte. Now i change it to 64 byte if it is Full speed. The FIFO size should always be set to the value in the endpoint descriptor, no matter what speed the connection is. The log file are attached.

Re: [PATCH] build some drivers only when compile-testing

2013-06-19 Thread Tomi Valkeinen
On 19/06/13 10:12, Jiri Slaby wrote: On 06/19/2013 09:10 AM, Tomi Valkeinen wrote: On 17/06/13 23:05, Jiri Slaby wrote: The last point I inclined to the Greg's argument to remove the EXPERT dependency. So currently I have what is attached... Comments? The patch looks a bit odd with the

[PATCH v2 1/1] usb: fix build error without CONFIG_USB_PHY

2013-06-19 Thread Peter Chen
on i386: drivers/built-in.o: In function `ci_hdrc_probe': core.c:(.text+0x20446b): undefined reference to `of_usb_get_phy_mode' Signed-off-by: Peter Chen peter.c...@freescale.com --- Changes for v2: - Using IS_ENABLED to MACRO define include/linux/usb/of.h | 16 ++-- 1 files

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

2013-06-19 Thread Roger Quadros
Hi Chao, On 06/19/2013 05:31 AM, Chao Xie wrote: Some controller need software to initialize PHY before add host controller, and shut down PHY after remove host controller. Add the generic code for these controllers so they do not need do it in its own host controller driver.

[PATCH 4/4] arm: omap: remove using usb_bind_phy for binding musb and phy

2013-06-19 Thread Kishon Vijay Abraham I
Now that MUSB for OMAP started using devm_usb_get_phy_by_name which does not require PHY library to already have the binding information, removed usb_bind_phy calls that binds the MUSB controller with the PHY from the board files. Signed-off-by: Kishon Vijay Abraham I kis...@ti.com Acked-by:

[PATCH 3/4] usb: musb: omap: use the new API to get PHY reference by label

2013-06-19 Thread Kishon Vijay Abraham I
After the devices are created using PLATFORM_DEVID_AUTO, devm_usb_get_phy_dev and usb_get_phy_dev can't be used reliably as it relies on the device_names passed in usb_bind_phy. So used the new API devm_usb_get_phy_by_name to get the PHY reference. Signed-off-by: Kishon Vijay Abraham I

[PATCH 2/4] usb: phy: add a new API to get PHY ref by label

2013-06-19 Thread Kishon Vijay Abraham I
After the devices are created using PLATFORM_DEVID_AUTO, devm_usb_get_phy_dev and usb_get_phy_dev can't be used reliably as it relies on the device_names passed in usb_bind_phy. So added a new API to get the PHY reference by PHY label (PHY label should be filled which creating the PHY).

[PATCH 0/4] usb: musb: fix USB enumeration issue in OMAP3 platform

2013-06-19 Thread Kishon Vijay Abraham I
In the case of non-dt boot, the platform specific initialization file (board file) will do usb_bind_phy that binds the usb controller with the PHY using device names. After the device names are created using PLATFORM_DEVID_AUTO, our original method of binding by device names doesn't work reliably

[PATCH 1/4] arm: omap: Add phy binding info for musb in plat data

2013-06-19 Thread Kishon Vijay Abraham I
In order for controllers to get PHY in case of non dt boot, the phy binding information (phy label) should be added in the platform data of the controller. Signed-off-by: Kishon Vijay Abraham I kis...@ti.com Acked-by: Felipe Balbi ba...@ti.com Tested-by: Tomi Valkeinen tomi.valkei...@ti.com ---

ATENÇÃO;

2013-06-19 Thread Administrador do Sistema
-- ATENÇÃO; Sua caixa de correio excedeu o limite de 5 GB de armazenamento, que é como definido pelo administrador, você está atualmente em execução no 10.9GB, você pode não ser capaz de enviar ou receber novas mensagens até que você re-validar a sua caixa de correio. Para revalidar sua

[PATCH 6/8] ehci-atmel.c: prepare clk before calling enable

2013-06-19 Thread Boris BREZILLON
Replace clk_enable/disable with clk_prepare_enable/disable_unprepare to avoid common clk framework warnings. Signed-off-by: Boris BREZILLON b.brezil...@overkiz.com --- drivers/usb/host/ehci-atmel.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH] usb: gadget: at91_udc: call at91udc_of_init only when needed

2013-06-19 Thread Arnd Bergmann
This avoids a build error in at91sam9261_9g10_defconfig: drivers/usb/gadget/at91_udc.c: In function 'at91udc_probe': drivers/usb/gadget/at91_udc.c:1685:34: warning: 'flags' may be used uninitialized in this function [-Wmaybe-uninitialized] board-vbus_active_low = (flags OF_GPIO_ACTIVE_LOW) ?

ATENÇÃO;

2013-06-19 Thread Administrador do Sistema
ATENÇÃO; Sua caixa de correio excedeu o limite de 5 GB de armazenamento, que é como definido pelo administrador, você está atualmente em execução no 10.9GB, você pode não ser capaz de enviar ou receber novas mensagens até que você re-validar a sua caixa de correio. Para revalidar sua caixa de

Re: [RFC PATCH v1 1/6] USB: HCD: support giveback of URB in tasklet context

2013-06-19 Thread Ming Lei
On Wed, Jun 19, 2013 at 10:59 AM, Ming Lei ming@canonical.com wrote: On Wed, Jun 19, 2013 at 12:05 AM, Alan Stern st...@rowland.harvard.edu wrote: On Tue, 18 Jun 2013, Ming Lei wrote: This patch implements the mechanism of giveback of URB in tasklet context, so that hardware interrupt

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

2013-06-19 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

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

2013-06-19 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

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

2013-06-19 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 manjunath.gou...@linaro.org Cc: Arnd Bergmann a...@arndb.de Cc: Alan Stern st...@rowland.harvard.edu Cc: Greg KH g...@kroah.com Cc:

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

2013-06-19 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

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

2013-06-19 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

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

2013-06-19 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

[PATCH 5/8] usb: gadget: at91_udc: prepare clk before calling enable

2013-06-19 Thread Boris BREZILLON
Replace clk_enable/disable with clk_prepare_enable/disable_unprepare to avoid common clk framework warnings. Signed-off-by: Boris BREZILLON b.brezil...@overkiz.com --- drivers/usb/gadget/at91_udc.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git

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

2013-06-19 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

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

2013-06-19 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

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

2013-06-19 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

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

2013-06-19 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

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

2013-06-19 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

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

2013-06-19 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

[PATCH] xhci: Add missing unlocks on error paths

2013-06-19 Thread Emil Goode
This patch adds missing unlocks on error paths in the xhci_free_streams and xhci_configure_endpoint functions. Signed-off-by: Emil Goode emilgo...@gmail.com --- drivers/usb/host/xhci.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c

[PATCH 1/1] net: add dm9620 net usb driver

2013-06-19 Thread Joseph CHANG
DM9620 is an USB2.0 network adapter rather than DM9601 USB1.1. This driver processed the RX data with 4 bytes header, TX 2 bytes header, make the control bit exactly right in PHY write operation, and optional IFF_ALLMULTI bit for RX control. Tested good for many platforms, include X86 desktop and

[PATCH 7/8] USB: ohci-at91: prepare clk before calling enable

2013-06-19 Thread Boris BREZILLON
Replace clk_enable/disable with clk_prepare_enable/disable_unprepare to avoid common clk framework warnings. Signed-off-by: Boris BREZILLON b.brezil...@overkiz.com --- drivers/usb/host/ohci-at91.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

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

2013-06-19 Thread Sergei Shtylyov
Hello. On 19-06-2013 16:12, Manjunath Goudar wrote: 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

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

2013-06-19 Thread Sergei Shtylyov
On 19-06-2013 16:12, Manjunath Goudar wrote: 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

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

2013-06-19 Thread Sergei Shtylyov
On 19-06-2013 16:12, Manjunath Goudar wrote: 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

Re: [PATCH 1/1] net: add dm9620 net usb driver

2013-06-19 Thread Andi Shyti
Hi Joseph, had a fast look... [ ... ] +static int dm9620_set_eeprom(struct net_device *net, + struct ethtool_eeprom *eeprom, u8 *data) +{ + struct usbnet *dev = netdev_priv(net); + int offset = eeprom-offset; + int len = eeprom-len; + int done; + + if

[RFC PATCH 5/6] ARM: dts: omap3beagle-xm: Add idle state pins for USB host

2013-06-19 Thread Roger Quadros
Add the Idle state pins for USB host and enable WAKEUP on DIR, DAT0-3, so that the PHY can wakeup the OMAP SoC from sleep on any USB activity (e.g. remote wakeup or connect/disconnect). CC: Benoît Cousson b-cous...@ti.com Signed-off-by: Roger Quadros rog...@ti.com ---

[RFC PATCH 6/6] ARM: OMAP3: Enable Hardware Save and Restore for USB Host

2013-06-19 Thread Roger Quadros
To ensure hardware context is restored while resuming from OFF mode we need to enable the Hardware SAR bit for the USB Host power domain. Signed-off-by: Roger Quadros rog...@ti.com --- arch/arm/mach-omap2/powerdomains3xxx_data.c |8 +--- 1 files changed, 1 insertions(+), 7 deletions(-)

[RFC PATCH 4/6] USB: ehci-omap: Suspend the controller during bus suspend

2013-06-19 Thread Roger Quadros
Runtime suspend the controller during bus suspend and resume it during bus resume. This will ensure that the USB Host power domain enters lower power state and does not prevent the SoC from endering deeper sleep states. Remote wakeup will come up as an interrupt while the controller is suspended,

[RFC PATCH 0/6] Suspend USB Host controller on bus suspend

2013-06-19 Thread Roger Quadros
Hi, This series attempts to suspend the OMAP EHCI host controller on USB Bus suspend. This will cause its parent, the OMAP USB Host Module as well as the USB TLL Module to be put in suspend and hence allow the USB power domain to be put in a lower power state. Then we no longer prevent the rest

[RFC PATCH 2/6] mfd: omap-usb-host: Put pins in IDLE state on suspend

2013-06-19 Thread Roger Quadros
In order to support wake up from suspend use the pinctrl framework to put the USB host pins in IDLE state during suspend. CC: Samuel Ortiz sa...@linux.intel.com Signed-off-by: Roger Quadros rog...@ti.com --- drivers/mfd/omap-usb-host.c | 46 +++ 1 files

[RFC PATCH 1/6] mfd: omap-usb-host: move initialization to module_init()

2013-06-19 Thread Roger Quadros
We no longer need to be initialized in any particular order so move driver initialization to the standard place i.e. module_init() CC: Samuel Ortiz sa...@linux.intel.com Signed-off-by: Roger Quadros rog...@ti.com --- drivers/mfd/omap-usb-host.c | 10 +- drivers/mfd/omap-usb-tll.c |

Re: [PATCH 0/7] Common Clock Framework support for Samsung S3C64xx

2013-06-19 Thread Kukjin Kim
On 06/06/13 08:57, Tomasz Figa wrote: This series is an attempt to move clock support on Samsung S3C64xx SoCs to Common Clock Framework. Looks good :) First, support for PLL types present on S3C64xx SoCs is added to Samsung Common Clock Framework driver. Then the main clock driver for

Re: [PATCH 0/7] Common Clock Framework support for Samsung S3C64xx

2013-06-19 Thread Tomasz Figa
On Wednesday 19 of June 2013 23:17:06 Kukjin Kim wrote: On 06/06/13 08:57, Tomasz Figa wrote: This series is an attempt to move clock support on Samsung S3C64xx SoCs to Common Clock Framework. Looks good :) Thanks. First, support for PLL types present on S3C64xx SoCs is added to

Re: [PATCH 2/7] clk: samsung: Add clock driver for S3C64xx SoCs

2013-06-19 Thread Kukjin Kim
On 06/13/13 06:38, Tomasz Figa wrote: [...] Another thing is that it's unlikely for any new SoC from S3C64xx series to show up, so basically the clock list is fixed. Sure. I can take this into clk-next along with patch #1, or if you prefer: Acked-by: Mike Turquettemturque...@linaro.org

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

2013-06-19 Thread Viresh Kumar
On 19 June 2013 20:12, Manjunath Goudar manjunath.gou...@linaro.org wrote: On 19 June 2013 18:13, Sergei Shtylyov sergei.shtyl...@cogentembedded.com On 19-06-2013 16:12, Manjunath Goudar wrote: struct usb_hcd *hcd = platform_get_drvdata(pdev); struct ohci_hcd *ohci =

Re: [RFC PATCH 0/1] Intel xhci: rework EHCI/xHCI port switching

2013-06-19 Thread Mathias Nyman
On 06/18/2013 05:39 PM, Alan Stern wrote: On Mon, 17 Jun 2013, Sarah Sharp wrote: Correct me if I'm wrong here. The original idea was to switch everything over to xHCI during some early stage (like when the xHCI controller is first passed to the pci-quirks.c code) and switch back to EHCI at

Re: [Bug] USB 2.0 Ports Dont Work on Sony Vaio Laptop

2013-06-19 Thread Alan Stern
On Wed, 19 Jun 2013, Ming Lei wrote: Hi, Recently, there is one bug report from Ubuntu community: USB 2.0 Ports Dont Work on Sony Vaio Laptop and the problem exists on upstream kernel too. The built-in two USB 2.0 devices can be recognized correctly, but external devices can't

Re: [RFC PATCH 0/6] Suspend USB Host controller on bus suspend

2013-06-19 Thread Alan Stern
On Wed, 19 Jun 2013, Roger Quadros wrote: Hi, This series attempts to suspend the OMAP EHCI host controller on USB Bus suspend. Why do you want to suspend the host controller during bus suspend? They are two different operations and should be carried out at two different times. The

Re: [Bug] USB 2.0 Ports Dont Work on Sony Vaio Laptop

2013-06-19 Thread Ming Lei
Hi, Thanks for your reply. On Wed, Jun 19, 2013 at 11:17 PM, Alan Stern st...@rowland.harvard.edu wrote: On Wed, 19 Jun 2013, Ming Lei wrote: Hi, Recently, there is one bug report from Ubuntu community: USB 2.0 Ports Dont Work on Sony Vaio Laptop and the problem exists on upstream

Re: [RFC PATCH v1 2/6] USB: disable IRQs deliberately when calling complete()

2013-06-19 Thread Alan Stern
On Wed, 19 Jun 2013, Ming Lei wrote: On Wed, Jun 19, 2013 at 12:36 AM, Alan Stern st...@rowland.harvard.edu wrote: On Tue, 18 Jun 2013, Ming Lei wrote: We disable local IRQs here in case of running complete() by tasklet to avoid possible deadlock because drivers may call spin_lock()

[PATCH 4/8] usb: musb: provide empty dma_controller_create() in PIO mode

2013-06-19 Thread Sebastian Andrzej Siewior
Add a dma_controller_create() returning NULL so a few ifdefs can dropped. Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de --- drivers/usb/musb/musb_dma.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/usb/musb/musb_dma.h b/drivers/usb/musb/musb_dma.h

Re: [RFC PATCH v1 1/6] USB: HCD: support giveback of URB in tasklet context

2013-06-19 Thread Alan Stern
On Wed, 19 Jun 2013, Ming Lei wrote: @@ -835,9 +839,11 @@ static int usb_rh_urb_dequeue(struct usb_hcd *hcd, struct urb *urb, int status) hcd-status_urb = NULL; usb_hcd_unlink_urb_from_ep(hcd, urb); -

[PATCH 2/8] usb: musb: musbhsdma: drop the controller check in dma_controller_destroy()

2013-06-19 Thread Sebastian Andrzej Siewior
This check is hardly required and alas is wrong. 'c' might be NULL but the chances are low that 'controller' after the container_of() becomes NULL. Since no other DMA implementation is doing that and musb-core does not call it with a NULL pointer it can dropped. Signed-off-by: Sebastian Andrzej

random musb bits, mostly dma, nothing serious

2013-06-19 Thread Sebastian Andrzej Siewior
Hi Felipe, here is a bunch of small musb patches. Sebastian -- To unsubscribe from this list: send the line unsubscribe linux-usb in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH 1/8] usb: musb: replace ifndef with ifdef for CONFIG_MUSB_PIO_ONLY

2013-06-19 Thread Sebastian Andrzej Siewior
The ifdef reads somehow better than an ifndef Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de --- drivers/usb/musb/musb_dma.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/usb/musb/musb_dma.h b/drivers/usb/musb/musb_dma.h index 1b6b827..8919ce2

[PATCH 6/8] usb: musb: core: call dma_controller_destroy() in the err path

2013-06-19 Thread Sebastian Andrzej Siewior
The cleanup in the error is missing the dma controller. The structure is allocated at runtime and ux500 allocates even a little more than just this struct. So cleanup! Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de --- drivers/usb/musb/musb_core.c | 2 ++ 1 file changed, 2

[PATCH 5/8] usb: musb: remove a few is_dma_capable() in init/exit code

2013-06-19 Thread Sebastian Andrzej Siewior
This patch removes is_dma_capable() and an ifdef in the init/exit path around init/de-init of the dma_controller. Since we have the empty stubs in the PIO code we can call it without gcc trouble. Earlier we had an ifdef and the is_dma_capable() macro where gcc ignored the if (0) path even that the

[PATCH 7/8] usb: musb: unmap reqs in musb_gadget_queue()'s error case

2013-06-19 Thread Sebastian Andrzej Siewior
If the descriptor is missing the reqeust is never unmapped. This patch changes this and renames the cleanup label to unlock since there is no cleanup done. The cleanup would revert the allocation of ressource (i.e. this dma mapping) but it does not, it simply unlocks and returns. Signed-off-by:

[PATCH 3/8] usb: musb: dma: merge -start/stop into create/destroy

2013-06-19 Thread Sebastian Andrzej Siewior
The core code creates a controller and immediately after that it calls the -start() callback. This one might drop an error but nobody cares. The same thing happens in the destroy corner: First -stop() called followed by destroy callback. So why not merge those two into the same function since

[PATCH 8/8] usb: musb: omap2430: make it compile again

2013-06-19 Thread Sebastian Andrzej Siewior
it does not compile since 09fc7d (usb: musb: fix incorrect usage of resource pointer). What makes me wonder most is if source of the Tested-by tag :) Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de --- drivers/usb/musb/omap2430.c | 4 ++-- 1 file changed, 2 insertions(+), 2

Re: [RFC PATCH v1 4/6] USB: EHCI: don't release ehci-lock if URB giveback in tasklet context

2013-06-19 Thread Alan Stern
On Wed, 19 Jun 2013, Ming Lei wrote: On Wed, Jun 19, 2013 at 12:43 AM, Alan Stern st...@rowland.harvard.edu wrote: On Tue, 18 Jun 2013, Ming Lei wrote: If HCD_BH is set for HC driver's flags, URB giveback will be done in tasklet context instead of interrupt context, so the ehci-lock

Re: [PATCH v7 1/9] drivers: phy: add generic PHY framework

2013-06-19 Thread Sylwester Nawrocki
Hi, On 06/13/2013 10:43 AM, Kishon Vijay Abraham I wrote: +/** + * phy_create() - create a new phy + * @dev: device that is creating the new phy + * @id: id of the phy + * @ops: function pointers for performing phy operations + * @label: label given to the phy + * @priv: private data from

Re: [Bug] USB 2.0 Ports Dont Work on Sony Vaio Laptop

2013-06-19 Thread Alan Stern
On Wed, 19 Jun 2013, Ming Lei wrote: There's a good chance this problem was caused by a change in the xhci-hcd driver. I am wondering why xhci-hcd may cause the problem since the affected hub is 'Intel Corp. Integrated Rate Matching Hub' which is connected to EHCI root hub. It's just a

Re: [RFC PATCH v1 5/6] USB: EHCI: improve interrupt qh unlink

2013-06-19 Thread Alan Stern
On Wed, 19 Jun 2013, Ming Lei wrote: The approach used in this patch is wrong. You shouldn't start the unlink, then wait, then finish the unlink. Consider what would happen What you mentioned above is just what the patch is doing, :-) start_unlink_intr() only sets the qh as

Re: [PATCH] build some drivers only when compile-testing

2013-06-19 Thread Mark Brown
On Tue, Jun 18, 2013 at 11:34:26AM +0300, Felipe Balbi wrote: MUSB alone has 8 different arch choices. Before, it used to be that core driver was dependendent on all of them, so whenever someone wanted to build MUSB for another arch, they had to introdude their glue code and modify the

Re: [RFC PATCH 2/6] mfd: omap-usb-host: Put pins in IDLE state on suspend

2013-06-19 Thread Kevin Hilman
Hi Roger, Roger Quadros rog...@ti.com writes: In order to support wake up from suspend use the pinctrl framework to put the USB host pins in IDLE state during suspend. CC: Samuel Ortiz sa...@linux.intel.com Signed-off-by: Roger Quadros rog...@ti.com You should use helpers for this now in

Re: [PATCH 1/2] musb: musb: dsps: support multiple instances

2013-06-19 Thread Sebastian Andrzej Siewior
On 06/18/2013 10:27 AM, Felipe Balbi wrote: --- a/arch/arm/boot/dts/am33xx.dtsi +++ b/arch/arm/boot/dts/am33xx.dtsi @@ -341,6 +341,14 @@ port1-mode = 3; power = 250; ti,hwmods = usb_otg_hs; + phys = nopphy0 nopphy1; + }; + + nopphy0: usbphy@0 { +

Re: [RFC PATCH 6/6] ARM: OMAP3: Enable Hardware Save and Restore for USB Host

2013-06-19 Thread Sergei Shtylyov
Hello. On 06/19/2013 06:05 PM, Roger Quadros wrote: To ensure hardware context is restored while resuming from OFF mode we need to enable the Hardware SAR bit for the USB Host power domain. Signed-off-by: Roger Quadros rog...@ti.com --- arch/arm/mach-omap2/powerdomains3xxx_data.c |8

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

2013-06-19 Thread Sergei Shtylyov
Hello. On 06/19/2013 06:42 PM, Manjunath Goudar wrote: 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()

Re: [RFC PATCH 4/6] USB: ehci-omap: Suspend the controller during bus suspend

2013-06-19 Thread Kevin Hilman
Hi Roger, Roger Quadros rog...@ti.com writes: Runtime suspend the controller during bus suspend and resume it during bus resume. This will ensure that the USB Host power domain enters lower power state and does not prevent the SoC from endering deeper sleep states. Remote wakeup will come

Re: [RFC PATCH 5/6] ARM: dts: omap3beagle-xm: Add idle state pins for USB host

2013-06-19 Thread Kevin Hilman
Roger Quadros rog...@ti.com writes: Add the Idle state pins for USB host and enable WAKEUP on DIR, DAT0-3, so that the PHY can wakeup the OMAP SoC from sleep on any USB activity (e.g. remote wakeup or connect/disconnect). CC: Benoît Cousson b-cous...@ti.com Signed-off-by: Roger Quadros

Hardware bug in Intel USB-2 hub?

2013-06-19 Thread Alan Stern
Sarah: This report surfaced in Bugzilla #59011 (see especially comments #38 and #39). Toralf reports, among other things, that the integrated rate-matching hub in his ThinkPad T420 (6 Series/C200 Series chipset) isn't behaving the way it should. The particular symptom is that when the hub is

Section mismatch in drivers/usb/gadget/fotg210-udc.c

2013-06-19 Thread Paul Gortmaker
Hi guys, This was seen on a linux-next (Jun18th tree) allmodconfig build: WARNING: drivers/usb/gadget/fotg210-udc.o(.data+0x0): Section mismatch in reference from the variable fotg210_driver to the function .init.text:fotg210_udc_probe() The variable fotg210_driver references the function

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

2013-06-19 Thread Alan Stern
On Wed, 12 Jun 2013, Manjunath Goudar wrote: 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. V2: -Set

question about USB interface probe order and synchronization

2013-06-19 Thread Thomas Pugliese
Hi, I am attempting to debug a problem where the hwa_hc module occasionally fails to start correctly when an HWA device is plugged in. An HWA device consists of two USB interfaces each with its own driver: the radio control interface (hwa_rc.ko), and the host controller interface (hwa_hc.ko).

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

2013-06-19 Thread Alan Stern
On Wed, 12 Jun 2013, Manjunath Goudar wrote: 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. V2: -Set

Re: question about USB interface probe order and synchronization

2013-06-19 Thread Greg KH
On Wed, Jun 19, 2013 at 03:33:51PM -0500, Thomas Pugliese wrote: Hi, I am attempting to debug a problem where the hwa_hc module occasionally fails to start correctly when an HWA device is plugged in. An HWA device consists of two USB interfaces each with its own driver: the radio control

[PATCH -next] usb: phy: tegra: remove duplicated include from phy-tegra-usb.c

2013-06-19 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn --- drivers/usb/phy/phy-tegra-usb.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/usb/phy/phy-tegra-usb.c b/drivers/usb/phy/phy-tegra-usb.c index

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

2013-06-19 Thread Chao Xie
On Wed, Jun 19, 2013 at 3:51 PM, Roger Quadros rog...@ti.com wrote: Hi Chao, On 06/19/2013 05:31 AM, Chao Xie wrote: Some controller need software to initialize PHY before add host controller, and shut down PHY after remove host controller. Add the generic code for these controllers so they

Re: [RFC PATCH v1 1/6] USB: HCD: support giveback of URB in tasklet context

2013-06-19 Thread Ming Lei
On Wed, Jun 19, 2013 at 11:37 PM, Alan Stern st...@rowland.harvard.edu wrote: On Wed, 19 Jun 2013, Ming Lei wrote: @@ -835,9 +839,11 @@ static int usb_rh_urb_dequeue(struct usb_hcd *hcd, struct urb *urb, int status) hcd-status_urb = NULL;

Re: [RFC PATCH v1 4/6] USB: EHCI: don't release ehci-lock if URB giveback in tasklet context

2013-06-19 Thread Ming Lei
On Wed, Jun 19, 2013 at 11:47 PM, Alan Stern st...@rowland.harvard.edu wrote: On Wed, 19 Jun 2013, Ming Lei wrote: On Wed, Jun 19, 2013 at 12:43 AM, Alan Stern st...@rowland.harvard.edu wrote: On Tue, 18 Jun 2013, Ming Lei wrote: If HCD_BH is set for HC driver's flags, URB giveback will

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

2013-06-19 Thread Jingoo Han
On Thursday, June 13, 2013 12:54 AM, Manjunath Goudar wrote: 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.

Re: [PATCH v2 1/1] usb: fix build error without CONFIG_USB_PHY

2013-06-19 Thread Randy Dunlap
On 06/19/13 00:20, Peter Chen wrote: on i386: drivers/built-in.o: In function `ci_hdrc_probe': core.c:(.text+0x20446b): undefined reference to `of_usb_get_phy_mode' Signed-off-by: Peter Chen peter.c...@freescale.com Reported-by: Randy Dunlap rdun...@infradead.org Acked-by: Randy Dunlap

Re: [PATCH v7 1/9] drivers: phy: add generic PHY framework

2013-06-19 Thread Kishon Vijay Abraham I
Hi, On Wednesday 19 June 2013 09:19 PM, Sylwester Nawrocki wrote: Hi, On 06/13/2013 10:43 AM, Kishon Vijay Abraham I wrote: +/** + * phy_create() - create a new phy + * @dev: device that is creating the new phy + * @id: id of the phy + * @ops: function pointers for performing phy operations +

Re: Detecting start/stop streaming for UVC webcam with bulk transfer mode

2013-06-19 Thread Chetan Nanda
Hi Laurent, Thanks for your mail, On Tue, Jun 18, 2013 at 6:12 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Chetan, On Tuesday 18 June 2013 11:17:40 Chetan Nanda wrote: Hi, I am currently working with UVC camera device which send data using bulk transfer for preview