From: Paolo Bonzini <pbonzini@redhat.com>
To: Bernhard Beschow <shentey@gmail.com>, qemu-devel@nongnu.org
Cc: qemu-arm@nongnu.org, "Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
qemu-ppc@nongnu.org,
"Marc-André Lureau" <marcandre.lureau@redhat.com>,
"Daniel P. Berrangé" <berrange@redhat.com>
Subject: Re: [PATCH 0/5] DTB build housekeeping
Date: Thu, 12 Jun 2025 08:09:04 +0200 [thread overview]
Message-ID: <f3a4daf7-19ca-4938-bce8-dbe7d1ff8649@redhat.com> (raw)
In-Reply-To: <20250610204131.2862-1-shentey@gmail.com>
On 6/10/25 22:41, Bernhard Beschow wrote:
> This series removes the bundled device tree blobs from the repository which are
> redundant to the bundled device tree sources. To make this work, the device tree
> compiler 'dtc' is now required for the boards which rely on these device trees.
> Building these board can be enforced with a 'dtc' build option which should
> help packagers. For simplicity, 'dtc' is searched for in the system only rather
> than also considering the 'dtc' submodule.
>
> I also tried unsing Meson's source_set::add(when: ...) for building only the
> needed DTBs but eventually found it to be too over-engineered.
>
> Last but not least two inconsistencies regarding QEMU_FILE_TYPE_DTB are fixed
> which seem to be latent bugs (one of which materializes in my e500-fdt branch).
Applied patches 1-3, thanks.
Paolo
> Bernhard Beschow (5):
> hw: Fix type constant for DTB files
> pc-bios/dtb/meson: Prefer target name to be outfile, not infile
> scripts/meson-buildoptions: Sort coroutine_backend choices
> lexicographically
> configure: Add dtc option
> configure: Require dtc for boards with bundled device trees
>
> MAINTAINERS | 2 +-
> meson.build | 12 ++++++++++++
> hw/arm/boot.c | 2 +-
> hw/ppc/e500.c | 2 +-
> Kconfig.host | 3 +++
> hw/microblaze/Kconfig | 4 ++--
> hw/ppc/Kconfig | 4 ++--
> meson_options.txt | 3 +++
> pc-bios/dtb/bamboo.dtb | Bin 3211 -> 0 bytes
> pc-bios/dtb/canyonlands.dtb | Bin 9779 -> 0 bytes
> pc-bios/dtb/meson.build | 19 ++++++++-----------
> pc-bios/dtb/petalogix-ml605.dtb | Bin 9882 -> 0 bytes
> pc-bios/dtb/petalogix-s3adsp1800.dtb | Bin 8161 -> 0 bytes
> scripts/meson-buildoptions.sh | 5 ++++-
> 14 files changed, 37 insertions(+), 19 deletions(-)
> delete mode 100644 pc-bios/dtb/bamboo.dtb
> delete mode 100644 pc-bios/dtb/canyonlands.dtb
> delete mode 100644 pc-bios/dtb/petalogix-ml605.dtb
> delete mode 100644 pc-bios/dtb/petalogix-s3adsp1800.dtb
>
prev parent reply other threads:[~2025-06-12 6:09 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-06-10 20:41 [PATCH 0/5] DTB build housekeeping Bernhard Beschow
2025-06-10 20:41 ` [PATCH 1/5] hw: Fix type constant for DTB files Bernhard Beschow
2025-06-11 10:31 ` Thomas Huth
2025-06-10 20:41 ` [PATCH 2/5] pc-bios/dtb/meson: Prefer target name to be outfile, not infile Bernhard Beschow
2025-06-11 10:38 ` Thomas Huth
2025-06-10 20:41 ` [PATCH 3/5] scripts/meson-buildoptions: Sort coroutine_backend choices lexicographically Bernhard Beschow
2025-06-11 0:37 ` BALATON Zoltan
2025-06-11 10:40 ` Thomas Huth
2025-06-11 12:59 ` Kohei Tokunaga
2025-06-10 20:41 ` [PATCH 4/5] configure: Add dtc option Bernhard Beschow
2025-06-11 0:37 ` BALATON Zoltan
2025-06-11 8:45 ` Daniel P. Berrangé
2025-06-11 11:45 ` BALATON Zoltan
2025-06-12 5:12 ` Paolo Bonzini
2025-06-12 5:46 ` Paolo Bonzini
2025-06-12 21:27 ` Bernhard Beschow
2025-06-12 12:39 ` BALATON Zoltan
2025-06-10 20:41 ` [PATCH 5/5] configure: Require dtc for boards with bundled device trees Bernhard Beschow
2025-06-12 6:09 ` Paolo Bonzini [this message]
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=f3a4daf7-19ca-4938-bce8-dbe7d1ff8649@redhat.com \
--to=pbonzini@redhat.com \
--cc=berrange@redhat.com \
--cc=edgar.iglesias@gmail.com \
--cc=marcandre.lureau@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=philmd@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=shentey@gmail.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.