On 6/10/20 11:47 AM, Hayes Wang wrote: > Marek Vasut [mailto:ma...@denx.de] >> Sent: Tuesday, June 09, 2020 10:22 PM > [...] >>> - bmcr = BMCR_ANENABLE | BMCR_ANRESTART; >>> + bmcr = BMCR_ANENABLE | BMCR_ANRESTART | BMCR_RESET; >> >> Do you really need to reset a PHY while resetting the autonegotiation ? >> I think if you reset the PHY , then the ANEG operation is restarted by >> default or undefined, no ? > > The setting is from the HW engineer. And I follow > the rule for a long time. I think they know what > they do.
If the hardware is OK with it, then that's fine.