a small patch that fixes the ohci warn "irq nobody cared" on shutdown.

2013-07-26 Thread Caizhiyong
From: caizhiyong When ohci-hcd is shutting down, call ohci_usb_reset reset ohci-hcd, the root hub generate an interrupt, but ohci->rh_state is OHCI_RH_HALTED, and ohci_irq ignore the interrupt, the kernel trigger warning "irq nobody cared". ehci-hcd is first disable interrupts, then reset ehci

[PATCH] USB: ohci_usb warn "irq nobody cared" on shutdown

2013-07-26 Thread Caizhiyong
From: caizhiyong When ohci-hcd is shutting down, call ohci_usb_reset reset ohci-hcd, the root hub generate an interrupt, but ohci->rh_state is OHCI_RH_HALTED, and ohci_irq ignore the interrupt, the kernel trigger warning "irq nobody cared". ehci-hcd is first disable interrupts, then reset ehci

Re: [PATCH 1/6] usb: phy-nop: Use RESET Controller for managing the reset line

2013-07-26 Thread Roger Quadros
On 07/25/2013 08:28 PM, Felipe Balbi wrote: > On Thu, Jul 18, 2013 at 11:53:05AM +0300, Roger Quadros wrote: >> Till now we were modelling the RESET line as a voltage regulator and >> using the regulator framework to manage it. >> >> [1] introduces a GPIO based reset controller driver. We use that

Re: [PATCH] USB: serial: add driver for Suunto ANT+ USB device

2013-07-26 Thread Daniele Forsi
2013/7/26 Greg Kroah-Hartman: > + Say Y here if you want to usb the Suunto ANT+ USB device. I think there might be a thinko: s/to usb/to use/ -- Daniele Forsi -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org Mo

[PATCH 0/2] usb: fix controller-PHY binding for OMAP3 platform

2013-07-26 Thread Kishon Vijay Abraham I
After the platform devices are created using PLATFORM_DEVID_AUTO, the device names given in usb_bind_phy (in board file) does not match with the actual device name causing the USB PHY library not to return the PHY reference when the MUSB controller request for the PHY in the non-dt boot case. So re

[PATCH 2/2] arm: omap: remove *.auto* from device names given in usb_bind_phy

2013-07-26 Thread Kishon Vijay Abraham I
Previously MUSB wrapper (OMAP) device used PLATFORM_DEVID_AUTO while creating MUSB core device. So in usb_bind_phy (binds the controller with the PHY), the device name of the controller had *.auto* in it. Since with using PLATFORM_DEVID_AUTO, there is no way to know the exact device name in advance

[PATCH 1/2] usb: musb: omap: remove using PLATFORM_DEVID_AUTO in omap2430.c

2013-07-26 Thread Kishon Vijay Abraham I
After the platform devices are created using PLATFORM_DEVID_AUTO, the device names given in usb_bind_phy (in board file) does not match with the actual device name causing the USB PHY library not to return the PHY reference when the MUSB controller request for the PHY in the non-dt boot case. So re

[PATCH 06/16 => 2/2] usb: musb: dsps: remove EOI access

2013-07-26 Thread Sebastian Andrzej Siewior
The EOI register is not present in the AM335x memory space according to the TRM and thus removed. Should any platform using the EOI register get merged then it may be used again if the register address is not zero. Signed-off-by: Sebastian Andrzej Siewior --- drivers/usb/musb/musb_dsps.c | 15 ++

[PATCH 06/16 => 1/2] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-26 Thread Sebastian Andrzej Siewior
The ti81xx platform is not fully supported right now. This patch renames the date structure to a am33xx prefix which is actually used. Signed-off-by: Sebastian Andrzej Siewior --- drivers/usb/musb/musb_dsps.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb/mus

Re: Help adding trace events to xHCI

2013-07-26 Thread Johannes Berg
[sorry for the late response!] > Yes, that does look inefficient. Would it make sense to add a couple > different trace event classes for different sized buffers, and call > those trace classes conditionally, based on the size of the formatted > string? Or would that be too much of a mess. I do

[PATCH v13 02/14] usb: chipidea: imx: remove vbus regulator operation

2013-07-26 Thread Peter Chen
Since we have added vbus reguatlor operation at common host file (chipidea/host.c), the glue layer vbus operation isn't needed any more. Tested-by: Marek Vasut Signed-off-by: Peter Chen --- drivers/usb/chipidea/ci_hdrc_imx.c | 30 +++--- 1 files changed, 7 insertions(+

[PATCH v13 13/14] usb: chipidea: udc: .pullup is valid when vbus is on at CI_HDRC_PULLUP_ON_VBUS

2013-07-26 Thread Peter Chen
When the flag CI_HDRC_PULLUP_ON_VBUS is set, .pullup should only be called when the vbus is active. When the CI_HDRC_PULLUP_ON_VBUS is set, the controller only begins to run when the vbus is on, So, it is only meaningful software set pullup/pulldown after the controller begins to run. Tested-by: M

[PATCH v13 12/14] usb: chipidea: udc: misuse flag CI_HDRC_REGS_SHARED and CI_HDRC_PULLUP_ON_VBUS

2013-07-26 Thread Peter Chen
CI_HDRC_REGS_SHARED stands for the controller registers is shared with other USB drivers, if all USB drivers are at chipidea/, it doesn't needed to set. CI_HDRC_PULLUP_ON_VBUS stands for pullup dp when the vbus is on. This flag doesn't need to set if the vbus is always on for gadget since dp has al

Re: Help adding trace events to xHCI

2013-07-26 Thread Johannes Berg
On Thu, 2013-07-11 at 15:29 -0400, Steven Rostedt wrote: > > Note that there's no easy way to dynamically allocate the right amount > > of space in the ringbuffer, or at least I haven't found one. We > > therefore have a static size, which is somewhat inefficient. > > Can you add a helper functio

[PATCH v13 01/14] usb: chipidea: add vbus regulator control

2013-07-26 Thread Peter Chen
For boards which have board level vbus control (eg, through gpio), we need to vbus operation according to below rules: - For host, we need open vbus before start hcd, and close it after remove hcd. - For otg, the vbus needs to be on/off when usb role switches. When the host roles begins, it opens v

[PATCH v13 07/14] usb: chipidea: add flag CI_HDRC_DUAL_ROLE_NOT_OTG

2013-07-26 Thread Peter Chen
Since we need otgsc to know vbus's status at some chipidea controllers even it is peripheral-only mode. Besides, some SoCs (eg, AR9331 SoC) don't have otgsc register even the DCCPARAMS_DC and DCCPARAMS_HC are both 1 at CAP_DCCPARAMS. We inroduce flag CI_HDRC_DUAL_ROLE_NOT_OTG to indicate if the co

[PATCH v13 08/14] usb: chipidea: disable all interrupts and clear all interrupts status

2013-07-26 Thread Peter Chen
During the initialization, it needs to disable all interrupts enable bit as well as clear all interrupts status bits to avoid exceptional interrupt. Tested-by: Marek Vasut Signed-off-by: Peter Chen --- drivers/usb/chipidea/core.c | 11 ++- 1 files changed, 10 insertions(+), 1 deletion

[PATCH v13 10/14] usb: chipidea: add vbus interrupt handler

2013-07-26 Thread Peter Chen
We add vbus interrupt handler at ci_otg_work, it uses OTGSC_BSV(at otgsc) to know it is connect or disconnet event. Meanwhile, we introduce two flags id_event and b_sess_valid_event to indicate it is an id interrupt or a vbus interrupt. Tested-by: Marek Vasut Signed-off-by: Peter Chen --- drive

[PATCH v13 00/14] Add tested id switch and vbus connect detect support for Chipidea

2013-07-26 Thread Peter Chen
This patchset adds tested otg id switch function and vbus connect and disconnect detection for chipidea driver. And fix kinds of bugs found at chipidea drivers after enabling id and vbus detection. This patch are fully tested at imx6 sabresd and imx28evk platform by me. Besides, marek tested it o

[PATCH v13 04/14] usb: chipidea: udc: otg_set_peripheral is useless for some chipidea users

2013-07-26 Thread Peter Chen
It is useless at below cases: - If we implement both usb host and device at chipidea driver. - If we don't need phy->otg. Tested-by: Marek Vasut Signed-off-by: Peter Chen --- drivers/usb/chipidea/udc.c |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/drivers/usb/ch

[PATCH v13 09/14] usb: chipidea: move otg relate things to otg file

2013-07-26 Thread Peter Chen
Move otg relate things to otg file. Tested-by: Marek Vasut Signed-off-by: Peter Chen --- drivers/usb/chipidea/core.c | 63 +-- drivers/usb/chipidea/otg.c | 57 +- drivers/usb/chipidea/otg.h |2 + 3 files chang

[PATCH v13 14/14] usb: chipidea: udc: fix the oops when plugs in usb cable after rmmod gadget

2013-07-26 Thread Peter Chen
When we rmmod gadget, the ci->driver needs to be cleared. Otherwise, we plug in usb cable again, the driver will consider gadget is there, in fact, it was removed. Besides, consolidate the calling of ci->driver->disconnect, when we do rmmod gadget, the gadget's disconnect should be called from udc

[PATCH v13 05/14] usb: chipidea: otg: Add otg file used to access otgsc

2013-07-26 Thread Peter Chen
This file is mainly used to access otgsc currently, it may add otg related things in the future. Tested-by: Marek Vasut Signed-off-by: Peter Chen --- drivers/usb/chipidea/Makefile |2 +- drivers/usb/chipidea/bits.h | 10 drivers/usb/chipidea/core.c |3 +- drivers/usb/chip

[PATCH v13 06/14] usb: chipidea: Add role init and destory APIs

2013-07-26 Thread Peter Chen
- The role's init will be called at probe procedure. - The role's destory will be called at fail patch at probe and driver's removal. - The role's start/stop will be called when specific role has started. Tested-by: Marek Vasut Signed-off-by: Peter Chen --- drivers/usb/chipidea/core.c | 10 ++

[PATCH v13 11/14] usb: chipidea: add wait vbus lower than OTGSC_BSV before role starts

2013-07-26 Thread Peter Chen
When the gadget role starts, we need to make sure the vbus is lower than OTGSC_BSV, or there will be an vbus interrupt since we use B_SESSION_VALID as vbus interrupt to indicate connect and disconnect. When the host role starts, it may not be useful to wait vbus to lower than OTGSC_BSV, but it can

[PATCH v13 03/14] usb: chipidea: imx: add return value check for devm_regulator_get

2013-07-26 Thread Peter Chen
- If devm_regulator_get returns -EPROBE_DEFER, we also return -EPROBE_DEFER to wait regulator being ready later. - If devm_regulator_get returns -ENODEV, we think there is no "vbus-supply" node at DT, it means this board doesn't need vbus control. - If devm_regulator_get returns other error values,

Re: [PATCH 0/2] USBNET: max rx/tx qlen change

2013-07-26 Thread Oliver Neukum
On Thu, 2013-07-25 at 22:19 +0800, Ming Lei wrote: > Hi, > > On Thu, Jul 25, 2013 at 3:38 PM, Oliver Neukum wrote: > > On Thu, 2013-07-25 at 13:47 +0800, Ming Lei wrote: > >> Hi, > >> > >> There are two patches on computing max rx/tx qlen, and fix one > >> performance problem on USB3 NIC. > > > >

Re: [PATCH 15/16] dmaengine: add transfered member to dma_async_tx_descriptor

2013-07-26 Thread Sebastian Andrzej Siewior
On Thu, Jul 25, 2013 at 05:32:35PM +0200, Lars-Peter Clausen wrote: > > I see. But it can't be reused before calling the callback if it is > > going to call the callback, right? > > So if this is a no-no, I'm left with an additional argument to the > > complete callback? > > Hm, maybe using dmaeng

Re: [PATCH] usb: dwc3: core: modify IO memory resource after deferred probe completes

2013-07-26 Thread Felipe Balbi
Hi, On Fri, Jul 26, 2013 at 09:48:26AM +0300, Ivan T. Ivanov wrote: > > > Also, this is not *modifying* what was passed, just skipping the xHCI > > > address space so we don't request_mem_region() an area we won't really > > > handle and prevent xhci-hcd.ko from probing. > > > > Hmm? platform_get

Re: [PATCH] usb: gadget: at91_udc: Check gpio lookup results

2013-07-26 Thread Felipe Balbi
On Thu, Jul 25, 2013 at 01:59:51PM -0700, Olof Johansson wrote: > On Thu, Jul 25, 2013 at 10:19 AM, Felipe Balbi wrote: > > On Thu, Jul 25, 2013 at 09:18:39AM -0700, Olof Johansson wrote: > >> That's what the code does today, and I wasn't trying to second-guess > >> their decisions on that. Chance

[PATCH 0/3][RESEND] USB: mos7840: fixes for v3.11

2013-07-26 Thread Johan Hovold
Here's a resend of the three remaining mos7840-fixes for v3.11 rebased on usb-linus of today. Tested using a mcs7820-device (and by faking the mcs7810 LED). Johan Johan Hovold (3): USB: mos7840: fix race in register handling USB: mos7840: fix device-type detection USB: mos7840: fix race i

[PATCH 3/3] USB: mos7840: fix race in led handling

2013-07-26 Thread Johan Hovold
Fix race in LED handling introduced by commit 0eafe4de ("USB: serial: mos7840: add support for MCS7810 devices") which reused the port control urb for manipulating the LED without making sure that the urb is not already in use. This could lead to the control urb being manipulated while in flight.

[PATCH 1/3] USB: mos7840: fix race in register handling

2013-07-26 Thread Johan Hovold
Fix race in mos7840_get_reg which unconditionally manipulated the control urb (which may already be in use) by adding a control-urb busy flag. Cc: sta...@vger.kernel.org Signed-off-by: Johan Hovold --- drivers/usb/serial/mos7840.c | 18 -- 1 file changed, 16 insertions(+), 2 dele

[PATCH 2/3] USB: mos7840: fix device-type detection

2013-07-26 Thread Johan Hovold
Fix race in device-type detection introduced by commit 0eafe4de ("USB: serial: mos7840: add support for MCS7810 devices") which used a static variable to hold the device type. Move type detection to probe and use serial data to store the device type. Cc: sta...@vger.kernel.org Signed-off-by: Joha

Re: [PATCH] musb: don't reset endpoint data toggle on blackfin

2013-07-26 Thread Scott Jiang
Hi Balbi, > On Thu, Jul 18, 2013 at 07:04:14PM -0400, Scott Jiang wrote: >> Reset endpoint data toggle would lead to failure for musb >> RTL version 1.9 on blackfin. > > is this specific to musb 1.9 or to blackfin ? > I'm not sure about this. It failed in musb 1.9 on bf527. But this bug didn't aff

Re: [PATCH] musb: don't reset endpoint data toggle on blackfin

2013-07-26 Thread Felipe Balbi
Hi, On Fri, Jul 26, 2013 at 05:59:22PM +0800, Scott Jiang wrote: > Hi Balbi, > > > On Thu, Jul 18, 2013 at 07:04:14PM -0400, Scott Jiang wrote: > >> Reset endpoint data toggle would lead to failure for musb > >> RTL version 1.9 on blackfin. > > > > is this specific to musb 1.9 or to blackfin ? >

Re: [PATCH] USB: serial: add driver for Suunto ANT+ USB device

2013-07-26 Thread Steinar H. Gunderson
On Thu, Jul 25, 2013 at 09:52:29PM -0700, Greg Kroah-Hartman wrote: > Steinar, I've tested the driver below with my device and it seems to > work. If you have any problems with it, please let me know, otherwise > I'll queue it up to get into the 3.11 kernel release soon. Backported to 3.10.3 and

Re: Help adding trace events to xHCI

2013-07-26 Thread Steven Rostedt
On Fri, 2013-07-26 at 11:19 +0200, Johannes Berg wrote: > My original though here was that we should be able to reserve (maximum) > space on the per-CPU buffer, and then relinquish some of it after the > tracepoint was written to the data, but I never had the time to check if > that was possible t

Re: [PATCH v2 3/7] usb: phy: msm: Move regulator usage to managed resource allocation

2013-07-26 Thread Ivan T. Ivanov
Hi Felipe, On Thu, 2013-07-25 at 16:43 +0300, Ivan T. Ivanov wrote: > On Wed, 2013-07-24 at 15:39 +0300, Felipe Balbi wrote: > > On Tue, Jul 09, 2013 at 06:47:09PM +0300, Ivan T. Ivanov wrote: > > > From: "Ivan T. Ivanov" > > > > > > This patch move global regulators variables to driver state

[PATCH - bugfix for 3.11 2/2] usb/gadget: f_phonet: remove unused preprocessor conditional

2013-07-26 Thread Andrzej Pietrasiewicz
The compatibility layer which the USBF_PHONET_INCLUDED was a part of is no longer present - the USBF_PHONET_INCLUDED is not #defined by anyone anymore, so the ifndef is always true. Removing it. Signed-off-by: Andrzej Pietrasiewicz Signed-off-by: Kyungmin Park Acked-by: Michal Nazarewicz --- d

[PATCH] usb: xhci: add missing dma-mapping.h includes

2013-07-26 Thread James Hogan
A randconfig build hit the following build errors because xhci.c and xhci-mem.c use dma mapping functions but don't include . Add the missing includes to fix the build errors. drivers/usb/host/xhci.c In function 'xhci_gen_setup': drivers/usb/host/xhci.c +4872 : error: implicit declaration of funct

[PATCH - bugfix for 3.11 1/2] usb/gadget: multi: fix error return code in cdc_do_config()

2013-07-26 Thread Andrzej Pietrasiewicz
Fix to return a negative error code from the error handling case instead of 0, as returned elsewhere in this function. Introduced by commit 59835a (usb: gadget: multi: use function framework for ACM.) Make rndis_do_config() consistent with cdc_do_config() in the way it handles returning the PTR_E

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

2013-07-26 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 v10 0/8] PHY framework

2013-07-26 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 v10 3/8] usb: phy: twl4030: use the new generic PHY framework

2013-07-26 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. Also twl4030-usb driver is moved to drivers/ph

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

2013-07-26 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 v10 8/8] usb: phy: twl4030-usb: remove *set_suspend* and *phy_init* ops

2013-07-26 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 Acked-by: Felipe Balbi Reviewed-by: Sylwester Nawrocki --- drivers/phy/phy-twl4030-usb.c | 57 ++---

[PATCH v10 4/8] arm: omap3: twl: add phy consumer data in twl4030_usb_data

2013-07-26 Thread Kishon Vijay Abraham I
The PHY framework uses the phy consumer data populated in platform data in the case of non-dt boot to return the reference to the PHY when the controller (PHY consumer) requests for it. So populated the phy consumer data in the platform data of twl usb. Signed-off-by: Kishon Vijay Abraham I ---

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

2013-07-26 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 Acked-by: Felipe Balbi Reviewed-by: Sylwester Nawrocki --- drivers/phy/phy-omap-usb2.c | 25 - 1 file changed,

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

2013-07-26 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. The omap-usb2 driver is also moved to driver/phy. However using the old USB PHY library cannot be completely removed because OTG is intertwined wi

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

2013-07-26 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

RE: [PATCH v2 01/21] usb/gadget: multi: fix error return code in cdc_do_config()

2013-07-26 Thread Andrzej Pietrasiewicz
On Thursday, July 25, 2013 7:33 PM Felipe Balbi wrote: > On Fri, Jul 19, 2013 at 12:28:13PM +0200, Andrzej Pietrasiewicz wrote: > > Fix to return a negative error code from the error handling case > > instead of 0, as returned elsewhere in this function. > > > > Introduced by commit 59835a (usb: ga

[RESEND PATCH v10 4/8] arm: omap3: twl: add phy consumer data in twl4030_usb_data

2013-07-26 Thread Kishon Vijay Abraham I
The PHY framework uses the phy consumer data populated in platform data in the case of non-dt boot to return the reference to the PHY when the controller (PHY consumer) requests for it. So populated the phy consumer data in the platform data of twl usb. Signed-off-by: Kishon Vijay Abraham I ---

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

2013-07-26 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

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

2013-07-26 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 Acked-by: Felipe Balbi Reviewed-by: Sylwester Nawrocki --- drivers/phy/phy-omap-usb2.c | 25 - 1 file changed,

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

2013-07-26 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. Also twl4030-usb driver is moved to drivers/ph

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

2013-07-26 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 Acked-by: Felipe Balbi Reviewed-by: Sylwester Nawrocki --- drivers/phy/phy-twl4030-usb.c | 57 ++---

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

2013-07-26 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

[RESEND PATCH v10 0/8] PHY framework

2013-07-26 Thread Kishon Vijay Abraham I
Resending with fixed device tree mailing address! 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 use

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

2013-07-26 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. The omap-usb2 driver is also moved to driver/phy. However using the old USB PHY library cannot be completely removed because OTG is intertwined wi

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

2013-07-26 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

Re: Help adding trace events to xHCI

2013-07-26 Thread Johannes Berg
On Fri, 2013-07-26 at 08:28 -0400, Steven Rostedt wrote: > > My original though here was that we should be able to reserve (maximum) > > space on the per-CPU buffer, and then relinquish some of it after the > > tracepoint was written to the data, but I never had the time to check if > > that was p

Re: Help adding trace events to xHCI

2013-07-26 Thread Steven Rostedt
On Fri, 2013-07-26 at 15:06 +0200, Johannes Berg wrote: > On Fri, 2013-07-26 at 08:28 -0400, Steven Rostedt wrote: > Ah, yes, that'd work. I was considering putting it into the trace event > handling itself so I don't have to allocate those buffers and put the > handling into every tracepoint, but

Re: Help adding trace events to xHCI

2013-07-26 Thread Johannes Berg
On Fri, 2013-07-26 at 09:17 -0400, Steven Rostedt wrote: > On Fri, 2013-07-26 at 15:06 +0200, Johannes Berg wrote: > > On Fri, 2013-07-26 at 08:28 -0400, Steven Rostedt wrote: > > > Ah, yes, that'd work. I was considering putting it into the trace event > > handling itself so I don't have to alloc

Re: Help adding trace events to xHCI

2013-07-26 Thread Steven Rostedt
On Fri, 2013-07-26 at 15:45 +0200, Johannes Berg wrote: > Well, right now I can live with allocation 110 bytes for each > tracepoint, this would just optimise it down. If I was in the middle of > writing one such event while an interrupt came in, I'd not be able to > reduce the ring-buffer allocat

Re: [PATCH] USB: serial: add driver for Suunto ANT+ USB device

2013-07-26 Thread Alan Stern
On Thu, 25 Jul 2013, Greg Kroah-Hartman wrote: > From: Greg Kroah-Hartman > > This adds a driver for the Suunto ANT+ USB device, exposing it as a usb > serial device. This lets the userspace "gant" program to talk to the > device to communicate over the ANT+ protocol to any devices it finds. >

Re: [PATCH 35/50] media: usb: cx231xx: spin_lock in complete() cleanup

2013-07-26 Thread Hans Verkuil
On 07/11/2013 11:05 AM, Ming Lei wrote: > Complete() will be run with interrupt enabled, so change to > spin_lock_irqsave(). > > Cc: Mauro Carvalho Chehab > Cc: Hans Verkuil > Cc: linux-me...@vger.kernel.org > Signed-off-by: Ming Lei > --- > drivers/media/usb/cx231xx/cx231xx-audio.c |6 +

Re: [PATCH] USB: ohci_usb warn "irq nobody cared" on shutdown

2013-07-26 Thread Alan Stern
On Fri, 26 Jul 2013, Caizhiyong wrote: > From: caizhiyong > > When ohci-hcd is shutting down, call ohci_usb_reset reset ohci-hcd, > the root hub generate an interrupt, but ohci->rh_state is > OHCI_RH_HALTED, and ohci_irq ignore the interrupt, the kernel trigger > warning "irq nobody cared". ehci

Re: After entering D3 cold: URB ffff88020ea41000 submitted while active

2013-07-26 Thread Ming Lei
On Thu, Jul 25, 2013 at 10:30 PM, Oleksii Shevchuk wrote: > https://bugzilla.kernel.org/show_bug.cgi?id=60621 > > Randomly after returning from S3 or enabling powersaving this trace goes to > dmesg: > > [13108.297051] ehci-pci :00:1a.0: power state changed by ACPI to D3cold > [13115.552813] -

[PATCH V3 1/2] USB: OHCI: make ohci-ep93xx a separate driver

2013-07-26 Thread Manjunath Goudar
Separate the OHCI EP93XX 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. Signed-off-by: Manjunath Goudar Acked-by: Alan Stern Cc: Arnd Bergmann Cc: Greg KH Cc: linux-usb@vger.kernel

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

2013-07-26 Thread Manjunath Goudar
Separate the OHCI pxa27x/pxa3xx 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. Signed-off-by: Manjunath Goudar Cc: Arnd Bergmann Cc: Greg KH Cc: Alan Stern Cc: linux-usb@vger.kern

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

2013-07-26 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. In V2: patch 1/2: -ohci_hcd_init() statements are removed, because by default it is called in ohci_setu

Re: [PATCH] USB: serial: add driver for Suunto ANT+ USB device

2013-07-26 Thread Greg Kroah-Hartman
On Fri, Jul 26, 2013 at 10:27:51AM -0400, Alan Stern wrote: > On Thu, 25 Jul 2013, Greg Kroah-Hartman wrote: > > > From: Greg Kroah-Hartman > > > > This adds a driver for the Suunto ANT+ USB device, exposing it as a usb > > serial device. This lets the userspace "gant" program to talk to the >

Re: [PATCH] usb: udc: add gadget state kobject uevent

2013-07-26 Thread Greg KH
On Fri, Jul 26, 2013 at 01:56:19PM +0800, Barry Song wrote: > 2013/7/18 Felipe Balbi : > > On Thu, Jul 18, 2013 at 05:28:19PM +0800, Rong Wang wrote: > >> Hi Felipe, > >> > >> Thanks, I'll test the patch. > >> > >> But sysfs_notify(&gadget->dev.kobj, NULL, "status"), status or state ? > >> I notice

Re: [PATCH] usb: gadget: mv_u3d_core: fix violation of locking discipline in mv_u3d_ep_disable()

2013-07-26 Thread Alexey Khoroshilov
Hi Felipe, On 07/25/2013 09:30 PM, Felipe Balbi wrote: > On Wed, Jul 24, 2013 at 12:20:17AM +0400, Alexey Khoroshilov wrote: >> mv_u3d_nuke() expects to be calles with ep->u3d->lock held, >> because mv_u3d_done() does. But mv_u3d_ep_disable() calls it >> without lock that can lead to unpleasant co

Re: [PATCH 14/16] usb: musb: dsps: add MUSB_DEVCTL_SESSION back after removal

2013-07-26 Thread Sebastian Andrzej Siewior
* Bin Liu | 2013-07-23 13:55:27 [-0500]: >Hi Sebastian, Hi Bin, >> In my testing the bit remains set. How is the bit supposed to come back >> after I connect a host device? >The bit remains even when no device is plugged and ID ping is float? After I set it, it remains set yes. >what platform do

Re: [PATCH] usb: gadget: at91_udc: Check gpio lookup results

2013-07-26 Thread Olof Johansson
On Fri, Jul 26, 2013 at 2:54 AM, Felipe Balbi wrote: > On Thu, Jul 25, 2013 at 01:59:51PM -0700, Olof Johansson wrote: >> On Thu, Jul 25, 2013 at 10:19 AM, Felipe Balbi wrote: >> > On Thu, Jul 25, 2013 at 09:18:39AM -0700, Olof Johansson wrote: >> >> That's what the code does today, and I wasn't

Re: [PATCH 14/16] usb: musb: dsps: add MUSB_DEVCTL_SESSION back after removal

2013-07-26 Thread Sebastian Andrzej Siewior
* Bin Liu | 2013-07-23 13:55:27 [-0500]: >Hi Sebastian, Hi Bin, >either by SRP or HNP, or something else. In TI 3.2 kernel, there is >workaround in otg_timer() to _toggle_ the SESSION bit to detect if ID >pin is grounded, which means a USB device is connected. I've been looking at http://downl

Ejected Nook (usb mass storage) prevents suspend

2013-07-26 Thread Andy Lutomirski
This is kernel 3.9.9-302.fc19.x86_64. I plugged in a BN Nook (a usb mass storage device), used it, and ejected it. This makes suspend fail: [50135.265514] PM: Entering freeze sleep [50135.265517] Suspending console(s) (use no_console_suspend to debug) [50135.287724] sd 7:0:0:0: [sdb] Synchronizi

[GIT PATCH] USB fixes for 3.11-rc3

2013-07-26 Thread Greg KH
The following changes since commit ad81f0545ef01ea651886dddac4bef6cec930092: Linux 3.11-rc1 (2013-07-14 15:18:27 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git/ tags/usb-3.11-rc3 for you to fetch changes up to 2c7b871b9102c497ba8

Re: Ejected Nook (usb mass storage) prevents suspend

2013-07-26 Thread Alan Stern
On Fri, 26 Jul 2013, Andy Lutomirski wrote: > This is kernel 3.9.9-302.fc19.x86_64. > > I plugged in a BN Nook (a usb mass storage device), used it, and > ejected it. This makes suspend fail: > > [50135.265514] PM: Entering freeze sleep > [50135.265517] Suspending console(s) (use no_console_sus

Re: [PATCH 14/16] usb: musb: dsps: add MUSB_DEVCTL_SESSION back after removal

2013-07-26 Thread Bin Liu
Sebastian, On Fri, Jul 26, 2013 at 11:31 AM, Sebastian Andrzej Siewior wrote: > * Bin Liu | 2013-07-23 13:55:27 [-0500]: > >>Hi Sebastian, > Hi Bin, > >>either by SRP or HNP, or something else. In TI 3.2 kernel, there is >>workaround in otg_timer() to _toggle_ the SESSION bit to detect if ID >>pi

Re: [PATCH 14/16] usb: musb: dsps: add MUSB_DEVCTL_SESSION back after removal

2013-07-26 Thread Bin Liu
On Fri, Jul 26, 2013 at 12:53 PM, Bin Liu wrote: > Sebastian, > > On Fri, Jul 26, 2013 at 11:31 AM, Sebastian Andrzej Siewior > wrote: >> * Bin Liu | 2013-07-23 13:55:27 [-0500]: >> >>>Hi Sebastian, >> Hi Bin, >> >>>either by SRP or HNP, or something else. In TI 3.2 kernel, there is >>>workaround

Re: [PATCH 14/16] usb: musb: dsps: add MUSB_DEVCTL_SESSION back after removal

2013-07-26 Thread Sebastian Andrzej Siewior
On 07/26/2013 08:17 PM, Bin Liu wrote: Hi Bin. >> In TI 3.2 kernel, the otg_timer() toggles the SESSION bit in >> OTG_STATE_B_IDLE state. So whenever the ID pin is grounded by >> connecting a device, the controller will try go into host mode. >> But I have no idea why your board is in a_idle not

Re: [PATCH 14/16] usb: musb: dsps: add MUSB_DEVCTL_SESSION back after removal

2013-07-26 Thread Bin Liu
Sebastian, On Fri, Jul 26, 2013 at 1:22 PM, Sebastian Andrzej Siewior wrote: > On 07/26/2013 08:17 PM, Bin Liu wrote: > > Hi Bin. > >>> In TI 3.2 kernel, the otg_timer() toggles the SESSION bit in >>> OTG_STATE_B_IDLE state. So whenever the ID pin is grounded by >>> connecting a device, the contr

RE: [PATCH] usb: dwc3: core: modify IO memory resource after deferred probe completes

2013-07-26 Thread Paul Zimmerman
> From: Felipe Balbi [mailto:ba...@ti.com] > Sent: Friday, July 26, 2013 2:54 AM > > > > Also, this is not *modifying* what was passed, just skipping the xHCI > > > > address space so we don't request_mem_region() an area we won't really > > > > handle and prevent xhci-hcd.ko from probing. > > > >

Re: Audio I/O parameters

2013-07-26 Thread Alan Stern
On Fri, 26 Jul 2013, Clemens Ladisch wrote: > Alan Stern wrote: > > On Thu, 25 Jul 2013, James Stone wrote: > >> The only slight difference I can see is that maybe the 3.10 uses > >> slightly higher CPU load than 3.5 at the ridiculously low latency of > >> 64 frames/period duplex. > > > > With the

Re: [PATCH V3 2/2] USB: OHCI: make ohci-pxa27x a separate driver

2013-07-26 Thread Alan Stern
On Fri, 26 Jul 2013, Manjunath Goudar wrote: > Separate the OHCI pxa27x/pxa3xx 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. > > Signed-off-by: Manjunath Goudar > Cc: Arnd Be

Re: NULL pointer dereference in xhci_free_dev

2013-07-26 Thread Sarah Sharp
Heinz and Frantisek, I believe your issue should be fixed by this patch: http://marc.info/?l=linux-usb&m=137476546506888&w=2 Please let me know if it doesn't. Sarah Sharp On Sat, May 11, 2013 at 04:09:25PM +0200, Heinz Diehl wrote: > On 11.05.2013, Frantisek Hrbata wrote: > > > I haven't tri

Re: [PATCH] chipidea: core: Move hw_phymode_configure() into probe

2013-07-26 Thread Rtp
Fabio Estevam writes: > Currently hw_phymode_configure() is located inside hw_device_reset(), which is > only called by chipidea udc driver. > > When operating in host mode, we also need to call hw_phymode_configure() in > order to properly configure the PHY mode, so move this function into prob

Re: [PATCH 14/16] usb: musb: dsps: add MUSB_DEVCTL_SESSION back after removal

2013-07-26 Thread Sebastian Andrzej Siewior
On 07/26/2013 08:29 PM, Bin Liu wrote: Hi Bin, >> [ 97.570852] musb-hdrc musb-hdrc.1.auto: Poll devctl 19 (a_wait_bcon) > I don't think you are testing with the 3.2 kernel you referred from > the ti.com link you mentioned above. > 3.2 kernel should have no '.auto' at the end of the device name.

Re: [PATCH 14/16] usb: musb: dsps: add MUSB_DEVCTL_SESSION back after removal

2013-07-26 Thread Bin Liu
Sebastian, On Fri, Jul 26, 2013 at 2:55 PM, Sebastian Andrzej Siewior wrote: > The bit is removed in otg_timer() because the phy gets into > OTG_STATE_A_WAIT_BCON. > > Now, this is current HEAD + my stuff to get second port working. I > downloaded v3.2 and browsed as I was looking for the workaro

[PATCH 1/4] usb: phy: rename nop_usb_xceiv => usb_phy_gen_xceiv

2013-07-26 Thread Sebastian Andrzej Siewior
The "nop" driver isn't a do-nothing-stub but supports a couple functions like clock on/off or is able to use a voltage regulator. This patch simply renames the driver to "generic" since it is easy possible to extend it by a simple function istead of writing a complete driver. Signed-off-by: Sebast

[PATCH 3/4] usb: musb: dsps: remove the hardcoded phy pieces

2013-07-26 Thread Sebastian Andrzej Siewior
dsps uses a nop driver which is added in dsps itself and does the PHY on/off calls within dsps. Since those calls are now moved the nop driver itself, we can now request the phy proper phy and remove those calls. Currently only the first musb interface is used so we only add one phy node for now.

[PATCH 4/4] usb: musb: dsps: use proper child nodes

2013-07-26 Thread Sebastian Andrzej Siewior
This moves the two instances from the big node into two child nodes. The glue layer ontop does almost nothing. There is one devices containing the (2) phy, (2) usb and later the dma engine. The usb device is the "glue device" which contains the musb device as a child. This is what we do ever since.

am335x: support for multiple USB instances

2013-07-26 Thread Sebastian Andrzej Siewior
Hi, The last series has been rebased ontop Felipe's testing branch. It contains only the support for multiple devices. I decided to keep the temporary change in the nop (now generic) phy driver which should ease the transition. I still have the problem that the second USB port does not work after

[PATCH 2/4] usb: phy: add am335x pieces to generic phy

2013-07-26 Thread Sebastian Andrzej Siewior
This patch copies the phy support bits from dsps into the generic phy driver. Most code can be re-used except for the on/off. The additional am335x can be removed once we have a phy driver that does more than this. Signed-off-by: Sebastian Andrzej Siewior --- drivers/usb/phy/phy-generic.c | 116

Re: [PATCH v2 3/7] usb: phy: msm: Move regulator usage to managed resource allocation

2013-07-26 Thread Felipe Balbi
On Fri, Jul 26, 2013 at 03:31:34PM +0300, Ivan T. Ivanov wrote: > > Hi Felipe, > > On Thu, 2013-07-25 at 16:43 +0300, Ivan T. Ivanov wrote: > > On Wed, 2013-07-24 at 15:39 +0300, Felipe Balbi wrote: > > > On Tue, Jul 09, 2013 at 06:47:09PM +0300, Ivan T. Ivanov wrote: > > > > From: "Ivan T. Ivan

Re: [PATCH] usb: gadget: at91_udc: Check gpio lookup results

2013-07-26 Thread Felipe Balbi
On Fri, Jul 26, 2013 at 09:23:35AM -0700, Olof Johansson wrote: > On Fri, Jul 26, 2013 at 2:54 AM, Felipe Balbi wrote: > > On Thu, Jul 25, 2013 at 01:59:51PM -0700, Olof Johansson wrote: > >> On Thu, Jul 25, 2013 at 10:19 AM, Felipe Balbi wrote: > >> > On Thu, Jul 25, 2013 at 09:18:39AM -0700, Ol

Re: Ejected Nook (usb mass storage) prevents suspend

2013-07-26 Thread Alan Stern
On Fri, 26 Jul 2013, Andy Lutomirski wrote: > This is kernel 3.9.9-302.fc19.x86_64. > > I plugged in a BN Nook (a usb mass storage device), used it, and > ejected it. This makes suspend fail: > > [50135.265514] PM: Entering freeze sleep > [50135.265517] Suspending console(s) (use no_console_sus

  1   2   >