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

From: Andrew Lunn
Date: Fri Aug 02 2024 - 19:00:07 EST


> + 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?

Andrew