Antonio Vázquez Blanco has uploaded this change for review.

View Change

Split cli_output declarations to a separate header.

This is a simple refactor that aims to simplify maintenance and to clarify file dependency inside the project.
Currently, many declarations reside in flash.h making it difficult to really understand file dependency.

Change-Id: I45d62bd219bdbed919788ae17a64aeb119a8aac4
Signed-off-by: Antonio Vázquez Blanco <antoniovazquezblanco@gmail.com>
---
M cli_classic.c
M cli_output.c
A include/cli_output.h
M include/flash.h
4 files changed, 25 insertions(+), 9 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/85/84985/1
diff --git a/cli_classic.c b/cli_classic.c
index 3343438..1669076 100644
--- a/cli_classic.c
+++ b/cli_classic.c
@@ -25,6 +25,7 @@
#include <stdbool.h>
#include <stdlib.h>
#include <cli_classic.h>
+#include "cli_output.h"
#include "flash.h"
#include "flashchips.h"
#include "fmap.h"
diff --git a/cli_output.c b/cli_output.c
index e5b829a..cf96bc0 100644
--- a/cli_output.c
+++ b/cli_output.c
@@ -15,11 +15,12 @@
* GNU General Public License for more details.
*/

+#include "cli_output.h"
+
#include <stdio.h>
#include <stdarg.h>
#include <string.h>
#include <errno.h>
-#include "flash.h"

enum flashrom_log_level verbose_screen = FLASHROM_MSG_INFO;
enum flashrom_log_level verbose_logfile = FLASHROM_MSG_DEBUG2;
diff --git a/include/cli_output.h b/include/cli_output.h
new file mode 100644
index 0000000..3f6107a
--- /dev/null
+++ b/include/cli_output.h
@@ -0,0 +1,21 @@
+/*
+ * This file is part of the flashrom project.
+ *
+ * SPDX-FileCopyrightText: 2024 Antonio Vázquez Blanco <antoniovazquezblanco@gmail.com>
+ * SPDX-License-Identifier: GPL-2.0-or-later
+ */
+#ifndef __CLI_OUTPUT_H__
+#define __CLI_OUTPUT_H__
+
+#include <stdarg.h>
+#include "flash.h"
+
+extern enum flashrom_log_level verbose_screen;
+extern enum flashrom_log_level verbose_logfile;
+int open_logfile(const char * const filename);
+int close_logfile(void);
+void start_logging(void);
+int flashrom_print_cb(enum flashrom_log_level level, const char *fmt, va_list ap);
+void flashrom_progress_cb(struct flashrom_flashctx *flashctx);
+
+#endif /* __CLI_OUTPUT_H__ */
diff --git a/include/flash.h b/include/flash.h
index d0e55af..f0b7f3b 100644
--- a/include/flash.h
+++ b/include/flash.h
@@ -680,14 +680,7 @@
/* cli_common.c */
void print_chip_support_status(const struct flashchip *chip);

-/* cli_output.c */
-extern enum flashrom_log_level verbose_screen;
-extern enum flashrom_log_level verbose_logfile;
-int open_logfile(const char * const filename);
-int close_logfile(void);
-void start_logging(void);
-int flashrom_print_cb(enum flashrom_log_level level, const char *fmt, va_list ap);
-void flashrom_progress_cb(struct flashrom_flashctx *flashctx);
+/* libflashrom.c */
/* Let gcc and clang check for correct printf-style format strings. */
int print(enum flashrom_log_level level, const char *fmt, ...)
#ifdef __MINGW32__

To view, visit change 84985. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newchange
Gerrit-Project: flashrom
Gerrit-Branch: main
Gerrit-Change-Id: I45d62bd219bdbed919788ae17a64aeb119a8aac4
Gerrit-Change-Number: 84985
Gerrit-PatchSet: 1
Gerrit-Owner: Antonio Vázquez Blanco <antoniovazquezblanco@gmail.com>