linux-kernel - Re: [RFT PATCH 0/4] usb: dwc2: Fix core reset and force mode delay problems

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <56FCE567.1050004@i2se.com>
Date:	Thu, 31 Mar 2016 10:52:55 +0200
From:	Stefan Wahren <stefan.wahren@...e.com>
To:	John Youn <johnyoun@...opsys.com>, linux-usb@...r.kernel.org,
	Doug Anderson <dianders@...omium.org>
Cc:	Felipe Balbi <balbi@...com>,
	Michael Niewoehner <linux@...ewoehner.de>,
	Tao Huang <huangtao@...k-chips.com>,
	Julius Werner <jwerner@...omium.org>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	linux-kernel@...r.kernel.org,
	Caesar Wang <caesar.upstream@...il.com>,
	Heiko Stuebner <heiko@...ech.de>,
	Felipe Balbi <balbi@...nel.org>,
	Remi Pommarel <repk@...plefau.lt>,
	Kever Yang <kever.yang@...k-chips.com>,
	Przemek Rudy <prudy1@...pl>
Subject: Re: [RFT PATCH 0/4] usb: dwc2: Fix core reset and force mode delay
 problems

Hi John,

Am 29.03.2016 um 04:36 schrieb John Youn:
> Hi,
>
> The following patch series addresses the core reset and force mode
> delay problems we have been seeing on dwc2 for some platforms.
>
> I think I have identified the source of the inconsistencies between
> platforms and this series attempts to address them.
>
> Basically everything stems from the IDDIG debounce filter delay, which
> is a function of the PHY clock speed and can range from 5-50 ms if
> enabled. This delay must be taken into account on core reset and force
> modes. A full explanation is provided in the patch commit log and code
> comments.
>
> The first two patches are prerequisites to the force mode fixes,
> including one patch that was sent separately by Przemek Rudy. I have
> resubmitted it with this series for convenience.
>
> Please help by reviewing and testing on your platforms.
>
> Patches were tested on:
> * Synopsys HAPS platform IP 3.20a OTG, dr_mode=OTG,HOST,PERIPHERAL

i applied the complete patch series on 4.6rc-1 and successfully tested
it with my Raspberry Pi B (IP 2.80a, dr_mode=HOST).

Probing and usual USB devices (network, keyboard) still works fine.

Do you need any other specific tests?

Regards
Stefan


>
> Regards,
> John
>
> John Youn (3):
>   usb: dwc2: gadget: Only initialize device if in device mode
>   usb: dwc2: Add delay to core soft reset
>   usb: dwc2: Properly account for the force mode delays
>
> Przemek Rudy (1):
>   usb: dwc2: do not override forced dr_mode in gadget setup
>
>  drivers/usb/dwc2/core.c     | 195 ++++++++++++++++++++++++++++----------------
>  drivers/usb/dwc2/core.h     |   2 +-
>  drivers/usb/dwc2/gadget.c   |  30 +++++--
>  drivers/usb/dwc2/hcd.c      |   6 +-
>  drivers/usb/dwc2/hw.h       |   1 +
>  drivers/usb/dwc2/platform.c |   9 +-
>  6 files changed, 161 insertions(+), 82 deletions(-)
>