Re: [PATCH] Revert "xhci: Limit USB2 port wake support for AMD Promontory hosts"

2017-09-15 Thread Kai-Heng Feng
On Mon, Aug 28, 2017 at 9:56 PM, Kai-Heng Feng wrote: > On Mon, Aug 28, 2017 at 6:14 PM, Mathias Nyman > wrote: >> On 28.08.2017 12:29, Greg KH wrote: >> >> Adding more people who were involved in the original patch. >> >> Users are now

Re: [PATCH] usb: dwc3: Fix the USB 3.0 hub detection bug after warm boot

2017-09-15 Thread gustavo panizzo
Hello On Fri, Sep 15, 2017 at 09:01:13AM +, gustavo panizzo wrote: Hello On Thu, Sep 07, 2017 at 01:51:31PM +0300, Felipe Balbi wrote: Hi, gustavo panizzo writes: --- drivers/usb/dwc3/core.c | 33 + 1 file changed, 33 insertions(+)

Re: [PATCH 5/5] usb: usb251xb: Add copyrights

2017-09-15 Thread Serge Semin
On Fri, Sep 15, 2017 at 04:53:13PM -0700, Greg KH wrote: > On Sat, Sep 16, 2017 at 02:45:17AM +0300, Serge Semin wrote: > > On Fri, Sep 15, 2017 at 04:40:28PM -0700, Greg KH > > wrote: > > > On Sat, Sep 16, 2017 at 02:31:13AM +0300, Serge

Re: [PATCH 1/5] usb: usb251xb: Add USB2517/i hub support

2017-09-15 Thread Serge Semin
On Fri, Sep 15, 2017 at 04:45:14PM -0700, Greg KH wrote: > On Sat, Sep 16, 2017 at 02:31:09AM +0300, Serge Semin wrote: > > USB2517i hubs are very like USB251xb devices series. They have almost > > the same configuration registers space except number of ports, led > >

Re: [PATCH 5/5] usb: usb251xb: Add copyrights

2017-09-15 Thread Greg KH
On Sat, Sep 16, 2017 at 02:45:17AM +0300, Serge Semin wrote: > On Fri, Sep 15, 2017 at 04:40:28PM -0700, Greg KH > wrote: > > On Sat, Sep 16, 2017 at 02:31:13AM +0300, Serge Semin wrote: > > > Signed-off-by: Serge Semin > > > --- > > >

Re: [PATCH 5/5] usb: usb251xb: Add copyrights

2017-09-15 Thread Serge Semin
On Fri, Sep 15, 2017 at 04:40:28PM -0700, Greg KH wrote: > On Sat, Sep 16, 2017 at 02:31:13AM +0300, Serge Semin wrote: > > Signed-off-by: Serge Semin > > --- > > drivers/usb/misc/usb251xb.c | 1 + > > 1 file changed, 1 insertions(+), 0

Re: [PATCH 1/5] usb: usb251xb: Add USB2517/i hub support

2017-09-15 Thread Greg KH
On Sat, Sep 16, 2017 at 02:31:09AM +0300, Serge Semin wrote: > USB2517i hubs are very like USB251xb devices series. They have almost > the same configuration registers space except number of ports, led > configurations and lack of battery settings. All these peculiarities > are reflected in this

Re: [PATCH 5/5] usb: usb251xb: Add copyrights

2017-09-15 Thread Greg KH
On Fri, Sep 15, 2017 at 04:40:28PM -0700, Greg KH wrote: > On Sat, Sep 16, 2017 at 02:31:13AM +0300, Serge Semin wrote: > > Signed-off-by: Serge Semin > > --- > > drivers/usb/misc/usb251xb.c | 1 + > > 1 file changed, 1 insertions(+), 0 deletion(-) > > I can't take

Re: [PATCH 5/5] usb: usb251xb: Add copyrights

2017-09-15 Thread Greg KH
On Sat, Sep 16, 2017 at 02:31:13AM +0300, Serge Semin wrote: > Signed-off-by: Serge Semin > --- > drivers/usb/misc/usb251xb.c | 1 + > 1 file changed, 1 insertions(+), 0 deletion(-) I can't take patches without any changelog text, sorry. > > diff --git

[PATCH 0/5] usb: usb251xb: Add USB2517i hub support and fix some bugs

2017-09-15 Thread Serge Semin
Primarily it was intended to just add USB2517 hub support to the driver. But after tests a bug and inconistency were discovered. So it was decided to perform following changes: Changelog v1: - Add USB2517/i hub specifics support to the driver - Fix property_u32 NULL-pointer dereference - Add new

[PATCH 1/5] usb: usb251xb: Add USB2517/i hub support

2017-09-15 Thread Serge Semin
USB2517i hubs are very like USB251xb devices series. They have almost the same configuration registers space except number of ports, led configurations and lack of battery settings. All these peculiarities are reflected in this patch. Signed-off-by: Serge Semin ---

[PATCH 4/5] usb: usb251xb: Use GPIO descriptor consumer interface

2017-09-15 Thread Serge Semin
The driver used to be developed with legacy GPIO API support. It's better to use descriptor-based interface for several reasons. First of all the legacy API doesn't support the ACTIVE_LOW/HIGH flag of dts nodes, which is essential since different hardware may have different GPIOs connectivity

[PATCH 5/5] usb: usb251xb: Add copyrights

2017-09-15 Thread Serge Semin
Signed-off-by: Serge Semin --- drivers/usb/misc/usb251xb.c | 1 + 1 file changed, 1 insertions(+), 0 deletion(-) diff --git a/drivers/usb/misc/usb251xb.c b/drivers/usb/misc/usb251xb.c index 1d47acd09..43afa04ad 100644 --- a/drivers/usb/misc/usb251xb.c +++

[PATCH 2/5] usb: usb251xb: Fix property_u32 NULL pointer dereference

2017-09-15 Thread Serge Semin
The methods like of_property_read_u32 utilizing the specified pointer permit only the pointer to a preallocated u32 storage as the third argument. As a result the driver crashes on NULL pointer dereference in case if "oc-delay-us" or "power-on-time-ms" declared in dts file. Signed-off-by: Serge

[PATCH 3/5] usb: usb251xb: Add max power/current dts nodes

2017-09-15 Thread Serge Semin
This parameters may be varied in accordance with hardware specifics. So lets add the corresponding settings to the usb251x driver dts specification. Signed-off-by: Serge Semin --- Documentation/devicetree/bindings/usb/usb251xb.txt | 6 ++

Re: usb/gadget: stalls in dummy_timer

2017-09-15 Thread Alan Stern
On Thu, 14 Sep 2017, Andrey Konovalov wrote: > On Thu, Sep 14, 2017 at 7:49 PM, Alan Stern wrote: > > On Thu, 14 Sep 2017, Andrey Konovalov wrote: > > > >> Looked at this a little more. > >> > >> dummy_timer() stucks in an infinite loop. It calls > >>

RE: xhci_hcd HC died; cleaning up with TUSB7340 and µPD720201

2017-09-15 Thread Chris Welch
> -Original Message- > From: Roger Quadros [mailto:rog...@ti.com] > Sent: Thursday, September 14, 2017 10:14 AM > To: Chris Welch ; linux-usb@vger.kernel.org > Subject: Re: xhci_hcd HC died; cleaning up with TUSB7340 and µPD720201 > > > Texas Instruments

[PATCH 0/2] max3421: add devicetree support

2017-09-15 Thread Jules Maselbas
Hi, This patchset adds devicetree support to the max3421 driver. Thoses modification are based on a previous (unapplied) patch series by Alexander Amelkin [1]. Since I am not a kernel expert I would a like to have your feedbacks on: 1. Under which condition pdata must be freed (in max3421_probe)

[PATCH 2/2] dt-bindings: max3421: Add bindings documentation

2017-09-15 Thread Jules Maselbas
Adds bindings documentation for the max3421 driver. Signed-off-by: Jules Maselbas --- .../devicetree/bindings/usb/maxim,max3421.txt | 24 ++ 1 file changed, 24 insertions(+) create mode 100644

[PATCH 1/2] usb: max3421: Add devicetree support

2017-09-15 Thread Jules Maselbas
Adds support for devicetree to the max3421 driver. Signed-off-by: Jules Maselbas --- drivers/usb/host/max3421-hcd.c | 87 +- 1 file changed, 86 insertions(+), 1 deletion(-) diff --git a/drivers/usb/host/max3421-hcd.c

Re: Interfacing a camera to UVC

2017-09-15 Thread Rail Shafigulin
Thanks for answers, everyone! I think I got what I needed. -- Rail Shafigulin Software Engineer Esencia Technologies -- *ESENCIA TECHNOLOGIES, INC.*3945 Freedom Circle, Suite #360, Santa Clara CA 95054 Phone: +1 408 736 8284 Fax:

Re: USB mouse

2017-09-15 Thread Alan Stern
On Fri, 15 Sep 2017, Bruce Korb wrote: > (Alan -- sorry for duplicate reply. iPad seems to really like HTML, so > it bounced.) > > On Thu, Sep 14, 2017 at 5:30 PM, Alan Stern wrote: > > On Thu, 14 Sep 2017, Bruce Korb wrote: > > The number of tagged commits has

Re: USB mouse

2017-09-15 Thread Bruce Korb
(Alan -- sorry for duplicate reply. iPad seems to really like HTML, so it bounced.) On Thu, Sep 14, 2017 at 5:30 PM, Alan Stern wrote: > On Thu, 14 Sep 2017, Bruce Korb wrote: > The number of tagged commits has nothing to do with it. Bisection > involves testing _all_

[PATCH v2 2/9] clk: at91: pmc: Save SCSR during suspend

2017-09-15 Thread Romain Izard
The contents of the System Clock Status Register (SCSR) needs to be restored into the System Clock Enable Register (SCER). As the bootloader will restore some clocks by itself, the issue can be missed as only the USB controller, the LCD controller, the Image Sensor controller and the programmable

[PATCH v2 1/9] clk: at91: pmc: Wait for clocks when resuming

2017-09-15 Thread Romain Izard
Wait for the syncronization of all clocks when resuming, not only the UPLL clock. Do not use regmap_read_poll_timeout, as it will call BUG() when interrupts are masked, which is the case in here. Signed-off-by: Romain Izard --- drivers/clk/at91/pmc.c | 24

[PATCH v2 5/9] mtd: nand: atmel: Report PMECC failures as errors

2017-09-15 Thread Romain Izard
It is not normal for the PMECC to fail when trying to fix ECC errors. Report these cases as errors. Signed-off-by: Romain Izard --- drivers/mtd/nand/atmel/pmecc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/mtd/nand/atmel/pmecc.c

[PATCH v2 4/9] mtd: nand: atmel: Avoid ECC errors when leaving backup mode

2017-09-15 Thread Romain Izard
During backup mode, the contents of all registers will be cleared as the SoC will be completely powered down. For a product that boots on NAND Flash memory, the bootloader will obviously use the related controller to read the Flash and correct any detected error in the memory, before handling back

[PATCH v2 6/9] ehci-atmel: Power down during suspend is normal

2017-09-15 Thread Romain Izard
When an Atmel SoC is suspended with the backup mode, the USB bus will be powered down. As this is expected, do not return an error to the driver core when ehci_resume detects it. Signed-off-by: Romain Izard --- drivers/usb/host/ehci-atmel.c | 3 ++- 1 file changed, 2

[PATCH v2 3/9] clk: at91: pmc: Support backup for programmable clocks

2017-09-15 Thread Romain Izard
From: Romain Izard When an AT91 programmable clock is declared in the device tree, register it into the Power Management Controller driver. On entering suspend mode, the driver saves and restores the Programmable Clock registers to support the backup mode for

[PATCH v2 8/9] atmel_flexcom: Support backup mode

2017-09-15 Thread Romain Izard
The controller used by a flexcom module is configured at boot, and left alone after this. As the configuration will be lost after backup mode, restore the state of the flexcom driver on resume. Signed-off-by: Romain Izard --- drivers/mfd/atmel-flexcom.c | 65

[PATCH v2 7/9] pwm: atmel-tcb: Support backup mode

2017-09-15 Thread Romain Izard
Save and restore registers for the PWM on suspend and resume, which makes hibernation and backup modes possible. Signed-off-by: Romain Izard --- drivers/pwm/pwm-atmel-tcb.c | 63 +++-- 1 file changed, 61 insertions(+), 2

[PATCH v2 9/9] tty/serial: atmel: Prevent a warning on suspend

2017-09-15 Thread Romain Izard
The atmel serial port driver reported the following warning on suspend: atmel_usart f802.serial: ttyS1: Unable to drain transmitter As the ATMEL_US_TXEMPTY status bit in ATMEL_US_CSR is always cleared when the transmitter is disabled, we need to know the transmitter's state to return the real

[PATCH v2 0/9] Various patches for SAMA5D2 backup mode

2017-09-15 Thread Romain Izard
While the core of the backup mode for SAMA5D2 has been integrated in v4.13, it is far from complete. Individual controllers in the chip have drivers that do not support the reset of the registers during suspend, and they need to be adapted to handle it. The first patch uses the clock wakeup code

incomplete MJPG frames through dwc2 USB host

2017-09-15 Thread Andrea Di Chiara
Hi, I'm using a RK3288 rockchip board whit kernel rockchip-linux release-4.4. It has two USB host and one of them has a DW controller. If I connect a UVC camera on it, I get into an incomplete MJPG frame error when enable stream with a image format greater or equal than 1920x1080. There isn't any

Re: [PATCH] usb: dwc3: Fix the USB 3.0 hub detection bug after warm boot

2017-09-15 Thread gustavo panizzo
Hello On Thu, Sep 07, 2017 at 01:51:31PM +0300, Felipe Balbi wrote: Hi, gustavo panizzo writes: --- drivers/usb/dwc3/core.c | 33 + 1 file changed, 33 insertions(+) diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index