On 28.02.19 20:49, Heinrich Schuchardt wrote:
> In  `efidebug boot add iPXE scsi 0:1 snp-arm64.efi --foo` a parameter is
> missing. Hence the command should not silently return as if everything were
> ok but should display the usage info.
>
> Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>


Reviewed-by: Alexander Graf <ag...@csgraf.de>


Alex


> ---
>  cmd/efidebug.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/cmd/efidebug.c b/cmd/efidebug.c
> index 5072a7b39bc..db96682c5a4 100644
> --- a/cmd/efidebug.c
> +++ b/cmd/efidebug.c
> @@ -491,7 +491,7 @@ static int do_efi_boot_add(cmd_tbl_t *cmdtp, int flag,
>  
>       id = (int)simple_strtoul(argv[1], &endp, 16);
>       if (*endp != '\0' || id > 0xffff)
> -             return CMD_RET_FAILURE;
> +             return CMD_RET_USAGE;
>  
>       sprintf(var_name, "Boot%04X", id);
>       p = var_name16;
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to