]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
net: stmmac: visconti: use PHY_INTF_SEL_x to select PHY interface
authorRussell King (Oracle) <rmk+kernel@armlinux.org.uk>
Tue, 11 Nov 2025 08:12:43 +0000 (08:12 +0000)
committerJakub Kicinski <kuba@kernel.org>
Thu, 13 Nov 2025 02:13:42 +0000 (18:13 -0800)
Convert dwmac-visconti to use the PHY_INTF_SEL_x definitions. The
original definitions used constant 0, BIT(0) (==1) and BIT(2) (==4)
to define these, but the values of the bits corresponds with the
PHY_INTF_SEL_x values, so it is highly likely that these are not
individual bits, but the PHY_INTF_SEL_x bitfield.

This removes this incorrect use of BIT().

Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Link: https://patch.msgid.link/E1vIjUZ-0000000Dqu5-2sDI@rmk-PC.armlinux.org.uk
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/stmicro/stmmac/dwmac-visconti.c

index bd65d42390543be82e698681dd6e86af94758b8c..7b6b048e1be050282f70237d57eff1b2e6721506 100644 (file)
@@ -42,9 +42,9 @@
 
 #define ETHER_CLK_SEL_RX_TX_CLK_EN (ETHER_CLK_SEL_RX_CLK_EN | ETHER_CLK_SEL_TX_CLK_EN)
 
-#define ETHER_CONFIG_INTF_MII 0
-#define ETHER_CONFIG_INTF_RGMII BIT(0)
-#define ETHER_CONFIG_INTF_RMII BIT(2)
+#define ETHER_CONFIG_INTF_MII  PHY_INTF_SEL_GMII_MII
+#define ETHER_CONFIG_INTF_RGMII        PHY_INTF_SEL_RGMII
+#define ETHER_CONFIG_INTF_RMII PHY_INTF_SEL_RMII
 
 struct visconti_eth {
        void __iomem *reg;