From: Mario Limonciello <mario.limoncie...@amd.com>

mark_seamless_boot_stream() can be called multiple times to run
the more expensive checks in dc_validate_boot_timing().

Refactor the function so that if those have already passed once
the function isn't called again.

Also add a message the first time that they have passed to let
the user know the stream will be used for seamless boot.

Signed-off-by: Mario Limonciello <mario.limoncie...@amd.com>
---
 .../gpu/drm/amd/display/dc/core/dc_resource.c  | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c 
b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
index 626f75b6ad003..78bad889ba065 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
@@ -3525,16 +3525,20 @@ static int acquire_resource_from_hw_enabled_state(
        return -1;
 }
 
-static void mark_seamless_boot_stream(
-               const struct dc  *dc,
-               struct dc_stream_state *stream)
+static void mark_seamless_boot_stream(const struct dc  *dc,
+                                     struct dc_stream_state *stream)
 {
        struct dc_bios *dcb = dc->ctx->dc_bios;
 
-       if (dc->config.allow_seamless_boot_optimization &&
-                       !dcb->funcs->is_accelerated_mode(dcb)) {
-               if (dc_validate_boot_timing(dc, stream->sink, &stream->timing))
-                       stream->apply_seamless_boot_optimization = true;
+       if (stream->apply_seamless_boot_optimization)
+               return;
+       if (!dc->config.allow_seamless_boot_optimization)
+               return;
+       if (dcb->funcs->is_accelerated_mode(dcb))
+               return;
+       if (dc_validate_boot_timing(dc, stream->sink, &stream->timing)) {
+               stream->apply_seamless_boot_optimization = true;
+               DC_LOG_INFO("Marked stream for seamless boot optimization\n");
        }
 }
 
-- 
2.43.0

Reply via email to