[PATCH 1/1] miiphy.h: Remove CONFIG_DM_xxx guards

Tom Rini trini at konsulko.com
Tue Jan 18 23:24:40 CET 2022


Function prototypes must not be guarded with ifdef tests.  Doing so
prevents us from doing:
if (CONFIG_IS_ENABLED(FOO))
  func();

as that results in a warning when CONFIG_FOO is not enabled.

Signed-off-by: Tom Rini <trini at konsulko.com>
---
I forgot that I had this applied when pushing before, as it's required
by the aspeed mii changes.

 include/miiphy.h | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/include/miiphy.h b/include/miiphy.h
index 8b77bac01e6a..77a00359587b 100644
--- a/include/miiphy.h
+++ b/include/miiphy.h
@@ -126,8 +126,6 @@ int bb_miiphy_write(struct mii_dev *miidev, int addr, int devad, int reg,
 #define ESTATUS_1000XF		0x8000
 #define ESTATUS_1000XH		0x4000
 
-#ifdef CONFIG_DM_MDIO
-
 /**
  * struct mdio_perdev_priv - Per-device class data for MDIO DM
  *
@@ -185,10 +183,6 @@ struct phy_device *dm_mdio_phy_connect(struct udevice *mdiodev, int phyaddr,
  */
 struct phy_device *dm_eth_phy_connect(struct udevice *ethdev);
 
-#endif
-
-#ifdef CONFIG_DM_MDIO_MUX
-
 /* indicates none of the child buses is selected */
 #define MDIO_MUX_SELECT_NONE	-1
 
@@ -206,5 +200,3 @@ struct mdio_mux_ops {
 #define mdio_mux_get_ops(dev) ((struct mdio_mux_ops *)(dev)->driver->ops)
 
 #endif
-
-#endif
-- 
2.25.1



More information about the U-Boot mailing list