Hi Petr, On 18. 03. 20 10:57, Petr Štetiar wrote: > At this moment unusable binaries are produced if bl31.bin file is > missing in order to allow passing of various CI tests. This intention of > broken binaries has to be now explicitly confirmed via new > BUILDBOT_BROKEN_BINARIES config option, so usable binaries are produced > by default from now on. > > Signed-off-by: Petr Štetiar <[email protected]> > --- > arch/arm/mach-zynqmp/mkimage_fit_atf.sh | 14 +++++++++----- > 1 file changed, 9 insertions(+), 5 deletions(-) > > diff --git a/arch/arm/mach-zynqmp/mkimage_fit_atf.sh > b/arch/arm/mach-zynqmp/mkimage_fit_atf.sh > index 1e770ba111d3..5effe05abdee 100755 > --- a/arch/arm/mach-zynqmp/mkimage_fit_atf.sh > +++ b/arch/arm/mach-zynqmp/mkimage_fit_atf.sh > @@ -29,11 +29,15 @@ else > fi > > if [ ! -f $BL31 ]; then > - echo "WARNING: BL31 file $BL31 NOT found, resulting binary is > non-functional" >&2 > - BL31=/dev/null > - # But U-Boot proper could be loaded in EL3 by specifying > - # firmware = "uboot"; > - # instead of "atf" in config node > + if [ "$BUILDBOT_BROKEN_BINARIES" = "y" ]; then > + BL31=/dev/null > + # But U-Boot proper could be loaded in EL3 by specifying > + # firmware = "uboot"; > + # instead of "atf" in config node > + else > + echo "ERROR: BL31 file $BL31 NOT found, resulting binary is > non-functional" >&2 > + exit 1 > + fi > fi > > cat << __HEADER_EOF >
I think instead of fixing it on several places we should merge things together and fix this issue there. Take a look at thread where we discussed it with Tom. https://lists.denx.de/pipermail/u-boot/2019-December/393556.html Thanks, Michal
