drm/sun4i: de3: Be explicit about supported modifiers
authorPiotr Oniszczuk <piotr.oniszczuk@gmail.com>
Sat, 5 Jun 2021 07:35:34 +0000 (09:35 +0200)
committerMaxime Ripard <maxime@cerno.tech>
Mon, 7 Jun 2021 11:33:46 +0000 (13:33 +0200)
Currently only linear formats are supported in sun4i-drm driver, but
SoCs like H6 supports AFBC variant of some of them in multiple cores
(GPU, VPU, DE3). Panfrost already implements AFBC compression and is
sometimes confused what should be default choice (linear, AFBC) if DRM
driver is not explicit about modifier support (MiniMyth2 distro with
MythTV app).

After some discussion with Daniel Stone on #panfrost IRC, it was decided
to make modifiers in sun4i-drm explicit, to avoid any kind of guessing,
not just in panfrost, but everywhere. In fact, long term idea is to make
modifier parameter in drm_universal_plane_init() mandatory (non NULL).

Signed-off-by: Piotr Oniszczuk <piotr.oniszczuk@gmail.com>
Signed-off-by: Jernej Skrabec <jernej.skrabec@gmail.com>
Acked-by: Daniel Stone <daniels@collabora.com>
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
Link: https://patchwork.freedesktop.org/patch/msgid/20210605073534.3786194-1-jernej.skrabec@gmail.com
drivers/gpu/drm/sun4i/sun8i_ui_layer.c
drivers/gpu/drm/sun4i/sun8i_vi_layer.c

index 0db164a774a1f1e80ea41c88371632f68070e773..e779855bcd6e3547d4db576929bbbec40199214f 100644 (file)
@@ -370,6 +370,11 @@ static const u32 sun8i_ui_layer_formats[] = {
        DRM_FORMAT_XRGB8888,
 };
 
+static const uint64_t sun8i_layer_modifiers[] = {
+       DRM_FORMAT_MOD_LINEAR,
+       DRM_FORMAT_MOD_INVALID
+};
+
 struct sun8i_ui_layer *sun8i_ui_layer_init_one(struct drm_device *drm,
                                               struct sun8i_mixer *mixer,
                                               int index)
@@ -392,7 +397,7 @@ struct sun8i_ui_layer *sun8i_ui_layer_init_one(struct drm_device *drm,
                                       &sun8i_ui_layer_funcs,
                                       sun8i_ui_layer_formats,
                                       ARRAY_SIZE(sun8i_ui_layer_formats),
-                                      NULL, type, NULL);
+                                      sun8i_layer_modifiers, type, NULL);
        if (ret) {
                dev_err(drm->dev, "Couldn't initialize layer\n");
                return ERR_PTR(ret);
index 46420780db598a409f2280c0c528b9529e96055f..1c86c2dd0bbf1a2871e9e3dfb5b7760782613cdd 100644 (file)
@@ -534,6 +534,11 @@ static const u32 sun8i_vi_layer_de3_formats[] = {
        DRM_FORMAT_YVU422,
 };
 
+static const uint64_t sun8i_layer_modifiers[] = {
+       DRM_FORMAT_MOD_LINEAR,
+       DRM_FORMAT_MOD_INVALID
+};
+
 struct sun8i_vi_layer *sun8i_vi_layer_init_one(struct drm_device *drm,
                                               struct sun8i_mixer *mixer,
                                               int index)
@@ -560,7 +565,8 @@ struct sun8i_vi_layer *sun8i_vi_layer_init_one(struct drm_device *drm,
        ret = drm_universal_plane_init(drm, &layer->plane, 0,
                                       &sun8i_vi_layer_funcs,
                                       formats, format_count,
-                                      NULL, DRM_PLANE_TYPE_OVERLAY, NULL);
+                                      sun8i_layer_modifiers,
+                                      DRM_PLANE_TYPE_OVERLAY, NULL);
        if (ret) {
                dev_err(drm->dev, "Couldn't initialize layer\n");
                return ERR_PTR(ret);