On Thu, 2025-11-27 at 19:49 +0200, Imre Deak wrote: > There is no reason to accept an invalid maximum sink compressed BPP > value (i.e. 0), fail the state computation in this case. > > Signed-off-by: Imre Deak <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_dp.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) >
Reviewed-by: Vinod Govindapillai <[email protected]> > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c > b/drivers/gpu/drm/i915/display/intel_dp.c > index f97ee8265836a..db7e49c17ca8d 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -2631,8 +2631,7 @@ intel_dp_compute_config_link_bpp_limits(struct > intel_dp *intel_dp, > dsc_sink_max_bpp = > intel_dp_dsc_sink_max_compressed_bpp(connector, > > crtc_state, > > limits->pipe.max_bpp / 3); > - dsc_max_bpp = dsc_sink_max_bpp ? > - min(dsc_sink_max_bpp, dsc_src_max_bpp) > : dsc_src_max_bpp; > + dsc_max_bpp = min(dsc_sink_max_bpp, > dsc_src_max_bpp); > dsc_max_bpp = min(dsc_max_bpp, joiner_max_bpp); > > max_link_bpp_x16 = min(max_link_bpp_x16, > fxp_q4_from_int(dsc_max_bpp));
