Re: [PATCH] [v2] printk: add dummy printk_force_console_enter/exit helpers
From: Andy Shevchenko
Date: Tue Nov 12 2024 - 10:21:57 EST
On Tue, Nov 12, 2024 at 03:29:15PM +0100, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@xxxxxxxx>
>
> The newly added interface is broken when PRINTK is disabled:
>
> drivers/tty/sysrq.c: In function '__handle_sysrq':
> drivers/tty/sysrq.c:601:9: error: implicit declaration of function 'printk_force_console_enter' [-Wimplicit-function-declaration]
> 601 | printk_force_console_enter();
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~
> drivers/tty/sysrq.c:611:25: error: implicit declaration of function 'printk_force_console_exit' [-Wimplicit-function-declaration]
> 611 | printk_force_console_exit();
> | ^~~~~~~~~~~~~~~~~~~~~~~~~
>
> Add empty stub functions for both.
Does it compile with `make W=1` without warnings/errors? If so, I am okay with
the approach.
Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
--
With Best Regards,
Andy Shevchenko