Swap the order of matrix and multiplier as designed in hardware.

Signed-off-by: Alex Hung <alex.h...@amd.com>
---
 .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_color.c  | 12 ++++++------
 .../drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c    |  8 ++++----
 2 files changed, 10 insertions(+), 10 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 edf05290d441..54ec12c1352f 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
@@ -1477,25 +1477,25 @@ amdgpu_dm_plane_set_colorop_properties(struct 
drm_plane_state *plane_state,
        if (ret)
                return ret;
 
-       /* 3x4 matrix */
+       /* Multiplier */
        colorop = colorop->next;
        if (!colorop) {
-               drm_dbg(dev, "no 3x4 matrix colorop found\n");
+               drm_dbg(dev, "no multiplier colorop found\n");
                return -EINVAL;
        }
 
-       ret = __set_dm_plane_colorop_3x4_matrix(plane_state, dc_plane_state, 
colorop);
+       ret = __set_dm_plane_colorop_multiplier(plane_state, dc_plane_state, 
colorop);
        if (ret)
                return ret;
 
-       /* Multiplier */
+       /* 3x4 matrix */
        colorop = colorop->next;
        if (!colorop) {
-               drm_dbg(dev, "no multiplier colorop found\n");
+               drm_dbg(dev, "no 3x4 matrix colorop found\n");
                return -EINVAL;
        }
 
-       ret = __set_dm_plane_colorop_multiplier(plane_state, dc_plane_state, 
colorop);
+       ret = __set_dm_plane_colorop_3x4_matrix(plane_state, dc_plane_state, 
colorop);
        if (ret)
                return ret;
 
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
index 33156531d751..d0777691862b 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_colorop.c
@@ -75,7 +75,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane 
*plane, struct drm_pr
 
        i++;
 
-       /* 3x4 matrix */
+       /* Multiplier */
        ops[i] = kzalloc(sizeof(struct drm_colorop), GFP_KERNEL);
        if (!ops[i]) {
                DRM_ERROR("KMS: Failed to allocate colorop\n");
@@ -83,7 +83,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane 
*plane, struct drm_pr
                goto cleanup;
        }
 
-       ret = drm_colorop_ctm_3x4_init(dev, ops[i], plane);
+       ret = drm_colorop_mult_init(dev, ops[i], plane);
        if (ret)
                goto cleanup;
 
@@ -91,7 +91,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane 
*plane, struct drm_pr
 
        i++;
 
-       /* Multiplier */
+       /* 3x4 matrix */
        ops[i] = kzalloc(sizeof(struct drm_colorop), GFP_KERNEL);
        if (!ops[i]) {
                DRM_ERROR("KMS: Failed to allocate colorop\n");
@@ -99,7 +99,7 @@ int amdgpu_dm_initialize_default_pipeline(struct drm_plane 
*plane, struct drm_pr
                goto cleanup;
        }
 
-       ret = drm_colorop_mult_init(dev, ops[i], plane);
+       ret = drm_colorop_ctm_3x4_init(dev, ops[i], plane);
        if (ret)
                goto cleanup;
 
-- 
2.43.0

Reply via email to