OSDN Git Service

drm/amd/display: Fix merging dynamic ODM+MPO configs on DCN32
authorDillon Varone <Dillon.Varone@amd.com>
Fri, 23 Sep 2022 18:00:09 +0000 (14:00 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 6 Oct 2022 16:02:17 +0000 (12:02 -0400)
[WHY&HOW?]
When merging ODM pipes that are using MPO, we must copy the stream_res
from the new top pipe to the bottom pipe so that the overlayed plane is
not pointing to the wrong stream assets.

Reviewed-by: Martin Leung <Martin.Leung@amd.com>
Reviewed-by: Jun Lei <Jun.Lei@amd.com>
Acked-by: Hamza Mahfooz <hamza.mahfooz@amd.com>
Signed-off-by: Dillon Varone <Dillon.Varone@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dml/dcn32/dcn32_fpu.c

index a56ee04..f3f98e9 100644 (file)
@@ -1598,6 +1598,9 @@ bool dcn32_internal_validate_bw(struct dc *dc,
                                        /*MPC split rules will handle this case*/
                                        pipe->bottom_pipe->top_pipe = NULL;
                                } else {
+                                       /* when merging an ODM pipes, the bottom MPC pipe must now point to
+                                        * the previous ODM pipe and its associated stream assets
+                                        */
                                        if (pipe->prev_odm_pipe->bottom_pipe) {
                                                /* 3 plane MPO*/
                                                pipe->bottom_pipe->top_pipe = pipe->prev_odm_pipe->bottom_pipe;
@@ -1607,6 +1610,8 @@ bool dcn32_internal_validate_bw(struct dc *dc,
                                                pipe->bottom_pipe->top_pipe = pipe->prev_odm_pipe;
                                                pipe->prev_odm_pipe->bottom_pipe = pipe->bottom_pipe;
                                        }
+
+                                       memcpy(&pipe->bottom_pipe->stream_res, &pipe->bottom_pipe->top_pipe->stream_res, sizeof(struct stream_resource));
                                }
                        }