Re: [PATCH v3 0/2] printk: CONFIG_BASE_SMALL fix for LOG_CPU_MAX_BUF_SHIFT and removal

From: Petr Mladek
Date: Mon Feb 05 2024 - 08:48:31 EST


On Mon 2024-02-05 00:29:43, Yoann Congal wrote:
> This series focuses on CONFIG_BASE_SMALL.
> The first patch fixes LOG_CPU_MAX_BUF_SHIFT when CONFIG_BASE_SMALL is
> used.
> The second patch globally replace CONFIG_BASE_SMALL usages by the
> equivalent !CONFIG_BASE_FULL.

Nit: I would personally do it the other way around and get rid of
CONFIG_BASE_SMALL.

CONFIG_BASE_FULL is the default and is used by most users. It is
the CONFIG_BASE_SMALL which needs special treating.

That said, I do not want to block this patchset because
of my preferences.

Best Regards,
Petr