public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: "oliver.graute@kococonnector.com" <oliver.graute@gmail.com>
To: Marcel Ziswiler <marcel.ziswiler@toradex.com>
Cc: "sbabic@denx.de" <sbabic@denx.de>,
	"ye.li@nxp.com" <ye.li@nxp.com>,
	"uboot-imx@nxp.com" <uboot-imx@nxp.com>,
	"gaurav.jain@nxp.com" <gaurav.jain@nxp.com>,
	"festevam@gmail.com" <festevam@gmail.com>,
	"peng.fan@nxp.com" <peng.fan@nxp.com>,
	"horia.geanta@nxp.com" <horia.geanta@nxp.com>,
	"denys.drozdov@toradex.com" <denys.drozdov@toradex.com>,
	"u-boot@lists.denx.de" <u-boot@lists.denx.de>
Subject: Re: [PATCH v2 8/8] imx: imx8: apalis: switch to binman
Date: Thu, 10 Nov 2022 17:35:18 +0100	[thread overview]
Message-ID: <20221110163518.GC17674@optiplex> (raw)
In-Reply-To: <7e74253e9886b9ca21520af393ff6a7fd323afc4.camel@toradex.com>

On 09/11/22, Marcel Ziswiler wrote:
> Hi Oliver
> 
> Thanks for working on this.
> 
> On Fri, 2022-11-04 at 16:03 +0100, Oliver Graute wrote:
> > Switch to use binman to pack images
> > 
> > Signed-off-by: Oliver Graute <oliver.graute@kococonnector.com>
> > ---
> > Changes for v2
> >  - use common imx8qm-u-boot.dtsi
> >  - guard SPL nodes with CONFIG_SPL
> 
> I don't think it is that simple. Even after guarding them SPL nodes the flash.bin one still references them,
> not? And yes, so far we never used SPL for them 8/8X as that honestly does not really make much sense given the
> SCFW does the actual RAM initialisation. Anyway, I am not very clear on how/what exactly that all means now.

On imx8qm-rom7720 we use the SCFW for RAM initalisation too but still use
SPL.  
> 
> Another topic is that flash.bin is now just the SPL whereas previously u-boot-dtb.imx was the entire thing. But
> again, that could have been related to us not using SPL.

yes, thats right, if I guard the SPL subnode in imx-boot node I get a 0 byte
flash.bin. 

At the moment I don't know the best way here. I would be grateful for
some hints here.

> >  CONFIG_FIT=y +CONFIG_FIT_EXTERNAL_OFFSET=0x3000
> 
> Also not quite sure what exactly that is now.

its introduced by commit 3814fcba12323b9f30f39ee5455f3f9a7e955c64

Best regards,

Oliver

  reply	other threads:[~2022-11-10 16:35 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-04 15:03 [PATCH v2 0/8] imx8: switch missing boards to binman Oliver Graute
2022-11-04 15:03 ` [PATCH v5 1/8] imx: imx8qm-rom7720: switch " Oliver Graute
2022-11-12 16:26   ` sbabic
2022-11-04 15:03 ` [PATCH v6 2/8] imx: imx8qm: cgtqmx8: " Oliver Graute
2022-11-12 16:26   ` sbabic
2022-11-04 15:03 ` [PATCH v2 3/8] imx: imx8qxp: imx8qxp_mek " Oliver Graute
2022-11-12 16:25   ` sbabic
2022-11-04 15:03 ` [PATCH v2 4/8] imx: imx8qm: imx8qm_mek " Oliver Graute
2022-11-12 16:26   ` sbabic
2022-11-04 15:03 ` [PATCH v2 5/8] imx: imx8qxp: giedi " Oliver Graute
2022-11-12 16:25   ` sbabic
2022-11-04 15:03 ` [PATCH v1 6/8] imx: imx8qxp: deneb " Oliver Graute
2022-11-12 16:26   ` sbabic
2022-11-04 15:03 ` [PATCH v2 7/8] imx: imx8x: colibri: " Oliver Graute
2022-11-12 16:25   ` sbabic
2022-11-04 15:03 ` [PATCH v2 8/8] imx: imx8: apalis: " Oliver Graute
2022-11-09 17:47   ` Marcel Ziswiler
2022-11-10 16:35     ` oliver.graute@kococonnector.com [this message]
2022-11-12 16:26   ` sbabic
2022-11-15 10:05     ` Marcel Ziswiler
2022-11-15 10:23       ` Stefano Babic
2022-11-04 16:31 ` [PATCH v2 0/8] imx8: switch missing boards " Fabio Estevam
2022-11-08 16:43   ` Stefano Babic
2022-11-08 22:16     ` Fabio Estevam
2022-11-09 15:44       ` Stefano Babic
2022-11-09 16:14       ` Oliver Graute
2022-11-09 16:19     ` Oliver Graute
2022-11-09 17:45       ` Fabio Estevam
2022-11-10 16:11         ` Tom Rini
2022-11-11 12:53         ` Oliver Graute
2022-11-11 17:40           ` Fabio Estevam
2022-11-11 17:55             ` Fabio Estevam
2022-11-16  9:47               ` Marcel Ziswiler
2022-11-18 13:41               ` Oliver Graute
2022-11-18 13:50                 ` Fabio Estevam
2022-11-22 14:23               ` Oliver Graute
2022-12-02  1:00                 ` Fabio Estevam
2022-11-18 12:35             ` Oliver Graute
2022-11-09 16:50   ` Marcel Ziswiler

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20221110163518.GC17674@optiplex \
    --to=oliver.graute@gmail.com \
    --cc=denys.drozdov@toradex.com \
    --cc=festevam@gmail.com \
    --cc=gaurav.jain@nxp.com \
    --cc=horia.geanta@nxp.com \
    --cc=marcel.ziswiler@toradex.com \
    --cc=peng.fan@nxp.com \
    --cc=sbabic@denx.de \
    --cc=u-boot@lists.denx.de \
    --cc=uboot-imx@nxp.com \
    --cc=ye.li@nxp.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox