Return early for !icl_is_hdr_plane() to reduce indent.

Signed-off-by: Jani Nikula <[email protected]>

---

'git show -w' for easy review
---
 drivers/gpu/drm/i915/display/intel_color.c | 63 +++++++++++-----------
 1 file changed, 32 insertions(+), 31 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_color.c 
b/drivers/gpu/drm/i915/display/intel_color.c
index cf910baa69f3..429eb6daa7d2 100644
--- a/drivers/gpu/drm/i915/display/intel_color.c
+++ b/drivers/gpu/drm/i915/display/intel_color.c
@@ -3970,45 +3970,46 @@ xelpd_program_plane_pre_csc_lut(struct intel_dsb *dsb,
        int i, lut_size = 128;
        u32 lut_val;
 
-       if (icl_is_hdr_plane(display, plane)) {
-               intel_de_write_dsb(display, dsb,
-                                  PLANE_PRE_CSC_GAMC_INDEX_ENH(pipe, plane, 0),
-                                  PLANE_PAL_PREC_AUTO_INCREMENT);
+       if (!icl_is_hdr_plane(display, plane))
+               return;
 
-               if (pre_csc_lut) {
-                       for (i = 0; i < lut_size; i++) {
-                               lut_val = 
drm_color_lut32_extract(pre_csc_lut[i].green, 24);
+       intel_de_write_dsb(display, dsb,
+                          PLANE_PRE_CSC_GAMC_INDEX_ENH(pipe, plane, 0),
+                          PLANE_PAL_PREC_AUTO_INCREMENT);
 
-                               intel_de_write_dsb(display, dsb,
-                                                  
PLANE_PRE_CSC_GAMC_DATA_ENH(pipe, plane, 0),
-                                                  lut_val);
-                       }
+       if (pre_csc_lut) {
+               for (i = 0; i < lut_size; i++) {
+                       lut_val = drm_color_lut32_extract(pre_csc_lut[i].green, 
24);
 
-                       /* Program the max register to clamp values > 1.0. */
-                       /* TODO: Restrict to 0x7ffffff */
-                       do {
-                               intel_de_write_dsb(display, dsb,
-                                                  
PLANE_PRE_CSC_GAMC_DATA_ENH(pipe, plane, 0),
-                                                  lut_val);
-                       } while (i++ < 130);
-               } else {
-                       for (i = 0; i < lut_size; i++) {
-                               lut_val = (i * ((1 << 24) - 1)) / (lut_size - 
1);
+                       intel_de_write_dsb(display, dsb,
+                                          PLANE_PRE_CSC_GAMC_DATA_ENH(pipe, 
plane, 0),
+                                          lut_val);
+               }
 
-                               intel_de_write_dsb(display, dsb,
-                                                  
PLANE_PRE_CSC_GAMC_DATA_ENH(pipe, plane, 0),
-                                                  lut_val);
-                       }
+               /* Program the max register to clamp values > 1.0. */
+               /* TODO: Restrict to 0x7ffffff */
+               do {
+                       intel_de_write_dsb(display, dsb,
+                                          PLANE_PRE_CSC_GAMC_DATA_ENH(pipe, 
plane, 0),
+                                          lut_val);
+               } while (i++ < 130);
+       } else {
+               for (i = 0; i < lut_size; i++) {
+                       lut_val = (i * ((1 << 24) - 1)) / (lut_size - 1);
 
-                       do {
-                               intel_de_write_dsb(display, dsb,
-                                                  
PLANE_PRE_CSC_GAMC_DATA_ENH(pipe, plane, 0),
-                                                  1 << 24);
-                       } while (i++ < 130);
+                       intel_de_write_dsb(display, dsb,
+                                          PLANE_PRE_CSC_GAMC_DATA_ENH(pipe, 
plane, 0),
+                                          lut_val);
                }
 
-               intel_de_write_dsb(display, dsb, 
PLANE_PRE_CSC_GAMC_INDEX_ENH(pipe, plane, 0), 0);
+               do {
+                       intel_de_write_dsb(display, dsb,
+                                          PLANE_PRE_CSC_GAMC_DATA_ENH(pipe, 
plane, 0),
+                                          1 << 24);
+               } while (i++ < 130);
        }
+
+       intel_de_write_dsb(display, dsb, PLANE_PRE_CSC_GAMC_INDEX_ENH(pipe, 
plane, 0), 0);
 }
 
 static void
-- 
2.47.3

Reply via email to