net: stmmac: remove unnecessary .use_prio queue initialisation

Several drivers (see below) explicitly set the queue .use_prio
configuration to false. However, as this structure is allocated using
devm_kzalloc(), all members default to zero unless otherwise explicitly
initialised. .use_prio isn't, so defaults to false. Remove these
unnecessary initialisations, leaving stmmac_platform.c as the only
file that .use_prio is set true.

$ grep 'use_prio =' *.c
dwmac-intel.c: plat->tx_queues_cfg[0].use_prio = false;
dwmac-intel.c: plat->rx_queues_cfg[0].use_prio = false;
dwmac-intel.c: plat->rx_queues_cfg[i].use_prio = false;
dwmac-intel.c: plat->tx_queues_cfg[i].use_prio = false;
dwmac-loongson.c: plat->tx_queues_cfg[0].use_prio = false;
dwmac-loongson.c: plat->rx_queues_cfg[0].use_prio = false;
stmmac_pci.c: plat->tx_queues_cfg[0].use_prio = false;
stmmac_pci.c: plat->rx_queues_cfg[0].use_prio = false;
stmmac_pci.c: plat->tx_queues_cfg[i].use_prio = false;
stmmac_pci.c: plat->rx_queues_cfg[i].use_prio = false;
stmmac_platform.c: plat->rx_queues_cfg[queue].use_prio = false;
stmmac_platform.c: plat->rx_queues_cfg[queue].use_prio = true;
stmmac_platform.c: plat->tx_queues_cfg[queue].use_prio = false;
stmmac_platform.c: plat->tx_queues_cfg[queue].use_prio = true;

Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Link: https://patch.msgid.link/E1vJvjV-0000000EVkC-0WAV@rmk-PC.armlinux.org.uk
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
index 1fd6faa..72f6acd 100644 (file)
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c
@@ -570,10 +570,6 @@ static void common_default_data(struct plat_stmmacenet_data *plat)
 
        plat->mdio_bus_data->needs_reset = true;
 
-       /* Disable Priority config by default */
-       plat->tx_queues_cfg[0].use_prio = false;
-       plat->rx_queues_cfg[0].use_prio = false;
-
        /* Disable RX queues routing by default */
        plat->rx_queues_cfg[0].pkt_route = 0x0;
 }
@@ -619,9 +615,6 @@ static int intel_mgbe_common_data(struct pci_dev *pdev,
        for (i = 0; i < plat->rx_queues_to_use; i++) {
                plat->rx_queues_cfg[i].mode_to_use = MTL_QUEUE_DCB;
 
-               /* Disable Priority config by default */
-               plat->rx_queues_cfg[i].use_prio = false;
-
                /* Disable RX queues routing by default */
                plat->rx_queues_cfg[i].pkt_route = 0x0;
        }
@@ -629,8 +622,6 @@ static int intel_mgbe_common_data(struct pci_dev *pdev,
        for (i = 0; i < plat->tx_queues_to_use; i++) {
                plat->tx_queues_cfg[i].mode_to_use = MTL_QUEUE_DCB;
 
-               /* Disable Priority config by default */
-               plat->tx_queues_cfg[i].use_prio = false;
                /* Default TX Q0 to use TSO and rest TXQ for TBS */
                if (i > 0)
                        plat->tx_queues_cfg[i].tbs_en = 1;

diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-loongson.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-loongson.c
index ed5e9ca..c64a24b 100644 (file)
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-loongson.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-loongson.c
@@ -98,10 +98,6 @@ static void loongson_default_data(struct pci_dev *pdev,
        /* Increase the default value for multicast hash bins */
        plat->multicast_filter_bins = 256;
 
-       /* Disable Priority config by default */
-       plat->tx_queues_cfg[0].use_prio = false;
-       plat->rx_queues_cfg[0].use_prio = false;
-
        /* Disable RX queues routing by default */
        plat->rx_queues_cfg[0].pkt_route = 0x0;
 

diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c
index ded4484..2f45b79 100644 (file)
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_pci.c
@@ -28,10 +28,6 @@ static void common_default_data(struct plat_stmmacenet_data *plat)
 
        plat->mdio_bus_data->needs_reset = true;
 
-       /* Disable Priority config by default */
-       plat->tx_queues_cfg[0].use_prio = false;
-       plat->rx_queues_cfg[0].use_prio = false;
-
        /* Disable RX queues routing by default */
        plat->rx_queues_cfg[0].pkt_route = 0x0;
 }
@@ -74,7 +70,6 @@ static int snps_gmac5_default_data(struct pci_dev *pdev,
 
        plat->tx_sched_algorithm = MTL_TX_ALGORITHM_WRR;
        for (i = 0; i < plat->tx_queues_to_use; i++) {
-               plat->tx_queues_cfg[i].use_prio = false;
                plat->tx_queues_cfg[i].mode_to_use = MTL_QUEUE_DCB;
                plat->tx_queues_cfg[i].weight = 25;
                if (i > 0)
@@ -83,7 +78,6 @@ static int snps_gmac5_default_data(struct pci_dev *pdev,
 
        plat->rx_sched_algorithm = MTL_RX_ALGORITHM_SP;
        for (i = 0; i < plat->rx_queues_to_use; i++) {
-               plat->rx_queues_cfg[i].use_prio = false;
                plat->rx_queues_cfg[i].mode_to_use = MTL_QUEUE_DCB;
                plat->rx_queues_cfg[i].pkt_route = 0x0;
        }

diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
index 7eb2251..4750843 100644 (file)
--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c
@@ -184,7 +184,6 @@ static int stmmac_mtl_setup(struct platform_device *pdev,
                if (of_property_read_u32(q_node, "snps,priority",
                                        &plat->rx_queues_cfg[queue].prio)) {
                        plat->rx_queues_cfg[queue].prio = 0;
-                       plat->rx_queues_cfg[queue].use_prio = false;
                } else {
                        plat->rx_queues_cfg[queue].use_prio = true;
                }
@@ -261,7 +260,6 @@ static int stmmac_mtl_setup(struct platform_device *pdev,
                if (of_property_read_u32(q_node, "snps,priority",
                                        &plat->tx_queues_cfg[queue].prio)) {
                        plat->tx_queues_cfg[queue].prio = 0;
-                       plat->tx_queues_cfg[queue].use_prio = false;
                } else {
                        plat->tx_queues_cfg[queue].use_prio = true;
                }