On 01/02/2023 18:02, Justin Green wrote:
Tested using "modetest -P" on an MT8195 device.

Signed-off-by: Justin Green <greenjus...@chromium.org>
---
  drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 21 +++++++++++++++++++--
  1 file changed, 19 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c 
b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
index a6255e847104..7d26f7055751 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
+++ b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
@@ -85,6 +85,22 @@ static const u32 mt8173_formats[] = {
        DRM_FORMAT_YUYV,
  };
+static const u32 mt8195_formats[] = {

I'd call it mt8195_ovl_formats, to make it explicit that it's only for the OVL.

Regards,
Matthias

+       DRM_FORMAT_XRGB8888,
+       DRM_FORMAT_ARGB8888,
+       DRM_FORMAT_ARGB2101010,
+       DRM_FORMAT_BGRX8888,
+       DRM_FORMAT_BGRA8888,
+       DRM_FORMAT_BGRA1010102,
+       DRM_FORMAT_ABGR8888,
+       DRM_FORMAT_XBGR8888,
+       DRM_FORMAT_RGB888,
+       DRM_FORMAT_BGR888,
+       DRM_FORMAT_RGB565,
+       DRM_FORMAT_UYVY,
+       DRM_FORMAT_YUYV,
+};
+
  struct mtk_disp_ovl_data {
        unsigned int addr;
        unsigned int gmc_bits;
@@ -616,8 +632,9 @@ static const struct mtk_disp_ovl_data 
mt8195_ovl_driver_data = {
        .fmt_rgb565_is_0 = true,
        .smi_id_en = true,
        .supports_afbc = true,
-       .formats = mt8173_formats,
-       .num_formats = ARRAY_SIZE(mt8173_formats),
+       .formats = mt8195_formats,
+       .num_formats = ARRAY_SIZE(mt8195_formats),
+       .supports_clrfmt_ext = true,
  };
static const struct of_device_id mtk_disp_ovl_driver_dt_match[] = {

Reply via email to