diff options
Diffstat (limited to 'common/recipes-kernel/linux/linux-yocto-4.19.8/1217-drm-amd-display-Don-t-re-program-planes-for-DPMS-cha.patch')
-rw-r--r-- | common/recipes-kernel/linux/linux-yocto-4.19.8/1217-drm-amd-display-Don-t-re-program-planes-for-DPMS-cha.patch | 62 |
1 files changed, 62 insertions, 0 deletions
diff --git a/common/recipes-kernel/linux/linux-yocto-4.19.8/1217-drm-amd-display-Don-t-re-program-planes-for-DPMS-cha.patch b/common/recipes-kernel/linux/linux-yocto-4.19.8/1217-drm-amd-display-Don-t-re-program-planes-for-DPMS-cha.patch new file mode 100644 index 00000000..ff78545f --- /dev/null +++ b/common/recipes-kernel/linux/linux-yocto-4.19.8/1217-drm-amd-display-Don-t-re-program-planes-for-DPMS-cha.patch @@ -0,0 +1,62 @@ +From 8cb4f28670ef1875259198e0cedf446996d611ba Mon Sep 17 00:00:00 2001 +From: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com> +Date: Wed, 23 Jan 2019 14:55:58 -0500 +Subject: [PATCH 1217/2940] drm/amd/display: Don't re-program planes for DPMS + changes + +[Why] +There are opt1c lock warnings and CRTC read timeouts when running the +"igt@kms_plane@plane-position-hole-dpms-pipe-*" tests. These are +caused by trying to reprogram planes that are not in the current +context. + +DPMS off removes the stream from the context. In this case: + +new_crtc_state->active_changed = true +new_crtc_state->mode_changed = false + +The planes are reprogrammed before the stream is removed from the +context because stream_state->mode_changed = false. + +For DPMS adds the stream and planes back to the context: + +new_crtc_state->active_changed = true +new_crtc_state->mode_changed = false + +The planes are also reprogrammed here before the stream is added to the +context because stream_state->mode_changed = true. They were not +previously in the current context so warnings occur here. + +[How] +Set stream_state->mode_changed = true when +new_crtc_state->active_changed = true too. + +This prevents reprogramming before the context is applied in DC. The +programming will be done after the context is applied. + +Change-Id: I3b405f3081d3ecca68b40e894e681ed0372e92df +Signed-off-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com> +Reviewed-by: Sun peng Li <Sunpeng.Li@amd.com> +Acked-by: Bhawanpreet Lakha <Bhawanpreet.Lakha@amd.com> +Acked-by: Tony Cheng <Tony.Cheng@amd.com> +--- + drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 ++- + 1 file changed, 2 insertions(+), 1 deletion(-) + +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 234ecae1ae83..d30c7ffa5561 100644 +--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c ++++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c +@@ -5007,7 +5007,8 @@ static void amdgpu_dm_commit_planes(struct drm_atomic_state *state, + static void amdgpu_dm_crtc_copy_transient_flags(struct drm_crtc_state *crtc_state, + struct dc_stream_state *stream_state) + { +- stream_state->mode_changed = crtc_state->mode_changed; ++ stream_state->mode_changed = ++ crtc_state->mode_changed || crtc_state->active_changed; + } + + static int amdgpu_dm_atomic_commit(struct drm_device *dev, +-- +2.17.1 + |