drm/i915/dp: Fail state computation for invalid max throughput BPP value

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.

Reviewed-by: Vinod Govindapillai <vinod.govindapillai@intel.com>
Reviewed-by: Luca Coelho <luciano.coelho@intel.com>
Signed-off-by: Imre Deak <imre.deak@intel.com>
Link: https://patch.msgid.link/20251215192357.172201-15-imre.deak@intel.com

diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c
index a89c9eec62b3307ec96ebdb309cdb96909f4791a..206184e6ba66bf22d5a975c941eb3c0ad4e952d0 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -2630,8 +2630,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;