[PATCH V2 24/24] ARM: imx8m: verdin-imx8mm: Enable USB Host support

[Marek Vasut]

On 4/20/21 2:33 AM, Tim Harvey wrote:

[...]

>> +/* USB Configs */
>> +#define CONFIG_EHCI_HCD_INIT_AFTER_RESET
>> +#define CONFIG_MXC_USB_PORTSC  (PORT_PTS_UTMI | PORT_PTS_PTW)
>> +#define CONFIG_USB_MAX_CONTROLLER_COUNT 2
>> +
>>   #endif /*_VERDIN_IMX8MM_H */
>>
>> --
>> 2.30.2
>>
> 
> Marek,
> 
> Thanks for your work on USB support for IMX8M!
> 
> I'm attempting to add USB support to the venice board following this
> example but I think there are still some things missing from the dt to
> make this work. I find that mx6_parse_dt_adds failes; Looks like it is
> required to have an alias that points to the phy but then it fails
> because the phy doesn't have a reg. Also, it would see the
> CONFIG_MXC_USB_PORTSC is no longer needed as that is now the default.

Have a look at this patch:

https://source.denx.de/u-boot/custodians/u-boot-usb/-/commit/8203ee09275c30766a1fd1cf19c60a904ee72f8c

That should fix it for you.

It would be good however if you could take the mx6 ventana and test the 
driver there, and possibly submit fixes in case the USB is still broken 
there. What would be even better is if you could implement the MXS PHY 
driver, so all the !CONFIG_PHY stuff can be removed for MX6 too. That 
shouldn't be much effort.