[Linux-stm32] [PATCH net-next 1/4] net: ethernet: stmmac: Add support for syscfg clock

David Miller davem at davemloft.net
Mon Nov 4 19:33:44 UTC 2019


From: Christophe Roullier <christophe.roullier at st.com>
Date: Mon, 4 Nov 2019 14:25:30 +0100

> +				if (dwmac->syscfg_clk)
> +					goto unprepare_syscfg;
>  				return ret;
 ...
> +unprepare_syscfg:
> +	clk_disable_unprepare(dwmac->syscfg_clk);
> +
> +	return ret;

This is so amazingly silly.  You're doing a goto instead of the
clk_disable_unprepare() call itself.

Please don't do this.



More information about the Linux-stm32 mailing list