[PATCH 6/9] iio: adc: meson_saradc: Make use of the helper function dev_err_probe()
Martin Blumenstingl
martin.blumenstingl at googlemail.com
Mon Sep 27 13:09:47 PDT 2021
Hello,
first of all: thanks for this patch!
On Mon, Sep 27, 2021 at 10:15 AM Cai Huoqing <caihuoqing at baidu.com> wrote:
[...]
> + if (IS_ERR(priv->clkin))
> + return dev_err_probe(&pdev->dev,
> + PTR_ERR(priv->clkin),
Is there any specific reason why you put PTR_ERR() on a separate line?
it would still fit into the line above and be below the old 80 chars
per line limit.
For priv->vref you already have it the way I am suggesting there.
[...]
> + if (IS_ERR(priv->core_clk))
> + return dev_err_probe(&pdev->dev,
> + PTR_ERR(priv->core_clk),
the same question as above applies here as well
Best regards,
Martin
More information about the linux-amlogic
mailing list