Re: [PATCH net 5/6] net: dsa: vsc73xx: allow phy resetting

From: Linus Walleij
Date: Fri Aug 02 2024 - 17:07:44 EST


On Fri, Aug 2, 2024 at 10:04 AM Pawel Dembicki <paweldembicki@xxxxxxxxx> wrote:

> Now, phy reset isn't a problem for vsc73xx switches.
> 'soft_reset' can be done normally.
>
> This commit removes the reset blockade in the 'vsc73xx_phy_write'
> function.
>
> Signed-off-by: Pawel Dembicki <paweldembicki@xxxxxxxxx>

Like Russell says, it needs an explanation.

I think it worked because since the phy write operations were
not properly implemented, the PHY relied on power-on
or firmware defaults before, so things just happened to work
on some systems. We were just lucky things worked if we didn't
reset the PHY.

With some explanation like that:
Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx>

Yours,
Linus Walleij