[PATCH v2 2/2] drm/kmb: Do not report 0 (success) in case of error

Chrisanthus, Anitha anitha.chrisanthus at intel.com
Thu May 27 16:30:16 UTC 2021


This is already fixed in the patch from Zhen Lei.

> -----Original Message-----
> From: Christophe JAILLET <christophe.jaillet at wanadoo.fr>
> Sent: Wednesday, May 26, 2021 11:10 PM
> To: Chrisanthus, Anitha <anitha.chrisanthus at intel.com>; Dea, Edmund J
> <edmund.j.dea at intel.com>; airlied at linux.ie; daniel at ffwll.ch;
> sam at ravnborg.org
> Cc: dri-devel at lists.freedesktop.org; linux-kernel at vger.kernel.org; kernel-
> janitors at vger.kernel.org; Christophe JAILLET <christophe.jaillet at wanadoo.fr>
> Subject: [PATCH v2 2/2] drm/kmb: Do not report 0 (success) in case of error
> 
> 'ret' is known to be 0 at this point.
> Reporting the error from the previous 'platform_get_irq()' call is likely,
> so add the missing assignment.
> 
> Fixes: 7f7b96a8a0a1 ("drm/kmb: Add support for KeemBay Display")
> Signed-off-by: Christophe JAILLET <christophe.jaillet at wanadoo.fr>
> ---
> v2: New patch
> ---
>  drivers/gpu/drm/kmb/kmb_drv.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/kmb/kmb_drv.c
> b/drivers/gpu/drm/kmb/kmb_drv.c
> index fa28e42da460..d9e10ac9847c 100644
> --- a/drivers/gpu/drm/kmb/kmb_drv.c
> +++ b/drivers/gpu/drm/kmb/kmb_drv.c
> @@ -138,6 +138,7 @@ static int kmb_hw_init(struct drm_device *drm,
> unsigned long flags)
>  	irq_lcd = platform_get_irq(pdev, 0);
>  	if (irq_lcd < 0) {
>  		drm_err(&kmb->drm, "irq_lcd not found");
> +		ret = irq_lcd;
>  		goto setup_fail;
>  	}
> 
> --
> 2.30.2



More information about the dri-devel mailing list