All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
To: "heiko.thiery@gmail.com" <heiko.thiery@gmail.com>
Cc: "sjg@chromium.org" <sjg@chromium.org>,
	Max Krummenacher <max.krummenacher@toradex.com>,
	"uboot-imx@nxp.com" <uboot-imx@nxp.com>,
	"hs@denx.de" <hs@denx.de>,
	"festevam@gmail.com" <festevam@gmail.com>,
	"peng.fan@nxp.com" <peng.fan@nxp.com>,
	"sbabic@denx.de" <sbabic@denx.de>,
	"jagan@amarulasolutions.com" <jagan@amarulasolutions.com>,
	"iliev@ronetix.at" <iliev@ronetix.at>,
	"marex@denx.de" <marex@denx.de>,
	"igor.opaniuk@gmail.com" <igor.opaniuk@gmail.com>,
	"u-boot@lists.denx.de" <u-boot@lists.denx.de>,
	"t.remmet@phytec.de" <t.remmet@phytec.de>,
	"frieder.schrempf@kontron.de" <frieder.schrempf@kontron.de>
Subject: Re: [PATCH v1 5/7] verdin-imx8mm: switch to use binman to pack images
Date: Mon, 23 Aug 2021 10:02:46 +0000	[thread overview]
Message-ID: <717a73b9ee216663e93f532b698910627e0bce64.camel@toradex.com> (raw)
In-Reply-To: <CAEyMn7ZvceO0k3K6+kVns2kqLrM4mgBMCdpFkPo+bufwEzFSVw@mail.gmail.com>

Hi Heiko

On Mon, 2021-08-23 at 08:37 +0200, Heiko Thiery wrote:

> ...
> > diff --git a/board/toradex/verdin-imx8mm/imximage.cfg b/board/toradex/verdin-imx8mm/imximage.cfg
> > index b8b25ff4201..9bd60e39fb5 100644
> > --- a/board/toradex/verdin-imx8mm/imximage.cfg
> > +++ b/board/toradex/verdin-imx8mm/imximage.cfg
> > @@ -1,16 +1,9 @@
> >  /* SPDX-License-Identifier: GPL-2.0+ */
> >  /*
> > - * Copyright 2020 Toradex
> > + * Copyright 2021 Toradex
> >   */
> > 
> >  #define __ASSEMBLY__
> > 
> > -FIT
> >  BOOT_FROM      emmc_fastboot
> > -LOADER         spl/u-boot-spl-ddr.bin  0x7E1000
> > -SECOND_LOADER  u-boot.itb              0x40200000 0x60000
> > -
> > -DDR_FW lpddr4_pmu_train_1d_imem.bin
> > -DDR_FW lpddr4_pmu_train_1d_dmem.bin
> > -DDR_FW lpddr4_pmu_train_2d_imem.bin
> > -DDR_FW lpddr4_pmu_train_2d_dmem.bin
> > +LOADER         mkimage.spl.mkimage     0x7E1000
> 
> I think you should use "spl.bin" here and not the
> "mkimage.spl.mkimage". As far as I understand the mkimage files are
> only binman temporary files.

Unfortunately, that does not seem to work:

binman: Error 1 running 'mkimage -d ./mkimage.spl.mkimage -n spl/u-boot-spl.cfgout -T imx8mimage -e 0x7e1000
./mkimage-out.spl.mkimage': spl.bin: Can't open: No such file or directory

Thinking about it, I guess, it is a chicken/egg problem. As it is that step of binman which actually creates
spl.bin from this mkimage.spl.mkimage intermediate artifact, not?

Cheers

Marcel

  parent reply	other threads:[~2021-08-23 10:02 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-20 20:52 [PATCH v1 0/7] board: toradex: verdin-imx8mm: target refresh Marcel Ziswiler
2021-08-20 20:52 ` [PATCH v1 1/7] imx: mkimage_fit_atf: fix legacy image generation Marcel Ziswiler
2021-08-22 10:07   ` ZHIZHIKIN Andrey
2021-08-23  6:56     ` Marcel Ziswiler
2021-08-23  7:50       ` ZHIZHIKIN Andrey
2021-08-23 10:59         ` Fabio Estevam
2021-08-23 15:53           ` ZHIZHIKIN Andrey
2021-08-23 17:27             ` Fabio Estevam
2021-08-20 20:52 ` [PATCH v1 2/7] imx8m: clean-up kconfig indentation Marcel Ziswiler
2021-08-20 20:52 ` [PATCH v1 3/7] verdin-imx8mm: fix ethernet Marcel Ziswiler
2021-08-20 20:52 ` [PATCH v1 4/7] ARM: dts: imx8mm-verdin: prepare for dek blob encapsulation Marcel Ziswiler
2021-08-20 20:52 ` [PATCH v1 5/7] verdin-imx8mm: switch to use binman to pack images Marcel Ziswiler
2021-08-23  6:37   ` Heiko Thiery
2021-08-23  7:00     ` Marcel Ziswiler
2021-08-23 10:02     ` Marcel Ziswiler [this message]
2021-08-23 10:43       ` Heiko Thiery
2021-08-20 20:52 ` [PATCH v1 6/7] verdin-imx8mm: enable sleep_moci output Marcel Ziswiler
2021-08-20 20:52 ` [PATCH v1 7/7] verdin-imx8mm: drop support for v1.0 hardware 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=717a73b9ee216663e93f532b698910627e0bce64.camel@toradex.com \
    --to=marcel.ziswiler@toradex.com \
    --cc=festevam@gmail.com \
    --cc=frieder.schrempf@kontron.de \
    --cc=heiko.thiery@gmail.com \
    --cc=hs@denx.de \
    --cc=igor.opaniuk@gmail.com \
    --cc=iliev@ronetix.at \
    --cc=jagan@amarulasolutions.com \
    --cc=marex@denx.de \
    --cc=max.krummenacher@toradex.com \
    --cc=peng.fan@nxp.com \
    --cc=sbabic@denx.de \
    --cc=sjg@chromium.org \
    --cc=t.remmet@phytec.de \
    --cc=u-boot@lists.denx.de \
    --cc=uboot-imx@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.