Re: [PATCH] Fix switched around host, otg in tegra20-usb-phy documentation

2013-07-29 Thread Mikko Perttunen
On 07/30/2013 09:44 AM, Felipe Balbi wrote: * PGP Signed by an unknown key On Tue, Jul 30, 2013 at 09:38:44AM +0300, Mikko Perttunen wrote: Signed-off-by: Mikko Perttunen return -ENOLOG; sorry, I can't apply without a commit log. Please mention that the previous commit had a typo which you'

[PATCH] usb: tegra: Fix typo in tegra20-usb-phy documentation

2013-07-29 Thread Mikko Perttunen
The device tree binding documentation for tegra20-usb-phy used to claim that the vbus-supply property is required for host phys and optional for otg phys. This should be the other way around. Signed-off-by: Mikko Perttunen --- Documentation/devicetree/bindings/usb/nvidia,tegra20-usb-phy.txt | 2

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

2013-07-29 Thread Kishon Vijay Abraham I
Hi, On Tuesday 30 July 2013 11:58 AM, Felipe Balbi wrote: > Hi, > > On Tue, Jul 30, 2013 at 11:55:04AM +0530, Kishon Vijay Abraham I wrote: >>> On Tue, Jul 30, 2013 at 11:41:23AM +0530, Kishon Vijay Abraham I wrote: >> diff --git a/arch/arm/mach-omap2/board-2430sdp.c >> b/arch/ar

Re: [PATCH] Fix switched around host, otg in tegra20-usb-phy documentation

2013-07-29 Thread Felipe Balbi
On Tue, Jul 30, 2013 at 09:38:44AM +0300, Mikko Perttunen wrote: > Signed-off-by: Mikko Perttunen return -ENOLOG; sorry, I can't apply without a commit log. Please mention that the previous commit had a typo which you're fixing. -- balbi signature.asc Description: Digital signature

Re: [alsa-devel] Buffer size for ALSA USB PCM audio

2013-07-29 Thread Takashi Iwai
At Mon, 29 Jul 2013 21:23:15 +0200, Daniel Mack wrote: > > On 29.07.2013 20:20, Alan Stern wrote: > > data_ep_set_params() checks snd_usb_endpoint_implicit_feedback_sink() > > in three places. It looks like only the second place is correct. > > > > Can you verify whether the other two are right

[PATCH] Fix switched around host, otg in tegra20-usb-phy documentation

2013-07-29 Thread Mikko Perttunen
Signed-off-by: Mikko Perttunen --- Documentation/devicetree/bindings/usb/nvidia,tegra20-usb-phy.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/usb/nvidia,tegra20-usb-phy.txt b/Documentation/devicetree/bindings/usb/nvidia,tegra20-usb-phy.

Re: [PATCH v2 2/3] Documentation: devicetree: phy-tegra-usb: Add vbus-supply property for host mode PHYs

2013-07-29 Thread Felipe Balbi
On Tue, Jul 30, 2013 at 09:10:12AM +0300, Mikko Perttunen wrote: > On 07/30/2013 12:23 AM, Felipe Balbi wrote: > >* PGP Signed by an unknown key > > > >Hi, > > > >On Tue, Jul 30, 2013 at 12:19:32AM +0300, Felipe Balbi wrote: > >>On Mon, Jul 29, 2013 at 09:24:46AM -0600, Stephen Warren wrote: > >>>O

Re: [PATCH] usb: gadget: f_rndis: fix error return code in rndis_bind()

2013-07-29 Thread Felipe Balbi
On Tue, Jul 30, 2013 at 07:54:02AM +0800, Wei Yongjun wrote: > From: Wei Yongjun > > Fix to return -EINVAL in the vendor param set error handling > case instead of 0, as done elsewhere in this function. > > Signed-off-by: Wei Yongjun this bug is present in v3.5+, do we need to send this to sta

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

2013-07-29 Thread Felipe Balbi
Hi, On Tue, Jul 30, 2013 at 11:55:04AM +0530, Kishon Vijay Abraham I wrote: > > On Tue, Jul 30, 2013 at 11:41:23AM +0530, Kishon Vijay Abraham I wrote: > diff --git a/arch/arm/mach-omap2/board-2430sdp.c > b/arch/arm/mach-omap2/board-2430sdp.c > index 244d8a5..17bb076 10

Re: [PATCH v3] usb: phy-samsung-usb: Simplify PMU register handling

2013-07-29 Thread Felipe Balbi
On Mon, Jul 29, 2013 at 02:44:13PM -0700, Julius Werner wrote: > This patch simplifies the way the phy-samsung-usb code finds the correct > power management register to enable PHY clock gating. Previously, the > code would calculate the register address from a device tree supplied > base address an

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

2013-07-29 Thread Kishon Vijay Abraham I
Hi, On Tuesday 30 July 2013 11:48 AM, Felipe Balbi wrote: > Hi, > > On Tue, Jul 30, 2013 at 11:41:23AM +0530, Kishon Vijay Abraham I wrote: diff --git a/arch/arm/mach-omap2/board-2430sdp.c b/arch/arm/mach-omap2/board-2430sdp.c index 244d8a5..17bb076 100644 ---

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

2013-07-29 Thread Felipe Balbi
Hi, On Tue, Jul 30, 2013 at 11:41:23AM +0530, Kishon Vijay Abraham I wrote: > >> diff --git a/arch/arm/mach-omap2/board-2430sdp.c > >> b/arch/arm/mach-omap2/board-2430sdp.c > >> index 244d8a5..17bb076 100644 > >> --- a/arch/arm/mach-omap2/board-2430sdp.c > >> +++ b/arch/arm/ma

[PATCH] usb: of: fix build breakage caused by recent patches

2013-07-29 Thread Felipe Balbi
commit 052a11d (usb: phy: make PHY driver selection possible by controller drivers) changed the rules on how drivers/usb/phy/of.c would be compiled and failed to update include/linux/usb/of.h accordingly. Because of that, we can fall into situations where of_usb_get_phy_mode() is redefined. In ord

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

2013-07-29 Thread Kishon Vijay Abraham I
Hi, On Tuesday 30 July 2013 11:31 AM, Felipe Balbi wrote: > Hi, > > On Tue, Jul 30, 2013 at 10:44:48AM +0530, Kishon Vijay Abraham I wrote: >>> On Mon, Jul 29, 2013 at 08:59:26PM +0530, Kishon Vijay Abraham I wrote: >> Previously MUSB wrapper (OMAP) device used PLATFORM_DEVID_AUTO while

Re: [PATCH v2 2/3] Documentation: devicetree: phy-tegra-usb: Add vbus-supply property for host mode PHYs

2013-07-29 Thread Mikko Perttunen
On 07/30/2013 12:23 AM, Felipe Balbi wrote: * PGP Signed by an unknown key Hi, On Tue, Jul 30, 2013 at 12:19:32AM +0300, Felipe Balbi wrote: On Mon, Jul 29, 2013 at 09:24:46AM -0600, Stephen Warren wrote: On 06/28/2013 06:33 AM, Mikko Perttunen wrote: Add vbus-supply as an optional property

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

2013-07-29 Thread Felipe Balbi
Hi, On Tue, Jul 30, 2013 at 10:44:48AM +0530, Kishon Vijay Abraham I wrote: > > On Mon, Jul 29, 2013 at 08:59:26PM +0530, Kishon Vijay Abraham I wrote: > Previously MUSB wrapper (OMAP) device used PLATFORM_DEVID_AUTO while > creating > MUSB core device. So in usb_bind_phy (binds th

Re: [PATCH v2 2/4] phy: phy-amxxxx-usb: Add PHY driver for amxxxx platform

2013-07-29 Thread George Cherian
On 7/29/2013 8:47 PM, Sebastian Andrzej Siewior wrote: * George Cherian | 2013-07-19 18:04:35 [+0530]: Adds phy driver support for am33xx platform, the host/device peripheral controller shall get this phy object to control the phy operations. If you rebase this on-top of the two instances patc

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

2013-07-29 Thread Kishon Vijay Abraham I
Hi, On Monday 29 July 2013 11:24 PM, Felipe Balbi wrote: > Hi, > > On Mon, Jul 29, 2013 at 08:59:26PM +0530, Kishon Vijay Abraham I wrote: Previously MUSB wrapper (OMAP) device used PLATFORM_DEVID_AUTO while creating MUSB core device. So in usb_bind_phy (binds the controller with

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

2013-07-29 Thread Kishon Vijay Abraham I
Hi, On Monday 29 July 2013 09:21 PM, Sylwester Nawrocki wrote: > On 07/26/2013 02:49 PM, Kishon Vijay Abraham I wrote: >> 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 pha

Re: [PATCH v2 1/4] usb: phy: phy-omap-control: Add API to power and wakeup

2013-07-29 Thread George Cherian
On 7/29/2013 7:55 PM, Sebastian Andrzej Siewior wrote: * George Cherian | 2013-07-19 18:04:34 [+0530]: diff --git a/drivers/usb/phy/phy-omap-control.c b/drivers/usb/phy/phy-omap-control.c index 1419ced..4f2502c 100644 --- a/drivers/usb/phy/phy-omap-control.c +++ b/drivers/usb/phy/phy-omap-cont

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

2013-07-29 Thread Peter Chen
On Tue, Jul 30, 2013 at 12:47:17AM +0200, Michael Grzeschik wrote: > On Fri, Jul 26, 2013 at 05:18:19PM +0800, Peter Chen wrote: > > - 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 t

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

2013-07-29 Thread Peter Chen
On Tue, Jul 30, 2013 at 12:30:25AM +0200, Michael Grzeschik wrote: > Hi Peter, > > On Fri, Jul 26, 2013 at 05:18:18PM +0800, Peter Chen wrote: > > Since we have added vbus reguatlor operation at common > > host file (chipidea/host.c), the glue layer vbus operation > > isn't needed any more. > > >

Re: [PATCH] usb: xhci: Fix Command Ring Stopped Event handling

2013-07-29 Thread Yang Bai
Hi there, We met a panic issue with a 3.5-based kernel, code at git://kernel.ubuntu.com/ubuntu/ubuntu-quantal.git. call trace as: [ 27.508480] BUG: unable to handle kernel NULL pointer dereference at 03c8 [ 27.544645] Call Trace: [ 27.545060] [ 27.545353] [] handle_set_deq_

Re: Panasonic DMC-ZS3 mass storage USB regression in 3.10.3

2013-07-29 Thread Marc Meledandri
On Mon, Jul 29, 2013 at 11:58 AM, Alan Stern wrote: > On Mon, 29 Jul 2013, Marc Meledandri wrote: > >> On Mon, Jul 29, 2013 at 11:24 AM, Alan Stern >> wrote: >> > On Sun, 28 Jul 2013, Marc Meledandri wrote: >> > >> >> I've hit a USB regression from 3.10.2 (working) -> 3.10.3 (busted) with my >>

[PATCH] usb: gadget: f_rndis: fix error return code in rndis_bind()

2013-07-29 Thread Wei Yongjun
From: Wei Yongjun Fix to return -EINVAL in the vendor param set error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/usb/gadget/f_rndis.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/usb/gadget/f_rndis.c

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

2013-07-29 Thread Michael Grzeschik
On Fri, Jul 26, 2013 at 05:18:19PM +0800, Peter Chen wrote: > - 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 ne

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

2013-07-29 Thread Michael Grzeschik
Hi Peter, On Fri, Jul 26, 2013 at 05:18:18PM +0800, Peter Chen wrote: > 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/chipid

RE: USB Interrupt Transaction Scheduling

2013-07-29 Thread Stoddard, Nate (GE Healthcare)
> -Original Message- > > 1. Does this number take the USB requirement that "no more than 90% > > of any frame be allocated for periodic full-speed transfers" into > > account? > > No. You can tell because the table says that there would be 37 bytes > remaining, which is less than 10%

RE: USB Interrupt Transaction Scheduling

2013-07-29 Thread Stoddard, Nate (GE Healthcare)
> > Becides just testing to see if we do follow these things, what is your end > goal > here? Are you trying to determine how many devices you should be able to > successfully plug into a machine and have them work properly? > > Or something else? In our first release, we are considering 8 de

[PATCH v3] usb: phy-samsung-usb: Simplify PMU register handling

2013-07-29 Thread Julius Werner
This patch simplifies the way the phy-samsung-usb code finds the correct power management register to enable PHY clock gating. Previously, the code would calculate the register address from a device tree supplied base address and add an offset based on the PHY type. Since every PHY has its own dev

Re: [PATCH v2 2/3] Documentation: devicetree: phy-tegra-usb: Add vbus-supply property for host mode PHYs

2013-07-29 Thread Stephen Warren
On 07/29/2013 03:23 PM, Felipe Balbi wrote: > Hi, > > On Tue, Jul 30, 2013 at 12:19:32AM +0300, Felipe Balbi wrote: >> On Mon, Jul 29, 2013 at 09:24:46AM -0600, Stephen Warren wrote: >>> On 06/28/2013 06:33 AM, Mikko Perttunen wrote: Add vbus-supply as an optional property for host mode

Re: [PATCH v2 2/3] Documentation: devicetree: phy-tegra-usb: Add vbus-supply property for host mode PHYs

2013-07-29 Thread Felipe Balbi
Hi, On Tue, Jul 30, 2013 at 12:19:32AM +0300, Felipe Balbi wrote: > On Mon, Jul 29, 2013 at 09:24:46AM -0600, Stephen Warren wrote: > > On 06/28/2013 06:33 AM, Mikko Perttunen wrote: > > > Add vbus-supply as an optional property for host mode phy-tegra-usb PHYs. > > > > Felipe, I see that you did

Re: [PATCH v2 2/3] Documentation: devicetree: phy-tegra-usb: Add vbus-supply property for host mode PHYs

2013-07-29 Thread Felipe Balbi
On Mon, Jul 29, 2013 at 09:24:46AM -0600, Stephen Warren wrote: > On 06/28/2013 06:33 AM, Mikko Perttunen wrote: > > Add vbus-supply as an optional property for host mode phy-tegra-usb PHYs. > > Felipe, I see that you didn't merge this one patch from this series. Was > there a reason for that? At

Re: USB Interrupt Transaction Scheduling

2013-07-29 Thread Alan Stern
On Mon, 29 Jul 2013, Stoddard, Nate (GE Healthcare) wrote: > According to the USB 2.0 specification, 19 full speed interrupt > endpoints (@ 64 bytes) can be scheduled. I have a couple of > questions about this specification and the Linux kernel's > implementation: > > 1. Does this number take t

Re: USB Interrupt Transaction Scheduling

2013-07-29 Thread Greg KH
On Mon, Jul 29, 2013 at 08:43:01PM +, Stoddard, Nate (GE Healthcare) wrote: > According to the USB 2.0 specification, 19 full speed interrupt endpoints (@ > 64 bytes) can be scheduled. I have a couple of questions about this > specification and the Linux kernel's implementation: > > 1. Doe

Re: Audio I/O parameters

2013-07-29 Thread James Stone
On Mon, Jul 29, 2013 at 9:41 PM, James Stone wrote: > On Mon, Jul 29, 2013 at 4:25 PM, Alan Stern wrote: >> On Sun, 28 Jul 2013, James Stone wrote: >> >>> On Sat, Jul 27, 2013 at 6:45 PM, Alan Stern >>> wrote: >>> > On Sat, 27 Jul 2013, James Stone wrote: >>> > >>> >> OK. So this seems to have

USB Interrupt Transaction Scheduling

2013-07-29 Thread Stoddard, Nate (GE Healthcare)
According to the USB 2.0 specification, 19 full speed interrupt endpoints (@ 64 bytes) can be scheduled. I have a couple of questions about this specification and the Linux kernel's implementation: 1. Does this number take the USB requirement that "no more than 90% of any frame be allocated

Re: Audio I/O parameters

2013-07-29 Thread James Stone
On Mon, Jul 29, 2013 at 4:25 PM, Alan Stern wrote: > On Sun, 28 Jul 2013, James Stone wrote: > >> On Sat, Jul 27, 2013 at 6:45 PM, Alan Stern >> wrote: >> > On Sat, 27 Jul 2013, James Stone wrote: >> > >> >> OK. So this seems to have solved the starting jack at low latencies >> >> problem, but I

Re: [PATCH 2/3] include: Convert ethernet mac address declarations to use ETH_ALEN

2013-07-29 Thread Rafael J. Wysocki
On Monday, July 29, 2013 12:34:24 PM Joe Perches wrote: > On Mon, 2013-07-29 at 13:59 +0200, Rafael J. Wysocki wrote: > > On Sunday, July 28, 2013 10:29:04 PM Joe Perches wrote: > > > It's convenient to have ethernet mac addresses use > > > ETH_ALEN to be able to grep for them a bit easier and > >

Re: Buffer size for ALSA USB PCM audio

2013-07-29 Thread Alan Stern
On Mon, 29 Jul 2013, Daniel Mack wrote: > On 29.07.2013 20:20, Alan Stern wrote: > > data_ep_set_params() checks snd_usb_endpoint_implicit_feedback_sink() > > in three places. It looks like only the second place is correct. > > > > Can you verify whether the other two are right, and explain to

Re: [PATCH 2/3] include: Convert ethernet mac address declarations to use ETH_ALEN

2013-07-29 Thread Joe Perches
On Mon, 2013-07-29 at 13:59 +0200, Rafael J. Wysocki wrote: > On Sunday, July 28, 2013 10:29:04 PM Joe Perches wrote: > > It's convenient to have ethernet mac addresses use > > ETH_ALEN to be able to grep for them a bit easier and > > also to ensure that the addresses are __aligned(2). [] > > diff

Re: Buffer size for ALSA USB PCM audio

2013-07-29 Thread Daniel Mack
On 29.07.2013 20:20, Alan Stern wrote: > data_ep_set_params() checks snd_usb_endpoint_implicit_feedback_sink() > in three places. It looks like only the second place is correct. > > Can you verify whether the other two are right, and explain to me if > they are? Which version are you looking a

Re: [PATCH v2 1/6] usb: phy: msm: Move mach depndend code to platform data

2013-07-29 Thread Ivan T. Ivanov
Hi, > > Cc: Bryan Huntsman > > Cc: Felipe Balbi > > Cc: Greg Kroah-Hartman > > Cc: linux-arm-ker...@lists.infradead.org > > Cc: linux-ker...@vger.kernel.org > > Cc: linux-usb@vger.kernel.org > > > > Signed-off-by: Ivan T. Ivanov > > --- > > arch/arm/mach-msm/board-msm7x30.c | 35 +

Re: [RFC] xhci: add trace for debug messages related to changing contexts

2013-07-29 Thread Sarah Sharp
Ah, looking over the patch again, there's a couple print statements related to configure endpoint and evaluate context changes that you missed. Are you familiar with completions? If not, I suggest you read the "Completions" section of Chapter 5 of Linux Device Drivers: http://lwn.net/Kernel/LDD3/

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

2013-07-29 Thread Bin Liu
Sebastian, On Mon, Jul 29, 2013 at 12:51 PM, Sebastian Andrzej Siewior wrote: > On 07/29/2013 07:26 PM, Bin Liu wrote: >> Sebastian, > > Hi Bin, > >> I did not say AM335xEVM is not able to run OTG mode. The problem is in >> OTG mode the SESSION bit will be cleared once the device is unplugged, >>

Re: Buffer size for ALSA USB PCM audio

2013-07-29 Thread Alan Stern
On Mon, 29 Jul 2013, Clemens Ladisch wrote: > Alan Stern wrote: > > Clemens remarked some time ago that keeping the queue full would be > > trivial, if only he knew how full it needed to be. The answer to that > > is given above. I have been trying to make the appropriate changes, > > but I'm no

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

2013-07-29 Thread Felipe Balbi
Hi, On Mon, Jul 29, 2013 at 08:59:26PM +0530, Kishon Vijay Abraham I wrote: > >> 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 *.aut

Re: [PATCH 0/2] usb: chipidea: fixes for v3.11-rc3

2013-07-29 Thread Greg KH
On Mon, Jul 29, 2013 at 01:09:55PM +0300, Alexander Shishkin wrote: > Hi, > > Here are two small chipidea fixes for v3.11. Thanks, now applied. greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo in

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

2013-07-29 Thread Sebastian Andrzej Siewior
On 07/29/2013 07:26 PM, Bin Liu wrote: > Sebastian, Hi Bin, > I did not say AM335xEVM is not able to run OTG mode. The problem is in > OTG mode the SESSION bit will be cleared once the device is unplugged, > then there is no way the SESSION bit will come back in the current > mainline kernel. Ye

Re: [PATCH v2 1/6] usb: phy: msm: Move mach depndend code to platform data

2013-07-29 Thread David Brown
On Mon, Jul 29, 2013 at 10:04:19AM +0300, Ivan T. Ivanov wrote: From: "Ivan T. Ivanov" This patch fix compilation error and is an intermediate step before the addition of DeviceTree support for newer targets. Fix suggested here: https://lkml.org/lkml/2013/6/19/381 Cc: David Brown Cc: Daniel W

Re: [RFC] xhci: add trace for debug messages related to changing contexts

2013-07-29 Thread Sarah Sharp
Hi Xenia! On Tue, Jul 23, 2013 at 06:14:55AM +0300, Xenia Ragiadakou wrote: > This patch defines a new trace event, which is called xhci_dbg_context_change > and belongs in the event class xhci_log_msg, and adds tracepoints for tracing > the debug messages related to context updates performed with

Re: [PATCH] pl2303: fix the upper baud rate limit check for type_0/1 chips

2013-07-29 Thread Sergei Shtylyov
Hello. On 07/29/2013 08:33 PM, Frank Schäfer wrote: Fixes the following regression that has been introduced recently with commit b2d6d98fc7: Please also specify that commit's summary line in parens. With type_0 and type_1 chips - all baud rates < 1228800 baud are rounded up to 1228800 b

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

2013-07-29 Thread Bin Liu
Sebastian, On Mon, Jul 29, 2013 at 11:53 AM, Sebastian Andrzej Siewior wrote: > So you say, am335x-evm is not able to run OTG mode and may only run in > host mode and as a consequence it must not call musb_platform_try_idle() > because it throws that one bit away and there is no way to bring it b

Re: Buffer size for ALSA USB PCM audio

2013-07-29 Thread Clemens Ladisch
Alan Stern wrote: > Clemens remarked some time ago that keeping the queue full would be > trivial, if only he knew how full it needed to be. The answer to that > is given above. I have been trying to make the appropriate changes, > but I'm not finding it "trivial". What I meant was that it would

Re: External HDD does not work with 3.11-rc2

2013-07-29 Thread Alan Stern
On Thu, 25 Jul 2013, Philipp Dreimann wrote: > On Wed, Jul 24, 2013 at 11:03 PM, Alan Stern > wrote: > > On Wed, 24 Jul 2013, Philipp Dreimann wrote: > > > >> Hello, > >> > >> one of my external HDDs does not work using 3.11-rc2. The drive works > >> using 3.10, and 3.9, except for a suspend/res

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

2013-07-29 Thread Sebastian Andrzej Siewior
* Bin Liu | 2013-07-26 22:07:14 [-0500]: >Sebastian, Hi Bin, >I have not tested it yet, but I believe I found why host mode works on >TI 3.2 kernel but not on mainline. Please look at Line 786 in 3.2 >kernel musb_core.c [1]. > >773 if ((int_usb & MUSB_INTR_DISCONNECT) && !musb->ignore_dis

Re: Kernel 3.10.3 "reset SuperSpeed USB device number 2 using xhci_hcd"

2013-07-29 Thread Stuart Foster
On 07/29/13 16:23, Alan Stern wrote: On Mon, 29 Jul 2013, Martin K. Petersen wrote: "Alan" == Alan Stern writes: Alan, Alan> It was introduced by commit 98dcc2946adb (SCSI: sd: Update WRITE Alan> SAME heuristics). This commit adds a call to scsi_get_vpd_page() Alan> in sd_read_write_same()

[PATCH] pl2303: fix the upper baud rate limit check for type_0/1 chips

2013-07-29 Thread Frank Schäfer
Fixes the following regression that has been introduced recently with commit b2d6d98fc7: With type_0 and type_1 chips - all baud rates < 1228800 baud are rounded up to 1228800 baud - the device silently runs at 9600 baud for all baud rates > 1228800 baud Signed-off-by: Frank Schäfer --- driver

Re: Panasonic DMC-ZS3 mass storage USB regression in 3.10.3

2013-07-29 Thread Alan Stern
On Mon, 29 Jul 2013, Marc Meledandri wrote: > On Mon, Jul 29, 2013 at 11:24 AM, Alan Stern > wrote: > > On Sun, 28 Jul 2013, Marc Meledandri wrote: > > > >> I've hit a USB regression from 3.10.2 (working) -> 3.10.3 (busted) with my > >> Panasonic DMC-ZS3 camera. I can no longer mount the mass st

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

2013-07-29 Thread Sebastian Andrzej Siewior
* Felipe Balbi | 2013-07-29 15:22:18 [+0300]: >On Fri, Jul 26, 2013 at 10:15:55PM +0200, Sebastian Andrzej Siewior wrote: >> 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

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

2013-07-29 Thread Sylwester Nawrocki
On 07/26/2013 02:49 PM, Kishon Vijay Abraham I wrote: > 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 provid

Re: Panasonic DMC-ZS3 mass storage USB regression in 3.10.3

2013-07-29 Thread Marc Meledandri
On Mon, Jul 29, 2013 at 11:24 AM, Alan Stern wrote: > On Sun, 28 Jul 2013, Marc Meledandri wrote: > >> I've hit a USB regression from 3.10.2 (working) -> 3.10.3 (busted) with my >> Panasonic DMC-ZS3 camera. I can no longer mount the mass storage device. > >> [ 197.976105] sd 6:0:0:0: [sdf] Write

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

2013-07-29 Thread Kishon Vijay Abraham I
On Monday 29 July 2013 08:58 PM, Kamil Debski wrote: > Hi Kishon, > > A small fix follows inline. > >> From: linux-media-ow...@vger.kernel.org [mailto:linux-media- >> ow...@vger.kernel.org] On Behalf Of Kishon Vijay Abraham I >> Sent: Friday, July 26, 2013 2:49 PM >> >> The PHY framework provides

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

2013-07-29 Thread Kishon Vijay Abraham I
Hi, On Monday 29 July 2013 08:36 PM, Felipe Balbi wrote: > Hi, > > On Fri, Jul 26, 2013 at 02:33:38PM +0530, Kishon Vijay Abraham I wrote: >> 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),

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

2013-07-29 Thread Kamil Debski
Hi Kishon, A small fix follows inline. > From: linux-media-ow...@vger.kernel.org [mailto:linux-media- > ow...@vger.kernel.org] On Behalf Of Kishon Vijay Abraham I > Sent: Friday, July 26, 2013 2:49 PM > > The PHY framework provides a set of APIs for the PHY drivers to > create/destroy a PHY and

Re: Kernel 3.10.3 "reset SuperSpeed USB device number 2 using xhci_hcd"

2013-07-29 Thread Martin K. Petersen
> "Alan" == Alan Stern writes: Alan> In the case of these bug reports, the scsi_report_opcode() call Alan> returns -EINVAL. Oh, right. Because I actually do an explicit SCSI version check in scsi_report_opcode(). That's OK, then... -- Martin K. Petersen Oracle Linux Engineering -- To

Re: Audio I/O parameters

2013-07-29 Thread Alan Stern
On Sun, 28 Jul 2013, James Stone wrote: > On Sat, Jul 27, 2013 at 6:45 PM, Alan Stern wrote: > > On Sat, 27 Jul 2013, James Stone wrote: > > > >> OK. So this seems to have solved the starting jack at low latencies > >> problem, but I am still getting sporadic cannot submit urb (err = -18) > >> un

Re: Panasonic DMC-ZS3 mass storage USB regression in 3.10.3

2013-07-29 Thread Alan Stern
On Sun, 28 Jul 2013, Marc Meledandri wrote: > I've hit a USB regression from 3.10.2 (working) -> 3.10.3 (busted) with my > Panasonic DMC-ZS3 camera. I can no longer mount the mass storage device. > [ 197.976105] sd 6:0:0:0: [sdf] Write Protect is off > [ 197.976114] sd 6:0:0:0: [sdf] Mode Sense

Re: [PATCH v2 2/3] Documentation: devicetree: phy-tegra-usb: Add vbus-supply property for host mode PHYs

2013-07-29 Thread Stephen Warren
On 06/28/2013 06:33 AM, Mikko Perttunen wrote: > Add vbus-supply as an optional property for host mode phy-tegra-usb PHYs. Felipe, I see that you didn't merge this one patch from this series. Was there a reason for that? At this stage, I would consider it reasonable for DT binding documentation ch

Re: Kernel 3.10.3 "reset SuperSpeed USB device number 2 using xhci_hcd"

2013-07-29 Thread Alan Stern
On Mon, 29 Jul 2013, Martin K. Petersen wrote: > > "Alan" == Alan Stern writes: > > Alan, > > Alan> It was introduced by commit 98dcc2946adb (SCSI: sd: Update WRITE > Alan> SAME heuristics). This commit adds a call to scsi_get_vpd_page() > Alan> in sd_read_write_same() without first checki

Re: [PATCH v2 2/4] phy: phy-amxxxx-usb: Add PHY driver for amxxxx platform

2013-07-29 Thread Sebastian Andrzej Siewior
* George Cherian | 2013-07-19 18:04:35 [+0530]: >Adds phy driver support for am33xx platform, the host/device >peripheral controller shall get this phy object to control the phy >operations. If you rebase this on-top of the two instances patches I've sent earlier then you can bury patch 3 and 4,

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

2013-07-29 Thread Felipe Balbi
Hi, On Fri, Jul 26, 2013 at 02:33:38PM +0530, Kishon Vijay Abraham I wrote: > 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

Re: Kernel 3.10.3 "reset SuperSpeed USB device number 2 using xhci_hcd"

2013-07-29 Thread Martin K. Petersen
> "Alan" == Alan Stern writes: Alan, Alan> It was introduced by commit 98dcc2946adb (SCSI: sd: Update WRITE Alan> SAME heuristics). This commit adds a call to scsi_get_vpd_page() Alan> in sd_read_write_same() without first checking Alan> sd_try_extended_inquiry(). As noted in the latter ro

Re: Buffer size for ALSA USB PCM audio

2013-07-29 Thread Alan Stern
On Mon, 29 Jul 2013, Takashi Iwai wrote: > > The existing code has a problem: Under some conditions the URB queue > > will be too short. EHCI requires at least 10 microframes on the queue > > at all times (even when an URB is completing and is therefore no longer > > on the queue) to avoid underr

Re: [GIT PULL] usb: fixes for v3.11-rc3

2013-07-29 Thread Greg KH
On Mon, Jul 29, 2013 at 03:17:58PM +0300, Felipe Balbi wrote: > Hi Greg, > > Here's my second set of fixes. Nothing major, mostly > related to recent conversion to configfs on the gadget > framework. > > Let me know if you want me to change anything. > > The following changes since commit 690c70

Re: [PATCH v2 1/4] usb: phy: phy-omap-control: Add API to power and wakeup

2013-07-29 Thread Sebastian Andrzej Siewior
* George Cherian | 2013-07-19 18:04:34 [+0530]: >diff --git a/drivers/usb/phy/phy-omap-control.c >b/drivers/usb/phy/phy-omap-control.c >index 1419ced..4f2502c 100644 >--- a/drivers/usb/phy/phy-omap-control.c >+++ b/drivers/usb/phy/phy-omap-control.c >@@ -46,6 +46,73 @@ struct device *omap_get_con

Re: Ejected Nook (usb mass storage) prevents suspend

2013-07-29 Thread Alan Stern
On Mon, 29 Jul 2013, Oliver Neukum wrote: > On Fri, 2013-07-26 at 16:31 -0400, Alan Stern wrote: > > > In addition to my earlier comment, the patch below should be applied. > > It will fix your immediate problem, although not in the best way. > > Alan, > > I think your diagnosis is correct, b

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

2013-07-29 Thread Felipe Balbi
On Mon, Jul 29, 2013 at 04:52:12PM +0300, Felipe Balbi wrote: > On Mon, Jul 29, 2013 at 05:15:58PM +0400, Alexey Khoroshilov wrote: > > On 07/29/2013 04:52 PM, Felipe Balbi wrote: > > > Hi, > > > > > > On Fri, Jul 26, 2013 at 07:26:05PM +0400, Alexey Khoroshilov wrote: > > >> On 07/25/2013 09:30 PM

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

2013-07-29 Thread Felipe Balbi
On Mon, Jul 29, 2013 at 05:15:58PM +0400, Alexey Khoroshilov wrote: > On 07/29/2013 04:52 PM, Felipe Balbi wrote: > > Hi, > > > > On Fri, Jul 26, 2013 at 07:26:05PM +0400, Alexey Khoroshilov wrote: > >> On 07/25/2013 09:30 PM, Felipe Balbi wrote: > >>> On Wed, Jul 24, 2013 at 12:20:17AM +0400, Alex

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

2013-07-29 Thread Alexey Khoroshilov
On 07/29/2013 04:52 PM, Felipe Balbi wrote: > Hi, > > On Fri, Jul 26, 2013 at 07:26:05PM +0400, Alexey Khoroshilov wrote: >> 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

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

2013-07-29 Thread Felipe Balbi
Hi, On Fri, Jul 26, 2013 at 07:26:05PM +0400, Alexey Khoroshilov wrote: > 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

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

2013-07-29 Thread Sebastian Andrzej Siewior
On 07/29/2013 01:39 PM, Vinod Koul wrote: >> @@ -416,6 +418,7 @@ struct dma_async_tx_descriptor { >> dma_cookie_t (*tx_submit)(struct dma_async_tx_descriptor *tx); >> dma_async_tx_callback callback; >> void *callback_param; >> +unsigned int transfered; > I think this should be do

Re: [RFC] ux500 dma & short transfers on MUSB

2013-07-29 Thread Sebastian Andrzej Siewior
On 07/29/2013 01:44 PM, Vinod Koul wrote: > Please look into the residue field > > /** > * struct dma_tx_state - filled in to report the status of > * a transfer. > * @last: last completed DMA cookie > * @used: last issued DMA cookie (i.e. the one in progress) > * @residue: the remaining numb

Re: [PATCH v2 3/6] usb: phy: msm: Migrate to Managed Device Resource allocation

2013-07-29 Thread Felipe Balbi
Hi, On Mon, Jul 29, 2013 at 10:04:21AM +0300, Ivan T. Ivanov wrote: > motg->irq = platform_get_irq(pdev, 0); > - if (!motg->irq) { > + if (motg->irq < 0) { looks like this particular hunk isn't part of $subject. -- balbi signature.asc Description: Digital signature

Re: [PATCH v2 1/6] usb: phy: msm: Move mach depndend code to platform data

2013-07-29 Thread Felipe Balbi
Hi, On Mon, Jul 29, 2013 at 10:04:19AM +0300, Ivan T. Ivanov wrote: > From: "Ivan T. Ivanov" > > This patch fix compilation error and is an intermediate step > before the addition of DeviceTree support for newer targets. > Fix suggested here: https://lkml.org/lkml/2013/6/19/381 > > Cc: David Br

Re: [RFC] ux500 dma & short transfers on MUSB

2013-07-29 Thread Vinod Koul
On Thu, Jul 18, 2013 at 01:21:55PM +0200, Sebastian Andrzej Siewior wrote: > On 07/11/2013 07:14 PM, Sebastian Andrzej Siewior wrote: > > Dan, Vinod, > > do you guys have an idea how the dma driver could inform its user how > much of the requested data got really transferred? This requirement > s

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

2013-07-29 Thread Felipe Balbi
On Fri, Jul 26, 2013 at 10:15:55PM +0200, Sebastian Andrzej Siewior wrote: > 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. >

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

2013-07-29 Thread Felipe Balbi
On Fri, Jul 26, 2013 at 10:15:54PM +0200, Sebastian Andrzej Siewior wrote: > 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

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

2013-07-29 Thread Vinod Koul
On Mon, Jul 22, 2013 at 08:10:06PM +0200, Sebastian Andrzej Siewior wrote: > In USB RX path it is possible that the we receive less bytes than > requested. Take the following example: > The driver for USB-to-UART submits an URB with 256 bytes in size and the > dmaengine driver in turn programs a tr

[GIT PULL] usb: fixes for v3.11-rc3

2013-07-29 Thread Felipe Balbi
Hi Greg, Here's my second set of fixes. Nothing major, mostly related to recent conversion to configfs on the gadget framework. Let me know if you want me to change anything. The following changes since commit 690c70bab1ebab0782cf9c316b726e2dddebc411: usb: phy: omap-usb3: fix dpll clock index

Re: [PATCH 2/3] include: Convert ethernet mac address declarations to use ETH_ALEN

2013-07-29 Thread Rafael J. Wysocki
On Sunday, July 28, 2013 10:29:04 PM Joe Perches wrote: > It's convenient to have ethernet mac addresses use > ETH_ALEN to be able to grep for them a bit easier and > also to ensure that the addresses are __aligned(2). > > Add #include as necessary. > > Signed-off-by: Joe Perches > --- > inclu

Re: [PATCH] chipidea: ci_hdrc_imx: Remove unused variable 'res'

2013-07-29 Thread Alexander Shishkin
Fabio Estevam writes: > From: Fabio Estevam > > 'res' is not used anywhere, so let's get rid of it. > > Signed-off-by: Fabio Estevam Applied and pushed, thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More ma

[PATCH 2/2] usb: chipidea: fix the build error with randconfig

2013-07-29 Thread Alexander Shishkin
From: Peter Chen Using below configs, the compile will have error: ERROR: "ehci_init_driver" undefined! .config: CONFIG_USB_CHIPIDEA=m CONFIG_USB_CHIPIDEA_HOST=y CONFIG_USB_CHIPIDEA_DEBUG=y The reason is chipidea host uses symbol from ehci, but ehci is not compiled. Let the chipidea host depen

[PATCH 1/2] usb: chipidea: cast PORTSC_PTS and DEVLC_PTS macros

2013-07-29 Thread Alexander Shishkin
From: Fabio Estevam Fix the following build warnings on x86: drivers/usb/chipidea/core.c: In function 'hw_phymode_configure': drivers/usb/chipidea/core.c:226:3: warning: large integer implicitly truncated to unsigned type [-Woverflow] drivers/usb/chipidea/core.c:230:3: warning: large integer im

[PATCH 0/2] usb: chipidea: fixes for v3.11-rc3

2013-07-29 Thread Alexander Shishkin
Hi, Here are two small chipidea fixes for v3.11. Fabio Estevam (1): usb: chipidea: cast PORTSC_PTS and DEVLC_PTS macros Peter Chen (1): usb: chipidea: fix the build error with randconfig drivers/usb/chipidea/Kconfig | 4 ++-- drivers/usb/chipidea/bits.h | 4 ++-- 2 files changed, 4 insert

Re: Buffer size for ALSA USB PCM audio

2013-07-29 Thread Takashi Iwai
At Thu, 25 Jul 2013 10:50:49 -0400 (EDT), Alan Stern wrote: > > On Thu, 25 Jul 2013, Takashi Iwai wrote: > > > > Besides, what's the reason for allocating 10 URBs if you're not going > > > to submit more than 2 of them at a time? > > > > I don't know how you deduced 10 urbs in your example, >

Re: [PATCH v2 1/3] net/usb/r815x: replace USB buffer from stack to DMA-able

2013-07-29 Thread David Miller
From: Oliver Neukum Date: Mon, 29 Jul 2013 07:20:24 +0200 > On Sat, 2013-07-27 at 20:21 -0700, David Miller wrote: >> From: Hayes Wang >> Date: Thu, 25 Jul 2013 15:59:02 +0800 >> >> > Some USB buffers use stack which may not be DMA-able. >> > Use the buffers from kmalloc to replace those one. >

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

2013-07-29 Thread Ivan T. Ivanov
From: "Ivan T. Ivanov" When deferred probe happens driver will try to ioremap multiple times and will fail. Memory resource.start variable is a global variable, modifications in this field will be accumulated on every probe. Fix this by moving the above operations after driver hold all required P

Re: Kernel 3.10.3 "reset SuperSpeed USB device number 2 using xhci_hcd"

2013-07-29 Thread Stuart Foster
On 07/28/13 20:28, Alan Stern wrote: On Sat, 27 Jul 2013, Stuart Foster wrote: On 07/27/13 20:34, Alan Stern wrote: On Sat, 27 Jul 2013, Stuart Foster wrote: On 07/27/13 15:58, Alan Stern wrote: On Sat, 27 Jul 2013, Stuart Foster wrote: Hi, I have started having problems with an external

Chipidea tree is out of date one month

2013-07-29 Thread Peter Chen
Hi Alex, Your chipidea tree is out of date more than one month, https://github.com/virtuoso/linux-ci/commits/ci-for-greg You have not queued any patches since June 26th, below is the oldest patch you have not queued. http://marc.info/?l=linux-usb&m=137227196006173&w=2 There are two patches you s

  1   2   >