[PATCH v4 5/9] sunxi: binman: Provide a default BL31 filename

[Jagan Teki]

Hi Samuel,

On Sun, Sep 13, 2020 at 3:05 AM Samuel Holland <samuel at sholland.org> wrote:
>
> Prior to commit 7f7f8aca8257 ("sunxi: Convert 64-bit boards to use
> binman"), if the BL31 environment variable was not defined, the firmware
> would be loaded from a file "bl31.bin" in the current directory. Restore
> that behavior by providing that as the default filename in case no entry
> arg is provided, which will be the case if the environment variable is
> unset.
>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> Signed-off-by: Samuel Holland <samuel at sholland.org>
> ---
>  arch/arm/dts/sunxi-u-boot.dtsi | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/dts/sunxi-u-boot.dtsi b/arch/arm/dts/sunxi-u-boot.dtsi
> index 82b97a9503f..6ab0c783ba5 100644
> --- a/arch/arm/dts/sunxi-u-boot.dtsi
> +++ b/arch/arm/dts/sunxi-u-boot.dtsi
> @@ -52,6 +52,7 @@
>  #endif
>
>                                         atf-bl31 {
> +                                               filename = "bl31.bin";

 I have dropped missing-msg = "atf-bl31-sunxi"; and used filename
section as above.

Build showing below messages.

  MKIMAGE u-boot.img
  MKIMAGE u-boot-dtb.img
  BINMAN  all
Image 'main-section' is missing external blobs and is non-functional: scp

Some images are invalid

Cannot we make scp as optional for the build?

Jagan.