RE: [EXT] Re: [PATCH v2 2/3] usb: chipidea: imx: support disabling runtime-pm

Xu Yang
2023-07-18T08:31:56Z

> -----Original Message-----
>
> Ciao Francesco,
>
> On Thu, 6 Jul 2023 12:23:43 +0200
> Francesco Dolcini <francesco@dolcini.it> wrote:
>
> > Hello Luca,
> >
> > On Tue, May 30, 2023 at 11:22:51AM +0000, Jun Li wrote:
> > > Yes, your understanding is correct, talked with Xu(in CC), he will take this
> > > soon.
> >
> > A series was posted
> >
> > I had no time to try or look at it yet.
>
> Thanks for keeping me up to date on this topic, which is still totally
> relevant to me.
>
> I looked at the series, but it does not seem to be addressing the
> problem with USB host not detecting new devices when VBUS is not
> directly connected, e.g. in the Colibri imx6ull SoM.
>
> Xu, do you confirm the series at the link is _not_ solving the problem
> being discussed here?

Have you tried this patchset? The upstream driver couldn't get correct
USB role from HW_USBPHY_CTRL register when the ID pin is float. This is
what this patchset is trying to fix. With this patch, condition
"(!vbus_is_on && !mxs_phy_is_otg_host(mxs_phy)" will always be false when
controller acts as host role, then __mxs_phy_disconnect_line(phy, true)
will never be called. So I think it doesn't matter whether VBUS is connected
or not when act as host mode. If you still have issue after apply this patchset,
please let me know.

Thanks,
Xu Yang

>
> Best regards,
>
> Luca
>