[PATCH] drm/exynos: dsi: silence warning about regulators during deferred probe

Krzysztof Kozlowski krzk at kernel.org
Wed Feb 26 12:00:14 UTC 2020


On Wed, 26 Feb 2020 at 11:11, Marek Szyprowski <m.szyprowski at samsung.com> wrote:
>
> Don't confuse user with meaningless warning about failure in getting
> regulators in case of deferred probe.

You are doing actually more than you explained here. You fixed
inaccurate error code (EPROBE_DEFER) being returned in case of
failures of getting regulators.

This change makes sense but commit message looks too innocent.

Best regards,
Krzysztof

>
> Signed-off-by: Marek Szyprowski <m.szyprowski at samsung.com>
> ---
>  drivers/gpu/drm/exynos/exynos_drm_dsi.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> index 08a8ce3f499c..a85365c56d4d 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> @@ -1773,8 +1773,9 @@ static int exynos_dsi_probe(struct platform_device *pdev)
>         ret = devm_regulator_bulk_get(dev, ARRAY_SIZE(dsi->supplies),
>                                       dsi->supplies);
>         if (ret) {
> -               dev_info(dev, "failed to get regulators: %d\n", ret);
> -               return -EPROBE_DEFER;
> +               if (ret != -EPROBE_DEFER)
> +                       dev_info(dev, "failed to get regulators: %d\n", ret);
> +               return ret;
>         }
>
>         dsi->clks = devm_kcalloc(dev,
> --
> 2.17.1
>


More information about the dri-devel mailing list