Re: [PATCH v5 5/5] Add ioctls to enable and disable local controls on an instrument

2015-11-22 Thread Dave Penkler
On Wed, Nov 18, 2015 at 11:41:30AM +0200, Andy Shevchenko wrote: > On Wed, Nov 18, 2015 at 10:38 AM, Dave Penkler wrote: > > These ioctls provide support for the USBTMC-USB488 control requests > > for REN_CONTROL, GO_TO_LOCAL and LOCAL_LOCKOUT > > Couple of comments below. > > > diff --git a/dri

Re: [PATCH v5 1/5] Implement an ioctl to support the USMTMC-USB488 READ_STATUS_BYTE operation.

2015-11-22 Thread Dave Penkler
On Wed, Nov 18, 2015 at 11:55:27AM +0200, Andy Shevchenko wrote: > On Wed, Nov 18, 2015 at 10:37 AM, Dave Penkler wrote: > > Background: > > When performing a read on an instrument that is executing a function > > that runs longer than the USB timeout the instrument may hang and > > require a devi

Re: Not enough host controller resources for new device state

2015-11-22 Thread Patrick Shirkey
On Sun, November 22, 2015 5:41 am, Alan Stern wrote: > On Sat, 21 Nov 2015, Patrick Shirkey wrote: > >> On Sat, November 21, 2015 9:35 am, Peter Stuge wrote: >> > Patrick Shirkey wrote: >> >> > You essentially have to educate yourself on silicon level (ie. what >> >> > hardware IP is being used in

Re: [PATCH v5 1/5] Implement an ioctl to support the USMTMC-USB488 READ_STATUS_BYTE operation.

2015-11-22 Thread Andy Shevchenko
On Sun, Nov 22, 2015 at 11:19 AM, Dave Penkler wrote: > On Wed, Nov 18, 2015 at 11:55:27AM +0200, Andy Shevchenko wrote: >> On Wed, Nov 18, 2015 at 10:37 AM, Dave Penkler wrote: >> > + switch (status) { >> > + case 0: /* SUCCESS */ >> > + if (data->iin_buffer[0] & 0x80)

Re: [PATCH v5 5/5] Add ioctls to enable and disable local controls on an instrument

2015-11-22 Thread Andy Shevchenko
On Sun, Nov 22, 2015 at 10:51 AM, Dave Penkler wrote: > On Wed, Nov 18, 2015 at 11:41:30AM +0200, Andy Shevchenko wrote: >> On Wed, Nov 18, 2015 at 10:38 AM, Dave Penkler wrote: >> > + if (rv < 0) { >> > + dev_err(dev, "simple usb_control_msg failed %d\n", rv); >> > +

[PATCH v3 0/2] cdc_acm: Ignore Infineon Flash Loader utility

2015-11-22 Thread Jonas Jonsson
In v3: - Better explanation of the device behaviour in the commit message. - Use USB_CLASS_CDC_DATA instead of 0x0a. In v2: - Include Tested-by: Daniele Palmas . -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More m

[PATCH v3 2/2] USB: serial: Another Infineon flash loader USB ID

2015-11-22 Thread Jonas Jonsson
The flash loader has been seen on a Telit UE910 modem. The flash loader is a bit special, it presents both an ACM and CDC Data interface but only the latter is useful. Unless a magic string is sent to the device it will disappear and the regular modem device appears instead. Signed-off-by: Jonas J

[PATCH v3 1/2] cdc_acm: Ignore Infineon Flash Loader utility

2015-11-22 Thread Jonas Jonsson
Some modems, such as the Telit UE910, are using an Infineon Flash Loader utility. It has two interfaces, 2/2/0 (Abstract Modem) and 10/0/0 (CDC Data). The latter can be used as a serial interface to upgrade the firmware of the modem. However, that isn't possible when the cdc-acm driver takes contro

Re: [4.3] kworker busy in pm_runtime_work

2015-11-22 Thread Daniel J Blueman
On 16 November 2015 at 23:22, Alan Stern wrote: > On Mon, 16 Nov 2015, Daniel J Blueman wrote: > >> Tuning USB suspend [1] in 4.3 on a Dell XPS 15 9553 (Skylake), I see a >> kworker thread spinning in rpm_suspend [2]. >> >> What is the most useful debug to get here beyond the immediate [3]? > > Yo

Re: [linux-sunxi] [PATCH 2/5] phy-sun4i-usb: Add support for the host usb-phys found on the H3 SoC

2015-11-22 Thread Hans de Goede
Hi, On 11/20/2015 08:49 PM, Priit Laes wrote: On Sun, 2015-11-15 at 20:46 +0100, Hans de Goede wrote: From: Reinder de Haan Note this commit only adds support for phys 1-3, phy 0, the otg phy, is not yet (fully) supported after this commit. This patch seems to be causing following compile

[PATCH v2] phy-sun4i-usb: Add support for the host usb-phys found on the H3 SoC

2015-11-22 Thread Hans de Goede
From: Reinder de Haan Note this commit only adds support for phys 1-3, phy 0, the otg phy, is not yet (fully) supported after this commit. Signed-off-by: Reinder de Haan Signed-off-by: Hans de Goede --- Changes in v2: -Change break; after dev_err() to return, as intended, fixing a compiler wa

Re: [Linux USB] enable/disable individual ports

2015-11-22 Thread Alan Stern
On Sat, 21 Nov 2015, Greg KH wrote: > > I saved a copy when you first posted it and have used it on quite a few > > occasions. Maybe the tool should be added to the usbutils collection so > > it could reach a somewhat wider audience? Yes, I know the usecases are > > all weird, but that still doe

Re: Not enough host controller resources for new device state

2015-11-22 Thread Alan Stern
On Sun, 22 Nov 2015, Patrick Shirkey wrote: > > It _is_ possible to disable xHCI at the system level. By sending the > > proper command to the appropriate sysfs file, you can unbind the > > xhci-hcd driver from the xHCI controller. This will disable the > > controller, and so all further USB com

Re: [PATCH] usb-storage: Fix scsi-sd failure "Invalid field in cdb" for USB adapter JMicron

2015-11-22 Thread Alan Stern
On Sat, 21 Nov 2015, Dmitry Katsubo wrote: > On 20/11/2015 17:08, Alan Stern wrote: > > On Fri, 20 Nov 2015, Dmitry Katsubo wrote: > > > >> From: Dmitry Katsubo > >> > >> The patch extends the family of SATA-to-USB JMicron adapters that need > >> FUA to be disabled and applies the same policy fo

Re: Not enough host controller resources for new device state

2015-11-22 Thread Patrick Shirkey
On Mon, November 23, 2015 2:43 am, Alan Stern wrote: > On Sun, 22 Nov 2015, Patrick Shirkey wrote: > >> > It _is_ possible to disable xHCI at the system level. By sending the >> > proper command to the appropriate sysfs file, you can unbind the >> > xhci-hcd driver from the xHCI controller. This

Re: [Linux USB] enable/disable individual ports

2015-11-22 Thread Peter Stuge
Alan Stern wrote: > It does print out a message, though not a big one. Would you like it > to do something more specific? A more verbose "usage" message, > perhaps? That's a good idea, and additionally I think it would be important to print (much) more information if the ioctl() fails. > For c

Re: Not enough host controller resources for new device state

2015-11-22 Thread Peter Stuge
Patrick Shirkey wrote: > > if the kernel does have support for xHCI, we assume that > > the user will prefer xHCI over EHCI if the motherboard has xHCI. > > Obviously the solution above should suffice for my purposes but out > of interest is it viable to make the switch accessible to run time > co

[GIT PULL] USB driver fixes for 4.4-rc2

2015-11-22 Thread Greg KH
The following changes since commit 8005c49d9aea74d382f474ce11afbbc7d7130bec: Linux 4.4-rc1 (2015-11-15 17:00:27 -0800) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git/ tags/usb-4.4-rc2 for you to fetch changes up to 19cd80a214821f4b558560

Re: [Linux USB] enable/disable individual ports

2015-11-22 Thread Greg KH
On Sun, Nov 22, 2015 at 07:26:08PM +0100, Peter Stuge wrote: > Alan Stern wrote: > > It does print out a message, though not a big one. Would you like it > > to do something more specific? A more verbose "usage" message, > > perhaps? > > That's a good idea, and additionally I think it would be i

Re: [PATCH v4 1/3] Documentation: dt-bindings: add async_irq to msm_hsusb

2015-11-22 Thread Rob Herring
On Fri, Nov 20, 2015 at 03:47:18PM -0800, Tim Bird wrote: > Add optional async_irq to msm_hsusb binding doc. > > Signed-off-by: Tim Bird Acked-by: Rob Herring > --- > Documentation/devicetree/bindings/usb/msm-hsusb.txt | 10 -- > 1 file changed, 8 insertions(+), 2 deletions(-) > > di

Re: [PATCH v2] phy-sun4i-usb: Add support for the host usb-phys found on the H3 SoC

2015-11-22 Thread Rob Herring
On Sun, Nov 22, 2015 at 12:29:24PM +0100, Hans de Goede wrote: > From: Reinder de Haan > > Note this commit only adds support for phys 1-3, phy 0, the otg phy, is > not yet (fully) supported after this commit. > > Signed-off-by: Reinder de Haan > Signed-off-by: Hans de Goede > --- > Changes in

Re: [PATCH 2/2] usb: phy: msm: fix connect/disconnect bug for dragonboard OTG port

2015-11-22 Thread Peter Chen
On Fri, Nov 20, 2015 at 06:58:59PM -0600, Felipe Balbi wrote: > > Hi, > > Tim Bird writes: > > On 11/16/2015 09:21 AM, Felipe Balbi wrote: > >> > >> Hi, > >> > >> Peter Chen writes: > >>> On Wed, Nov 11, 2015 at 09:48:00AM -0800, Tim Bird wrote: > > > On 11/10/2015 07:14 PM, Pe

Re: Not enough host controller resources for new device state

2015-11-22 Thread Alan Stern
On Mon, 23 Nov 2015, Patrick Shirkey wrote: > >> Is there a method to determine if the system has multiple or single > >> controllers available? > > > > lspci will tell you. > > > >> Do you or anyone else here have a link to the sysfs command to unbind > >> the > >> controller? > > > > If you prov

Re: Not enough host controller resources for new device state

2015-11-22 Thread Alan Stern
On Sun, 22 Nov 2015, Peter Stuge wrote: > Patrick Shirkey wrote: > > > if the kernel does have support for xHCI, we assume that > > > the user will prefer xHCI over EHCI if the motherboard has xHCI. > > > > Obviously the solution above should suffice for my purposes but out > > of interest is it

Re: [Linux USB] enable/disable individual ports

2015-11-22 Thread Alan Stern
On Sun, 22 Nov 2015, Greg KH wrote: > On Sun, Nov 22, 2015 at 07:26:08PM +0100, Peter Stuge wrote: > > Alan Stern wrote: > > > It does print out a message, though not a big one. Would you like it > > > to do something more specific? A more verbose "usage" message, > > > perhaps? > > > > That's

Re: [Linux USB] enable/disable individual ports

2015-11-22 Thread Peter Stuge
Alan Stern wrote: > > > > It does print out a message, though not a big one. Would you like it > > > > to do something more specific? A more verbose "usage" message, > > > > perhaps? > > > > > > That's a good idea, and additionally I think it would be important to > > > print (much) more informa

[PATCH v2] phy: add phy-hi6220-usb

2015-11-22 Thread Zhangfei Gao
Support hi6220 use phy for HiKey board Acked-by: Rob Herring Signed-off-by: Zhangfei Gao --- Rebase to Linux 4.4-rc1 .../devicetree/bindings/phy/phy-hi6220-usb.txt | 16 ++ drivers/phy/Kconfig| 9 ++ drivers/phy/Makefile |

Re: [PATCH] usb: dwc2: add support of hi6220

2015-11-22 Thread zhangfei
On 11/20/2015 11:34 PM, Felipe Balbi wrote: John Youn writes: That's weird. I just sync'd to your testing/next and it seems to apply fine. Same with the series from Gregory Herrero. Any chance it's something to do with your local repo? odd. Seems like it works if I apply manually with

Re: [PATCH v2] phy-sun4i-usb: Add support for the host usb-phys found on the H3 SoC

2015-11-22 Thread Kishon Vijay Abraham I
Hi, On Sunday 22 November 2015 04:59 PM, Hans de Goede wrote: > From: Reinder de Haan > > Note this commit only adds support for phys 1-3, phy 0, the otg phy, is > not yet (fully) supported after this commit. > > Signed-off-by: Reinder de Haan > Signed-off-by: Hans de Goede > --- > Changes in

Re: Not enough host controller resources for new device state

2015-11-22 Thread Patrick Shirkey
On Mon, November 23, 2015 1:24 pm, Alan Stern wrote: > On Mon, 23 Nov 2015, Patrick Shirkey wrote: > >> >> Is there a method to determine if the system has multiple or single >> >> controllers available? >> > >> > lspci will tell you. >> > >> >> Do you or anyone else here have a link to the sysfs

Re: [PATCH v6 4/4] usb: gadget: udc-core: independent registration of gadgets and gadget drivers

2015-11-22 Thread Peter Chen
On Fri, Nov 20, 2015 at 11:27:36AM -0500, Alan Stern wrote: > On Fri, 20 Nov 2015, Marek Szyprowski wrote: > > > From: Ruslan Bilovol > > > > Change behavior during registration of gadgets and > > gadget drivers in udc-core. Instead of previous > > approach when for successful probe of usb gadge

Re: [PATCH v6 4/4] usb: gadget: udc-core: independent registration of gadgets and gadget drivers

2015-11-22 Thread Peter Chen
On Fri, Nov 20, 2015 at 09:54:12AM +0100, Marek Szyprowski wrote: > From: Ruslan Bilovol > > Change behavior during registration of gadgets and > gadget drivers in udc-core. Instead of previous > approach when for successful probe of usb gadget driver > at least one usb gadget should be already r