From: Jose Abreu <jose.ab...@synopsys.com>
Date: Wed, 26 Jun 2019 15:47:35 +0200

> @@ -122,6 +122,8 @@ static void dwxgmac2_dma_axi(void __iomem *ioaddr, struct 
> stmmac_axi *axi)
>       }
>  
>       writel(value, ioaddr + XGMAC_DMA_SYSBUS_MODE);
> +     writel(GENMASK(29, 0), ioaddr + XGMAC_TX_EDMA_CTRL);
> +     writel(GENMASK(29, 0), ioaddr + XGMAC_RX_EDMA_CTRL);
>  }

This mask is magic and there is no indication what the bits mean and
in particular what it means to set bits 0 -- 29

You have to document what these bits mean and thus what these register
writes actually do.

Reply via email to