summary refs log tree commit diff
path: root/drivers/gpu
diff options
context:
space:
mode:
authorJoshua Ashton <joshua@froggi.es>2023-11-02 07:32:16 +0000
committerPierre-Loup A. Griffais <pgriffais@valvesoftware.com>2023-11-02 16:08:21 -0700
commita2a64d6e93fc63f9a12294399a520c5abd207484 (patch)
treeb453299d62e534671adcae9fd9a4e4f735f5f73f /drivers/gpu
parent01d948a85d4664a07571913e38479978ad2da5d1 (diff)
downloadlinux-a2a64d6e93fc63f9a12294399a520c5abd207484.tar.gz
drm/amd/display: Remove erroneous color_mgmt_changed check in amdgpu_dm_plane_set_color_properties
Pure luck we didn't run into this before.

Could be a newly allocated dc_plane_state.

Signed-off-by: Joshua Ashton <joshua@froggi.es>
Diffstat (limited to 'drivers/gpu')
-rw-r--r--drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_color.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_color.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_color.c
index c84692612bd2..86a2d449cf07 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_color.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_color.c
@@ -1031,8 +1031,15 @@ amdgpu_dm_plane_set_color_properties(struct drm_plane_state *plane_state,
 	int ret;
 
 	/* We have nothing to do here, return */
+	/*
+	 * JoshA: WE HAVE TO DO THIS EVERY TIME.
+	 * It's on a new dc_plane_state allocation, none of this data is here!
+	 * !!!!!!!!
+	 * This was always true before we duped properties due to LUCK and the
+	 * properties matching.
 	if (!plane_state->color_mgmt_changed)
 		return 0;
+	*/
 
 	dc_plane_state->hdr_mult = dc_fixpt_from_s3132(dm_plane_state->hdr_mult);