DistroKit Mailinglist
 help / color / mirror / Atom feed
From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: distrokit@pengutronix.de
Cc: Ahmad Fatoum <a.fatoum@pengutronix.de>
Subject: [DistroKit] [PATCH 8/8] v8a: barebox: patch in support for BeaglePlay RTL8211F-VD Ethernet PHY
Date: Wed, 21 Jan 2026 09:07:01 +0100	[thread overview]
Message-ID: <20260121080813.4099332-9-a.fatoum@pengutronix.de> (raw)
In-Reply-To: <20260121080813.4099332-1-a.fatoum@pengutronix.de>

The BeaglePlay has a RTL8211F-VD Gigabit Ethernet PHY at mdio0-phy00,
which detects link up, but fails to communicate when used with the
barebox generic PHY driver.

Networking works under Linux as its Realtek PHY driver already matches the
RTL8211F-VD PHY's ID, so sync the barebox driver against it.

Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
 ...-realtek-sync-with-Linux-v6.19-state.patch | 863 ++++++++++++++++++
 .../patches/barebox-2026.01.0/series          |   1 +
 2 files changed, 864 insertions(+)
 create mode 100644 configs/platform-v8a/patches/barebox-2026.01.0/0001-net-phy-realtek-sync-with-Linux-v6.19-state.patch
 create mode 100644 configs/platform-v8a/patches/barebox-2026.01.0/series

diff --git a/configs/platform-v8a/patches/barebox-2026.01.0/0001-net-phy-realtek-sync-with-Linux-v6.19-state.patch b/configs/platform-v8a/patches/barebox-2026.01.0/0001-net-phy-realtek-sync-with-Linux-v6.19-state.patch
new file mode 100644
index 000000000000..b7b1adbbc868
--- /dev/null
+++ b/configs/platform-v8a/patches/barebox-2026.01.0/0001-net-phy-realtek-sync-with-Linux-v6.19-state.patch
@@ -0,0 +1,863 @@
+From cc0a6b72f486dbcfa42d6f4cd9e102e616f093b2 Mon Sep 17 00:00:00 2001
+From: Ahmad Fatoum <a.fatoum@pengutronix.de>
+Date: Tue, 20 Jan 2026 20:34:34 +0100
+Subject: [PATCH] net: phy: realtek: sync with Linux v6.19 state
+
+The BeaglePlay has a RTL8211F-VD Gigabit Ethernet PHY at mdio0-phy00,
+which detects link up, but fails to communicate when used with the
+barebox generic PHY driver.
+
+The Linux Realtek PHY driver already matches the RTL8211F-VD PHY's
+ID, so bring the barebox driver up to date.
+
+Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
+---
+ drivers/net/phy/phy-core.c     |  54 +++++
+ drivers/net/phy/phy.c          |  63 +++++
+ drivers/net/phy/realtek.c      | 411 ++++++++++++++++++++++++++++++++-
+ include/linux/phy.h            |  19 ++
+ include/linux/string_choices.h |  79 +++++++
+ 5 files changed, 614 insertions(+), 12 deletions(-)
+
+diff --git a/drivers/net/phy/phy-core.c b/drivers/net/phy/phy-core.c
+index b12f54ed3878..85e8c4b3e2c7 100644
+--- a/drivers/net/phy/phy-core.c
++++ b/drivers/net/phy/phy-core.c
+@@ -23,6 +23,37 @@ int phy_modify(struct phy_device *phydev, u32 regnum, u16 mask, u16 set)
+ 	return ret < 0 ? ret : 0;
+ }
+ 
++/**
++ * phy_modify_changed - Function for modifying a PHY register
++ * @phydev: the phy_device struct
++ * @regnum: register number to modify
++ * @mask: bit mask of bits to clear
++ * @set: new value of bits set in mask to write to @regnum
++ *
++ * NOTE: MUST NOT be called from interrupt context,
++ * because the bus read/write functions may wait for an interrupt
++ * to conclude the operation.
++ *
++ * Returns negative errno, 0 if there was no change, and 1 in case of change
++ */
++int phy_modify_changed(struct phy_device *phydev, u32 regnum, u16 mask, u16 set)
++{
++	int new, ret;
++
++	ret = phy_read(phydev, regnum);
++	if (ret < 0)
++		return ret;
++
++	new = (ret & ~mask) | set;
++	if (new == ret)
++		return 0;
++
++	ret = phy_write(phydev, regnum, (ret & ~mask) | set);
++
++	return ret < 0 ? ret : 1;
++}
++EXPORT_SYMBOL_GPL(phy_modify_changed);
++
+ static int phy_read_page(struct phy_device *phydev)
+ {
+ 	struct phy_driver *phydrv = to_phy_driver(phydev->dev.driver);
+@@ -184,3 +215,26 @@ int phy_modify_paged(struct phy_device *phydev, int page, u32 regnum,
+ 
+ 	return phy_restore_page(phydev, oldpage, ret);
+ }
++
++/**
++ * phy_modify_paged_changed() - Function for modifying a paged register
++ * @phydev: a pointer to a &struct phy_device
++ * @page: the page for the phy
++ * @regnum: register number
++ * @mask: bit mask of bits to clear
++ * @set: bit mask of bits to set
++ *
++ * Returns negative errno, 0 if there was no change, and 1 in case of change
++ */
++int phy_modify_paged_changed(struct phy_device *phydev, int page, u32 regnum,
++			     u16 mask, u16 set)
++{
++	int ret = 0, oldpage;
++
++	oldpage = phy_select_page(phydev, page);
++	if (oldpage >= 0)
++		ret = phy_modify_changed(phydev, regnum, mask, set);
++
++	return phy_restore_page(phydev, oldpage, ret);
++}
++EXPORT_SYMBOL(phy_modify_paged_changed);
+diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c
+index 9635c816c330..5ca3d9254af8 100644
+--- a/drivers/net/phy/phy.c
++++ b/drivers/net/phy/phy.c
+@@ -1077,6 +1077,69 @@ int genphy_config_init(struct phy_device *phydev)
+ 	return 0;
+ }
+ 
++/**
++ * phy_poll_reset - Safely wait until a PHY reset has properly completed
++ * @phydev: The PHY device to poll
++ *
++ * Description: According to IEEE 802.3, Section 2, Subsection 22.2.4.1.1, as
++ *   published in 2008, a PHY reset may take up to 0.5 seconds.  The MII BMCR
++ *   register must be polled until the BMCR_RESET bit clears.
++ *
++ *   Furthermore, any attempts to write to PHY registers may have no effect
++ *   or even generate MDIO bus errors until this is complete.
++ *
++ *   Some PHYs (such as the Marvell 88E1111) don't entirely conform to the
++ *   standard and do not fully reset after the BMCR_RESET bit is set, and may
++ *   even *REQUIRE* a soft-reset to properly restart autonegotiation.  In an
++ *   effort to support such broken PHYs, this function is separate from the
++ *   standard phy_init_hw() which will zero all the other bits in the BMCR
++ *   and reapply all driver-specific and board-specific fixups.
++ */
++static int phy_poll_reset(struct phy_device *phydev)
++{
++	/* Poll until the reset bit clears (50ms per retry == 0.6 sec) */
++	int ret, val;
++
++	ret = phy_read_poll_timeout(phydev, MII_BMCR, val, !(val & BMCR_RESET),
++				    600000);
++	if (ret)
++		return ret;
++	/* Some chips (smsc911x) may still need up to another 1ms after the
++	 * BMCR_RESET bit is cleared before they are usable.
++	 */
++	mdelay(1);
++	return 0;
++}
++
++int genphy_soft_reset(struct phy_device *phydev)
++{
++	u16 res = BMCR_RESET;
++	int ret;
++
++	if (phydev->autoneg == AUTONEG_ENABLE)
++		res |= BMCR_ANRESTART;
++
++	ret = phy_modify(phydev, MII_BMCR, BMCR_ISOLATE, res);
++	if (ret < 0)
++		return ret;
++
++	/* Clause 22 states that setting bit BMCR_RESET sets control registers
++	 * to their default value. Therefore the POWER DOWN bit is supposed to
++	 * be cleared after soft reset.
++	 */
++
++	ret = phy_poll_reset(phydev);
++	if (ret)
++		return ret;
++
++	/* BMCR may be reset to defaults */
++	if (phydev->autoneg == AUTONEG_DISABLE)
++		ret = genphy_setup_forced(phydev);
++
++	return ret;
++}
++EXPORT_SYMBOL(genphy_soft_reset);
++
+ int phy_driver_register(struct phy_driver *phydrv)
+ {
+ 	phydrv->drv.bus = &mdio_bus_type;
+diff --git a/drivers/net/phy/realtek.c b/drivers/net/phy/realtek.c
+index 8ab3d630912b..77d7b9aaf20b 100644
+--- a/drivers/net/phy/realtek.c
++++ b/drivers/net/phy/realtek.c
+@@ -1,6 +1,6 @@
+-// SPDX-License-Identifier: GPL-2.0-or-later
+-/*
+- * drivers/net/phy/realtek.c
++// SPDX-License-Identifier: GPL-2.0+
++// SPDX-Comment: Origin-URL: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/net/phy/realtek/realtek_main.c?id=4f0638b12451112de4138689fa679315c8d388dc
++/* drivers/net/phy/realtek.c
+  *
+  * Driver for Realtek PHYs
+  *
+@@ -11,6 +11,8 @@
+ #include <common.h>
+ #include <init.h>
+ #include <linux/bitops.h>
++#include <linux/mdio.h>
++#include <linux/string_choices.h>
+ #include <linux/phy.h>
+ 
+ #define RTL821x_PHYSR				0x11
+@@ -20,23 +22,102 @@
+ #define RTL821x_INER				0x12
+ #define RTL8211B_INER_INIT			0x6400
+ #define RTL8211E_INER_LINK_STATUS		BIT(10)
++#define RTL8211F_INER_PME			BIT(7)
+ #define RTL8211F_INER_LINK_STATUS		BIT(4)
+ 
+ #define RTL821x_INSR				0x13
+ 
++#define RTL821x_EXT_PAGE_SELECT			0x1e
++
+ #define RTL821x_PAGE_SELECT			0x1f
+ 
+ #define RTL8211F_INSR				0x1d
+ 
++#define RTL821x_SET_EXT_PAGE			0x07
++
++/* RTL8211E extension page 164/0xa4 */
++#define RTL8211E_RGMII_EXT_PAGE			0xa4
++#define RTL8211E_RGMII_DELAY			0x1c
++#define RTL8211E_CTRL_DELAY			BIT(13)
++#define RTL8211E_TX_DELAY			BIT(12)
++#define RTL8211E_RX_DELAY			BIT(11)
++#define RTL8211E_DELAY_MASK			GENMASK(13, 11)
++
++/* RTL8211F PHY configuration */
++#define RTL8211F_PHYCR_PAGE			0xa43
++#define RTL8211F_PHYCR1				0x18
++#define RTL8211F_ALDPS_PLL_OFF			BIT(1)
++#define RTL8211F_ALDPS_ENABLE			BIT(2)
++#define RTL8211F_ALDPS_XTAL_OFF			BIT(12)
++
++#define RTL8211F_PHYCR2				0x19
++#define RTL8211F_CLKOUT_EN			BIT(0)
++#define RTL8211F_PHYCR2_PHY_EEE_ENABLE		BIT(5)
++
++/* RTL8211F(D)(I)-VD-CG CLKOUT configuration is specified via magic values
++ * to undocumented register pages. The names here do not reflect the datasheet.
++ * Unlike other PHY models, CLKOUT configuration does not go through PHYCR2.
++ */
++#define RTL8211FVD_CLKOUT_PAGE			0xd05
++#define RTL8211FVD_CLKOUT_REG			0x11
++#define RTL8211FVD_CLKOUT_EN			BIT(8)
++
++/* RTL8211F RGMII configuration */
++#define RTL8211F_RGMII_PAGE			0xd08
++
++#define RTL8211F_TXCR				0x11
+ #define RTL8211F_TX_DELAY			BIT(8)
++
++#define RTL8211F_RXCR				0x15
+ #define RTL8211F_RX_DELAY			BIT(3)
+ 
+ #define RTL8201F_ISR				0x1e
+ #define RTL8201F_IER				0x13
+ 
++#define RTL821x_EXT_PAGE_SELECT			0x1e
++
++/* RTL822X_VND2_XXXXX registers are only accessible when phydev->is_c45
++ * is set, they cannot be accessed by C45-over-C22.
++ */
++#define RTL822X_VND2_C22_REG(reg)		(0xa400 + 2 * (reg))
++
++#define RTL8221B_VND2_INER			0xa4d2
++#define RTL8221B_VND2_INER_LINK_STATUS		BIT(4)
++
++#define RTL8221B_VND2_INSR			0xa4d4
++
+ #define RTL8366RB_POWER_SAVE			0x15
+ #define RTL8366RB_POWER_SAVE_ON			BIT(12)
+ 
++#define RTL_VND2_PHYSR				0xa434
++#define RTL_VND2_PHYSR_DUPLEX			BIT(3)
++#define RTL_VND2_PHYSR_SPEEDL			GENMASK(5, 4)
++#define RTL_VND2_PHYSR_SPEEDH			GENMASK(10, 9)
++#define RTL_VND2_PHYSR_MASTER			BIT(11)
++#define RTL_VND2_PHYSR_SPEED_MASK		(RTL_VND2_PHYSR_SPEEDL | RTL_VND2_PHYSR_SPEEDH)
++
++#define	RTL_MDIO_AN_10GBT_STAT			0xa5d6
++
++#define RTL_GENERIC_PHYID			0x001cc800
++#define RTL_8211FVD_PHYID			0x001cc878
++#define RTL_8221B				0x001cc840
++#define RTL_8221B_VB_CG				0x001cc849
++#define RTL_8221B_VM_CG				0x001cc84a
++#define RTL_8251B				0x001cc862
++#define RTL_8261C				0x001cc890
++
++MODULE_DESCRIPTION("Realtek PHY driver");
++MODULE_AUTHOR("Johnson Leung");
++MODULE_LICENSE("GPL");
++
++struct rtl821x_priv {
++	bool enable_aldps;
++	bool disable_clk_out;
++	struct clk *clk;
++	/* rtl8211f */
++	u16 iner;
++};
++
+ static int rtl821x_read_page(struct phy_device *phydev)
+ {
+ 	return phy_read(phydev, RTL821x_PAGE_SELECT);
+@@ -47,6 +128,28 @@ static int rtl821x_write_page(struct phy_device *phydev, int page)
+ 	return phy_write(phydev, RTL821x_PAGE_SELECT, page);
+ }
+ 
++static int rtl821x_probe(struct phy_device *phydev)
++{
++	struct device *dev = &phydev->dev;
++	struct rtl821x_priv *priv;
++
++	priv = xzalloc(sizeof(*priv));
++
++	priv->clk = clk_get_optional_enabled(dev, NULL);
++	if (IS_ERR(priv->clk))
++		return dev_err_probe(dev, PTR_ERR(priv->clk),
++				     "failed to get phy clock\n");
++
++	priv->enable_aldps = of_property_read_bool(dev->of_node,
++						   "realtek,aldps-enable");
++	priv->disable_clk_out = of_property_read_bool(dev->of_node,
++						      "realtek,clkout-disable");
++
++	phydev->priv = priv;
++
++	return 0;
++}
++
+ static int rtl8211_config_aneg(struct phy_device *phydev)
+ {
+ 	int ret;
+@@ -78,9 +181,8 @@ static int rtl8211c_config_init(struct phy_device *phydev)
+ 	return genphy_config_init(phydev);
+ }
+ 
+-static int rtl8211f_config_init(struct phy_device *phydev)
++static int rtl8211f_config_rgmii_delay(struct phy_device *phydev)
+ {
+-	struct device *dev = &phydev->dev;
+ 	u16 val_txdly, val_rxdly;
+ 	int ret;
+ 
+@@ -109,23 +211,169 @@ static int rtl8211f_config_init(struct phy_device *phydev)
+ 		return 0;
+ 	}
+ 
+-	ret = phy_modify_paged(phydev, 0xd08, 0x11, RTL8211F_TX_DELAY,
+-			       val_txdly);
++	ret = phy_modify_paged_changed(phydev, RTL8211F_RGMII_PAGE,
++				       RTL8211F_TXCR, RTL8211F_TX_DELAY,
++				       val_txdly);
+ 	if (ret < 0) {
+-		dev_err(dev, "Failed to update the TX delay register\n");
++		phydev_err(phydev, "Failed to update the TX delay register: %pe\n",
++			   ERR_PTR(ret));
+ 		return ret;
++	} else if (ret) {
++		phydev_dbg(phydev,
++			   "%s 2ns TX delay (and changing the value from pin-strapping RXD1 or the bootloader)\n",
++			   str_enable_disable(val_txdly));
++	} else {
++		phydev_dbg(phydev,
++			   "2ns TX delay was already %s (by pin-strapping RXD1 or bootloader configuration)\n",
++			   str_enabled_disabled(val_txdly));
+ 	}
+ 
+-	ret = phy_modify_paged(phydev, 0xd08, 0x15, RTL8211F_RX_DELAY,
+-			       val_rxdly);
++	ret = phy_modify_paged_changed(phydev, RTL8211F_RGMII_PAGE,
++				       RTL8211F_RXCR, RTL8211F_RX_DELAY,
++				       val_rxdly);
+ 	if (ret < 0) {
+-		dev_err(dev, "Failed to update the RX delay register\n");
++		phydev_err(phydev, "Failed to update the RX delay register: %pe\n",
++			   ERR_PTR(ret));
+ 		return ret;
++	} else if (ret) {
++		phydev_dbg(phydev,
++			   "%s 2ns RX delay (and changing the value from pin-strapping RXD0 or the bootloader)\n",
++			   str_enable_disable(val_rxdly));
++	} else {
++		phydev_dbg(phydev,
++			   "2ns RX delay was already %s (by pin-strapping RXD0 or bootloader configuration)\n",
++			   str_enabled_disabled(val_rxdly));
+ 	}
+ 
+ 	return 0;
+ }
+ 
++static int rtl8211f_config_clk_out(struct phy_device *phydev)
++{
++	struct rtl821x_priv *priv = phydev->priv;
++	struct phy_driver *drv = to_phy_driver(phydev->dev.driver);
++	int ret;
++
++	/* The value is preserved if the device tree property is absent */
++	if (!priv->disable_clk_out)
++		return 0;
++
++	if (drv->phy_id == RTL_8211FVD_PHYID)
++		ret = phy_modify_paged(phydev, RTL8211FVD_CLKOUT_PAGE,
++				       RTL8211FVD_CLKOUT_REG,
++				       RTL8211FVD_CLKOUT_EN, 0);
++	else
++		ret = phy_modify_paged(phydev, RTL8211F_PHYCR_PAGE,
++				       RTL8211F_PHYCR2, RTL8211F_CLKOUT_EN, 0);
++	if (ret)
++		return ret;
++
++	return genphy_soft_reset(phydev);
++}
++
++/* Advance Link Down Power Saving (ALDPS) mode changes crystal/clock behaviour,
++ * which causes the RXC clock signal to stop for tens to hundreds of
++ * milliseconds.
++ *
++ * Some MACs need the RXC clock to support their internal RX logic, so ALDPS is
++ * only enabled based on an opt-in device tree property.
++ */
++static int rtl8211f_config_aldps(struct phy_device *phydev)
++{
++	struct rtl821x_priv *priv = phydev->priv;
++	u16 mask = RTL8211F_ALDPS_PLL_OFF |
++		   RTL8211F_ALDPS_ENABLE |
++		   RTL8211F_ALDPS_XTAL_OFF;
++
++	/* The value is preserved if the device tree property is absent */
++	if (!priv->enable_aldps)
++		return 0;
++
++	return phy_modify_paged(phydev, RTL8211F_PHYCR_PAGE, RTL8211F_PHYCR1,
++				mask, mask);
++}
++
++static int rtl8211f_config_phy_eee(struct phy_device *phydev)
++{
++	/* Disable PHY-mode EEE so LPI is passed to the MAC */
++	return phy_modify_paged(phydev, RTL8211F_PHYCR_PAGE, RTL8211F_PHYCR2,
++				RTL8211F_PHYCR2_PHY_EEE_ENABLE, 0);
++}
++
++static int rtl8211f_config_init(struct phy_device *phydev)
++{
++	struct device *dev = &phydev->dev;
++	int ret;
++
++	ret = rtl8211f_config_aldps(phydev);
++	if (ret) {
++		dev_err(dev, "aldps mode configuration failed: %pe\n",
++			ERR_PTR(ret));
++		return ret;
++	}
++
++	ret = rtl8211f_config_rgmii_delay(phydev);
++	if (ret)
++		return ret;
++
++	ret = rtl8211f_config_clk_out(phydev);
++	if (ret) {
++		dev_err(dev, "clkout configuration failed: %pe\n",
++			ERR_PTR(ret));
++		return ret;
++	}
++
++	return rtl8211f_config_phy_eee(phydev);
++}
++
++static int rtl8211e_config_init(struct phy_device *phydev)
++{
++	int ret = 0, oldpage;
++	u16 val;
++
++	/* enable TX/RX delay for rgmii-* modes, and disable them for rgmii. */
++	switch (phydev->interface) {
++	case PHY_INTERFACE_MODE_RGMII:
++		val = RTL8211E_CTRL_DELAY | 0;
++		break;
++	case PHY_INTERFACE_MODE_RGMII_ID:
++		val = RTL8211E_CTRL_DELAY | RTL8211E_TX_DELAY | RTL8211E_RX_DELAY;
++		break;
++	case PHY_INTERFACE_MODE_RGMII_RXID:
++		val = RTL8211E_CTRL_DELAY | RTL8211E_RX_DELAY;
++		break;
++	case PHY_INTERFACE_MODE_RGMII_TXID:
++		val = RTL8211E_CTRL_DELAY | RTL8211E_TX_DELAY;
++		break;
++	default: /* the rest of the modes imply leaving delays as is. */
++		return 0;
++	}
++
++	/* According to a sample driver there is a 0x1c config register on the
++	 * 0xa4 extension page (0x7) layout. It can be used to disable/enable
++	 * the RX/TX delays otherwise controlled by RXDLY/TXDLY pins.
++	 * The configuration register definition:
++	 * 14 = reserved
++	 * 13 = Force Tx RX Delay controlled by bit12 bit11,
++	 * 12 = RX Delay, 11 = TX Delay
++	 * 10:0 = Test && debug settings reserved by realtek
++	 */
++	oldpage = phy_select_page(phydev, 0x7);
++	if (oldpage < 0)
++		goto err_restore_page;
++
++	ret = phy_write(phydev, RTL821x_EXT_PAGE_SELECT, 0xa4);
++	if (ret)
++		goto err_restore_page;
++
++	ret = phy_modify(phydev, 0x1c, RTL8211E_CTRL_DELAY
++			 | RTL8211E_TX_DELAY | RTL8211E_RX_DELAY,
++			 val);
++
++err_restore_page:
++	return phy_restore_page(phydev, oldpage, ret);
++}
++
+ static int rtl8366rb_config_init(struct phy_device *phydev)
+ {
+ 	int ret;
+@@ -137,17 +385,91 @@ static int rtl8366rb_config_init(struct phy_device *phydev)
+ 	ret = phy_set_bits(phydev, RTL8366RB_POWER_SAVE,
+ 			   RTL8366RB_POWER_SAVE_ON);
+ 	if (ret) {
+-		dev_err(&phydev->dev, "error enabling power management\n");
++		dev_err(&phydev->dev,
++			"error enabling power management\n");
+ 	}
+ 
+ 	return ret;
+ }
+ 
++/* get actual speed to cover the downshift case */
++static void rtlgen_decode_physr(struct phy_device *phydev, int val)
++{
++	/* bit 3
++	 * 0: Half Duplex
++	 * 1: Full Duplex
++	 */
++	if (val & RTL_VND2_PHYSR_DUPLEX)
++		phydev->duplex = DUPLEX_FULL;
++	else
++		phydev->duplex = DUPLEX_HALF;
++
++	switch (val & RTL_VND2_PHYSR_SPEED_MASK) {
++	case 0x0000:
++		phydev->speed = SPEED_10;
++		break;
++	case 0x0010:
++		phydev->speed = SPEED_100;
++		break;
++	case 0x0020:
++		phydev->speed = SPEED_1000;
++		break;
++	case 0x0200:
++		phydev->speed = SPEED_10000;
++		break;
++	case 0x0210:
++		phydev->speed = SPEED_2500;
++		break;
++	default:
++		break;
++	}
++}
++
++static int rtlgen_read_status(struct phy_device *phydev)
++{
++	int ret, val;
++
++	ret = genphy_read_status(phydev);
++	if (ret < 0)
++		return ret;
++
++	if (!phydev->link)
++		return 0;
++
++	val = phy_read_paged(phydev, 0xa43, 0x12);
++	if (val < 0)
++		return val;
++
++	rtlgen_decode_physr(phydev, val);
++
++	return 0;
++}
++
++static int rtl822x_read_status(struct phy_device *phydev)
++{
++	int lpadv, ret;
++
++	ret = rtlgen_read_status(phydev);
++	if (ret < 0)
++		return ret;
++
++	if (phydev->autoneg == AUTONEG_DISABLE)
++		return 0;
++
++	lpadv = phy_read_mmd(phydev, MDIO_MMD_VEND2, RTL_MDIO_AN_10GBT_STAT);
++	if (lpadv < 0)
++		return lpadv;
++
++	return 0;
++}
++
+ static struct phy_driver realtek_drvs[] = {
+ 	{
+ 		PHY_ID_MATCH_EXACT(0x00008201),
+ 		.drv.name	= "RTL8201CP Ethernet",
+ 		.features	= PHY_BASIC_FEATURES,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
+ 	}, {
+ 		PHY_ID_MATCH_EXACT(0x001cc816),
+ 		.drv.name	= "RTL8201F Fast Ethernet",
+@@ -158,6 +480,13 @@ static struct phy_driver realtek_drvs[] = {
+ 		PHY_ID_MATCH_EXACT(0x001cc840),
+ 		.drv.name	= "RTL8226B_RTL8221B 2.5Gbps PHY",
+ 		.features	= PHY_GBIT_FEATURES,
++		.read_status	= rtl822x_read_status,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
++	}, {
++		PHY_ID_MATCH_MODEL(0x001cc880),
++		.drv.name	= "RTL8208 Fast Ethernet",
++		.features	= PHY_BASIC_FEATURES,
+ 		.read_page	= rtl821x_read_page,
+ 		.write_page	= rtl821x_write_page,
+ 	}, {
+@@ -165,28 +494,41 @@ static struct phy_driver realtek_drvs[] = {
+ 		.drv.name	= "RTL8211 Gigabit Ethernet",
+ 		.features	= PHY_GBIT_FEATURES,
+ 		.config_aneg	= rtl8211_config_aneg,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
+ 	}, {
+ 		PHY_ID_MATCH_EXACT(0x001cc912),
+ 		.drv.name	= "RTL8211B Gigabit Ethernet",
+ 		.features	= PHY_GBIT_FEATURES,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
+ 	}, {
+ 		PHY_ID_MATCH_EXACT(0x001cc913),
+ 		.drv.name	= "RTL8211C Gigabit Ethernet",
+ 		.features	= PHY_GBIT_FEATURES,
+ 		.config_init	= rtl8211c_config_init,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
+ 	}, {
+ 		PHY_ID_MATCH_EXACT(0x001cc914),
+ 		.drv.name	= "RTL8211DN Gigabit Ethernet",
+ 		.features	= PHY_GBIT_FEATURES,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
+ 	}, {
+ 		PHY_ID_MATCH_EXACT(0x001cc915),
+ 		.drv.name	= "RTL8211E Gigabit Ethernet",
+ 		.features	= PHY_GBIT_FEATURES,
++		.config_init	= &rtl8211e_config_init,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
+ 	}, {
+ 		PHY_ID_MATCH_EXACT(0x001cc916),
+ 		.drv.name	= "RTL8211F Gigabit Ethernet",
+ 		.features	= PHY_GBIT_FEATURES,
++		.probe		= rtl821x_probe,
+ 		.config_init	= &rtl8211f_config_init,
++		.read_status	= rtlgen_read_status,
+ 		.read_page	= rtl821x_read_page,
+ 		.write_page	= rtl821x_write_page,
+ 	}, {
+@@ -196,11 +538,56 @@ static struct phy_driver realtek_drvs[] = {
+ 		.config_init	= &rtl8211f_config_init,
+ 		.read_page	= rtl821x_read_page,
+ 		.write_page	= rtl821x_write_page,
++	}, {
++		PHY_ID_MATCH_EXACT(RTL_8211FVD_PHYID),
++		.drv.name	= "RTL8211F-VD Gigabit Ethernet",
++		.features	= PHY_GBIT_FEATURES,
++		.probe		= rtl821x_probe,
++		.config_init	= &rtl8211f_config_init,
++		.read_status	= rtlgen_read_status,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
++	}, {
++		PHY_ID_MATCH_EXACT(0x001cc838),
++		.drv.name	= "RTL8226-CG 2.5Gbps PHY",
++		.features	= PHY_GBIT_FEATURES,
++		.read_status	= rtl822x_read_status,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
++	}, {
++		PHY_ID_MATCH_EXACT(0x001cc848),
++		.drv.name	= "RTL8226B-CG_RTL8221B-CG 2.5Gbps PHY",
++		.features	= PHY_GBIT_FEATURES,
++		.read_status	= rtl822x_read_status,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
++	}, {
++		PHY_ID_MATCH_EXACT(0x001cc849),
++		.drv.name	= "RTL8221B-VB-CG 2.5Gbps PHY",
++		.features	= PHY_GBIT_FEATURES,
++		.read_status	= rtl822x_read_status,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
++	}, {
++		PHY_ID_MATCH_EXACT(0x001cc84a),
++		.drv.name	= "RTL8221B-VM-CG 2.5Gbps PHY",
++		.features	= PHY_GBIT_FEATURES,
++		.read_status	= rtl822x_read_status,
++		.read_page	= rtl821x_read_page,
++		.write_page	= rtl821x_write_page,
+ 	}, {
+ 		PHY_ID_MATCH_EXACT(0x001cc961),
+ 		.drv.name	= "RTL8366RB Gigabit Ethernet",
+ 		.features	= PHY_GBIT_FEATURES,
+ 		.config_init	= &rtl8366rb_config_init,
++	}, {
++		PHY_ID_MATCH_EXACT(0x001cc942),
++		.drv.name	= "RTL8365MB-VC Gigabit Ethernet",
++		.features	= PHY_GBIT_FEATURES,
++	}, {
++		PHY_ID_MATCH_EXACT(0x001cc960),
++		.drv.name	= "RTL8366S Gigabit Ethernet",
++		.features	= PHY_GBIT_FEATURES,
+ 	},
+ };
+ 
+diff --git a/include/linux/phy.h b/include/linux/phy.h
+index 04b82d63a51e..26b1136b8a5d 100644
+--- a/include/linux/phy.h
++++ b/include/linux/phy.h
+@@ -15,6 +15,7 @@
+ #include <linux/list.h>
+ #include <linux/ethtool.h>
+ #include <linux/mii.h>
++#include <linux/iopoll.h>
+ 
+ #define PHY_DEFAULT_FEATURES    (SUPPORTED_Autoneg | \
+ 				 SUPPORTED_TP | \
+@@ -452,11 +453,27 @@ int phy_read_paged(struct phy_device *phydev, int page, u32 regnum);
+ int phy_write_paged(struct phy_device *phydev, int page, u32 regnum, u16 val);
+ int phy_modify_paged(struct phy_device *phydev, int page, u32 regnum,
+ 		     u16 mask, u16 set);
++int phy_modify_paged_changed(struct phy_device *phydev, int page, u32 regnum,
++			     u16 mask, u16 set);
+ 
+ struct phy_device *phy_device_create(struct mii_bus *bus, int addr, int phy_id);
+ int phy_register_device(struct phy_device* dev);
+ void phy_unregister_device(struct phy_device *phydev);
+ 
++#define phy_read_poll_timeout(phydev, regnum, val, cond, \
++				timeout_us) \
++({ \
++	int __ret, __val; \
++	__ret = read_poll_timeout(__val = phy_read, val, \
++				  __val < 0 || (cond), \
++		timeout_us, phydev, regnum); \
++	if (__val < 0) \
++		__ret = __val; \
++	if (__ret) \
++		dev_err(&((phydev)->dev), "%s failed: %d\n", __func__, __ret); \
++	__ret; \
++})
++
+ /**
+  * phy_read - Convenience function for reading a given PHY register
+  * @phydev: the phy_device struct
+@@ -479,6 +496,7 @@ static inline int phy_write(struct phy_device *phydev, u32 regnum, u16 val)
+ }
+ 
+ int phy_modify(struct phy_device *phydev, u32 regnum, u16 mask, u16 set);
++int phy_modify_changed(struct phy_device *phydev, u32 regnum, u16 mask, u16 set);
+ 
+ /**
+  * phy_set_bits - Convenience function for setting bits in a PHY register
+@@ -531,6 +549,7 @@ int phy_update_status(struct phy_device *phydev);
+ int phy_wait_aneg_done(struct phy_device *phydev);
+ 
+ /* Generic PHY support and helper functions */
++int genphy_soft_reset(struct phy_device *phydev);
+ int genphy_config_init(struct phy_device *phydev);
+ int genphy_restart_aneg(struct phy_device *phydev);
+ int genphy_config_aneg(struct phy_device *phydev);
+diff --git a/include/linux/string_choices.h b/include/linux/string_choices.h
+index 265573343f54..ee84087d4b26 100644
+--- a/include/linux/string_choices.h
++++ b/include/linux/string_choices.h
+@@ -2,8 +2,87 @@
+ #ifndef _LINUX_STRING_CHOICES_H_
+ #define _LINUX_STRING_CHOICES_H_
+ 
++/*
++ * Here provide a series of helpers in the str_$TRUE_$FALSE format (you can
++ * also expand some helpers as needed), where $TRUE and $FALSE are their
++ * corresponding literal strings. These helpers can be used in the printing
++ * and also in other places where constant strings are required. Using these
++ * helpers offers the following benefits:
++ *  1) Reducing the hardcoding of strings, which makes the code more elegant
++ *     through these simple literal-meaning helpers.
++ *  2) Unifying the output, which prevents the same string from being printed
++ *     in various forms, such as enable/disable, enabled/disabled, en/dis.
++ *  3) Deduping by the linker, which results in a smaller binary file.
++ */
++
+ #include <linux/types.h>
+ 
++static inline const char *str_assert_deassert(bool v)
++{
++	return v ? "assert" : "deassert";
++}
++#define str_deassert_assert(v)		str_assert_deassert(!(v))
++
++static inline const char *str_enable_disable(bool v)
++{
++	return v ? "enable" : "disable";
++}
++#define str_disable_enable(v)		str_enable_disable(!(v))
++
++static inline const char *str_enabled_disabled(bool v)
++{
++	return v ? "enabled" : "disabled";
++}
++#define str_disabled_enabled(v)		str_enabled_disabled(!(v))
++
++static inline const char *str_hi_lo(bool v)
++{
++	return v ? "hi" : "lo";
++}
++#define str_lo_hi(v)		str_hi_lo(!(v))
++
++static inline const char *str_high_low(bool v)
++{
++	return v ? "high" : "low";
++}
++#define str_low_high(v)		str_high_low(!(v))
++
++static inline const char *str_input_output(bool v)
++{
++	return v ? "input" : "output";
++}
++#define str_output_input(v)	str_input_output(!(v))
++
++static inline const char *str_on_off(bool v)
++{
++	return v ? "on" : "off";
++}
++#define str_off_on(v)		str_on_off(!(v))
++
++static inline const char *str_read_write(bool v)
++{
++	return v ? "read" : "write";
++}
++#define str_write_read(v)		str_read_write(!(v))
++
++static inline const char *str_true_false(bool v)
++{
++	return v ? "true" : "false";
++}
++#define str_false_true(v)		str_true_false(!(v))
++
++static inline const char *str_up_down(bool v)
++{
++	return v ? "up" : "down";
++}
++#define str_down_up(v)		str_up_down(!(v))
++
++static inline const char *str_yes_no(bool v)
++{
++	return v ? "yes" : "no";
++}
++#define str_no_yes(v)		str_yes_no(!(v))
++
+ /**
+  * str_plural - Return the simple pluralization based on English counts
+  * @num: Number used for deciding pluralization
+-- 
+2.47.3
+
diff --git a/configs/platform-v8a/patches/barebox-2026.01.0/series b/configs/platform-v8a/patches/barebox-2026.01.0/series
new file mode 100644
index 000000000000..12e7d62e2c0d
--- /dev/null
+++ b/configs/platform-v8a/patches/barebox-2026.01.0/series
@@ -0,0 +1 @@
+0001-net-phy-realtek-sync-with-Linux-v6.19-state.patch
-- 
2.47.3




      parent reply	other threads:[~2026-01-21  8:08 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-21  8:06 [DistroKit] [PATCH 0/8] barebox: update to v2026.01.0 and extend config Ahmad Fatoum
2026-01-21  8:06 ` [DistroKit] [PATCH 1/8] barebox: set CONFIG_NAME across all platforms Ahmad Fatoum
2026-01-21  8:06 ` [DistroKit] [PATCH 2/8] barebox: update barebox v2025.11.0 -> v2026.01.0 Ahmad Fatoum
2026-01-21  8:06 ` [DistroKit] [PATCH 3/8] barebox: enable some useful interactive tools Ahmad Fatoum
2026-01-21  8:06 ` [DistroKit] [PATCH 4/8] v8a: barebox: enable Rockchip power domain driver Ahmad Fatoum
2026-01-21  8:06 ` [DistroKit] [PATCH 5/8] barebox: enable bootloader dm-verity support Ahmad Fatoum
2026-01-21  8:06 ` [DistroKit] [PATCH 6/8] barebox: enable oftree fallback Ahmad Fatoum
2026-01-21  8:07 ` [DistroKit] [PATCH 7/8] barebox: r5: disable CONFIG_DEBUG_PROBES Ahmad Fatoum
2026-01-21  8:07 ` Ahmad Fatoum [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260121080813.4099332-9-a.fatoum@pengutronix.de \
    --to=a.fatoum@pengutronix.de \
    --cc=distrokit@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox