Re: [Bug] usb: dwc2: host function broken in OTG mode on bcm283x

2016-08-22 Thread Stefan Wahren
Hi John, > John Youn hat am 23. August 2016 um 00:08 > geschrieben: > > > On 8/22/2016 1:47 PM, Stefan Wahren wrote: > > Hi John, > > > >> John Youn hat am 22. August 2016 um 22:06 > >> geschrieben: > >> > >> > >> On 8/20/2016 6:03 AM, Stefan

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

2016-08-22 Thread Brian Norris
+ 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 salt. On Wed, Aug 10, 2016 at 04:32:15PM -0400, robert.f...@collabora.com wrote: > From: Robert Foss > >

Re: [PATCH v6 4/8] usb: core: add power sequence handling for USB devices

2016-08-22 Thread Peter Chen
On Mon, Aug 22, 2016 at 12:09:54PM -0400, Alan Stern wrote: > > > + > > > +static int hub_of_pwrseq_on(struct device_node *np, struct usb_hub *hub) > > > +{ > > > + struct pwrseq *pwrseq; > > > + struct usb_pwrseq_node *pwrseq_node; > > > + int ret; > > > + > > > + pwrseq = pwrseq_alloc_generic();

Re: musb: am3358: having problem with high-speed on usb1 at peripheral

2016-08-22 Thread Ayaka
從我的 iPad 傳送 > Bin Liu 於 2016年8月19日 上午12:08 寫道: > > Hi, > >> On Fri, Aug 12, 2016 at 10:23:15AM +0800, ayaka wrote: >> Hello all: >> I recently add a support for customize am3358 board using the branch >> processor-sdk-linux-03.00.00 from Ti git. But I meet a problem with musb

[PATCH] usb: phy: phy-generic: Check clk_prepare_enable() error

2016-08-22 Thread Fabio Estevam
From: Fabio Estevam clk_prepare_enable() may fail, so we should better check its return value and propagate it in the case of failure. Signed-off-by: Fabio Estevam --- drivers/usb/phy/phy-generic.c | 8 ++-- 1 file changed, 6 insertions(+), 2

Re: [PATCH v6 2/8] power: add power sequence library

2016-08-22 Thread Peter Chen
On Mon, Aug 22, 2016 at 12:23:31PM +0200, Sebastian Reichel wrote: > Hi Peter, > > On Mon, Aug 22, 2016 at 02:51:58PM +0800, Peter Chen wrote: > > On Mon, Aug 15, 2016 at 05:13:12PM +0800, Peter Chen wrote: > > > We have an well-known problem that the device needs to do some power > > > sequence

[PATCH v7] usb: ohci-at91: Forcibly suspend ports while USB suspend

2016-08-22 Thread Wenyou Yang
The usb controller does not manage correctly the suspend mode for the ehci. In echi mode, there is no way to suspend without any device connected to it. This is why this specific control is added to fix this issue. Since the suspend mode works in ohci mode, this specific control works by suspend

chipidea: udc: kernel panic in isr_setup_status_phase

2016-08-22 Thread Clemens Gruber
Hi, I am using an i.MX6Q embedded board, acting as a (ethernet) gadget with RNDIS function, connected over an USB OTG cable to a PC. Most of the time it works fine, but in some mysterious circumstances, a kernel panic occurs, just after attaching the OTG cable, connecting it to the other machine:

Bug 153551: Kernel panic on Nexus 5X USB unplug while tethering

2016-08-22 Thread Jose Marino
I'm using my phone (Nexus 5X running Android) to tether a USB connection to my laptop (XPS 15 9550). I plug the phone through the USB-C connection and in the phone I select USB tethering. Initially things look normal: a usb0 network interface appears in the laptop and it tries to get an IP

[PATCH v2 1/2] usb: dwc3: Add revision numbers for the USB 3.0 IP

2016-08-22 Thread John Youn
Add revision number constants for the 3.00a and 3.10a releases. Signed-off-by: John Youn --- drivers/usb/dwc3/core.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h index 3d94acd..002e647 100644 ---

[PATCH v2 2/2] usb: dwc3: Add ENDXFER command polling

2016-08-22 Thread John Youn
ENDXFER polling is available on version 3.10a and later of the DWC_usb3 (USB 3.0) controller. With this feature, the software can poll the CMDACT bit in the DEPCMD register after issuing an ENDXFER command. This feature is enabled by writing GUCTL2[14]. This feature is NOT available on the

Re: [Bug] usb: dwc2: host function broken in OTG mode on bcm283x

2016-08-22 Thread John Youn
On 8/22/2016 1:47 PM, Stefan Wahren wrote: > Hi John, > >> John Youn hat am 22. August 2016 um 22:06 >> geschrieben: >> >> >> On 8/20/2016 6:03 AM, Stefan Wahren wrote: >>> Hi John, >>> >>> >> >> Hi Stefan, >> >> Why doesn't DT work? I think all the properties are there

Re: UBSAN: Undefined behaviour in linux-4.7.2/drivers/usb/core/devio.c:1713:25

2016-08-22 Thread Vittorio Zecca
I can reproduce the UBSAN report by inserting in the USB receptacle a HUAWEI Mobile Broadband E353 HSPA+ USB Stick. Alan Stern asked for /sys/kernel/debug/usb/devices: cat debug-usb-devices T: Bus=01 Lev=00 Prnt=00 Port=00 Cnt=00 Dev#= 1 Spd=480 MxCh= 2 B: Alloc= 0/800 us ( 0%), #Int= 0,

Re: UBSAN: Undefined behaviour in linux-4.7.2/drivers/usb/core/devio.c:1713:25

2016-08-22 Thread Alan Stern
On Mon, 22 Aug 2016, Bjørn Mork wrote: > Alan Stern writes: > > > On Mon, 22 Aug 2016, Bjørn Mork wrote: > > > >> Alan Stern writes: > >> > >> > On Sun, 21 Aug 2016, Jiri Slaby wrote: > >> > > >> >> Cc: proper lists. > >> >> > >> >>

[PATCH] USB: fix typo in wMaxPacketSize validation

2016-08-22 Thread Alan Stern
The maximum value allowed for wMaxPacketSize of a high-speed interrupt endpoint is 1024 bytes, not 1023. Signed-off-by: Alan Stern Fixes: aed9d65ac327 ("USB: validate wMaxPacketValue entries in endpoint descriptors") CC: --- [as1809]

Re: UBSAN: Undefined behaviour in linux-4.7.2/drivers/usb/core/devio.c:1713:25

2016-08-22 Thread Bjørn Mork
Alan Stern writes: > On Mon, 22 Aug 2016, Bjørn Mork wrote: > >> Alan Stern writes: >> >> > On Sun, 21 Aug 2016, Jiri Slaby wrote: >> > >> >> Cc: proper lists. >> >> >> >> ep->desc.bInterval seems to be 0 here. > >> > As far as I can see,

Re: [Bug] usb: dwc2: host function broken in OTG mode on bcm283x

2016-08-22 Thread Stefan Wahren
Hi John, > John Youn hat am 22. August 2016 um 22:06 > geschrieben: > > > On 8/20/2016 6:03 AM, Stefan Wahren wrote: > > Hi John, > > > > > > Hi Stefan, > > Why doesn't DT work? I think all the properties are there to set > these. it only works for the future

Re: UBSAN: Undefined behaviour in linux-4.7.2/drivers/usb/core/devio.c:1713:25

2016-08-22 Thread Alan Stern
On Mon, 22 Aug 2016, Bjørn Mork wrote: > Alan Stern writes: > > > On Sun, 21 Aug 2016, Jiri Slaby wrote: > > > >> Cc: proper lists. > >> > >> ep->desc.bInterval seems to be 0 here. > > As far as I can see, this isn't possible. The usb_parse_endpoint() > > routine

Re: UBSAN: Undefined behaviour in linux-4.7.2/drivers/usb/core/devio.c:1713:25

2016-08-22 Thread Bjørn Mork
Alan Stern writes: > On Sun, 21 Aug 2016, Jiri Slaby wrote: > >> Cc: proper lists. >> >> ep->desc.bInterval seems to be 0 here. >> >> On 08/21/2016, 12:42 PM, Vittorio Zecca wrote: >> > I am not sure this is the right place so please bear with me... >> > From

Re: [Bug] usb: dwc2: host function broken in OTG mode on bcm283x

2016-08-22 Thread John Youn
On 8/20/2016 6:03 AM, Stefan Wahren wrote: > Hi John, > > on bcm2835 in OTG mode the USB host function broken (USB devices don't > enumerate) with the following settings: > > CONFIG_USB=y > CONFIG_USB_OTG=y > CONFIG_USB_DWC2=y > CONFIG_USB_DWC2_DUAL_ROLE=y > CONFIG_USB_PHY=y >

Re: [PATCH v2 0/7] musb: sunxi: Add support for run-time changing dr-mode through sysfs

2016-08-22 Thread Rask Ingemann Lambertsen
On Fri, Aug 19, 2016 at 04:25:40PM -0500, Bin Liu wrote: > Hi, > > On Mon, Aug 15, 2016 at 09:21:25PM +0200, Hans de Goede wrote: > > Hi All, > > > > Here is a patch series which implements run-time changing the dr-mode > > of sunxi musb controllers through the (already existing) musb "mode" > >

Re: [patch] USB: serial: option: add WeTelecom WM-D200

2016-08-22 Thread Dan Williams
On Mon, 2016-08-22 at 20:07 +0300, Aleksandr Makarov wrote: > 22.08.2016 18:03, Dan Williams пишет: > > > > On Sat, 2016-08-20 at 14:50 +0300, Aleksandr Makarov wrote: > > > > > > USB: serial: option: add WeTelecom WM-D200 > > > > > > Add support for WeTelecom WM-D200. > > > > > > T:  Bus=03

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

2016-08-22 Thread Alan Stern
On Mon, 22 Aug 2016, Ritesh Raj Sarraf wrote: > Hello Mathias, > > I finally reproduced the bug. So the linked usbmon log should have it. But > its a > whopping 22 MiB. It seems it takes a couple of hours of the OS running to > trigger the bug. > >

Re: [PATCH 2/2] usb: dwc3: Add ENDXFER command polling

2016-08-22 Thread John Youn
On 8/22/2016 12:49 AM, Felipe Balbi wrote: > > Hi, > > John Youn writes: >> ENDXFER polling is available on version 3.10a and later of the >> DWC_usb3 (USB 3.0) controller. With this feature, the software can poll >> the CMDACT bit in the DEPCMD register after issuing an

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

2016-08-22 Thread Ritesh Raj Sarraf
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On Mon, 2016-08-22 at 21:01 +0530, Ritesh Raj Sarraf wrote: > >  > > Even after fresh unload/load, the acutal problematic devid "0bda:0129", does > > not appear. I'll report back again after a fresh reboot. > >  > >

Re: UBSAN: Undefined behaviour in linux-4.7.2/drivers/usb/core/devio.c:1713:25

2016-08-22 Thread Alan Stern
On Sun, 21 Aug 2016, Jiri Slaby wrote: > Cc: proper lists. > > ep->desc.bInterval seems to be 0 here. > > On 08/21/2016, 12:42 PM, Vittorio Zecca wrote: > > I am not sure this is the right place so please bear with me... > > From Vittorio Zecca > > > > After compiling kernel 4.7.2 with ubsan I

Re: [patch] USB: serial: option: add WeTelecom WM-D200

2016-08-22 Thread Aleksandr Makarov
22.08.2016 18:03, Dan Williams пишет: > On Sat, 2016-08-20 at 14:50 +0300, Aleksandr Makarov wrote: >> USB: serial: option: add WeTelecom WM-D200 >> >> Add support for WeTelecom WM-D200. >> >> T: Bus=03 Lev=01 Prnt=01 Port=01 Cnt=01 Dev#= 4 Spd=12 MxCh= 0 >> D: Ver= 1.10 Cls=00(>ifc ) Sub=00

Re: [PATCH 2/2] dmaengine: cppi41: Add basic PM runtime support

2016-08-22 Thread Vinod Koul
On Mon, Aug 22, 2016 at 07:21:48AM -0700, Tony Lindgren wrote: > Hi, > > * Vinod Koul [160821 23:34]: > > On Fri, Aug 19, 2016 at 03:59:40PM -0700, Tony Lindgren wrote: > > @@ -349,6 +358,12 @@ static dma_cookie_t cppi41_tx_submit(struct > > dma_async_tx_descriptor *tx) >

Re: [PATCH v6] usb: ohci-at91: Forcibly suspend ports while USB suspend

2016-08-22 Thread Alan Stern
On Mon, 22 Aug 2016, Wenyou Yang wrote: > The usb controller does not manage correctly the suspend mode for > the ehci. In echi mode, there is no way to suspend without any > device connected to it. This is why this specific control is added > to fix this issue. Since the suspend mode works in

Re: [PATCH v6 4/8] usb: core: add power sequence handling for USB devices

2016-08-22 Thread Alan Stern
On Mon, 22 Aug 2016, Peter Chen wrote: > On Mon, Aug 15, 2016 at 05:13:14PM +0800, Peter Chen wrote: > > Some hard-wired USB devices need to do power sequence to let the > > device work normally, the typical power sequence like: enable USB > > PHY clock, toggle reset pin, etc. But current Linux

Re: [PATCH v2 7/7] musb: sunxi: Add support for platform_set_mode

2016-08-22 Thread Bin Liu
On Mon, Aug 22, 2016 at 05:55:50PM +0200, Hans de Goede wrote: > Hi, > > On 22-08-16 17:38, Bin Liu wrote: > >On Mon, Aug 22, 2016 at 05:32:57PM +0200, Hans de Goede wrote: > > > > >>>When switching from host to peripheral mode, if an usb device is still > >>>plugged and enumerated,

Re: [PATCH v2 0/7] musb: sunxi: Add support for run-time changing dr-mode through sysfs

2016-08-22 Thread Bin Liu
Hi, On Mon, Aug 22, 2016 at 05:50:25PM +0200, Hans de Goede wrote: > Hi, > > On 22-08-16 16:08, Bin Liu wrote: > >Hi, > > > >On Sun, Aug 21, 2016 at 11:29:02AM +0200, Hans de Goede wrote: > >>Hi, > >> > >>On 19-08-16 23:25, Bin Liu wrote: > >>>Hi, > >>> > >>>On Mon, Aug 15, 2016 at 09:21:25PM

Re: [PATCH v2 7/7] musb: sunxi: Add support for platform_set_mode

2016-08-22 Thread Hans de Goede
Hi, On 22-08-16 17:38, Bin Liu wrote: On Mon, Aug 22, 2016 at 05:32:57PM +0200, Hans de Goede wrote: When switching from host to peripheral mode, if an usb device is still plugged and enumerated, how do you handle the device disconnect? The phy code will report vbus low for long enough

Re: [PATCH v2 0/7] musb: sunxi: Add support for run-time changing dr-mode through sysfs

2016-08-22 Thread Hans de Goede
Hi, On 22-08-16 16:08, Bin Liu wrote: Hi, On Sun, Aug 21, 2016 at 11:29:02AM +0200, Hans de Goede wrote: Hi, On 19-08-16 23:25, Bin Liu wrote: Hi, On Mon, Aug 15, 2016 at 09:21:25PM +0200, Hans de Goede wrote: Hi All, Here is a patch series which implements run-time changing the dr-mode

Re: [PATCH v2 7/7] musb: sunxi: Add support for platform_set_mode

2016-08-22 Thread Bin Liu
On Mon, Aug 22, 2016 at 05:32:57PM +0200, Hans de Goede wrote: > HI, > > On 22-08-16 17:24, Bin Liu wrote: > >On Mon, Aug 22, 2016 at 05:08:56PM +0200, Hans de Goede wrote: > >>Hi, > >> > >>On 22-08-16 16:11, Bin Liu wrote: > >>>Hi, > >>> > >>>On Sun, Aug 21, 2016 at 12:10:26PM +0200, Hans de

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

2016-08-22 Thread Ritesh Raj Sarraf
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Dropping Alan from To list. On Mon, 2016-08-22 at 20:09 +0530, Ritesh Raj Sarraf wrote: > On Mon, 2016-08-22 at 18:11 +0530, Ritesh Raj Sarraf wrote: > > > > > > I'll post back again once I have captured the relevant logs. > > > > Thank you for

Re: [PATCH v2 7/7] musb: sunxi: Add support for platform_set_mode

2016-08-22 Thread Hans de Goede
HI, On 22-08-16 17:24, Bin Liu wrote: On Mon, Aug 22, 2016 at 05:08:56PM +0200, Hans de Goede wrote: Hi, On 22-08-16 16:11, Bin Liu wrote: Hi, On Sun, Aug 21, 2016 at 12:10:26PM +0200, Hans de Goede wrote: Hi, On 19-08-16 23:30, Bin Liu wrote: Hi, On Mon, Aug 15, 2016 at 09:21:32PM

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

2016-08-22 Thread Alan Stern
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 > --- > drivers/usb/host/ohci-omap.c | 1 - > 1 file

Re: [PATCH v2 7/7] musb: sunxi: Add support for platform_set_mode

2016-08-22 Thread Bin Liu
On Mon, Aug 22, 2016 at 05:08:56PM +0200, Hans de Goede wrote: > Hi, > > On 22-08-16 16:11, Bin Liu wrote: > >Hi, > > > >On Sun, Aug 21, 2016 at 12:10:26PM +0200, Hans de Goede wrote: > >>Hi, > >> > >>On 19-08-16 23:30, Bin Liu wrote: > >>>Hi, > >>> > >>>On Mon, Aug 15, 2016 at 09:21:32PM +0200,

Re: [PATCH v2 7/7] musb: sunxi: Add support for platform_set_mode

2016-08-22 Thread Hans de Goede
Hi, On 22-08-16 16:11, Bin Liu wrote: Hi, On Sun, Aug 21, 2016 at 12:10:26PM +0200, Hans de Goede wrote: Hi, On 19-08-16 23:30, Bin Liu wrote: Hi, On Mon, Aug 15, 2016 at 09:21:32PM +0200, Hans de Goede wrote: This allows run-time dr_mode switching support via the "mode" musb sysfs

Re: [patch] USB: serial: option: add WeTelecom WM-D200

2016-08-22 Thread Dan Williams
On Sat, 2016-08-20 at 14:50 +0300, Aleksandr Makarov wrote: > USB: serial: option: add WeTelecom WM-D200 > > Add support for WeTelecom WM-D200. > > T:  Bus=03 Lev=01 Prnt=01 Port=01 Cnt=01 Dev#=  4 Spd=12  MxCh= 0 > D:  Ver= 1.10 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs=  1 > P:  Vendor=22de

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

2016-08-22 Thread Ritesh Raj Sarraf
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On Mon, 2016-08-22 at 18:11 +0530, Ritesh Raj Sarraf wrote: > > I'll post back again once I have captured the relevant logs. > > Thank you for looking into this. It got more tricky to pick the logs. When the bug is hit, the device vanishes from

Re: [PATCH 2/2] dmaengine: cppi41: Add basic PM runtime support

2016-08-22 Thread Tony Lindgren
Hi, * Vinod Koul [160821 23:34]: > On Fri, Aug 19, 2016 at 03:59:40PM -0700, Tony Lindgren wrote: > @@ -349,6 +358,12 @@ static dma_cookie_t cppi41_tx_submit(struct > dma_async_tx_descriptor *tx) > > static int cppi41_dma_alloc_chan_resources(struct dma_chan *chan) > > {

Re: [PATCH 0/4] Implement PM runtime for musb-core based on session bit

2016-08-22 Thread Tony Lindgren
* Andreas Kemnade [160821 22:29]: > Hi, > > On Thu, 18 Aug 2016 15:46:46 -0700 > Tony Lindgren wrote: > > > Hi all, > > > > Here's a series of patches to simplify musb PM runtime support > > further. > > > > I finally figured out that we can get rid of

Re: [PATCH v2 0/7] musb: sunxi: Add support for run-time changing dr-mode through sysfs

2016-08-22 Thread Bin Liu
On Mon, Aug 22, 2016 at 09:08:29AM -0500, Bin Liu wrote: > Hi, > > On Sun, Aug 21, 2016 at 11:29:02AM +0200, Hans de Goede wrote: > > Hi, > > > > On 19-08-16 23:25, Bin Liu wrote: > > >Hi, > > > > > >On Mon, Aug 15, 2016 at 09:21:25PM +0200, Hans de Goede wrote: > > >>Hi All, > > >> > > >>Here

Re: [PATCH v2 7/7] musb: sunxi: Add support for platform_set_mode

2016-08-22 Thread Bin Liu
Hi, On Sun, Aug 21, 2016 at 12:10:26PM +0200, Hans de Goede wrote: > Hi, > > On 19-08-16 23:30, Bin Liu wrote: > >Hi, > > > >On Mon, Aug 15, 2016 at 09:21:32PM +0200, Hans de Goede wrote: > >>This allows run-time dr_mode switching support via the "mode" musb > >>sysfs attribute. > >> >

Re: [PATCH v2 0/7] musb: sunxi: Add support for run-time changing dr-mode through sysfs

2016-08-22 Thread Bin Liu
Hi, On Sun, Aug 21, 2016 at 11:29:02AM +0200, Hans de Goede wrote: > Hi, > > On 19-08-16 23:25, Bin Liu wrote: > >Hi, > > > >On Mon, Aug 15, 2016 at 09:21:25PM +0200, Hans de Goede wrote: > >>Hi All, > >> > >>Here is a patch series which implements run-time changing the dr-mode > >>of sunxi musb

Re: cdc_acm bug? read buffer bytes shifted

2016-08-22 Thread Oliver Neukum
On Sat, 2016-08-20 at 14:14 +0200, Julio Guerra wrote: > >> Another example: > >>> 00 00 00 00 00 00 00 01 > >> ... > >>> 01 00 00 00 00 00 00 00 > >> ... > >>> 00 01 00 00 00 00 00 00 > >> ... > >>> 00 00 01 00 00 00 00 00 > >> after a random number of times, while usbmon shows the usb payload >

Re: [PATCH] mmc: host: vub300: don't print error when allocating urb fails

2016-08-22 Thread Ulf Hansson
On 11 August 2016 at 23:03, Wolfram Sang wrote: > kmalloc will print enough information in case of failure. > > Signed-off-by: Wolfram Sang Thanks, applied for next! Kind regards Uffe > --- > drivers/mmc/host/vub300.c | 2 -- > 1

Re: [PATCH V9 1/1] usb:serial: Add Fintek F81532/534 driver

2016-08-22 Thread Johan Hovold
On Tue, May 31, 2016 at 09:51:20AM +0800, Ji-Ze Hong (Peter Hong) wrote: > This driver is for Fintek F81532/F81534 USB to Serial Ports IC. > > F81532 spec: > https://drive.google.com/file/d/0B8vRwwYO7aMFOTRRMmhWQVNvajQ/view? > usp=sharing > > F81534 spec: >

Re: [PATCH 2/4] phy: rockchip-usb: use rockchip_usb_phy_reset to reset phy during wakeup

2016-08-22 Thread Heiko Stübner
Am Montag, 22. August 2016, 17:17:41 schrieb Kishon Vijay Abraham I: > Hi, > > On Sunday 21 August 2016 02:02 AM, Randy Li wrote: > > It is a hardware bug in RK3288, the only way to solve it is to > > reset the phy. > > > > Signed-off-by: Randy Li > > --- > > > >

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

2016-08-22 Thread Ritesh Raj Sarraf
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On Mon, 2016-08-22 at 11:24 +0300, Mathias Nyman wrote: > * A few disconnects, were these real disconnects done by you?: No. Nothing was even plugged into the USB ports. And the resets keep recurring. >   Aug 18 23:55:57 learner kernel: usb 1-4:

[PATCHv6 3/3] mfd: intel_soc_pmic_bxtwc: add support for USB Type-C PHY on WhiskeyCove

2016-08-22 Thread Heikki Krogerus
Intel WhiskeyCove PMIC has also a USB Type-C PHY, so let's create a device for it. Signed-off-by: Heikki Krogerus Cc: Lee Jones --- drivers/mfd/intel_soc_pmic_bxtwc.c | 11 +++ 1 file changed, 11 insertions(+) diff --git

[PATCHv6 2/3] usb: typec: add driver for Intel Whiskey Cove PMIC USB Type-C PHY

2016-08-22 Thread Heikki Krogerus
This adds driver for the USB Type-C PHY on Intel WhiskeyCove PMIC which is available on some of the Intel Broxton SoC based platforms. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/Kconfig | 14 ++ drivers/usb/typec/Makefile | 1 +

[PATCHv6 0/3] USB Type-C Connector class

2016-08-22 Thread Heikki Krogerus
The USB Type-C class is meant to provide unified interface to the userspace to present the USB Type-C ports in a system. Changes since v5: - Only updating the roles based on driver notifications - Added MODULE_ALIAS for the WhiskeyCove module - Including the patch that creates the actual platform

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

2016-08-22 Thread Heikki Krogerus
The purpose of USB Type-C connector class is to provide unified interface for the user space to get the status and basic information about USB Type-C connectors on a system, control over data role swapping, and when the port supports USB Power Delivery, also control over power role swapping and

Re: [PATCH 2/4] phy: rockchip-usb: use rockchip_usb_phy_reset to reset phy during wakeup

2016-08-22 Thread Kishon Vijay Abraham I
Hi, On Sunday 21 August 2016 02:02 AM, Randy Li wrote: > It is a hardware bug in RK3288, the only way to solve it is to > reset the phy. > > Signed-off-by: Randy Li > --- > drivers/phy/phy-rockchip-usb.c | 20 > 1 file changed, 20 insertions(+) > > diff

Re: [PATCH 4/4] ARM: dts: rockchip: Point rk3288 dwc2 usb at the full PHY reset

2016-08-22 Thread Sergei Shtylyov
Hello. On 8/21/2016 3:19 PM, ayaka wrote: The "host1" port (AKA the dwc2 port that isn't the OTG port) on rk3288 has a hardware errata that causes everything to get confused when we get a remote wakeup. We'll use the reset that's in the CRU to reset the port when it's in a bad state. Note

Re: [PATCH v6 2/8] power: add power sequence library

2016-08-22 Thread Sebastian Reichel
Hi Peter, On Mon, Aug 22, 2016 at 02:51:58PM +0800, Peter Chen wrote: > On Mon, Aug 15, 2016 at 05:13:12PM +0800, Peter Chen wrote: > > We have an well-known problem that the device needs to do some power > > sequence before it can be recognized by related host, the typical > > example like

(U.N.O/W.B.O/18/08/2016/5/9/82).

2016-08-22 Thread U / N
Reimbursement Payment contact Mark Ford: claims.treas@gmail.com -- 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: How the EHCI HC driver make the decision to suspend some USB devices?

2016-08-22 Thread Oliver Neukum
On Fri, 2016-08-19 at 10:16 -0400, Alan Stern wrote: > On Fri, 19 Aug 2016, ludeng wrote: > > > Hi, > > How the EHCI HC driver make the decision to suspend some USB devices, > > but not to suspend some others? We notice that for some USB Video > > Cameras, when they are enumerated and there is no

[PATCH 4/4] usb: renesas_usbhs: set quirk_avoids_skb_reserve if USB-DMAC is used

2016-08-22 Thread Yoshihiro Shimoda
This patch sets the quirk_avoids_skb_reserve flag to improve performance of a network gadget driver (e.g. f_ncm.c) if USB-DMAC is used. For example (on r8a7795 board + f_ncm.c + iperf udp mode / receiving): - without this patch: 90.3 Mbits/sec - withthis patch: 273 Mbits/sec

[PATCH 2/4] usb: gadget: u_ether: add a flag to avoid skb_reserve() calling

2016-08-22 Thread Yoshihiro Shimoda
This patch adds a flag "no_skb_reserve" in struct eth_dev. So, if a peripheral driver sets the quirk_avoids_skb_reserve flag, upper network gadget drivers (e.g. f_ncm.c) can avoid skb_reserve() calling using the flag as well. Signed-off-by: Yoshihiro Shimoda ---

[PATCH 1/4] usb: gadget: add a new quirk to avoid skb_reserve in u_ether.c

2016-08-22 Thread Yoshihiro Shimoda
Some platforms (e.g. USB-DMAC on R-Car SoCs) has memory alignment restriction. If memory alignment is not match, the usb peripheral driver decides not to use the DMA controller. Then, the performance is not good. In the case of u_ether.c, since it calls skb_reserve() in rx_submit(), it is

[PATCH 3/4] usb: gadget: f_ncm: add support for no_skb_reserve

2016-08-22 Thread Yoshihiro Shimoda
This patch adds to support no_skb_reserve function to improve performance for some platforms. About the detail, please refer to the commit log of "quirk_avoids_skb_reserve" in include/linux/usb/gadget.h. Signed-off-by: Yoshihiro Shimoda ---

[PATCH 0/4] usb: gadget: add a new quirk to avoid skb_reserve in u_ether.c

2016-08-22 Thread Yoshihiro Shimoda
This patch set is based on the latest Felipe's usb.git / testing/next branch. (commit id = fcb8eae8d83c12770f8de915df01f7191dc697a3) I submitted this patch set as RFC in last month and nobody said anything. So, I submitted this without RFC :) Changes from RFC version: - rebase on the latest

Re: [PATCH] usb: chipidea: support generic phy in PM code path

2016-08-22 Thread Jisheng Zhang
On Mon, 22 Aug 2016 13:53:37 +0530 Kishon Vijay Abraham I wrote: > Hi, > > On Monday 22 August 2016 01:44 PM, Jisheng Zhang wrote: > > + Kishon > > > > Hi Peter, Kishon, > > > > On Mon, 22 Aug 2016 15:29:07 +0800 Peter Chen wrote: > > > >> On Mon, Aug 22, 2016 at 02:39:21PM +0800, Jisheng

Re: pwc over musb: 100% frame drop (lost) on high resolution stream

2016-08-22 Thread Matwey V. Kornilov
2016-08-22 1:00 GMT+03:00 Alan Stern : > On Sun, 21 Aug 2016, Matwey V. Kornilov wrote: > >> In both cases (with or without HCD_BH), usb_hcd_giveback_urb is called >> every 0.01 sec. It is not clear why behavior is so different. > > What behavior are you asking about?

Re: [PATCH] usb: chipidea: support generic phy in PM code path

2016-08-22 Thread Kishon Vijay Abraham I
Hi, On Monday 22 August 2016 01:44 PM, Jisheng Zhang wrote: > + Kishon > > Hi Peter, Kishon, > > On Mon, 22 Aug 2016 15:29:07 +0800 Peter Chen wrote: > >> On Mon, Aug 22, 2016 at 02:39:21PM +0800, Jisheng Zhang wrote: >>> Hi Peter, >>> >>> On Mon, 22 Aug 2016 13:33:08 +0800 Peter Chen wrote:

Re: [PATCH] usb: chipidea: support generic phy in PM code path

2016-08-22 Thread Jisheng Zhang
+ Kishon Hi Peter, Kishon, On Mon, 22 Aug 2016 15:29:07 +0800 Peter Chen wrote: > On Mon, Aug 22, 2016 at 02:39:21PM +0800, Jisheng Zhang wrote: > > Hi Peter, > > > > On Mon, 22 Aug 2016 13:33:08 +0800 Peter Chen wrote: > > > > > On Fri, Aug 19, 2016 at 08:10:13PM +0800, Jisheng Zhang

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

2016-08-22 Thread Mathias Nyman
On 19.08.2016 13:26, Ritesh Raj Sarraf wrote: Hello Mathia, On Fri, 2016-08-19 at 11:42 +0300, Mathias Nyman wrote: It'd help to know if it really is a bug or something else. Could you add xhci debugging?. The current logs mainly show usb core reporting errors on urbs returned (or timeout)

Re: [PATCH 2/2] usb: dwc3: Add ENDXFER command polling

2016-08-22 Thread Felipe Balbi
Hi, John Youn writes: > ENDXFER polling is available on version 3.10a and later of the > DWC_usb3 (USB 3.0) controller. With this feature, the software can poll > the CMDACT bit in the DEPCMD register after issuing an ENDXFER command. > This feature is enabled by writing

Re: [PATCH v4 10/10] usb: gadget: f_hid: use alloc_ep_req()

2016-08-22 Thread Felipe Balbi
Hi, John Youn writes: > On 8/8/2016 1:30 PM, Felipe F. Tonello wrote: >> Use gadget's framework allocation function instead of directly calling >> usb_ep_alloc_request(). >> >> Signed-off-by: Felipe F. Tonello >> --- >>

[PATCH v2] usb: renesas_usbhs: gadget: fix return value check in usbhs_mod_gadget_probe()

2016-08-22 Thread Felipe Balbi
From: Wei Yongjun In case of error, the function usb_get_phy() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Fixes: b5a2875605ca ("usb: renesas_usbhs: Allow an OTG PHY driver to provide VBUS") Cc:

Re: [PATCH 3/3] USB: serial: use IS_ENABLED() instead of checking for built-in or module

2016-08-22 Thread Johan Hovold
On Thu, Aug 18, 2016 at 09:45:06AM -0400, Javier Martinez Canillas wrote: > The IS_ENABLED() macro checks if a Kconfig symbol has been enabled either > built-in or as a module, use that macro instead of open coding the same. > > Using the macro makes the code more readable by helping abstract

Re: [PATCH -next] xhci: fix non static symbol warning

2016-08-22 Thread Felipe Balbi
Hi, Wei Yongjun writes: > Fixes the following sparse warning: > > drivers/usb/host/xhci-ring.c:608:6: warning: > symbol 'xhci_unmap_td_bounce_buffer' was not declared. Should it be static? > > Signed-off-by: Wei Yongjun Reviewed-by: Felipe Balbi

Re: [PATCH] usb: chipidea: support generic phy in PM code path

2016-08-22 Thread Peter Chen
On Mon, Aug 22, 2016 at 02:39:21PM +0800, Jisheng Zhang wrote: > Hi Peter, > > On Mon, 22 Aug 2016 13:33:08 +0800 Peter Chen wrote: > > > On Fri, Aug 19, 2016 at 08:10:13PM +0800, Jisheng Zhang wrote: > > > Support generic phy in PM code path: call phy_power_off/phy_power_on > > > in

RE: [PATCH -next] usb: renesas_usbhs: gadget: fix return value check in usbhs_mod_gadget_probe()

2016-08-22 Thread Felipe Balbi
Hi, Yoshihiro Shimoda writes: >> From: Wei Yongjun >> Sent: Saturday, August 13, 2016 10:28 AM >> >> In case of error, the function usb_get_phy() returns ERR_PTR() and never >> returns NULL. The NULL test in the return value check should be replaced >> with

RE: [PATCH/RFC 0/4] usb: gadget: add a new quirk to avoid skb_reserve in u_ether.c

2016-08-22 Thread Yoshihiro Shimoda
Hi Felipe-san, > From: Felipe Balbi > Sent: Thursday, August 18, 2016 4:35 PM > > Yoshihiro-san, > > Yoshihiro Shimoda writes: > > This patch set is based on the latest Felipe's usb.git / testing/next > > branch. > > (commit id =

Re: [PATCH v6 4/8] usb: core: add power sequence handling for USB devices

2016-08-22 Thread Peter Chen
On Mon, Aug 15, 2016 at 05:13:14PM +0800, Peter Chen wrote: > Some hard-wired USB devices need to do power sequence to let the > device work normally, the typical power sequence like: enable USB > PHY clock, toggle reset pin, etc. But current Linux USB driver > lacks of such code to do it, it may

RE: [PATCH -next] usb: renesas_usbhs: gadget: fix return value check in usbhs_mod_gadget_probe()

2016-08-22 Thread Yoshihiro Shimoda
Hi, > From: Wei Yongjun > Sent: Saturday, August 13, 2016 10:28 AM > > In case of error, the function usb_get_phy() returns ERR_PTR() and never > returns NULL. The NULL test in the return value check should be replaced > with IS_ERR(). > > Signed-off-by: Wei Yongjun Thank

Re: [PATCH v6 2/8] power: add power sequence library

2016-08-22 Thread Peter Chen
On Mon, Aug 15, 2016 at 05:13:12PM +0800, Peter Chen wrote: > We have an well-known problem that the device needs to do some power > sequence before it can be recognized by related host, the typical > example like hard-wired mmc devices and usb devices. > > This power sequence is hard to be

Re: [PATCH 2/2] dmaengine: cppi41: Add basic PM runtime support

2016-08-22 Thread Vinod Koul
On Fri, Aug 19, 2016 at 03:59:40PM -0700, Tony Lindgren wrote: > Let's keep the device enabled between cppi41_dma_issue_pending() > and dmaengine_desc_get_callback_invoke() and rely on the PM runtime > autoidle timeout elsewhere. > > As the PM runtime is for whole device, not for each channel, >

Re: [PATCH] usb: chipidea: support generic phy in PM code path

2016-08-22 Thread Jisheng Zhang
Hi Peter, On Mon, 22 Aug 2016 13:33:08 +0800 Peter Chen wrote: > On Fri, Aug 19, 2016 at 08:10:13PM +0800, Jisheng Zhang wrote: > > Support generic phy in PM code path: call phy_power_off/phy_power_on > > in ci_controller_suspend/ci_controller_resume. > > > > Signed-off-by: Jisheng Zhang