On Wed, Jul 24, 2019 at 11:04:41PM +0200, Wolfram Sang wrote: > > > > What about adding a short comment explaining the situation? > > > > Do you mean in the DT itself? > > Yes, so we don't forget why it is there. Or do you think this is not > needed? I'd think it is useful but won't insist. Sure, how about something like this: /* TX clock internal delay mode is required for reliable * 1Gbps communication using the KSZ9031RNX phy present on * the Ebisu board, however, TX clock internal delay mode * isn't supported on r8a77990. Thus, limit speed to * 100Mbps for reliable communication. */ max-speed = <100>;