Re: [PATCH v2 00/25] bootstd: Add a boot menu
Hi Tom, On Mon, 5 Dec 2022 at 17:05, Simon Glass wrote: > > Hi Heinrich (and anyone else), > > On Thu, 10 Nov 2022 at 15:15, Simon Glass wrote: > > > > Hi Heinrich, > > > > On Mon, 7 Nov 2022 at 16:35, Simon Glass wrote: > > > > > > Hi Heinrich, > > > > > > On Mon, 7 Nov 2022 at 12:15, Heinrich Schuchardt > > > wrote: > > > > > > > > On 11/4/22 23:48, Simon Glass wrote: > > > > > So far standard boot lacks a boot menu, although it is possible to > > > > > create > > > > > a rudimentary one using the existing 'bootmenu' command. > > > > > > > > > > Even then, this text-based menu offer only basic functionality and > > > > > does > > > > > not take full advantage of the displays which are common on many > > > > > devices. > > > > > > > > > > This series provides a 'bootflow menu' command which allows the user > > > > > to > > > > > select from the available bootflows. An attempt is made to show the > > > > > name > > > > > of the available operating systems, by reading more information into > > > > > the > > > > > bootflow. A logo can be read also, where supported, so that this can > > > > > be > > > > > presented to the user when an option is highlighted. > > > > > > > > > > Full use is made of TrueType fonts, if enabled. For cases where only a > > > > > serial console is available, it falls back to a simple text-based > > > > > menu. > > > > > > > > Please, add the link to your design document > > > > > > > > https://docs.google.com/document/d/1VQeApnLlH6xKm_OI36AhWkJLUEd9OXEvIJXB8aM2de8/edit?resourcekey=0-DwgHpR2S8vJEJzvvwPb-AQ#heading=h.17wg41voij6q > > > > is broken. > > > > > > What happens when you click that? It works for me. > > > > > > > > > > > in future version of this series. > > > > > > > > The series leaves us with duplicate code in > > > > > > > > bootmenu_choice_entry() and eficonfig_choice_entry() as well as > > > > bootmenu_loop() and bootmenu_autoboot_loop(). > > > > > > Yes OK, but that is the case today and my series actually removes some > > > duplicated code, so perhaps that could be cleaned up later? > > > > > > > > > > > The bootmenu command relies heavily on ANSI sequences but VIDEO_ANSI is > > > > disabled by default for CONFIG_EFI_LOADER=n which means that the > > > > bootmenu command will not work anymore. > > > > > > Does it not work, or does it just work but in a serial fashion? I > > > don't see ANSI codes as being necessary to show a menu. > > > > > > > > > > > > > > > > > All of this is implementing using a new 'expo' construct, a > > > > > collection of > > > > > > > > Expo is not an English word. Expo is typically used as name of trade > > > > fairs. Transaction probably is the right word to use here. > > > > > > That is debatable I think. Transaction is quite generic and appears in > > > U-Boot >400 times. I think it will just be confusing, like the word > > > 'metadata' used in the FWU stuff. > > > > > > Expo is short for exposition. My use of it is somewhat archaic > > > perhaps, but even for the meaning you mention, a public exposition is > > > not a bad description of what is provided here. > > > > > > I am not 100% convinced about 'expo' either. Do you have any other ideas? > > > > > > > > > > > Files expo.c and scene.c are in boot/ which does not match a generic GUI > > > > feature. They should be placed in lib/. > > > > > > Yes I was wondering about that, but thought that boot/ made at least > > > some sense since the menu will only ever be used for booting...? > > > > > > I can move it, but I am a little nervous about that, since lib/ > > > normally has utility libraries. Perhaps lib/expo would be better? > > > > Just to say that I replied to your comments on the doc also, so let me > > know what you think. > > I'd like to get this applied now that -next is opening. Do you have > any more comments? What are your thoughts on apply this now, -next? I am starting to wonder if we should have a VBE tree, as we do for EFI? Regards, Simon
Re: [PATCH v2 00/25] bootstd: Add a boot menu
Hi Heinrich (and anyone else), On Thu, 10 Nov 2022 at 15:15, Simon Glass wrote: > > Hi Heinrich, > > On Mon, 7 Nov 2022 at 16:35, Simon Glass wrote: > > > > Hi Heinrich, > > > > On Mon, 7 Nov 2022 at 12:15, Heinrich Schuchardt wrote: > > > > > > On 11/4/22 23:48, Simon Glass wrote: > > > > So far standard boot lacks a boot menu, although it is possible to > > > > create > > > > a rudimentary one using the existing 'bootmenu' command. > > > > > > > > Even then, this text-based menu offer only basic functionality and does > > > > not take full advantage of the displays which are common on many > > > > devices. > > > > > > > > This series provides a 'bootflow menu' command which allows the user to > > > > select from the available bootflows. An attempt is made to show the name > > > > of the available operating systems, by reading more information into the > > > > bootflow. A logo can be read also, where supported, so that this can be > > > > presented to the user when an option is highlighted. > > > > > > > > Full use is made of TrueType fonts, if enabled. For cases where only a > > > > serial console is available, it falls back to a simple text-based menu. > > > > > > Please, add the link to your design document > > > > > > https://docs.google.com/document/d/1VQeApnLlH6xKm_OI36AhWkJLUEd9OXEvIJXB8aM2de8/edit?resourcekey=0-DwgHpR2S8vJEJzvvwPb-AQ#heading=h.17wg41voij6q > > > is broken. > > > > What happens when you click that? It works for me. > > > > > > > > in future version of this series. > > > > > > The series leaves us with duplicate code in > > > > > > bootmenu_choice_entry() and eficonfig_choice_entry() as well as > > > bootmenu_loop() and bootmenu_autoboot_loop(). > > > > Yes OK, but that is the case today and my series actually removes some > > duplicated code, so perhaps that could be cleaned up later? > > > > > > > > The bootmenu command relies heavily on ANSI sequences but VIDEO_ANSI is > > > disabled by default for CONFIG_EFI_LOADER=n which means that the > > > bootmenu command will not work anymore. > > > > Does it not work, or does it just work but in a serial fashion? I > > don't see ANSI codes as being necessary to show a menu. > > > > > > > > > > > > > All of this is implementing using a new 'expo' construct, a collection > > > > of > > > > > > Expo is not an English word. Expo is typically used as name of trade > > > fairs. Transaction probably is the right word to use here. > > > > That is debatable I think. Transaction is quite generic and appears in > > U-Boot >400 times. I think it will just be confusing, like the word > > 'metadata' used in the FWU stuff. > > > > Expo is short for exposition. My use of it is somewhat archaic > > perhaps, but even for the meaning you mention, a public exposition is > > not a bad description of what is provided here. > > > > I am not 100% convinced about 'expo' either. Do you have any other ideas? > > > > > > > > Files expo.c and scene.c are in boot/ which does not match a generic GUI > > > feature. They should be placed in lib/. > > > > Yes I was wondering about that, but thought that boot/ made at least > > some sense since the menu will only ever be used for booting...? > > > > I can move it, but I am a little nervous about that, since lib/ > > normally has utility libraries. Perhaps lib/expo would be better? > > Just to say that I replied to your comments on the doc also, so let me > know what you think. I'd like to get this applied now that -next is opening. Do you have any more comments? Regards, Simon
Re: [PATCH v2 00/25] bootstd: Add a boot menu
Hi Heinrich, On Mon, 7 Nov 2022 at 16:35, Simon Glass wrote: > > Hi Heinrich, > > On Mon, 7 Nov 2022 at 12:15, Heinrich Schuchardt wrote: > > > > On 11/4/22 23:48, Simon Glass wrote: > > > So far standard boot lacks a boot menu, although it is possible to create > > > a rudimentary one using the existing 'bootmenu' command. > > > > > > Even then, this text-based menu offer only basic functionality and does > > > not take full advantage of the displays which are common on many devices. > > > > > > This series provides a 'bootflow menu' command which allows the user to > > > select from the available bootflows. An attempt is made to show the name > > > of the available operating systems, by reading more information into the > > > bootflow. A logo can be read also, where supported, so that this can be > > > presented to the user when an option is highlighted. > > > > > > Full use is made of TrueType fonts, if enabled. For cases where only a > > > serial console is available, it falls back to a simple text-based menu. > > > > Please, add the link to your design document > > > > https://docs.google.com/document/d/1VQeApnLlH6xKm_OI36AhWkJLUEd9OXEvIJXB8aM2de8/edit?resourcekey=0-DwgHpR2S8vJEJzvvwPb-AQ#heading=h.17wg41voij6q > > is broken. > > What happens when you click that? It works for me. > > > > > in future version of this series. > > > > The series leaves us with duplicate code in > > > > bootmenu_choice_entry() and eficonfig_choice_entry() as well as > > bootmenu_loop() and bootmenu_autoboot_loop(). > > Yes OK, but that is the case today and my series actually removes some > duplicated code, so perhaps that could be cleaned up later? > > > > > The bootmenu command relies heavily on ANSI sequences but VIDEO_ANSI is > > disabled by default for CONFIG_EFI_LOADER=n which means that the > > bootmenu command will not work anymore. > > Does it not work, or does it just work but in a serial fashion? I > don't see ANSI codes as being necessary to show a menu. > > > > > > > > > All of this is implementing using a new 'expo' construct, a collection of > > > > Expo is not an English word. Expo is typically used as name of trade > > fairs. Transaction probably is the right word to use here. > > That is debatable I think. Transaction is quite generic and appears in > U-Boot >400 times. I think it will just be confusing, like the word > 'metadata' used in the FWU stuff. > > Expo is short for exposition. My use of it is somewhat archaic > perhaps, but even for the meaning you mention, a public exposition is > not a bad description of what is provided here. > > I am not 100% convinced about 'expo' either. Do you have any other ideas? > > > > > Files expo.c and scene.c are in boot/ which does not match a generic GUI > > feature. They should be placed in lib/. > > Yes I was wondering about that, but thought that boot/ made at least > some sense since the menu will only ever be used for booting...? > > I can move it, but I am a little nervous about that, since lib/ > normally has utility libraries. Perhaps lib/expo would be better? Just to say that I replied to your comments on the doc also, so let me know what you think. Regards, Simon
Re: [PATCH v2 00/25] bootstd: Add a boot menu
Hi Heinrich, On Mon, 7 Nov 2022 at 12:15, Heinrich Schuchardt wrote: > > On 11/4/22 23:48, Simon Glass wrote: > > So far standard boot lacks a boot menu, although it is possible to create > > a rudimentary one using the existing 'bootmenu' command. > > > > Even then, this text-based menu offer only basic functionality and does > > not take full advantage of the displays which are common on many devices. > > > > This series provides a 'bootflow menu' command which allows the user to > > select from the available bootflows. An attempt is made to show the name > > of the available operating systems, by reading more information into the > > bootflow. A logo can be read also, where supported, so that this can be > > presented to the user when an option is highlighted. > > > > Full use is made of TrueType fonts, if enabled. For cases where only a > > serial console is available, it falls back to a simple text-based menu. > > Please, add the link to your design document > > https://docs.google.com/document/d/1VQeApnLlH6xKm_OI36AhWkJLUEd9OXEvIJXB8aM2de8/edit?resourcekey=0-DwgHpR2S8vJEJzvvwPb-AQ#heading=h.17wg41voij6q > is broken. What happens when you click that? It works for me. > > in future version of this series. > > The series leaves us with duplicate code in > > bootmenu_choice_entry() and eficonfig_choice_entry() as well as > bootmenu_loop() and bootmenu_autoboot_loop(). Yes OK, but that is the case today and my series actually removes some duplicated code, so perhaps that could be cleaned up later? > > The bootmenu command relies heavily on ANSI sequences but VIDEO_ANSI is > disabled by default for CONFIG_EFI_LOADER=n which means that the > bootmenu command will not work anymore. Does it not work, or does it just work but in a serial fashion? I don't see ANSI codes as being necessary to show a menu. > > > > > All of this is implementing using a new 'expo' construct, a collection of > > Expo is not an English word. Expo is typically used as name of trade > fairs. Transaction probably is the right word to use here. That is debatable I think. Transaction is quite generic and appears in U-Boot >400 times. I think it will just be confusing, like the word 'metadata' used in the FWU stuff. Expo is short for exposition. My use of it is somewhat archaic perhaps, but even for the meaning you mention, a public exposition is not a bad description of what is provided here. I am not 100% convinced about 'expo' either. Do you have any other ideas? > > Files expo.c and scene.c are in boot/ which does not match a generic GUI > feature. They should be placed in lib/. Yes I was wondering about that, but thought that boot/ made at least some sense since the menu will only ever be used for booting...? I can move it, but I am a little nervous about that, since lib/ normally has utility libraries. Perhaps lib/expo would be better? Regards, Simon > > Best regards > > Heinrich > > > scenes (like menu screens) which can be navigated by the user to view > > information and select options. This is fairly general and should be able > > to cope with a wider array of use cases, with less hacking of the menu > > code, such as is currently needed for CMD_BOOTEFI_BOOTMGR. > > > > Of course it would be possible to enhance the existing menu rather than > > creating a new setup. Instead it seems better to make the existing menu > > use expo, if code space permits. It avoids the event-loop problem and > > should be more extensible, given its loosely coupled components and use of > > IDs instead of pointers. Further motivation is provided in the > > documentation. > > > > For now the CLI keypress-decoding code is split out to be used by the new > > menu. The key codes defined by menu.h are reused also. > > > > This is of course just a starting point. Some ideas for future work are > > included in the documentation. > > > > Changes in v2: > > - Drop the _add suffix on expo creation function > > - Fix 'touse' typo > > - Fix pylint warning in mkdir_cond() > > - Put strings in a separate structure referenced by ID > > - Rebase to master > > - Rename vidconsole_get_font() to vidconsole_get_font_size() > > - Update for new API > > > > Simon Glass (25): > >sandbox: Enable mmc command and legacy images > >cli: Move readline character-processing to a state machine > >bootmenu: Add a few comments > >menu: Rename KEY_... to BKEY_... > >menu: Update bootmenu_autoboot_loop() to return the code > >menu: Update bootmenu_loop() to return the code > >menu: Use a switch statement > >menu: Make use of CLI character processing > >image: Add a function to find a script in an image > >image: Move common image code to image_board and command > >video: Enable VIDEO_ANSI by default only with EFI > >video: truetype: Rename the metrics function > >video: Fix unchnaged typo > >video: Add font functions to the vidconsole API > >bootstd: Read the Operating System name for distro/scripts > >
Re: [PATCH v2 00/25] bootstd: Add a boot menu
On 11/4/22 23:48, Simon Glass wrote: So far standard boot lacks a boot menu, although it is possible to create a rudimentary one using the existing 'bootmenu' command. Even then, this text-based menu offer only basic functionality and does not take full advantage of the displays which are common on many devices. This series provides a 'bootflow menu' command which allows the user to select from the available bootflows. An attempt is made to show the name of the available operating systems, by reading more information into the bootflow. A logo can be read also, where supported, so that this can be presented to the user when an option is highlighted. Full use is made of TrueType fonts, if enabled. For cases where only a serial console is available, it falls back to a simple text-based menu. Please, add the link to your design document https://docs.google.com/document/d/1VQeApnLlH6xKm_OI36AhWkJLUEd9OXEvIJXB8aM2de8/edit?resourcekey=0-DwgHpR2S8vJEJzvvwPb-AQ#heading=h.17wg41voij6q is broken. in future version of this series. The series leaves us with duplicate code in bootmenu_choice_entry() and eficonfig_choice_entry() as well as bootmenu_loop() and bootmenu_autoboot_loop(). The bootmenu command relies heavily on ANSI sequences but VIDEO_ANSI is disabled by default for CONFIG_EFI_LOADER=n which means that the bootmenu command will not work anymore. All of this is implementing using a new 'expo' construct, a collection of Expo is not an English word. Expo is typically used as name of trade fairs. Transaction probably is the right word to use here. Files expo.c and scene.c are in boot/ which does not match a generic GUI feature. They should be placed in lib/. Best regards Heinrich scenes (like menu screens) which can be navigated by the user to view information and select options. This is fairly general and should be able to cope with a wider array of use cases, with less hacking of the menu code, such as is currently needed for CMD_BOOTEFI_BOOTMGR. Of course it would be possible to enhance the existing menu rather than creating a new setup. Instead it seems better to make the existing menu use expo, if code space permits. It avoids the event-loop problem and should be more extensible, given its loosely coupled components and use of IDs instead of pointers. Further motivation is provided in the documentation. For now the CLI keypress-decoding code is split out to be used by the new menu. The key codes defined by menu.h are reused also. This is of course just a starting point. Some ideas for future work are included in the documentation. Changes in v2: - Drop the _add suffix on expo creation function - Fix 'touse' typo - Fix pylint warning in mkdir_cond() - Put strings in a separate structure referenced by ID - Rebase to master - Rename vidconsole_get_font() to vidconsole_get_font_size() - Update for new API Simon Glass (25): sandbox: Enable mmc command and legacy images cli: Move readline character-processing to a state machine bootmenu: Add a few comments menu: Rename KEY_... to BKEY_... menu: Update bootmenu_autoboot_loop() to return the code menu: Update bootmenu_loop() to return the code menu: Use a switch statement menu: Make use of CLI character processing image: Add a function to find a script in an image image: Move common image code to image_board and command video: Enable VIDEO_ANSI by default only with EFI video: truetype: Rename the metrics function video: Fix unchnaged typo video: Add font functions to the vidconsole API bootstd: Read the Operating System name for distro/scripts bootstd: Allow reading a logo for the OS menu: Factor out menu-keypress decoding expo: Add basic implementation expo: Add support for scenes expo: Add support for scene menus expo: Add basic tests bootstd: Support creating a boot menu bootstd: Add a test for the bootstd menu bootstd: Support setting a theme for the menu expo: Add documentation .../cmd_stm32prog/cmd_stm32prog.c | 2 +- arch/sandbox/dts/test.dts | 11 + boot/Kconfig | 12 + boot/Makefile | 3 + boot/bootflow.c | 1 + boot/bootflow_internal.h | 47 ++ boot/bootflow_menu.c | 284 + boot/bootmeth-uclass.c| 69 ++- boot/bootmeth_distro.c| 36 ++ boot/bootmeth_script.c| 40 +- boot/bootstd-uclass.c | 2 + boot/expo.c | 170 ++ boot/image-board.c| 133 + boot/scene.c | 414 ++ boot/scene_internal.h | 123 boot/scene_menu.c | 390 + cmd/bootflow.c|
[PATCH v2 00/25] bootstd: Add a boot menu
So far standard boot lacks a boot menu, although it is possible to create a rudimentary one using the existing 'bootmenu' command. Even then, this text-based menu offer only basic functionality and does not take full advantage of the displays which are common on many devices. This series provides a 'bootflow menu' command which allows the user to select from the available bootflows. An attempt is made to show the name of the available operating systems, by reading more information into the bootflow. A logo can be read also, where supported, so that this can be presented to the user when an option is highlighted. Full use is made of TrueType fonts, if enabled. For cases where only a serial console is available, it falls back to a simple text-based menu. All of this is implementing using a new 'expo' construct, a collection of scenes (like menu screens) which can be navigated by the user to view information and select options. This is fairly general and should be able to cope with a wider array of use cases, with less hacking of the menu code, such as is currently needed for CMD_BOOTEFI_BOOTMGR. Of course it would be possible to enhance the existing menu rather than creating a new setup. Instead it seems better to make the existing menu use expo, if code space permits. It avoids the event-loop problem and should be more extensible, given its loosely coupled components and use of IDs instead of pointers. Further motivation is provided in the documentation. For now the CLI keypress-decoding code is split out to be used by the new menu. The key codes defined by menu.h are reused also. This is of course just a starting point. Some ideas for future work are included in the documentation. Changes in v2: - Drop the _add suffix on expo creation function - Fix 'touse' typo - Fix pylint warning in mkdir_cond() - Put strings in a separate structure referenced by ID - Rebase to master - Rename vidconsole_get_font() to vidconsole_get_font_size() - Update for new API Simon Glass (25): sandbox: Enable mmc command and legacy images cli: Move readline character-processing to a state machine bootmenu: Add a few comments menu: Rename KEY_... to BKEY_... menu: Update bootmenu_autoboot_loop() to return the code menu: Update bootmenu_loop() to return the code menu: Use a switch statement menu: Make use of CLI character processing image: Add a function to find a script in an image image: Move common image code to image_board and command video: Enable VIDEO_ANSI by default only with EFI video: truetype: Rename the metrics function video: Fix unchnaged typo video: Add font functions to the vidconsole API bootstd: Read the Operating System name for distro/scripts bootstd: Allow reading a logo for the OS menu: Factor out menu-keypress decoding expo: Add basic implementation expo: Add support for scenes expo: Add support for scene menus expo: Add basic tests bootstd: Support creating a boot menu bootstd: Add a test for the bootstd menu bootstd: Support setting a theme for the menu expo: Add documentation .../cmd_stm32prog/cmd_stm32prog.c | 2 +- arch/sandbox/dts/test.dts | 11 + boot/Kconfig | 12 + boot/Makefile | 3 + boot/bootflow.c | 1 + boot/bootflow_internal.h | 47 ++ boot/bootflow_menu.c | 284 + boot/bootmeth-uclass.c| 69 ++- boot/bootmeth_distro.c| 36 ++ boot/bootmeth_script.c| 40 +- boot/bootstd-uclass.c | 2 + boot/expo.c | 170 ++ boot/image-board.c| 133 + boot/scene.c | 414 ++ boot/scene_internal.h | 123 boot/scene_menu.c | 390 + cmd/bootflow.c| 44 +- cmd/bootmenu.c| 19 +- cmd/eficonfig.c | 38 +- cmd/font.c| 11 +- cmd/source.c | 140 + common/Makefile | 6 +- common/cli_getch.c| 208 +++ common/cli_readline.c | 150 + common/command.c | 19 + common/menu.c | 157 +++-- configs/sandbox_defconfig | 2 + configs/sandbox_flattree_defconfig| 2 + configs/snow_defconfig| 4 + configs/tools-only_defconfig | 2 + doc/develop/expo.rst | 188 ++ doc/develop/index.rst | 1 + drivers/usb/gadget/f_sdp.c| 2 +-