NVIDIA / edk2-nvidia

NVIDIA EDK2 platform support
Other
78 stars 36 forks source link

fix(eqos): Use correct TX clock name #76

Closed danielfullmer closed 1 year ago

danielfullmer commented 1 year ago

EqosDeviceDxe would fail to set the TX clock rate with the following output:

PhyLinkAdjustEmacConfig: Failed to set clock frequency Not Found

This prevented the network from working at 100Base-T or 10Base-T. (Tested on Xavier NX devkit). The clock name used in the device tree from NVIDIA's sources is "eqos_tx".

Signed-off-by: Daniel Fullmer dfullmer@anduril.com

swatisrik commented 1 year ago

The suggested change makes it specific to Eqos and removes support for other hardware. The change needs to check for which clock compat string is present and use that for programming

jbrasen commented 1 year ago

To clarify we will work on an updated patch and post it that handles both cases (as we have some device trees on platforms with different names) Thank you for pointing this out.