[igt-dev] [PATCH i-g-t 1/2] lib/igt_kms: Clear pipe color management properties on reset.

Maarten Lankhorst
2019-04-03T10:30:25Z

Just like we try to sanitize all properties in igt_plane_reset, we
should do the same for pipe properties. Loading a wrong lut could
affect passing and failing tests, so we need to be careful and set
sane defaults for everything.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
lib/igt_kms.c | 9 +++++++++
1 file changed, 9 insertions(+)

diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 1e2415bf9c89..a975ff1499ca 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -1795,6 +1795,15 @@ static void igt_pipe_reset(igt_pipe_t *pipe)
igt_pipe_obj_set_prop_value(pipe, IGT_CRTC_ACTIVE, 0);
igt_pipe_obj_clear_prop_changed(pipe, IGT_CRTC_OUT_FENCE_PTR);

+ if (igt_pipe_obj_has_prop(pipe, IGT_CRTC_CTM))
+ igt_pipe_obj_set_prop_value(pipe, IGT_CRTC_CTM, 0);
+
+ if (igt_pipe_obj_has_prop(pipe, IGT_CRTC_GAMMA_LUT))
+ igt_pipe_obj_set_prop_value(pipe, IGT_CRTC_GAMMA_LUT, 0);
+
+ if (igt_pipe_obj_has_prop(pipe, IGT_CRTC_DEGAMMA_LUT))
+ igt_pipe_obj_set_prop_value(pipe, IGT_CRTC_DEGAMMA_LUT, 0);
+
pipe->out_fence_fd = -1;
}