From: Jimmy Kizito <jimmy.kiz...@amd.com>

[Why]
When copying a stream, the encoder assigned to it is copied too.
Encoder assignment should only happen when executing the encoder
assignment function link_encs_assign().

[How]
Clear the link encoder pointer for copied stream.

Reviewed-by: Meenakshikumar Somasundaram <meenakshikumar.somasunda...@amd.com>
Reviewed-by: Jun Lei <jun....@amd.com>
Acked-by: Agustin Gutierrez Sanchez <agustin.gutier...@amd.com>
Signed-off-by: Jimmy Kizito <jimmy.kiz...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/core/dc_stream.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_stream.c 
b/drivers/gpu/drm/amd/display/dc/core/dc_stream.c
index f0f54f4d3d9b..57cf4cb82370 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc_stream.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_stream.c
@@ -202,6 +202,10 @@ struct dc_stream_state *dc_copy_stream(const struct 
dc_stream_state *stream)
        new_stream->stream_id = new_stream->ctx->dc_stream_id_count;
        new_stream->ctx->dc_stream_id_count++;
 
+       /* If using dynamic encoder assignment, wait till stream committed to 
assign encoder. */
+       if (new_stream->ctx->dc->res_pool->funcs->link_encs_assign)
+               new_stream->link_enc = NULL;
+
        kref_init(&new_stream->refcount);
 
        return new_stream;
-- 
2.25.1

Reply via email to