DistroKit Mailinglist
 help / color / mirror / Atom feed
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    |



      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