[PATCH] bootmenu: Select CFB_CONSOLE_ANSI if CFB_CONSOLE

Pali Rohár pali at kernel.org
Tue Dec 29 12:26:23 CET 2020


On Tuesday 29 December 2020 11:42:56 Heinrich Schuchardt wrote:
> 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.

That is because I added this "hint" into Kconfig. So it does not have to
be specified in documentation rst file.

> 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?

Because I did not know about this option. It was not mentioned in
documentation rst file.


More information about the U-Boot mailing list