From: Sung Lee <sung....@amd.com>

[WHY]
Failing validation when building scaling parameters causes corruption to
occur due to pipe splitting with smaller pixel widths than HW supports.
This needs to fail silently for now to hide the corruption until the
corruption itself can be fixed.

[HOW]
Do not fail validation if building scaling params fails.

Signed-off-by: Sung Lee <sung....@amd.com>
Reviewed-by: Dmytro Laktyushkin <dmytro.laktyush...@amd.com>
Acked-by: Rodrigo Siqueira <rodrigo.sique...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c 
b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c
index 4190ee592e6d..d00de61ac720 100644
--- a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c
+++ b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c
@@ -2824,8 +2824,8 @@ bool dcn20_fast_validate_bw(
                                        dcn20_split_stream_for_mpc(
                                                        &context->res_ctx, 
dc->res_pool,
                                                        pipe, hsplit_pipe);
-                                       if 
(!resource_build_scaling_params(pipe) || 
!resource_build_scaling_params(hsplit_pipe))
-                                               goto validate_fail;
+                                       resource_build_scaling_params(pipe);
+                                       
resource_build_scaling_params(hsplit_pipe);
                                }
                                pipe_split_from[hsplit_pipe->pipe_idx] = 
pipe_idx;
                        }
-- 
2.26.2

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to