On Wed, Nov 18, 2020 at 10:52:51AM -0800, Jakub Kicinski wrote:
> On Mon, 16 Nov 2020 08:43:01 -0800 Ruslan V. Sushko wrote:
> > From: Andrew Lunn <and...@lunn.ch>
> >
> > When the switch is hardware reset, it reads the contents of the
> > EEPROM. This can contain instructions for programming values into
> > registers and to perform waits between such programming. Reading the
> > EEPROM can take longer than the 100ms mv88e6xxx_hardware_reset() waits
> > after deasserting the reset GPIO. So poll the EEPROM done bit to
> > ensure it is complete.
> >
> > Signed-off-by: Andrew Lunn <and...@lunn.ch>
> > Signed-off-by: Ruslan Sushko <r...@sushko.dev>
>
> Andrew, do we need this in net?
I was wondering about that. I actually recommend leaving the EEPROM
empty. The driver has no idea what magic the EEPROM has done, and so
will stomp over it, or make assumptions which are not true about
register values.
But Zodiac has valid use cases of putting stuff into the EEPROM, and
they are aware of the danger. And this patch has got lost at least
once, causing lots of head scratching. So getting it into 5.10 makes
sense for them. I don't think it needs to go further back.
Not sure what Fixes: tag to use.
Andrew
