Thread (39 messages) 39 messages, 4 authors, 2025-08-22

Re: [PATCH net-next v10 05/15] net: phy: dp83822: Add support for phy_port representation

From: Maxime Chevallier <maxime.chevallier@bootlin.com>
Date: 2025-08-04 13:51:31
Also in: linux-arm-msm, linux-devicetree, lkml, netdev

On Sat, 26 Jul 2025 22:50:08 +0200
Andrew Lunn [off-list ref] wrote:
quoted
+#if IS_ENABLED(CONFIG_OF_MDIO)
+		if (dp83822->fx_enabled && dp83822->fx_sd_enable)
+			dp83822->fx_signal_det_low =
+				device_property_present(&phydev->mdio.dev,
+							"ti,link-loss-low");
+
+		/* ti,fiber-mode is still used for backwards compatibility, but
+		 * has been replaced with the mdi node definition, see
+		 * ethernet-port.yaml
+		 */
+		if (!dp83822->fx_enabled)
+			dp83822->fx_enabled =
+				device_property_present(&phydev->mdio.dev,
+							"ti,fiber-mode");  
Could be my grep fu is broken but:

~/linux$ grep -r fiber-mode arch/*
~/linux$ 

So it does not even appear to be used. If it is not used, do we have
to consider backwards compatibility?

Maybe consider marking the property deprecated and point to the new
binding?
I'd love that :)

Let's mark it as deprecated then, I'll do that for the next iteration.

Maxime
	Andrew
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help