[PATCH v2] usb: ehci-mx6: Add i.MX95 OTG support

[Tim Harvey]

On Wed, Jul 23, 2025 at 7:34 PM Peng Fan <peng.fan at oss.nxp.com> wrote:
>
> Hi Tim,
>
> On Mon, Jul 21, 2025 at 11:02:05AM -0700, Tim Harvey wrote:
> >When the usb node is defined dr_mode="otg" ehci_usb_phy_mode() is called
> >to determine the mode from status registers.
> >
> >The IMX95RM does not currently define the USBNC STATUS register but it is
> >assumed to be an omission as the first three registers are defined.
> >It has been expirimentally verified that the USBNC_PHY_STATUS register
> >at offset 0x23C bit4 (USBNC_PHYSTATUS_ID_DIG) reads 0 when USB_ID is GND
> >and 1 when floating.
> >
> >Use is_imx9() as this driver works for i.MX91, i.MX93 and i.MX95 and all
> >of these determine the role based on the USBNC_PHY_STATUS register.
>
> Just a question, does your board use USB2.0 port for SDP? And any public
> patches ready for that?
>

Hi Peng,

Yes, I am using the USB2.0 interface for SDP. Aside from this patch
the usb2 nodes need to be added to the dt (usbphynop, usb2, usbmisc
which are already in kernel but not yet in uboot dts/upstream).

However, unlike SDP on the USB3.0 interface which I was able to get
working, I was getting stuck somewhere between the ATF and U-Boot
proper on USB2.0. I can no longer boot software on my imx95_19x19_evk
after the changes merged for B0 so I'm unable to test and compare
notes at the moment.

Best Regards,

Tim


> >
> >Fixes: 801b5fafd35d "(usb: ehci-mx6: Add i.MX95 support")
> >Signed-off-by: Tim Harvey <tharvey at gateworks.com>
>
> LGTM:
>
> Reviewed-by: Peng Fan <peng.fan at nxp.com>