[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