Re: [PATCH v3] imx: imx8qm: cgtqmx8: switch to binman

[Oliver Graute <oliver.graute@gmail.com>]

On 24/10/22, Stefano Babic wrote:
> On 24.10.22 13:59, Oliver Graute wrote:
> > On 24/10/22, Stefano Babic wrote:
> > > On 24.10.22 13:37, Fabio Estevam wrote:
> > > > Hi Stefano and Oliver,
> > > > 
> > > > On Tue, May 3, 2022 at 7:49 AM Oliver Graute
> > > > <oliver.graute@kococonnector.com> wrote:
> > > > > 
> > > > > Switch to use binman to pack images
> > > > > 
> > > > > Signed-off-by: Oliver Graute <oliver.graute@kococonnector.com>
> > > > > Reviewed-by: Fabio Estevam <festevam@denx.de>
> > > > > Reviewed-by: Simon Glass <sjg@chromium.org>
> > > > > ---
> > > > > Changes for v3
> > > > >    -added Reviewed-by
> > > > >    -rebased on master
> > > > > 
> > > > >    arch/arm/dts/imx8qm-cgtqmx8.dts     | 1 +
> > > > >    arch/arm/mach-imx/imx8/Kconfig      | 1 +
> > > > >    board/congatec/cgtqmx8/imximage.cfg | 4 ++--
> > > > >    configs/cgtqmx8_defconfig           | 2 +-
> > > > >    doc/board/congatec/cgtqmx8.rst      | 3 +--
> > > > >    5 files changed, 6 insertions(+), 5 deletions(-)
> > > > 
> > > > Just realized that v3 does not include imx8qm-u-boot.dtsi and this breaks CI.
> > > > 
> > > 
> > > Right - as well as V4, this is what I tried to merge and breaks. I have drop
> > > it.
> > > 
> > > > The previous v2 did include it:
> > > > https://patchwork.ozlabs.org/project/uboot/patch/20220221090139.10066-1-oliver.graute@kococonnector.com/
> > > 
> > > Thanks for noting this - Oliver, can you check and repost a V5, then ?
> > 
> > The arch/arm/dts/imx8qm-u-boot.dtsi  is applied on the binman Patch for
> > "PATCH v3 imx: imx8qm-rom7720: switch to binman"
> > 
> > So if you first apply the patches for the imx8qm-rom7720 Board it should work.
> 
> 
> Ok, thanks for tipp - I apply both and will test again

I saw the CI Pipeline failing. Perhaps the reason is a
missing CONFIG_FIT_EXTERNAL_OFFSET?

+++ b/configs/cgtqmx8_defconfig
@@ -20,6 +20,7 @@ CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
 CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0x80200000
 CONFIG_REMAKE_ELF=y
 CONFIG_FIT=y
+CONFIG_FIT_EXTERNAL_OFFSET=0x3000
 CONFIG_SPL_LOAD_FIT=y
 # CONFIG_USE_SPL_FIT_GENERATOR is not set

Can you confirm?

 Best regards,

 Oliver