realtek: eth: simplify mac port control access

The ethernet driver uses an indirection for access to the MAC_PORT_CTRL
register (aka. MAC_L2_PORT_CTRL on RTL93xx). It defines the base address
and adds up the cpu port offset. This is not needed as the driver does
not handle the non-cpu ports. Use direct register access instead and
avoid register confusion by always using the "_L2_" naming.

While we are here:

- Drop the functions and use defines instead
- Add CPU port defines for better readability

Signed-off-by: Markus Stockhausen <markus.stockhausen@gmx.de>
Link: https://github.com/openwrt/openwrt/pull/21691
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>

diff --git a/target/linux/realtek/files-6.12/drivers/net/ethernet/rtl838x_eth.c b/target/linux/realtek/files-6.12/drivers/net/ethernet/rtl838x_eth.c
index 76e3c3d62382cf2d5d50ddebafce8618066e2a39..826dd72eba9dcc3e0e770b864446613af8658395 100644 (file)
--- a/target/linux/realtek/files-6.12/drivers/net/ethernet/rtl838x_eth.c
+++ b/target/linux/realtek/files-6.12/drivers/net/ethernet/rtl838x_eth.c
@@ -447,7 +447,7 @@ static irqreturn_t rteth_93xx_net_irq(int irq, void *dev_id)
 static void rteth_nic_reset(struct rteth_ctrl *ctrl, int reset_mask)
 {
        pr_info("RESETTING CPU_PORT %d\n", ctrl->r->cpu_port);
-       sw_w32_mask(0x3, 0, ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+       sw_w32_mask(0x3, 0, ctrl->r->mac_l2_port_ctrl);
        mdelay(100);
 
        /* Reset NIC (SW_NIC_RST) and queues (SW_Q_RST) */
@@ -549,7 +549,7 @@ static void rtl838x_hw_en_rxtx(struct rteth_ctrl *ctrl)
        sw_w32_mask(0, RX_EN | TX_EN, ctrl->r->dma_if_ctrl);
 
        /* Restart TX/RX to CPU port */
-       sw_w32_mask(0x0, 0x3, ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+       sw_w32_mask(0x0, 0x3, ctrl->r->mac_l2_port_ctrl);
        /* Set Speed, duplex, flow control
         * FORCE_EN | LINK_EN | NWAY_EN | DUP_SEL
         * | SPD_SEL = 0b10 | FORCE_FC_EN | PHY_MASTER_SLV_MANUAL_EN
@@ -558,7 +558,7 @@ static void rtl838x_hw_en_rxtx(struct rteth_ctrl *ctrl)
        sw_w32(0x6192F, ctrl->r->mac_force_mode_ctrl + ctrl->r->cpu_port * 4);
 
        /* Enable CRC checks on CPU-port */
-       sw_w32_mask(0, BIT(3), ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+       sw_w32_mask(0, BIT(3), ctrl->r->mac_l2_port_ctrl);
 }
 
 static void rtl839x_hw_en_rxtx(struct rteth_ctrl *ctrl)
@@ -573,7 +573,7 @@ static void rtl839x_hw_en_rxtx(struct rteth_ctrl *ctrl)
        sw_w32_mask(0, RX_EN | TX_EN, ctrl->r->dma_if_ctrl);
 
        /* Restart TX/RX to CPU port, enable CRC checking */
-       sw_w32_mask(0x0, 0x3 | BIT(3), ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+       sw_w32_mask(0x0, 0x3 | BIT(3), ctrl->r->mac_l2_port_ctrl);
 
        /* CPU port joins Lookup Miss Flooding Portmask */
        /* TODO: The code below should also work for the RTL838x */
@@ -611,7 +611,7 @@ static void rtl93xx_hw_en_rxtx(struct rteth_ctrl *ctrl)
        sw_w32_mask(0, RX_EN_93XX | TX_EN_93XX, ctrl->r->dma_if_ctrl);
 
        /* Restart TX/RX to CPU port, enable CRC checking */
-       sw_w32_mask(0x0, 0x3 | BIT(4), ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+       sw_w32_mask(0x0, 0x3 | BIT(4), ctrl->r->mac_l2_port_ctrl);
 
        if (ctrl->r->family_id == RTL9300_FAMILY_ID)
                sw_w32_mask(0, BIT(ctrl->r->cpu_port), RTL930X_L2_UNKN_UC_FLD_PMSK);
@@ -757,7 +757,7 @@ static void rtl838x_hw_stop(struct rteth_ctrl *ctrl)
        u32 clear_irq = ctrl->r->family_id == RTL8380_FAMILY_ID ? 0x000fffff : 0x007fffff;
 
        /* Disable RX/TX from/to CPU-port */
-       sw_w32_mask(0x3, 0, ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+       sw_w32_mask(0x3, 0, ctrl->r->mac_l2_port_ctrl);
 
        /* Disable traffic */
        if (ctrl->r->family_id == RTL9300_FAMILY_ID || ctrl->r->family_id == RTL9310_FAMILY_ID)
@@ -1272,7 +1272,7 @@ static void rteth_mac_link_down(struct phylink_config *config,
 
        pr_debug("In %s\n", __func__);
        /* Stop TX/RX to port */
-       sw_w32_mask(0x03, 0, ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+       sw_w32_mask(0x03, 0, ctrl->r->mac_l2_port_ctrl);
 }
 
 static void rteth_mac_link_up(struct phylink_config *config,
@@ -1285,7 +1285,7 @@ static void rteth_mac_link_up(struct phylink_config *config,
 
        pr_debug("In %s\n", __func__);
        /* Restart TX/RX to port */
-       sw_w32_mask(0, 0x03, ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+       sw_w32_mask(0, 0x03, ctrl->r->mac_l2_port_ctrl);
 }
 
 static void rteth_set_mac_hw(struct net_device *dev, u8 *mac)
@@ -1418,9 +1418,9 @@ static int rteth_83xx_set_features(struct net_device *dev, netdev_features_t fea
 
        if ((features ^ dev->features) & NETIF_F_RXCSUM) {
                if (!(features & NETIF_F_RXCSUM))
-                       sw_w32_mask(BIT(3), 0, ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+                       sw_w32_mask(BIT(3), 0, ctrl->r->mac_l2_port_ctrl);
                else
-                       sw_w32_mask(0, BIT(3), ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+                       sw_w32_mask(0, BIT(3), ctrl->r->mac_l2_port_ctrl);
        }
 
        return 0;
@@ -1432,9 +1432,9 @@ static int rteth_93xx_set_features(struct net_device *dev, netdev_features_t fea
 
        if ((features ^ dev->features) & NETIF_F_RXCSUM) {
                if (!(features & NETIF_F_RXCSUM))
-                       sw_w32_mask(BIT(4), 0, ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+                       sw_w32_mask(BIT(4), 0, ctrl->r->mac_l2_port_ctrl);
                else
-                       sw_w32_mask(0, BIT(4), ctrl->r->mac_port_ctrl(ctrl->r->cpu_port));
+                       sw_w32_mask(0, BIT(4), ctrl->r->mac_l2_port_ctrl);
        }
 
        return 0;
@@ -1465,9 +1465,9 @@ static const struct net_device_ops rteth_838x_netdev_ops = {
 
 static const struct rteth_config rteth_838x_cfg = {
        .family_id = RTL8380_FAMILY_ID,
-       .cpu_port = 28,
+       .cpu_port = RTETH_838X_CPU_PORT,
        .net_irq = rteth_83xx_net_irq,
-       .mac_port_ctrl = rtl838x_mac_port_ctrl,
+       .mac_l2_port_ctrl = RTETH_838X_MAC_L2_PORT_CTRL,
        .dma_if_intr_sts = RTL838X_DMA_IF_INTR_STS,
        .dma_if_intr_msk = RTL838X_DMA_IF_INTR_MSK,
        .dma_if_ctrl = RTL838X_DMA_IF_CTRL,
@@ -1509,9 +1509,9 @@ static const struct net_device_ops rteth_839x_netdev_ops = {
 
 static const struct rteth_config rteth_839x_cfg = {
        .family_id = RTL8390_FAMILY_ID,
-       .cpu_port = 52,
+       .cpu_port = RTETH_839X_CPU_PORT,
        .net_irq = rteth_83xx_net_irq,
-       .mac_port_ctrl = rtl839x_mac_port_ctrl,
+       .mac_l2_port_ctrl = RTETH_839X_MAC_L2_PORT_CTRL,
        .dma_if_intr_sts = RTL839X_DMA_IF_INTR_STS,
        .dma_if_intr_msk = RTL839X_DMA_IF_INTR_MSK,
        .dma_if_ctrl = RTL839X_DMA_IF_CTRL,
@@ -1553,9 +1553,9 @@ static const struct net_device_ops rteth_930x_netdev_ops = {
 
 static const struct rteth_config rteth_930x_cfg = {
        .family_id = RTL9300_FAMILY_ID,
-       .cpu_port = 28,
+       .cpu_port = RTETH_930X_CPU_PORT,
        .net_irq = rteth_93xx_net_irq,
-       .mac_port_ctrl = rtl930x_mac_port_ctrl,
+       .mac_l2_port_ctrl = RTETH_930X_MAC_L2_PORT_CTRL,
        .dma_if_intr_rx_runout_sts = RTL930X_DMA_IF_INTR_RX_RUNOUT_STS,
        .dma_if_intr_rx_done_sts = RTL930X_DMA_IF_INTR_RX_DONE_STS,
        .dma_if_intr_tx_done_sts = RTL930X_DMA_IF_INTR_TX_DONE_STS,
@@ -1602,9 +1602,9 @@ static const struct net_device_ops rteth_931x_netdev_ops = {
 
 static const struct rteth_config rteth_931x_cfg = {
        .family_id = RTL9310_FAMILY_ID,
-       .cpu_port = 56,
+       .cpu_port = RTETH_931X_CPU_PORT,
        .net_irq = rteth_93xx_net_irq,
-       .mac_port_ctrl = rtl931x_mac_port_ctrl,
+       .mac_l2_port_ctrl = RTETH_931X_MAC_L2_PORT_CTRL,
        .dma_if_intr_rx_runout_sts = RTL931X_DMA_IF_INTR_RX_RUNOUT_STS,
        .dma_if_intr_rx_done_sts = RTL931X_DMA_IF_INTR_RX_DONE_STS,
        .dma_if_intr_tx_done_sts = RTL931X_DMA_IF_INTR_TX_DONE_STS,

diff --git a/target/linux/realtek/files-6.12/drivers/net/ethernet/rtl838x_eth.h b/target/linux/realtek/files-6.12/drivers/net/ethernet/rtl838x_eth.h
index 8bc317ae5b034bad93af26829c8ad796a3929e8a..d289f4c04640e6082fbf1de34095481f828c22e1 100644 (file)
--- a/target/linux/realtek/files-6.12/drivers/net/ethernet/rtl838x_eth.h
+++ b/target/linux/realtek/files-6.12/drivers/net/ethernet/rtl838x_eth.h
@@ -5,6 +5,11 @@
 
 /* Register definition */
 
+#define RTETH_838X_CPU_PORT                    28
+#define RTETH_839X_CPU_PORT                    52
+#define RTETH_930X_CPU_PORT                    28
+#define RTETH_931X_CPU_PORT                    56
+
 /*
  * Reset
  */
@@ -17,13 +22,17 @@
 #define RTL839X_IMR_PORT_LINK_STS_CHG          (0x0068)
 #define RTL839X_ISR_PORT_LINK_STS_CHG          (0x00a0)
 
-/* Per port MAC control */
-#define RTL838X_MAC_PORT_CTRL                  (0xd560)
-#define RTL839X_MAC_PORT_CTRL                  (0x8004)
-#define RTL930X_MAC_L2_PORT_CTRL               (0x3268)
-#define RTL930X_MAC_PORT_CTRL                  (0x3260)
-#define RTL931X_MAC_L2_PORT_CTRL               (0x6000)
-#define RTL931X_MAC_PORT_CTRL                  (0x6004)
+/*
+ * CPU port MAC control. On RTL93XX the functionality of the MAC port control register is
+ * split into MAC_L2_PORT_CTRL and MAC_PORT_CTRL and the L2 register holds the important
+ * bits for the driver. To avoid confusion on splitted models use the L2 naming convention
+ * for all targets.
+ */
+
+#define RTETH_838X_MAC_L2_PORT_CTRL            (0xd560 + (RTETH_838X_CPU_PORT << 7))
+#define RTETH_839X_MAC_L2_PORT_CTRL            (0x8004 + (RTETH_839X_CPU_PORT << 7))
+#define RTETH_930X_MAC_L2_PORT_CTRL            (0x3268 + (RTETH_930X_CPU_PORT << 6))
+#define RTETH_931X_MAC_L2_PORT_CTRL            (0x6000 + (RTETH_931X_CPU_PORT << 7))
 
 /* DMA interrupt control and status registers */
 #define RTL838X_DMA_IF_CTRL                    (0x9f58)
@@ -224,31 +233,6 @@
 /* Default MTU with jumbo frames support */
 #define DEFAULT_MTU 9000
 
-inline int rtl838x_mac_port_ctrl(int p)
-{
-       return RTL838X_MAC_PORT_CTRL + (p << 7);
-}
-
-inline int rtl839x_mac_port_ctrl(int p)
-{
-       return RTL839X_MAC_PORT_CTRL + (p << 7);
-}
-
-/* On the RTL931XX, the functionality of the MAC port control register is split up
- * into RTL931X_MAC_L2_PORT_CTRL and RTL931X_MAC_PORT_CTRL the functionality used
- * by the Ethernet driver is in the same bits now in RTL931X_MAC_L2_PORT_CTRL
- */
-
-inline int rtl930x_mac_port_ctrl(int p)
-{
-       return RTL930X_MAC_L2_PORT_CTRL + (p << 6);
-}
-
-inline int rtl931x_mac_port_ctrl(int p)
-{
-       return RTL931X_MAC_L2_PORT_CTRL + (p << 7);
-}
-
 inline int rtl838x_dma_if_rx_ring_size(int i)
 {
        return RTL838X_DMA_IF_RX_RING_SIZE + ((i >> 3) << 2);
@@ -424,7 +408,7 @@ struct rteth_config {
        int family_id;
        int cpu_port;
        irqreturn_t (*net_irq)(int irq, void *dev_id);
-       int (*mac_port_ctrl)(int port);
+       int mac_l2_port_ctrl;
        int dma_if_intr_sts;
        int dma_if_intr_msk;
        int dma_if_intr_rx_runout_sts;