On Mon, 11 Jan 2021 19:35:37 +0800 Joakim Zhang wrote: > In current driver, buffer2 available only when hardware supports split > header. Wrongly set buffer2 valid in stmmac_rx_refill when refill buffer > address. You can see that desc3 is 0x81000000 after initialization, but > turn out to be 0x83000000 after refill. > > Fixes: 67afd6d1cfdf ("net: stmmac: Add Split Header support and enable it in > XGMAC cores") > Signed-off-by: Joakim Zhang <qiangqing.zh...@nxp.com>
Please make sure there are no new sparse warnings (build flags: W=1 C=1): drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c:509:25: warning: invalid assignment: |= drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c:509:25: left side has type restricted __le32 drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c:509:25: right side has type unsigned long drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c:511:25: warning: invalid assignment: &= drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c:511:25: left side has type restricted __le32 drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c:511:25: right side has type unsigned long