* linux@xxxxxxxxxxx (linux@xxxxxxxxxxx) wrote:
> From: "Dr. David Alan Gilbert" <linux@xxxxxxxxxxx>
>
> 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@xxxxxxxxxxx>
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 |_______/
