The pipe BPP value shouldn't be set outside of the source's / sink's
valid pipe BPP range, ensure this when increasing the minimum pipe BPP
value to 30 due to HDR.

Fixes: ba49a4643cf53 ("drm/i915/dp: Set min_bpp limit to 30 in HDR mode")
Cc: Chaitanya Kumar Borah <[email protected]>
Cc: <[email protected]> # v6.18+
Signed-off-by: Imre Deak <[email protected]>
---
 drivers/gpu/drm/i915/display/intel_dp.c | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_dp.c 
b/drivers/gpu/drm/i915/display/intel_dp.c
index 2b8f43e211741..4d8f480cf803f 100644
--- a/drivers/gpu/drm/i915/display/intel_dp.c
+++ b/drivers/gpu/drm/i915/display/intel_dp.c
@@ -2697,6 +2697,7 @@ intel_dp_compute_config_limits(struct intel_dp *intel_dp,
                               bool dsc,
                               struct link_config_limits *limits)
 {
+       struct intel_display *display = to_intel_display(intel_dp);
        bool is_mst = intel_crtc_has_type(crtc_state, INTEL_OUTPUT_DP_MST);
        struct intel_connector *connector =
                to_intel_connector(conn_state->connector);
@@ -2709,8 +2710,7 @@ intel_dp_compute_config_limits(struct intel_dp *intel_dp,
        limits->min_lane_count = intel_dp_min_lane_count(intel_dp);
        limits->max_lane_count = intel_dp_max_lane_count(intel_dp);
 
-       limits->pipe.min_bpp = intel_dp_in_hdr_mode(conn_state) ? 30 :
-                               intel_dp_min_bpp(crtc_state->output_format);
+       limits->pipe.min_bpp = intel_dp_min_bpp(crtc_state->output_format);
        if (is_mst) {
                /*
                 * FIXME: If all the streams can't fit into the link with their
@@ -2726,6 +2726,16 @@ intel_dp_compute_config_limits(struct intel_dp *intel_dp,
                                                        
respect_downstream_limits);
        }
 
+       if (intel_dp_in_hdr_mode(conn_state)) {
+               if (limits->pipe.min_bpp <= 30 && limits->pipe.max_bpp >= 30)
+                       limits->pipe.min_bpp = 30;
+               else
+                       drm_dbg_kms(display->drm,
+                                   "[CONNECTOR:%d:%s] HDR min 30 bpp outside 
of valid pipe bpp range (%d-%d)\n",
+                                   connector->base.base.id, 
connector->base.name,
+                                   limits->pipe.min_bpp, limits->pipe.max_bpp);
+       }
+
        if (dsc && !intel_dp_dsc_compute_pipe_bpp_limits(connector, limits))
                return false;
 
-- 
2.49.1

Reply via email to