[U-Boot] [PATCH v3 1/8] cmd: add efishell command

AKASHI Takahiro
2019-01-07T05:08:53Z

On Sun, Dec 30, 2018 at 06:10:51PM +0100, Heinrich Schuchardt wrote:
> On 12/30/18 4:44 PM, Heinrich Schuchardt wrote:
> > On 12/18/18 6:05 AM, AKASHI Takahiro wrote:
> >> Currently, there is no easy way to add or modify UEFI variables.
> >> In particular, bootmgr supports BootOrder/BootXXXX variables, it is
> >> quite hard to define them as u-boot variables because they are represented
> >> in a complicated and encoded format.
> >>
> >> The new command, efishell, helps address these issues and give us
> >> more friendly interfaces:
> >> * efishell boot add: add BootXXXX variable
> >> * efishell boot rm: remove BootXXXX variable
> >> * efishell boot dump: display all BootXXXX variables
> >> * efishell boot order: set/display a boot order (BootOrder)
> >> * efishell setvar: set an UEFI variable (with limited functionality)
> >> * efishell dumpvar: display all UEFI variables
> >>
> >> As the name suggests, this command basically provides a subset fo UEFI
> >> shell commands with simplified functionality.
> >>
> >> Signed-off-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
>
> The behavior is a bit unexpected:
>
> => efishell boot order 200
> => efishell boot order
> 1: Boot00C8: (not defined)
> exit not allowed from main input shell.
>
> I would expect 'efishell boot order' to take a 4 digit hexadecimal
> number and to do no conversion from decimal to hexadecimal.

OK, but we should allow a less-than-4-digit number.

> I was also surprised to see 'exit not allowed from main input shell.'

I cannot reproduce this problem.

-Takahiro Akashi

> Best regards
>
> Heinrich