This patch fixes the naming of the registers:

s/PIPE_DSS_CTL/ICL_PIPE_DSS_CTL

And also fix the hex values to lower case, to match
rest of the definitions.

Manasi noticed this with the patch that was merged.

Fixes - 8b1b558d690aa (drm/i915/icl: Add DSS_CTL Registers)
Suggested-by: Manasi Navare <manasi.d.nav...@intel.com>
Cc: Jani Nikula <jani.nik...@intel.com>
Cc: Manasi Navare <manasi.d.nav...@intel.com>
Signed-off-by: Anusha Srivatsa <anusha.sriva...@intel.com>
---
 drivers/gpu/drm/i915/i915_reg.h | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index aef1a30ff9f6..c0e6e14fe9fa 100644
--- a/drivers/gpu/drm/i915/i915_reg.h
+++ b/drivers/gpu/drm/i915/i915_reg.h
@@ -10050,7 +10050,7 @@ enum skl_power_gate {
 #define  OVERLAP_PIXELS(pixels)                        ((pixels) << 16)
 #define  LEFT_DL_BUF_TARGET_DEPTH_MASK         (0xfff << 0)
 #define  LEFT_DL_BUF_TARGET_DEPTH(pixels)      ((pixels) << 0)
-#define  MAX_DL_BUFFER_TARGET_DEPTH            0x5A0
+#define  MAX_DL_BUFFER_TARGET_DEPTH            0x5a0
 
 #define DSS_CTL2                               _MMIO(0x67404)
 #define  LEFT_BRANCH_VDSC_ENABLE               (1 << 31)
@@ -10058,20 +10058,20 @@ enum skl_power_gate {
 #define  RIGHT_DL_BUF_TARGET_DEPTH_MASK                (0xfff << 0)
 #define  RIGHT_DL_BUF_TARGET_DEPTH(pixels)     ((pixels) << 0)
 
-#define _PIPE_DSS_CTL1_PB                      0x78200
-#define _PIPE_DSS_CTL1_PC                      0x78400
-#define PIPE_DSS_CTL1(pipe)                    _MMIO_PIPE((pipe) - PIPE_B, \
-                                                          _PIPE_DSS_CTL1_PB, \
-                                                          _PIPE_DSS_CTL1_PC)
+#define _ICL_PIPE_DSS_CTL1_PB                  0x78200
+#define _ICL_PIPE_DSS_CTL1_PC                  0x78400
+#define ICL_PIPE_DSS_CTL1(pipe)                        _MMIO_PIPE((pipe) - 
PIPE_B, \
+                                                          
_ICL_PIPE_DSS_CTL1_PB, \
+                                                          
_ICL_PIPE_DSS_CTL1_PC)
 #define  BIG_JOINER_ENABLE                     (1 << 29)
 #define  MASTER_BIG_JOINER_ENABLE              (1 << 28)
 #define  VGA_CENTERING_ENABLE                  (1 << 27)
 
-#define _PIPE_DSS_CTL2_PB                      0x78204
-#define _PIPE_DSS_CTL2_PC                      0x78404
-#define PIPE_DSS_CTL2(pipe)                    _MMIO_PIPE((pipe) - PIPE_B, \
-                                                          _PIPE_DSS_CTL2_PB, \
-                                                          _PIPE_DSS_CTL2_PC)
+#define _ICL_PIPE_DSS_CTL2_PB                  0x78204
+#define _ICL_PIPE_DSS_CTL2_PC                  0x78404
+#define ICL_PIPE_DSS_CTL2(pipe)                        _MMIO_PIPE((pipe) - 
PIPE_B, \
+                                                          
_ICL_PIPE_DSS_CTL2_PB, \
+                                                          
_ICL_PIPE_DSS_CTL2_PC)
 
 #define BXT_P_DSI_REGULATOR_CFG                        _MMIO(0x160020)
 #define  STAP_SELECT                                   (1 << 0)
-- 
2.17.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to