From: Wayne Lin <wayne....@amd.com>

[Why]
For mst streams under same topology, stream->link->link_enc_hw_inst are the 
same and
hence can't distinguish the crc window setting.

[How]
Firstly adjust dc_stream_forward_crc_window to accept assignment of phy_id. 
Follow up
another patch to determine the phy_id at dm layer.

Reviewed-by: HaoPing Liu <haoping....@amd.com>
Signed-off-by: Wayne Lin <wayne....@amd.com>
Signed-off-by: Aurabindo Pillai <aurabindo.pil...@amd.com>
---
 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c | 6 ++++--
 drivers/gpu/drm/amd/display/dc/core/dc.c              | 4 ++--
 drivers/gpu/drm/amd/display/dc/dc_stream.h            | 1 +
 3 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c
index f936a35fa9eb..2679ce9a0980 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c
@@ -89,6 +89,7 @@ static void amdgpu_dm_set_crc_window_default(struct drm_crtc 
*crtc, struct dc_st
        struct amdgpu_display_manager *dm = &drm_to_adev(drm_dev)->dm;
        struct amdgpu_crtc *acrtc = to_amdgpu_crtc(crtc);
        bool was_activated;
+       uint8_t phy_id = stream->link->link_enc_hw_inst;
 
        spin_lock_irq(&drm_dev->event_lock);
        was_activated = acrtc->dm_irq_params.window_param.activated;
@@ -106,7 +107,7 @@ static void amdgpu_dm_set_crc_window_default(struct 
drm_crtc *crtc, struct dc_st
                /* stop ROI update on this crtc */
                
flush_work(&dm->secure_display_ctxs[crtc->index].notify_ta_work);
                
flush_work(&dm->secure_display_ctxs[crtc->index].forward_roi_work);
-               dc_stream_forward_crc_window(stream, NULL, true);
+               dc_stream_forward_crc_window(stream, NULL, phy_id, true);
        }
 }
 
@@ -175,7 +176,8 @@ amdgpu_dm_forward_crc_window(struct work_struct *work)
        stream = to_amdgpu_crtc(crtc)->dm_irq_params.stream;
 
        mutex_lock(&dm->dc_lock);
-       dc_stream_forward_crc_window(stream, &secure_display_ctx->rect, false);
+       dc_stream_forward_crc_window(stream, &secure_display_ctx->rect,
+               stream->link->link_enc_hw_inst, false);
        mutex_unlock(&dm->dc_lock);
 }
 
diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c 
b/drivers/gpu/drm/amd/display/dc/core/dc.c
index dc14c0d05e8f..d3bcb646545c 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -579,7 +579,7 @@ dc_stream_forward_dmcu_crc_window(struct dmcu *dmcu,
 
 bool
 dc_stream_forward_crc_window(struct dc_stream_state *stream,
-               struct rect *rect, bool is_stop)
+               struct rect *rect, uint8_t phy_id, bool is_stop)
 {
        struct dmcu *dmcu;
        struct dc_dmub_srv *dmub_srv;
@@ -598,7 +598,7 @@ dc_stream_forward_crc_window(struct dc_stream_state *stream,
        if (i == MAX_PIPES)
                return false;
 
-       mux_mapping.phy_output_num = stream->link->link_enc_hw_inst;
+       mux_mapping.phy_output_num = phy_id;
        mux_mapping.otg_output_num = pipe->stream_res.tg->inst;
 
        dmcu = dc->res_pool->dmcu;
diff --git a/drivers/gpu/drm/amd/display/dc/dc_stream.h 
b/drivers/gpu/drm/amd/display/dc/dc_stream.h
index 413970588a26..c46fe603c665 100644
--- a/drivers/gpu/drm/amd/display/dc/dc_stream.h
+++ b/drivers/gpu/drm/amd/display/dc/dc_stream.h
@@ -541,6 +541,7 @@ bool dc_stream_get_crtc_position(struct dc *dc,
 #if defined(CONFIG_DRM_AMD_SECURE_DISPLAY)
 bool dc_stream_forward_crc_window(struct dc_stream_state *stream,
                struct rect *rect,
+               uint8_t phy_id,
                bool is_stop);
 #endif
 
-- 
2.47.1

Reply via email to