Re: [PATCH net-next v4 0/4] net: mdio: support dynamic OF device changes

From
James Hilliard <james.hilliard1@gmail.com>
Date
2026-08-07 16:41:18
Also in
imx, linux-omap, linux-pm, linux-usb, lkml, netdev
Reply to
Andrew Lunn, 2026-08-07
On Fri, Aug 7, 2026 at 9:10 AM Andrew Lunn [off-list ref] wrote:
On Fri, Aug 07, 2026 at 08:59:47AM -0600, James Hilliard wrote:
quoted
On Fri, Aug 7, 2026 at 7:16 AM Andrew Lunn [off-list ref] wrote:
quoted
On Thu, Aug 06, 2026 at 09:54:19PM -0600, James Hilliard wrote:
quoted
MDIO buses enumerate firmware children only when the bus is registered. A
later devicetree overlay or status transition is ignored, unlike on I2C and
SPI buses. Supporting live reconfiguration also means that population and
removal can race address scanning, PHY attachment and bus teardown.
Please make the bootloader work around the issues, and drop this
patchset.
This is just fixing OF_DYNAMIC for mdio now, it's no longer used
by the AC200/AC300 PHY patches anymore.
Meaning it has no users?
I think OF_OVERLAY depends on OF_DYNAMIC?

I guess it can also be used like this for live dt modification:
https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git/tree/Documentation/devicetree/configfs-overlays.txt?h=topic/overlays
We don't add code unless it has a user. To do so pointlessly adds
Maintenance burden.
AFAIU the patches other than the last one are still relevant regardless
since they are fixing a bunch of generic mdio safety issues in code that
has users since MDIO maps already change at runtime through non-OF
paths(i.e. SFP insertion/removal).
   Andrew
Thread (16 messages) 16 messages, 3 authors, last reply 2026-08-08
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help