[PATCH net] net: ethernet: mtk_eth_soc: fix typo for declaration MT7988 ESW capability
Michal Swiatkowski
michal.swiatkowski at linux.intel.com
Mon May 12 22:21:25 PDT 2025
On Tue, May 13, 2025 at 05:27:30AM +0100, Daniel Golle wrote:
> From: Bo-Cun Chen <bc-bocun.chen at mediatek.com>
>
> Since MTK_ESW_BIT is a bit number rather than a bitmap, it causes
> MTK_HAS_CAPS to produce incorrect results. This leads to the ETH
> driver not declaring MAC capabilities correctly for the MT7988 ESW.
>
> Fixes: 445eb6448ed3 ("net: ethernet: mtk_eth_soc: add basic support for MT7988 SoC")
> Signed-off-by: Bo-Cun Chen <bc-bocun.chen at mediatek.com>
> Signed-off-by: Daniel Golle <daniel at makrotopia.org>
> ---
> drivers/net/ethernet/mediatek/mtk_eth_soc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.c b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
> index 22a532695fb0..6c92072b4c28 100644
> --- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
> +++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
> @@ -4748,7 +4748,7 @@ static int mtk_add_mac(struct mtk_eth *eth, struct device_node *np)
> }
>
> if (mtk_is_netsys_v3_or_greater(mac->hw) &&
> - MTK_HAS_CAPS(mac->hw->soc->caps, MTK_ESW_BIT) &&
> + MTK_HAS_CAPS(mac->hw->soc->caps, MTK_ESW) &&
> id == MTK_GMAC1_ID) {
> mac->phylink_config.mac_capabilities = MAC_ASYM_PAUSE |
> MAC_SYM_PAUSE |
Looks like other usage of MTK_HAS_CAPS is fine, thanks for fixing.
Reviewed-by: Michal Swiatkowski <michal.swiatkowski at linux.intel.com>
> --
> 2.49.0
More information about the Linux-mediatek
mailing list