Hi Vladimir!

On Fri, 2024-10-04 at 14:57 +0300, Vladimir Oltean wrote:

On Fri, Oct 04, 2024 at 01:36:54PM +0200, A. Sverdlin wrote:

...

From: Anatolij Gustschin agust@denx.de

Accessing device registers seems to be not reliable, the chip revision is sometimes detected wrongly (0 instead of expected 1).

Ensure that the chip reset is performed via reset GPIO and then wait for 'Device Ready' status in HW_CFG register before doing any register initializations.

Signed-off-by: Anatolij Gustschin agust@denx.de [alex: reworked using read_poll_timeout()] Signed-off-by: Alexander Sverdlin alexander.sverdlin@siemens.com

---

Reviewed-by: Vladimir Oltean olteanv@gmail.com

I see you're missing a Fixes: tag (meaning in this case: backport down to all stable tree containing this commit). I think you can just post it as a reply to this email, without resending, and it should get picked up by maintainers through the same mechanism as Reviewed-by: tags.

Well, the only meaningful would be the very first commit for lan9303:

Fixes: a1292595e006 ("net: dsa: add new DSA switch driver for the SMSC-LAN9303") Cc: stable@vger.kernel.org