There is no reason to accept an invalid maximum sink compressed BPP
value (i.e. 0), fail the state computation in this case.

Reviewed-by: Vinod Govindapillai <[email protected]>
Reviewed-by: Luca Coelho <[email protected]>
Signed-off-by: Imre Deak <[email protected]>
---
 drivers/gpu/drm/i915/display/intel_dp.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_dp.c 
b/drivers/gpu/drm/i915/display/intel_dp.c
index dc2486aa12010..074aa472d33c8 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -2622,8 +2622,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));
-- 
2.49.1

Reply via email to