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)
commitb36a6bb8a151c056e1046e9d5b1192d90d9941c9
treefa2e7f08571d04c47679cf0140e0add140d5fdf0
parent6800234ceee01416657b05efcf233f6bdc68272c
drm/sun4i: de3: Be explicit about supported modifiers

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