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

Reply via email to