[PATCH v1] dwc2: gadget: Fix ISOC IN DDMA PID bitfield value calculation

2018-05-30 Thread Minas Harutyunyan
PID bitfield in descriptor should be set based on particular request length, not based on EP's mc value. PID value can't be set to 0 even request length is 0. If request length is 0 then SP bit should be set along PID. Signed-off-by: Minas Harutyunyan --- drivers/usb/dwc2/gadget.c | 15

RE: [PATCH 21/22] USB: gadget: udc: renesas_usb3: no need to check return value of debugfs_create functions

2018-05-30 Thread Yoshihiro Shimoda
> From: Greg Kroah-Hartman, Sent: Wednesday, May 30, 2018 12:31 AM > > When calling debugfs functions, there is no need to ever check the > return value. The function can work or not, but the code logic should > never do something different based on this. > > Cc: Felipe Balbi > Cc: Yoshihiro Sh

[PATCH v2] dwc2: gadget: Fix ISOC IN DDMA PID bitfield value calculation

2018-05-30 Thread Minas Harutyunyan
PID bitfield in descriptor should be set based on particular request length, not based on EP's mc value. PID value can't be set to 0 even request length is 0. If request length is 0 then SP bit should be set along PID. Signed-off-by: Minas Harutyunyan --- drivers/usb/dwc2/gadget.c | 15

Re: [PATCH 01/22] USB: typec: tcpm: no need to check return value of debugfs_create_dir()

2018-05-30 Thread Heikki Krogerus
+Guenter On Tue, May 29, 2018 at 05:30:46PM +0200, Greg Kroah-Hartman wrote: > When calling debugfs functions, there is no need to ever check the > return value. The function can work or not, but the code logic should > never do something different based on this. > > Clean up the tcpm.c code to

Re: [PATCH 01/22] USB: typec: tcpm: no need to check return value of debugfs_create_dir()

2018-05-30 Thread Greg Kroah-Hartman
On Wed, May 30, 2018 at 01:30:20PM +0300, Heikki Krogerus wrote: > +Guenter > > On Tue, May 29, 2018 at 05:30:46PM +0200, Greg Kroah-Hartman wrote: > > When calling debugfs functions, there is no need to ever check the > > return value. The function can work or not, but the code logic should > >

Re: [RFT/PATCH 00/38] usb: dwc3: gadget: Rework & Refactoring

2018-05-30 Thread Felipe Balbi
Hi, Laurent Pinchart writes: >> I've been working on this series for a while now. I feels like >> after this series the transfer management code is far easier to >> read and understand. >> >> Based on my tests, I have no regressions. Tested g_mass_st

[bug report] usb: musb: break the huge isr musb_stage0_irq() into small functions

2018-05-30 Thread Dan Carpenter
Hello Bin Liu, The patch bcb8fd3a2fba: "usb: musb: break the huge isr musb_stage0_irq() into small functions" from May 21, 2018, leads to the following static checker warning: drivers/usb/musb/musb_core.c:797 musb_handle_intr_connect() error: we previously assumed 'musb->hcd' coul

Re: [PATCH 01/22] USB: typec: tcpm: no need to check return value of debugfs_create_dir()

2018-05-30 Thread Guenter Roeck
On 05/30/2018 03:30 AM, Heikki Krogerus wrote: +Guenter On Tue, May 29, 2018 at 05:30:46PM +0200, Greg Kroah-Hartman wrote: When calling debugfs functions, there is no need to ever check the return value. The function can work or not, but the code logic should never do something different base

Re: [PATCH 01/22] USB: typec: tcpm: no need to check return value of debugfs_create_dir()

2018-05-30 Thread Guenter Roeck
On 05/30/2018 03:36 AM, Greg Kroah-Hartman wrote: On Wed, May 30, 2018 at 01:30:20PM +0300, Heikki Krogerus wrote: +Guenter On Tue, May 29, 2018 at 05:30:46PM +0200, Greg Kroah-Hartman wrote: When calling debugfs functions, there is no need to ever check the return value. The function can wor

Re: xHCI host controller not responding, assume dead" on Dell XPS 13 9370

2018-05-30 Thread Alan Stern
On Wed, 30 May 2018, Rindt, Daniel wrote: > Dear Mr. Stern, > > i stumbled over your conversation regarding the sleep thing: > https://www.spinics.net/lists/linux-usb/msg168671.html > > I am not familiar with the handling of the mailing list, for that > reason i wrote you. Discussions like this

Re: [PATCH] arm64: dts: msm8996: Use dwc3-qcom glue driver for USB

2018-05-30 Thread Doug Anderson
Hi, On Wed, May 30, 2018 at 4:04 AM, Manu Gautam wrote: > Move from dwc3-of-simple to dwc3-qcom glue driver to > support peripheral mode which requires qscratch wrapper > programming on VBUS event. Fixes: a4333c3a6ba9 ("usb: dwc3: Add Qualcomm DWC3 glue driver") > Signed-off-by: Manu Gautam >

Re: [PATCH 17/22] USB: udc: atmel_usba_udc: no need to check return value of debugfs_create functions

2018-05-30 Thread Alexandre Belloni
Hi, On 29/05/2018 17:31:02+0200, Greg Kroah-Hartman wrote: > When calling debugfs functions, there is no need to ever check the > return value. The function can work or not, but the code logic should > never do something different based on this. > > There is also no need to keep the file dentrie

Re: [PATCH 17/22] USB: udc: atmel_usba_udc: no need to check return value of debugfs_create functions

2018-05-30 Thread Greg Kroah-Hartman
On Wed, May 30, 2018 at 07:31:13PM +0200, Alexandre Belloni wrote: > Hi, > > On 29/05/2018 17:31:02+0200, Greg Kroah-Hartman wrote: > > When calling debugfs functions, there is no need to ever check the > > return value. The function can work or not, but the code logic should > > never do somethi

Re: [PATCH 17/22] USB: udc: atmel_usba_udc: no need to check return value of debugfs_create functions

2018-05-30 Thread Alexandre Belloni
On 29/05/2018 17:31:02+0200, Greg Kroah-Hartman wrote: > When calling debugfs functions, there is no need to ever check the > return value. The function can work or not, but the code logic should > never do something different based on this. > > There is also no need to keep the file dentries aro

Re: [PATCH v2] usb: xhci: force all memory allocations to node

2018-05-30 Thread Adam Wallis
Mathias, On 5/22/2018 2:55 PM, Adam Wallis wrote: > The xhci driver forces DMA memory to be node aware, however, there are > several ring-related memory allocations that are not memory node aware. > This patch resolves those *alloc functions to be allocated on the proper > memory node. > > Signed

Re: [Query] checking hub port status while USB 2.0 port is resuming.

2018-05-30 Thread Alan Stern
On Mon, 28 May 2018, Anshuman Gupta wrote: > > It sounds like you want the code in usb_port_resume() to check and see > > whether the port has already received a wakeup signal and hasn't > > finished processing it yet. > yes. > > > > There's no way to do this for ports on external hubs, but in

Re: [PATCH v6 01/15] dt-bindings: connector: add properties for typec

2018-05-30 Thread Rob Herring
On Mon, May 28, 2018 at 10:52:33AM +0800, Li Jun wrote: > Add bindings supported by current typec driver, so user can pass > all those properties via dt. > > Signed-off-by: Li Jun > --- > .../bindings/connector/usb-connector.txt | 44 +++ > include/dt-bindings/usb/pd.h

Re: [RFT/PATCH 18/38] usb: dwc3: gadget: check for Missed Isoc from event status

2018-05-30 Thread Paul Zimmerman
Hi Felipe, Felipe Balbi writes: < snip > > thinking about this a little more. This extra list_empty() check is > not wrong at all :-) I've amended this series with the 3 patches > below. I'll resend the series once I've given more time for people to > test. Patches have been updated to the bran

Re: [RFT/PATCH 18/38] usb: dwc3: gadget: check for Missed Isoc from event status

2018-05-30 Thread Felipe Balbi
Hi, Paul Zimmerman writes: > Hi Felipe, > > Felipe Balbi writes: > > < snip > > >> thinking about this a little more. This extra list_empty() check is >> not wrong at all :-) I've amended this series with the 3 patches >> below. I'll resend the series once I've given more time for people to >>