On Thu, 2025-11-27 at 19:49 +0200, Imre Deak wrote: > There is no reason to accept a minimum/maximum link BPP value above the > maximum throughput BPP value, fail the state computation in this case. > > Signed-off-by: Imre Deak <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_dp.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c > b/drivers/gpu/drm/i915/display/intel_dp.c > index 545d872a30403..f97ee8265836a 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -2638,8 +2638,6 @@ intel_dp_compute_config_link_bpp_limits(struct intel_dp > *intel_dp, > max_link_bpp_x16 = min(max_link_bpp_x16, > fxp_q4_from_int(dsc_max_bpp)); > > throughput_max_bpp_x16 = > dsc_throughput_quirk_max_bpp_x16(connector, crtc_state); > - throughput_max_bpp_x16 = clamp(throughput_max_bpp_x16, > - limits->link.min_bpp_x16, > max_link_bpp_x16); > if (throughput_max_bpp_x16 < max_link_bpp_x16) { > max_link_bpp_x16 = throughput_max_bpp_x16; >
Reviewed-by: Luca Coelho <[email protected]> -- Cheers, Luca.
