Re: [PATCH] net: mdiobus: Move all reset registration to `mdiobus_register_reset()`
From: Csókás Bence
Date: Mon Aug 25 2025 - 11:05:27 EST
Hi,
On 2025. 08. 25. 16:54, Russell King (Oracle) wrote:
Now, I could nitpick your "because the PHY was never reset" - that's
untrue. The common problem is the PHY is _held_ in reset mode making
the PHY unresponsive on the MDIO bus.
In our case, the problem is that refclk is turned off during init, and
then before PHY probe, it is turned back on, but reset is not being
asserted.
If your goal is to fix this, then rather than submitting piecemeal
patches with no explanation, I suggest you work on the problem and
come up with a solution as a series of patches (with commit
descriptions that explain _what_ and _why_ changes are being made)
and submit it with a cover message explaining the overall issue
that is being addressed.
That means we can review the patch series as a whole rather than
being drip-fed individual patches, which is going to take a very
long time to make forward progress.
We are still working on the comprehensive solution. I thought that in
the meantime, these small and lower-risk pieces could be reviewed and
picked up.
Bence