chipidea USB controller on IMX6solo processor does not give interrupt on quick connection and discconection

2017-05-02 Thread Anshuman Tripathy
Hello All, I am working on wanboard and the processor there is IMX6solo series . The USB controller is chipidea one . The drivers are all chipidea opensource driver. I have a requirement of connecting Iphone to it , where it will launch carplay on detection of carplay device . The flow is :

Re: [PATCH] USB: xhci: fix lock-inversion problem

2017-05-02 Thread Mathias Nyman
On 01.05.2017 22:41, Alan Stern wrote: With threaded interrupts, bottom-half handlers are called with interrupts enabled. Therefore they can't safely use spin_lock(); they have to use spin_lock_irqsave(). Lockdep warns about a violation occurring in xhci_irq():

Re: USB: serial: ftdi_sio: fix extreme low-latency setting

2017-05-02 Thread Johan Hovold
On Fri, Apr 28, 2017 at 11:27:20AM +0200, Anthony Mallet wrote: > Hello, > > Regarding this commit: > https://github.com/torvalds/linux/commit/c6dce262 > > The change breaks things badly as regular users cannot > programmatically get the previous 1ms latency setting anymore. This is > even more h

Re: VFS: busy inodes on changed media or resized disk sr0

2017-05-02 Thread Oliver Neukum
Am Freitag, den 28.04.2017, 16:56 -0400 schrieb Alan Stern: > On Fri, 28 Apr 2017, Cristian wrote: > > > Hello, > > > > Message dmesg 4.11.0-041100rc8-generic: > > [  383.133909] VFS: busy inodes on changed media or resized disk sr0 > > [  451.239627] VFS: busy inodes on changed media or resized

Re: Kernel / USB bug

2017-05-02 Thread Oliver Neukum
Am Montag, den 01.05.2017, 20:04 +0200 schrieb Nicolas Repentin: > Anyone ? > > > Le 30/04/2017 à 18:34, Nicolas Repentin a écrit : > > > > Hi all, > > > > I got some bugs on USB 3.0 storage. > > > > I've created a bugzilla here: > > > > https://bugzilla.kernel.org/show_bug.cgi?id=189631 > >

Re: USB: serial: ftdi_sio: fix extreme low-latency setting

2017-05-02 Thread Anthony Mallet
On Tuesday 2 May 2017, at 10:34, Johan Hovold wrote: > On Fri, Apr 28, 2017 at 11:27:20AM +0200, Anthony Mallet wrote: > > This is due to commit 557aaa7 > > https://github.com/torvalds/linux/commit/557aaa7ffab639d0190b935a041b16ae44606342 > > that silently/partially ignores the ASYNC_LOW_LATENCY f

Re: USB: serial: ftdi_sio: fix extreme low-latency setting

2017-05-02 Thread Johan Hovold
On Tue, May 02, 2017 at 12:05:48PM +0200, Anthony Mallet wrote: > On Tuesday 2 May 2017, at 10:34, Johan Hovold wrote: > > On Fri, Apr 28, 2017 at 11:27:20AM +0200, Anthony Mallet wrote: > > > This is due to commit 557aaa7 > > > https://github.com/torvalds/linux/commit/557aaa7ffab639d0190b935a041b

Chipidea USB controller hangs in peripheral mode under high memory bus pressure

2017-05-02 Thread Laurent Pinchart
Hello, I ran into an issue with a Xilinx Zynq XC7Z010 system. The system acts as a USB peripheral, using the UVC gadget driver. When transferring high bandwidth data over USB in isochronous mode, complete system freezes are occasionally noticed. The problem was traced to the following code fro

Re: Chipidea USB controller hangs in peripheral mode under high memory bus pressure

2017-05-02 Thread Laurent Pinchart
On Tuesday 02 May 2017 15:07:03 Laurent Pinchart wrote: > Hello, > > I ran into an issue with a Xilinx Zynq XC7Z010 system. The system acts as a > USB peripheral, using the UVC gadget driver. > > When transferring high bandwidth data over USB in isochronous mode, complete > system freezes are occ

Re: [PATCH] usb: musb: musb_cppi41: Update an error message

2017-05-02 Thread Bin Liu
On Tue, May 02, 2017 at 11:21:54AM +0530, Sekhar Nori wrote: > Hi Alexandre, > > On Friday 28 April 2017 09:34 PM, Alexandre Bailon wrote: > > If dma_request_slave_channel() failed to return a channel, > > then the driver will print an error and request to defer probe. > > Update the error message

Re: [PATCH] usb: hub: judge BOS field in usb_reset_and_verify_device()

2017-05-02 Thread Greg KH
On Tue, May 02, 2017 at 02:31:40PM +0800, wangjianxing5...@163.com wrote: > From: "Jianxing.Wang" > > When notebook with intel-3165-wifi suspend and resume,crash for > udev->bos is NULL. I'm sorry, I don't understand, how can udev->bos be NULL here? Why does the wifi device matter to the USB de

Re: [PATCH] usb: hub: judge BOS field in usb_reset_and_verify_device()

2017-05-02 Thread Guenter Roeck
On 05/02/2017 06:19 AM, Greg KH wrote: On Tue, May 02, 2017 at 02:31:40PM +0800, wangjianxing5...@163.com wrote: From: "Jianxing.Wang" When notebook with intel-3165-wifi suspend and resume,crash for udev->bos is NULL. I'm sorry, I don't understand, how can udev->bos be NULL here? Why does t

Re: [PATCH] usb: hub: judge BOS field in usb_reset_and_verify_device()

2017-05-02 Thread jianxing
>That check was added recently with 7b2db29fbb4e7, which also explains how it >can happen. Question here is if the problem with intel-3165-wifi was seen >on a kernel which doesn't have that patch. @Guenter Roeck, Thanks for your reminding,I didn't see your patch. I just do test with kernel-4.10,

Re: [usb-host] question about pointer dereference before null check

2017-05-02 Thread Alan Stern
On Mon, 1 May 2017, Gustavo A. R. Silva wrote: > Hello everybody, > > While taking a look into Coverity ID 100828 I ran into the following > piece of code at drivers/usb/host/ehci-sched.c:1096: > > u32 addr; > int think_time; > int hs_transfers; > > addr =

Re: [PATCH] usb: gadget: udc: add null check before pointer dereference

2017-05-02 Thread Alan Stern
On Mon, 1 May 2017, Gustavo A. R. Silva wrote: > Add null check before dereferencing dev->regs pointer inside > net2280_led_shutdown() function. > > Addresses-Coverity-ID: 101783 > Signed-off-by: Gustavo A. R. Silva > --- > drivers/usb/gadget/udc/net2280.c | 8 +--- > 1 file changed, 5 inse

Re: [PATCH] USB: xhci: fix lock-inversion problem

2017-05-02 Thread Alan Stern
On Tue, 2 May 2017, Mathias Nyman wrote: > On 01.05.2017 22:41, Alan Stern wrote: > > With threaded interrupts, bottom-half handlers are called with > > interrupts enabled. Therefore they can't safely use spin_lock(); they > > have to use spin_lock_irqsave(). Lockdep warns about a violation > >

Re: [PATCH] usb: musb: musb_cppi41: Update an error message

2017-05-02 Thread Sekhar Nori
On Tuesday 02 May 2017 06:24 PM, Bin Liu wrote: > On Tue, May 02, 2017 at 11:21:54AM +0530, Sekhar Nori wrote: >> Hi Alexandre, >> >> On Friday 28 April 2017 09:34 PM, Alexandre Bailon wrote: >>> If dma_request_slave_channel() failed to return a channel, >>> then the driver will print an error and

Re: [PATCH] USB: xhci: fix lock-inversion problem

2017-05-02 Thread Mathias Nyman
On 02.05.2017 17:34, Alan Stern wrote: On Tue, 2 May 2017, Mathias Nyman wrote: On 01.05.2017 22:41, Alan Stern wrote: With threaded interrupts, bottom-half handlers are called with interrupts enabled. Therefore they can't safely use spin_lock(); they have to use spin_lock_irqsave(). Lockdep

Re: [PATCH] usb: musb: musb_cppi41: Update an error message

2017-05-02 Thread Bin Liu
On Tue, May 02, 2017 at 08:07:44PM +0530, Sekhar Nori wrote: > On Tuesday 02 May 2017 06:24 PM, Bin Liu wrote: > > On Tue, May 02, 2017 at 11:21:54AM +0530, Sekhar Nori wrote: > >> Hi Alexandre, > >> > >> On Friday 28 April 2017 09:34 PM, Alexandre Bailon wrote: > >>> If dma_request_slave_channel()

Re: [usb-host] question about pointer dereference before null check

2017-05-02 Thread Gustavo A. R. Silva
Hi Alan, Quoting Alan Stern : On Mon, 1 May 2017, Gustavo A. R. Silva wrote: Hello everybody, While taking a look into Coverity ID 100828 I ran into the following piece of code at drivers/usb/host/ehci-sched.c:1096: u32 addr; int think_time; int hs_transf

RE: Asmedia USB 1343 crashes

2017-05-02 Thread David Laight
From: Thomas Fjellstrom > Sent: 01 May 2017 14:40 > I've got a 970 Pro gaming aura motherboard with an Asmedia 1343 Usb 3.1 > controller. It's been consistently throwing errors and eventually crashing and > becomming unresponsive. ... I've an earlier Asmedia 1042 controller. It has a bug (which I

Re: [PATCH] usb: gadget: udc: add null check before pointer dereference

2017-05-02 Thread Gustavo A. R. Silva
Hi Alan, Quoting Alan Stern : On Mon, 1 May 2017, Gustavo A. R. Silva wrote: Add null check before dereferencing dev->regs pointer inside net2280_led_shutdown() function. Addresses-Coverity-ID: 101783 Signed-off-by: Gustavo A. R. Silva --- drivers/usb/gadget/udc/net2280.c | 8 +--- 1 f

[PATCH] usb: host: remove unnecessary null check

2017-05-02 Thread Gustavo A. R. Silva
Remove unnecessary null check. udev->tt cannot ever be NULL when this section of code runs. Addresses-Coverity-ID: 100828 Cc: Alan Stern Signed-off-by: Gustavo A. R. Silva --- drivers/usb/host/ehci-sched.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/host/ehci

Re: [PATCH] usb: musb: musb_cppi41: Update an error message

2017-05-02 Thread Alexandre Bailon
On 05/01/2017 04:47 PM, Bin Liu wrote: Hi, On Fri, Apr 28, 2017 at 06:04:54PM +0200, Alexandre Bailon wrote: If dma_request_slave_channel() failed to return a channel, then the driver will print an error and request to defer probe. Update the error message to explain we are defrering probe.

[PATCH v2] usb: gadget: udc: add null check before pointer dereference

2017-05-02 Thread Gustavo A. R. Silva
Add null check before dereferencing dev->regs pointer inside net2280_led_shutdown() function. Addresses-Coverity-ID: 101783 Cc: Alan Stern Signed-off-by: Gustavo A. R. Silva --- Changes in v2: Move the net2280_led_shutdown() call later. drivers/usb/gadget/udc/net2280.c | 5 +++-- 1 file chan

Re: [PATCH] usb: musb: musb_cppi41: Update an error message

2017-05-02 Thread Sergei Shtylyov
On 05/02/2017 07:50 PM, Alexandre Bailon wrote: If dma_request_slave_channel() failed to return a channel, then the driver will print an error and request to defer probe. Update the error message to explain we are defrering probe. Signed-off-by: Alexandre Bailon --- drivers/usb/musb/musb_cpp

[PATCH] USB: serial: ftdi_sio: fix setting latency for unprivileged users

2017-05-02 Thread Anthony Mallet
Commit 557aaa7ffab6 ("ft232: support the ASYNC_LOW_LATENCY flag") enables unprivileged users to set the FTDI latency timer, but there was a logic flaw that skipped sending the corresponding USB control message to the device. Signed-off-by: Anthony Mallet --- Here is the patch, feel free to upda

Re: Kernel / USB bug

2017-05-02 Thread Nicolas Repentin
Hi I got this bug : using last kernel on Archlinux (4.10.13-1, since 4.4), an USB 3.0 external HDD dies all the time when writing on it. If you write slow, it works, but with high copy, it freezes. Tested on multiple computers, multiples disks, and multiple external disks. On dmesg I got this :

Re: [PATCH] usb: host: remove unnecessary null check

2017-05-02 Thread Alan Stern
On Tue, 2 May 2017, Gustavo A. R. Silva wrote: > Remove unnecessary null check. udev->tt cannot ever be NULL when this > section of code runs. > > Addresses-Coverity-ID: 100828 > Cc: Alan Stern > Signed-off-by: Gustavo A. R. Silva > --- > drivers/usb/host/ehci-sched.c | 2 +- > 1 file changed,

Re: [PATCH v2] usb: gadget: udc: add null check before pointer dereference

2017-05-02 Thread Alan Stern
On Tue, 2 May 2017, Gustavo A. R. Silva wrote: > Add null check before dereferencing dev->regs pointer inside > net2280_led_shutdown() function. > > Addresses-Coverity-ID: 101783 > Cc: Alan Stern > Signed-off-by: Gustavo A. R. Silva > --- > Changes in v2: > Move the net2280_led_shutdown() cal

Re: VFS: busy inodes on changed media or resized disk sr0

2017-05-02 Thread Cristian
Hello, Message in Linux 4.11.0-041100rc8-generic Regards, -- Cristian 2017-05-02 5:46 GMT-03:00 Oliver Neukum : > Am Freitag, den 28.04.2017, 16:56 -0400 schrieb Alan Stern: >> On Fri, 28 Apr 2017, Cristian wrote: >> >> > Hello, >> > >> > Message dmesg 4.11.0-041100rc8-generic: >> > [ 383.13

[usb-gadget-udc] question about null check after calling phys_to_virt() function

2017-05-02 Thread Gustavo A. R. Silva
Hello everybody, While looking into Coverity ID 145958 I ran into the following piece of code at drivers/usb/gadget/udc/amd5536udc.c:852: } else if (i == buf_len) { /* first td */ td = (struct udc_data_dma *)phys_to_virt( req->td_data->next);

dwc3 udc driver

2017-05-02 Thread Ed Lizard
I'm having trouble seeing the udc driver for dwc3 in /sys/class/udc. We are moving platforms from musb on 2.6.37 to dwc3 on 4.6. Whew, a lot has changed since we were stuck on the old kernel. It looks like libusbgx and udc is the current method for a complex video/web gadget ( uvc and a coupl

Re: [PATCH v8 1/5] x86: add simple udelay calibration

2017-05-02 Thread Boris Ostrovsky
On 03/21/2017 04:01 AM, Lu Baolu wrote: > Add a simple udelay calibration in x86 architecture-specific > boot-time initializations. This will get a workable estimate > for loops_per_jiffy. Hence, udelay() could be used after this > initialization. This breaks Xen PV guests since at this point, and

[PATCH] Fix for new version of realtek r8153

2017-05-02 Thread jake Briggs
From: jake Fix for new version of realtek r8153 not being recognised by the r8152 driver. The new version of the device with idVendor=0bda, idProduct=8153 is 6010 and treating it as if it was a 5c30 allow it to work. Error log: Apr 19 09:55:13 devotron kernel: [ 330.167074] usb 2-2.2: New USB

Re: dwc3 udc driver

2017-05-02 Thread Bin Liu
On Tue, May 02, 2017 at 05:05:32PM -0400, Ed Lizard wrote: > I'm having trouble seeing the udc driver for dwc3 in /sys/class/udc. > > We are moving platforms from musb on 2.6.37 to dwc3 on 4.6. Whew, a > lot has changed since we were stuck on the old kernel. > > It looks like libusbgx and udc is

Re: chipidea USB controller on IMX6solo processor does not give interrupt on quick connection and discconection

2017-05-02 Thread Peter Chen
On Tue, May 02, 2017 at 12:47:46PM +0530, Anshuman Tripathy wrote: > Hello All, > > > I am working on wanboard and the processor there is IMX6solo series . > The USB controller is chipidea one . > > The drivers are all chipidea opensource driver. > > > I have a requirement of connecting Iphone

Re: dwc3 udc driver

2017-05-02 Thread Bin Liu
On Tue, May 02, 2017 at 08:14:10PM -0500, Bin Liu wrote: > On Tue, May 02, 2017 at 05:05:32PM -0400, Ed Lizard wrote: > > I'm having trouble seeing the udc driver for dwc3 in /sys/class/udc. > > > > We are moving platforms from musb on 2.6.37 to dwc3 on 4.6. Whew, a > > lot has changed since we we

Re: Asmedia USB 1343 crashes

2017-05-02 Thread Thomas Fjellstrom
On Monday, May 1, 2017 12:08:55 PM MDT Thomas Fjellstrom wrote: > On Monday, May 1, 2017 1:57:53 PM MDT Alan Stern wrote: > > On Mon, 1 May 2017, Thomas Fjellstrom wrote: > > > On Monday, May 1, 2017 10:54:12 AM MDT Alan Stern wrote: > > > > On Mon, 1 May 2017, Thomas Fjellstrom wrote: > > > > > I'

Re: Asmedia USB 1343 crashes

2017-05-02 Thread Thomas Fjellstrom
On Tuesday, May 2, 2017 8:04:28 PM MDT Thomas Fjellstrom wrote: > On Monday, May 1, 2017 12:08:55 PM MDT Thomas Fjellstrom wrote: > > On Monday, May 1, 2017 1:57:53 PM MDT Alan Stern wrote: > > > On Mon, 1 May 2017, Thomas Fjellstrom wrote: > > > > On Monday, May 1, 2017 10:54:12 AM MDT Alan Stern

Re: Chipidea USB controller hangs in peripheral mode under high memory bus pressure

2017-05-02 Thread Peter Chen
On Tue, May 02, 2017 at 03:07:03PM +0300, Laurent Pinchart wrote: > Hello, > > I ran into an issue with a Xilinx Zynq XC7Z010 system. The system acts as a > USB peripheral, using the UVC gadget driver. > > When transferring high bandwidth data over USB in isochronous mode, complete > system fre

Re: I'd like to donate a MacBook Pro

2017-05-02 Thread Alex Henrie
2017-05-01 3:03 GMT-06:00 Lukas Wunner : > On Mon, 1 May 2017 00:27:41 -0600, Alex Henrie wrote: >> I am confident that this is a common problem because I have found >> various other users complaining about it: >> >> https://bbs.archlinux.org/viewtopic.php?pid=1613363#p1613363 >> https://bbs.archli