On Wed, Oct 09, 2024 at 12:07:53PM +0530, Suraj Kandpal wrote: > Use the blob variable instead of post_csc_lut as it may end up > being null.
Not possible. We always have a LUT for each half in split gamma mode, and if we don't then someone screwed up much earlier before we end up here. > > Signed-off-by: Suraj Kandpal <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_color.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_color.c > b/drivers/gpu/drm/i915/display/intel_color.c > index caf1af039960..22b7090c4f6f 100644 > --- a/drivers/gpu/drm/i915/display/intel_color.c > +++ b/drivers/gpu/drm/i915/display/intel_color.c > @@ -1503,7 +1503,7 @@ static void ivb_load_luts(const struct intel_crtc_state > *crtc_state) > ivb_load_lut_10(crtc_state, pre_csc_lut, PAL_PREC_SPLIT_MODE | > PAL_PREC_INDEX_VALUE(0)); > ivb_load_lut_ext_max(crtc_state); > - ivb_load_lut_10(crtc_state, post_csc_lut, PAL_PREC_SPLIT_MODE | > + ivb_load_lut_10(crtc_state, blob, PAL_PREC_SPLIT_MODE | > PAL_PREC_INDEX_VALUE(512)); > break; > case GAMMA_MODE_MODE_10BIT: > @@ -1531,7 +1531,7 @@ static void bdw_load_luts(const struct intel_crtc_state > *crtc_state) > bdw_load_lut_10(crtc_state, pre_csc_lut, PAL_PREC_SPLIT_MODE | > PAL_PREC_INDEX_VALUE(0)); > ivb_load_lut_ext_max(crtc_state); > - bdw_load_lut_10(crtc_state, post_csc_lut, PAL_PREC_SPLIT_MODE | > + bdw_load_lut_10(crtc_state, blob, PAL_PREC_SPLIT_MODE | > PAL_PREC_INDEX_VALUE(512)); > break; > case GAMMA_MODE_MODE_10BIT: > -- > 2.43.2 -- Ville Syrjälä Intel
