Re: [PATCH] SCSI: Don't store LUN bits in CDB[1] for USB mass-storage devices

2014-09-04 Thread Christoph Hellwig
Thanks, applied. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: Fwd: Status of chipidea msm USB reset patch

2014-09-04 Thread Peter Chen
On Thu, Sep 04, 2014 at 07:47:40AM -0700, Tim Bird wrote: > On Fri, Aug 15, 2014 at 12:08 AM, Ivan T. Ivanov wrote: > > On Fri, 2014-08-15 at 08:23 +0800, Peter Chen wrote: > >> On Thu, Aug 14, 2014 at 11:54:02AM -0500, Felipe Balbi wrote: > >> > Hi, > >> > > >> > On Thu, Aug 14, 2014 at 09:53:10A

Re: [PATCH v8] usb:serial:pl2303: add GPIOs interface on PL2303

2014-09-04 Thread Wang YanQing
On Thu, Sep 04, 2014 at 06:44:31PM +0200, Benjamin Henrion wrote: > On Thu, Sep 4, 2014 at 6:14 PM, Benjamin Henrion wrote: > > I have subscribed to the lkml. > > > > Can you make me a favour, send me your email as you posted on the LKML > > in mbox format attached to an email? > > > > I am trying

Re: [PATCH v2] usb: usbip: fix usbip.h path in userspace tool

2014-09-04 Thread Valentina Manea
On Fri, Sep 5, 2014 at 1:58 AM, Piotr Król wrote: > Fixes: 588b48caf65c ("usbip: move usbip userspace code out of staging") > which introduced build failure by not changing uapi/usbip.h include path > according to new location. > > Signed-off-by: Piotr Król > --- > include/uapi/linux/Kbuild

Re: Fwd: Status of chipidea msm USB reset patch

2014-09-04 Thread Peter Chen
On Thu, Sep 04, 2014 at 07:47:40AM -0700, Tim Bird wrote: > On Fri, Aug 15, 2014 at 12:08 AM, Ivan T. Ivanov wrote: > > On Fri, 2014-08-15 at 08:23 +0800, Peter Chen wrote: > >> On Thu, Aug 14, 2014 at 11:54:02AM -0500, Felipe Balbi wrote: > >> > Hi, > >> > > >> > On Thu, Aug 14, 2014 at 09:53:10A

Re: [RFC PATCH 0/7] usb: gadget: add reset API at usb_gadget_driver

2014-09-04 Thread Peter Chen
On Thu, Sep 04, 2014 at 11:04:38AM -0400, Alan Stern wrote: > On Thu, 4 Sep 2014, Peter Chen wrote: > > > Hi Felipe & Alan, how about using below steps for this reset/vbus/pullup > > changes? It mainly uses Alan's suggestion. > > > > Step 1: > > The initial implementation in the four gadget >

Re: [PATCH v3 0/6] usb: host: change TPL support behaviour

2014-09-04 Thread Peter Chen
On Thu, Sep 04, 2014 at 11:12:42AM -0400, Alan Stern wrote: > On Thu, 4 Sep 2014, Peter Chen wrote: > > > On Wed, Sep 03, 2014 at 09:48:15PM -0400, Alan Stern wrote: > > > On Thu, 4 Sep 2014, Peter Chen wrote: > > > > > > > > > > Hi Greg & Alan, any comments for this patchset? > > > > > > > > >

Re: [PATCH v2] usb: usbip: fix usbip.h path in userspace tool

2014-09-04 Thread Greg KH
On Fri, Sep 05, 2014 at 12:58:02AM +0200, Piotr Król wrote: > Fixes: 588b48caf65c ("usbip: move usbip userspace code out of staging") > which introduced build failure by not changing uapi/usbip.h include path > according to new location. > > Signed-off-by: Piotr Król > --- > include/uapi/linux/K

Re: [PATCH] usb: usbip: fix usbip.h path in userspace tool

2014-09-04 Thread Piotr Król
On Fri, Sep 05, 2014 at 12:55:39AM +0200, Piotr Król wrote: Sorry for that I send second time the same patch. Please ignore it. Thanks, Piotr > Fixes: 588b48caf65c ("usbip: move usbip userspace code out of staging") > which introduced build failure by not changing uapi/usbip.h include path > acc

[PATCH v2] usb: usbip: fix usbip.h path in userspace tool

2014-09-04 Thread Piotr Król
Fixes: 588b48caf65c ("usbip: move usbip userspace code out of staging") which introduced build failure by not changing uapi/usbip.h include path according to new location. Signed-off-by: Piotr Król --- include/uapi/linux/Kbuild | 1 + tools/usb/usbip/libsrc/usbip_common.h | 2 +- 2 f

[PATCH] usb: usbip: fix usbip.h path in userspace tool

2014-09-04 Thread Piotr Król
Fixes: 588b48caf65c ("usbip: move usbip userspace code out of staging") which introduced build failure by not changing uapi/usbip.h include path according to new location. Signed-off-by: Piotr Król --- tools/usb/usbip/libsrc/usbip_common.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

Re: [PATCH v8] usb:serial:pl2303: add GPIOs interface on PL2303

2014-09-04 Thread Benjamin Henrion
On Thu, Sep 4, 2014 at 7:35 PM, Wang YanQing wrote: > On Sun, Aug 31, 2014 at 11:24:56PM +0800, Wang YanQing wrote: >> PL2303 USB Serial devices may has GPIOs, this patch add >> basic PL2303 gpio support. >> >> Known issue: >> If gpios are in use(export to userspace through sysfs interface, etc),

Re: Similar Elan Touchscreen reset behaviour

2014-09-04 Thread Greg KH
On Thu, Sep 04, 2014 at 02:30:40PM -0700, Sarah Sharp wrote: > On Sat, Aug 16, 2014 at 12:37:19AM -0700, Eric Decker wrote: > > Hi Sarah, > > > > I talked to you a while back about the ELAN touchscreen in my ASUS laptop > > doing a repeated disconnect. > > > > It just came back. So you are prob

[PATCH v5 2/2] usb: hcd: add generic PHY support

2014-09-04 Thread Sergei Shtylyov
Add the generic PHY support, analogous to the USB PHY support. Intended it to be used with the PCI EHCI/OHCI drivers and the xHCI platform driver. Signed-off-by: Sergei Shtylyov Signed-off-by: Yoshihiro Shimoda --- Changes in version 5: - renamed the new 'gen_phy' field of 'struct usb_phy' back

[PATCH v5 1/2] usb: rename phy to usb_phy in HCD

2014-09-04 Thread Sergei Shtylyov
From: Antoine Tenart The USB PHY member of the HCD structure is renamed to 'usb_phy' and modifications are done in all drivers accessing it. This is in preparation to adding the generic PHY support. Signed-off-by: Antoine Tenart [Sergei: added missing 'drivers/usb/misc/lvstest.c' file, resolved

[PATCH v5 0/2] Add generic PHY support to USB HCD

2014-09-04 Thread Sergei Shtylyov
Hello. This patchset is against the usb-next' branch of Greg KH's 'usb.git' repo. Here I add support for the generic PHY to the 'struct usb_hcd' (having to rename the existing 'phy' field to 'usb_phy' beforehand). This was mainly intended to be used with the PCI OHCI/EHCI drivers and also xHCI

Re: ax88179_178a hang over xhci

2014-09-04 Thread Sarah Sharp
Hi Andrea, I'm no longer the xHCI driver maintainer. Please work with Mathias, and Cc the linux-usb mailing list. Sarah Sharp On Wed, Aug 20, 2014 at 04:12:49PM +0200, Andrea Arcangeli wrote: > Hi Sarah, > > just a short followup on this. I still had 1gbps hangs with the > 0b95:1790 ASIX Elect

Re: Similar Elan Touchscreen reset behaviour

2014-09-04 Thread Sarah Sharp
On Sat, Aug 16, 2014 at 12:37:19AM -0700, Eric Decker wrote: > Hi Sarah, > > I talked to you a while back about the ELAN touchscreen in my ASUS laptop > doing a repeated disconnect. > > It just came back. So you are probably right about it being flakey h/w. > > Returning it at this point proba

Re: Problem with USB-to-SATA adapters (was: AS2105-based enclosure size issues with >2TB HDDs)

2014-09-04 Thread Dale R. Worley
> From: James Bottomley > Before we embark on elaborate hacks, why don't we just make the capacity > writeable (by root) in sysfs from userspace (will require block change)? > We can then encode all the nasty heuristics (including gpt reading) in > userspace as a udev rule. Looking in from the o

Re: [PATCH] usb: usbip: fix usbip.h path in userspace tool

2014-09-04 Thread Greg KH
On Thu, Sep 04, 2014 at 09:58:11PM +0300, Valentina Manea wrote: > On Thu, Sep 4, 2014 at 5:14 AM, Greg KH wrote: > > > >> This patch is a pretty nasty hack to get it to compile within the > >> kernel tree. I wonder whether we can make this self-contained by > >> duplicating the header's content

Re: RCU bug with v3.17-rc3 ?

2014-09-04 Thread Felipe Balbi
Hi, On Thu, Sep 04, 2014 at 02:25:35PM -0500, Felipe Balbi wrote: > On Thu, Sep 04, 2014 at 12:16:42PM -0700, Paul E. McKenney wrote: > > On Thu, Sep 04, 2014 at 01:40:21PM -0500, Felipe Balbi wrote: > > > Hi, > > > > > > I keep triggering the following Oops with -rc3 when writing to the mass > >

Re: [PATCH RESEND v3 01/12] usb: dwc2/gadget: fix phy disable sequence

2014-09-04 Thread Felipe Balbi
Hi, On Thu, Sep 04, 2014 at 07:37:20PM +, Paul Zimmerman wrote: > > From: Felipe Balbi [mailto:ba...@ti.com] > > Sent: Thursday, September 04, 2014 12:18 PM > > > > On Thu, Sep 04, 2014 at 12:04:23PM -0700, Paul Zimmerman wrote: > > > From: Kamil Debski > > > > > > When the driver is removed

RE: [PATCH RESEND v3 01/12] usb: dwc2/gadget: fix phy disable sequence

2014-09-04 Thread Paul Zimmerman
> From: Felipe Balbi [mailto:ba...@ti.com] > Sent: Thursday, September 04, 2014 12:18 PM > > On Thu, Sep 04, 2014 at 12:04:23PM -0700, Paul Zimmerman wrote: > > From: Kamil Debski > > > > When the driver is removed s3c_hsotg_phy_disable is called three times > > instead of once. This results in d

Re: RCU bug with v3.17-rc3 ?

2014-09-04 Thread Felipe Balbi
On Thu, Sep 04, 2014 at 12:16:42PM -0700, Paul E. McKenney wrote: > On Thu, Sep 04, 2014 at 01:40:21PM -0500, Felipe Balbi wrote: > > Hi, > > > > I keep triggering the following Oops with -rc3 when writing to the mass > > storage gadget driver: > > v3.17-rc3, correct? yup, as in subject ;-) > I

Re: [PATCH RESEND v3 12/12] usb: dwc2/gadget: avoid disabling ep0

2014-09-04 Thread Felipe Balbi
On Thu, Sep 04, 2014 at 12:04:34PM -0700, Paul Zimmerman wrote: > From: Robert Baldyga > > Endpoint 0 should not be disabled, so we start loop counter from number 1. > > Signed-off-by: Robert Baldyga > Signed-off-by: Paul Zimmerman Looks like this would fix a modprobe -r followed by modprobe

Re: [PATCH RESEND v3 09/12] usb: dwc2/gadget: delay enabling irq once hardware is configured properly

2014-09-04 Thread Felipe Balbi
On Thu, Sep 04, 2014 at 12:04:31PM -0700, Paul Zimmerman wrote: > From: Marek Szyprowski > > This patch fixes kernel panic/interrupt storm/etc issues if bootloader > left s3c-hsotg module in enabled state. Now interrupt handler is enabled > only after proper configuration of hardware registers. >

Re: [PATCH RESEND v3 08/12] usb: dwc2/gadget: do not call disconnect method in pullup

2014-09-04 Thread Felipe Balbi
On Thu, Sep 04, 2014 at 12:04:30PM -0700, Paul Zimmerman wrote: > From: Marek Szyprowski > > This leads to potential spinlock recursion in composite framework, other > udc drivers also don't call it directly from pullup method. > > Signed-off-by: Marek Szyprowski > Signed-off-by: Robert Baldyga

Re: [PATCH RESEND v3 07/12] usb: dwc2/gadget: break infinite loop in endpoint disable code

2014-09-04 Thread Felipe Balbi
On Thu, Sep 04, 2014 at 12:04:29PM -0700, Paul Zimmerman wrote: > From: Marek Szyprowski > > This patch fixes possible freeze caused by infinite loop in interrupt > context. > > Signed-off-by: Marek Szyprowski > Signed-off-by: Robert Baldyga > Signed-off-by: Paul Zimmerman Cc stable ? Revie

Re: [PATCH RESEND v3 02/12] usb: dwc2/gadget: fix phy initialization sequence

2014-09-04 Thread Felipe Balbi
On Thu, Sep 04, 2014 at 12:04:24PM -0700, Paul Zimmerman wrote: > From: Kamil Debski > > In the Generic PHY Framework a NULL phy is considered to be a valid phy > thus the "if (hsotg->phy)" check does not give us the information whether > the Generic PHY Framework is used. > > In addition to the

Re: [PATCH RESEND v3 01/12] usb: dwc2/gadget: fix phy disable sequence

2014-09-04 Thread Felipe Balbi
On Thu, Sep 04, 2014 at 12:04:23PM -0700, Paul Zimmerman wrote: > From: Kamil Debski > > When the driver is removed s3c_hsotg_phy_disable is called three times > instead of once. This results in decreasing of the phy reference counter > below zero and thus consecutive inserts of the module fails.

Re: RCU bug with v3.17-rc3 ?

2014-09-04 Thread Paul E. McKenney
On Thu, Sep 04, 2014 at 01:40:21PM -0500, Felipe Balbi wrote: > Hi, > > I keep triggering the following Oops with -rc3 when writing to the mass > storage gadget driver: v3.17-rc3, correct? I take it that the test passes on some earlier version?

[PATCH RESEND v3 10/12] usb: dwc2/gadget: assign TX FIFO dynamically

2014-09-04 Thread Paul Zimmerman
From: Robert Baldyga Because we have not enough memory to have each TX FIFO of size at least 3072 bytes (the maximum single packet size with 3 transactions per microframe), we create four FIFOs of lenght 1024, and four of length 3072 bytes, and assing them to endpoints dynamically according to ma

[PATCH RESEND v3 11/12] usb: dwc2/gadget: disable clock when it's not needed

2014-09-04 Thread Paul Zimmerman
From: Robert Baldyga When device is stopped or suspended clock is not needed so we can disable it for this time. Signed-off-by: Robert Baldyga Signed-off-by: Paul Zimmerman --- drivers/usb/dwc2/gadget.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/usb/dwc2/gadget.c b/d

[PATCH RESEND v3 08/12] usb: dwc2/gadget: do not call disconnect method in pullup

2014-09-04 Thread Paul Zimmerman
From: Marek Szyprowski This leads to potential spinlock recursion in composite framework, other udc drivers also don't call it directly from pullup method. Signed-off-by: Marek Szyprowski Signed-off-by: Robert Baldyga Signed-off-by: Paul Zimmerman --- drivers/usb/dwc2/gadget.c | 1 - 1 file

[PATCH RESEND v3 12/12] usb: dwc2/gadget: avoid disabling ep0

2014-09-04 Thread Paul Zimmerman
From: Robert Baldyga Endpoint 0 should not be disabled, so we start loop counter from number 1. Signed-off-by: Robert Baldyga Signed-off-by: Paul Zimmerman --- drivers/usb/dwc2/gadget.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/dwc2/gadget.c b/drivers/usb

[PATCH RESEND v3 07/12] usb: dwc2/gadget: break infinite loop in endpoint disable code

2014-09-04 Thread Paul Zimmerman
From: Marek Szyprowski This patch fixes possible freeze caused by infinite loop in interrupt context. Signed-off-by: Marek Szyprowski Signed-off-by: Robert Baldyga Signed-off-by: Paul Zimmerman --- drivers/usb/dwc2/gadget.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/dwc2

[PATCH RESEND v3 09/12] usb: dwc2/gadget: delay enabling irq once hardware is configured properly

2014-09-04 Thread Paul Zimmerman
From: Marek Szyprowski This patch fixes kernel panic/interrupt storm/etc issues if bootloader left s3c-hsotg module in enabled state. Now interrupt handler is enabled only after proper configuration of hardware registers. Signed-off-by: Marek Szyprowski Signed-off-by: Robert Baldyga Signed-off

[PATCH RESEND v3 05/12] usb: dwc2/gadget: hide some not really needed debug messages

2014-09-04 Thread Paul Zimmerman
From: Marek Szyprowski Some DWC2/s3c-hsotg debug messages are really useless for typical user, so hide them behind dev_dbg(). Signed-off-by: Marek Szyprowski Signed-off-by: Robert Baldyga Signed-off-by: Paul Zimmerman --- drivers/usb/dwc2/gadget.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH RESEND v3 06/12] usb: dwc2/gadget: ensure that all fifos have correct memory buffers

2014-09-04 Thread Paul Zimmerman
From: Marek Szyprowski Print warning if FIFOs are configured in such a way that they don't fit into the SPRAM available on the s3c hsotg module. Signed-off-by: Marek Szyprowski Signed-off-by: Robert Baldyga Signed-off-by: Paul Zimmerman --- drivers/usb/dwc2/core.h | 1 + drivers/usb/dwc2/

[PATCH RESEND v3 03/12] usb: dwc2/gadget: move phy bus length initialization

2014-09-04 Thread Paul Zimmerman
From: Kamil Debski This patch moves the part of code that initializes the PHY bus width. This results in simpler code and removes the need to check whether the Generic PHY Framework is used. Signed-off-by: Kamil Debski Signed-off-by: Marek Szyprowski Signed-off-by: Robert Baldyga Signed-off-b

[PATCH RESEND v3 04/12] usb: dwc2/gadget: Fix comment text

2014-09-04 Thread Paul Zimmerman
From: Andrzej Pietrasiewicz Adjust the debug text to the name of the printed variable. Signed-off-by: Andrzej Pietrasiewicz Signed-off-by: Robert Baldyga Signed-off-by: Paul Zimmerman --- drivers/usb/dwc2/gadget.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/us

[PATCH RESEND v3 02/12] usb: dwc2/gadget: fix phy initialization sequence

2014-09-04 Thread Paul Zimmerman
From: Kamil Debski In the Generic PHY Framework a NULL phy is considered to be a valid phy thus the "if (hsotg->phy)" check does not give us the information whether the Generic PHY Framework is used. In addition to the above this patch also removes phy_init from probe and phy_exit from remove. T

[PATCH RESEND v3 01/12] usb: dwc2/gadget: fix phy disable sequence

2014-09-04 Thread Paul Zimmerman
From: Kamil Debski When the driver is removed s3c_hsotg_phy_disable is called three times instead of once. This results in decreasing of the phy reference counter below zero and thus consecutive inserts of the module fails. This patch removes calls to s3c_hsotg_phy_disable from s3c_hsotg_remove

[PATCH RESEND v3 00/12] usb: dwc2/gadget: fix series (for v3.18)

2014-09-04 Thread Paul Zimmerman
From: Robert Baldyga I'm resending this patchset rebased on linux-next. Now it should apply. Thanks, Robert Baldyga Changelog: v3: https://lkml.org/lkml/2014/8/4/617 - use endpoint index instead of FIFO index for EPFIFO - extend patch description v2: https://lkml.org/lkml/2014/7/16/199 - add p

Re: [PATCH] usb: usbip: fix usbip.h path in userspace tool

2014-09-04 Thread Valentina Manea
On Thu, Sep 4, 2014 at 5:14 AM, Greg KH wrote: > >> This patch is a pretty nasty hack to get it to compile within the >> kernel tree. I wonder whether we can make this self-contained by >> duplicating the header's content inside usbip_common.h. > > Ick, no, don't do that, use the .h file, that is

RCU bug with v3.17-rc3 ?

2014-09-04 Thread Felipe Balbi
Hi, I keep triggering the following Oops with -rc3 when writing to the mass storage gadget driver: | # modprobe g_mass_storage stall=0 removable=1 file=/dev/sda | [ 44.883554] Number of LUNs=8 | [ 44.886709] Mass Storage Function, version: 2009/09/11 | [ 44.892303] LUN: removable file: (no

Re: [PATCH v8] usb:serial:pl2303: add GPIOs interface on PL2303

2014-09-04 Thread Wang YanQing
On Sun, Aug 31, 2014 at 11:24:56PM +0800, Wang YanQing wrote: > PL2303 USB Serial devices may has GPIOs, this patch add > basic PL2303 gpio support. > > Known issue: > If gpios are in use(export to userspace through sysfs interface, etc), > then call pl2303_release(unplug usb-serial convertor, mod

Re: [PATCH 3/5] usb: phy: twl4030-usb: Move code from twl4030_phy_power to the runtime PM calls

2014-09-04 Thread Tony Lindgren
* Kishon Vijay Abraham I [140904 06:51]: > Hi Tony, > > On Thursday 28 August 2014 04:58 AM, Tony Lindgren wrote: > > We don't need twl4030_phy_power() any longer now that we have > > the runtime PM calls. Let's get rid of it as it's confusing. > > No functional changes, just move the code and us

Re: [PATCH v8] usb:serial:pl2303: add GPIOs interface on PL2303

2014-09-04 Thread Linus Walleij
On Sun, Aug 31, 2014 at 5:24 PM, Wang YanQing wrote: > PL2303 USB Serial devices may has GPIOs, this patch add > basic PL2303 gpio support. > > Known issue: > If gpios are in use(export to userspace through sysfs interface, etc), > then call pl2303_release(unplug usb-serial convertor, modprobe -r

Re: [PATCH v2 3/3] gpio: add support for the Diolan DLN-2 USB GPIO driver

2014-09-04 Thread Linus Walleij
On Sat, Aug 30, 2014 at 12:00 AM, Octavian Purdila wrote: > From: Daniel Baluta > > This patch adds GPIO and IRQ support for the Diolan DLN-2 GPIO module. > > Information about the USB protocol interface can be found in the > Programmer's Reference Manual [1], see section 2.9 for the GPIO > modu

Re: [PATCH v3 0/6] usb: host: change TPL support behaviour

2014-09-04 Thread Alan Stern
On Thu, 4 Sep 2014, Peter Chen wrote: > On Wed, Sep 03, 2014 at 09:48:15PM -0400, Alan Stern wrote: > > On Thu, 4 Sep 2014, Peter Chen wrote: > > > > > > > > Hi Greg & Alan, any comments for this patchset? > > > > > > > > > > In patch 2/6, why did you move the !is_targeted(udev) code from > > >

Re: [RFC PATCH 0/7] usb: gadget: add reset API at usb_gadget_driver

2014-09-04 Thread Alan Stern
On Thu, 4 Sep 2014, Peter Chen wrote: > Hi Felipe & Alan, how about using below steps for this reset/vbus/pullup > changes? It mainly uses Alan's suggestion. > > Step 1: > The initial implementation in the four gadget > drivers can be very simple: It calls the disconnect handler. >

Re: Fwd: Status of chipidea msm USB reset patch

2014-09-04 Thread Tim Bird
On Fri, Aug 15, 2014 at 12:08 AM, Ivan T. Ivanov wrote: > On Fri, 2014-08-15 at 08:23 +0800, Peter Chen wrote: >> On Thu, Aug 14, 2014 at 11:54:02AM -0500, Felipe Balbi wrote: >> > Hi, >> > >> > On Thu, Aug 14, 2014 at 09:53:10AM -0700, Tim Bird wrote: >> > > Ping. Anybody know the status of this

RE: EHCI Isochronous schedule full?

2014-09-04 Thread Alan Stern
On Thu, 4 Sep 2014, Christian Melki wrote: > Hi Alan. > > The machine has only two cameras inserted (no hids, nothing else). > While the cameras have an interrupt endpoint, I disabled it in > uvc_video. Still I don't get more than two cameras into the schedule. > So there are no interrupt endpoin

Re: gadget serial driver - configuration value

2014-09-04 Thread Alan Stern
On Thu, 4 Sep 2014, Anjana V Kumar wrote: > >> We see that, the three configurations listed in serial driver (CDC > >> ACM, CDC OBEX, generic serial) cannot be present together as per the > >> current implementation. Is there a specific reason why the > >> configuration values were set as 1, 2 and

Re: [GIT PULL] USB-serial fixes for v3.17-rc4

2014-09-04 Thread Greg Kroah-Hartman
On Thu, Sep 04, 2014 at 10:40:40AM +0200, Johan Hovold wrote: > Hi Greg, > > Here's a few device id updates for v3.17-rc4. > > I added the zte_ev one to my tree when it missed rc3, but didn't ask you to > drop it from your queue so you have already applied that one. > > Thanks, > Johan > > >

Re: [PATCH 3/5] usb: phy: twl4030-usb: Move code from twl4030_phy_power to the runtime PM calls

2014-09-04 Thread Kishon Vijay Abraham I
Hi Tony, On Thursday 28 August 2014 04:58 AM, Tony Lindgren wrote: > We don't need twl4030_phy_power() any longer now that we have > the runtime PM calls. Let's get rid of it as it's confusing. > No functional changes, just move the code and use res instead > of ret as we are not returning that va

Re: [rfc]your patch to deal with devices that need continous data traffic

2014-09-04 Thread Oliver Neukum
On Thu, 2014-09-04 at 13:36 +0200, Johan Hovold wrote: > On Thu, Sep 04, 2014 at 12:44:37PM +0200, Oliver Neukum wrote: > > Hi, > > > > I also got a problem with such a device and I took your patch > > and applied it to this device. What do you think? The only > > substantial change I made was not

How to control charging on USB OTG port

2014-09-04 Thread Marcus Weseloh
Hi all, I've got an ARM development board with Allwinner A20 SoC and AXP209 charging controller for the attached battery. I'm using the USB OTG port for a MIDI USB gadget and would like to control how and when my board tries to charge the battery via the OTG USB connection. When attached to a USB

Re: [PATCH] usb: usbip: fix usbip.h path in userspace tool

2014-09-04 Thread Sergei Shtylyov
Hello. On 9/4/2014 3:31 AM, Piotr Król wrote: Fixes: 588b48caf65c ("usbip: move usbip userspace code out of staging") which introduced build failure by not changing uapi/usbip.h include path according to new location. Signed-off-by: Piotr Król --- tools/usb/usbip/libsrc/usbip_common.h | 2

Re: [PATCH] usb: usbip: fix usbip.h path in userspace tool

2014-09-04 Thread Greg KH
On Thu, Sep 04, 2014 at 11:59:40AM +0300, Valentina Manea wrote: > On Thu, Sep 4, 2014 at 2:41 AM, Greg KH wrote: > > > > Shouldn't you be able to use the same "path" as ch9.h uses? It's a uapi > > .h file, no relative paths should be needed. > > > > For the userspace tool, the build system only

Re: [rfc]your patch to deal with devices that need continous data traffic

2014-09-04 Thread Johan Hovold
On Thu, Sep 04, 2014 at 12:44:37PM +0200, Oliver Neukum wrote: > Hi, > > I also got a problem with such a device and I took your patch > and applied it to this device. What do you think? The only > substantial change I made was not counting unrequested input > for autosuspend. Ah, good to know th

Re: usb: gadget: How should an udc handle reception of more data than req.length?

2014-09-04 Thread Andreas Larsson
On 2014-09-03 18:07, Alan Stern wrote: On Wed, 3 Sep 2014, Andreas Larsson wrote: Hi! In short: How should a udc driver handle a situation when the host sends more data than req.length in the receiving OUT request? What should be reported to the gadget driver? What should be done with the exce

[PATCH] usb: gadget: midi: Fix ignored "index" and "id" module parameters

2014-09-04 Thread Marcus Weseloh
The MIDI USB gadget driver has "index" and "id" parameters which should determine the alsa sound card index and id to use when registering the card. Those parameters had no effect, as the relevant information is only set on the midi structure after f_midi_register_card is called. This patch moves

[rfc]your patch to deal with devices that need continous data traffic

2014-09-04 Thread Oliver Neukum
Hi, I also got a problem with such a device and I took your patch and applied it to this device. What do you think? The only substantial change I made was not counting unrequested input for autosuspend. Regards Oliver >From 7babd79010390e9bee19cf2ac2a0f374a59b1bed Mon Sep

Re: [PATCH resend 2/2] usb: serial: xsens_mt: always bind to interface number 1

2014-09-04 Thread Frans Klaver
On Thu, Sep 04, 2014 at 11:22:26AM +0200, Johan Hovold wrote: > On Thu, Sep 04, 2014 at 11:05:47AM +0200, Frans Klaver wrote: > > > > The new version is applied, built and tested with some actual hardware > > on 3.17-rc3. > > Good. Both patches applied now. Thanks again, Frans -- To unsubscribe

Re: [PATCH resend 2/2] usb: serial: xsens_mt: always bind to interface number 1

2014-09-04 Thread Johan Hovold
On Thu, Sep 04, 2014 at 11:05:47AM +0200, Frans Klaver wrote: > On Thu, Sep 04, 2014 at 10:15:22AM +0200, Johan Hovold wrote: > > On Thu, Sep 04, 2014 at 09:12:05AM +0200, Frans Klaver wrote: > > > > > > static int xsens_mt_probe(struct usb_serial *serial, > > > >

[PATCH net-next v3 2/2] r8152: use eth_hw_addr_random

2014-09-04 Thread Hayes Wang
If the hw doesn't have a valid MAC address, give a random one and set it to the hw. Signed-off-by: Hayes Wang --- drivers/net/usb/r8152.c | 35 +++ 1 file changed, 19 insertions(+), 16 deletions(-) diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c in

Re: [PATCH resend 2/2] usb: serial: xsens_mt: always bind to interface number 1

2014-09-04 Thread Frans Klaver
On Thu, Sep 04, 2014 at 10:15:22AM +0200, Johan Hovold wrote: > On Thu, Sep 04, 2014 at 09:12:05AM +0200, Frans Klaver wrote: > > > > static int xsens_mt_probe(struct usb_serial *serial, > > > const struct usb_device_id *id) > > > { > > > - if (!has_

[PATCH net-next v3 0/2] r8152: random MAC address

2014-09-04 Thread Hayes Wang
If the interface has invalid MAC address, it couldn't be used. In order to let it work normally, give a random one. v3: Remove ether_addr_copy(dev->perm_addr, dev->dev_addr); v2: Use "%pM" format specifier for printing a MAC address. Hayes Wang (2): r8152: change the location of rt

Re: [PATCH] usb: usbip: fix usbip.h path in userspace tool

2014-09-04 Thread Valentina Manea
On Thu, Sep 4, 2014 at 2:41 AM, Greg KH wrote: > > Shouldn't you be able to use the same "path" as ch9.h uses? It's a uapi > .h file, no relative paths should be needed. > For the userspace tool, the build system only looks for headers in the system default locations, such as /usr/include. Hence

[GIT PULL] USB-serial fixes for v3.17-rc4

2014-09-04 Thread Johan Hovold
Hi Greg, Here's a few device id updates for v3.17-rc4. I added the zte_ev one to my tree when it missed rc3, but didn't ask you to drop it from your queue so you have already applied that one. Thanks, Johan The following changes since commit 69e273c0b0a3c337a521d083374c918dc52c666f: Linux

Re: [PATCH resend 2/2] usb: serial: xsens_mt: always bind to interface number 1

2014-09-04 Thread Johan Hovold
On Thu, Sep 04, 2014 at 09:12:05AM +0200, Frans Klaver wrote: > > static int xsens_mt_probe(struct usb_serial *serial, > > const struct usb_device_id *id) > > { > > - if (!has_required_endpoints(serial->interface->cur_altsetting)) > > -

[PATCH net-next v3 1/2] r8152: change the location of rtl8152_set_mac_address

2014-09-04 Thread Hayes Wang
Exchange the location of rtl8152_set_mac_address() and set_ethernet_addr(). Then, the set_ethernet_addr() could set the MAC address by calling rtl8152_set_mac_address() later. Signed-off-by: Hayes Wang --- drivers/net/usb/r8152.c | 34 +- 1 file changed, 17 insert

RE: EHCI Isochronous schedule full?

2014-09-04 Thread Christian Melki
Hi Alan. The machine has only two cameras inserted (no hids, nothing else). While the cameras have an interrupt endpoint, I disabled it in uvc_video. Still I don't get more than two cameras into the schedule. So there are no interrupt endpoints besides the one used for the internal hubs. They a

[PATCH 2/2] usb: serial: xsens_mt: always bind to interface number 1

2014-09-04 Thread Frans Klaver
Probe is testing if the current interface provides two bulk endpoints. While this achieves the goal of only binding to the correct interface, we already know we can find the device on interface number 1. Stop checking the endpoints and just return successfully when interface number 1 is probed. Si

Re: [PATCH resend 2/2] usb: serial: xsens_mt: always bind to interface number 1

2014-09-04 Thread Frans Klaver
Johan, I noticed I submitted an error here. Will resend later today. On Mon, Sep 1, 2014 at 11:39 AM, Frans Klaver wrote: > Probe is testing if the current interface provides two bulk endpoints. > While this achieves the goal of only binding to the correct interface, > we already know we can fin

RE: [PATCH v3] uas: replace WARN_ON_ONCE() with lockdep_assert_held()

2014-09-04 Thread Sharma, Sanjeev
-Original Message- From: gre...@linuxfoundation.org [mailto:gre...@linuxfoundation.org] Sent: Tuesday, August 19, 2014 3:01 PM To: Sharma, Sanjeev Cc: kra...@redhat.com; mdharm-...@one-eyed-alien.net; linux-usb@vger.kernel.org; linux-ker...@vger.kernel.org; linux-s...@vger.kernel.org; Ha