RE: [PATCH 1/2] usb: configfs: allow UDC binding rule configured as binding to *any* UDC

2016-05-04 Thread Du, Changbin
Hi, > > On most platforms, there is only one device controller available. > > In this case, we desn't care the UDC's name. So let's ignore the > > name by setting 'UDC' to 'any'. > > Hmm libubsgx allows to do this for a very long time. You simply pass > NULL instead of pointer to usbg_udc. > >

[PATCH v8 2/7] usb: mux: add generic code for dual role port mux

2016-05-04 Thread Lu Baolu
Several Intel platforms implement USB dual role by having completely separate xHCI and dwc3 IPs in PCH or SOC silicons. These two IPs share a single USB port. There is another external port mux which controls where the data lines should go. While the USB controllers are part of the silicon, the

[PATCH v8 5/7] mfd: intel_vuport: Add Intel virtual USB port MFD Driver

2016-05-04 Thread Lu Baolu
Some Intel platforms have an USB port mux controlled by GPIOs. There's a single ACPI platform device that provides 1) USB ID extcon device; 2) USB vbus regulator device; and 3) USB port switch device. This MFD driver will split these 3 devices for their respective drivers. [baolu: removed .owner

[PATCH v8 0/7] usb: add support for Intel dual role port mux

2016-05-04 Thread Lu Baolu
Intel SOC chips are featured with USB dual role. The host role is provided by Intel xHCI IP, and the gadget role is provided by IP from designware. Tablet platform designs always share a single port for both host and gadget controllers. There is a mux to switch the port to the right controller

[PATCH v8 1/7] regulator: fixed: add support for ACPI interface

2016-05-04 Thread Lu Baolu
Add support to retrieve fixed voltage configure information through ACPI interface. This is needed for Intel Bay Trail devices, where a GPIO is used to control the USB vbus. Signed-off-by: Lu Baolu --- drivers/regulator/fixed.c | 46

[PATCH v8 3/7] usb: mux: add driver for Intel gpio controlled port mux

2016-05-04 Thread Lu Baolu
In some Intel platforms, a single usb port is shared between USB host and device controller. The shared port is under control of GPIO pins. This patch adds the support for USB GPIO controlled port mux. [baolu: removed .owner per platform_no_drv_owner.cocci] Signed-off-by: David Cohen

[PATCH v8 6/7] usb: pci-quirks: add Intel USB drcfg mux device

2016-05-04 Thread Lu Baolu
In some Intel platforms, a single usb port is shared between USB host and device controllers. The shared port is under control of a switch which is defined in the Intel vendor defined extended capability for xHCI. This patch adds the support to detect and create the platform device for the port

[PATCH v8 7/7] MAINTAINERS: add maintainer entry for Intel USB dual role mux drivers

2016-05-04 Thread Lu Baolu
Add a maintainer entry for Intel USB dual role mux drivers and add myself as a maintainer. Signed-off-by: Lu Baolu --- MAINTAINERS | 10 ++ 1 file changed, 10 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 1d5b4be..6ab9e02 100644 --- a/MAINTAINERS

[PATCH v8 4/7] usb: mux: add driver for Intel drcfg controlled port mux

2016-05-04 Thread Lu Baolu
Several Intel PCHs and SOCs have an internal mux that is used to share one USB port between device controller and host controller. The mux is handled through the Dual Role Configuration Register. Signed-off-by: Heikki Krogerus Signed-off-by: Lu Baolu

Re: [PATCH] usb: dwc3: usb/dwc3: fake dissconnect event when turn off pullup

2016-05-04 Thread John Youn
On 5/4/2016 3:42 AM, Felipe Balbi wrote: > > Hi, > > John Youn writes: >>> John Youn writes: > "Du, Changbin" writes: >> Hi, Balbi, >> >> The step to reproduce this issue is: >> 1) connect device to a

Re: [PATCH 0/3] usb: USB Type-C Class and driver for UCSI

2016-05-04 Thread Guenter Roeck
On Tue, Feb 09, 2016 at 07:01:20PM +0200, Heikki Krogerus wrote: > Hi, > > The OS, or more precisely the user space, needs to be able to control > a few things regarding USB Type-C ports. The first thing that must be > allowed to be controlled is the data role. USB Type-C ports will > select the

Re: [REGRESSION] asix: Lots of asix_rx_fixup() errors and slow transmissions

2016-05-04 Thread John Stultz
On Tue, May 3, 2016 at 3:54 AM, Dean Jenkins wrote: > On 03/05/16 11:04, Guodong Xu wrote: >> >> did you test on ARM 64-bit system or ARM 32-bit? I ask because HiKey >> is an ARM 64-bit system. I suggest we should be careful on that. I saw >> similar issues when

[PATCH v5 0/3] USB: serial: cp210x: Bugfixes and cleanup in CRTSCTS flag code

2016-05-04 Thread Konstantin Shkolnyy
This patch series fixes bugs and replaces magic numbers with symbolic names in CRTSCTS flag code. v5: Fixed 2 compile errors in PATCH 2/3, otherwise no change. v4: Same series of patches, fixed names and defines by feedback. v3: Regenerated the patches correctly against the latest usb-next

[PATCH v5 1/3] USB: serial: cp210x: Fixed RTS mode setting by the CRTSCTS flag.

2016-05-04 Thread Konstantin Shkolnyy
A bug in the CRTSCTS handling caused RTS to alternate between CRTSCTS=0 => "RTS transmits active signal" and CRTSCTS=1 => "RTS receives flow control" instead of CRTSCTS=0 => "RTS is statically active" and CRTSCTS=1 => "RTS receives flow control" This only happened after first having enabled

[PATCH v5 3/3] USB: serial: cp210x: Cleaned up CRTSCTS flag code.

2016-05-04 Thread Konstantin Shkolnyy
The CRTSCTS flag code cleared (and inconsistently) bits unrelated to CRTSCTS functionality. It was also harder than necessary to read. Signed-off-by: Konstantin Shkolnyy --- drivers/usb/serial/cp210x.c | 32 ++-- 1 file changed, 10

[PATCH v5 2/3] USB: serial: cp210x: Got rid of magic numbers in CRTSCTS flag code.

2016-05-04 Thread Konstantin Shkolnyy
Replaced magic numbers used in the CRTSCTS flag code with symbolic names from the chip specification. Signed-off-by: Konstantin Shkolnyy --- drivers/usb/serial/cp210x.c | 109 ++-- 1 file changed, 84 insertions(+), 25

Re: [PATCH 0/2] musb_host: move DMA engine check from musb_tx_dma_set_mode_cppi_tusb() to its caller

2016-05-04 Thread Bin Liu
Hi, On Wed, May 04, 2016 at 11:56:15PM +0300, Sergei Shtylyov wrote: > Hello. > > On 05/04/2016 11:46 PM, Bin Liu wrote: > > Commit 754fe4a92c07 ("usb: musb: Remove ifdefs for TX DMA for > musb_host.c") > looks incomplete: the DMA engine checks are done outside the Mentor/UX500 >

Re: [patch] usbnet: smsc95xx: silence an uninitialized variable warning

2016-05-04 Thread David Miller
From: Dan Carpenter Date: Wed, 4 May 2016 09:22:01 +0300 > If the call to fn() fails then "buf" is uninitialized. Just return the > error code in that case. > > Signed-off-by: Dan Carpenter Applied. Please, in the future, use a consistent

Re: [patch] usbnet/smsc75xx: silence uninitialized variable warning

2016-05-04 Thread David Miller
From: Dan Carpenter Date: Wed, 4 May 2016 09:21:02 +0300 > If the fn() calls fail then "buf" is uninitialized. Just return early > in that situation. > > Signed-off-by: Dan Carpenter Applied. -- To unsubscribe from this list: send the line

Re: [PATCH 0/2] musb_host: move DMA engine check from musb_tx_dma_set_mode_cppi_tusb() to its caller

2016-05-04 Thread Sergei Shtylyov
Hello. On 05/04/2016 11:46 PM, Bin Liu wrote: Commit 754fe4a92c07 ("usb: musb: Remove ifdefs for TX DMA for musb_host.c") looks incomplete: the DMA engine checks are done outside the Mentor/UX500 handler but inside the CPPI/TUSB handler. Move the checks out of the CPPI/ TUSB handler into its

Re: [PATCHv2] musb_host: fix lockup on rxcsr_h_error

2016-05-04 Thread Bin Liu
Hi, On Wed, May 04, 2016 at 01:19:27PM -0700, Greg KH wrote: > On Wed, May 04, 2016 at 02:17:15PM -0500, Bin Liu wrote: > > Hi, > > > > On Wed, May 04, 2016 at 10:02:16PM +0300, Sergei Shtylyov wrote: > > > Hello. > > > > > > On 05/04/2016 09:56 PM, Bin Liu wrote: > > > > > > yes, it

Re: [PATCH 0/2] musb_host: move DMA engine check from musb_tx_dma_set_mode_cppi_tusb() to its caller

2016-05-04 Thread Bin Liu
Hi, On Wed, May 04, 2016 at 11:06:58PM +0300, Sergei Shtylyov wrote: > On 05/04/2016 11:01 PM, Bin Liu wrote: > > >>Commit 754fe4a92c07 ("usb: musb: Remove ifdefs for TX DMA for musb_host.c") > >>looks incomplete: the DMA engine checks are done outside the Mentor/UX500 > >>handler but inside

Re: [PATCH v3 1/2] usb: host: ehci-tegra: Grab the correct UTMI pads reset

2016-05-04 Thread Thierry Reding
On Wed, May 04, 2016 at 11:14:50AM -0600, Stephen Warren wrote: > On 05/04/2016 08:39 AM, Thierry Reding wrote: > > From: Thierry Reding > > > > There are three EHCI controllers on Tegra SoCs, each with its own reset > > line. However, the first controller contains a set of

Re: [PATCH v3 2/2] usb: host: ehci-tegra: Avoid getting the same reset twice

2016-05-04 Thread Thierry Reding
On Wed, May 04, 2016 at 11:23:20AM -0600, Stephen Warren wrote: > On 05/04/2016 08:40 AM, Thierry Reding wrote: > > From: Thierry Reding > > > > Starting with commit 0b52297f2288 ("reset: Add support for shared reset > > controls") there is a reference count for reset control

Re: [PATCHv2] musb_host: fix lockup on rxcsr_h_error

2016-05-04 Thread Greg KH
On Wed, May 04, 2016 at 02:17:15PM -0500, Bin Liu wrote: > Hi, > > On Wed, May 04, 2016 at 10:02:16PM +0300, Sergei Shtylyov wrote: > > Hello. > > > > On 05/04/2016 09:56 PM, Bin Liu wrote: > > > > yes, it also works with that reset and go to finish: > > > > diff --git

Re: [PATCH v3 2/2] usb: host: ehci-tegra: Avoid getting the same reset twice

2016-05-04 Thread Thierry Reding
On Wed, May 04, 2016 at 07:22:54PM +0200, Philipp Zabel wrote: > Hi Thierry, > > Am Mittwoch, den 04.05.2016, 16:40 +0200 schrieb Thierry Reding: > > From: Thierry Reding > > > > Starting with commit 0b52297f2288 ("reset: Add support for shared reset > > controls") there is

Re: [PATCH 0/2] musb_host: move DMA engine check from musb_tx_dma_set_mode_cppi_tusb() to its caller

2016-05-04 Thread Sergei Shtylyov
On 05/04/2016 11:01 PM, Bin Liu wrote: Commit 754fe4a92c07 ("usb: musb: Remove ifdefs for TX DMA for musb_host.c") looks incomplete: the DMA engine checks are done outside the Mentor/UX500 handler but inside the CPPI/TUSB handler. Move the checks out of the CPPI/ TUSB handler into its caller,

Re: [PATCH 0/2] musb_host: move DMA engine check from musb_tx_dma_set_mode_cppi_tusb() to its caller

2016-05-04 Thread Bin Liu
Hi, On Wed, May 04, 2016 at 01:51:58AM +0300, Sergei Shtylyov wrote: > Commit 754fe4a92c07 ("usb: musb: Remove ifdefs for TX DMA for musb_host.c") > looks incomplete: the DMA engine checks are done outside the Mentor/UX500 > handler but inside the CPPI/TUSB handler. Move the checks out of the

Re: [PATCH v4 3/7] phy: Add set_mode callback

2016-05-04 Thread David Lechner
On 05/04/2016 01:39 PM, Bin Liu wrote: Have you already tested this? I never tried changing mode via sysfs, but by quickly reviewing the code, I am wondering how it works. the core only calls ops->set_mode() but nothing else. To really switch the mode, the driver has to talk to the root hub,

Re: [PATCHv2] musb_host: fix lockup on rxcsr_h_error

2016-05-04 Thread Bin Liu
Hi, On Wed, May 04, 2016 at 10:02:16PM +0300, Sergei Shtylyov wrote: > Hello. > > On 05/04/2016 09:56 PM, Bin Liu wrote: > > yes, it also works with that reset and go to finish: > > diff --git a/drivers/usb/musb/musb_host.c > b/drivers/usb/musb/musb_host.c >

Re: [PATCHv2] musb_host: fix lockup on rxcsr_h_error

2016-05-04 Thread Sergei Shtylyov
Hello. On 05/04/2016 09:56 PM, Bin Liu wrote: yes, it also works with that reset and go to finish: diff --git a/drivers/usb/musb/musb_host.c b/drivers/usb/musb/musb_host.c index c3d5fc9..8cd98e7 100644 --- a/drivers/usb/musb/musb_host.c +++ b/drivers/usb/musb/musb_host.c @@ -1599,6 +1599,10

Re: [PATCHv2] musb_host: fix lockup on rxcsr_h_error

2016-05-04 Thread Bin Liu
Hi, On Wed, May 04, 2016 at 04:49:50PM +0200, Yegor Yefremov wrote: > On Tue, May 3, 2016 at 4:35 PM, Bin Liu wrote: > > Hi, > > > > On Tue, May 03, 2016 at 04:25:58PM +0200, Yegor Yefremov wrote: > >> On Tue, May 3, 2016 at 3:48 PM, Bin Liu wrote: > >> > Hi, > >> >

Re: [PATCH v4 3/7] phy: Add set_mode callback

2016-05-04 Thread Bin Liu
On Wed, May 04, 2016 at 01:20:36PM -0500, David Lechner wrote: > On 05/04/2016 01:10 PM, Bin Liu wrote: > >Hi, > > > >On Thu, Apr 14, 2016 at 01:35:14PM -0500, David Lechner wrote: > >>The initial use for this is for PHYs that have a mode related to USB OTG. > >>There are several SoCs (e.g. TI

Re: [RFT PATCH 1/3] usb: misc: usb3503: Fix HUB mode after bootloader initialization

2016-05-04 Thread Mark Brown
On Wed, May 04, 2016 at 02:01:57PM +0200, Krzysztof Kozlowski wrote: > This looks like pwrseq for MMC devices so the idea is to: > 1. Move drivers/mmc/core/pwrseq* to separate directory > (drivers/power/pwrseq ?) > 2. Add support for pwrseq to USB core, > 3. Add new pwrseq driver (or extend

Re: [PATCH v4 3/7] phy: Add set_mode callback

2016-05-04 Thread David Lechner
On 05/04/2016 01:10 PM, Bin Liu wrote: Hi, On Thu, Apr 14, 2016 at 01:35:14PM -0500, David Lechner wrote: The initial use for this is for PHYs that have a mode related to USB OTG. There are several SoCs (e.g. TI OMAP and DA8xx) that have a mode setting in the USB PHY to override OTG VBUS and

Re: [PATCH v4 3/7] phy: Add set_mode callback

2016-05-04 Thread Bin Liu
Hi, On Thu, Apr 14, 2016 at 01:35:14PM -0500, David Lechner wrote: > The initial use for this is for PHYs that have a mode related to USB OTG. > There are several SoCs (e.g. TI OMAP and DA8xx) that have a mode setting > in the USB PHY to override OTG VBUS and ID signals. > > Of course, the enum

Re: [GIT PULL] usb: Chipidea and OTG FSM changes for v4.7-rc1

2016-05-04 Thread Greg KH
On Wed, May 04, 2016 at 01:58:43PM +0800, Peter Chen wrote: > The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca: > > Linux 4.6-rc1 (2016-03-26 16:03:24 -0700) > > are available in the git repository at: > >

Re: [PATCH v3 2/2] usb: host: ehci-tegra: Avoid getting the same reset twice

2016-05-04 Thread Stephen Warren
On 05/04/2016 08:40 AM, Thierry Reding wrote: From: Thierry Reding Starting with commit 0b52297f2288 ("reset: Add support for shared reset controls") there is a reference count for reset control assertions. The goal is to allow resets to be shared by multiple devices and an

Re: [PATCH v3 2/2] usb: host: ehci-tegra: Avoid getting the same reset twice

2016-05-04 Thread Philipp Zabel
Hi Thierry, Am Mittwoch, den 04.05.2016, 16:40 +0200 schrieb Thierry Reding: > From: Thierry Reding > > Starting with commit 0b52297f2288 ("reset: Add support for shared reset > controls") there is a reference count for reset control assertions. The > goal is to allow resets

Re: [PATCH net-next 4/5] treewide: replace dev->trans_start update with helper

2016-05-04 Thread Antonio Quartulli
On Tue, May 03, 2016 at 04:33:13PM +0200, Florian Westphal wrote: > Replace all trans_start updates with netif_trans_update helper. > change was done via spatch: > > struct net_device *d; > @@ > - d->trans_start = jiffies > + netif_trans_update(d) > > Compile tested only. > > Cc:

Re: [PATCH v3 1/2] usb: host: ehci-tegra: Grab the correct UTMI pads reset

2016-05-04 Thread Stephen Warren
On 05/04/2016 08:39 AM, Thierry Reding wrote: From: Thierry Reding There are three EHCI controllers on Tegra SoCs, each with its own reset line. However, the first controller contains a set of UTMI configuration registers that are shared with its siblings. These registers

Re: [PATCH v4 0/7] da8xx USB PHY (was da8xx USB clocks)

2016-05-04 Thread David Lechner
On 04/14/2016 01:35 PM, David Lechner wrote: The "da8xx USB clocks" patch series was growing a bit too big, so on the advice of Sekhar Nori, I am splitting it into two parts. This part contains everything in drivers/ and the other part will contain everything in arch/arm/mach-davinci. This

RE: [PATCH] usb: xhci: ring: fix off-by-one error

2016-05-04 Thread David Laight
> From: Mathias Nyman > Sent: 03 May 2016 12:11 ... > > @@ -109,7 +109,7 @@ static int last_trb(struct xhci_hcd *xhci, struct > > xhci_ring *ring, > > struct xhci_segment *seg, union xhci_trb *trb) > > { > > if (ring == xhci->event_ring) > > - return trb ==

Re: MUSB driver on AM3352 dropping USB packets

2016-05-04 Thread Bin Liu
Hi, On Wed, May 04, 2016 at 03:48:50PM +, Andrew Goodbody wrote: > Hi, > > I have been investigating communication issues with iPads. When the > system is busy it seems that the musb driver is silently dropping > occasional packets. I have a usbmon trace that does not show the > packet and I

MUSB driver on AM3352 dropping USB packets

2016-05-04 Thread Andrew Goodbody
Hi, I have been investigating communication issues with iPads. When the system is busy it seems that the musb driver is silently dropping occasional packets. I have a usbmon trace that does not show the packet and I have a trace from a hardware USB analyser that does show the packet. So the

Re: [PATCH v3 1/2] usb: host: ehci-tegra: Grab the correct UTMI pads reset

2016-05-04 Thread Thierry Reding
On Wed, May 04, 2016 at 07:57:10AM -0700, Greg Kroah-Hartman wrote: > On Wed, May 04, 2016 at 04:39:59PM +0200, Thierry Reding wrote: > > From: Thierry Reding > > > > There are three EHCI controllers on Tegra SoCs, each with its own reset > > line. However, the first

Re: [PATCH v3 1/2] usb: host: ehci-tegra: Grab the correct UTMI pads reset

2016-05-04 Thread Greg Kroah-Hartman
On Wed, May 04, 2016 at 04:39:59PM +0200, Thierry Reding wrote: > From: Thierry Reding > > There are three EHCI controllers on Tegra SoCs, each with its own reset > line. However, the first controller contains a set of UTMI configuration > registers that are shared with its

[PATCH v3 2/2] usb: host: ehci-tegra: Avoid getting the same reset twice

2016-05-04 Thread Thierry Reding
From: Thierry Reding Starting with commit 0b52297f2288 ("reset: Add support for shared reset controls") there is a reference count for reset control assertions. The goal is to allow resets to be shared by multiple devices and an assert will take effect only when all instances

Re: [PATCH] usb: host: ehci-tegra: Avoid getting the same reset twice

2016-05-04 Thread Thierry Reding
On Tue, May 03, 2016 at 03:16:56PM -0400, Alan Stern wrote: > On Tue, 3 May 2016, Thierry Reding wrote: > > > From: Thierry Reding > > > > Starting with commit 0b52297f2288 ("reset: Add support for shared reset > > controls") there is a reference count for reset control

Re: [PATCHv2] musb_host: fix lockup on rxcsr_h_error

2016-05-04 Thread Yegor Yefremov
On Tue, May 3, 2016 at 4:35 PM, Bin Liu wrote: > Hi, > > On Tue, May 03, 2016 at 04:25:58PM +0200, Yegor Yefremov wrote: >> On Tue, May 3, 2016 at 3:48 PM, Bin Liu wrote: >> > Hi, >> > >> > On Tue, May 03, 2016 at 12:03:52PM +0200, Yegor Yefremov wrote: >> >> On Thu,

[PATCH v3 1/2] usb: host: ehci-tegra: Grab the correct UTMI pads reset

2016-05-04 Thread Thierry Reding
From: Thierry Reding There are three EHCI controllers on Tegra SoCs, each with its own reset line. However, the first controller contains a set of UTMI configuration registers that are shared with its siblings. These registers will only be reset as part of the first

Re: [PATCH v2 2/2] usb: host: ehci-tegra: Avoid getting the same reset twice

2016-05-04 Thread Thierry Reding
On Wed, May 04, 2016 at 04:22:02PM +0200, Thierry Reding wrote: > From: Thierry Reding > > Starting with commit 0b52297f2288 ("reset: Add support for shared reset > controls") there is a reference count for reset control assertions. The > goal is to allow resets to be shared

Re: [PATCH] usb: misc: usbtest: fix pattern tests for scatterlists.

2016-05-04 Thread Alan Stern
On Wed, 4 May 2016, Felipe Balbi wrote: > > multiple of 512 bytes and the maxpacket size is 1024. Then you either > > that's not common case for testusb. One of the test cases (see below) > exercises exactly small sg entries: > > # try to trigger short OUT processing bugs >

[PATCH v2 1/2] usb: host: ehci-tegra: Grab the correct UTMI pads reset

2016-05-04 Thread Thierry Reding
From: Thierry Reding There are three EHCI controllers on Tegra SoCs, each with its own reset line. However, the first controller contains a set of UTMI configuration registers that are shared with its siblings. These registers will only be reset as part of the first

[PATCH v2 2/2] usb: host: ehci-tegra: Avoid getting the same reset twice

2016-05-04 Thread Thierry Reding
From: Thierry Reding Starting with commit 0b52297f2288 ("reset: Add support for shared reset controls") there is a reference count for reset control assertions. The goal is to allow resets to be shared by multiple devices and an assert will take effect only when all instances

[PATCH] usb: Use (foo *) instead of (foo*).

2016-05-04 Thread Sandhya Bankar
Use (foo *) instead of (foo*). Signed-off-by: Sandhya Bankar --- drivers/usb/gadget/config.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/gadget/config.c b/drivers/usb/gadget/config.c index e6c0542..17a6077 100644 ---

Re: [PATCH v7 10/14] usb: otg: add hcd companion support

2016-05-04 Thread Roger Quadros
On 04/05/16 16:17, Rob Herring wrote: > On Mon, May 02, 2016 at 03:18:53PM +0300, Roger Quadros wrote: >> From: Yoshihiro Shimoda >> >> Since some host controller (e.g. EHCI) needs a companion host controller >> (e.g. OHCI), this patch adds such a configuration

Re: [EXT] Re: [PATCH v4 2/3] USB: serial: cp210x: Got rid of magic numbers in CRTSCTS flag code.

2016-05-04 Thread Johan Hovold
On Wed, May 04, 2016 at 01:17:26PM +, Konstantin Shkolnyy wrote: > > -Original Message- > > From: Johan Hovold [mailto:jhov...@gmail.com] On Behalf Of Johan Hovold > > Sent: Wednesday, May 04, 2016 07:55 > > To: Konstantin Shkolnyy > > Cc: Johan Hovold; Konstantin Shkolnyy;

RE: [EXT] Re: [PATCH v4 2/3] USB: serial: cp210x: Got rid of magic numbers in CRTSCTS flag code.

2016-05-04 Thread Konstantin Shkolnyy
> -Original Message- > From: Johan Hovold [mailto:jhov...@gmail.com] On Behalf Of Johan Hovold > Sent: Wednesday, May 04, 2016 07:55 > To: Konstantin Shkolnyy > Cc: Johan Hovold; Konstantin Shkolnyy; linux-usb@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: Re: [EXT] Re: [PATCH

Re: [PATCH v7 10/14] usb: otg: add hcd companion support

2016-05-04 Thread Rob Herring
On Mon, May 02, 2016 at 03:18:53PM +0300, Roger Quadros wrote: > From: Yoshihiro Shimoda > > Since some host controller (e.g. EHCI) needs a companion host controller > (e.g. OHCI), this patch adds such a configuration to use it in the OTG > core. > >

Re: [PATCH v7 09/14] usb: of: add an API to get OTG device from USB controller node

2016-05-04 Thread Rob Herring
On Mon, May 02, 2016 at 03:18:52PM +0300, Roger Quadros wrote: > The OTG controller and the USB controller an be linked via the > 'otg-controller' property in the USB controller's device node. > > of_usb_get_otg() can be used to get the OTG controller device > from the USB controller's device

RE: [EXT] Re: [PATCH v4 2/3] USB: serial: cp210x: Got rid of magic numbers in CRTSCTS flag code.

2016-05-04 Thread Konstantin Shkolnyy
> -Original Message- > From: linux-usb-ow...@vger.kernel.org [mailto:linux-usb- > ow...@vger.kernel.org] On Behalf Of Johan Hovold > Sent: Wednesday, May 04, 2016 02:29 > To: Konstantin Shkolnyy > Cc: jo...@kernel.org; linux-usb@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject:

Re: [EXT] Re: [PATCH v4 2/3] USB: serial: cp210x: Got rid of magic numbers in CRTSCTS flag code.

2016-05-04 Thread Johan Hovold
On Wed, May 04, 2016 at 12:46:17PM +, Konstantin Shkolnyy wrote: > > -Original Message- From: linux-usb-ow...@vger.kernel.org > > [mailto:linux-usb- ow...@vger.kernel.org] On Behalf Of Johan Hovold > > Sent: Wednesday, May 04, 2016 02:29 To: Konstantin Shkolnyy Cc: > >

[PATCH 2/2] usb-midi: correct speed checking

2016-05-04 Thread Oliver Neukum
Allow for SS+ USB devices Signed-off-by: Oliver Neukum --- sound/usb/midi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/usb/midi.c b/sound/usb/midi.c index 47de8af..7ba9292 100644 --- a/sound/usb/midi.c +++ b/sound/usb/midi.c @@ -911,6 +911,7 @@ static void

[PATCH 1/2] usb-audio: correct speed checking

2016-05-04 Thread Oliver Neukum
Allow handling SS+ USB devices correctly. Signed-off-by: Oliver Neukum --- sound/usb/card.c | 4 sound/usb/helper.c | 1 + 2 files changed, 5 insertions(+) diff --git a/sound/usb/card.c b/sound/usb/card.c index 3fc6358..69860da 100644 --- a/sound/usb/card.c +++

Re: Re: [PATCH] USB: serial: option: add more ZTE device ids

2016-05-04 Thread Johan Hovold
On Wed, May 04, 2016 at 04:34:22PM +0800, 刘磊 wrote: > dear all: > thank you very much. > Excuse me for last submit mail, i missed some device ID. we need to add > some other device ID to linux kernel, that has been already used in our > corporation's device. > we need some help from

Re: [RFT PATCH 1/3] usb: misc: usb3503: Fix HUB mode after bootloader initialization

2016-05-04 Thread Krzysztof Kozlowski
On 05/03/2016 08:00 PM, Rob Herring wrote: > On Mon, May 02, 2016 at 11:55:01AM +0100, Mark Brown wrote: >> On Mon, May 02, 2016 at 11:49:12AM +0200, Krzysztof Kozlowski wrote: >> >>> This VDD regulator supply actually is not a usb3503 USB HUB regulator >>> supply... but a supply to the LAN

Re: [PATCH] usb: xhci-mtk: fixup mouse wakeup failure during system suspend

2016-05-04 Thread Felipe Balbi
Hi, chunfeng yun writes: >> chunfeng yun writes: >> > On Tue, 2016-05-03 at 12:33 +0300, Felipe Balbi wrote: >> >> Hi, >> >> >> >> chunfeng yun writes: >> >> >> chunfeng yun writes:

Re: [PATCH net-next 4/5] treewide: replace dev->trans_start update with helper

2016-05-04 Thread Mugunthan V N
On Tuesday 03 May 2016 08:03 PM, Florian Westphal wrote: > Replace all trans_start updates with netif_trans_update helper. > change was done via spatch: > > struct net_device *d; > @@ > - d->trans_start = jiffies > + netif_trans_update(d) > > Compile tested only. > > Cc:

Re: [PATCH] usb: xhci-mtk: fixup mouse wakeup failure during system suspend

2016-05-04 Thread chunfeng yun
Hi, On Wed, 2016-05-04 at 11:03 +0300, Felipe Balbi wrote: > Hi, > > chunfeng yun writes: > > On Tue, 2016-05-03 at 12:33 +0300, Felipe Balbi wrote: > >> Hi, > >> > >> chunfeng yun writes: > >> >> chunfeng yun

Re: [PATCH] usb: dwc3: usb/dwc3: fake dissconnect event when turn off pullup

2016-05-04 Thread Felipe Balbi
Hi, John Youn writes: >> John Youn writes: "Du, Changbin" writes: > Hi, Balbi, > > The step to reproduce this issue is: > 1) connect device to a host and wait its enumeration. > 2) trigger software

Re: [PATCH] usb: gadget: f_fs: Fix kernel panic for SuperSpeed

2016-05-04 Thread Felipe Balbi
Hi, Jim Lin writes: >>> In f_fs.c >>> " >>> static int __ffs_data_do_os_desc(enum ffs_os_desc_type type, >>>struct usb_os_desc_header *h, void *data, >>>unsigned len, void *priv) >>> { >>> struct ffs_data *ffs = priv; >>>

Re: [linux-sunxi] [PATCH] musb: sunxi: Ignore VBus errors in host-only mode

2016-05-04 Thread Michal Suchanek
On 14 September 2015 at 22:25, Maxime Ripard wrote: > On Thu, Sep 10, 2015 at 08:38:38PM +0200, Hans de Goede wrote: >> Hi, >> >> On 10-09-15 20:30, Maxime Ripard wrote: >> >On Thu, Sep 10, 2015 at 08:23:23PM +0200, Hans de Goede wrote: >> >>Hi, >> >> >> >>On

Re: [PATCH] usb: misc: usbtest: fix pattern tests for scatterlists.

2016-05-04 Thread Mathias Nyman
On 03.05.2016 17:37, Alan Stern wrote: On Tue, 3 May 2016, Felipe Balbi wrote: Hi, Alan Stern writes: On Mon, 2 May 2016, Mathias Nyman wrote: The current implemenentation restart the sent pattern for each entry in the sg list. The receiving end expects a

[PATCH] usb: Remove unnecessary space before operator ','.

2016-05-04 Thread Sandhya Bankar
Remove unnecessary space before operator ','. Signed-off-by: Sandhya Bankar --- drivers/usb/phy/phy-twl6030-usb.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/usb/phy/phy-twl6030-usb.c b/drivers/usb/phy/phy-twl6030-usb.c

[PATCH] usb: Remove unnecessary space before open square bracket.

2016-05-04 Thread Sandhya Bankar
Remove unnecessary space before open square bracket. Signed-off-by: Sandhya Bankar --- drivers/usb/isp1760/isp1760-if.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/isp1760/isp1760-if.c b/drivers/usb/isp1760/isp1760-if.c index

Re: [RESEND PATCH v10 4/4] power: wm831x_power: Support USB charger current limit management

2016-05-04 Thread Mark Brown
On Wed, May 04, 2016 at 08:59:23AM +0530, Manish Badarkhe wrote: > > They're in the silicon, it's just a table of values that were put into > > the silicon at design time. The defines would just be TABLE_ENTRY_1 or > > whatever. > Thanks for the clarification, In that case,

Re: [PATCH v6 09/12] usb: gadget: udc: adapt to OTG core

2016-05-04 Thread Roger Quadros
On 04/05/16 11:03, Jun Li wrote: > Hi > >> -Original Message- >> From: Roger Quadros [mailto:rog...@ti.com] >> Sent: Wednesday, May 04, 2016 2:37 PM >> To: Peter Chen >> Cc: Jun Li ; st...@rowland.harvard.edu; ba...@kernel.org; >>

Re:Re: [PATCH] USB: serial: option: add more ZTE device ids

2016-05-04 Thread 刘磊
dear all: thank you very much. Excuse me for last submit mail, i missed some device ID. we need to add some other device ID to linux kernel, that has been already used in our corporation's device. we need some help from yours. looking forward for you reply, thanks! Signed-off-by: lei

RE: [PATCH v6 09/12] usb: gadget: udc: adapt to OTG core

2016-05-04 Thread Jun Li
Hi > -Original Message- > From: Roger Quadros [mailto:rog...@ti.com] > Sent: Wednesday, May 04, 2016 2:37 PM > To: Peter Chen > Cc: Jun Li ; st...@rowland.harvard.edu; ba...@kernel.org; > gre...@linuxfoundation.org; peter.c...@freescale.com; >

Re: [PATCH net-next 4/5] treewide: replace dev->trans_start update with helper

2016-05-04 Thread Felipe Balbi
Hi, Florian Westphal writes: > Replace all trans_start updates with netif_trans_update helper. > change was done via spatch: > > struct net_device *d; > @@ > - d->trans_start = jiffies > + netif_trans_update(d) > > Compile tested only. > > Cc:

Re: [PATCH 1/2] usb: configfs: allow UDC binding rule configured as binding to *any* UDC

2016-05-04 Thread Krzysztof Opasiak
On 05/03/2016 05:04 AM, changbin...@intel.com wrote: > From: "Du, Changbin" > > On most platforms, there is only one device controller available. > In this case, we desn't care the UDC's name. So let's ignore the > name by setting 'UDC' to 'any'. Hmm libubsgx allows to

Re: [PATCH] usb: misc: usbtest: fix pattern tests for scatterlists.

2016-05-04 Thread Felipe Balbi
Hi, Alan Stern writes: >> Alan Stern writes: >> > On Mon, 2 May 2016, Mathias Nyman wrote: >> > >> >> The current implemenentation restart the sent pattern for each entry in >> >> the sg list. The receiving end expects a continuous pattern,

Re: [PATCH] usb: gadget: f_fs: Fix kernel panic for SuperSpeed

2016-05-04 Thread Jim Lin
On 2016年04月29日 19:57, Felipe Balbi wrote: * PGP Signed by an unknown key Hi, Jim Lin writes: On 2016年04月28日 20:21, Felipe Balbi wrote: I also attach git log of system/core/adb/usb_linux_client.cpp of Android N for your reference. " Author: Badhri Jagan Sridharan

Re: [PATCH] usb: xhci-mtk: fixup mouse wakeup failure during system suspend

2016-05-04 Thread Felipe Balbi
Hi, chunfeng yun writes: > On Tue, 2016-05-03 at 12:33 +0300, Felipe Balbi wrote: >> Hi, >> >> chunfeng yun writes: >> >> chunfeng yun writes: >> >> > On Thu, 2016-04-21 at 10:04 +0800, Chunfeng Yun wrote: >> >>

Re: [PATCH v6 09/12] usb: gadget: udc: adapt to OTG core

2016-05-04 Thread Peter Chen
On Wed, May 04, 2016 at 09:37:29AM +0300, Roger Quadros wrote: > Peter, > > On 04/05/16 06:35, Peter Chen wrote: > > On Tue, May 03, 2016 at 06:44:46PM +0300, Roger Quadros wrote: > >> Hi, > >> > >> On 03/05/16 10:06, Jun Li wrote: > >>> Hi > >>> > >>> /** > >>> + *

Re: [PATCH] drivers: usb: dwc3 : Configure DMA properties and ops from DT

2016-05-04 Thread Felipe Balbi
Hi, Rajesh Bhagat writes: > On certain platforms (e.g. ARM64) the dma_ops needs to be explicitly set > to be able to do DMA allocations, so use the of_dma_configure() helper > to populate the dma properties and assign an appropriate dma_ops. > > Signed-off-by: Rajesh

Re: [REGRESSION] asix: Lots of asix_rx_fixup() errors and slow transmissions

2016-05-04 Thread Dean Jenkins
On 04/05/16 01:28, David B. Robins wrote: Here is the code snippet from the patch with my annotations between # #, I will try to explain my intentions. Feel free to point out any flaws: if (rx->remaining && (rx->remaining + sizeof(u32) <= skb->len)) { # Only runs when

Re: [PATCH] usb: serial: ti_usb_3410_5052: add MOXA UPORT 11x0 support

2016-05-04 Thread Johan Hovold
On Tue, May 03, 2016 at 01:46:51PM +0200, Mathieu OTHACEHE wrote: > > No, I was trying to say that the we should not attempt to load a > > firmware on the "ti_usb-v%04x-p%04x.fw" format before loading the moxa > > firmware. > > For MTS devices (mts_*.fw) and for devices using generic firmware

Re: [PATCH v4 2/3] USB: serial: cp210x: Got rid of magic numbers in CRTSCTS flag code.

2016-05-04 Thread Johan Hovold
On Tue, May 03, 2016 at 07:52:23PM -0500, Konstantin Shkolnyy wrote: > Replaced magic numbers used in the CRTSCTS flag code with symbolic names > from the chip specification. > > Signed-off-by: Konstantin Shkolnyy This patch does not even compile. Please be more

Re: [PATCH] USB: serial: option: add more ZTE device ids

2016-05-04 Thread Johan Hovold
On Tue, May 03, 2016 at 02:44:19PM -0700, Greg Kroah-Hartman wrote: > From: lei liu > > More ZTE device ids. > > Signed-off-by: lei liu > Cc: stable > [properly sort them - gregkh] > Signed-off-by: Greg Kroah-Hartman

Re: [PATCH v6 09/12] usb: gadget: udc: adapt to OTG core

2016-05-04 Thread Roger Quadros
Peter, On 04/05/16 06:35, Peter Chen wrote: > On Tue, May 03, 2016 at 06:44:46PM +0300, Roger Quadros wrote: >> Hi, >> >> On 03/05/16 10:06, Jun Li wrote: >>> Hi >>> >>> /** >>> + * usb_gadget_start - start the usb gadget controller and >>> +connect to bus >>> + *

[patch] usbnet: smsc95xx: silence an uninitialized variable warning

2016-05-04 Thread Dan Carpenter
If the call to fn() fails then "buf" is uninitialized. Just return the error code in that case. Signed-off-by: Dan Carpenter diff --git a/drivers/net/usb/smsc95xx.c b/drivers/net/usb/smsc95xx.c index 2edc2bc..d9d2806 100644 --- a/drivers/net/usb/smsc95xx.c +++

[patch] usbnet/smsc75xx: silence uninitialized variable warning

2016-05-04 Thread Dan Carpenter
If the fn() calls fail then "buf" is uninitialized. Just return early in that situation. Signed-off-by: Dan Carpenter diff --git a/drivers/net/usb/smsc75xx.c b/drivers/net/usb/smsc75xx.c index c369db9..9af9799 100644 --- a/drivers/net/usb/smsc75xx.c +++

[GIT PULL] usb: Chipidea and OTG FSM changes for v4.7-rc1

2016-05-04 Thread Peter Chen
The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca: Linux 4.6-rc1 (2016-03-26 16:03:24 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/peter.chen/usb.git/ tags/usb-ci-v4.7-rc1 for you to fetch changes up to