Re: [PATCH] drm/kmb: Remove unused const and defines

["Dr. David Alan Gilbert" <linux@treblig.org>]

* linux@treblig.org (linux@treblig.org) wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> layer_irqs[] was added in 2020 as part of
> commit 7f7b96a8a0a1 ("drm/kmb: Add support for KeemBay Display")
> but isn't used.
> 
> Remove it.
> 
> It was the only user of the LCD_INT_VL0, LCD_INT_VL1, LCD_INT_GL0
> and LCD_INT_GL1 defines.
> 
> Remove them.
> 
> commit c026565fe9be ("drm/kmb: Enable alpha blended second plane")
> added a second copy of the POSSIBLE_CRTCS define.
> 
> Remove it.
> 
> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>

Hi,
  Can someone please review or pick up this one please.

Thanks,

Dave

> ---
>  drivers/gpu/drm/kmb/kmb_plane.c | 7 -------
>  drivers/gpu/drm/kmb/kmb_plane.h | 5 -----
>  2 files changed, 12 deletions(-)
> 
> diff --git a/drivers/gpu/drm/kmb/kmb_plane.c b/drivers/gpu/drm/kmb/kmb_plane.c
> index 9e0562aa2bcb..5a8c7cbf27b0 100644
> --- a/drivers/gpu/drm/kmb/kmb_plane.c
> +++ b/drivers/gpu/drm/kmb/kmb_plane.c
> @@ -17,13 +17,6 @@
>  #include "kmb_plane.h"
>  #include "kmb_regs.h"
>  
> -const u32 layer_irqs[] = {
> -	LCD_INT_VL0,
> -	LCD_INT_VL1,
> -	LCD_INT_GL0,
> -	LCD_INT_GL1
> -};
> -
>  /* Conversion (yuv->rgb) matrix from myriadx */
>  static const u32 csc_coef_lcd[] = {
>  	1024, 0, 1436,
> diff --git a/drivers/gpu/drm/kmb/kmb_plane.h b/drivers/gpu/drm/kmb/kmb_plane.h
> index b51144044fe8..ad8b9a67c57f 100644
> --- a/drivers/gpu/drm/kmb/kmb_plane.h
> +++ b/drivers/gpu/drm/kmb/kmb_plane.h
> @@ -25,14 +25,9 @@
>  
>  #define LCD_INT_GL0_ERR (LAYER2_DMA_FIFO_OVERFLOW | LAYER2_DMA_FIFO_UNDERFLOW)
>  #define LCD_INT_GL1_ERR (LAYER3_DMA_FIFO_OVERFLOW | LAYER3_DMA_FIFO_UNDERFLOW)
> -#define LCD_INT_VL0 (LAYER0_DMA_DONE | LAYER0_DMA_IDLE | LCD_INT_VL0_ERR)
> -#define LCD_INT_VL1 (LAYER1_DMA_DONE | LAYER1_DMA_IDLE | LCD_INT_VL1_ERR)
> -#define LCD_INT_GL0 (LAYER2_DMA_DONE | LAYER2_DMA_IDLE | LCD_INT_GL0_ERR)
> -#define LCD_INT_GL1 (LAYER3_DMA_DONE | LAYER3_DMA_IDLE | LCD_INT_GL1_ERR)
>  #define LCD_INT_DMA_ERR (LCD_INT_VL0_ERR | LCD_INT_VL1_ERR \
>  		| LCD_INT_GL0_ERR | LCD_INT_GL1_ERR)
>  
> -#define POSSIBLE_CRTCS 1
>  #define to_kmb_plane(x) container_of(x, struct kmb_plane, base_plane)
>  
>  #define POSSIBLE_CRTCS		1
> -- 
> 2.48.1
> 
-- 
 -----Open up your eyes, open up your mind, open up your code -------   
/ Dr. David Alan Gilbert    |       Running GNU/Linux       | Happy  \ 
\        dave @ treblig.org |                               | In Hex /
 \ _________________________|_____ http://www.treblig.org   |_______/