[PATCH u-boot 0/3] Revert broken Bootmenu commits
Tom Rini
trini at konsulko.com
Wed Jun 14 21:51:21 CEST 2023
On Sun, Jun 11, 2023 at 02:53:21PM +0200, Pali Rohár wrote:
> These 3 commits broke support for U-Boot Bootmenu on Nokia N900.
> Issues were reported more than month ago, but nobody reacted to them.
> So revert these broken commits for now, to fix U-Boot Bootmenu support.
>
> With these revered commits, U-Boot Bootmenu from master branch is
> working fine again on Nokia N900.
>
> Pali Rohár (3):
> Revert "video: Enable VIDEO_ANSI by default only with EFI"
> Revert "menu: Factor out menu-keypress decoding"
> Revert "menu: Make use of CLI character processing"
>
> cmd/bootmenu.c | 9 ++--
> cmd/eficonfig.c | 12 ++---
> common/cli_getch.c | 12 ++---
> common/menu.c | 122 ++++++++++++++++++++++++++----------------
> drivers/video/Kconfig | 7 +--
> include/cli.h | 4 +-
> include/menu.h | 17 +-----
> 7 files changed, 91 insertions(+), 92 deletions(-)
Following up over here, while
https://patchwork.ozlabs.org/project/uboot/patch/20230612201434.861700-1-sjg@chromium.org/
addresses some of the issues, but not all (as it clearly isn't working
in all of the cases Pali has explained), looking in to the VIDEO_ANSI
part of this too, all three of these reverts are related seemingly to
something escape-character related. I'm not taking any of the revert
commits in just yet, but will by the next -rc if we don't have something
that fixes all of the issues.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230614/9fdc8cd6/attachment.sig>
More information about the U-Boot
mailing list