From: "Leo (Hanghong) Ma" <[email protected]>

[Why && How]
Previous change for Coverity has caused regression on visual confirm
so fix it by reverting the part that affects visual confirm.

Reviewed-by: Chris Park <[email protected]>
Signed-off-by: Leo (Hanghong) Ma <[email protected]>
Signed-off-by: Roman Li <[email protected]>
---
 drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c 
b/drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c
index 9291f078ffde..8486c144d81c 100644
--- a/drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c
+++ b/drivers/gpu/drm/amd/display/dc/dc_dmub_srv.c
@@ -519,7 +519,8 @@ void dc_dmub_srv_get_visual_confirm_color_cmd(struct dc 
*dc, struct pipe_ctx *pi
        union dmub_rb_cmd cmd = { 0 };
        unsigned int panel_inst = 0;
 
-       if (!dc_get_edp_link_panel_inst(dc, pipe_ctx->stream->link, 
&panel_inst))
+       if (!dc_get_edp_link_panel_inst(dc, pipe_ctx->stream->link, 
&panel_inst) &&
+                       dc->debug.visual_confirm == VISUAL_CONFIRM_DISABLE)
                return;
 
        memset(&cmd, 0, sizeof(cmd));
-- 
2.34.1

Reply via email to