From: Alexander Sverdlin <alexander.sverdlin@gmail.com>
To: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Vincent Jardin <vjardin@free.fr>,
Romain Naour <romain.naour@smile.fr>,
arnout@rnout.be, buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH v3] boot/uboot: support custom DTS with CONFIG_OF_UPSTREAM
Date: Sun, 18 Oct 2026 21:45:45 +0200 [thread overview]
Message-ID: <c5aca2bcccb4018ee1d1923ce3262177977003eb.camel@gmail.com> (raw)
In-Reply-To: <atUeDroTYvzii7To@windsurf>
Hi Thomas!
On Sun, 2026-10-18 at 21:32 +0200, Thomas Petazzoni wrote:
> > Since U-Boot 2024.07, boards with CONFIG_OF_UPSTREAM build their device
> > trees from dts/upstream/src/<arch>/<vendor>/ instead of arch/<arch>/dts/,
> > so the files copied by BR2_TARGET_UBOOT_CUSTOM_DTS_PATH were misplaced.
> >
> > Derive the destination from the U-Boot configuration, including the vendor
> > sub-directory taken from CONFIG_DEFAULT_DEVICE_TREE.
> >
> > Signed-off-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>
>
> Thanks for the patch. However, I don't really understand the
> use-case. If your board DT is not in the upstream Linux kernel, why
> would you enable CONFIG_OF_UPSTREAM in the first place?
>
> The whole point of CONFIG_OF_UPSTREAM=y is to tell U-Boot "use the DT
> from the upstream kernel". But if you have to copy your own DT, it
> means your board DT is not in upstream Linux. So, just disable
> CONFIG_OF_UPSTREAM in your U-Boot configuration, and use the existing
> BR2_TARGET_UBOOT_CUSTOM_DTS_PATH option that copies to arch/ARCH/dts/
> and does the job.
>
> Could you clarify why we would want this change? I'm kind of missing
> the point.
This is indeed a valid point!
I've been looking into it from the point of implementing a BSP for a board
which have been already merged into upstream Linux, but not into U-Boot
yet, but you are right, for this intermediate step I'm not forced to
use CONFIG_OF_UPSTREAM even if it would be used for the same board at
some point in future (when the DTs will be merged into U-Boot as well).
Sorry for the noise, let's ignore the patch for now!
--
Alexander Sverdlin.
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2026-10-18 19:46 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-03 21:06 [Buildroot] [PATCH v3] boot/uboot: support custom DTS with CONFIG_OF_UPSTREAM Alexander Sverdlin
2026-10-18 19:32 ` Thomas Petazzoni via buildroot
2026-10-18 19:45 ` Alexander Sverdlin [this message]
2026-10-19 7:42 ` Thomas Petazzoni via buildroot
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=c5aca2bcccb4018ee1d1923ce3262177977003eb.camel@gmail.com \
--to=alexander.sverdlin@gmail.com \
--cc=arnout@rnout.be \
--cc=buildroot@buildroot.org \
--cc=romain.naour@smile.fr \
--cc=thomas.petazzoni@bootlin.com \
--cc=vjardin@free.fr \
/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.