Re: [PATCH] usb: dwc2: host: Fix remote wakeup when not in DWC2_L2

2015-10-30 Thread Herrero, Gregory
On Fri, Oct 30, 2015 at 03:21:29AM +, John Youn wrote: > On 10/29/2015 9:43 AM, Doug Anderson wrote: > > John, > > > > On Thu, Oct 22, 2015 at 1:05 PM, Douglas Anderson > > wrote: > >> In commit 734643dfbdde ("usb: dwc2: host: add flag to reflect bus > >> state") we changed dwc2_port_suspend

Re: [PATCH] usb: dwc2: host: Fix remote wakeup when not in DWC2_L2

2015-10-29 Thread John Youn
On 10/29/2015 9:43 AM, Doug Anderson wrote: > John, > > On Thu, Oct 22, 2015 at 1:05 PM, Douglas Anderson > wrote: >> In commit 734643dfbdde ("usb: dwc2: host: add flag to reflect bus >> state") we changed dwc2_port_suspend() not to set the lx_state >> anymore (instead it sets the new bus_suspen

Re: [PATCH] usb: dwc2: host: Fix remote wakeup when not in DWC2_L2

2015-10-29 Thread Doug Anderson
Hi, On Thu, Oct 29, 2015 at 3:49 PM, Heiko Stuebner wrote: > Am Donnerstag, 22. Oktober 2015, 13:05:03 schrieb Douglas Anderson: >> In commit 734643dfbdde ("usb: dwc2: host: add flag to reflect bus >> state") we changed dwc2_port_suspend() not to set the lx_state >> anymore (instead it sets the n

Re: [PATCH] usb: dwc2: host: Fix remote wakeup when not in DWC2_L2

2015-10-29 Thread Heiko Stuebner
Am Donnerstag, 22. Oktober 2015, 13:05:03 schrieb Douglas Anderson: > In commit 734643dfbdde ("usb: dwc2: host: add flag to reflect bus > state") we changed dwc2_port_suspend() not to set the lx_state > anymore (instead it sets the new bus_suspended variable). This > introduced a bug where we woul

Re: [PATCH] usb: dwc2: host: Fix remote wakeup when not in DWC2_L2

2015-10-29 Thread Doug Anderson
John, On Thu, Oct 22, 2015 at 1:05 PM, Douglas Anderson wrote: > In commit 734643dfbdde ("usb: dwc2: host: add flag to reflect bus > state") we changed dwc2_port_suspend() not to set the lx_state > anymore (instead it sets the new bus_suspended variable). This > introduced a bug where we would f

[PATCH] usb: dwc2: host: Fix remote wakeup when not in DWC2_L2

2015-10-22 Thread Douglas Anderson
In commit 734643dfbdde ("usb: dwc2: host: add flag to reflect bus state") we changed dwc2_port_suspend() not to set the lx_state anymore (instead it sets the new bus_suspended variable). This introduced a bug where we would fail to detect device insertions if: 1. Plug empty hub into dwc2 2. Plug