On Fri, 2 Aug 2024 11:45:21 +0100 Russell King (Oracle) wrote: > Subject: [PATCH RFC v3 0/14] net: stmmac: convert stmmac "pcs" to phylink we have a build error here inside the tasty layered cake that is the op handling in this driver (from patch 2 to 13, inclusive): In file included from drivers/net/ethernet/stmicro/stmmac/common.h:26, from drivers/net/ethernet/stmicro/stmmac/stmmac.h:20, from drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c:19: drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c: In function ‘stmmac_ethtool_set_link_ksettings’: drivers/net/ethernet/stmicro/stmmac/hwif.h:15:17: error: too many arguments to function ‘priv->hw->mac->pcs_ctrl_ane’ 15 | (__priv)->hw->__module->__cname((__arg0), ##__args); \ | ^ drivers/net/ethernet/stmicro/stmmac/hwif.h:485:9: note: in expansion of macro ‘stmmac_do_void_callback’ 485 | stmmac_do_void_callback(__priv, mac, pcs_ctrl_ane, __priv, __args) | ^~~~~~~~~~~~~~~~~~~~~~~ drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c:419:17: note: in expansion of macro ‘stmmac_pcs_ctrl_ane’ 419 | stmmac_pcs_ctrl_ane(priv, priv->ioaddr, 1, priv->hw->ps, 0); | ^~~~~~~~~~~~~~~~~~~