[PATCH 17/26] drm/amd/display: Skip Invalid Streams from DSC Policy

[Rodrigo Siqueira]

From: Fangzhi Zuo <jerry....@amd.com>

Streams with invalid new connector state should be elimiated from
dsc policy.

Reviewed-by: Aurabindo Pillai <aurabindo.pil...@amd.com>
Signed-off-by: Fangzhi Zuo <jerry....@amd.com>
Signed-off-by: Rodrigo Siqueira <rodrigo.sique...@amd.com>
---
 .../drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
index 68bd8e6b948d..32b025c92c63 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
@@ -1120,6 +1120,7 @@ static int compute_mst_dsc_configs_for_link(struct 
drm_atomic_state *state,
        int i, k, ret;
        bool debugfs_overwrite = false;
        uint16_t fec_overhead_multiplier_x1000 = 
get_fec_overhead_multiplier(dc_link);
+       struct drm_connector_state *new_conn_state;
 
        memset(params, 0, sizeof(params));
 
@@ -1127,7 +1128,7 @@ static int compute_mst_dsc_configs_for_link(struct 
drm_atomic_state *state,
                return PTR_ERR(mst_state);
 
        /* Set up params */
-       DRM_DEBUG_DRIVER("%s: MST_DSC Set up params for %d streams\n", 
__func__, dc_state->stream_count);
+       DRM_DEBUG_DRIVER("%s: MST_DSC Try to set up params from %d streams\n", 
__func__, dc_state->stream_count);
        for (i = 0; i < dc_state->stream_count; i++) {
                struct dc_dsc_policy dsc_policy = {0};
 
@@ -1143,6 +1144,14 @@ static int compute_mst_dsc_configs_for_link(struct 
drm_atomic_state *state,
                if (!aconnector->mst_output_port)
                        continue;
 
+               new_conn_state = drm_atomic_get_new_connector_state(state, 
&aconnector->base);
+
+               if (!new_conn_state) {
+                       DRM_DEBUG_DRIVER("%s:%d MST_DSC Skip the stream 0x%p 
with invalid new_conn_state\n",
+                                       __func__, __LINE__, stream);
+                       continue;
+               }
+
                stream->timing.flags.DSC = 0;
 
                params[count].timing = &stream->timing;
@@ -1175,6 +1184,8 @@ static int compute_mst_dsc_configs_for_link(struct 
drm_atomic_state *state,
                count++;
        }
 
+       DRM_DEBUG_DRIVER("%s: MST_DSC Params set up for %d streams\n", 
__func__, count);
+
        if (count == 0) {
                ASSERT(0);
                return 0;
-- 
2.45.2