Re: [PATCH v7 usb-next 3/4] usb: core: hcd: integrate the PHY roothub wrapper

2017-10-25 Thread Manu Gautam
Hi, On 10/24/2017 3:27 AM, Martin Blumenstingl wrote: > This integrates the PHY roothub wrapper into the core hcd > infrastructure. Multiple PHYs which are part of the roothub devicetree > node (which is a sub-node of the sysdev's node) are now managed > (= powered on/off when needed), by the

Re: [PATCH 2/3] usb: phy: remove phy-msm-usb.c

2017-10-25 Thread Manu Gautam
Hi, On 10/26/2017 3:31 AM, Alex Elder wrote: > No Qualcomm SoC requires the "phy-msm-usb.c" USB phy driver support > any more, so remove the code. > > Suggested-by: Stephen Boyd > Signed-off-by: Alex Elder > Acked-by: Bjorn Andersson

Re: [PATCH 1/3] usb: host: remove ehci-msm.c

2017-10-25 Thread Manu Gautam
Hi, On 10/26/2017 3:31 AM, Alex Elder wrote: > No Qualcomm SoC requires the "ehci-msm.c" code any more. So remove it. > > Suggested-by: Stephen Boyd > Signed-off-by: Alex Elder > Acked-by: Bjorn Andersson > Acked-by: Andy

[PATCH v4] arm64: dts: ls1088a: Add USB support

2017-10-25 Thread yinbo.zhu
From: "yinbo.zhu" Add USB support on ls1088ardb Signed-off-by: yinbo.zhu Signed-off-by: Ran Wang --- Change in v4: - Change status from "enable" to "disabled" then enable it on dts file. - Add the

[PATCH] usb: wusbcore: wa-xfer: mark expected switch fall-through

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/wusbcore/wa-xfer.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/wusbcore/wa-xfer.c

Re: [PATCH] usb: dwc2: host: Don't retry NAKed transactions right away

2017-10-25 Thread Doug Anderson
Hi, On Wed, Oct 25, 2017 at 2:08 PM, Douglas Anderson wrote: > On rk3288-veyron devices on Chrome OS it was found that plugging in an > Arduino-based USB device could cause the system to lockup, especially > if the CPU Frequency was at one of the slower operating points

[PATCH 1/3] usb: host: remove ehci-msm.c

2017-10-25 Thread Alex Elder
No Qualcomm SoC requires the "ehci-msm.c" code any more. So remove it. Suggested-by: Stephen Boyd Signed-off-by: Alex Elder Acked-by: Bjorn Andersson Acked-by: Andy Gross --- drivers/usb/host/Makefile

[PATCH 0/3] usb: remove some unused code

2017-10-25 Thread Alex Elder
This series deletes some Qualcomm USB code is no longer needed by any Qualcomm hardware. -Alex Alex Elder (3): usb: host: remove ehci-msm.c usb: phy: remove phy-msm-usb.c usb: phy: remove phy-qcom-8x16-usb.c drivers/usb/host/Makefile |

[PATCH 3/3] usb: phy: remove phy-qcom-8x16-usb.c

2017-10-25 Thread Alex Elder
No Qualcomm SoC requires the "phy-qcom-8x16-usb.c" USB phy driver support any more, so remove the code. Suggested-by: Stephen Boyd Signed-off-by: Alex Elder Acked-by: Bjorn Andersson Acked-by: Andy Gross

[PATCH 2/3] usb: phy: remove phy-msm-usb.c

2017-10-25 Thread Alex Elder
No Qualcomm SoC requires the "phy-msm-usb.c" USB phy driver support any more, so remove the code. Suggested-by: Stephen Boyd Signed-off-by: Alex Elder Acked-by: Bjorn Andersson Acked-by: Andy Gross ---

Re: usb: dwc2: NMI watchdog: BUG: soft lockup - CPU#0 stuck for 146s

2017-10-25 Thread Doug Anderson
Hi, On Mon, Oct 16, 2017 at 1:49 PM, Julius Werner wrote: >> d9a14b00 339317035 C Ii:1:004:1 -32:1 0 >> d9a14b00 339317049 S Ii:1:004:1 -115:1 10 < >> d9a14b00 339318040 C Ii:1:004:1 -32:1 0 >> d9a14b00 339318057 S Ii:1:004:1 -115:1 10 < >> d9a14b00 339319042 C Ii:1:004:1

[PATCH] usb: dwc2: host: Don't retry NAKed transactions right away

2017-10-25 Thread Douglas Anderson
On rk3288-veyron devices on Chrome OS it was found that plugging in an Arduino-based USB device could cause the system to lockup, especially if the CPU Frequency was at one of the slower operating points (like 100 MHz / 200 MHz). Upon tracing, I found that the following was happening: * The USB

Re: [PATCH] rndis_host: support Novatel Verizon USB730L — Linux USB

2017-10-25 Thread Bjørn Mork
Gal Shalif writes: > Bjoren> You can always unbind the driver, right? And usb_modeswitch will > even do it for you. > I'm not a USB mode switch expert, so please explain what is the mode switch > file that you suggest. > > As I already said before - my kernel configuration will

[PATCH 5/9] usb: host: ehci-hcd: mark expected switch fall-through

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/host/ehci-hcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/9] usb: host: fotg210-hcd: mark expected switch fall-through

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/host/fotg210-hcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 0/9] mark expected switch fall-throughs

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, this patchset aims to mark switch cases where we are expecting to fall through. In Kees Cook words: "This is an unfortunate omission in the C language, and thankfully both gcc and clang have stepped up to solve this the same way static analyzers

[PATCH 4/9] usb: host: ohci-hcd: mark expected switch fall-through

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/host/ohci-hcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 2/9] usb: host: xhci: mark expected switch fall-through

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/host/xhci.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/host/xhci.c

Re: [PATCH 6/9] usb: host: isp1362-hcd: mark expected switch fall-through

2017-10-25 Thread Gustavo A. R. Silva
Greg, Quoting "Gustavo A. R. Silva" : In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/host/isp1362-hcd.c | 1 + 1 file changed, 1

[PATCH 8/9] usb: host: xhci-hub: mark expected switch fall-through

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/host/xhci-hub.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/host/xhci-hub.c

[PATCH 9/9] usb: host: pci-quirks: mark expected switch fall-through

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/host/pci-quirks.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 6/9] usb: host: isp1362-hcd: mark expected switch fall-through

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/host/isp1362-hcd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/host/isp1362-hcd.c

[PATCH 7/9] usb: host: oxu210hp-hcd: mark expected switch fall-through

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/host/oxu210hp-hcd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 3/9] usb: host: xhci-mem: mark expected switch fall-through

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/host/xhci-mem.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/host/xhci-mem.c

[PATCH 3/4] usb: xhci: Fix build warning

2017-10-25 Thread Corentin Labbe
This patch fix the following build warning: drivers/usb/host/xhci-ring.c:1895:19: warning: variable 'urb_priv' set but not used [-Wunused-but-set-variable] Signed-off-by: Corentin Labbe --- drivers/usb/host/xhci-ring.c | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH 2/4] usb: xhci: Fix build warning

2017-10-25 Thread Corentin Labbe
This patch fix the following build warning: drivers/usb/host/xhci.c:2853:23: warning: variable 'ep' set but not used [-Wunused-but-set-variable] Signed-off-by: Corentin Labbe --- drivers/usb/host/xhci.c | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH 1/4] usb: xhci: Fix build warning

2017-10-25 Thread Corentin Labbe
This patch fix the following build warnings: drivers/usb/host/xhci.c:3378:6: warning: variable 'last_freed_endpoint' set but not used [-Wunused-but-set-variable] Signed-off-by: Corentin Labbe --- drivers/usb/host/xhci.c | 3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH 4/4] usb: xhci: Fix build warning

2017-10-25 Thread Corentin Labbe
This patch fix the following build warnings: drivers/usb/host/xhci-ring.c:2011:20: warning: variable 'ep_ring' set but not used [-Wunused-but-set-variable] Signed-off-by: Corentin Labbe --- drivers/usb/host/xhci-ring.c | 2 -- 1 file changed, 2 deletions(-) diff --git

Re: [PATCH] rndis_host: support Novatel Verizon USB730L — Linux USB

2017-10-25 Thread Bjørn Mork
Oliver Neukum writes: > Am Dienstag, den 24.10.2017, 23:17 +0300 schrieb Gal Shalif: >> The option driver is compiled as a module. >> The option driver patch, and other related patches are listed within >> my previous email: > > Hi, > > but you are adding the device

[PATCH] usb: gadget: udc: dummy_hcd: mark expected switch fall-throughs

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/gadget/udc/dummy_hcd.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

[PATCH] usb: gadget: composite: mark expected switch fall-throughs

2017-10-25 Thread Gustavo A. R. Silva
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Gustavo A. R. Silva --- drivers/usb/gadget/composite.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

Re: [PATCH] drivers/hid: Convert timers to use timer_setup()

2017-10-25 Thread kbuild test robot
-timer_setup/20171025-230118 config: i386-randconfig-x008-201743 (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 reproduce: # save the attached .config to linux build tree make ARCH=i386 All error/warnings (new ones prefixed by >>): In file included from i

Re: [PATCH] drivers/hid: Convert timers to use timer_setup()

2017-10-25 Thread kbuild test robot
-timer_setup/20171025-230118 config: x86_64-randconfig-x011-201743 (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 reproduce: # save the attached .config to linux build tree make ARCH=x86_64 All error/warnings (new ones prefixed by >>): In file included from i

Re: [RFC/PATCH 0/4] usb: core: Add Support for getting PTM_STATUS

2017-10-25 Thread Alan Stern
On Wed, 25 Oct 2017, Felipe Balbi wrote: > Hi, > > The following series was compile-tested only (so far, at least). I > wanted to get some comments from folks to see what you guys think > about this before running tests. > > I don't have any device available which would support PTM_STATUS so I

[PATCH] usb-storage: make use of srb local variable

2017-10-25 Thread Alan Stern
Commit 8b52291a0743 ("usb-storage: fix deadlock involving host lock and scsi_done") added a local variable to usb_stor_control_thread() in the usb-storage driver. This local variable holds the value of us->srb, for use after the host lock has been released. But as long as we have the value in a

Re: [PATCH] USB: EHCI: don't reread PORTSC after disabling port power

2017-10-25 Thread Alan Stern
On Tue, 24 Oct 2017, Uwe [iso-8859-1] Kleine-K�nig wrote: > Now after removing my instrumentation the situation isn't what I > described before. Instead of being spammed by messages about > over-current I get only a single message (good) and power stays on. I > wonder if that is acceptable. > >

Re: [PATCH] usb: usbatm: Convert timers to use timer_setup()

2017-10-25 Thread Kees Cook
This also uses timer_setup_on_stack() (only in -next). If it's okay, I'll carry it in the timers tree. Thanks! -Kees On Tue, Oct 24, 2017 at 12:08 PM, Kees Cook wrote: > In preparation for unconditionally passing the struct timer_list pointer to > all timer callbacks,

Re: [PATCH] usb: usbtest: Convert timers to use timer_setup()

2017-10-25 Thread Kees Cook
Sorry, I sent this one but forgot that timer_setup_on_stack() is in -next only. If it's okay, I can carry it in the timers tree. Thanks! -Kees On Tue, Oct 24, 2017 at 12:27 PM, Felipe Balbi wrote: > Kees Cook writes: > >> In preparation for

Re: [PATCHv4 2/2] phy: exynos5-usbdrd: Calibrate LOS levels for exynos5420/5800

2017-10-25 Thread Felipe Balbi
Hi, Kishon Vijay Abraham I writes: > Felipe, > > On Monday 09 October 2017 05:30 PM, Andrzej Pietrasiewicz wrote: >> From: Vivek Gautam >> >> Adding phy calibration sequence for USB 3.0 DRD PHY present on >> Exynos5420/5800 systems. >> This calibration

Re: [PATCHv4 2/2] phy: exynos5-usbdrd: Calibrate LOS levels for exynos5420/5800

2017-10-25 Thread Kishon Vijay Abraham I
Felipe, On Monday 09 October 2017 05:30 PM, Andrzej Pietrasiewicz wrote: > From: Vivek Gautam > > Adding phy calibration sequence for USB 3.0 DRD PHY present on > Exynos5420/5800 systems. > This calibration facilitates setting certain PHY parameters viz. > the

[RFC/PATCH 1/4] usb: core: add Status Type definitions

2017-10-25 Thread Felipe Balbi
USB 3.1 added a PTM_STATUS type. Let's add a define for it and following patches will let usb_get_status() accept the new argument. Signed-off-by: Felipe Balbi --- drivers/usb/core/message.c | 4 ++-- include/uapi/linux/usb/ch9.h | 4 2 files changed, 6

[RFC/PATCH 0/4] usb: core: Add Support for getting PTM_STATUS

2017-10-25 Thread Felipe Balbi
Hi, The following series was compile-tested only (so far, at least). I wanted to get some comments from folks to see what you guys think about this before running tests. I don't have any device available which would support PTM_STATUS so I guess I'd have to implement it on g_zero, if at all

[RFC/PATCH 3/4] usb: core: add a 'type' parameter to usb_get_status()

2017-10-25 Thread Felipe Balbi
This new 'type' parameter will allows interested drivers to request for PTM status or Standard status. Signed-off-by: Felipe Balbi --- drivers/staging/wlan-ng/hfa384x_usb.c | 8 +++--- drivers/usb/core/driver.c | 3 ++- drivers/usb/core/hub.c

[RFC/PATCH 2/4] usb: core: rename usb_get_status() 'type' argument to 'recip'

2017-10-25 Thread Felipe Balbi
This makes it a lot clearer that we're expecting a recipient as the argument. A follow-up patch will use the argument 'type' as the status type selector (standard or ptm). Signed-off-by: Felipe Balbi --- drivers/usb/core/message.c | 6 +++--- include/linux/usb.h

[RFC/PATCH 4/4] usb: core: add two usb get status helpers

2017-10-25 Thread Felipe Balbi
Now that we have more than one USB Status Type, we can provide helpers to the users so that we reduce the amount of arguments that need to be passed. This will also reduce the amount of mistakes when using this API. Signed-off-by: Felipe Balbi ---

Re: [PATCH] usb: gadget: f_phonet: mark expected switch fall-throughs

2017-10-25 Thread Greg Kroah-Hartman
On Wed, Oct 25, 2017 at 12:25:36PM +0300, Felipe Balbi wrote: > > Hi, > > "Gustavo A. R. Silva" writes: > > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > > where we are expecting to fall through. > > > > Addresses-Coverity-ID: 115004 > >

Re: [PATCH] usb: gadget: f_phonet: mark expected switch fall-throughs

2017-10-25 Thread Felipe Balbi
Hi, "Gustavo A. R. Silva" writes: > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > where we are expecting to fall through. > > Addresses-Coverity-ID: 115004 > Addresses-Coverity-ID: 115005 > Signed-off-by: Gustavo A. R. Silva

Donation!

2017-10-25 Thread MH
I have a donation gift for you. Reply asap -- 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: [PATCH] rndis_host: support Novatel Verizon USB730L — Linux USB

2017-10-25 Thread Oliver Neukum
Am Dienstag, den 24.10.2017, 23:17 +0300 schrieb Gal Shalif: > The option driver is compiled as a module. > The option driver patch, and other related patches are listed within > my previous email: Hi, but you are adding the device unconditionally to the HID blacklist. Thus the HID part becomes

Re: [PATCH] drivers/hid: Convert timers to use timer_setup()

2017-10-25 Thread Benjamin Tissoires
On Oct 24 2017 or thereabouts, Allen Pais wrote: > Switch to using the new timer_setup() and from_timer() > for drivers/hid/* > > Cc: Kees Cook > Signed-off-by: Allen Pais > --- > > Note:This patch is only compile tested. Looks good enough for me: