Hi,Yes, maybe a quirk is more proper. As you mentioned,
William Wu <william.wu@xxxxxxxxxxxxxx> writes:
Add snps,phyif_utmi_16_bits devicetree property. USB2 phythis needs a quirk_ prefix...
interface is hardware property, and it's platform dependent,right, that's fine. But also note on section 8.1.1 Table 8-1 where it
so we need to configure it in devicetree to set the core to
support a UTMI+ PHY with an 8- or 16-bit interface.
And refer to the dwc3 databook, the GUSB2PHYCFG.USBTRDTIM
must set to the corresponding value according to the usb2
phy interface.
states:
|-------------+------------------------------------------------------------|
| GUSB2PHYCFG | Program the following PHY configuration fields: USBTrdTim, |
| | FSIntf, PHYIf, TOUTCal, or leave the default values if |
| | the correct power-on values were selected during |
| | coreConsultant configuration. Note: The PHY must not |
| | be enabled for auto-resume in device mode. Hence the |
| | field GUSB2PHYCFG[15] (ULPIAutoRes) must be written |
| | with '0' during the power-on initialization in case |
| | the reset value is '1'. |
| | |
|-------------+------------------------------------------------------------|
You only need this because your core was badly configured in
coreConsultant.