diff options
| author | Jani Nikula <jani.nikula@intel.com> | 2026-06-03 18:14:28 +0300 |
|---|---|---|
| committer | Jani Nikula <jani.nikula@intel.com> | 2026-06-10 12:03:54 +0300 |
| commit | e9e982ff338ebf30f7461b9db9a987e56bc22d3e (patch) | |
| tree | 331f054a00ff564615d52abadc2d2cc3181b6f97 | |
| parent | 88b30ff4bc52f959e46599b14ab16674e8ebfde8 (diff) | |
drm/i915/color: clean up variables in xelpd_program_plane_pre_csc_lut()
Use plain int for counting. Initialize lut_size at declaration. Remove
unnecessary lut_val initialization. Remove extra u32 v and just use
lut_val.
Reviewed-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Link: https://patch.msgid.link/92d8a508ab18dfd33b6e5573e6edf433f1bbd321.1780499355.git.jani.nikula@intel.com
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
| -rw-r--r-- | drivers/gpu/drm/i915/display/intel_color.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/drivers/gpu/drm/i915/display/intel_color.c b/drivers/gpu/drm/i915/display/intel_color.c index 458508bcf1f4..73ab879915c3 100644 --- a/drivers/gpu/drm/i915/display/intel_color.c +++ b/drivers/gpu/drm/i915/display/intel_color.c @@ -3967,12 +3967,10 @@ xelpd_program_plane_pre_csc_lut(struct intel_dsb *dsb, enum pipe pipe = to_intel_plane(state->plane)->pipe; enum plane_id plane = to_intel_plane(state->plane)->id; const struct drm_color_lut32 *pre_csc_lut = plane_state->hw.degamma_lut->data; - u32 i, lut_size; - u32 lut_val = 1 << 24; + int i, lut_size = 128; + u32 lut_val; if (icl_is_hdr_plane(display, plane)) { - lut_size = 128; - intel_de_write_dsb(display, dsb, PLANE_PRE_CSC_GAMC_INDEX_ENH(pipe, plane, 0), PLANE_PAL_PREC_AUTO_INCREMENT); @@ -3995,10 +3993,11 @@ xelpd_program_plane_pre_csc_lut(struct intel_dsb *dsb, } while (i++ < 130); } else { for (i = 0; i < lut_size; i++) { - u32 v = (i * ((1 << 24) - 1)) / (lut_size - 1); + lut_val = (i * ((1 << 24) - 1)) / (lut_size - 1); intel_de_write_dsb(display, dsb, - PLANE_PRE_CSC_GAMC_DATA_ENH(pipe, plane, 0), v); + PLANE_PRE_CSC_GAMC_DATA_ENH(pipe, plane, 0), + lut_val); } do { |