Re: [PATCH 10/10] drm/panel: simple: Use dev_err_probe() to simplify code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Fri, Sep 23, 2022 at 6:59 PM Yuan Can <yuancan@xxxxxxxxxx> wrote:
>
> In the probe path, dev_err() can be replaced with dev_err_probe()
> which will check if error code is -EPROBE_DEFER and prints the
> error name. It also sets the defer probe reason which can be
> checked later through debugfs.
>
> Signed-off-by: Yuan Can <yuancan@xxxxxxxxxx>
> ---
>  drivers/gpu/drm/panel/panel-simple.c | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
> index 0cb3be26e2e6..1607824dc2b3 100644
> --- a/drivers/gpu/drm/panel/panel-simple.c
> +++ b/drivers/gpu/drm/panel/panel-simple.c
> @@ -575,12 +575,9 @@ static int panel_simple_probe(struct device *dev, const struct panel_desc *desc)
>
>         panel->enable_gpio = devm_gpiod_get_optional(dev, "enable",
>                                                      GPIOD_OUT_LOW);
> -       if (IS_ERR(panel->enable_gpio)) {
> -               err = PTR_ERR(panel->enable_gpio);
> -               if (err != -EPROBE_DEFER)
> -                       dev_err(dev, "failed to request GPIO: %d\n", err);
> -               return err;
> -       }
> +       if (IS_ERR(panel->enable_gpio))
> +               return dev_err_probe(dev, PTR_ERR(panel->enable_gpio),
> +                                    "failed to request GPIO\n");

Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx>

I'd be happy to land this patch and the panel-edp one into drm-misc
just because I've touched those panel drivers in the past. I'd tend to
leave the other panel drivers to others unless you really get stuck.

For now I'll sit tight because I think you can make a 2nd fix to the
panel-edp one and put them into the same patch.

-Doug



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux