All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
To: "tharvey@gateworks.com" <tharvey@gateworks.com>
Cc: "festevam@gmail.com" <festevam@gmail.com>,
	"u-boot@lists.denx.de" <u-boot@lists.denx.de>,
	"uboot-imx@nxp.com" <uboot-imx@nxp.com>,
	"sjg@chromium.org" <sjg@chromium.org>,
	"sbabic@denx.de" <sbabic@denx.de>,
	"frieder.schrempf@kontron.de" <frieder.schrempf@kontron.de>,
	"trini@konsulko.com" <trini@konsulko.com>
Subject: Re: [RFC PATCH v1 2/5] arm64: dts: imx8mm: add common -binman.dtsi
Date: Wed, 6 Oct 2021 23:51:42 +0000	[thread overview]
Message-ID: <95b23e758219dce3005644a3a3220e9b55177331.camel@toradex.com> (raw)
In-Reply-To: <CAJ+vNU3ebOkfAohNmWYroUECVo9C6oi30VEcv6cDzpgU2tZvBw@mail.gmail.com>

Hi Tim

On Thu, 2021-08-26 at 07:57 -0700, Tim Harvey wrote:
> ...
> Marcel,
> 
> This would break imx8mm-venice.
> 
> The of-list, @fdt-SEQ and @config-SEQ are required to support
> automatic generation of fdt and config nodes when you have multiple
> device-trees. See tools/binman/entries.rst for details.
> 
> Currently, imx8mm-venice is the only board that uses multiple dtbs in
> CONFIG_OF_LIST so this would be the only board broken by your patch.
> I'm curious if just using the of-list, @fdt-SEQ and @config-SEQ as
> above works fine for boards with a single dtb?

Yes, I verified that now and will be sending a v2 with that shortly. Thanks!

> Otherwise I suppose
> boards like mine can include your common "imx8mm-binman.dtsi" but will
> have to leave the &binman node override.
> 
> Best regards,
> 
> Tim

Cheers

Marcel

  parent reply	other threads:[~2021-10-06 23:51 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-26 12:14 [RFC PATCH v1 0/5] arm64: dts: imx8mm: add common -binman.dtsi and further clean-up Marcel Ziswiler
2021-08-26 12:14 ` [RFC PATCH v1 1/5] imx8mm-cl-iot-gate: fix imximage intermediate binary naming Marcel Ziswiler
2021-08-26 12:14 ` [RFC PATCH v1 2/5] arm64: dts: imx8mm: add common -binman.dtsi Marcel Ziswiler
2021-08-26 14:57   ` Tim Harvey
2021-08-26 16:01     ` Marcel Ziswiler
2021-10-06 23:51     ` Marcel Ziswiler [this message]
2021-08-27  9:21   ` Teresa Remmet
2021-08-27 10:59     ` Marcel Ziswiler
2021-08-27 15:12       ` Tim Harvey
2021-08-28 12:55         ` Marcel Ziswiler
2021-08-26 12:14 ` [RFC PATCH v1 3/5] arm64: dts: imx8mm-binman.dtsi: alphabetically re-order properties Marcel Ziswiler
2021-08-26 12:14 ` [RFC PATCH v1 4/5] arm64: dts: imx8mm-binman.dtsi: explicitly add spl filename Marcel Ziswiler
2021-08-26 12:14 ` [RFC PATCH v1 5/5] arm64: dts: imx8mm-binman.dtsi: improve odd blob-ext naming Marcel Ziswiler
2021-08-26 14:58 ` [RFC PATCH v1 0/5] arm64: dts: imx8mm: add common -binman.dtsi and further clean-up Tim Harvey
2021-09-02 17:08 ` Fabio Estevam
2021-10-06 23:53   ` 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=95b23e758219dce3005644a3a3220e9b55177331.camel@toradex.com \
    --to=marcel.ziswiler@toradex.com \
    --cc=festevam@gmail.com \
    --cc=frieder.schrempf@kontron.de \
    --cc=sbabic@denx.de \
    --cc=sjg@chromium.org \
    --cc=tharvey@gateworks.com \
    --cc=trini@konsulko.com \
    --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.