Re: [PATCH net,stable] qmi_wwan/cdc_ether: add device ID for HP lt2523 (Novatel E371) WWAN card

2017-02-01 Thread Dan Williams
On Tue, 2017-01-24 at 10:36 +0100, Bjørn Mork wrote: > Bjørn Mork writes: > > > From: Dan Williams > > Woops! I didn't intend to blame this on you Dan.  Sorry. I reused > your > commit message, and obviously unintentionally also the "author" > line... > > But

[PATCH 17/17] usb: musb: dsps: make dsps_musb_clear_ep_rxintr() static

2017-02-01 Thread Bin Liu
From: Wei Yongjun Fixes the following sparse warning: drivers/usb/musb/musb_dsps.c:270:6: warning: symbol 'dsps_musb_clear_ep_rxintr' was not declared. Should it be static? Signed-off-by: Wei Yongjun Signed-off-by: Bin Liu ---

[PATCH 07/17] usb: musb: da8xx: Add support of suspend / resume

2017-02-01 Thread Bin Liu
From: Alexandre Bailon Implement PM methods specifics for da8xx glue. The only thing to do is to power off the phy. As the registers are in retention during suspend, there is no need to save them. Signed-off-by: Alexandre Bailon Signed-off-by: Bin

[PATCH 10/17] usb: musb: sunxi: Uses the resource-managed extcon API when registering extcon notifier

2017-02-01 Thread Bin Liu
From: Chanwoo Choi This patch just uses the resource-managed extcon API when registering the extcon notifier. Signed-off-by: Chanwoo Choi Acked-by: Maxime Ripard Signed-off-by: Bin Liu ---

[PATCH 14/17] usb: musb: Add support for optional VBUS irq to dsps glue layer

2017-02-01 Thread Bin Liu
From: Tony Lindgren We can now configure the PMIC interrupt to provide us VBUS events. In that case we don't need to constantly poll the status and can make it optional. This is only wired up for the mini-B interface on beaglebone. Note that eventually we should get also the

[PATCH 08/17] usb: musb: Add a quirk to preserve the session during suspend

2017-02-01 Thread Bin Liu
From: Alexandre Bailon On da8xx, VBUS is not maintained during suspend when musb is in host mode. On resume, all the connected devices will be disconnected and then will be enumerated again. This happens because MUSB_DEVCTL is cleared during suspend. Add a quirk to not

[PATCH 16/17] usb: musb: sunxi: add support for the variant in H3/V3s SoC

2017-02-01 Thread Bin Liu
From: Icenowy Zheng Allwinner H3/V3s features a variant of MUSB controller, which lacks one endpoint. Add support for it. Signed-off-by: Icenowy Zheng Acked-by: Maxime Ripard [b-...@ti.com: added usb: to commit subject

[PATCH 11/17] usb: musb: debugfs: allow forcing host mode together with speed in testmode

2017-02-01 Thread Bin Liu
From: Pali Rohár Based on the musb ug, force_host bit is allowed to be set along with force_hs or force_fs bit. It could help to implement forced host mode via testmode on Nokia N900. Signed-off-by: Pali Rohár Signed-off-by: Bin Liu

[PATCH 15/17] usb: musb: omap2430: constify dev_pm_ops structures

2017-02-01 Thread Bin Liu
From: Bhumika Goyal Declare dev_pm_ops structures as const as they are only stored in the pm field of a device_driver structure. This field is of type const, so dev_pm_ops structures having similar properties can be declared const too. Size details after cross compiling the

[PATCH 09/17] usb: musb: da8xx: Fix host mode suspend

2017-02-01 Thread Bin Liu
From: Alexandre Bailon On da8xx, VBUS is not maintained during suspend when musb is in host mode. On resume, all the connected devices will be disconnected and then will be enumerated again. This happens because MUSB_DEVCTL is cleared during suspend. Use the quirk

[PATCH 13/17] usb: musb: blackfin: fix unused warnings on suspend/resume

2017-02-01 Thread Bin Liu
From: Jérémy Lefaure When CONFIG_PM_SLEEP is disabled, SIMPLE_DEV_PM_OPS does not use bfin_resume and bfin_suspend even if CONFIG_PM is enabled: drivers/usb/musb/blackfin.c:602:12: warning: ‘bfin_resume’ defined but not used [-Wunused-function] static int

[PATCH 12/17] usb: musb: dsps: switch to static id for musb-hdrc platform devices

2017-02-01 Thread Bin Liu
The dsps glue uses PLATFORM_DEVID_AUTO when creating the musb-hdrc platform devices, this causes that the id will change in each system depending on the order of driver probe, the order of the usb instances defined in device-tree, or the list of enabled devices which use also PLATFORM_DEVID_AUTO

[PATCH 05/17] usb: musb: am35x: remove redundant code

2017-02-01 Thread Bin Liu
The session is cleared in the core whenever musb_platform_disable() is called, so clearing it in the glue driver *_musb_disable() is redundant. Signed-off-by: Bin Liu --- drivers/usb/musb/am35x.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/usb/musb/am35x.c

[PATCH 06/17] usb: musb: davinci: remove redundant code

2017-02-01 Thread Bin Liu
The session is cleared in the core whenever musb_platform_disable() is called, so clearing it in the glue driver *_musb_disable() is redundant. Signed-off-by: Bin Liu --- drivers/usb/musb/davinci.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/usb/musb/davinci.c

[PATCH 04/17] usb: musb: da8xx: remove redundant code

2017-02-01 Thread Bin Liu
The session is cleared in the core whenever musb_platform_disable() is called, so clearing it in the glue driver *_musb_disable() is redundant. Signed-off-by: Bin Liu --- drivers/usb/musb/da8xx.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/usb/musb/da8xx.c

[PATCH 00/17] musb patches for v4.11-rc1

2017-02-01 Thread Bin Liu
Hi Greg, This musb patch set is for next v4.11-rc1. It adds suspend/resume support in DA8xx glue, adds H3/V3s SoC support, fixes incorrect CPPI reference in DA8xx glue, switch to static id naming for musb-hdrc device to give consistent user experience, and a few compile warning fixes and

[PATCH 03/17] usb: musb: dsps: remove redundant code

2017-02-01 Thread Bin Liu
The session is cleared in the core whenever musb_platform_disable() is called, so clearing it in the glue driver *_musb_disable() is redundant. Signed-off-by: Bin Liu --- drivers/usb/musb/musb_dsps.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/usb/musb/musb_dsps.c

[PATCH 02/17] usb: musb: remove musb_generic_disable function

2017-02-01 Thread Bin Liu
musb_generic_disable() only has two lines of code. So remove it and let the callers directly call those two lines. Signed-off-by: Bin Liu --- drivers/usb/musb/musb_core.c | 26 ++ 1 file changed, 10 insertions(+), 16 deletions(-) diff --git

[PATCH 01/17] usb: musb: da8xx: Remove CPPI 3.0 quirk and methods

2017-02-01 Thread Bin Liu
From: Alexandre Bailon DA8xx driver is registering and using the CPPI 3.0 DMA controller but actually, the DA8xx has a CPPI 4.1 DMA controller. Remove the CPPI 3.0 quirk and methods. Fixes: f8e9f34f80a2 ("usb: musb: Fix up DMA related macros") Fixes: 7f6283ed6fe8 ("usb:

Re: [PATCH V2 1/2] dt-bindings: leds: document new led-triggers property

2017-02-01 Thread Rafał Miłecki
On 02/01/2017 10:26 PM, Jacek Anaszewski wrote: On 02/01/2017 04:56 PM, Rafał Miłecki wrote: On 01/31/2017 10:34 PM, Jacek Anaszewski wrote: On 01/31/2017 05:11 PM, Rafał Miłecki wrote: Thanks a lot Jacek for this explanation (and sorry but I needed a bit of time to think about this). I can

Re: [PATCH V2 1/2] dt-bindings: leds: document new led-triggers property

2017-02-01 Thread Jacek Anaszewski
On 02/01/2017 04:56 PM, Rafał Miłecki wrote: > On 01/31/2017 10:34 PM, Jacek Anaszewski wrote: >> On 01/31/2017 05:11 PM, Rafał Miłecki wrote: >>> On 01/25/2017 10:04 PM, Jacek Anaszewski wrote: On 01/25/2017 10:03 AM, Rafał Miłecki wrote: > On 21 January 2017 at 22:42, Jacek Anaszewski

Re: [PATCH 4.10-rc3 11/13] net: liquidio: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-01 Thread Felix Manlunas
Russell King wrote on Tue [2017-Jan-31 19:19:19 +]: > drivers/net/ethernet/cavium/liquidio/lio_vf_main.c:30: error: expected > declaration specifiers or '...' before string constant > drivers/net/ethernet/cavium/liquidio/lio_vf_main.c:30: warning: data >

Re: [PATCH] usb: gadget: udc: atmel: fix debug output

2017-02-01 Thread Nicolas Ferre
Le 01/02/2017 à 18:00, Alexandre Belloni a écrit : > On 01/02/2017 at 17:41:55 +0100, Arnd Bergmann wrote: >> The debug output now contains the wrong variable, as seen from the compiler >> warning: >> >> drivers/usb/gadget/udc/atmel_usba_udc.c: In function 'usba_ep_enable': >>

Re: [PATCH] usb: gadget: udc: atmel: fix debug output

2017-02-01 Thread Alexandre Belloni
On 01/02/2017 at 17:41:55 +0100, Arnd Bergmann wrote: > The debug output now contains the wrong variable, as seen from the compiler > warning: > > drivers/usb/gadget/udc/atmel_usba_udc.c: In function 'usba_ep_enable': > drivers/usb/gadget/udc/atmel_usba_udc.c:632:550: error: 'ept_cfg' may be used

Re: [PATCH 1/2] fs: configfs: make qw_sign attribute symmetric

2017-02-01 Thread Stefan Agner
On 2017-02-01 00:06, Greg KH wrote: > On Tue, Jan 31, 2017 at 06:19:16PM -0800, Stefan Agner wrote: >> Currently qw_sign requires UTF-8 character to set, but returns UTF-16 >> when read. This isn't obvious when simply using cat since the null >> characters are not visible, but hexdump unveils the

Re: [PATCH 2/2] fs: configfs: use hexadecimal values and new line

2017-02-01 Thread Stefan Agner
On 2017-02-01 01:02, Felipe Balbi wrote: > Hi, > > Greg KH writes: >> On Tue, Jan 31, 2017 at 06:19:17PM -0800, Stefan Agner wrote: >>> Other unsigned properties return hexadecimal values, follow this >>> convention when printing b_vendor_code too. Also add newlines

[PATCH] usb: gadget: udc: atmel: fix debug output

2017-02-01 Thread Arnd Bergmann
The debug output now contains the wrong variable, as seen from the compiler warning: drivers/usb/gadget/udc/atmel_usba_udc.c: In function 'usba_ep_enable': drivers/usb/gadget/udc/atmel_usba_udc.c:632:550: error: 'ept_cfg' may be used uninitialized in this function [-Werror=maybe-uninitialized]

Re: [PATCH 5/6] DT bindings documentation for Broadcom IPROC USB Device controller.

2017-02-01 Thread Rob Herring
On Mon, Jan 30, 2017 at 01:26:12PM +0530, Raviteja Garimella wrote: > The device node is used for UDCs integrated into Broadcom's > iProc family of SoCs'. The UDC is based on Synopsys Designware > Cores AHB Subsystem USB Device Controller IP. > > Signed-off-by: Raviteja Garimella

Re: [PATCH V2 1/2] dt-bindings: leds: document new led-triggers property

2017-02-01 Thread Rafał Miłecki
On 01/31/2017 10:34 PM, Jacek Anaszewski wrote: On 01/31/2017 05:11 PM, Rafał Miłecki wrote: On 01/25/2017 10:04 PM, Jacek Anaszewski wrote: On 01/25/2017 10:03 AM, Rafał Miłecki wrote: On 21 January 2017 at 22:42, Jacek Anaszewski wrote: On 01/21/2017 05:24 PM,

Re: [PATCH 4.10-rc3 05/13] net: bgmac: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-01 Thread Rafał Miłecki
On 01/31/2017 08:18 PM, Russell King wrote: drivers/net/ethernet/broadcom/bgmac.c:1015:17: error: dereferencing pointer to incomplete type 'struct mii_bus' drivers/net/ethernet/broadcom/bgmac.c:1185:2: error: implicit declaration of function 'phy_start' [-Werror=implicit-function-declaration]

Re: [PATCH 4.10-rc3 07/13] net: mvneta: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-01 Thread Thomas Petazzoni
Hello, On Tue, 31 Jan 2017 19:18:59 +, Russell King wrote: > drivers/net/ethernet/marvell/mvneta.c:2694:26: error: storage size of > 'status' isn't known > drivers/net/ethernet/marvell/mvneta.c:2695:26: error: storage size of > 'changed' isn't known >

Re: [PATCH 4.10-rc3 03/13] net: macb: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-02-01 Thread Nicolas Ferre
Le 31/01/2017 à 20:18, Russell King a écrit : > drivers/net/ethernet/cadence/macb.h:862:33: sparse: expected ; at end of > declaration > drivers/net/ethernet/cadence/macb.h:862:33: sparse: Expected } at end of > struct-union-enum-specifier > drivers/net/ethernet/cadence/macb.h:862:33: sparse:

Re: [PATCH 2/2] fs: configfs: use hexadecimal values and new line

2017-02-01 Thread Felipe Balbi
Hi, Greg KH writes: > On Tue, Jan 31, 2017 at 06:19:17PM -0800, Stefan Agner wrote: >> Other unsigned properties return hexadecimal values, follow this >> convention when printing b_vendor_code too. Also add newlines to >> the OS Descriptor support related

Re: Re: Re: Subject: [PATCH v1] USB:Core: BugFix: Proper handling of Race Condition when two USB class drivers try to call init_usb_class simultaneously

2017-02-01 Thread gre...@linuxfoundation.org
On Wed, Feb 01, 2017 at 07:24:44AM +, Ajay Kaher wrote: >   > >> At boot time, probe function of multiple connected devices > >> (proprietary devices) execute simultaneously. > > > >What exactly do you mean here? How can probe happen "simultaneously"? > >The USB core prevents this, right? >

RE: Re: Re: Subject: [PATCH v1] USB:Core: BugFix: Proper handling of Race Condition when two USB class drivers try to call init_usb_class simultaneously

2017-02-01 Thread Ajay Kaher
  >> At boot time, probe function of multiple connected devices >> (proprietary devices) execute simultaneously. > >What exactly do you mean here? How can probe happen "simultaneously"? >The USB core prevents this, right? I have observed two scenarios to call probe function: Scenario #1: Driver

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

2017-02-01 Thread Greg Kroah-Hartman
On Wed, Feb 01, 2017 at 12:10:17AM +0100, Rafael J. Wysocki wrote: > On Tue, Jan 3, 2017 at 7:33 AM, 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

Re: [PATCH 1/2] fs: configfs: make qw_sign attribute symmetric

2017-02-01 Thread Greg KH
On Tue, Jan 31, 2017 at 06:19:16PM -0800, Stefan Agner wrote: > Currently qw_sign requires UTF-8 character to set, but returns UTF-16 > when read. This isn't obvious when simply using cat since the null > characters are not visible, but hexdump unveils the true string: > > # echo MSFT100 >

Re: [PATCH 2/2] fs: configfs: use hexadecimal values and new line

2017-02-01 Thread Greg KH
On Tue, Jan 31, 2017 at 06:19:17PM -0800, Stefan Agner wrote: > Other unsigned properties return hexadecimal values, follow this > convention when printing b_vendor_code too. Also add newlines to > the OS Descriptor support related properties, like other sysfs > files use. configfs is not sysfs,

Re: [PATCH 1/2] fs: configfs: make qw_sign attribute symmetric

2017-02-01 Thread Greg KH
On Tue, Jan 31, 2017 at 06:19:16PM -0800, Stefan Agner wrote: > Currently qw_sign requires UTF-8 character to set, but returns UTF-16 > when read. This isn't obvious when simply using cat since the null > characters are not visible, but hexdump unveils the true string: > > # echo MSFT100 >