Re: [PATCH v6 2/2] board: kontron: pitx-imx8m: Add Kontron pitx-imx8m board support

From: Heiko Thiery <heiko.thiery@gmail.com>
To: Stefano Babic <sbabic@denx.de>, Simon Glass <sjg@chromium.org>
Cc: u-boot@lists.denx.de, "Fabio Estevam" <festevam@gmail.com>,
	"NXP i . MX U-Boot Team" <uboot-imx@nxp.com>,
	"Peng Fan" <peng.fan@nxp.com>, "Ye Li" <ye.li@nxp.com>,
	"Patrick Wildt" <patrick@blueri.se>,
	"Michael Walle" <michael@walle.cc>,
	"Thomas Schäfer" <thomas.schaefer@kontron.com>
Subject: Re: [PATCH v6 2/2] board: kontron: pitx-imx8m: Add Kontron pitx-imx8m board support
Date: Thu, 21 Oct 2021 15:57:22 +0200	[thread overview]
Message-ID: <CAEyMn7bBgst-+SshW+=tGv0+cFj7DD=Qjv5HYmbpf4JERUHA2Q@mail.gmail.com> (raw)
In-Reply-To: <ea6be4f8-d8d4-7773-0ae2-d10cdf07f63b@denx.de>

Hi Stefano, Hi Simon,

Am Do., 21. Okt. 2021 um 15:29 Uhr schrieb Stefano Babic <sbabic@denx.de>:
>
> On 21.10.21 15:17, Heiko Thiery wrote:
> > Hi Stefano,
> >
> >
> > Am Do., 21. Okt. 2021 um 13:31 Uhr schrieb Stefano Babic <sbabic@denx.de>:
> >>
> >> Hi Heiko,
> >>
> >> On 14.10.21 20:08, Heiko Thiery wrote:
> >>> The Kontron pitx-imx8m is an NXP i.MX8MQ based board in the pITX
> >>> form factor.
> >>>
> >>
> >> There is still something using binman that does not work with CI.
> >> Without CI, we load all firmware including signed_hdmi_imx8m.bin.
> >>
> >> But on CI, file is not loaded, but instead of warnings, build for this
> >> board fails with :
> >>
> >> +Image 'main-section' is missing external blobs and is non-functional:
> >> blob-ext@1 blob-ext@2 blob-ext@3 blob-ext@4
> >> +binman: Error 1 running 'mkimage -d ./mkimage.spl.mkimage -n
> >> spl/u-boot-spl.cfgout -T imx8mimage -e 0x7e1000
> >> ./mkimage-out.spl.mkimage': signed_hdmi_imx8m.bin: Can't open: No such
> >> file or directory
> >> +
> >>
> >> Could you take a look ?
> >
> > Sure. Could you point me to the right CI configuration?
> >
>
> Configuration is in .gitlab-ci.yml. For thsi board, buildman is called
> as part of aarch64 target, that is
>
>         buildman -o /tmp -P -E -W arm -x aarch64
>
> > I need to understand how the required imx-firmware files are copied to
> > the u-boot dir. This seems to work already for e. g. the imx8mm
> > boards. These require the lpdd* train fw files.

Here we seem to hit an issue with the CI. blobs (lpddr*) are required
for the imx8mm boards. Binman reports the missing files only as a
warning and buildman does not care about that:
----
+WARNING 'mkimage.flash.mkimage' not found, resulting binary is not-functional
----

With the imx8mq boards switching to binman we also need an additional
fw part (signed_hdmi_imx8m.bin). But other than the lpddr firmware as
blob the signed_hmdi'_imx8m.bin is passed to mkimage and mkimage exits
with an error. Thats leads to the error:
----
binman: Error 1 running 'mkimage -d ./mkimage.spl.mkimage -n
spl/u-boot-spl.cfgout -T imx8mimage -e 0x7e1000
./mkimage-out.spl.mkimage': signed_hdmi_imx8m.bin: Can't open: No such
file or directory
----

Currently I do not have an idea how to handle that. Anyone else?

-- 
Heiko