Hi,
We have a bunch of functions that create planes properties that will take a
default value, but it isn't actually enforced in the plane state.
This leads to drivers having multiple strategies to work around that issue,
most of them being a variation of forcing a value at plane state reset time.
Others work fine by luck, or have entirely ignored the issue.
This series aims at making sure the default value set by the call to the
function isn't ignored, and then making sure all drivers behave consistently.
Let me know what you think,
Maxime
Changes from v1:
- Collected tags
- Squashed some patches
Dave Stevenson (3):
drm/object: Add drm_object_property_get_default_value() function
drm/object: Add default zpos value at reset
drm/object: Add default color encoding and range value at reset
Maxime Ripard (19):
drm/komeda: plane: switch to plane reset helper
drm/tegra: plane: switch to plane reset helper
drm/tegra: hub: Fix zpos initial value mismatch
drm/omap: plane: Fix zpos initial value mismatch
drm/amd/display: Fix color encoding mismatch
drm/tegra: plane: Remove redundant zpos initialisation
drm/komeda: plane: Remove redundant zpos initialisation
drm/exynos: plane: Remove redundant zpos initialisation
drm/imx: ipuv3-plane: Remove redundant zpos initialisation
drm/msm/mdp5: Remove redundant zpos initialisation
drm/nouveau/kms: Remove redundant zpos initialisation
drm/omap: plane: Remove redundant zpos initialisation
drm/rcar: plane: Remove redundant zpos initialisation
drm/sti: plane: Remove redundant zpos initialisation
drm/sun4i: layer: Remove redundant zpos initialisation
drm/komeda: plane: Remove redundant color encoding and range
initialisation
drm/armada: overlay: Remove redundant color encoding and range
initialisation
drm/imx: ipuv3-plane: Remove redundant color encoding and range
initialisation
drm/omap: plane: Remove redundant color encoding and range
initialisation
.../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-
.../gpu/drm/arm/display/komeda/komeda_plane.c | 13 +----
drivers/gpu/drm/armada/armada_overlay.c | 2 -
drivers/gpu/drm/drm_atomic_state_helper.c | 25 +++++++++
drivers/gpu/drm/drm_mode_object.c | 53 +++++++++++++++----
drivers/gpu/drm/exynos/exynos_drm_plane.c | 5 +-
drivers/gpu/drm/imx/ipuv3-plane.c | 8 +--
drivers/gpu/drm/msm/disp/mdp5/mdp5_plane.c | 16 +++---
drivers/gpu/drm/nouveau/dispnv50/wndw.c | 2 -
drivers/gpu/drm/omapdrm/omap_plane.c | 22 ++++----
drivers/gpu/drm/rcar-du/rcar_du_plane.c | 1 -
drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 1 -
drivers/gpu/drm/sti/sti_cursor.c | 2 +-
drivers/gpu/drm/sti/sti_gdp.c | 2 +-
drivers/gpu/drm/sti/sti_hqvdp.c | 2 +-
drivers/gpu/drm/sti/sti_plane.c | 6 ---
drivers/gpu/drm/sti/sti_plane.h | 1 -
drivers/gpu/drm/sun4i/sun4i_layer.c | 16 +++---
drivers/gpu/drm/tegra/hub.c | 2 +-
drivers/gpu/drm/tegra/plane.c | 6 +--
include/drm/drm_mode_object.h | 7 +++
21 files changed, 111 insertions(+), 83 deletions(-)
--
2.35.1
