Commit 68861a3b authored by Bartosz Golaszewski's avatar Bartosz Golaszewski Committed by Jakub Kicinski

net: stmmac: replace the tso_en field with a flag

Drop the boolean field of the plat_stmmacenet_data structure in favor of a
simple bitfield flag.
Signed-off-by: default avatarBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Reviewed-by: default avatarAndrew Halaney <ahalaney@redhat.com>
Link: https://lore.kernel.org/r/20230710090001.303225-6-brgl@bgdev.plReviewed-by: default avatarSimon Horman <simon.horman@corigine.com>
Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent d8daff28
...@@ -113,7 +113,7 @@ static int dwc_eth_dwmac_config_dt(struct platform_device *pdev, ...@@ -113,7 +113,7 @@ static int dwc_eth_dwmac_config_dt(struct platform_device *pdev,
/* dwc-qos needs GMAC4, AAL, TSO and PMT */ /* dwc-qos needs GMAC4, AAL, TSO and PMT */
plat_dat->has_gmac4 = 1; plat_dat->has_gmac4 = 1;
plat_dat->dma_cfg->aal = 1; plat_dat->dma_cfg->aal = 1;
plat_dat->tso_en = 1; plat_dat->flags |= STMMAC_FLAG_TSO_EN;
plat_dat->pmt = 1; plat_dat->pmt = 1;
return 0; return 0;
......
...@@ -458,8 +458,7 @@ static int intel_mgbe_common_data(struct pci_dev *pdev, ...@@ -458,8 +458,7 @@ static int intel_mgbe_common_data(struct pci_dev *pdev,
plat->has_gmac = 0; plat->has_gmac = 0;
plat->has_gmac4 = 1; plat->has_gmac4 = 1;
plat->force_sf_dma_mode = 0; plat->force_sf_dma_mode = 0;
plat->tso_en = 1; plat->flags |= (STMMAC_FLAG_TSO_EN | STMMAC_FLAG_SPH_DISABLE);
plat->flags |= STMMAC_FLAG_SPH_DISABLE;
/* Multiplying factor to the clk_eee_i clock time /* Multiplying factor to the clk_eee_i clock time
* period to make it closer to 100 ns. This value * period to make it closer to 100 ns. This value
......
...@@ -783,7 +783,8 @@ static int qcom_ethqos_probe(struct platform_device *pdev) ...@@ -783,7 +783,8 @@ static int qcom_ethqos_probe(struct platform_device *pdev)
if (ethqos->has_emac_ge_3) if (ethqos->has_emac_ge_3)
plat_dat->dwmac4_addrs = &data->dwmac4_addrs; plat_dat->dwmac4_addrs = &data->dwmac4_addrs;
plat_dat->pmt = 1; plat_dat->pmt = 1;
plat_dat->tso_en = of_property_read_bool(np, "snps,tso"); if (of_property_read_bool(np, "snps,tso"))
plat_dat->flags |= STMMAC_FLAG_TSO_EN;
if (of_device_is_compatible(np, "qcom,qcs404-ethqos")) if (of_device_is_compatible(np, "qcom,qcs404-ethqos"))
plat_dat->rx_clk_runs_in_lpi = 1; plat_dat->rx_clk_runs_in_lpi = 1;
if (data->has_integrated_pcs) if (data->has_integrated_pcs)
......
...@@ -290,7 +290,7 @@ static int tegra_mgbe_probe(struct platform_device *pdev) ...@@ -290,7 +290,7 @@ static int tegra_mgbe_probe(struct platform_device *pdev)
} }
plat->has_xgmac = 1; plat->has_xgmac = 1;
plat->tso_en = 1; plat->flags |= STMMAC_FLAG_TSO_EN;
plat->pmt = 1; plat->pmt = 1;
plat->bsp_priv = mgbe; plat->bsp_priv = mgbe;
......
...@@ -5677,7 +5677,7 @@ static netdev_features_t stmmac_fix_features(struct net_device *dev, ...@@ -5677,7 +5677,7 @@ static netdev_features_t stmmac_fix_features(struct net_device *dev,
features &= ~NETIF_F_CSUM_MASK; features &= ~NETIF_F_CSUM_MASK;
/* Disable tso if asked by ethtool */ /* Disable tso if asked by ethtool */
if ((priv->plat->tso_en) && (priv->dma_cap.tsoen)) { if ((priv->plat->flags & STMMAC_FLAG_TSO_EN) && (priv->dma_cap.tsoen)) {
if (features & NETIF_F_TSO) if (features & NETIF_F_TSO)
priv->tso = true; priv->tso = true;
else else
...@@ -7250,7 +7250,7 @@ int stmmac_dvr_probe(struct device *device, ...@@ -7250,7 +7250,7 @@ int stmmac_dvr_probe(struct device *device,
ndev->hw_features |= NETIF_F_HW_TC; ndev->hw_features |= NETIF_F_HW_TC;
} }
if ((priv->plat->tso_en) && (priv->dma_cap.tsoen)) { if ((priv->plat->flags & STMMAC_FLAG_TSO_EN) && (priv->dma_cap.tsoen)) {
ndev->hw_features |= NETIF_F_TSO | NETIF_F_TSO6; ndev->hw_features |= NETIF_F_TSO | NETIF_F_TSO6;
if (priv->plat->has_gmac4) if (priv->plat->has_gmac4)
ndev->hw_features |= NETIF_F_GSO_UDP_L4; ndev->hw_features |= NETIF_F_GSO_UDP_L4;
......
...@@ -77,7 +77,7 @@ static int snps_gmac5_default_data(struct pci_dev *pdev, ...@@ -77,7 +77,7 @@ static int snps_gmac5_default_data(struct pci_dev *pdev,
plat->clk_csr = 5; plat->clk_csr = 5;
plat->has_gmac4 = 1; plat->has_gmac4 = 1;
plat->force_sf_dma_mode = 1; plat->force_sf_dma_mode = 1;
plat->tso_en = 1; plat->flags |= STMMAC_FLAG_TSO_EN;
plat->pmt = 1; plat->pmt = 1;
/* Set default value for multicast hash bins */ /* Set default value for multicast hash bins */
......
...@@ -525,7 +525,8 @@ stmmac_probe_config_dt(struct platform_device *pdev, u8 *mac) ...@@ -525,7 +525,8 @@ stmmac_probe_config_dt(struct platform_device *pdev, u8 *mac)
plat->has_gmac4 = 1; plat->has_gmac4 = 1;
plat->has_gmac = 0; plat->has_gmac = 0;
plat->pmt = 1; plat->pmt = 1;
plat->tso_en = of_property_read_bool(np, "snps,tso"); if (of_property_read_bool(np, "snps,tso"))
plat->flags |= STMMAC_FLAG_TSO_EN;
} }
if (of_device_is_compatible(np, "snps,dwmac-3.610") || if (of_device_is_compatible(np, "snps,dwmac-3.610") ||
...@@ -538,7 +539,8 @@ stmmac_probe_config_dt(struct platform_device *pdev, u8 *mac) ...@@ -538,7 +539,8 @@ stmmac_probe_config_dt(struct platform_device *pdev, u8 *mac)
if (of_device_is_compatible(np, "snps,dwxgmac")) { if (of_device_is_compatible(np, "snps,dwxgmac")) {
plat->has_xgmac = 1; plat->has_xgmac = 1;
plat->pmt = 1; plat->pmt = 1;
plat->tso_en = of_property_read_bool(np, "snps,tso"); if (of_property_read_bool(np, "snps,tso"))
plat->flags |= STMMAC_FLAG_TSO_EN;
} }
dma_cfg = devm_kzalloc(&pdev->dev, sizeof(*dma_cfg), dma_cfg = devm_kzalloc(&pdev->dev, sizeof(*dma_cfg),
......
...@@ -208,6 +208,7 @@ struct dwmac4_addrs { ...@@ -208,6 +208,7 @@ struct dwmac4_addrs {
#define STMMAC_FLAG_SPH_DISABLE BIT(1) #define STMMAC_FLAG_SPH_DISABLE BIT(1)
#define STMMAC_FLAG_USE_PHY_WOL BIT(2) #define STMMAC_FLAG_USE_PHY_WOL BIT(2)
#define STMMAC_FLAG_HAS_SUN8I BIT(3) #define STMMAC_FLAG_HAS_SUN8I BIT(3)
#define STMMAC_FLAG_TSO_EN BIT(4)
struct plat_stmmacenet_data { struct plat_stmmacenet_data {
int bus_id; int bus_id;
...@@ -271,7 +272,6 @@ struct plat_stmmacenet_data { ...@@ -271,7 +272,6 @@ struct plat_stmmacenet_data {
struct reset_control *stmmac_ahb_rst; struct reset_control *stmmac_ahb_rst;
struct stmmac_axi *axi; struct stmmac_axi *axi;
int has_gmac4; int has_gmac4;
bool tso_en;
int rss_en; int rss_en;
int mac_port_sel_speed; int mac_port_sel_speed;
bool en_tx_lpi_clockgating; bool en_tx_lpi_clockgating;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment