Re: crash by cdc_acm driver in kernels 4.8-rc1/5

On Tue, Sep 20, 2016 at 03:05:14PM +0200, Oliver Neukum wrote: > > I cannot replicate it. Could you please provide "lsusb -v"? > > Regards > Oliver It concerns these type of modems: http://www.ebay.nl/itm/191933738340 http://www.ebay.nl/itm/121590899044 lsusb: Bus 002

g_webcam Isoch high bandwidth transfer

Hi, I am trying to check Isoch high bandwidth transfer with g_webcam.ko in high-speed connection. First I hacked webcam.c as follows to enable 640x480@30fps mode. diff --git a/drivers/usb/gadget/legacy/webcam.c b/drivers/usb/gadget/legacy/webcam.c index 72c976b..9eb315f 100644 ---

Re: 4.8-rc: runtime PM trying to activate child device host6 but parent (2-1.2:1.0) is not active

On Tue, 20 Sep 2016, Carsten Mattner wrote: > On Tue, Sep 20, 2016 at 2:36 PM, Oliver Neukum wrote: > > On Tue, 2016-09-20 at 14:11 +0200, Carsten Mattner wrote: > >> 4.8-rc5 was fine, but 4.8-rc6 and 4.8-rc7 on a Lenovo Thinkpad x220 report > >> this > >> warning or error

Re: [PATCHv2] usb: musb: Fix unbalanced platform_disable

* Laurent Pinchart [160919 23:36]: > Hi Tony, > > On Monday 19 Sep 2016 15:41:50 Tony Lindgren wrote: > > * Laurent Pinchart [160919 13:35]: > > > On Sunday 18 Sep 2016 08:19:02 Tony Lindgren wrote: > > >>> [5.711303] []

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

-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On Tue, 2016-09-20 at 11:43 -0400, Alan Stern wrote: > > Yes. But that'd also mean to write that value upon every suspend/resume > cycle > > because the rtsx usb driver still declares support for autosuspend. > > Should that be dropped ? > > No,

Re: [PATCHv2] usb: musb: Fix unbalanced platform_disable

Hi Tony, On Monday 19 Sep 2016 15:41:50 Tony Lindgren wrote: > * Laurent Pinchart [160919 13:35]: > > On Sunday 18 Sep 2016 08:19:02 Tony Lindgren wrote: > >>> [5.711303] [] (_raw_spin_unlock_irqrestore) from > >>> [] > >>> (musb_gadget_queue+0x128/0x4ac) >

Re: [PATCH] usb: misc: legousbtower: Fix NULL pointer deference

On Tue, 2016-09-20 at 08:22 +0200, Greg KH wrote: > When you sent this the last time (back on Sept 2), I sent you a > "simpler" patch, the same one below, and asked you if that worked > instead. I never heard back, could you test this patch and see if it > resolves the issue for you? As far as I

[PATCH net 5/5] r8152: disable ALDPS and EEE before setting PHY

Disable ALDPS and EEE to avoid the possible failure when setting the PHY. Signed-off-by: Hayes Wang --- drivers/net/usb/r8152.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index

Re: USB hot-plug not working (ASUS TP301UA-C4028T)

Hi Oliver! Here you are. dmesg signals when plugging AC in: http://paste.opensuse.org/57485b34 dmesg signals when unplugging AC: http://paste.opensuse.org/5a8e9910 And just in case, dmesg signals when plugging a USB device when AC is plugged in: http://paste.opensuse.org/45faee84 Hope this

Re: [PATCH] usb: misc: legousbtower: Fix NULL pointer deference

Hi greg, Many appologies, I did not see your patch attached/forgot it existed when I fixed this yesterday. I created this patch based on your feedback from my original email on September 2nd. I have tested this patch with an emulated device and it fixes the vulnerability. Again, applogies

Re: [RFC PATCH] xhci: do not halt the secondary HCD

On 19.09.2016 11:23, Joel Stanley wrote: Hi Mathias, On Mon, Sep 19, 2016 at 4:33 PM, Greg KH wrote: On Mon, Sep 19, 2016 at 04:05:45PM +0930, Joel Stanley wrote: We can't halt the secondary HCD, because it's also the primary HCD, which will cause problems if we

Re: USB hot-plug not working (ASUS TP301UA-C4028T)

On Tue, 2016-09-20 at 11:11 +1200, Pierre de Villemereuil wrote: > Dear Oliver, > > Sorry about my last message, the bug is actually still going on. I found > something interesting though: when AC is plugged, USB is working totally OK, > but when on battery, then here comes trouble... Could

Re: [PATCH] usb: misc: legousbtower: Fix NULL pointer deference

On Mon, Sep 19, 2016 at 07:09:51PM +0100, James wrote: > This patch fixes a NULL pointer dereference caused by a race codition in the > probe function of the legousbtower driver. It re-structures the probe > function to only register the interface after successfully reading the > board's firmware

[PATCH net 2/5] r8152: move enabling PHY

Move enabling PHY to init(), otherwise some other settings may fail. Signed-off-by: Hayes Wang --- drivers/net/usb/r8152.c | 43 +-- 1 file changed, 25 insertions(+), 18 deletions(-) diff --git a/drivers/net/usb/r8152.c

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

Hi, John Youn writes: > There's also a TCM python tool somewhere which helps with this. I > haven't used f_tcm in a long while, but Sebastian and I used it long > back to test streams with dwc3. Have you tried it recently? Or do you know of anyone

Re: [PATCH] usb: misc: legousbtower: Fix NULL pointer deference

On Tue, Sep 20, 2016 at 10:18:21AM +0200, Oliver Neukum wrote: > On Tue, 2016-09-20 at 08:22 +0200, Greg KH wrote: > > When you sent this the last time (back on Sept 2), I sent you a > > "simpler" patch, the same one below, and asked you if that worked > > instead. I never heard back, could you

Re: cdc_acm bug? read buffer bytes shifted

On Fri, 2016-09-16 at 18:24 +0200, Julio Guerra wrote: > > 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

[PATCH net 0/5] r8152: correct the flow of PHY

First, to enable the PHY as early as possible. Some settings may fail if the PHY is power down. Move the other PHY settings to hw_phy_cfg() to make sure the order is correct. Finally, disable ALDPS and EEE before updating the PHY for RTL8153. Hayes Wang (5): r8152: move some functions

[PATCH net 4/5] r8152: remove r8153_enable_eee

Remove r8153_enable_eee(). Signed-off-by: Hayes Wang --- drivers/net/usb/r8152.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index 9ce5bd5..e7a05dd 100644 --- a/drivers/net/usb/r8152.c

[PATCH net 3/5] r8152: move PHY settings to hw_phy_cfg

Move the PHY relative settings together to hw_phy_cfg(). Signed-off-by: Hayes Wang --- drivers/net/usb/r8152.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index dbf11ba..9ce5bd5

[PATCH net 1/5] r8152: move some functions

Move the following functions forward. r8152_mmd_indirect() r8152_mmd_read() r8152_mmd_write() r8152_eee_en() r8152b_enable_eee() r8153_eee_en() r8153_enable_eee() r8152b_enable_fc() r8153_aldps_en() Signed-off-by: Hayes Wang

Re: USB hot-plug not working (ASUS TP301UA-C4028T)

On Tue, 2016-09-20 at 20:58 +1200, Pierre de Villemereuil wrote: > Hi Oliver! > > Here you are. > > dmesg signals when plugging AC in: > http://paste.opensuse.org/57485b34 > > dmesg signals when unplugging AC: > http://paste.opensuse.org/5a8e9910 > > And just in case, dmesg signals when

Re: [PATCH] mmc: rtsx_usb_sdmmc: Handle runtime PM while changing led

On 19 September 2016 at 20:02, Alan Stern wrote: > On Mon, 19 Sep 2016, Ulf Hansson wrote: > >> On 18 September 2016 at 04:30, Alan Stern wrote: >> > On Sat, 17 Sep 2016, Ulf Hansson wrote: >> > >> >> Each access of the parent device (usb

Re: [PATCH] mmc: rtsx_usb_sdmmc: Handle runtime PM while changing led

On Mon, 2016-09-19 at 14:02 -0400, Alan Stern wrote: > > We can for sure enable autosuspend for the sdmmc device, although as > > soon as an SD card will be detected the rtsx driver will increase > the > > runtime PM usage count. The count is decreased when the card is > > removed (or failed to be

Re: [PATCH] usb: misc: legousbtower: Fix NULL pointer deference

On Tue, Sep 20, 2016 at 10:09:56AM +0100, James wrote: > Hi greg, > > Many appologies, I did not see your patch attached/forgot it existed > when I > fixed this yesterday. I created this patch based on your > feedback from my original email on September 2nd. I have tested this patch > with

4.8-rc: runtime PM trying to activate child device host6 but parent (2-1.2:1.0) is not active

4.8-rc5 was fine, but 4.8-rc6 and 4.8-rc7 on a Lenovo Thinkpad x220 report this warning or error (cannot tell) anytime a USB mass storage device is inserted into a usb port (2.0 or 3.0): runtime PM trying to activate child device host6 but parent (2-1.2:1.0) is not active The device appears to

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

-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On Mon, 2016-09-19 at 13:48 -0400, Alan Stern wrote: > > This ought to help.  Ritesh, please apply this patch on top of the  > two earlier ones and let's see what happens. > > Alan Stern > > Please find the logs at the following links. On this

Re: 4.8-rc: runtime PM trying to activate child device host6 but parent (2-1.2:1.0) is not active

On Tue, 2016-09-20 at 14:11 +0200, Carsten Mattner wrote: > 4.8-rc5 was fine, but 4.8-rc6 and 4.8-rc7 on a Lenovo Thinkpad x220 report > this > warning or error (cannot tell) anytime a USB mass storage device is inserted > into a usb port (2.0 or 3.0): > > runtime PM trying to activate child

Re: 4.8-rc: runtime PM trying to activate child device host6 but parent (2-1.2:1.0) is not active

On Tue, Sep 20, 2016 at 02:15:02PM +0200, Carsten Mattner wrote: > On Tue, Sep 20, 2016 at 2:11 PM, Carsten Mattner > wrote: > > 4.8-rc5 was fine, but 4.8-rc6 and 4.8-rc7 on a Lenovo Thinkpad x220 report > > this > > In hindsight, I'm not entirely certain if rc5 was

Re: 4.8-rc: runtime PM trying to activate child device host6 but parent (2-1.2:1.0) is not active

On Tue, Sep 20, 2016 at 2:11 PM, Carsten Mattner wrote: > 4.8-rc5 was fine, but 4.8-rc6 and 4.8-rc7 on a Lenovo Thinkpad x220 report > this In hindsight, I'm not entirely certain if rc5 was fine, so please don't take that as a fact. > warning or error (cannot tell)

Re: Memory barrier needed with wake_up_process()?

Hi, Felipe Balbi writes: > [ Unknown signature status ] > > Hi, > > Alan Stern writes: > > [...] > >>> $ grep -RnH "raise_exception\|fsg_main_thread" /tmp/trace.txt >>> /tmp/trace.txt:111743: irq/17-dwc3-3527 [003] d..1

Re: 4.8-rc: runtime PM trying to activate child device host6 but parent (2-1.2:1.0) is not active

On Tue, Sep 20, 2016 at 5:41 PM, Alan Stern wrote: > I suspect the problem has been there all along, but it simply wasn't > reported until commit 71723f95463d ("PM / runtime: print error when > activating a child to unactive parent") was merged in 4.8-rc1. Is this

Re: [PATCHv2] usb: musb: Fix unbalanced platform_disable

* Tony Lindgren [160920 10:11]: > * Laurent Pinchart [160919 23:36]: > > No, I perform the following steps: > > > > - Connect the panda board to the USB through USB (which powers the board on) > > - Let the board boot over NFS > > - Log in as

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

On 9/20/2016 1:29 AM, Felipe Balbi wrote: > > Hi, > > John Youn writes: >> There's also a TCM python tool somewhere which helps with this. I >> haven't used f_tcm in a long while, but Sebastian and I used it long >> back to test streams with dwc3. > >

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

-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hello Alan, On Tue, 2016-09-20 at 10:16 -0400, Alan Stern wrote: > This is a lot better.  No more I/O errors. > > We still have irregular suspends and resumes, but that's to be  > expected.  More worrying are the spontaneous disconnects.  They

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

On Tue, 20 Sep 2016, Ritesh Raj Sarraf wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA512 > > Hello Alan, > > On Tue, 2016-09-20 at 10:16 -0400, Alan Stern wrote: > > This is a lot better.  No more I/O errors. > > > > We still have irregular suspends and resumes, but that's to be  > >

Re: PROBLEM: DWC3 USB 3.0 not working on Odroid-XU4 with Exynos 5422

Hi guys, On Di, 2016-08-30 at 10:32 +0530, Anand Moon wrote: > Hi All > > Adding Vivek Gautam. > > On 29 August 2016 at 16:35, Michael Niewöhner > wrote: > > > > Hi Mathias, > > On Mo, 2016-08-29 at 13:59 +0300, Mathias Nyman wrote: > > > > > > On 29.08.2016 10:28,

Re: 转发: [PATCH] usb: serial: update CH34x driver in drivers/usb/serial

On 2016-09-15 01:03, Aidan Thornton wrote: thanks, greg k-h It looks like someone by the name of Grigori Goronzy (CCed) had a patch series or four attempting to do this that just never went anywhere like all the other attempts. Might be worth someone talking to him or looking at his patches.

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

On 9/20/2016 11:35 AM, John Youn wrote: > On 9/20/2016 1:29 AM, Felipe Balbi wrote: >> >> Hi, >> >> John Youn writes: >>> There's also a TCM python tool somewhere which helps with this. I >>> haven't used f_tcm in a long while, but Sebastian and I used it long

Re: 4.8-rc: runtime PM trying to activate child device host6 but parent (2-1.2:1.0) is not active

On Tue, 20 Sep 2016, Carsten Mattner wrote: > On Tue, Sep 20, 2016 at 5:41 PM, Alan Stern wrote: > > > I suspect the problem has been there all along, but it simply wasn't > > reported until commit 71723f95463d ("PM / runtime: print error when > > activating a child

Re: 4.8-rc: runtime PM trying to activate child device host6 but parent (2-1.2:1.0) is not active

On Tue, Sep 20, 2016 at 9:49 PM, Alan Stern wrote: > On Tue, 20 Sep 2016, Carsten Mattner wrote: > > > On Tue, Sep 20, 2016 at 5:41 PM, Alan Stern > > wrote: > > > > > I suspect the problem has been there all along, but it simply wasn't > >

Re: [PATCH net 0/5] r8152: correct the flow of PHY

From: Hayes Wang Date: Tue, 20 Sep 2016 16:22:04 +0800 > First, to enable the PHY as early as possible. Some settings may fail if the > PHY is power down. > > Move the other PHY settings to hw_phy_cfg() to make sure the order is correct. > > Finally, disable ALDPS and

Re: 4.8-rc: runtime PM trying to activate child device host6 but parent (2-1.2:1.0) is not active

On Tue, Sep 20, 2016 at 2:36 PM, Oliver Neukum wrote: > On Tue, 2016-09-20 at 14:11 +0200, Carsten Mattner wrote: >> 4.8-rc5 was fine, but 4.8-rc6 and 4.8-rc7 on a Lenovo Thinkpad x220 report >> this >> warning or error (cannot tell) anytime a USB mass storage device is

Re: crash by cdc_acm driver in kernels 4.8-rc1/5

On Mon, 2016-09-12 at 04:43 +0200, Wim Osterholt wrote: > > A laptop, more broken than the rest, does not output anything after > inserting. Later on it crashes. No system.map file in /boot. > After booting with the dongle inserted it spits out: I cannot replicate it. Could you please provide

[PATCH] cdc-acm: hardening against malicious devices

This should fix the last holes against malicious devices still open in cdc-acm. It cannot go into stable due to the introduction of the common parser. The fix for stable already merged also covers the problems this patch fixes. Signed-off-by: Oliver Neukum ---

Re: [RESEND PATCH 1/4] usb: dbc: early driver for xhci debug capability

On 29.08.2016 08:26, Lu Baolu wrote: xHCI debug capability (DbC) is an optional but standalone functionality provided by an xHCI host controller. Software learns this capability by walking through the extended capability list of the host. xHCI specification describes DbC in section 7.6. This

Re: Memory barrier needed with wake_up_process()?

On Tue, 20 Sep 2016, Felipe Balbi wrote: > And here's trace output (complete, scroll to bottom). It seems to me > like the thread didn't wake up at all. It didn't even try to > execute. I'll add some more traces and try to get better information > about what's going on. > > > > # tracer: nop >

Re: 4.8-rc: runtime PM trying to activate child device host6 but parent (2-1.2:1.0) is not active

On Tue, Sep 20, 2016 at 3:45 PM, Carsten Mattner wrote: > On Tue, Sep 20, 2016 at 2:36 PM, Oliver Neukum wrote: >> On Tue, 2016-09-20 at 14:11 +0200, Carsten Mattner wrote: >>> 4.8-rc5 was fine, but 4.8-rc6 and 4.8-rc7 on a Lenovo Thinkpad x220 report

[PATCH] extcon: usb-gpio: Add VBUS detection support

Driver can now work with both ID and VBUS pins or either one of them. There can be the following 3 cases 1) Both ID and VBUS GPIOs are available: ID = LOW -> USB_HOST active, USB inactive ID = HIGH -> USB_HOST inactive, USB state is same as VBUS. 2) Only ID GPIO is available: ID = LOW ->

Re: [RESEND PATCH 0/4] usb: early: add support for early printk through USB3 debug port

On 29.08.2016 08:26, Lu Baolu wrote: xHCI debug capability (DbC) is an optional but standalone functionality provided by an xHCI host controller. With DbC hardware initialized, the system will present a debug device through the USB3 debug port (normally the first USB3 port). The debug device is

Re: [PATCH] mmc: rtsx_usb_sdmmc: Handle runtime PM while changing led

On Tue, 20 Sep 2016, Ulf Hansson wrote: > >> I am wondering what you think would be a good autosuspend timeout in > >> this case? It seems to me that the only thing the rtsx driver really > >> care about is to tell the parent device that it needs to be runtime > >> resumed during a certain

Re: [PATCH v2 1/6] usb: dwc2: add support for Meson8b and GXBB SoCs

On Sun, Sep 11, 2016 at 03:41:06PM +0200, Martin Blumenstingl wrote: > From: Jerome Brunet > > Add compatible strings for amlogic Meson8b and GXBB SoCs with the > corresponding configuration parameters. > > Signed-off-by: Jerome Brunet >

Re: [PATCH] mmc: rtsx_usb_sdmmc: Handle runtime PM while changing led

On Tue, 20 Sep 2016, Oliver Neukum wrote: > On Mon, 2016-09-19 at 14:02 -0400, Alan Stern wrote: > > > We can for sure enable autosuspend for the sdmmc device, although as > > > soon as an SD card will be detected the rtsx driver will increase > > the > > > runtime PM usage count. The count is

Re: [PATCH v2 2/6] Documentation: dt-bindings: Add documentation for the Meson USB2 PHYs

On Sun, Sep 11, 2016 at 03:41:07PM +0200, Martin Blumenstingl wrote: > Add the documentation for the bindings for the Meson8b and GXBB USB2 > PHYs. > > Signed-off-by: Martin Blumenstingl > --- > .../devicetree/bindings/phy/meson-usb2-phy.txt | 27 >

Re: [PATCHv2] usb: musb: Fix unbalanced platform_disable

* Andreas Kemnade [160919 22:05]: > On Mon, 19 Sep 2016 09:02:50 -0700 > Tony Lindgren wrote: > > > * Andreas Kemnade [160918 23:00]: > > > On Sun, 18 Sep 2016 08:19:02 -0700 > > > Tony Lindgren wrote: > > > > >

[RESEND PATCH v6, 3/5] usb: xhci-mtk: make IPPC register optional

Make IPPC register optional to support host side of dual-role mode, due to it is moved into common glue layer for simplification. Signed-off-by: Chunfeng Yun --- drivers/usb/host/xhci-mtk.c | 36 +--- 1 file changed, 29 insertions(+),

[RESEND PATCH v6, 5/5] arm64: dts: mediatek: add USB3 DRD driver

USB3 DRD driver is added for MT8173-EVB, and xHCI driver becomes its subnode Signed-off-by: Chunfeng Yun --- arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 46 +-- arch/arm64/boot/dts/mediatek/mt8173.dtsi| 29 + 2 files

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

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) modes. Signed-off-by: Chunfeng Yun --- drivers/usb/Kconfig|2 +

[RESEND PATCH v6, 0/5] Add MediaTek USB3 DRD Driver

>From 99e428a1808c8ca91ff473d487b52ca5d355d875 Mon Sep 17 00:00:00 2001 From: Chunfeng Yun Date: Mon, 5 Sep 2016 10:27:07 +0800 Subject: [PATCH v6, 0/5] Add MediaTek USB3 DRD Driver These patches introduce the MediaTek USB3 dual-role controller driver. The driver can

[RESEND PATCH v6, 2/5] dt-bindings: mt8173-mtu3: add devicetree bindings

add a DT binding doc for MediaTek USB3 DRD driver Signed-off-by: Chunfeng Yun Acked-by: Rob Herring --- .../devicetree/bindings/usb/mt8173-mtu3.txt| 87 1 file changed, 87 insertions(+) create mode 100644

[RESEND PATCH v6, 1/5] dt-bindings: mt8173-xhci: support host side of dual-role mode

Some resources, such as IPPC register etc, shared with device driver are moved into common glue layer when xHCI driver is the host side of dual-role mode and they should be changed as optional properties if they are required ones before. For clarity, add a new part of binding to support host side