Re: [PATCH v2 4/4] usb: dwc3: Workaround for super-speed host on dra7 in dual-role mode

2017-02-23 Thread Vivek Gautam
On 02/24/2017 06:27 AM, Peter Chen wrote: On Thu, Feb 23, 2017 at 02:04:50PM +0530, Vivek Gautam wrote: On 02/16/2017 06:36 PM, Roger Quadros wrote: dra7 OTG core limits the host controller to USB2.0 (high-speed) mode when we're operating in dual-role. We work around that by bypassing the

Re: [PATCH] usb: add CONFIG_USB_PCI for system have both PCI HW and non-PCI based USB HW

2017-02-23 Thread kbuild test robot
Hi yuan, [auto build test ERROR on usb/usb-testing] [also build test ERROR on v4.10] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [Kernel] usbhid 3-2:1.1: couldn't find an input interrupt endpoint

2017-02-23 Thread Alan Stern
On Thu, 23 Feb 2017, Cristian wrote: > Hello, > > lsusb -v; https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1667370 Bus 003 Device 003: ID 0458:0186 KYE Systems Corp. (Mouse Systems) Couldn't open device, some information will be missing Device Descriptor: bLength18

[PATCH v2 2/2] usb: gadget: pxa27x: Test for a valid argument pointer

2017-02-23 Thread Petr Cvek
A call usb_put_phy(udc->transceiver) must be tested for a valid pointer. Use an already existing test for usb_unregister_notifier call. Reported-by: Robert Jarzmik Signed-off-by: Petr Cvek --- drivers/usb/gadget/udc/pxa27x_udc.c | 5 +++-- 1 file

[PATCH v2 1/2] usb: gadget: pxa27x: Remove duplicate function prototype

2017-02-23 Thread Petr Cvek
Functions udc_enable() and udc_disable() have a duplicated prototype. Remove it. Signed-off-by: Petr Cvek --- drivers/usb/gadget/udc/pxa27x_udc.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/usb/gadget/udc/pxa27x_udc.c b/drivers/usb/gadget/udc/pxa27x_udc.c

[PATCH v2 0/2] usb: gadget: pxa27x: Duplicated definition and valid pointer fixes

2017-02-23 Thread Petr Cvek
Two simple fixes for pxa27x gadget UDC. First one remove function declaration duplication, second one add test for a valid pointer before function call as it can be called with an error value in the pointer. Changes from v1: * Shorter titles, corrected Robert Jarzmik's signature Petr Cvek (2):

Re: [PATCH v2 4/4] usb: dwc3: Workaround for super-speed host on dra7 in dual-role mode

2017-02-23 Thread Peter Chen
On Thu, Feb 23, 2017 at 02:04:50PM +0530, Vivek Gautam wrote: > > > On 02/16/2017 06:36 PM, Roger Quadros wrote: > >dra7 OTG core limits the host controller to USB2.0 (high-speed) mode > >when we're operating in dual-role. > > > >We work around that by bypassing the OTG core and reading the >

dwc2 gadget issues (was: RK3288 USB OTG controller in peripheral mode)

2017-02-23 Thread Heiko Stuebner
Hi Francesco, Am Donnerstag, 23. Februar 2017, 19:11:37 CET schrieb Francesco Lavra: > I'm having trouble getting the RK3288 OTG controller (the one at > ff58) to work in peripheral mode. I'm using a Firefly Reload board, > and I know the hardware is fine because I can successfully use the

Renesas/NEC 720202 ExpressCard USB 3.0 - HDD unmounted after S3 resume

2017-02-23 Thread Alessio Corbucci
Hi, I have a problem with my new ExpressCard USB 3.0 Renesas 720202: - PROBLEM DESCRIPTION: an external USB 3.0 drive is connected to the Renesas ExpressCard USB 3.0 controller with a single ext4 primary partition on it, mounted, with only a text file opened with geany; when I put the system on

Re: v4.10-rc8 (-rc6) boot regression on Intel desktop, does not boot after cold boots, boots after reboot

2017-02-23 Thread Pavel Machek
On Thu 2017-02-23 17:28:26, Frederic Weisbecker wrote: > On Tue, Feb 14, 2017 at 08:27:43PM +0100, Pavel Machek wrote: > > On Tue 2017-02-14 18:59:56, Pavel Machek wrote: > > > Hi! > > > > > > > > > > Hmm. I moved keyboard between USB ports, and now 4.10-rc6 no > > > > > > > longer > > > > > > >

RE:

2017-02-23 Thread Qin's Yanjun
How are you today and your family? I require your attention and honest co-operation about some issues which i will really want to discuss with you which. Looking forward to read from you soon. Qin's __ Sky Silk, http://aknet.kz -- To unsubscribe from this list:

Re: [Kernel] usbhid 3-2:1.1: couldn't find an input interrupt endpoint

2017-02-23 Thread Cristian
Hello, lsusb -v; https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1667370 Regards, -- Cristian 2017-02-23 12:40 GMT-03:00 Alan Stern : > On Thu, 23 Feb 2017, Greg KH wrote: > >> On Tue, Feb 21, 2017 at 01:56:22PM -0300, Cristian wrote: >> > 2017-02-21 2:04

Re: v4.10-rc8 (-rc6) boot regression on Intel desktop, does not boot after cold boots, boots after reboot

2017-02-23 Thread Frederic Weisbecker
On Tue, Feb 14, 2017 at 08:27:43PM +0100, Pavel Machek wrote: > On Tue 2017-02-14 18:59:56, Pavel Machek wrote: > > Hi! > > > > > > > > Hmm. I moved keyboard between USB ports, and now 4.10-rc6 no longer > > > > > > boots. v4.6 works ok. Let me try with keyboard unplugged... no, I > > > > > >

Re: [RFC PATCH] xhci: Use Cached ring during endpoint ring allocation

2017-02-23 Thread Mathias Nyman
On 22.02.2017 17:05, Anurag Kumar Vulisha wrote: Currently during endpoint initialization, a new endpoint ring is alloacte using xhci_ring_alloc(), if this function fails to allocate ring a cached ring(if available) is assigned to endpoint ring. This patch modifies the code that during endpoint

Re: [Kernel] usbhid 3-2:1.1: couldn't find an input interrupt endpoint

2017-02-23 Thread Alan Stern
On Thu, 23 Feb 2017, Greg KH wrote: > On Tue, Feb 21, 2017 at 01:56:22PM -0300, Cristian wrote: > > 2017-02-21 2:04 GMT-03:00 Greg KH : > > > On Mon, Feb 20, 2017 at 03:28:37PM -0300, Cristian wrote: > > >> Hello, > > >> > > >> dmesg: > > >> [2.465370] usbhid

Re: Advanced Format SAT devices show incorrect physical block size

2017-02-23 Thread Pali Rohár
On Monday 30 January 2017 18:43:12 Pali Rohár wrote: > On Monday 30 January 2017 17:17:03 Alan Stern wrote: > > On Sun, 29 Jan 2017, Pali Rohár wrote: > > > On Wednesday 11 January 2017 16:23:29 Alan Stern wrote: > > > > On Tue, 10 Jan 2017, James Bottomley wrote: > > > > > On Tue, 2017-01-10 at

Re: [Kernel] usbhid 3-2:1.1: couldn't find an input interrupt endpoint

2017-02-23 Thread Greg KH
On Tue, Feb 21, 2017 at 01:56:22PM -0300, Cristian wrote: > 2017-02-21 2:04 GMT-03:00 Greg KH : > > On Mon, Feb 20, 2017 at 03:28:37PM -0300, Cristian wrote: > >> Hello, > >> > >> dmesg: > >> [2.465370] usbhid 3-2:1.1: couldn't find an input interrupt endpoint > > >

Re: [PATCH v2 4/4] usb: dwc3: Workaround for super-speed host on dra7 in dual-role mode

2017-02-23 Thread Vivek Gautam
On 02/16/2017 06:36 PM, Roger Quadros wrote: dra7 OTG core limits the host controller to USB2.0 (high-speed) mode when we're operating in dual-role. We work around that by bypassing the OTG core and reading the extcon framework directly for ID/VBUS events. Signed-off-by: Roger Quadros

[PATCH v2 05/14] usb: dwc2: core: Add hibernated flag

2017-02-23 Thread Vardan Mikayelyan
True if core is hibernated. Signed-off-by: Vardan Mikayelyan Signed-off-by: John Youn --- drivers/usb/dwc2/core.h | 2 ++ drivers/usb/dwc2/platform.c | 1 + 2 files changed, 3 insertions(+) diff --git a/drivers/usb/dwc2/core.h

[PATCH v2 10/14] usb: dwc2: Add dwc2_enter_hibernation(), dwc2_exit_hibernation()

2017-02-23 Thread Vardan Mikayelyan
These are wrapper functions which are calling device or host enter/exit hibernation functions. Signed-off-by: Vardan Mikayelyan Signed-off-by: John Youn --- drivers/usb/dwc2/core.c | 38 ++ drivers/usb/dwc2/core.h

[PATCH v2 09/14] usb: dwc2: Add host/device hibernation functions

2017-02-23 Thread Vardan Mikayelyan
Add host/device hibernation functions which must be wrapped by core's dwc2_enter_hibernation()/dwc2_exit_hibernation() functions. Make dwc2_backup_global_registers dwc2_restore_global_register non-static to use them in both host/gadget sides. Added function names:

[PATCH v2 06/14] usb: dwc2: gadget: Add remote_wakeup_allowed flag

2017-02-23 Thread Vardan Mikayelyan
It will be set once corresponding set_feature command comes. True if device is allowed to wake-up host by remote-wakeup signalling. This is preparation for remote wake-up support implementation, it will not be implemented until gadget stack provide interface for bringing remote wake-up

[PATCH v2 08/14] usb: dwc2: Add helper functions for restore routine

2017-02-23 Thread Vardan Mikayelyan
Add common (host/device) helper functions, which will be called while exiting from hibernation, from both sides. dwc2_restore_essential_regs() dwc2_hib_restore_common() Signed-off-by: Vardan Mikayelyan Signed-off-by: John Youn ---

[PATCH v2 07/14] usb: dwc2: Changes in registers backup/restore functions

2017-02-23 Thread Vardan Mikayelyan
Move hptxfsiz to host register's backup/restore functions, not needed to have it in global register's backup/restore functions. Add backup for glpmcfg, and read/write for gi2cctl and pcgcctl. As requires programming guide. Affected functions: dwc2_backup_host_registers()

[PATCH v2 04/14] usb: dwc2: gadget: Fix dwc2_restore_device_registers

2017-02-23 Thread Vardan Mikayelyan
Add parameter remote_wakeup to dwc2_restore_device_registers() to be able to restore device registers according to programming guide for dwc-otg. It says that in case of rem_wakeup DCTL must not be restored here. Remove setting of DCTL_PWRONPRGDONE from this function, because it will be done in

[PATCH v2 02/14] usb: dwc2: Add hibernation field into dwc2_hw_params

2017-02-23 Thread Vardan Mikayelyan
Add parameter and it's initialization, needed for hibernation. Reimplement dwc2_set_param_power_down() to support hibernation too. Now 'power_down' parameter can be initialized with 0, 1 or 2. 0 - No 1 - Partial power down 2 - Hibernation Signed-off-by: Vardan Mikayelyan

[PATCH v2 03/14] usb: dwc2: gadget: Moved dtxfsiz backup array place

2017-02-23 Thread Vardan Mikayelyan
Moved dtxfsiz from dwc2_gregs_backup to dwc2_dregs_backup, because it is device register. Signed-off-by: Vardan Mikayelyan Signed-off-by: John Youn --- drivers/usb/dwc2/core.c | 8 ++-- drivers/usb/dwc2/core.h | 4 ++--

[PATCH v2 12/14] usb: dwc2: Add dwc2_handle_gpwrdn_intr() handler

2017-02-23 Thread Vardan Mikayelyan
The GPWRDN interrupts are those that occur in both Host and Device mode while core is in hibernated state. Export dwc2_core_init to be able to use it in GPWRDN_IDSTS interrupt handler. Here we have duplicated init functions in host and gadget sides so I have left things as it was(used

[PATCH v2 11/14] usb: dwc2: Allow entering hibernation from USB_SUSPEND interrupt

2017-02-23 Thread Vardan Mikayelyan
Do changes to allow entering hibernated state from USB_SUSPEND interrupt. All code is added under if conditions and mustn't impact existing functionality. Signed-off-by: Vardan Mikayelyan Signed-off-by: John Youn --- drivers/usb/dwc2/core_intr.c |

[PATCH v2 14/14] usb: dwc2: Enable power down

2017-02-23 Thread Vardan Mikayelyan
From: John Youn Enable the power down option based on the core capability. Signed-off-by: John Youn Signed-off-by: Vardan Mikayelyan --- drivers/usb/dwc2/params.c | 16 +++- 1 file changed, 15 insertions(+), 1

[PATCH v2 13/14] usb: dwc2: Change hub-control to allow hibernation

2017-02-23 Thread Vardan Mikayelyan
Affected cases: ClearPortFeature's USB_PORT_FEAT_SUSPEND SetPortFeature's USB_PORT_FEAT_SUSPEND USB_PORT_FEAT_RESET Signed-off-by: Vardan Mikayelyan Signed-off-by: John Youn --- drivers/usb/dwc2/hcd.c | 16 +--- 1 file changed, 13

[PATCH v2 00/14] usb: dwc2: Add hibernation feature support.

2017-02-23 Thread Vardan Mikayelyan
The functions with name hibernation are misnamed originally. They were only responsible for partial power down and not for hibernation. This patch set adds the real hibernation support for dwc2 driver and renames existing functions to power_down.xisting functions to power_down. Changes in v2:

[PATCH v2 01/14] usb: dwc2: Rename hibernation to partial_power_down

2017-02-23 Thread Vardan Mikayelyan
No-op change, only rename. This code was misnamed originally. It was only responsible for partial power down and not for hibernation. Rename core_params->hibernation to core_params->power_down, dwc2_set_param_hibernation() to dwc2_set_param_power_down(). Signed-off-by: Vardan Mikayelyan