]> git.ipfire.org Git - thirdparty/openwrt.git/commitdiff
realtek: eth: remove fixed dsa coupling
authorMarkus Stockhausen <markus.stockhausen@gmx.de>
Fri, 20 Feb 2026 19:48:54 +0000 (20:48 +0100)
committerRobert Marko <robimarko@gmail.com>
Mon, 23 Feb 2026 21:03:02 +0000 (22:03 +0100)
The ethernet driver has a hard link to the dsa driver. Especially
the setup_tc() function must be available when the driver loads.
Decouple it by using a dsa callback recipe.

Suggested-by: Balázs Triszka <info@balika011.hu>
Signed-off-by: Markus Stockhausen <markus.stockhausen@gmx.de>
Link: https://github.com/openwrt/openwrt/pull/22121
Signed-off-by: Robert Marko <robimarko@gmail.com>
target/linux/realtek/files-6.12/drivers/net/ethernet/rtl838x_eth.c

index 4dfe1452eb7691f50541ddaffcd8c960eb0f1453..6b83163fb4a32c4038e39ab2b52074e48802d7fd 100644 (file)
@@ -24,8 +24,6 @@
 #include <asm/mach-rtl-otto/mach-rtl-otto.h>
 #include "rtl838x_eth.h"
 
-int rtl83xx_setup_tc(struct net_device *dev, enum tc_setup_type type, void *type_data);
-
 #define RTETH_OWN_CPU          1
 #define RTETH_RX_RING_SIZE     128
 #define RTETH_RX_RINGS         2
@@ -1366,6 +1364,23 @@ static struct phylink_pcs *rteth_mac_select_pcs(struct phylink_config *config,
        return &ctrl->pcs;
 }
 
+static int rteth_setup_tc(struct net_device *dev, enum tc_setup_type type, void *type_data)
+{
+    struct dsa_switch *ds;
+    struct dsa_port *dp;
+
+    if (!netdev_uses_dsa(dev))
+        return -EOPNOTSUPP;
+
+    dp = dev->dsa_ptr;
+    ds = dp->ds;
+
+    if (!ds->ops->port_setup_tc)
+        return -EOPNOTSUPP;
+
+    return ds->ops->port_setup_tc(ds, dp->index, type, type_data);
+}
+
 static const struct net_device_ops rteth_838x_netdev_ops = {
        .ndo_open = rteth_open,
        .ndo_stop = rteth_stop,
@@ -1376,7 +1391,7 @@ static const struct net_device_ops rteth_838x_netdev_ops = {
        .ndo_tx_timeout = rteth_tx_timeout,
        .ndo_set_features = rteth_83xx_set_features,
        .ndo_fix_features = rteth_fix_features,
-       .ndo_setup_tc = rtl83xx_setup_tc,
+       .ndo_setup_tc = rteth_setup_tc,
 };
 
 static const struct rteth_config rteth_838x_cfg = {
@@ -1423,7 +1438,7 @@ static const struct net_device_ops rteth_839x_netdev_ops = {
        .ndo_tx_timeout = rteth_tx_timeout,
        .ndo_set_features = rteth_83xx_set_features,
        .ndo_fix_features = rteth_fix_features,
-       .ndo_setup_tc = rtl83xx_setup_tc,
+       .ndo_setup_tc = rteth_setup_tc,
 };
 
 static const struct rteth_config rteth_839x_cfg = {
@@ -1470,7 +1485,7 @@ static const struct net_device_ops rteth_930x_netdev_ops = {
        .ndo_tx_timeout = rteth_tx_timeout,
        .ndo_set_features = rteth_93xx_set_features,
        .ndo_fix_features = rteth_fix_features,
-       .ndo_setup_tc = rtl83xx_setup_tc,
+       .ndo_setup_tc = rteth_setup_tc,
 };
 
 static const struct rteth_config rteth_930x_cfg = {
@@ -1520,6 +1535,7 @@ static const struct net_device_ops rteth_931x_netdev_ops = {
        .ndo_tx_timeout = rteth_tx_timeout,
        .ndo_set_features = rteth_93xx_set_features,
        .ndo_fix_features = rteth_fix_features,
+       .ndo_setup_tc = rteth_setup_tc,
 };
 
 static const struct rteth_config rteth_931x_cfg = {