[PATCH v5 17/32] drm/amd/display: mark plane as needing reset if color props change

[Melissa Wen]

From: Joshua Ashton <jos...@froggi.es>

We should reset a plane state if at least one of the color management
properties differs from old and new state.

Reviewed-by: Harry Wentland <harry.wentl...@amd.com>
Signed-off-by: Joshua Ashton <jos...@froggi.es>
Co-developed-by: Melissa Wen <m...@igalia.com>
Signed-off-by: Melissa Wen <m...@igalia.com>
---
 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index 8bf7866ee1af..8f82d9837a41 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -9632,6 +9632,10 @@ static bool should_reset_plane(struct drm_atomic_state 
*state,
         */
        for_each_oldnew_plane_in_state(state, other, old_other_state, 
new_other_state, i) {
                struct amdgpu_framebuffer *old_afb, *new_afb;
+               struct dm_plane_state *dm_new_other_state, *dm_old_other_state;
+
+               dm_new_other_state = to_dm_plane_state(new_other_state);
+               dm_old_other_state = to_dm_plane_state(old_other_state);
 
                if (other->type == DRM_PLANE_TYPE_CURSOR)
                        continue;
@@ -9668,6 +9672,17 @@ static bool should_reset_plane(struct drm_atomic_state 
*state,
                    old_other_state->color_encoding != 
new_other_state->color_encoding)
                        return true;
 
+               /* HDR/Transfer Function changes. */
+               if (dm_old_other_state->degamma_tf != 
dm_new_other_state->degamma_tf ||
+                   dm_old_other_state->degamma_lut != 
dm_new_other_state->degamma_lut ||
+                   dm_old_other_state->hdr_mult != 
dm_new_other_state->hdr_mult ||
+                   dm_old_other_state->shaper_lut != 
dm_new_other_state->shaper_lut ||
+                   dm_old_other_state->shaper_tf != 
dm_new_other_state->shaper_tf ||
+                   dm_old_other_state->lut3d != dm_new_other_state->lut3d ||
+                   dm_old_other_state->blend_lut != 
dm_new_other_state->blend_lut ||
+                   dm_old_other_state->blend_tf != 
dm_new_other_state->blend_tf)
+                       return true;
+
                /* Framebuffer checks fall at the end. */
                if (!old_other_state->fb || !new_other_state->fb)
                        continue;
-- 
2.40.1