[PATCH 04/11] net: dwc_eth_qos: Scrub ifdeffery
Marek Vasut
marex at denx.de
Sat Mar 9 03:11:25 CET 2024
Replace ifdef CONFIG_CLK with if (CONFIG_IS_ENABLED(CLK)) to improve code
build coverage. Some of the functions printed debug("%s: OK\n", __func__);
on exit with and without CLK enabled, some did not, make it consistent and
print nothing if CLK is disabled.
Signed-off-by: Marek Vasut <marex at denx.de>
---
Cc: Christophe Roullier <christophe.roullier at st.com>
Cc: Joe Hershberger <joe.hershberger at ni.com>
Cc: Patrice Chotard <patrice.chotard at foss.st.com>
Cc: Patrick Delaunay <patrick.delaunay at foss.st.com>
Cc: Ramon Fried <rfried.dev at gmail.com>
Cc: u-boot at dh-electronics.com
Cc: uboot-stm32 at st-md-mailman.stormreply.com
---
drivers/net/dwc_eth_qos_stm32.c | 25 ++++++++++++-------------
1 file changed, 12 insertions(+), 13 deletions(-)
diff --git a/drivers/net/dwc_eth_qos_stm32.c b/drivers/net/dwc_eth_qos_stm32.c
index 7520a136ed0..d7ec0c9be36 100644
--- a/drivers/net/dwc_eth_qos_stm32.c
+++ b/drivers/net/dwc_eth_qos_stm32.c
@@ -46,21 +46,22 @@
static ulong eqos_get_tick_clk_rate_stm32(struct udevice *dev)
{
-#ifdef CONFIG_CLK
- struct eqos_priv *eqos = dev_get_priv(dev);
+ struct eqos_priv __maybe_unused *eqos = dev_get_priv(dev);
+
+ if (!CONFIG_IS_ENABLED(CLK))
+ return 0;
return clk_get_rate(&eqos->clk_master_bus);
-#else
- return 0;
-#endif
}
static int eqos_start_clks_stm32(struct udevice *dev)
{
-#ifdef CONFIG_CLK
- struct eqos_priv *eqos = dev_get_priv(dev);
+ struct eqos_priv __maybe_unused *eqos = dev_get_priv(dev);
int ret;
+ if (!CONFIG_IS_ENABLED(CLK))
+ return 0;
+
debug("%s(dev=%p):\n", __func__, dev);
ret = clk_enable(&eqos->clk_master_bus);
@@ -89,12 +90,10 @@ static int eqos_start_clks_stm32(struct udevice *dev)
}
eqos->clk_ck_enabled = true;
}
-#endif
debug("%s: OK\n", __func__);
return 0;
-#ifdef CONFIG_CLK
err_disable_clk_tx:
clk_disable(&eqos->clk_tx);
err_disable_clk_rx:
@@ -104,20 +103,20 @@ err_disable_clk_master_bus:
err:
debug("%s: FAILED: %d\n", __func__, ret);
return ret;
-#endif
}
static int eqos_stop_clks_stm32(struct udevice *dev)
{
-#ifdef CONFIG_CLK
- struct eqos_priv *eqos = dev_get_priv(dev);
+ struct eqos_priv __maybe_unused *eqos = dev_get_priv(dev);
+
+ if (!CONFIG_IS_ENABLED(CLK))
+ return 0;
debug("%s(dev=%p):\n", __func__, dev);
clk_disable(&eqos->clk_tx);
clk_disable(&eqos->clk_rx);
clk_disable(&eqos->clk_master_bus);
-#endif
debug("%s: OK\n", __func__);
return 0;
--
2.43.0
More information about the U-Boot
mailing list