drm/amd/display: Add dm_logger_append_va API

Same as dm_logger_append, except it takes a va_list instead of a
variable number of arguments. dm_logger_append is now a minimal wrapper
around dm_logger_append_va.

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/display/dc/basics/logger.c b/drivers/gpu/drm/amd/display/dc/basics/logger.c
index 2ff5b46..180a9d6 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/basics/logger.c
+++ b/drivers/gpu/drm/amd/display/dc/basics/logger.c
@@ -311,6 +311,18 @@ void dm_logger_append(
        struct log_entry *entry,
        const char *msg,
        ...)
+{
+       va_list args;
+
+       va_start(args, msg);
+       dm_logger_append_va(entry, msg, args);
+       va_end(args);
+}
+
+void dm_logger_append_va(
+       struct log_entry *entry,
+       const char *msg,
+       va_list args)
 {
        struct dal_logger *logger;
 
@@ -325,11 +337,8 @@ void dm_logger_append(
                dal_logger_should_log(logger, entry->type)) {
 
                uint32_t size;
-               va_list args;
                char buffer[LOG_MAX_LINE_SIZE];
 
-               va_start(args, msg);
-
                size = dm_log_to_buffer(
                        buffer, LOG_MAX_LINE_SIZE, msg, args);
 
@@ -338,8 +347,6 @@ void dm_logger_append(
                } else {
                        append_entry(entry, "LOG_ERROR, line too long\n", 27);
                }
-
-               va_end(args);
        }
 }
 

diff --git a/drivers/gpu/drm/amd/display/include/logger_interface.h b/drivers/gpu/drm/amd/display/include/logger_interface.h
index 8e1fe70..28dee96 100644 (file)
--- a/drivers/gpu/drm/amd/display/include/logger_interface.h
+++ b/drivers/gpu/drm/amd/display/include/logger_interface.h
@@ -57,6 +57,11 @@ void dm_logger_append(
                const char *msg,
                ...);
 
+void dm_logger_append_va(
+               struct log_entry *entry,
+               const char *msg,
+               va_list args);
+
 void dm_logger_open(
                struct dal_logger *logger,
                struct log_entry *entry,