From: Dillon Varone <[email protected]>

To prevent invalid HW programming, streams should be revalidated first
before committing to HW.

Reviewed-by: Aric Cyr <[email protected]>
Signed-off-by: Dillon Varone <[email protected]>
Signed-off-by: Rodrigo Siqueira <[email protected]>
---
 drivers/gpu/drm/amd/display/dc/core/dc.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c 
b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 81f5996f2edb..dfa36368ae63 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -2153,6 +2153,11 @@ enum dc_status dc_commit_streams(struct dc *dc, struct 
dc_commit_streams_params
                struct dc_stream_state *stream = params->streams[i];
                struct dc_stream_status *status = dc_stream_get_status(stream);
 
+               /* revalidate streams */
+               res = dc_validate_stream(dc, stream);
+               if (res != DC_OK)
+                       return res;
+
                dc_stream_log(dc, stream);
 
                set[i].stream = stream;
-- 
2.45.2

Reply via email to