diff options
| author | Jani Nikula <jani.nikula@intel.com> | 2026-06-03 18:14:35 +0300 |
|---|---|---|
| committer | Jani Nikula <jani.nikula@intel.com> | 2026-06-10 12:08:13 +0300 |
| commit | 5de2ad00a044d3fe85b46eb33a1d911d6eb88567 (patch) | |
| tree | 8100630e427bb9141fbc7c7966d049dce6dfb0fb /drivers | |
| parent | bbc23003e00517144ce65901e5f74ea2c0ceb5d3 (diff) | |
| download | linux-next-5de2ad00a044d3fe85b46eb33a1d911d6eb88567.tar.gz linux-next-5de2ad00a044d3fe85b46eb33a1d911d6eb88567.zip | |
drm/i915/color: deduplicate loops in xelpd_program_plane_post_csc_lut()
Now that the pre_csc_lut and non-pre_csc_lut paths look similar,
deduplicate the loops and just determine the value based on pre_csc_lut
vs. not.
Reviewed-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Link: https://patch.msgid.link/4b3f55002a1c5b7067ccd6688d802fc284410e03.1780499355.git.jani.nikula@intel.com
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/gpu/drm/i915/display/intel_color.c | 22 |
1 files changed, 8 insertions, 14 deletions
diff --git a/drivers/gpu/drm/i915/display/intel_color.c b/drivers/gpu/drm/i915/display/intel_color.c index d705f745bd7d..87ced9f6ff40 100644 --- a/drivers/gpu/drm/i915/display/intel_color.c +++ b/drivers/gpu/drm/i915/display/intel_color.c @@ -4017,28 +4017,22 @@ xelpd_program_plane_post_csc_lut(struct intel_dsb *dsb, /* TODO: Add macro */ intel_de_write_dsb(display, dsb, PLANE_POST_CSC_GAMC_SEG0_INDEX_ENH(pipe, plane, 0), PLANE_PAL_PREC_AUTO_INCREMENT); - if (post_csc_lut) { - for (i = 0; i < lut_size + 3; i++) { + + for (i = 0; i < lut_size + 3; i++) { + if (post_csc_lut) { if (i < lut_size) lut_val = drm_color_lut32_extract(post_csc_lut[i].green, 24); /* else clamp to the last LUT value to prevent step discontinuity */ - - intel_de_write_dsb(display, dsb, - PLANE_POST_CSC_GAMC_DATA_ENH(pipe, plane, 0), - lut_val); - } - } else { - /*TODO: Add for segment 0 */ - for (i = 0; i < lut_size + 3; i++) { + } else { if (i < lut_size) lut_val = (i * ((1 << 24) - 1)) / (lut_size - 1); else lut_val = 1 << 24; - - intel_de_write_dsb(display, dsb, - PLANE_POST_CSC_GAMC_DATA_ENH(pipe, plane, 0), - lut_val); } + + intel_de_write_dsb(display, dsb, + PLANE_POST_CSC_GAMC_DATA_ENH(pipe, plane, 0), + lut_val); } intel_de_write_dsb(display, dsb, PLANE_POST_CSC_GAMC_INDEX_ENH(pipe, plane, 0), 0); |