Re: [PATCH v12 00/13] Add tested id switch and vbus connect detect support for Chipidea

2013-07-25 Thread Peter Chen
On Thu, Jul 25, 2013 at 07:55:23AM +0200, Marek Vasut wrote: Hi Peter, I have not tried the manual switching, but first, you need to close your vbus supply. I think we can close this issue, I will now be also getting MX28EVK. Thanks for all your help! Great. What's the

Re: [PATCH v12 00/13] Add tested id switch and vbus connect detect support for Chipidea

2013-07-25 Thread Marek Vasut
Dear 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 is fully tested at imx6 sabresd platform. My chipidea repo:

Re: usb cdc-acm driver cause windows xp sp3 blue screen randomly

2013-07-25 Thread Oliver Neukum
On Thu, 2013-07-25 at 10:06 +0800, Yingchun Li wrote: Ok, but the linux-cdc-acm.inf is provieded by us, so I think there should be something I can learn. Thanks anyway! And the driver binds and operates. The inf file works. The kernel driver is buggy. The driver comes from the vendor. As such

Re: [PATCH v4] staging: usbip: replace pr_warning() with dev_warn().

2013-07-25 Thread navin patidar
On Thu, Jul 25, 2013, Greg KH gre...@linuxfoundation.org said: On Thu, Jul 25, 2013 at 10:19:31AM +0530, navin patidar wrote: -pr_warning(Unable to start control thread\n); +struct device *dev; + +if (ud-side == USBIP_STUB) +dev = container_of(ud,

Re: External HDD does not work with 3.11-rc2

2013-07-25 Thread Philipp Dreimann
On Wed, Jul 24, 2013 at 11:03 PM, Alan Stern st...@rowland.harvard.edu 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/resume issue described here:

[PATCH - bugfix for 3.11] usb/gadget: free opts struct on error recovery

2013-07-25 Thread Andrzej Pietrasiewicz
Fix memory leaks introduced in commits: 40d133d7f542616cf9538508a372306e626a16e9 usb: gadget: f_ncm: convert to new function interface with backward compatibility fee562a6450b7806f1fbbe1469a67b5395b5c10a usb: gadget: f_ecm: convert to new function interface with backward compatibility

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

2013-07-25 Thread Oliver Neukum
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. I am afraid bisectability on SS is impaired. Otherwise this looks good. Regards Oliver -- To unsubscribe from this

Re: USB_QUIRK_RESET_RESUME not working with xhci

2013-07-25 Thread Oleksij Rempel
Hi Sarah, Are there any news on this issue? suddenly i don't have usb analyser to find what actually happening (That is way too expensive for Hobbyist). But probably thinkpenguin.com can provide you some sample to track it down (CC them). Am 22.07.2013 20:54, schrieb Oleksij Rempel: Am

Re: [PATCH 01/15] drivers: phy: add generic PHY framework

2013-07-25 Thread Arnd Bergmann
On Thursday 25 July 2013, Kishon Vijay Abraham I wrote: On Thursday 25 July 2013 12:02 AM, Arnd Bergmann wrote: On Tuesday 23 July 2013, Tomasz Figa wrote: On Tuesday 23 of July 2013 17:14:20 Alan Stern wrote: On Tue, 23 Jul 2013, Tomasz Figa wrote: Where would you want to have those

[PATCH v2 3/3] net/usb/r8152: adjust relative ocp function

2013-07-25 Thread Hayes Wang
- fix the conversion between cpu and __le32 - replace some pla_ocp and usb_ocp functions with generic_ocp function Signed-off-by: Hayes Wang hayesw...@realtek.com --- drivers/net/usb/r8152.c | 66 + 1 file changed, 23 insertions(+), 43

[PATCH v2 2/3] net/usb/r8152: make sure the USB buffer is DMA-able

2013-07-25 Thread Hayes Wang
Allocate the required memory before calling usb_control_msg. And the additional memory copy is necessary. Signed-off-by: Hayes Wang hayesw...@realtek.com --- drivers/net/usb/r8152.c | 60 - 1 file changed, 35 insertions(+), 25 deletions(-) diff

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

2013-07-25 Thread Hayes Wang
Some USB buffers use stack which may not be DMA-able. Use the buffers from kmalloc to replace those one. Signed-off-by: Hayes Wang hayesw...@realtek.com --- drivers/net/usb/r815x.c | 44 +++- 1 file changed, 27 insertions(+), 17 deletions(-) diff --git

Re: usb cdc-acm driver cause windows xp sp3 blue screen randomly

2013-07-25 Thread Yingchun Li
1) try to use different timeout 2) try to use C# or other DotNet language. If you can re-write your host application, another choice is to use a different driver like WinUSB to your device and then use libusb-1.0 API to communicate with your device. Ok, this way work for me, I will try to

Re: usb cdc-acm driver cause windows xp sp3 blue screen randomly

2013-07-25 Thread Xiaofan Chen
On Thu, Jul 25, 2013 at 4:02 PM, Yingchun Li sword.l.dra...@gmail.com wrote: 1) try to use different timeout 2) try to use C# or other DotNet language. If you can re-write your host application, another choice is to use a different driver like WinUSB to your device and then use libusb-1.0

Re: Buffer size for ALSA USB PCM audio

2013-07-25 Thread Takashi Iwai
At Wed, 24 Jul 2013 11:43:58 -0400 (EDT), Alan Stern wrote: On Wed, 24 Jul 2013, Takashi Iwai wrote: I don't understand. Consider a simple playback example. Suppose the user wants to keep the latency low, so he requests 2 periods per buffer. snd-usb-audio ignores this value and

Re: v3.11-rc1 USB regressions

2013-07-25 Thread Daniel Mack
On 25.07.2013 02:30, Aaro Koskinen wrote: On Wed, Jul 24, 2013 at 09:04:28PM +0200, Daniel Mack wrote: On 24.07.2013 20:51, Aaro Koskinen wrote: When I revert fe4cb0912f8e737f8e4b8b38b9e692f8062f5423 and 8b125df5b24cfb0ec7fa1971e343cc0badc1827d, it works like before (3.10): I'm now running

Re: [PATCH 01/15] drivers: phy: add generic PHY framework

2013-07-25 Thread Sylwester Nawrocki
On 07/24/2013 08:32 PM, Arnd Bergmann wrote: On Tuesday 23 July 2013, Tomasz Figa wrote: On Tuesday 23 of July 2013 17:14:20 Alan Stern wrote: On Tue, 23 Jul 2013, Tomasz Figa wrote: Where would you want to have those phy_address arrays stored? There are no board files when booting with DT.

Re: [PATCH 01/15] drivers: phy: add generic PHY framework

2013-07-25 Thread Mark Brown
On Wed, Jul 24, 2013 at 08:32:03PM +0200, Arnd Bergmann wrote: Sorry for jumping in to the middle of the discussion, but why does a *new* framework even bother defining an interface for board files? Can't we just drop any interfaces for platform data passing in the phy framework and put the

Re: [PATCH] staging: ozwpan: Fix coding style.

2013-07-25 Thread Rupesh Gujare
On 24/07/13 15:06, Rupesh Gujare wrote: This patch fixes coding style issues reported by Dan here:- http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2012-June/027767.html Signed-off-by: Rupesh Gujare rupesh.guj...@atmel.com --- drivers/staging/ozwpan/ozcdev.c | 15

Re: [PATCH 01/15] drivers: phy: add generic PHY framework

2013-07-25 Thread Laurent Pinchart
Hi Arnd, On Wednesday 24 July 2013 20:32:03 Arnd Bergmann wrote: On Tuesday 23 July 2013, Tomasz Figa wrote: On Tuesday 23 of July 2013 17:14:20 Alan Stern wrote: On Tue, 23 Jul 2013, Tomasz Figa wrote: Where would you want to have those phy_address arrays stored? There are no board

Re: [PATCH] staging: ozwpan: Fix coding style.

2013-07-25 Thread Dan Carpenter
On Thu, Jul 25, 2013 at 10:30:30AM +0100, Rupesh Gujare wrote: Greg, Did you missed this patch ? Or did I made another mistake. :( ? Chillax. It has only been one day. Ask again after two or three weeks. regards, dan carpenter -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH 1/1] TX throttling bug-fixing patch of AX88179_178A

2013-07-25 Thread Eric Dumazet
On Thu, 2013-07-25 at 13:25 +0800, Ming Lei wrote: On Thu, Jul 25, 2013 at 1:10 PM, Eric Dumazet eric.duma...@gmail.com wrote: On Thu, 2013-07-25 at 10:28 +0800, Ming Lei wrote: It depends if size of sg buffer(except for last one) in the sg list can be divided by usb endpoint's max

Re: [PATCH] staging: ozwpan: Fix coding style.

2013-07-25 Thread Rupesh Gujare
On 25/07/13 11:39, Dan Carpenter wrote: On Thu, Jul 25, 2013 at 10:30:30AM +0100, Rupesh Gujare wrote: Greg, Did you missed this patch ? Or did I made another mistake. :( ? Chillax. It has only been one day. Ask again after two or three weeks. All right Dan, Probably I was getting too

Re: [PATCH 01/15] drivers: phy: add generic PHY framework

2013-07-25 Thread Arnd Bergmann
On Thursday 25 July 2013, Laurent Pinchart wrote: On Wednesday 24 July 2013 20:32:03 Arnd Bergmann wrote: On Tuesday 23 July 2013, Tomasz Figa wrote: On Tuesday 23 of July 2013 17:14:20 Alan Stern wrote: On Tue, 23 Jul 2013, Tomasz Figa wrote: Where would you want to have those

Re: [PATCH 01/15] drivers: phy: add generic PHY framework

2013-07-25 Thread Laurent Pinchart
Hi Arnd, On Thursday 25 July 2013 13:00:49 Arnd Bergmann wrote: On Thursday 25 July 2013, Laurent Pinchart wrote: On Wednesday 24 July 2013 20:32:03 Arnd Bergmann wrote: On Tuesday 23 July 2013, Tomasz Figa wrote: On Tuesday 23 of July 2013 17:14:20 Alan Stern wrote: On Tue, 23 Jul

Suunto ANT+ stick

2013-07-25 Thread Steinar H. Gunderson
Hi, I have a Suunto ANT+ stick (designed to communicate with various training equipment) that works as a USB serial device. Unfortunately, it seems like its PCI IDs are missing; it doesn't come up unless I add vendor= and product= to the usbserial loading line explicitly. When I do, however, it

Re: [PATCH 01/15] drivers: phy: add generic PHY framework

2013-07-25 Thread Mark Brown
On Thu, Jul 25, 2013 at 01:00:49PM +0200, Arnd Bergmann wrote: I'm not saying that we can't support legacy board files with the common PHY framework, but I'd expect things to be much easier if we focus on those platforms that are actively being worked on for now, to bring an end to the

Re: [PATCH v12 00/13] Add tested id switch and vbus connect detect support for Chipidea

2013-07-25 Thread Marek Vasut
Dear Peter Chen, On Thu, Jul 25, 2013 at 07:55:23AM +0200, Marek Vasut wrote: Hi Peter, I have not tried the manual switching, but first, you need to close your vbus supply. I think we can close this issue, I will now be also getting MX28EVK. Thanks for all your help! Great.

Re: Audio I/O parameters

2013-07-25 Thread James Stone
On Wed, Jul 24, 2013 at 2:42 AM, James Stone jamesmst...@gmail.com wrote: On Wed, Jul 24, 2013 at 12:23 AM, James Stone jamesmst...@gmail.com wrote: On Tue, Jul 23, 2013 at 11:50 PM, Torstein Hegge he...@resisty.net wrote: On Tue, Jul 23, 2013 at 21:13:23 +0100, James Stone wrote: On Fri, Jul

Re: Audio I/O parameters

2013-07-25 Thread Clemens Ladisch
James Stone wrote: Ok -from the bisect, the problem of not being able to get to sub 64-frames per period starts with: http://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/?id=fc600432cd23e35c85de2ff4468d816d6938a112 This is a merge, which includes this commit:

Re: Audio I/O parameters

2013-07-25 Thread James Stone
On Thu, Jul 25, 2013 at 2:12 PM, Clemens Ladisch clem...@ladisch.de wrote: James Stone wrote: Ok -from the bisect, the problem of not being able to get to sub 64-frames per period starts with:

Re: [PATCH 1/1] TX throttling bug-fixing patch of AX88179_178A

2013-07-25 Thread Ben Hutchings
On Thu, 2013-07-25 at 13:25 +0800, Ming Lei wrote: On Thu, Jul 25, 2013 at 1:10 PM, Eric Dumazet eric.duma...@gmail.com wrote: On Thu, 2013-07-25 at 10:28 +0800, Ming Lei wrote: It depends if size of sg buffer(except for last one) in the sg list can be divided by usb endpoint's max

Re: [PATCH 7/7] usb: phy: msm: Lindent the code

2013-07-25 Thread Ivan T. Ivanov
Hi, On Wed, 2013-07-24 at 15:22 +0300, Felipe Balbi wrote: On Mon, Jun 24, 2013 at 06:27:44PM +0300, Ivan T. Ivanov wrote: From: Ivan T. Ivanov iiva...@mm-sol.com Cc: Felipe Balbi ba...@ti.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: linux-usb@vger.kernel.org Cc:

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

2013-07-25 Thread Ivan T. Ivanov
Hi, On Wed, 2013-07-24 at 15:38 +0300, Felipe Balbi wrote: Hi, On Tue, Jul 09, 2013 at 06:47:08PM +0300, Ivan T. Ivanov wrote: From: Ivan T. Ivanov iiva...@mm-sol.com Use managed device resources to clean up the probe/remove and get DT support for free. Signed-off-by: Ivan T.

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

2013-07-25 Thread Ivan T. Ivanov
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 iiva...@mm-sol.com This patch move global regulators variables to driver state structire and move allocation of the regulators to be devm managed.

Re: External HDD does not work with 3.11-rc2

2013-07-25 Thread Alan Stern
On Thu, 25 Jul 2013, Philipp Dreimann wrote: kernel log, lsusb and usbmon are attached. I tried usb2 and 3 ports, both have the same issue with this drive. Other drives work. The problem is caused by some program on your computer sending a command to the drive that it can't handle (SCSI

Re: [PATCH 2/2] Remove static sizing of usb_device_id arrays

2013-07-25 Thread Anders Hammarquist
In a message of Wed, 24 Jul 2013 15:52:08 -0700, Greg KH writes: On Sat, Jun 22, 2013 at 08:55:59PM +0200, Anders Hammarquist wrote: Signed-off-by: Anders Hammarquist i...@iko.pp.se --- drivers/usb/serial/ti_usb_3410_5052.c | 29 - 1 file changed, 24

Re: [PATCH V4] usb: Add Device Tree support to XHCI Platform driver

2013-07-25 Thread Felipe Balbi
Hi, On Thu, Jul 25, 2013 at 02:10:46AM +0400, Sergei Shtylyov wrote: + compatible = xhci-platform; It again looks somewhat like a driver name, not a device name. What made you change the value from usb-xhci, Al? Look at [eo]hci-omap.txt in that directory. I asked. usb-xhci is

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

2013-07-25 Thread Ming Lei
Hi, On Thu, Jul 25, 2013 at 3:38 PM, Oliver Neukum oneu...@suse.de 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. I am afraid bisectability on SS is impaired. Otherwise this

Re: [PATCH 1/5] usb: xhci: add the suspend/resume functionality

2013-07-25 Thread Felipe Balbi
HI, On Wed, Jul 24, 2013 at 10:54:47AM -0700, Sarah Sharp wrote: +#ifdef CONFIG_PM +static int xhci_plat_suspend(struct device *dev) +{ + struct usb_hcd *hcd = dev_get_drvdata(dev); + struct xhci_hcd *xhci = hcd_to_xhci(hcd); + + return

Re: [PATCH 7/7] usb: phy: msm: Lindent the code

2013-07-25 Thread Felipe Balbi
Hi, On Thu, Jul 25, 2013 at 04:40:51PM +0300, Ivan T. Ivanov wrote: diff --git a/drivers/usb/phy/phy-msm-usb.c b/drivers/usb/phy/phy-msm-usb.c index 6d05085..111f454 100644 --- a/drivers/usb/phy/phy-msm-usb.c +++ b/drivers/usb/phy/phy-msm-usb.c @@ -64,8 +64,8 @@ static int

Re: [PATCH 1/5] usb: phy: phy-nop: add support for am335x PHY

2013-07-25 Thread Felipe Balbi
Hi, On Fri, Jul 05, 2013 at 03:32:54PM +0200, Sebastian Andrzej Siewior wrote: The am335x PHY code is well hidden in multiple places. The glue layer uses the nop and does up/down in the background. This patch copies the power on / power off part out of dsps so it can be removed later in dsps.

Re: [PATCH 2/5] arm: dts: am33xx: add USB phy nodes

2013-07-25 Thread Felipe Balbi
On Fri, Jul 05, 2013 at 04:56:17PM +0200, Sebastian Andrzej Siewior wrote: On 07/05/2013 04:41 PM, Ruchika Kharwar wrote: On 07/05/2013 08:32 AM, Sebastian Andrzej Siewior wrote: The memory address contains three pieces that is the reset module which is currently the only one used and

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

2013-07-25 Thread Felipe Balbi
Hi, On Fri, Jul 05, 2013 at 03:32:56PM +0200, Sebastian Andrzej Siewior wrote: 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.

After entering D3 cold: URB ffff88020ea41000 submitted while active

2013-07-25 Thread Oleksii Shevchuk
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] [ cut here ] [13115.552841] WARNING: at

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

2013-07-25 Thread Felipe Balbi
On Fri, Jul 05, 2013 at 03:32:57PM +0200, Sebastian Andrzej Siewior wrote: This moves the two instances from the big node into two child nodes. The glue layer ontop does almost nothing. This could be two indepentant child nodes but I have no idea how 'ti,hwmods = usb_otg_hs;' affects the two

Re: [PATCH 5/5] musb: musb: dsps: remove instances variable

2013-07-25 Thread Felipe Balbi
On Fri, Jul 05, 2013 at 03:32:58PM +0200, Sebastian Andrzej Siewior wrote: Remove the instances variable, there is no single use of it. Signed-off-by: Sebastian Andrzej Siewior bige...@linutronix.de nice :-) -- balbi signature.asc Description: Digital signature

Re: [PATCH 2/2] usb: dwc3: ep0: don't change to configured state too early

2013-07-25 Thread Felipe Balbi
On Wed, Jul 24, 2013 at 02:01:27PM -0400, Alan Stern wrote: On Wed, 24 Jul 2013, Felipe Balbi wrote: Hi, On Tue, Jul 23, 2013 at 12:53:53PM -0400, Alan Stern wrote: I see. Doesn't the mass-storage gadget also use delayed status when going into the _un_configured state?

Re: [PATCH 01/16] usb: musb: dsps: init / shutdown the phy

2013-07-25 Thread Felipe Balbi
On Mon, Jul 22, 2013 at 08:09:52PM +0200, Sebastian Andrzej Siewior wrote: If the init / shutdown function of the phy moves out of dsps into the phy driver, then dsps needs to call the callbacks of the phy driver to ensure that this happens. Signed-off-by: Sebastian Andrzej Siewior

Re: [PATCH 02/16] usb: phy: phy-nop: add support for am335x PHY

2013-07-25 Thread Felipe Balbi
On Mon, Jul 22, 2013 at 08:09:53PM +0200, Sebastian Andrzej Siewior wrote: The am335x PHY code is well hidden in multiple places. The glue layer uses the nop and does up/down in the background. This patch copies the power on / power off part out of dsps so it can be removed later in dsps. At

Re: [PATCH V4] usb: Add Device Tree support to XHCI Platform driver

2013-07-25 Thread Alan Cooper
On Wed, Jul 24, 2013 at 6:10 PM, Sergei Shtylyov sergei.shtyl...@cogentembedded.com wrote: Hello. On 07/25/2013 01:21 AM, Sarah Sharp wrote: It looks like all the feedback has been addressed, but I'm no device tree expert. Felipe, Matthijs, and Sergei, does this look good? If so, I'll

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-25 Thread Sebastian Andrzej Siewior
* Bin Liu | 2013-07-23 13:23:57 [-0500]: Hi Sebastian, Hi Liu, On Mon, Jul 22, 2013 at 1:09 PM, Sebastian Andrzej Siewior bige...@linutronix.de wrote: This patch renames the type struct from ti81xx_driver_data to am33xx_driver_data since it is not used for ti81xx anymore. The EOI member is

Re: [PATCH 02/16] usb: phy: phy-nop: add support for am335x PHY

2013-07-25 Thread Sebastian Andrzej Siewior
* Felipe Balbi | 2013-07-25 17:33:30 [+0300]: On Mon, Jul 22, 2013 at 08:09:53PM +0200, Sebastian Andrzej Siewior wrote: right, I replied to the wrong series, please add a patch renaming file and functions to phy-generic.c/usb_phy_gen_ Sorry, I can't parse that. You want to rename phy-nop.c to

Re: Buffer size for ALSA USB PCM audio

2013-07-25 Thread Alan Stern
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, I made it up. :-) but in general, ep-nurbs is calculated so that the driver

Re: [PATCH 1/1] TX throttling bug-fixing patch of AX88179_178A

2013-07-25 Thread Ming Lei
On Thu, Jul 25, 2013 at 7:01 PM, Eric Dumazet eric.duma...@gmail.com wrote: On Thu, 2013-07-25 at 13:25 +0800, Ming Lei wrote: On Thu, Jul 25, 2013 at 1:10 PM, Eric Dumazet eric.duma...@gmail.com wrote: On Thu, 2013-07-25 at 10:28 +0800, Ming Lei wrote: It depends if size of sg

Re: [PATCH - bugfix for 3.11] usb/gadget: free opts struct on error recovery

2013-07-25 Thread Michal Nazarewicz
On Thu, Jul 25 2013, Andrzej Pietrasiewicz wrote: Fix memory leaks introduced in commits: 40d133d7f542616cf9538508a372306e626a16e9 usb: gadget: f_ncm: convert to new function interface with backward compatibility fee562a6450b7806f1fbbe1469a67b5395b5c10a usb: gadget: f_ecm: convert to new

Re: [PATCH 2/2] usb: dwc3: ep0: don't change to configured state too early

2013-07-25 Thread Alan Stern
On Thu, 25 Jul 2013, Felipe Balbi wrote: On Wed, Jul 24, 2013 at 02:01:27PM -0400, Alan Stern wrote: On Wed, 24 Jul 2013, Felipe Balbi wrote: Hi, On Tue, Jul 23, 2013 at 12:53:53PM -0400, Alan Stern wrote: I see. Doesn't the mass-storage gadget also use delayed status when

Re: [PATCH - bugfix for 3.11] usb/gadget: ether: put_usb_function on unbind

2013-07-25 Thread Michal Nazarewicz
On Wed, Jul 24 2013, Andrzej Pietrasiewicz wrote: Fix bugs introduced in 9c62ce83e4258bacc459faf57bf2ed83cce6be08 usb: gadget: ether: convert to new interface of f_ecm 94b5573e97729f0e1496d23b69cbe2c6b24ec0c3 usb: gadget: ether: convert to new interface of f_eem

Re: [PATCH 1/1] TX throttling bug-fixing patch of AX88179_178A

2013-07-25 Thread Ben Hutchings
On Thu, 2013-07-25 at 22:52 +0800, Ming Lei wrote: [...] On Thu, Jul 25, 2013 at 9:34 PM, Ben Hutchings bhutchi...@solarflare.com wrote: Not that I have any experience with USB drivers, but perhaps usb_sg_init()? USB SG library doesn't support submitting SG URB asynchronously, but that

Re: [PATCH V4] usb: Add Device Tree support to XHCI Platform driver

2013-07-25 Thread Sergei Shtylyov
On 25.07.2013 18:39, Alan Cooper wrote: It looks like all the feedback has been addressed, but I'm no device tree expert. Felipe, Matthijs, and Sergei, does this look good? If so, I'll queue to my xhci tree. Not quite there yet. Too bad I couldn't notice all the small issues at

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

2013-07-25 Thread Lars-Peter Clausen
On 07/22/2013 08:10 PM, 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 transfer of 256 bytes.

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

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, 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: gadget: multi: use function framework for ACM.) Signed-off-by: Andrzej Pietrasiewicz

Re: [PATCH v2 02/21] usb/gadget: f_phonet: remove unused preprocessor conditional

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: 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

Re: [PATCH v2] usb: host: xhci: Enable XHCI_SPURIOUS_SUCCESS for all controllers with xhci 1.0

2013-07-25 Thread Shuduo Sang
On Thu, Jul 25, 2013 at 12:30 AM, Sarah Sharp sarah.a.sh...@linux.intel.com wrote: On Mon, Jul 22, 2013 at 03:58:19PM +0800, Shuduo Sang wrote: On Mon, Jul 22, 2013 at 3:23 PM, George Cherian george.cher...@ti.com wrote: Yes, I run below script to capture picture. #!/bin/bash #

Re: [PATCH 1/1] TX throttling bug-fixing patch of AX88179_178A

2013-07-25 Thread Eric Dumazet
On Thu, 2013-07-25 at 22:52 +0800, Ming Lei wrote: Maybe need to try it with TSO enabled, in my test on ax88179_178a NIC after applying your disabling TSO patch, tx throughput is less than 600Mbps, but rx is close to 900Mbps. It looks like TCP stack could for this case allocate linear skbs

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-25 Thread Bin Liu
Sebastian, On Thu, Jul 25, 2013 at 9:41 AM, Sebastian Andrzej Siewior bige...@linutronix.de wrote: * Bin Liu | 2013-07-23 13:23:57 [-0500]: Hi Sebastian, Hi Liu, On Mon, Jul 22, 2013 at 1:09 PM, Sebastian Andrzej Siewior bige...@linutronix.de wrote: This patch renames the type struct from

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

2013-07-25 Thread Sebastian Andrzej Siewior
On 07/25/2013 04:57 PM, Lars-Peter Clausen wrote: I'm not sure that this will work. Once you've submitted the descriptor it is owned by the dmaengine driver and you are not allowed to dereference it anymore since the dmaengine driver might free the memory at any time. You should only reference

[Pull Request] xhci: Bug fixes, now with more tags!

2013-07-25 Thread Sarah Sharp
The following changes since commit 94190301ffa059c2d127b3a67ec5d161d5c62681: usb: option: add TP-LINK MA260 (2013-07-23 16:07:51 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/sarah/xhci.git tags/for-usb-linus-2013-07-25 for you to fetch

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

2013-07-25 Thread Oliver Neukum
On Thu, 2013-07-25 at 17:30 +0300, Oleksii Shevchuk wrote: Randomly after returning from S3 or enabling powersaving this trace goes to dmesg: [13115.552920] [a02f3a30] usb_submit_urb+0x360/0x370 [usbcore] [13115.552922] [a042fe96] wdm_int_callback+0x1c6/0x200 [cdc_wdm]

[PATCH 5/5] xhci: fix null pointer dereference on ring_doorbell_for_active_rings

2013-07-25 Thread Sarah Sharp
From: Oleksij Rempel li...@rempel-privat.de in some cases where device is attched to xhci port and do not responding, for example ath9k_htc with stalled firmware, kernel will crash on ring_doorbell_for_active_rings. This patch check if pointer exist before it is used. This patch should be

Re: Audio I/O parameters

2013-07-25 Thread Clemens Ladisch
James Stone wrote: On Thu, Jul 25, 2013 at 2:12 PM, Clemens Ladisch wrote: The commit that you found by bisection got reverted later: http://git.kernel.org/linus/015618b902ae Please check that the code of Fix URB cancellation at stream start is in your current kernel. well, in 3.10, it is

Re: [PATCH v2 04/21] usb/gadget: create a utility module for mass_storage

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: Converting to configfs requires making the f_mass_storage.c a module. But first we need to get rid of #include storage_common.c. This patch makes storage_common.c a separately compiled file, which is built as a utility module named u_ms.ko.

[PATCH 2/5] usb: xhci: Mark two functions __maybe_unused

2013-07-25 Thread Sarah Sharp
From: Olof Johansson o...@lixom.net Resolves the following build warnings: drivers/usb/host/xhci.c:332:13: warning: 'xhci_msix_sync_irqs' defined but not used [-Wunused-function] drivers/usb/host/xhci.c:3901:12: warning: 'xhci_change_max_exit_latency' defined but not used [-Wunused-function]

[PATCH 3/5] usb: fix build warning in pci-quirks.h when CONFIG_PCI is not enabled

2013-07-25 Thread Sarah Sharp
From: Randy Dunlap rdun...@infradead.org Fix warning when CONFIG_PCI is not enabled (from commit 296365781903226a3fb8758901eaeec09d2798e4). drivers/usb/host/pci-quirks.h: warning: its scope is only this definition or declaration, which is probably not what you want [enabled by default]

[PATCH 1/5] xhci: Avoid NULL pointer deref when host dies.

2013-07-25 Thread Sarah Sharp
When the host controller fails to respond to an Enable Slot command, and the host fails to respond to the register write to abort the command ring, the xHCI driver will assume the host is dead, and call usb_hc_died(). The USB device's slot_id is still set to zero, and the pointer stored at

[PATCH 4/5] usb: host: xhci: Enable XHCI_SPURIOUS_SUCCESS for all controllers with xhci 1.0

2013-07-25 Thread Sarah Sharp
From: George Cherian george.cher...@ti.com Xhci controllers with hci_version 0.96 gives spurious success events on short packet completion. During webcam capture the ERROR Transfer event TRB DMA ptr not part of current TD was observed. The same application works fine with synopsis controllers

Re: [PATCH v2 05/21] usb/gadget: f_mass_storage: factor out a header file

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: In order to prepare for the new function interface the f_mass_storage.c needs to be compiled as a module, and so a header file will be required. This patch factors out some code to a new f_mass_storage.h. Signed-off-by: Andrzej Pietrasiewicz

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-25 Thread Bin Liu
Sebastian, On Thu, Jul 25, 2013 at 10:16 AM, Sebastian Andrzej Siewior bige...@linutronix.de wrote: On 07/25/2013 05:12 PM, Bin Liu wrote: Sebastian, Hi Bin, Is it really there or was it never there and it has been added to TRM by accident? The EOI register IS in the USB subsystem of

Re: Audio I/O parameters

2013-07-25 Thread Alan Stern
On Thu, 25 Jul 2013, James Stone wrote: On Thu, Jul 25, 2013 at 2:12 PM, Clemens Ladisch clem...@ladisch.de wrote: James Stone wrote: Ok -from the bisect, the problem of not being able to get to sub 64-frames per period starts with:

Re: [PATCH v2 06/21] usb/gadget: f_mass_storage: add a level of indirection for luns storage

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: This is needed to prepare for configfs integration. So far the luns have been allocated during gadget's initialization, based on the nluns module parameter's value; the exact number is known when the gadget is initialized and that number of

Re: [PATCH v2 07/21] usb/gadget: f_mass_storage: use usb_gstrings_attach

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com Acked-by: Michal Nazarewicz min...@mina86.com --- drivers/usb/gadget/f_mass_storage.c | 25 - 1

Patch queue status?

2013-07-25 Thread Alan Stern
Greg: What's the status of your USB patch queue? I've seen a bunch of notices about patches being applied -- is the queue empty yet? I ask because of this: http://marc.info/?l=linux-usbm=137356910824266w=2 which hasn't been merged yet. Alan Stern -- To unsubscribe from this list:

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

2013-07-25 Thread Oleksii Shevchuk
lspci -s 00:14.0 -vvv 00:14.0 USB controller: Intel Corporation 7 Series/C210 Series Chipset Family USB xHCI Host Controller (rev 04) (prog-if 30 [XHCI]) Subsystem: Lenovo Device 21fa Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B-

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

2013-07-25 Thread Oliver Neukum
On Thu, 2013-07-25 at 18:45 +0300, Oleksii Shevchuk wrote: lspci -s 00:14.0 -vvv No. It happens with the wdm driver. What net device are you using? Regards Oliver -- To unsubscribe from this list: send the line unsubscribe linux-usb in the body of a message to

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

2013-07-25 Thread Oleksii Shevchuk
lsusb -s 001:004 Bus 001 Device 004: ID 1199:9013 Sierra Wireless, Inc. - Hardware | manufacturer: 'Sierra Wireless Inc' | model: 'Sierra Wireless MC8355 - Gobi 3000(TM) Module' | revision: 'D3200-STSUGN-1580 1 [Jul 08 2011

Re: [PATCH v2] usb: host: Faraday fotg210-hcd driver

2013-07-25 Thread Yuan-Hsin Chen
Hi, On Thu, Jul 25, 2013 at 7:10 AM, Greg KH gre...@linuxfoundation.org wrote: On Wed, Jun 19, 2013 at 07:53:04PM +, Yuan-Hsin Chen wrote: FOTG210 is an OTG controller which can be configured as an USB2.0 host. FOTG210 host is an ehci-like controller with some differences. First,

Re: [PATCH v2 08/21] usb/gadget: f_mass_storage: split fsg_common initialization into a number of functions

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: When configfs is in place, the things related to intialization of struct fsg_common will be split over a number of places. This patch adds several functions which together cover the former intialization routine fsg_common_init. When configfs

Re: [PATCH v2 09/21] usb/gadget: f_mass_storage: use fsg_common_setup in fsg_common_init

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com Acked-by: Michal

Re: [xhci] null pointer dereference on ring_doorbell_for_active_rings

2013-07-25 Thread Mathias Nyman
Hi I just started looking at this issue as well from xhci perspective. From your Oops backtrace I can see that ring_doorbell_for_active_rings() was called from handle_cmd_completion() This should only happend if XHCI_RESET_EP_QUIRK is set, i.e. you have a Fresco Logic host controller. In

Re: [PATCH v2 11/21] usb/gadget: f_mass_storage: use fsg_common_set_nluns in fsg_common_init

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com Acked-by: Michal

Re: [PATCH v2 13/21] usb/gadget: f_mass_storage: use fsg_common_set_cdev in fsg_common_init

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com Acked-by: Michal

Re: [PATCH v2 12/21] usb/gadget: f_mass_storage: use fsg_common_set_ops/_private_data in fsg_common_init

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com Acked-by: Michal

Re: [PATCH v2 14/21] usb/gadget: f_mass_storage: use fsg_common_create_luns in fsg_common_init

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com Acked-by: Michal

Re: [PATCH v2 15/21] usb/gadget: f_mass_storage: use fsg_common_set_inquiry_string in fsg_common_init

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com Acked-by: Michal

Re: [PATCH v2 16/21] usb/gadget: f_mass_storage: use fsg_common_run_thread in fsg_common_init

2013-07-25 Thread Michal Nazarewicz
On Fri, Jul 19 2013, Andrzej Pietrasiewicz wrote: fsg_common_init is a lengthy function. Now there are helper functions which cover all parts of it. Use them. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com Acked-by: Michal

Re: [PATCH 06/16] usb: musb: dsps: rename ti81xx_driver_data to am33xx_driver_data

2013-07-25 Thread Felipe Balbi
Hi, On Thu, Jul 25, 2013 at 10:28:37AM -0500, Bin Liu wrote: Sebastian, On Thu, Jul 25, 2013 at 10:16 AM, Sebastian Andrzej Siewior bige...@linutronix.de wrote: On 07/25/2013 05:12 PM, Bin Liu wrote: Sebastian, Hi Bin, Is it really there or was it never there and it has been

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

2013-07-25 Thread Felipe Balbi
Hi, On Tue, Jul 23, 2013 at 11:55:50AM -0700, Olof Johansson wrote: This resolves the following valid build warning: drivers/usb/gadget/at91_udc.c:1685:34: warning: 'flags' may be used uninitialized in this function [-Wmaybe-uninitialized] I switched from ? : to !! mostly to save from

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

2013-07-25 Thread Olof Johansson
On Thu, Jul 25, 2013 at 9:14 AM, Felipe Balbi ba...@ti.com wrote: Hi, On Tue, Jul 23, 2013 at 11:55:50AM -0700, Olof Johansson wrote: This resolves the following valid build warning: drivers/usb/gadget/at91_udc.c:1685:34: warning: 'flags' may be used uninitialized in this function

Re: Patch queue status?

2013-07-25 Thread Greg KH
On Thu, Jul 25, 2013 at 11:34:22AM -0400, Alan Stern wrote: Greg: What's the status of your USB patch queue? I've seen a bunch of notices about patches being applied -- is the queue empty yet? No it isn't empty yet, I tried to pick out most of the 3.11 patches from it for USB already, but

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

2013-07-25 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com 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

  1   2   >