[PATCH] hwrng: meson - Improve error handling for core clock
Martin Blumenstingl
martin.blumenstingl at googlemail.com
Sun Sep 19 13:07:15 PDT 2021
Hi Uwe,
On Tue, Sep 14, 2021 at 4:24 PM Uwe Kleine-König
<u.kleine-koenig at pengutronix.de> wrote:
[...]
> - data->core_clk = devm_clk_get(dev, "core");
> - if (IS_ERR(data->core_clk))
> - data->core_clk = NULL;
> + data->core_clk = devm_clk_get_optional(dev, "core");
> + if (IS_ERR(data->core_clk)) {
> + ret = PTR_ERR(data->core_clk);
> + if (ret != -EPROBE_DEFER)
> + dev_err(dev, "Failed to get core clock: %pe\n",
> + data->core_clk);
> +
> + return ret;
I suggest simplifying this by using dev_err_probe as this:
- allows you to get rid of the if (ret != -EPROBE_DEFER)
- means that the message is shown in debugfs' "devices_deferred"
Best regards,
Martin
More information about the linux-amlogic
mailing list