Re: [PATCH net-next v3 2/3] net: stmmac: Integrate dwxgmac4 into stmmac hwif handling

From: Jitendra Vegiraju
Date: Mon Aug 05 2024 - 20:36:54 EST


Hi Andrew,
On Fri, Aug 2, 2024 at 3:59 PM Andrew Lunn <andrew@xxxxxxx> wrote:
>
> > + user_ver = stmmac_get_user_version(priv, GMAC4_VERSION);
> > + if (priv->synopsys_id == DWXGMAC_CORE_4_00 &&
> > + user_ver == DWXGMAC_USER_VER_X22)
> > + mac->dma = &dwxgmac400_dma_ops;
>
> I know nothing about this hardware....
>
> Does priv->synopsys_id == DWXGMAC_CORE_4_0 not imply
> dwxgmac400_dma_ops?
>
> Could a user synthesise DWXGMAC_CORE_4_00 without using
> dwxgmac400_dma_ops? Could dwxgmac500_dma_ops or dwxgmac100_dma_ops be
> used?
Yes, the user can choose between Enhanced DMA , Hyper DMA , Normal DMA.
This SoC support has chosen Hyper DMA for future expandability.

>
> Andrew