All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chirag Shilwant <c-shilwant@ti.com>
To: <afd@ti.com>, <meta-ti@lists.yoctoproject.org>
Cc: Denys Dmytriyenko <denys@konsulko.com>,
	Ryan Eatmon <reatmon@ti.com>,
	"Raghavendra, Vignesh" <vigneshr@ti.com>,
	Praneeth Bajjuri <praneeth@ti.com>
Subject: Re: [meta-ti][scarthgap/master][PATCH 2/2] meta-ti-bsp: Add all tiboot3.bin variants by wildcard
Date: Tue, 1 Jul 2025 17:36:20 +0530	[thread overview]
Message-ID: <112eaca7-e4ba-411a-847c-cb352eae23da@ti.com> (raw)
In-Reply-To: <20250627193003.575911-2-afd@ti.com>

Hi Andrew,
On 28/06/25 01:00, Andrew Davis via lists.yoctoproject.org wrote:
> U-Boot generates all the valid versions of tiboot3.bin and picks the
> default based on the board's binman files. No need to duplicate this
> logic here, simply deploy what U-Boot gives us.
>
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>   meta-ti-bsp/conf/machine/include/am62axx.inc |  6 ------
>   meta-ti-bsp/conf/machine/include/am62dxx.inc |  6 ------
>   meta-ti-bsp/conf/machine/include/am62pxx.inc |  6 ------
>   meta-ti-bsp/conf/machine/include/am62xx.inc  |  9 ---------
>   meta-ti-bsp/conf/machine/include/am64xx.inc  |  9 ---------
>   meta-ti-bsp/conf/machine/include/am65xx.inc  |  2 --
>   meta-ti-bsp/conf/machine/include/am68.inc    |  6 ------
>   meta-ti-bsp/conf/machine/include/am69.inc    |  6 ------
>   meta-ti-bsp/conf/machine/include/j7200.inc   | 15 ---------------
>   meta-ti-bsp/conf/machine/include/j721e.inc   |  6 ------
>   meta-ti-bsp/conf/machine/include/j721s2.inc  |  9 ---------
>   meta-ti-bsp/conf/machine/include/j722s.inc   |  6 ------
>   meta-ti-bsp/conf/machine/include/j742s2.inc  |  6 ------
>   meta-ti-bsp/conf/machine/include/j784s4.inc  |  9 ---------
>   meta-ti-bsp/conf/machine/include/k3.inc      |  1 +
>   15 files changed, 1 insertion(+), 101 deletions(-)
>

<snip>

> diff --git a/meta-ti-bsp/conf/machine/include/k3.inc b/meta-ti-bsp/conf/machine/include/k3.inc
> index 9b85f867..d0f0bd62 100644
> --- a/meta-ti-bsp/conf/machine/include/k3.inc
> +++ b/meta-ti-bsp/conf/machine/include/k3.inc
> @@ -44,6 +44,7 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 alsa pci optee-ftpm"
>   IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
>   
>   IMAGE_BOOT_FILES ?= "${SPL_BINARYNAME} u-boot.${UBOOT_SUFFIX} tiboot3.bin"
> +IMAGE_BOOT_FILES += "tiboot3-*-evm.bin"


This commit seems to undo the effect of [0] which resolved SITSW-7159
There was a request to not package tiboot3-am62ax-gp-evm.bin inside vfat 
boot partition of wic image
but k3-am62a-sk-binman.dtsi [1] would still have it for Astc/vlab folks 
who know how to use the u-boot sources.


Hence, even if this approach to ship all tiboot3.bin generated by u-boot 
build seems logically correct
but it ends up deploying tiboot3 for GP samples which were only used 
internally for initial device bring-up
& not available publicly.


[0] 
https://git.yoctoproject.org/meta-ti/commit/?h=scarthgap-next&id=492a9aa4a92787185b41023e1026369d4028c1eb

[1] 
https://git.ti.com/cgit/ti-u-boot/ti-u-boot/tree/arch/arm/dts/k3-am62a-sk-binman.dtsi?h=11.01.04#n104


Chirag


>   IMAGE_EFI_BOOT_FILES ?= "${IMAGE_BOOT_FILES}"
>   
>   EFI_PROVIDER ?= "grub-efi"
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#18710): https://lists.yoctoproject.org/g/meta-ti/message/18710
> Mute This Topic: https://lists.yoctoproject.org/mt/113867726/7030289
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [c-shilwant@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>


  parent reply	other threads:[~2025-07-01 12:06 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-27 19:30 [meta-ti][scarthgap/master][PATCH 1/2] meta-ti-bsp: conf: machine: Deploy all versions of tiboot3.bin Andrew Davis
2025-06-27 19:30 ` [meta-ti][scarthgap/master][PATCH 2/2] meta-ti-bsp: Add all tiboot3.bin variants by wildcard Andrew Davis
2025-06-27 19:48   ` Ryan Eatmon
2025-06-27 20:02     ` Andrew Davis
2025-07-01 12:06   ` Chirag Shilwant [this message]
2025-07-01 13:25     ` Andrew Davis
2025-07-01 15:36       ` Ryan Eatmon
2025-06-27 19:45 ` [meta-ti][scarthgap/master][PATCH 1/2] meta-ti-bsp: conf: machine: Deploy all versions of tiboot3.bin Ryan Eatmon
2025-06-27 19:58   ` Andrew Davis
2025-07-01 21:22 ` Ryan Eatmon

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=112eaca7-e4ba-411a-847c-cb352eae23da@ti.com \
    --to=c-shilwant@ti.com \
    --cc=afd@ti.com \
    --cc=denys@konsulko.com \
    --cc=meta-ti@lists.yoctoproject.org \
    --cc=praneeth@ti.com \
    --cc=reatmon@ti.com \
    --cc=vigneshr@ti.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.