Hi John, On 9/24/2018 22:52, John Stultz wrote: > On Sun, Sep 23, 2018 at 10:57 PM, Artur Petrosyan > <arthur.petrosyan@xxxxxxxxxxxx> wrote: >> Hi John, >> >> On 9/21/2018 05:05, John Stultz wrote: >>> On Thu, Sep 20, 2018 at 7:17 AM, Artur Petrosyan >>> <Arthur.Petrosyan@xxxxxxxxxxxx> wrote: >>>> On 5/23/2018 01:57, John Stultz wrote: >>>>> Its done automatically, when the OTG cable is detected it the host >>>>> ports are disabled and when the OTG port is empty the host ports are >>>>> enabled. >>>>> >>>>> Let me know if you need anything else! >>>>> >>>> >>>> Please apply the patch set with this cover letter "[PATCH 0/3] usb: >>>> dwc2: Fix hibernation for switching between host and device modes." >>> >>> Sorry, can you send the patches to me, or point me to a git tree? I'm >>> not seeing that thread in my mailbox or on google. >>> >>>> Enable the power down on his devices. Let me know if you still see any >>>> issue. If there is no issue, please provide Tested-by tag. >>> >>> Would be happy to test it, thought I'm traveling tomorrow, so I may >>> not be able to validate till monday. >>> >>> thanks >>> -john >>> >> >> You can find the patch set following to this link. >> >> https://urldefense.proofpoint.com/v2/url?u=https-3A__marc.info_-3Fl-3Dlinux-2Dusb-26m-3D153745139408236-26w-3D2&d=DwIBaQ&c=DPL6_X_6JkXFx7AXWqB0tg&r=9hPBFKCJ_nBjJhGVrrlYOeOQjP_HlVzYqrC_D7niMJI&m=dfaz_WctFBdFgywM2g9w0XVhgOdAS4gIsj9o23RnyvY&s=_Nz3ephAop_nqrfcICii4OgMkV14Mi4yaKk8-qPqdis&e= > > I applied those three patches, and it seems to work ok. > > Just to be clear, was there anything else I was needing to do while testing it? > > Otherwise, > Tested-by: John Stultz <john.stultz@xxxxxxxxxx> #On HiKey > > thanks > -john > Just a clarification by this commit "[PATCH] usb: dwc2: Fix HiKey regression caused by power_down feature" https://marc.info/?l=linux-usb&m=152669095513248&w=2 the power_down is disabled setting "p->power_down = false;" in "dwc2_set_his_params" function. Could you please clarify that the testes done for those 3 patches were done enabling "p->power_down = true;" in "dwc2_set_his_params" function. Regards, Artur
