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

[David Miller <davem@xxxxxxxxxxxxx>]

From: Christophe Roullier <christophe.roullier@xxxxxx>
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.