Re: [PATCH] doc: update bootefi man-page

[AKASHI Takahiro <takahiro.akashi@linaro.org>]

On Mon, Apr 11, 2022 at 09:10:47PM +0200, Heinrich Schuchardt wrote:
> A image_size parameter has been added to the bootefi parameter.
> Describe all parameters.
> 
> Correct how the description of how the device-path in the loaded
> image protocol is determined.
> 
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
> ---
>  doc/usage/cmd/bootefi.rst | 19 ++++++++++++++++---
>  1 file changed, 16 insertions(+), 3 deletions(-)
> 
> diff --git a/doc/usage/cmd/bootefi.rst b/doc/usage/cmd/bootefi.rst
> index 31279fc0cb..bb354b2255 100644
> --- a/doc/usage/cmd/bootefi.rst
> +++ b/doc/usage/cmd/bootefi.rst
> @@ -9,7 +9,7 @@ Synopsis
>  
>  ::
>  
> -    bootefi [image_addr] [fdt_addr]
> +    bootefi [image_addr] [fdt_addr [image_size]]


I think that the syntax like
        bootefi [image_addr[:image_size]] [fdt_addr]
would be more aligned with other commands, say booti/bootz
and it would look much easier to understand and less confusing.

# I remember I've got a similar comment when I posted a patch for nvedit_efi.c.

-Takahiro Akashi


>      bootefi bootmgr [fdt_addr]
>      bootefi hello [fdt_addr]
>      bootefi selftest [fdt_addr]
> @@ -41,13 +41,26 @@ command sequence to run a UEFI application might look like
>      load mmc 0:1 $kernel_addr_r /EFI/grub/grubaa64.efi
>      bootefi $kernel_addr_r $fdt_addr_r
>  
> -The last file loaded defines the image file path in the loaded image protocol.
> -Hence the executable should always be loaded last.
> +The last UEFI binary loaded defines the image file path in the loaded image
> +protocol.
>  
>  The value of the environment variable *bootargs* is converted from UTF-8 to
>  UTF-16 and passed as load options in the loaded image protocol to the UEFI
>  binary.
>  
> +image_addr
> +    Address of the UEFI binary.
> +
> +fdt_addr
> +    Address of the device-tree or '-'. If no address is specifiy, the
> +    environment variable $fdt_addr is used as first fallback, the address of
> +    U-Boot's internal device-tree $fdtcontroladdr as second fallback.
> +    When using ACPI no device-tree shall be specified.
> +
> +image_size
> +    Size of the UEFI binary file. This argument is only needed if *image_addr*
> +    does not match the address of the last loaded UEFI binary.
> +
>  Note
>      UEFI binaries that are contained in FIT images are launched via the
>      *bootm* command.
> -- 
> 2.34.1
>