[openwrt/openwrt] realtek: pcs: remove auto-negotiation helper functions
LEDE Commits
lede-commits at lists.infradead.org
Wed Feb 11 15:34:16 PST 2026
hauke pushed a commit to openwrt/openwrt.git, branch main:
https://git.openwrt.org/d1d3cff5b337cfd96e66fa4b093c335c220f2055
commit d1d3cff5b337cfd96e66fa4b093c335c220f2055
Author: Jan Hoffmann <jan at 3e8.eu>
AuthorDate: Mon Feb 9 00:48:46 2026 +0100
realtek: pcs: remove auto-negotiation helper functions
There is now an implementation of .set_autoneg and .restart_autoneg for
all variants. Remove the helper function which checks for it, and just
call the operation directly.
Signed-off-by: Jan Hoffmann <jan at 3e8.eu>
Link: https://github.com/openwrt/openwrt/pull/21934
Signed-off-by: Hauke Mehrtens <hauke at hauke-m.de>
---
.../files-6.12/drivers/net/pcs/pcs-rtl-otto.c | 27 ++--------------------
1 file changed, 2 insertions(+), 25 deletions(-)
diff --git a/target/linux/realtek/files-6.12/drivers/net/pcs/pcs-rtl-otto.c b/target/linux/realtek/files-6.12/drivers/net/pcs/pcs-rtl-otto.c
index 72379c1d7f..47980913f3 100644
--- a/target/linux/realtek/files-6.12/drivers/net/pcs/pcs-rtl-otto.c
+++ b/target/linux/realtek/files-6.12/drivers/net/pcs/pcs-rtl-otto.c
@@ -3689,29 +3689,6 @@ static int rtpcs_931x_setup_serdes(struct rtpcs_serdes *sds,
/* Common functions */
-static int rtpcs_sds_set_autoneg(struct rtpcs_serdes *sds, unsigned int neg_mode,
- const unsigned long *advertising)
-{
- if (!sds->ops->set_autoneg) {
- dev_warn(sds->ctrl->dev, "set_autoneg not implemented for SDS %u, skipping\n",
- sds->id);
- return 0;
- }
-
- return sds->ops->set_autoneg(sds, neg_mode, advertising);
-}
-
-static void rtpcs_sds_restart_autoneg(struct rtpcs_serdes *sds)
-{
- if (!sds->ops->restart_autoneg) {
- dev_warn(sds->ctrl->dev, "restart_autoneg not implemented for SDS %u, skipping\n",
- sds->id);
- return;
- }
-
- sds->ops->restart_autoneg(sds);
-}
-
static void rtpcs_pcs_get_state(struct phylink_pcs *pcs, struct phylink_link_state *state)
{
struct rtpcs_link *link = rtpcs_phylink_pcs_to_link(pcs);
@@ -3780,7 +3757,7 @@ static void rtpcs_pcs_an_restart(struct phylink_pcs *pcs)
struct rtpcs_serdes *sds = link->sds;
mutex_lock(&ctrl->lock);
- rtpcs_sds_restart_autoneg(sds);
+ sds->ops->restart_autoneg(sds);
mutex_unlock(&ctrl->lock);
}
@@ -3815,7 +3792,7 @@ static int rtpcs_pcs_config(struct phylink_pcs *pcs, unsigned int neg_mode,
sds->id, phy_modes(interface));
}
- ret = rtpcs_sds_set_autoneg(sds, neg_mode, advertising);
+ ret = sds->ops->set_autoneg(sds, neg_mode, advertising);
out:
mutex_unlock(&ctrl->lock);
More information about the lede-commits
mailing list