| Message ID | 20241112-fix_qcom_ethqos_to_support_xgmac-v1-1-f0c93b27f9b2@quicinc.com |
|---|---|
| State | New |
| Headers | show |
| Series | net: stmmac: dwmac-qcom-ethqos: Enable support for XGMAC | expand |
On 11/13/2024 6:51 PM, Andrew Lunn wrote: > On Tue, Nov 12, 2024 at 06:08:10PM -0800, Sagar Cheluvegowda wrote: >> All Qualcomm platforms have only supported EMAC version 4 until >> now whereas in future we will also be supporting XGMAC version >> which has higher capabilities than its peer. As both has_gmac4 >> and has_xgmac fields cannot co-exist, make sure to disable the >> former flag when has_xgmac is enabled. > > If you say they are mutually exclusive, how can it happen that both > are enabled? > > To me, this feels like you are papering over a bug somewhere else. > > Andrew We can set either has_gmac4 or has_xgmac flags by using below dtsi properties as well. But since Qualcomm only supported GMAC4 version in all of its chipsets until now, we had enabled has_gmac4 flag by default within dwmac_qcom_ethqos.c instead of adding any of the below entries in the dtsi. But this will create problem for us as we start supporting Xgmac version in the future, so we are trying to add this change so that our driver can support Xgmac version when "snps,dwxgmac" is defined in the dtsi and we are keeping the default supported configuration as gmac4. if (of_device_is_compatible(np, "snps,dwmac-4.00") || of_device_is_compatible(np, "snps,dwmac-4.10a") || of_device_is_compatible(np, "snps,dwmac-4.20a") || of_device_is_compatible(np, "snps,dwmac-5.10a") || of_device_is_compatible(np, "snps,dwmac-5.20")) { plat->has_gmac4 = 1; plat->has_gmac = 0; plat->pmt = 1; if (of_property_read_bool(np, "snps,tso")) plat->flags |= STMMAC_FLAG_TSO_EN; } if (of_device_is_compatible(np, "snps,dwxgmac")) { plat->has_xgmac = 1; plat->pmt = 1; if (of_property_read_bool(np, "snps,tso")) plat->flags |= STMMAC_FLAG_TSO_EN; }
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-qcom-ethqos.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-qcom-ethqos.c index 901a3c1959fa..2f813f7ab196 100644 --- a/drivers/net/ethernet/stmicro/stmmac/dwmac-qcom-ethqos.c +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-qcom-ethqos.c @@ -872,6 +872,8 @@ static int qcom_ethqos_probe(struct platform_device *pdev) plat_dat->dump_debug_regs = rgmii_dump; plat_dat->ptp_clk_freq_config = ethqos_ptp_clk_freq_config; plat_dat->has_gmac4 = 1; + if (plat_dat->has_xgmac) + plat_dat->has_gmac4 = 0; if (ethqos->has_emac_ge_3) plat_dat->dwmac4_addrs = &data->dwmac4_addrs; plat_dat->pmt = 1;
All Qualcomm platforms have only supported EMAC version 4 until now whereas in future we will also be supporting XGMAC version which has higher capabilities than its peer. As both has_gmac4 and has_xgmac fields cannot co-exist, make sure to disable the former flag when has_xgmac is enabled. We want to keep the default capabilities as EMAC4 and enable XGMAC support from the dtsi based on the platform needs. Signed-off-by: Sagar Cheluvegowda <quic_scheluve@quicinc.com> --- drivers/net/ethernet/stmicro/stmmac/dwmac-qcom-ethqos.c | 2 ++ 1 file changed, 2 insertions(+) --- base-commit: 28955f4fa2823e39f1ecfb3a37a364563527afbc change-id: 20241112-fix_qcom_ethqos_to_support_xgmac-d1a81ea9cbfc Best regards,