On Thu, Jan 19, 2023 at 11:45 PM Marek Vasut <ma...@denx.de> wrote:
>
> The return is never triggered due to the goto just above it.
> Drop it. No functional change.
>
> Signed-off-by: Marek Vasut <ma...@denx.de>
> ---
> Cc: "Ariel D'Alessandro" <ariel.dalessan...@collabora.com>
> Cc: "NXP i.MX U-Boot Team" <uboot-...@nxp.com>
> Cc: Andrey Zhizhikin <andrey.zhizhi...@leica-geosystems.com>
> Cc: Fabio Estevam <feste...@gmail.com>
> Cc: Joe Hershberger <joe.hershber...@ni.com>
> Cc: Lukasz Majewski <lu...@denx.de>
> Cc: Marcel Ziswiler <marcel.ziswi...@toradex.com>
> Cc: Marek Vasut <ma...@denx.de>
> Cc: Michael Trimarchi <mich...@amarulasolutions.com>
> Cc: Peng Fan <peng....@nxp.com>
> Cc: Ramon Fried <rfried....@gmail.com>
> Cc: Sean Anderson <sean...@gmail.com>
> Cc: Stefano Babic <sba...@denx.de>
> Cc: Tim Harvey <thar...@gateworks.com>
> Cc: Tommaso Merciai <tommaso.merc...@amarulasolutions.com>
> Cc: u-boot@lists.denx.de
> ---
> drivers/net/dwc_eth_qos.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/net/dwc_eth_qos.c b/drivers/net/dwc_eth_qos.c
> index afc47b56ff5..44d05908894 100644
> --- a/drivers/net/dwc_eth_qos.c
> +++ b/drivers/net/dwc_eth_qos.c
> @@ -1373,7 +1373,6 @@ static int eqos_probe_resources_tegra186(struct udevice
> *dev)
> if (ret) {
> pr_err("clk_get_by_name(ptp_ref) failed: %d", ret);
> goto err_free_clk_rx;
> - return ret;
> }
>
> ret = clk_get_by_name(dev, "tx", &eqos->clk_tx);
> --
> 2.39.0
>
Reviewed-by: Ramon Fried <rfried....@gmail.com>