Hi Ilias, On Tue, 19 Jul 2022 at 17:03, Ilias Apalodimas <[email protected]> wrote: > > Hello Kojima-san > > On Fri, 15 Jul 2022 at 17:45, Masahisa Kojima > <[email protected]> wrote: > > > > Add documentation for eficonfig command. > > > > Signed-off-by: Masahisa Kojima <[email protected]> > > --- > > No change sinch v8 > > > > Changes in v8: > > - command name is changed from "efimenu" to "eficonfig" > > > > Newly created in v7 > > > > doc/usage/cmd/eficonfig.rst | 49 +++++++++++++++++++++++++++++++++++++ > > doc/usage/index.rst | 1 + > > [...] > > > + > > +The "eficonfig" command is enabled by:: > > + > > + CONFIG_CMD_EFICONFIG=y > > + > > +If CONFIG_BOOTMENU_DISABLE_UBOOT_CONSOLE is enabled, user can not enter > > +U-Boot console. In this case, bootmenu can be used to invoke "eficonfig":: > > + > > + CONFIG_USE_PREBOOT=y > > + CONFIG_PREBOOT="setenv bootmenu_0 UEFI Maintenance Menu=eficonfig" > > Can't we just set it as the default bootcmd? Does it have to be a > preboot command?
"eficonfig" command is for updating UEFI variables(BOOT#### and BootOrder, etc.) only. User can not boot the system with "eficonfig" command, so I think it is not the desired option to set it as default bootcmd. Thanks, Masahisa Kojima > > Thanks > /Ilias > > + > > +See also > > +-------- > > +* :doc:`bootmenu<bootmenu>` provides a simple mechanism for creating menus > > with different boot items > > diff --git a/doc/usage/index.rst b/doc/usage/index.rst > > index 8b98629d6b..1afbb6bc5d 100644 > > --- a/doc/usage/index.rst > > +++ b/doc/usage/index.rst > > @@ -35,6 +35,7 @@ Shell commands > > cmd/conitrace > > cmd/dm > > cmd/echo > > + cmd/eficonfig > > cmd/env > > cmd/event > > cmd/exception > > -- > > 2.17.1 > >
