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-...@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
signature.asc
Description: PGP signature