[PATCH] Revert "usb: dwc3: gadget: always decrement by 1"

2016-08-26 Thread John Youn
This reverts commit 6f8245b4e37c ("usb: dwc3: gadget: always decrement by 1"). We can't always decrement this value. We should decrement only if the calculation of free slots results in a LINK TRB being among one of the free slots (dequeue < enqueue). Otherwise, if the LINK TRB is not among the

Re: UAS and f_tcm -- is anyone using it?

2016-08-26 Thread Alan Stern
On Fri, 26 Aug 2016, Felipe Balbi wrote: > Hi, > > John Youn writes: > > I was wondering if anyone is using the f_tcm function? Specifically > > for UAS in superspeed with streams? Any idea if it is being using in > > production for this anywhere? > > > > I've been

[PATCH v2 00/19] MIPS: SEAD3 device tree conversion

2016-08-26 Thread Paul Burton
Although the SEAD3 board has already made some minimal use of device tree, until now most peripherals have been left probed by platform code. This series converts all SEAD3 peripherals to instead be probed from device tree. The amount of platform code is significantly reduced leaving SEAD3 primed

[PATCH v2 07/19] usb: host: ehci-sead3: Remove SEAD-3 EHCI code

2016-08-26 Thread Paul Burton
The SEAD-3 board is now probing its EHCI controller using the generic EHCI driver & its generic-ehci device tree binding. Remove the unused SEAD-3 specific EHCI code. Signed-off-by: Paul Burton --- Changes in v2: - New patch, removing SEAD-3 EHCI code instead of

Re: [PATCHv6 1/3] usb: USB Type-C connector class

2016-08-26 Thread Vincent Palatin
On Thu, Aug 25, 2016 at 1:59 PM, Heikki Krogerus wrote: > Hi, > > On Wed, Aug 24, 2016 at 04:08:23PM +0200, Vincent Palatin wrote: >> Sorry if I'm making redundant comments with previous discussions, I >> might have missed a few threads. >> >> >> On Mon, Aug 22,

Re: [PATCHv6 1/3] usb: USB Type-C connector class

2016-08-26 Thread Heikki Krogerus
Hi Vincent, On Fri, Aug 26, 2016 at 03:16:16PM +0200, Vincent Palatin wrote: > >> > +What: /sys/class/typec//current_vconn_role > >> > +Date: June 2016 > >> > +Contact: Heikki Krogerus > >> > +Description: > >> > + Shows the

Re: xHCI problem? [was Re: Erratic USB device behavior and device loss]

2016-08-26 Thread Alan Stern
On Fri, 26 Aug 2016, Ritesh Raj Sarraf wrote: > I was able to reproduce the resets after disabling LPM too. > > [ 5026.849192] systemd[1]: apt-daily.timer: Adding 6h 13min 53.496331s random > time. > [ 5027.078155] systemd[1]: apt-daily.timer: Adding 5h 55min 19.813619s random > time. > [

Re: chipidea: udc: kernel panic in isr_setup_status_phase

2016-08-26 Thread Peter Chen
On Fri, Aug 26, 2016 at 01:47:40AM +0200, Clemens Gruber wrote: > On Wed, Aug 24, 2016 at 04:11:02PM +0800, Peter Chen wrote: > > UEI is an error interrupt, and software have not handled it, so it will > > not affect ci->status. > > > > > Should we only call isr_tr_complete_handler if UI && !UEI

Re: [PATCH v4 1/2] mfd: ti-smusbdig: Add support for the TI SM-USB-DIG

2016-08-26 Thread Johan Hovold
On Tue, Aug 09, 2016 at 09:55:16AM -0500, Andrew F. Davis wrote: > The TI SM-USB-DIG is a USB to SPI/I2C/1Wire/GPIO adapter. > Add MFD core support. > > Signed-off-by: Andrew F. Davis > --- > drivers/mfd/Kconfig | 9 +++ > drivers/mfd/Makefile| 2 + >

Re: chipidea: udc: kernel panic in isr_setup_status_phase

2016-08-26 Thread Peter Chen
On Fri, Aug 26, 2016 at 01:47:40AM +0200, Clemens Gruber wrote: > On Wed, Aug 24, 2016 at 04:11:02PM +0800, Peter Chen wrote: > > UEI is an error interrupt, and software have not handled it, so it will > > not affect ci->status. > > > > > Should we only call isr_tr_complete_handler if UI && !UEI

Bestätigung deiner Email Adresse

2016-08-26 Thread Newsletter
Hallo! Wir freuen uns, dass du dich für unseren kostenlosen Newsletter entschieden hast. Ab deiner Eintragung bekommst du von uns regelmäßig, komplett kostenfrei, alle aktuellen News zu dem von dir gewählten Thema auf deine E-Mailadresse zugestellt. Schließe deine Eintragung ab indem du

Re: [RESEND PATCH, v5 4/5] usb: Add MediaTek USB3 DRD Driver

2016-08-26 Thread chunfeng yun
Hi, On Thu, 2016-08-25 at 10:32 +0200, Oliver Neukum wrote: > On Thu, 2016-08-25 at 11:05 +0800, Chunfeng Yun wrote: > > This patch adds support for the MediaTek USB3 controller > > integrated into MT8173. It can be configured as Dual-Role > > Device (DRD), Peripheral Only and Host Only (xHCI)

Re: UAS and f_tcm -- is anyone using it?

2016-08-26 Thread Felipe Balbi
Hi, John Youn writes: > I was wondering if anyone is using the f_tcm function? Specifically > for UAS in superspeed with streams? Any idea if it is being using in > production for this anywhere? > > I've been trying to get the tcm gadget running without success. It >

Re: xHCI problem? [was Re: Erratic USB device behavior and device loss]

2016-08-26 Thread Ritesh Raj Sarraf
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hello Alan, On Thu, 2016-08-25 at 19:04 +0530, Ritesh Raj Sarraf wrote: > > As you mentioned above, there's another aspect to power management  > > besides runtime PM, namely Link Power Management.  Perhaps the device  > > can't handle LPM. > >  >

Re: [PATCH v2 4/6] usb: dwc3: gadget: add remaining sg entries to ring

2016-08-26 Thread Felipe Balbi
Hi, John Youn writes: >> Felipe Balbi writes: On your testing/next, I see considerable slow down and file corruption in mass storage. After bisecting, this patch seems to be the first one that shows problems. The

Re: [PATCH 4/4] usb: dwc3: gadget: always decrement by 1

2016-08-26 Thread Felipe Balbi
Hi, John Youn writes: > On 8/25/2016 12:38 AM, Felipe Balbi wrote: >> We need to decrement in both cases (enq > deq and >> enq < deq) >> >> Signed-off-by: Felipe Balbi >> --- >> drivers/usb/dwc3/gadget.c | 5 + >> 1 file changed, 1

Re: [PATCH] usb: musb: Fix locking errors for host only mode

2016-08-26 Thread Tony Lindgren
Hi, * Bin Liu [160825 10:19]: > Hi, > > On Thu, Aug 18, 2016 at 03:40:38PM -0700, Tony Lindgren wrote: > > If we have USB gadgets disabled and USB_MUSB_HOST set, we get > > errors "possible irq lock inverssion dependency detected" > > errors during boot. > > > > Let's fix the

Re: [PATCH] usb: musb: Fix locking errors for host only mode

2016-08-26 Thread Bin Liu
On Fri, Aug 26, 2016 at 07:57:39AM -0700, Tony Lindgren wrote: > * Bin Liu [160826 07:57]: > > On Fri, Aug 26, 2016 at 07:39:05AM -0700, Tony Lindgren wrote: > > > Hi, > > > > > > * Bin Liu [160825 10:19]: > > > > Hi, > > > > > > > > On Thu, Aug 18, 2016 at

Re: [PATCH] usb: musb: Fix locking errors for host only mode

2016-08-26 Thread Tony Lindgren
* Bin Liu [160826 08:15]: > > BTY, sorry for beeing slow on reviewing patches lately. There are a > few critical things in my work coming up during my vacation. > (maintaining musb is not part of my paid job...) No problem, we all know how that goes. Tony -- To unsubscribe from

Re: xHCI problem? [was Re: Erratic USB device behavior and device loss]

2016-08-26 Thread Ritesh Raj Sarraf
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hello Alan, On Fri, 2016-08-26 at 10:14 -0400, Alan Stern wrote: > > [ 8758.487792] usb 1-4: USB disconnect, device number 13 > > [ 8759.711773] usb 1-4: new high-speed USB device number 16 using xhci_hcd > > [ 8759.885543] usb 1-4: New USB device

Re: [PATCH v2 07/19] usb: host: ehci-sead3: Remove SEAD-3 EHCI code

2016-08-26 Thread Alan Stern
On Fri, 26 Aug 2016, Paul Burton wrote: > The SEAD-3 board is now probing its EHCI controller using the generic > EHCI driver & its generic-ehci device tree binding. Remove the unused > SEAD-3 specific EHCI code. > > Signed-off-by: Paul Burton > > --- > > Changes in

Re: [Umap2][5/11][22b8:2d93] NULL pointer dereference

2016-08-26 Thread Binyamin Sharet (bsharet)
> > On 26 Aug 2016, at 09:38, Oliver Neukum wrote: > > >> Hi Oliver, >> >> It will take me some time to test it on v4.4.6-v7, so I have tried applying >> the patch >> to v4.4 but it failed in the 4th hunk (line 47). >> >> Please advise, should I: >> >> - manually edit the

Re: [PATCH 1/2] usb: ohci-sa1111: remove machine_has_neponset()

2016-08-26 Thread Russell King - ARM Linux
On Fri, Aug 26, 2016 at 01:18:25PM -0400, Alan Stern wrote: > On Fri, 26 Aug 2016, Russell King wrote: > > > The neponset is a daughter board for the Assabet platform, which has a > > SA chip on it. If we're initialising the SA OHCI, and we're > > part of a neponset, the host platform

Re: [PATCH 1/2] usb: ohci-sa1111: remove machine_has_neponset()

2016-08-26 Thread Russell King - ARM Linux
On Fri, Aug 26, 2016 at 02:20:50PM -0400, Alan Stern wrote: > On Fri, 26 Aug 2016, Russell King - ARM Linux wrote: > > > On Fri, Aug 26, 2016 at 01:18:25PM -0400, Alan Stern wrote: > > > On Fri, 26 Aug 2016, Russell King wrote: > > > > > > > The neponset is a daughter board for the Assabet

Re: [PATCH 1/2] usb: ohci-sa1111: remove machine_has_neponset()

2016-08-26 Thread Alan Stern
On Fri, 26 Aug 2016, Russell King - ARM Linux wrote: > On Fri, Aug 26, 2016 at 01:18:25PM -0400, Alan Stern wrote: > > On Fri, 26 Aug 2016, Russell King wrote: > > > > > The neponset is a daughter board for the Assabet platform, which has a > > > SA chip on it. If we're initialising the

Re: [PATCH] USB: ohci-omap - avoid including mach/irqs.h

2016-08-26 Thread Russell King - ARM Linux
On Fri, Aug 26, 2016 at 01:20:53PM -0400, Alan Stern wrote: > On Fri, 26 Aug 2016, Russell King - ARM Linux wrote: > > > On Mon, Aug 22, 2016 at 11:28:21AM -0400, Alan Stern wrote: > > > On Fri, 19 Aug 2016, Russell King wrote: > > > > > > > ohci-omap doesn't need to include mach/irqs.h -

Re: [PATCH 4/4] usb: dwc3: gadget: always decrement by 1

2016-08-26 Thread John Youn
On 8/26/2016 3:06 AM, Felipe Balbi wrote: > > Hi, > > John Youn writes: >> On 8/25/2016 12:38 AM, Felipe Balbi wrote: >>> We need to decrement in both cases (enq > deq and >>> enq < deq) >>> >>> Signed-off-by: Felipe Balbi >>> --- >>>

Re: [PATCH V4] leds: trigger: Introduce an USB port trigger

2016-08-26 Thread Rafał Miłecki
On 25 August 2016 at 14:49, Greg KH wrote: > On Thu, Aug 25, 2016 at 10:03:52AM +0200, Rafał Miłecki wrote: >> +static void usbport_trig_activate(struct led_classdev *led_cdev) >> +{ >> + struct usbport_trig_data *usbport_data; >> + int err; >> + >> +

[PATCH 2/2] usb: ohci-sa1111: remove mach/hardware.h include

2016-08-26 Thread Russell King
The mach/hardware.h include doesn't seem to be necessary to build ohci-sa, so let's remove it to kill off an unnecessary platform specific include. Signed-off-by: Russell King --- drivers/usb/host/ohci-sa.c | 1 - 1 file changed, 1 deletion(-) diff --git

[PATCH 1/2] usb: ohci-sa1111: remove machine_has_neponset()

2016-08-26 Thread Russell King
The neponset is a daughter board for the Assabet platform, which has a SA chip on it. If we're initialising the SA OHCI, and we're part of a neponset, the host platform must be an Assabet. This allows us to eliminate machine_has_neponset() from this driver, replacing it instead with

Re: [PATCH] USB: ohci-omap - avoid including mach/irqs.h

2016-08-26 Thread Russell King - ARM Linux
On Mon, Aug 22, 2016 at 11:28:21AM -0400, Alan Stern wrote: > On Fri, 19 Aug 2016, Russell King wrote: > > > ohci-omap doesn't need to include mach/irqs.h - nothing within this > > driver needs anything from this header file. Remove this include. > > > > Signed-off-by: Russell King

Re: [PATCH 1/2] usb: ohci-sa1111: remove machine_has_neponset()

2016-08-26 Thread Alan Stern
On Fri, 26 Aug 2016, Russell King wrote: > The neponset is a daughter board for the Assabet platform, which has a > SA chip on it. If we're initialising the SA OHCI, and we're > part of a neponset, the host platform must be an Assabet. > > This allows us to eliminate

Re: [PATCH] USB: ohci-omap - avoid including mach/irqs.h

2016-08-26 Thread Alan Stern
On Fri, 26 Aug 2016, Russell King - ARM Linux wrote: > On Mon, Aug 22, 2016 at 11:28:21AM -0400, Alan Stern wrote: > > On Fri, 19 Aug 2016, Russell King wrote: > > > > > ohci-omap doesn't need to include mach/irqs.h - nothing within this > > > driver needs anything from this header file. Remove

Re: [PATCH] usb: musb: Fix locking errors for host only mode

2016-08-26 Thread Bin Liu
On Fri, Aug 26, 2016 at 07:39:05AM -0700, Tony Lindgren wrote: > Hi, > > * Bin Liu [160825 10:19]: > > Hi, > > > > On Thu, Aug 18, 2016 at 03:40:38PM -0700, Tony Lindgren wrote: > > > If we have USB gadgets disabled and USB_MUSB_HOST set, we get > > > errors "possible irq lock

Re: [PATCH] usb: musb: Fix locking errors for host only mode

2016-08-26 Thread Tony Lindgren
* Bin Liu [160826 07:57]: > On Fri, Aug 26, 2016 at 07:39:05AM -0700, Tony Lindgren wrote: > > Hi, > > > > * Bin Liu [160825 10:19]: > > > Hi, > > > > > > On Thu, Aug 18, 2016 at 03:40:38PM -0700, Tony Lindgren wrote: > > > > If we have USB gadgets disabled and

Re: [PATCH RFC V3.5] leds: trigger: Introduce an USB port trigger

2016-08-26 Thread Rafał Miłecki
On 25 August 2016 at 20:48, Jacek Anaszewski wrote: > On 08/25/2016 04:30 PM, Alan Stern wrote: >> >> On Thu, 25 Aug 2016, Jacek Anaszewski wrote: >> >>> I'd see it as follows: >>> >>> #cat available_ports >>> #1-1 1-2 2-1 >>> >>> #echo "1-1" > new_port >>> >>> #cat

Re: [PATCH RFC V3.5] leds: trigger: Introduce an USB port trigger

2016-08-26 Thread Pavel Machek
On Thu 2016-08-25 11:04:38, Jacek Anaszewski wrote: > On 08/25/2016 10:29 AM, Rafał Miłecki wrote: > >On 25 August 2016 at 10:03, Jacek Anaszewski > >wrote: > >>On 08/24/2016 07:52 PM, Rafał Miłecki wrote: > >>> > >>>From: Rafał Miłecki > >>> >

[PATCH 1/1] usb: musb: Fix locking errors for host only mode

2016-08-26 Thread Bin Liu
From: Tony Lindgren If we have USB gadgets disabled and USB_MUSB_HOST set, we get errors "possible irq lock inverssion dependency detected" errors during boot. Let's fix the issue by adding start_musb flag and start the controller after we're out of the spinlock protected

[PATCH 0/1] musb-fixes for v4.8-rc4

2016-08-26 Thread Bin Liu
Hi Greg, Here are musb fixes for v4.8-rc4. Only one fix for locking error. Please let me know if any change is needed. Regards, -Bin. Tony Lindgren (1): usb: musb: Fix locking errors for host only mode drivers/usb/musb/musb_virthub.c | 7 ++- 1 file changed, 6 insertions(+), 1

Re: UAS and f_tcm -- is anyone using it?

2016-08-26 Thread John Youn
On 8/26/2016 7:09 AM, Alan Stern wrote: > On Fri, 26 Aug 2016, Felipe Balbi wrote: > >> Hi, >> >> John Youn writes: >>> I was wondering if anyone is using the f_tcm function? Specifically >>> for UAS in superspeed with streams? Any idea if it is being using in >>>

Re: [PATCH RFC V3.5] leds: trigger: Introduce an USB port trigger

2016-08-26 Thread Pavel Machek
On Thu 2016-08-25 20:48:04, Jacek Anaszewski wrote: > On 08/25/2016 04:30 PM, Alan Stern wrote: > >On Thu, 25 Aug 2016, Jacek Anaszewski wrote: > > > >>I'd see it as follows: > >> > >>#cat available_ports > >>#1-1 1-2 2-1 > >> > >>#echo "1-1" > new_port > >> > >>#cat observed_ports > >>#1-1 > >> >

Re: UAS and f_tcm -- is anyone using it?

2016-08-26 Thread John Youn
On 8/26/2016 12:48 AM, Felipe Balbi wrote: > > Hi, > > John Youn writes: >> I was wondering if anyone is using the f_tcm function? Specifically >> for UAS in superspeed with streams? Any idea if it is being using in >> production for this anywhere? >> >> I've been trying

Re: [PACTH,v6,1/2] usb: xhci: plat: Enable runtime PM

2016-08-26 Thread Brian Norris
Hi, On Wed, Aug 24, 2016 at 04:48:01PM -0400, Robert Foss wrote: > On 2016-08-22 11:23 PM, Brian Norris wrote: > >+ others > > > >Hi Robert and Felipe, > > > >I have a few questions for one or both of you. I'm not really an expert > >on runtime PM, so please take my questions with a grain of

Re: [PACTH,v6,1/2] usb: xhci: plat: Enable runtime PM

2016-08-26 Thread Brian Norris
Corrected a bouncing email address (sorry for the noise) On Fri, Aug 26, 2016 at 03:11:36PM -0700, Brian Norris wrote: > Hi, > > On Wed, Aug 24, 2016 at 04:48:01PM -0400, Robert Foss wrote: > > On 2016-08-22 11:23 PM, Brian Norris wrote: > > >+ others > > > > > >Hi Robert and Felipe, > > > > >

[no subject]

2016-08-26 Thread Aleksandr Makarov
unsubscribe linux-usb -- 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: [Umap2][5/11][22b8:2d93] NULL pointer dereference

2016-08-26 Thread Oliver Neukum
> Hi Oliver, > > It will take me some time to test it on v4.4.6-v7, so I have tried applying > the patch > to v4.4 but it failed in the 4th hunk (line 47). > > Please advise, should I: > > - manually edit the file on v4.4 and retest? Please do that. Regards Oliver

Re: [PATCH 00/44] usb: don't print on ENOMEM

2016-08-26 Thread Lothar Waßmann
Hi, On Thu, 25 Aug 2016 19:38:52 +0200 Wolfram Sang wrote: > Here is my next series to save memory by removing unneeded strings. It removes > in the usb subsystem all unspecific error messages after calling malloc-based > functions, i.e. (devm_)k[zcm]alloc. kmalloc prints enough information in