Re: [PATCH 1/6] net: hisilicon: add support for hisi_femac core on Hi3798MV200

From: Andrew Lunn
Date: Thu Feb 15 2024 - 18:58:06 EST


> + for (i = 0; i < CLK_NUM; i++) {
> + priv->clks[i] = devm_clk_get_enabled(&pdev->dev, clk_strs[i]);
> + if (IS_ERR(priv->clks[i])) {
> + dev_err(dev, "failed to get enabled clk %s: %ld\n", clk_strs[i],
> + PTR_ERR(priv->clks[i]));
> + ret = -ENODEV;
> + goto out_free_netdev;
> + }

The clk API has devm_clk_bulk_ versions. Please take a look at them, and see
if it will simplify the code.

Andrew