From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id E17CDC369AB for ; Wed, 16 Apr 2025 00:11:49 +0000 (UTC) Received: from mailout4.zoneedit.com (mailout4.zoneedit.com [64.68.198.64]) by mx.groups.io with SMTP id smtpd.web10.8501.1744762303816630540 for ; Tue, 15 Apr 2025 17:11:44 -0700 Authentication-Results: mx.groups.io; dkim=none (message not signed); spf=pass (domain: denix.org, ip: 64.68.198.64, mailfrom: denis@denix.org) Received: from localhost (localhost [127.0.0.1]) by mailout4.zoneedit.com (Postfix) with ESMTP id AE30F40C61; Wed, 16 Apr 2025 00:11:42 +0000 (UTC) Received: from mailout4.zoneedit.com ([127.0.0.1]) by localhost (zmo14-pco.easydns.vpn [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id qaFJ2_kO5DO4; Wed, 16 Apr 2025 00:11:42 +0000 (UTC) Received: from mail.denix.org (pool-100-15-87-159.washdc.fios.verizon.net [100.15.87.159]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mailout4.zoneedit.com (Postfix) with ESMTPSA id 7DD5E40A46; Wed, 16 Apr 2025 00:11:40 +0000 (UTC) Received: by mail.denix.org (Postfix, from userid 1000) id DB98D1662FD; Tue, 15 Apr 2025 20:11:39 -0400 (EDT) Date: Tue, 15 Apr 2025 20:11:39 -0400 From: Denys Dmytriyenko To: anshuld@ti.com Cc: meta-ti@lists.yoctoproject.org, vigneshr@ti.com Subject: Re: [meta-ti] [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs Message-ID: <20250416001139.GS13634@denix.org> References: <20250415093645.435281-1-anshuld@ti.com> <20250415093645.435281-3-anshuld@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250415093645.435281-3-anshuld@ti.com> User-Agent: Mutt/1.5.20 (2009-06-14) List-Id: X-Webhook-Received: from li982-79.members.linode.com [45.33.32.79] by aws-us-west-2-korg-lkml-1.web.codeaurora.org with HTTPS for ; Wed, 16 Apr 2025 00:11:49 -0000 X-Groupsio-URL: https://lists.yoctoproject.org/g/meta-ti/message/18472 On Tue, Apr 15, 2025 at 03:06:44PM +0530, Anshul Dalal via lists.yoctoproject.org wrote: > Falcon boot requires tispl_falcon.bin to be on the same filesystem > partition as the fitImage. > > This patch creates a new u-boot package called > 'u-boot-ti-staging-falcon' that is included in IMAGE_INSTALL when > ti-falcon is enabled which allows us to deploy the required files to > rootfs. > > Signed-off-by: Anshul Dalal > --- > meta-ti-bsp/conf/machine/include/ti-falcon.inc | 1 + > meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 11 +++++++++++ > 2 files changed, 12 insertions(+) > > diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc > index b571c04d..9f93c683 100644 > --- a/meta-ti-bsp/conf/machine/include/ti-falcon.inc > +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc > @@ -1,3 +1,4 @@ > +IMAGE_INSTALL:append = " u-boot-ti-staging-falcon" Why += cannot be used here? > KERNEL_CLASSES:remove = "kernel-fitimage" > KERNEL_CLASSES += "kernel-fitimage-legacyhs" > UBOOT_DTB_LOADADDRESS = "0x82000000" > diff --git a/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc b/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc > index 41d4abdd..76803ae3 100644 > --- a/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc > +++ b/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc > @@ -14,6 +14,8 @@ require u-boot-mergeconfig.inc > > FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:" > > +FILES:${PN}-falcon = "boot/tispl_falcon.bin" > + > SUMMARY = "u-boot bootloader for TI devices" > > LICENSE = "GPL-2.0-or-later" > @@ -373,4 +375,13 @@ do_deploy:append:am62pxx() { > install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR} > } > > +do_install:append:ti-falcon() { > + install -m 0644 ${B}/tispl_falcon.bin ${D}/boot > +} > + > +PACKAGES:prepend:am62xx-evm = "${PN}-falcon " > +PACKAGES:prepend:am62axx-evm = "${PN}-falcon " > +PACKAGES:prepend:am62pxx-evm = "${PN}-falcon " > +PACKAGES:prepend:am62xx-lp-evm = "${PN}-falcon " Shouldn't these be specific to ti-falcon only? > + > TOOLCHAIN = "gcc"