From: Robert Schwebel <r.schwebel@pengutronix.de>
To: Ahmad Fatoum <a.fatoum@pengutronix.de>
Cc: Alexander Dahl <ada@thorsis.com>,
distrokit@pengutronix.de, Roland Hieber <rhi@pengutronix.de>
Subject: Re: [DistroKit] [PATCH v2 0/3] v7a: barebox: Support FIT images
Date: Mon, 11 Mar 2024 13:05:46 +0100 [thread overview]
Message-ID: <Ze7zms8T9JIJtRLY@pengutronix.de> (raw)
In-Reply-To: <20240306092530.1448784-1-a.fatoum@pengutronix.de>
Applied to next.
On Wed, Mar 06, 2024 at 10:25:27AM +0100, Ahmad Fatoum wrote:
> In coordination with Alex, I am sending a v2 with a fix for the rpi
> regression reported by Roland. The fix will be part of v2024.03.0, which
> is not yet released. Since then, v2024.02.0 was released, but I think
> we should just update to v2024.03.0, once that's released and add FIT
> support on top of v2024.01.0.
>
> Changelog is alongside the individual patches.
>
> Original coverletter:
>
> Hei hei,
>
> while working on a BSP using DistroKit as a base layer I try to support
> boards supported by DistroKit already plus some more boards where the
> necessary things are in my upper layer. One task is to use a FIT image
> for kernel and dts usable with both U-Boot and barebox. This is
> possible with this series and some tweaks which hit ptxdist master
> lately.
>
> One nasty problem is bootstate.dtsi referenced in barebox config option
> CONFIG_EXTERNAL_DTS_FRAGMENTS as
> '${PTXDIST_PLATFORMCONFIGDIR}/dts/bootstate.dtsi' currently. When using
> DistroKit as a base layer that file can not be found. I experimented
> with '$(call ptx/in-platformconfigdir, …' as adviced in docs, but that's
> probably not meant for Kconfig? Build runs successfully, but no
> bootstate.dtsi is included at all then, leading to barebox not booting
> anything because it can not find its state.
>
> The only solution I could come up with is to use a symbolic link with a
> relative path in the upper layer to bootstate.dtsi in the base layer,
> but I don't consider that elegant. And it would have to be done in each
> layer stacked on top for everyone using DistroKit as a base layer.
> If anyone can propose a better solution for that, please advice!
>
> Otherwise I think all three patches are useful for themselves. I could
> not test on other platforms however so the two patches touching barebox
> might be somewhat incomplete?
>
>
> Alexander Dahl (3):
> platform: v7a: barebox: Upgrade from 2023.12.0 to 2024.01.0
> blspec-entry: Backport fix from ptxdist master
> platform: v7a: barebox: Enable FIT image support
>
> .../platform-v7a/barebox-am335x-mlo.config | 10 +-
> configs/platform-v7a/barebox-am335x.config | 19 ++-
> .../platform-v7a/barebox-am335x.config.diff | 3 +-
> configs/platform-v7a/barebox-at91.config | 19 ++-
> configs/platform-v7a/barebox-at91.config.diff | 3 +-
> configs/platform-v7a/barebox-mx6.config | 20 ++-
> configs/platform-v7a/barebox-mx6.config.diff | 4 +-
> configs/platform-v7a/barebox-rpi2.config | 20 ++-
> configs/platform-v7a/barebox-rpi2.config.diff | 2 +-
> configs/platform-v7a/barebox-stm32mp.config | 21 ++-
> .../platform-v7a/barebox-stm32mp.config.diff | 3 +-
> configs/platform-v7a/barebox-vexpress.config | 19 ++-
> .../platform-v7a/barebox-vexpress.config.diff | 3 +-
> configs/platform-v7a/barebox.config | 20 ++-
> ...ression-booting-without-VideoCore-DT.patch | 130 ++++++++++++++++++
> .../patches/barebox-2024.01.0/series | 1 +
> configs/platform-v7a/platformconfig | 4 +-
> rules/blspec-entry.make | 2 +-
> 18 files changed, 265 insertions(+), 38 deletions(-)
> create mode 100644 configs/platform-v7a/patches/barebox-2024.01.0/0001-ARM-rpi-fix-regression-booting-without-VideoCore-DT.patch
> create mode 100644 configs/platform-v7a/patches/barebox-2024.01.0/series
>
> --
> 2.39.2
>
>
>
--
Pengutronix e.K. | Dipl.-Ing. Robert Schwebel |
Steuerwalder Str. 21 | https://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 |
prev parent reply other threads:[~2024-03-11 12:05 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-06 9:25 Ahmad Fatoum
2024-03-06 9:25 ` [DistroKit] [PATCH v2 1/3] platform: v7a: barebox: Upgrade from 2023.12.0 to 2024.01.0 Ahmad Fatoum
2024-03-13 11:59 ` Roland Hieber
2024-03-06 9:25 ` [DistroKit] [PATCH v2 2/3] blspec-entry: Backport fix from ptxdist master Ahmad Fatoum
2024-03-06 9:25 ` [DistroKit] [PATCH v2 3/3] platform: v7a: barebox: Enable FIT image support Ahmad Fatoum
2024-03-11 12:05 ` Robert Schwebel [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=Ze7zms8T9JIJtRLY@pengutronix.de \
--to=r.schwebel@pengutronix.de \
--cc=a.fatoum@pengutronix.de \
--cc=ada@thorsis.com \
--cc=distrokit@pengutronix.de \
--cc=rhi@pengutronix.de \
/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