net: dwc_eth_qos: Scrub ifdeffery
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. Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com> Signed-off-by: Marek Vasut <marex@denx.de> Reviewed-by: Christophe ROULLIER <christophe.roullier@foss.st.com>
This commit is contained in:
parent
d100c1abb7
commit
b204c2a9ae
|
@ -46,21 +46,22 @@
|
||||||
|
|
||||||
static ulong eqos_get_tick_clk_rate_stm32(struct udevice *dev)
|
static ulong eqos_get_tick_clk_rate_stm32(struct udevice *dev)
|
||||||
{
|
{
|
||||||
#ifdef CONFIG_CLK
|
struct eqos_priv __maybe_unused *eqos = dev_get_priv(dev);
|
||||||
struct eqos_priv *eqos = dev_get_priv(dev);
|
|
||||||
|
if (!CONFIG_IS_ENABLED(CLK))
|
||||||
|
return 0;
|
||||||
|
|
||||||
return clk_get_rate(&eqos->clk_master_bus);
|
return clk_get_rate(&eqos->clk_master_bus);
|
||||||
#else
|
|
||||||
return 0;
|
|
||||||
#endif
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static int eqos_start_clks_stm32(struct udevice *dev)
|
static int eqos_start_clks_stm32(struct udevice *dev)
|
||||||
{
|
{
|
||||||
#ifdef CONFIG_CLK
|
struct eqos_priv __maybe_unused *eqos = dev_get_priv(dev);
|
||||||
struct eqos_priv *eqos = dev_get_priv(dev);
|
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
|
if (!CONFIG_IS_ENABLED(CLK))
|
||||||
|
return 0;
|
||||||
|
|
||||||
debug("%s(dev=%p):\n", __func__, dev);
|
debug("%s(dev=%p):\n", __func__, dev);
|
||||||
|
|
||||||
ret = clk_enable(&eqos->clk_master_bus);
|
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;
|
eqos->clk_ck_enabled = true;
|
||||||
}
|
}
|
||||||
#endif
|
|
||||||
|
|
||||||
debug("%s: OK\n", __func__);
|
debug("%s: OK\n", __func__);
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
#ifdef CONFIG_CLK
|
|
||||||
err_disable_clk_tx:
|
err_disable_clk_tx:
|
||||||
clk_disable(&eqos->clk_tx);
|
clk_disable(&eqos->clk_tx);
|
||||||
err_disable_clk_rx:
|
err_disable_clk_rx:
|
||||||
|
@ -104,20 +103,20 @@ err_disable_clk_master_bus:
|
||||||
err:
|
err:
|
||||||
debug("%s: FAILED: %d\n", __func__, ret);
|
debug("%s: FAILED: %d\n", __func__, ret);
|
||||||
return ret;
|
return ret;
|
||||||
#endif
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static int eqos_stop_clks_stm32(struct udevice *dev)
|
static int eqos_stop_clks_stm32(struct udevice *dev)
|
||||||
{
|
{
|
||||||
#ifdef CONFIG_CLK
|
struct eqos_priv __maybe_unused *eqos = dev_get_priv(dev);
|
||||||
struct eqos_priv *eqos = dev_get_priv(dev);
|
|
||||||
|
if (!CONFIG_IS_ENABLED(CLK))
|
||||||
|
return 0;
|
||||||
|
|
||||||
debug("%s(dev=%p):\n", __func__, dev);
|
debug("%s(dev=%p):\n", __func__, dev);
|
||||||
|
|
||||||
clk_disable(&eqos->clk_tx);
|
clk_disable(&eqos->clk_tx);
|
||||||
clk_disable(&eqos->clk_rx);
|
clk_disable(&eqos->clk_rx);
|
||||||
clk_disable(&eqos->clk_master_bus);
|
clk_disable(&eqos->clk_master_bus);
|
||||||
#endif
|
|
||||||
|
|
||||||
debug("%s: OK\n", __func__);
|
debug("%s: OK\n", __func__);
|
||||||
return 0;
|
return 0;
|
||||||
|
|
Loading…
Reference in New Issue
Block a user