[PATCH 03/11] drm/mediatek: ovl: Add supports_plane_colorops flag

Nícolas F. R. A. Prado posted 11 patches 1 month, 2 weeks ago
[PATCH 03/11] drm/mediatek: ovl: Add supports_plane_colorops flag
Posted by Nícolas F. R. A. Prado 1 month, 2 weeks ago
Add a supports_plane_colorops flag to the driver data to allow SoCs that
support per-plane color operations to enable support for it in the
driver.

Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
---
 drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
index 8e20b45411fc..46238c21b0ac 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
+++ b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
@@ -150,6 +150,7 @@ struct mtk_disp_ovl_data {
 	const u32 *formats;
 	size_t num_formats;
 	bool supports_clrfmt_ext;
+	bool supports_plane_colorops;
 };
 
 /*

-- 
2.51.0

Re: [PATCH 03/11] drm/mediatek: ovl: Add supports_plane_colorops flag
Posted by AngeloGioacchino Del Regno 3 days, 18 hours ago
Il 23/12/25 20:44, Nícolas F. R. A. Prado ha scritto:
> Add a supports_plane_colorops flag to the driver data to allow SoCs that
> support per-plane color operations to enable support for it in the
> driver.
> 
> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>

You should merge this patch [03/11] with [05/11] - so that you introduce the member
in the same commit where you use it.

This patch is a one-liner and it's not doing anything complicated at all :-)

Cheers,
Angelo

> ---
>   drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
> index 8e20b45411fc..46238c21b0ac 100644
> --- a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
> +++ b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
> @@ -150,6 +150,7 @@ struct mtk_disp_ovl_data {
>   	const u32 *formats;
>   	size_t num_formats;
>   	bool supports_clrfmt_ext;
> +	bool supports_plane_colorops;
>   };
>   
>   /*
> 



Re: [PATCH 03/11] drm/mediatek: ovl: Add supports_plane_colorops flag
Posted by Nícolas F. R. A. Prado 3 days, 14 hours ago
On Fri, 2026-02-06 at 11:07 +0100, AngeloGioacchino Del Regno wrote:
> Il 23/12/25 20:44, Nícolas F. R. A. Prado ha scritto:
> > Add a supports_plane_colorops flag to the driver data to allow SoCs
> > that
> > support per-plane color operations to enable support for it in the
> > driver.
> > 
> > Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
> 
> You should merge this patch [03/11] with [05/11] - so that you
> introduce the member
> in the same commit where you use it.
> 
> This patch is a one-liner and it's not doing anything complicated at
> all :-)

Sure, will do in v2 :).

-- 
Thanks,

Nícolas
Re: [PATCH 03/11] drm/mediatek: ovl: Add supports_plane_colorops flag
Posted by Macpaul Lin (林智斌) 1 month, 2 weeks ago
On Tue, 2025-12-23 at 16:44 -0300, Nícolas F. R. A. Prado wrote:
> Add a supports_plane_colorops flag to the driver data to allow SoCs
> that
> support per-plane color operations to enable support for it in the
> driver.
> 
> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
> ---
>  drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
> b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
> index 8e20b45411fc..46238c21b0ac 100644
> --- a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
> +++ b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
> @@ -150,6 +150,7 @@ struct mtk_disp_ovl_data {
>         const u32 *formats;
>         size_t num_formats;
>         bool supports_clrfmt_ext;
> +       bool supports_plane_colorops;
>  };
> 
>  /*
> 
> --
> 2.51.0
> 
> 

Reviewed-by: Macpaul Lin <macpaul.lin@mediatek.com>

BR,
Macpaul Lin