From mboxrd@z Thu Jan 1 00:00:00 1970 From: AKASHI Takahiro Date: Mon, 7 Jan 2019 14:08:53 +0900 Subject: [U-Boot] [PATCH v3 1/8] cmd: add efishell command In-Reply-To: References: <20181218050510.20308-1-takahiro.akashi@linaro.org> <20181218050510.20308-2-takahiro.akashi@linaro.org> Message-ID: <20190107050852.GD9033@linaro.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > > 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