[PATCH] bootmenu: Select CFB_CONSOLE_ANSI if CFB_CONSOLE

Heinrich Schuchardt xypron.glpk at gmx.de
Tue Dec 29 11:42:56 CET 2020


On 12/29/20 4:32 AM, Simon Glass wrote:
> Hi Paul,
>
> On Sat, 26 Dec 2020 at 14:04, Pali Rohár <pali at kernel.org> wrote:
>>
>> CMD_BOOTMENU uses ANSI terminal. Therefore if U-Boot has enabled
>> CFB_CONSOLE then bootmenu needs also CFB_CONSOLE_ANSI to work.
>>
>> Define this dependency in Kconfig and remove information about this
>> requirement from documentation file.
>>
>> Signed-off-by: Pali Rohár <pali at kernel.org>
>> ---
>>   cmd/Kconfig            | 1 +
>>   doc/usage/bootmenu.rst | 5 -----
>>   2 files changed, 1 insertion(+), 5 deletions(-)
>
> This driver is past the driver model migration deadline so we
> shouldn't accept patches, except perhaps to migrate or remove it.

All boards selecting CONFIG_CFB_CONSOLE also select
CONFIG_CFB_CONSOLE_ANSI. So this change probably is not needed for
existing boards. CONFIG_CFB_CONSOLE_ANSI=y is mentioned in
doc/usage/bootmenu.rst.

Why does CMD_BOOTMENU not select CONFIG_VIDEO_ANSI which is the setting
used by all driver model video drivers? Shouldn't that setting be
mentioned in doc/usage/bootmenu.rst?

Best regards

Heinrich


More information about the U-Boot mailing list