On Sun, Jan 16, 2022 at 23:02, Andrew Lunn <andrew@xxxxxxx> wrote: > On Sun, Jan 16, 2022 at 10:15:26PM +0100, Tobias Waldekranz wrote: >> Once an MDIO read transaction is initiated, we must read back the data >> register within 16 MDC cycles after the transaction completes. Outside >> of this window, reads may return corrupt data. >> >> Therefore, disable local interrupts in the critical section, to >> maximize the probability that we can satisfy this requirement. > > Since this is for net, a Fixes: tag would be nice. Maybe that would be > for the commit which added this driver, or maybe when the DTSI files > for the SOCs which have this errata we added? Alright, I wasn't sure how to tag WAs for errata since it is more about the hardware than the driver. Should I send a v2 even if nothing else pops up, or is this more of a if-you're-sending-a-v2-anyway type of comment?
