All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andreas Dannenberg via buildroot <buildroot@buildroot.org>
To: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Dario Binacchi <dario.binacchi@amarulasolutions.com>,
	linux-amarula@amarulasolutions.com,
	Asaf Kahlon <asafka7@gmail.com>,
	Xuanhao Shi <X15000177@gmail.com>,
	James Hilliard <james.hilliard1@gmail.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	buildroot@buildroot.org, Romain Naour <romain.naour@smile.fr>,
	michael@amarulasolutions.com,
	Alexander Sverdlin <alexander.sverdlin@gmail.com>,
	bryce@redpinelabs.com, Anand Gadiyar <gadiyar@ti.com>
Subject: Re: [Buildroot] [PATCH v9 05/22] boot/ti-k3-r5-loader: bump to version 2024.01
Date: Wed, 20 Mar 2024 04:17:15 -0500	[thread overview]
Message-ID: <20240320091715.a65mdipyyp3qbk35@dasso> (raw)
In-Reply-To: <Zfp-rk1BDxhDnkZi@landeda>

Hi Yann,

On Wed, Mar 20, 2024 at 07:14:06AM +0100, Yann E. MORIN wrote:
> Andreas, All,
> 
> On 2024-03-19 21:18 -0500, Andreas Dannenberg via buildroot spake thusly:
> > On Tue, Mar 19, 2024 at 11:09:32PM +0100, Yann E. MORIN wrote:
> > > On 2024-03-04 16:32 +0100, Dario Binacchi spake thusly:
> > > > All in-tree configs with the ti-k3-r5 bootloader use a custom version,
> > > > so this patch is mostly for the menuconfig default version
> [--SNIP--]
> > > Speaking of that, by the way, ti-k3-r5-loader really is uboot, so I
> > > think that it should share:
> > >  1. the same DL_DIR:   TI_K3_R5_LOADER_DL_SUBDIR = uboot
> > >  2. the same hash file:  have ti-k3-r5-loader.hash be a symlink to
> > >     uboot.hash  (and have a xomment at the top of that hash file that it
> > >     is shared and that old hashes should/can be kept)
> > The entire boot chain for TI K3 devices (like all of AM62x) comprising
> > what's known as ti-k3-r5-loaer in Buildroot as well as U-Boot SPL/U-Boot
> > [proper] should _always_ be built from the same U-Boot source commit, as
> > this is how it's developed, tested, and used by pretty much everybody.
> > So your suggestion to more closely tie those things together makes a lot
> > of sense IMHO.
> 
> Ah, interesting, thanks for the feedback.
> 
> So, it looks like for the situation for ti-k3-r5-loaer vs. U-Boot is
> very similar to the one for barebox: it can build a "base" barebox, and
> an "aux" barebox; they both share the same implementation (the
> barebox-package mini-infra), they each provide their own (def|.)config
> file, and they each have their own set of (Buildroot) options (the aux
> one has fewer options).
> 
> So it looks like this is what we should have done for ti-k3-r5-loaer,
> no?

I'm not familiar with barebox but what you are describing looks similar.
Note that ti-k3-r5-loader (building the initial boot binary) is building
for 32-bit ARMv7, and the next boot stages (U-Boot SPL, U-Boot) are all
64-bit ARMv8, so this might complicate any possible unification efforts.
 
> Also, does it make sense to use ti-k3-r5-loaer without U-Boot?

Yes, you could do "Falcon Boot" where the ti-k3-r5-loader would directly
load the Linux kernel, greatly simplifing the boot flow and booting much
MUCH faster. Actually it's a much-requested feature from our customers
(you know, the real world use cases :) so I wrote a technical note [1]
about that last year on how to set this up manually but it would be
great if we could bring this into Buildroot as a package/option somehow
at some point, after we get the base support accepted into the tree.

[1]
https://e2e.ti.com/support/processors-group/processors/f/processors-forum/1205638/faq-am625-how-to-boot-from-r5-u-boot-spl-directly-into-the-linux-kernel-skipping-a53-spl-and-a53-u-boot-falcon-mode


--
Andreas Dannenberg
Texas Instruments Inc




> 
> Regards,
> Yann E. MORIN.
> 
> -- 
> .-----------------.--------------------.------------------.--------------------.
> |  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> | +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
> | +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
> '------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2024-03-20  9:17 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-04 15:32 [Buildroot] [PATCH v9 00/22] Add support for AM62x-SK HS-FS devices Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 01/22] configs/ti_am62x_sk_defconfig: fix BR2_TARGET_OPTEE_OS_PLATFORM setting Dario Binacchi
2024-03-19 21:37   ` Yann E. MORIN
2024-03-04 15:32 ` [Buildroot] [PATCH v9 02/22] configs/ti_am64x_sk_defconfig: " Dario Binacchi
2024-03-19 21:38   ` Yann E. MORIN
2024-03-04 15:32 ` [Buildroot] [PATCH v9 03/22] configs/ti_am62x_sk_defconfig: explicitly set the ti-k3-r5-loader version Dario Binacchi
2024-03-19 21:40   ` Yann E. MORIN
2024-03-04 15:32 ` [Buildroot] [PATCH v9 04/22] configs/ti_am64x_sk_defconfig: " Dario Binacchi
2024-03-19 21:40   ` Yann E. MORIN
2024-03-04 15:32 ` [Buildroot] [PATCH v9 05/22] boot/ti-k3-r5-loader: bump to version 2024.01 Dario Binacchi
2024-03-19 22:09   ` Yann E. MORIN
2024-03-20  2:18     ` Andreas Dannenberg via buildroot
2024-03-20  6:14       ` Yann E. MORIN
2024-03-20  9:17         ` Andreas Dannenberg via buildroot [this message]
2024-03-24 16:22     ` Dario Binacchi
2024-03-26  7:22       ` Andreas Dannenberg via buildroot
2024-03-04 15:32 ` [Buildroot] [PATCH v9 06/22] package/python-attrs: add host variant Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 07/22] package/python-rpds-py: " Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 08/22] package/python-referencing: " Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 09/22] package/python-jsonschema-specifications: " Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 10/22] package/python-jsonschema: " Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 11/22] boot/ti-k3-r5-loader: set binman environment Dario Binacchi
2024-03-19 15:16   ` Bryan Brattlof via buildroot
2024-04-03  8:28     ` Romain Naour
2024-04-03  8:34       ` Michael Nazzareno Trimarchi
2024-04-03  9:37         ` Romain Naour
2024-04-03  9:46           ` Michael Nazzareno Trimarchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 12/22] boot/uboot: add BR2_TARGET_UBOOT_USE_BINMAN option Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 13/22] package/ti-k3: move TI_K3_{SECTYPE, SOC} out of the image-gen scope Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 14/22] package/ti-k3: disable HS-FS for AM65 SOC Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 15/22] boot/ti-k3/ti-k3-r5-loader: install tiboot3.bin Dario Binacchi
2024-03-19 15:11   ` Bryan Brattlof via buildroot
2024-03-20  9:20     ` Andreas Dannenberg via buildroot
2024-03-04 15:32 ` [Buildroot] [PATCH v9 16/22] configs/ti_am62x_sk_defconfig: bump U-Boot version to 2024.01 Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 17/22] configs/ti_am62x_sk_defconfig: bump Linux version to 6.6.18 Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 18/22] board/ti/am62x-sk: generalize post-build.sh Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 19/22] board/ti/am62x-sk: move post-{build, image}.sh to board/ti/common/am6xx Dario Binacchi
2024-03-13 10:56   ` Romain Naour
2024-03-13 11:29     ` Michael Nazzareno Trimarchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 20/22] configs/ti_am64x_sk_defconfig: bump U-Boot version to 2024.01 Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 21/22] configs/ti_am64x_sk_defconfig: bump Linux version to 6.6.18 Dario Binacchi
2024-03-04 15:32 ` [Buildroot] [PATCH v9 22/22] boot/ti-k3-image-gen: remove package Dario Binacchi
2024-03-13 11:00   ` Romain Naour
2024-03-13 11:28     ` Michael Nazzareno Trimarchi
2024-03-24 17:20     ` Dario Binacchi
2024-03-27 14:55       ` Romain Naour
2024-03-28 14:32         ` Heiko Thiery
2024-03-18  7:35   ` Michael Nazzareno Trimarchi
2024-03-12  6:42 ` [Buildroot] [PATCH v9 00/22] Add support for AM62x-SK HS-FS devices Andreas Dannenberg via buildroot
2024-03-19 22:24 ` Yann E. MORIN

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=20240320091715.a65mdipyyp3qbk35@dasso \
    --to=buildroot@buildroot.org \
    --cc=X15000177@gmail.com \
    --cc=alexander.sverdlin@gmail.com \
    --cc=asafka7@gmail.com \
    --cc=bryce@redpinelabs.com \
    --cc=dannenberg@ti.com \
    --cc=dario.binacchi@amarulasolutions.com \
    --cc=gadiyar@ti.com \
    --cc=james.hilliard1@gmail.com \
    --cc=linux-amarula@amarulasolutions.com \
    --cc=michael@amarulasolutions.com \
    --cc=romain.naour@smile.fr \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=yann.morin.1998@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.