Re: query on DWC3

2015-01-18 Thread sundeep subbaraya
Hi Felipe, On Thu, Jan 8, 2015 at 10:27 PM, Felipe Balbi ba...@ti.com wrote: Hi, On Tue, Jan 06, 2015 at 12:39:55PM +0530, sundeep subbaraya wrote: On Sun, Dec 14, 2014 at 08:39:18AM +0530, sundeep subbaraya wrote: Hi Paul, As per my understanding, for BULK OUT we do queue a request

[PATCH net-next 4/7] r8152: check RTL8152_UNPLUG for rtl8152_close

2015-01-18 Thread Hayes Wang
It is unnecessary to accress the hw register if the device is unplugged. Signed-off-by: Hayes Wang hayesw...@realtek.com --- drivers/net/usb/r8152.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index 21f853f..9be642e 100644

[PATCH net-next 2/7] r8152: adjust lpm timer

2015-01-18 Thread Hayes Wang
Set LPM timer to 500us, except for RTL_VER_04 which doesn't link at USB 3.0. Signed-off-by: Hayes Wang hayesw...@realtek.com --- drivers/net/usb/r8152.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index

[PATCH net-next 3/7] r8152: check linking status with netif_carrier_ok

2015-01-18 Thread Hayes Wang
Replace (tp-speed LINK_STATUS) with netif_carrier_ok(). Signed-off-by: Hayes Wang hayesw...@realtek.com --- drivers/net/usb/r8152.c | 23 +-- 1 file changed, 9 insertions(+), 14 deletions(-) diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index

[PATCH net-next 6/7] r8152: replace get_protocol with vlan_get_protocol

2015-01-18 Thread Hayes Wang
vlan_get_protocol() has been defined and use it to replace get_protocol(). Signed-off-by: Hayes Wang hayesw...@realtek.com --- drivers/net/usb/r8152.c | 16 ++-- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index

[PATCH net-next 5/7] r8152: adjust the link feed for hw_features

2015-01-18 Thread Hayes Wang
Keep NETIF_F_HW_VLAN_CTAG_RX and NETIF_F_HW_VLAN_CTAG_TX at the same line. Signed-off-by: Hayes Wang hayesw...@realtek.com --- drivers/net/usb/r8152.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index 9be642e..f66ffbd

[PATCH net-next 7/7] r8152: use BIT macro

2015-01-18 Thread Hayes Wang
Use BIT macro to replace (1 bits). Signed-off-by: Hayes Wang hayesw...@realtek.com --- drivers/net/usb/r8152.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c index 46440ed..c3a0224 100644

[PATCH v2 5/5] xhci: add a quirk for device disconnection errata for Synopsis Designware USB3 core

2015-01-18 Thread Sneeker Yeh
This issue is defined by a three-way race at disconnect, between 1) Class driver interrupt endpoint resheduling attempts if the ISR gave an ep error event due to device detach (it would try 3 times) 2) Disconnect interrupt on PORTSC_CSC, which is cleared by hub thread asynchronously 3) The

[PATCH v2 4/5] xhci: Platform: Set Synopsis device disconnection quirk based on platform data

2015-01-18 Thread Sneeker Yeh
If an xhci platform has Synopsis device disconnection errata then enable XHCI_DISCONNECT_QUIRK quirk flag. Signed-off-by: Sneeker Yeh sneeker@tw.fujitsu.com --- drivers/usb/host/xhci-plat.c |3 +++ include/linux/usb/xhci_pdriver.h |4 2 files changed, 7 insertions(+) diff

[PATCH net-next 0/7] r8152: adjust the code

2015-01-18 Thread Hayes Wang
Code adjustment. Hayes Wang (7): r8152: adjust rx_bottom r8152: adjust lpm timer r8152: check linking status with netif_carrier_ok r8152: check RTL8152_UNPLUG for rtl8152_close r8152: adjust the link feed for hw_features r8152: replace get_protocol with vlan_get_protocol r8152: use

[PATCH net-next 1/7] r8152: adjust rx_bottom

2015-01-18 Thread Hayes Wang
If a error occurs when submitting rx, skip the remaining submissions and try to submit them again next time. Signed-off-by: Hayes Wang hayesw...@realtek.com --- drivers/net/usb/r8152.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/drivers/net/usb/r8152.c

Re: [PATCH v3 1/3] usb: udc: add usb_gadget_find_udc

2015-01-18 Thread Peter Chen
On Fri, Jan 16, 2015 at 10:15:45AM -0600, Felipe Balbi wrote: On Fri, Jan 16, 2015 at 11:05:07AM -0500, Alan Stern wrote: On Fri, 16 Jan 2015, Peter Chen wrote: This is an internal API, and is used to find corresponding udc according to gadget. Signed-off-by: Peter Chen

[PATCH 1/2] usb: renesas_usbhs: add usbhsf_dma_init_pdev() function

2015-01-18 Thread Yoshihiro Shimoda
To add support for requesting DT DMA in the future, this patch adds usbhsf_dma_init_pdev() function. Signed-off-by: Yoshihiro Shimoda yoshihiro.shimoda...@renesas.com --- drivers/usb/renesas_usbhs/fifo.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git

[PATCH 2/2] usb: renesas_usbhs: add support for requesting DT DMA

2015-01-18 Thread Yoshihiro Shimoda
This patch adds dma_request_slave_channel_reason() calling to request dma slave channels for multiplatform environment. Signed-off-by: Yoshihiro Shimoda yoshihiro.shimoda...@renesas.com --- .../devicetree/bindings/usb/renesas_usbhs.txt |2 ++ drivers/usb/renesas_usbhs/fifo.c

[PATCH 0/2] usb: renesas_usbhs: add support for requesting DT DMA

2015-01-18 Thread Yoshihiro Shimoda
This patch series is based on Felipe's usb.git / testing/next branch. (commit id = 05cf6e00b8ff2b80c8b2d1e69f07cdd6c830efe0) Yoshihiro Shimoda (2): usb: renesas_usbhs: add usbhsf_dma_init_pdev() function usb: renesas_usbhs: add support for requesting DT DMA

[PATCH 1/1] usb: serial: Fintek F81232 driver improvement

2015-01-18 Thread Peter Hung
The original driver completed with TX function, but RX/MSR/MCR/LSR is not workable with this driver. So we rewrite it to make this device workable. This patch is tested with PassMark BurnInTest with Cycle-to-115200 + MCR/MSR check for 15mins checked with Suspend-To-RAM/DISK Signed-off-by: Peter

Re: [PATCHv3 1/5] USB: host: ehci_atmel: Add suspend/resume support

2015-01-18 Thread Boris Brezillon
Hi Sylvain, On Sat, 17 Jan 2015 23:49:00 +0100 Sylvain Rochet sylvain.roc...@finsecur.com wrote: Hi Sergei, On Sun, Jan 18, 2015 at 01:22:38AM +0300, Sergei Shtylyov wrote: There's little inconsistency in your patch subjects: you're using '_' but the files you're modifying are

Re: [PATCH v3 3/3] usb: udc: add usb_udc_activation_handler

2015-01-18 Thread Peter Chen
On Fri, Jan 16, 2015 at 11:11:56AM -0500, Alan Stern wrote: On Fri, 16 Jan 2015, Peter Chen wrote: During this API, the deactivation count will be update, and it will try to connect or disconnect gadget. It can be used to enable functions for gadget. Signed-off-by: Peter Chen

[PATCH 1/2] Add hub port switchable when checking port power

2015-01-18 Thread Chechun Kuo
In check_port_resume_type, we will check port power status to determine whether the status should change to -ENODEV. If all ports power control of the hub are handled at once, the port power status may not represent the real case and cause resume fail. At this patch, we add

[PATCH 2/2] Add HCS_PPC when getting root hub status

2015-01-18 Thread Chechun Kuo
for some platform ehci controller, it is possible there is no port power control capability in the root hub. And we add port power control determination when getting root hub port status. Signed-off-by: Chechun Kuo vichy@gmail.com --- drivers/usb/host/ehci-hub.c | 2 +- 1 file changed, 1

Re: Difference between run time and normal suspend

2015-01-18 Thread vichy
Is 3rd situation you mentioned above the function will be called is hcd-bus_suspend? hcd-bus_suspend will be called first, and shortly afterward ehci-suspend will be called (if this is an EHCI controller). Fourthly, you should realize that controller itself will also be put into runtime

[PATCH 2/2] USB: gadget: atmel_usba_udc: Enable/disable USB PLL on Vbus change

2015-01-18 Thread Sylvain Rochet
Prepare_enable on rising edge, disable_unprepare on falling edge. Reduce power consumption if USB PLL is not already necessary for OHCI or EHCI. If USB host is not connected we can sleep with USB PLL stopped. This driver does not support suspend/resume yet, not suspending if we are still attached

[PATCHv2 1/2] USB: gadget: atmel_usba_udc: Fixed vbus_prev initial state

2015-01-18 Thread Sylvain Rochet
If vbus gpio is high at init, we should set vbus_prev to true accordingly to the current vbus state. Without that, we skip the first vbus interrupt because the saved vbus state is not consistent. Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com ---

[PATCH v4 09/11] usb: storage: adjust module reference for scsi host

2015-01-18 Thread Akinobu Mita
The unusual usb storage drivers (ums-alauda, ums-cypress, ...) depend on usb-storage module. The module reference of these scsi host is initialized to usb-storage's one. Because these drivers use usb_stor_probe1() which is defined in usb-storage module and calls scsi_host_alloc(). So these

[PATCH v4 07/11] scsi: move module reference from scsi_host_template to Scsi_Host

2015-01-18 Thread Akinobu Mita
While accessing a scsi_device, the use count of the underlying LLDD module is incremented. The module reference is retrieved through .module field of struct scsi_host_template. This mapping between scsi_device and underlying LLDD module works well except some drivers which consist with the core

[PATCH v4 00/11] scsi: fix module reference mismatch for scsi host

2015-01-18 Thread Akinobu Mita
While accessing a scsi_device, the use count of the underlying LLDD module is incremented. The module reference is retrieved through .module field of struct scsi_host_template. This mapping between scsi_device and underlying LLDD module works well except some drivers which consist with the core

[PATCHv2 0/2] USB: gadget: atmel_usba_udc: Enable/disable USB PLL on Vbus change

2015-01-18 Thread Sylvain Rochet
Prepare_enable on rising edge, disable_unprepare on falling edge. Reduce power consumption if USB PLL is not already necessary for OHCI or EHCI. If USB host is not connected we can sleep with USB PLL stopped. This driver does not support suspend/resume yet, not suspending if we are still attached

Re: hard lockup with USB3380

2015-01-18 Thread Paul Jones
Ricardo, I think I figured out the problem: my 3380 was running in legacy adapter mode. I am now capable of connecting both g_mass_storage and g_ether without any kernel panics after ensuring the 3380 is in enhanced adapter mode. My only concern is the speed from my Mac to Linux: - g_ether: scp

Re: Fatal Exception on device when Hotsyncing Palm in linux

2015-01-18 Thread Henri Manson
Ubuntu 9.04: Linux ubuntu 2.6.28-11-generic #42-Ubuntu SMP Fri Apr 17 01:57:59 UTC 2009 i686 GNU/Linux Ubuntu 10.04.4: Linux ubuntu 2.6.32-38-generic #83-Ubuntu SMP Wed Jan 4 11:13:04 UTC 2012 i686 GNU/Linux On Fri, Jan 16, 2015 at 12:24 PM, Johan Hovold jo...@kernel.org wrote: On Thu, Jan 15,

Re: [PATCH 01/12] mfd: syscon: Add atmel-matrix registers definition

2015-01-18 Thread Lee Jones
On Wed, 14 Jan 2015, Alexandre Belloni wrote: From: Boris Brezillon boris.brezil...@free-electrons.com AT91 SoCs have a memory range reserved for internal bus configuration. Expose those registers so that drivers can make use of the matrix syscon declared in at91 DTs. Signed-off-by:

Re: [PATCH 04/12] mfd: syscon: Add Atmel SMC binding doc

2015-01-18 Thread Lee Jones
On Wed, 14 Jan 2015, Alexandre Belloni wrote: From: Boris Brezillon boris.brezil...@free-electrons.com The SMC registers are used to configure Atmel EBI (External Bus Interface) to interface with standard memory devices (NAND, NOR, SRAM or specialized devices like FPGAs). Declare this

Re: [PATCH 03/12] mfd: syscon: Add atmel-smc registers definition

2015-01-18 Thread Lee Jones
On Wed, 14 Jan 2015, Alexandre Belloni wrote: From: Boris Brezillon boris.brezil...@free-electrons.com Atmel AT91 SoCs have a memory range reserved for SMC (Static Memory Controller) configuration. Expose those registers so that drivers can make use of the smc syscon declared in at91 DTs.

Re: [PATCH 02/12] mfd: syscon: Add Atmel Matrix bus DT binding documentation

2015-01-18 Thread Lee Jones
On Wed, 14 Jan 2015, Alexandre Belloni wrote: From: Boris Brezillon boris.brezil...@free-electrons.com The Matrix registers are provided to configure internal bus behavior on at91 SoCs. Some registers might be accessed by several drivers (e.g. to configure external memory bus timings),

Re: [PATCH 01/12] mfd: syscon: Add atmel-matrix registers definition

2015-01-18 Thread Boris Brezillon
Hi Lee, On Sun, 18 Jan 2015 12:52:59 + Lee Jones lee.jo...@linaro.org wrote: On Wed, 14 Jan 2015, Alexandre Belloni wrote: From: Boris Brezillon boris.brezil...@free-electrons.com AT91 SoCs have a memory range reserved for internal bus configuration. Expose those registers so

[PATCHv4 4/6] USB: host: ehci-atmel: Move global variables to private struct

2015-01-18 Thread Sylvain Rochet
This patch move Atmel EHCI global variables (clocks ptr and clocked boolean) to private struct atmel_ehci_priv, stored in ehci-priv. Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com --- drivers/usb/host/ehci-atmel.c | 79 +++ 1 file changed, 50

[PATCHv4 6/6] USB: host: ohci-at91: usb_hcd_at91_probe(), remove useless stack initialisation

2015-01-18 Thread Sylvain Rochet
struct usb_hcd *hcd = NULL; ... hcd = usb_create_hcd(driver, dev, at91); This patch remove *hcd useless initialisation Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com --- drivers/usb/host/ohci-at91.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCHv4 0/6] USB: host: Atmel OHCI and EHCI drivers improvements

2015-01-18 Thread Sylvain Rochet
Suspend/resume support for EHCI. struct dev_pm_ops for OHCI. PLL stop for all sleep modes for OHCI. Removed global variables from both. Removed at91_suspend_entering_slow_clock() from both. Changes since v3: * Using struct dev_pm_ops instead of static struct platform_driver resume and

[PATCHv4 3/6] USB: host: ohci-at91: Stop/start USB PLL for all sleep modes

2015-01-18 Thread Sylvain Rochet
Disable/unprepare clocks without testing the sleep target_state, removed the at91_suspend_entering_slow_clock() call (which is only a target_state == PM_SUSPEND_MEM). Other kind of suspend now benefit from the power save induced by this PLL deactivation. The resume penalty is about 500 us, which

[PATCHv4 5/6] USB: host: ohci-at91: Move global variables to private struct

2015-01-18 Thread Sylvain Rochet
This patch move AT91 OHCI global variables (clocks ptr and clocked boolean) to private struct ohci_at91_priv, stored in ohci-priv. Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com --- drivers/usb/host/ohci-at91.c | 86 ++-- 1 file changed, 52

[PATCHv4 2/6] USB: host: ohci-at91: Use struct dev_pm_ops instead of struct platform_driver

2015-01-18 Thread Sylvain Rochet
This patch replace struct platform_driver.{resume,suspend} PM bindings to a new struct dev_pm_ops. Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com --- drivers/usb/host/ohci-at91.c | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git

[PATCHv4 1/6] USB: host: ehci-atmel: Add suspend/resume support

2015-01-18 Thread Sylvain Rochet
This patch add suspend/resume support for Atmel EHCI, mostly about disabling and unpreparing clocks so USB PLL is stopped before entering sleep state. Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com --- drivers/usb/host/ehci-atmel.c | 31 +++ 1 file changed,

Re: [PATCHv3 1/5] USB: host: ehci_atmel: Add suspend/resume support

2015-01-18 Thread Sergei Shtylyov
Hello. On 01/18/2015 01:49 AM, Sylvain Rochet wrote: There's little inconsistency in your patch subjects: you're using '_' but the files you're modifying are named using '-'... Indeed. @@ -187,6 +217,8 @@ static struct platform_driver ehci_atmel_driver = { .probe =

Re: Fatal Exception on device when Hotsyncing Palm in linux

2015-01-18 Thread Oliver Neukum
On Sat, 2015-01-17 at 00:06 +0100, Henri Manson wrote: I figured out how to get debug output, enclosed the results. How can I get usbserial debug output on Ubuntu 9.04? It appears it wasn't a module yet back then. out-crash: debug output form Ubuntu 14.04.1 which causes the Palm m505 to

Re: [PATCHv3 0/5] USB: host: Atmel OHCI and EHCI drivers improvements

2015-01-18 Thread Sylvain Rochet
Hello Alan, On Sun, Jan 18, 2015 at 12:20:49PM -0500, Alan Stern wrote: On Sat, 17 Jan 2015, Sylvain Rochet wrote: Sylvain Rochet (5): USB: host: ehci_atmel: Add suspend/resume support USB: host: ohci_at91: Stop/start USB PLL for all sleep modes USB: host: ehci_atmel: Move global

Re: [PATCH 2/2] USB: gadget: atmel_usba_udc: Enable/disable USB PLL on Vbus change

2015-01-18 Thread Boris Brezillon
On Sun, 18 Jan 2015 15:51:21 +0100 Sylvain Rochet sylvain.roc...@finsecur.com wrote: Prepare_enable on rising edge, disable_unprepare on falling edge. Reduce power consumption if USB PLL is not already necessary for OHCI or EHCI. If USB host is not connected we can sleep with USB PLL stopped.

[PATCH 001/001] usbhid: Fix initialisation and force effect modifications for the Microsoft Sidewinder Force Feedback Pro 2 joystick

2015-01-18 Thread Jim Keir
From: Jim Keir jimk...@yahoo.co.uk Signed-off-by: Jim Keir jimk...@yahoo.co.uk Currently the SWFF2 driver fails during initialisation, making the force capability of the joystick unusable. Further, there is a long-standing bug in the same driver where commands to update force parameters are

Re: Difference between run time and normal suspend

2015-01-18 Thread Alan Stern
On Sun, 18 Jan 2015, vichy wrote: after tracing the source, I only can find the ehci_suspend is called when system hibernate/suspend. rpm_suspend use below method to find out suspend callback if (dev-pm_domain) callback = dev-pm_domain-ops.runtime_suspend; else if (dev-type

Re: [PATCH 1/2] Add hub port switchable when checking port power

2015-01-18 Thread Alan Stern
On Sun, 18 Jan 2015, Chechun Kuo wrote: In check_port_resume_type, we will check port power status to determine whether the status should change to -ENODEV. If all ports power control of the hub are handled at once, the port power status may not represent the real case and cause resume

Re: [PATCHv3 0/5] USB: host: Atmel OHCI and EHCI drivers improvements

2015-01-18 Thread Alan Stern
On Sat, 17 Jan 2015, Sylvain Rochet wrote: Sylvain Rochet (5): USB: host: ehci_atmel: Add suspend/resume support USB: host: ohci_at91: Stop/start USB PLL for all sleep modes USB: host: ehci_atmel: Move global variables to private struct USB: host: ohci_at91: Move global variables to

[PATCHv3 1/2] USB: gadget: atmel_usba_udc: Fixed vbus_prev initial state

2015-01-18 Thread Sylvain Rochet
If vbus gpio is high at init, we should set vbus_prev to true accordingly to the current vbus state. Without that, we skip the first vbus interrupt because the saved vbus state is not consistent. Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com ---

[PATCHv3 2/2] USB: gadget: atmel_usba_udc: Enable/disable USB PLL on Vbus change

2015-01-18 Thread Sylvain Rochet
Prepare_enable on rising edge, disable_unprepare on falling edge. Reduce power consumption if USB PLL is not already necessary for OHCI or EHCI. If USB host is not connected we can sleep with USB PLL stopped. This driver does not support suspend/resume yet, not suspending if we are still attached

[PATCHv3 0/2] USB: gadget: atmel_usba_udc: Enable/disable USB PLL on Vbus change

2015-01-18 Thread Sylvain Rochet
Prepare_enable on rising edge, disable_unprepare on falling edge. Reduce power consumption if USB PLL is not already necessary for OHCI or EHCI. If USB host is not connected we can sleep with USB PLL stopped. This driver does not support suspend/resume yet, not suspending if we are still attached

Re: [PATCH v2 3/4] mfd: dln2: add start/stop RX URBs helpers

2015-01-18 Thread Lee Jones
On Tue, 16 Dec 2014, Octavian Purdila wrote: This is in preparation for adding suspend / resume support. Please re-submit this set with the Acks you have received. Also draft a cover-letter with the current status of set, what you need next etc. Signed-off-by: Octavian Purdila

Re: [PATCH 2/2] Add HCS_PPC when getting root hub status

2015-01-18 Thread Alan Stern
On Sun, 18 Jan 2015, Chechun Kuo wrote: for some platform ehci controller, it is possible there is no port power control capability in the root hub. And we add port power control determination when getting root hub port status. Why do you need to do this? Doesn't the current code always

Re: [PATCH 2/2] USB: gadget: atmel_usba_udc: Enable/disable USB PLL on Vbus change

2015-01-18 Thread Boris Brezillon
On Sun, 18 Jan 2015 15:51:21 +0100 Sylvain Rochet sylvain.roc...@finsecur.com wrote: Prepare_enable on rising edge, disable_unprepare on falling edge. Reduce power consumption if USB PLL is not already necessary for OHCI or EHCI. If USB host is not connected we can sleep with USB PLL stopped.

[PATCH 20/23] musb_virthub: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/musb/musb_virthub.c |7 --- 1 file changed, 4 insertions(+), 3

[PATCH 21/23] renesas_usbhs: mod_host: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare number to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/renesas_usbhs/mod_host.c |3 ++- 1 file changed, 2 insertions(+), 1

[PATCH 22/23] usbip: vhci_hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare number to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/usbip/vhci_hcd.c |3 ++- 1 file changed, 2 insertions(+), 1

[PATCH 23/23] wusbcore: rh: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/wusbcore/rh.c |4 ++-- 1 file changed, 2 insertions(+), 2

Re: [patch 1/6] usb: gadget: uvc: fix some error codes

2015-01-18 Thread Laurent Pinchart
Hi Dan, Thank you for the patch. On Wednesday 14 January 2015 23:59:48 Dan Carpenter wrote: We're basically saying ERR_CAST(NULL) and PTR_ERR(NULL) here, which is nonsensical. Signed-off-by: Dan Carpenter dan.carpen...@oracle.com Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com

Re: [patch 3/6] usb: gadget: uvc: remove an impossible condition

2015-01-18 Thread Laurent Pinchart
Hi Dan, Thank you for the patch. On Thursday 15 January 2015 00:02:18 Dan Carpenter wrote: num is a u32 so (num 0x) is never true. Also the range is already checked in kstrtou32(). Signed-off-by: Dan Carpenter dan.carpen...@oracle.com Acked-by: Laurent Pinchart

Re: [patch 4/6] usb: gadget: uvc: memory leak in uvcg_frame_make()

2015-01-18 Thread Laurent Pinchart
Hi Dan, Thank you for the patch. On Thursday 15 January 2015 00:03:08 Dan Carpenter wrote: We need to add a kfree(h) on an error path. Signed-off-by: Dan Carpenter dan.carpen...@oracle.com Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com diff --git

Re: [patch 6/6] usb: gadget: uvc: cleanup UVCG_FRAME_ATTR macro

2015-01-18 Thread Laurent Pinchart
Hi Dan, Thank you for the patch. On Thursday 15 January 2015 00:06:35 Dan Carpenter wrote: 1) Change conv an vnoc to to_cpu_endian to to_little_endian. 2) No need to check the limit because that is already handled in kstrtoXX so delete that parameter along with the check. 3) By using a

Re: [PATCH 1/3] Revert usb: gadget: uvc: cleanup __uvcg_fill_strm()

2015-01-18 Thread Laurent Pinchart
Hi Andrzej, Thank you for the patch. On Friday 16 January 2015 15:14:26 Andrzej Pietrasiewicz wrote: This reverts commit c0b53cb16250 (usb: gadget: uvc: cleanup __uvcg_fill_strm()). I can't find that commit in Linus' master, next/master or Felipe's next branch. If the patch hasn't been

Re: [PATCH 2/3] usb: gadget: uvc: preserve the address passed to kfree()

2015-01-18 Thread Laurent Pinchart
Hi Andrzej, Thank you for the patch. On Friday 16 January 2015 15:14:27 Andrzej Pietrasiewicz wrote: __uvcg_fill_strm() called from __uvcg_iter_stream_cls() might have advanced the data even if __uvcg_iter_stream_cls() returns an error, so use a backup copy as an argument to kfree().

Re: [PATCH 3/3] usb: gadget: uvc: comments for iterating over streaming hierarchy

2015-01-18 Thread Laurent Pinchart
Hi Andrzej, Thank you for the patch. On Friday 16 January 2015 15:14:28 Andrzej Pietrasiewicz wrote: The purpose of the functions and their parametrs might not be obvious to the reader, so explain it. Signed-off-by: Andrzej Pietrasiewicz andrze...@samsung.com ---

Re: [PATCHv2 2/2] USB: host: ohci_at91: Stop/start USB PLL for all sleep modes

2015-01-18 Thread Alexandre Belloni
On 17/01/2015 at 16:36:35 +0100, Sylvain Rochet wrote : Disable/unprepare clocks without testing the sleep target_state, removed the at91_suspend_entering_slow_clock() call (which is only a target_state == PM_SUSPEND_MEM). Other kind of suspend now benefit from the power save induced by this

Re: Suspected (out of tree) HCI issue

2015-01-18 Thread Vincent Pelletier
Hello, (please keep me cc'ed, I'm not subscribed) On Thu, 27 Nov 2014 13:00:28 -0500 (EST), Alan Stern st...@rowland.harvard.edu wrote: If your hardware resembles, say, the DWC2 hardware then maybe the DWC2 driver (drivers/usb/dwc2/) can be made to work with it. I've worked on this and have

[PATCH 00/23] Use USB_CHAR_* in the host drivers

2015-01-18 Thread Sergei Shtylyov
Hello. Here's a set of 23 patches against the 'usb-next' branch of Greg KH's 'usb.git' repo plus 2 FHCI patches I posted a week ago. It's to replace the use of the hex numbers where the USB_CHAR_* #define's should have been used. [01/23] ehci-hub: use HUB_CHAR_* [02/23] fhci-hub: use

[PATCH 02/23] fhci-hub: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare number to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/fhci-hub.c |2 +- 1 file changed, 1 insertion(+), 1

[PATCHv5 0/6] USB: host: Atmel OHCI and EHCI drivers improvements

2015-01-18 Thread Sylvain Rochet
USB: host: Atmel OHCI and EHCI drivers improvements Suspend/resume support for EHCI. struct dev_pm_ops for OHCI. Removed global variables from both. Fixed OHCI wake up support for STANDBY(wake-up enabled) and MEM(wake-up disabled) sleep targets. Changes since v4: * Re-add

[PATCH 01/23] ehci-hub: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/ehci-hub.c |8 1 file changed, 4 insertions(+), 4

[PATCHv5 3/6] USB: host: ehci-atmel: Move global variables to private struct

2015-01-18 Thread Sylvain Rochet
This patch move Atmel EHCI global variables (clocks ptr and clocked boolean) to private struct atmel_ehci_priv, stored in ehci-priv. Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com --- drivers/usb/host/ehci-atmel.c | 79 +++ 1 file changed, 50

[PATCHv5 1/6] USB: host: ehci-atmel: Add suspend/resume support

2015-01-18 Thread Sylvain Rochet
This patch add suspend/resume support for Atmel EHCI, mostly about disabling and unpreparing clocks so USB PLL is stopped before entering sleep state. Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com --- drivers/usb/host/ehci-atmel.c | 31 +++ 1 file changed,

[PATCHv5 5/6] USB: host: ohci-at91: Move global variables to private struct

2015-01-18 Thread Sylvain Rochet
This patch move AT91 OHCI global variables (clocks ptr and clocked boolean) to private struct ohci_at91_priv, stored in ohci-priv. Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com --- drivers/usb/host/ohci-at91.c | 86 ++-- 1 file changed, 52

[PATCHv5 6/6] USB: host: ohci-at91: usb_hcd_at91_probe(), remove useless stack initialisation

2015-01-18 Thread Sylvain Rochet
struct usb_hcd *hcd = NULL; ... hcd = usb_create_hcd(driver, dev, at91); This patch remove *hcd useless initialisation Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com --- drivers/usb/host/ohci-at91.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCHv5 2/6] USB: host: ohci-at91: Use struct dev_pm_ops instead of struct platform_driver

2015-01-18 Thread Sylvain Rochet
This patch replace struct platform_driver.{resume,suspend} PM bindings to a new struct dev_pm_ops. Signed-off-by: Sylvain Rochet sylvain.roc...@finsecur.com --- drivers/usb/host/ohci-at91.c | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git

[PATCHv5 4/6] USB: host: ohci-at91: Fix wake-up support

2015-01-18 Thread Sylvain Rochet
This device needs to be continuously clocked to provide wake up support, previously, if STANDBY target were chosen the device were enable_irq_wake()-prepared and clock still active and if MEM target were chosen the device were also enable_irq_wake()-prepared but not clocked anymore, which is

[PATCH 03/23] fotg210-hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/fotg210-hcd.c |4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH 04/23] fusbh200-hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/fusbh200-hcd.c |4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH 05/23] imx21-hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/imx21-hcd.c |5 ++--- 1 file changed, 2 insertions(+), 3

[PATCH 07/23] isp1362-hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. While at it, avoid duplicate computation when calling DBG(). Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com ---

[PATCH 06/23] isp116x-hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare number to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/isp116x-hcd.c |5 - 1 file changed, 4 insertions(+), 1

[PATCH 08/23] isp1760-hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/isp1760-hcd.c |6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH 09/23] max3421-hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare number to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/max3421-hcd.c |3 ++- 1 file changed, 2 insertions(+), 1

[PATCH 10/23] ohci-at91: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. While at it, stop setting already set HUB_CHAR_INDV_PORT_LPSM once again. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com ---

[PATCH 11/23] ohci-hub: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. While at it, fix the indentation. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/ohci-hub.c | 10 +- 1

[PATCH 12/23] ohci-s3c2410: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. While at it, stop setting already set HUB_CHAR_INDV_PORT_LPSM once again. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com ---

[PATCH 14/23] r8a66597-hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare number to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/r8a66597-hcd.c |3 ++- 1 file changed, 2 insertions(+), 1

[PATCH 15/23] sl811-hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/sl811-hcd.c |6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH 16/23] u132-hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/u132-hcd.c | 10 +- 1 file changed, 5 insertions(+), 5

[PATCH 17/23] uhci-hub: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare number to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/host/uhci-hub.c |5 +++-- 1 file changed, 3 insertions(+), 2

[PATCH 18/23] dwc2: hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare number to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/dwc2/hcd.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)

[PATCH 19/23] dummy_hcd: use HUB_CHAR_*

2015-01-18 Thread Sergei Shtylyov
Fix using the bare numbers to set the 'wHubCharacteristics' field of the Hub Descriptor while the values are #define'd in linux/usb/ch11.h. Signed-off-by: Sergei Shtylyov sergei.shtyl...@cogentembedded.com --- drivers/usb/gadget/udc/dummy_hcd.c |8 ++-- 1 file changed, 6

Control message failures kill entire XHCI stack

2015-01-18 Thread Devin Heitmueller
Hello, I'm debugging some issues on a couple of different USB TV tuners which boil down to the following error: xhci_hcd :00:14.0: xHCI host not responding to stop endpoint command. This is followed by the XHCI driver disconnecting *all* USB devices from the controller. I've done a bit of

Re: [PATCH 001/001] usbhid: Fix initialisation and force effect modifications for the Microsoft Sidewinder Force Feedback Pro 2 joystick

2015-01-18 Thread Alan Stern
On Sun, 18 Jan 2015, Jim Keir wrote: From: Jim Keir jimk...@yahoo.co.uk Signed-off-by: Jim Keir jimk...@yahoo.co.uk Currently the SWFF2 driver fails during initialisation, making the force capability of the joystick unusable. Further, there is a long-standing bug in the same driver where

Re: [PATCHv4 3/6] USB: host: ohci-at91: Stop/start USB PLL for all sleep modes

2015-01-18 Thread Sylvain Rochet
Hello, On Sun, Jan 18, 2015 at 08:36:03PM +0100, Sylvain Rochet wrote: Disable/unprepare clocks without testing the sleep target_state, removed the at91_suspend_entering_slow_clock() call (which is only a target_state == PM_SUSPEND_MEM). Other kind of suspend now benefit from the power save