Re: [PATCH net-next v2 5/5] net: dsa: microchip: check erratum workaround through indirect register read

[Oleksij Rempel <o.rempel@pengutronix.de>]

Hi Pieter,

On Wed, Jul 31, 2024 at 12:34:03PM +0200, vtpieter@gmail.com wrote:
> From: Pieter Van Trappen <pieter.van.trappen@cern.ch>
> 
> Check the erratum workaround application which ensures in addition
> that indirect register write and read work as expected.

> Commit b7fb7729c94f ("net: dsa: microchip: fix register write order in
> ksz8_ind_write8()") would have been found faster like this.

No. It would be not found faster, because there are nearly no active
users/develpers participating in the development/testing of this part of
the driver. I still do not have access to any KSZ879x variant and the
initial erratum fix was done on request without ability to actually
test it on my side. You seems to be the first one for long time making
something for this series :)

The strategy to re-read configs after writing them is also not
consequent in the driver and would not really help here too. It will
show you an error only if the driver is accidentally writing to a
read-only register.

> Also fix the register naming as in the datasheet.
> 
> Signed-off-by: Pieter Van Trappen <pieter.van.trappen@cern.ch>

Nacked-by: Oleksij Rempel <o.rempel@pengutronix.de>

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |