Nico Huber has submitted this change. ( https://review.coreboot.org/c/coreboot/+/60470 )
Change subject: console: Make get_log_level a public function ......................................................................
console: Make get_log_level a public function
Other drivers may need to know the coreboot log level hence, export this function rather than marking it static.
Signed-off-by: Subrata Banik subratabanik@google.com Change-Id: I56349f22c71c9db757b2be8eeb2dbfe959f80397 Reviewed-on: https://review.coreboot.org/c/coreboot/+/60470 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Angel Pons th3fanbus@gmail.com Reviewed-by: Nico Huber nico.h@gmx.de --- M src/console/init.c M src/include/console/console.h 2 files changed, 3 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Nico Huber: Looks good to me, but someone else must approve Angel Pons: Looks good to me, approved
diff --git a/src/console/init.c b/src/console/init.c index 4427d68..eb6b637 100644 --- a/src/console/init.c +++ b/src/console/init.c @@ -13,7 +13,7 @@ static int console_inited; static int console_loglevel;
-static inline int get_log_level(void) +int get_log_level(void) { if (console_inited == 0) return -1; diff --git a/src/include/console/console.h b/src/include/console/console.h index 8849df3..5546aa9 100644 --- a/src/include/console/console.h +++ b/src/include/console/console.h @@ -47,6 +47,7 @@ ENV_LIBAGESA || (ENV_SMM && CONFIG(DEBUG_SMI)))
#if __CONSOLE_ENABLE__ +int get_log_level(void); void console_init(void); int console_log_level(int msg_level);
@@ -62,6 +63,7 @@
enum { CONSOLE_LOG_NONE = 0, CONSOLE_LOG_FAST, CONSOLE_LOG_ALL }; #else +static inline int get_log_level(void) { return -1; } static inline void console_init(void) {} static inline int console_log_level(int msg_level) { return 0; } static inline int