| Message ID | 20260216230422.635024-2-ivitro@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | [meta-ti,master,v2] j784s4: move TI_CORE_INITRAMFS_KERNEL_MODULES to j784s4-evm.conf | expand |
meta-ti / na / 20260216230422.635024-2-ivitro
PRC Results: PASS
=========================================================
check-yocto-patches: PASS
=========================================================
Patches
----------------------------------------
All patches passed
=========================================================
apply-yocto-patch: PASS
=========================================================
master
=====================
Summary:
- Patch Series: [meta-ti][master][PATCH v2] j784s4: move TI_CORE_INITRAMFS_KERNEL_MODULES to j784s4-evm.conf
- Submitter: From: Vitor Soares <ivitro@gmail.com>
From: Vitor Soares <vitor.soares@toradex.com>
- Date: Date: Mon, 16 Feb 2026 23:04:20 +0000
- Num Patches: 1
- Mailing List (public inbox) Commit SHA: f82153d1c2ea00de3fe0137a0fb077adf1538de2
Applied to:
- Repository: lcpd-prc-meta-ti
- Base Branch: master-wip
- Commit Author: Ryan Eatmon <reatmon@ti.com>
- Commit Subject: conf: Add falcon support natively
- Commit SHA: e9a0af0f3815c30ebcca6d098cfb935cffbe2311
Patches
----------------------------------------
All patches applied
=========================================================
check-yocto-repo: PASS
=========================================================
master
=====================
PASS
=========================================================
yocto-check-layers: PASS
=========================================================
master - PASS
=====================
All checks passed
On Mon, 2026-02-16 at 23:04 +0000, Vitor Soares wrote: > From: Vitor Soares <vitor.soares@toradex.com> > > TI_CORE_INITRAMFS_KERNEL_MODULES is set in j784s4.inc, causing all > machines based on the J784S4 SoC to unconditionally build and deploy > an initramfs. The cdns-pltfrm and ti-j721e-ufs modules are required > by the EVM to access UFS storage and are not a SoC-level requirement. > > Moving it to j784s4-evm.conf scopes initramfs creation to the board > that needs it, allowing other integrators to rely on the empty default > without explicit overrides. > > Signed-off-by: Vitor Soares <vitor.soares@toradex.com> > --- > Changes in v2: > - Added project + branch specifier to subject line > > Tested on Toradex Aquila AM69 with bsp-mainline. EVM not tested. > > meta-ti-bsp/conf/machine/include/j784s4.inc | 3 --- > meta-ti-bsp/conf/machine/j784s4-evm.conf | 4 ++++ > 2 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/meta-ti-bsp/conf/machine/include/j784s4.inc b/meta-ti- > bsp/conf/machine/include/j784s4.inc > index 4dc3a71b4444..f29cd69bf9f5 100644 > --- a/meta-ti-bsp/conf/machine/include/j784s4.inc > +++ b/meta-ti-bsp/conf/machine/include/j784s4.inc > @@ -14,6 +14,3 @@ OPTEEMACHINE = "k3-j784s4" > > MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS += "cadence-mhdp-fw cnm-wave-fw ti-eth- > fw-j784s4" > > -TI_CORE_INITRAMFS_KERNEL_MODULES = "kernel-module-cdns-pltfrm kernel-module- > ti-j721e-ufs" > -TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_6 = "" > -TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_12 = "" > diff --git a/meta-ti-bsp/conf/machine/j784s4-evm.conf b/meta-ti- > bsp/conf/machine/j784s4-evm.conf > index bf53b07c33f4..333bd24d938a 100644 > --- a/meta-ti-bsp/conf/machine/j784s4-evm.conf > +++ b/meta-ti-bsp/conf/machine/j784s4-evm.conf > @@ -27,3 +27,7 @@ KERNEL_DEVICETREE = " \ > " > > UBOOT_MACHINE = "j784s4_evm_a72_defconfig" > + > +TI_CORE_INITRAMFS_KERNEL_MODULES = "kernel-module-cdns-pltfrm kernel-module- > ti-j721e-ufs" > +TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_6 = "" > +TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_12 = "" Hello, The recent commit 135dfe3f2620 ("ti-bsp: Make ti-6_18 the default BSP") making the ti-6_18 the default BSP appears to have broken our builds, which is what this patch was trying to prevent. With ti-6_18 now default, all J784s4 based machines are forced into initramfs build through TI_CORE_INITRAMFS_KERNEL_MODULES in j784s4.inc, regardless of whether they actually need these EVM specific modules. Our Toradex Aquila AM69/TDA4 SoM doesn't use UFS storage and doens't need initrafs. Our build now fail because the SoC include forces image levl integration decisions. Can this patch be considered, or am I missing something about how to handle this scenario? Thanks, Vitor Soares
On 3/3/2026 1:20 PM, Vitor Soares wrote: > On Mon, 2026-02-16 at 23:04 +0000, Vitor Soares wrote: >> From: Vitor Soares <vitor.soares@toradex.com> >> >> TI_CORE_INITRAMFS_KERNEL_MODULES is set in j784s4.inc, causing all >> machines based on the J784S4 SoC to unconditionally build and deploy >> an initramfs. The cdns-pltfrm and ti-j721e-ufs modules are required >> by the EVM to access UFS storage and are not a SoC-level requirement. >> >> Moving it to j784s4-evm.conf scopes initramfs creation to the board >> that needs it, allowing other integrators to rely on the empty default >> without explicit overrides. >> >> Signed-off-by: Vitor Soares <vitor.soares@toradex.com> >> --- >> Changes in v2: >> - Added project + branch specifier to subject line >> >> Tested on Toradex Aquila AM69 with bsp-mainline. EVM not tested. >> >> meta-ti-bsp/conf/machine/include/j784s4.inc | 3 --- >> meta-ti-bsp/conf/machine/j784s4-evm.conf | 4 ++++ >> 2 files changed, 4 insertions(+), 3 deletions(-) >> >> diff --git a/meta-ti-bsp/conf/machine/include/j784s4.inc b/meta-ti- >> bsp/conf/machine/include/j784s4.inc >> index 4dc3a71b4444..f29cd69bf9f5 100644 >> --- a/meta-ti-bsp/conf/machine/include/j784s4.inc >> +++ b/meta-ti-bsp/conf/machine/include/j784s4.inc >> @@ -14,6 +14,3 @@ OPTEEMACHINE = "k3-j784s4" >> >> MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS += "cadence-mhdp-fw cnm-wave-fw ti-eth- >> fw-j784s4" >> >> -TI_CORE_INITRAMFS_KERNEL_MODULES = "kernel-module-cdns-pltfrm kernel-module- >> ti-j721e-ufs" >> -TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_6 = "" >> -TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_12 = "" >> diff --git a/meta-ti-bsp/conf/machine/j784s4-evm.conf b/meta-ti- >> bsp/conf/machine/j784s4-evm.conf >> index bf53b07c33f4..333bd24d938a 100644 >> --- a/meta-ti-bsp/conf/machine/j784s4-evm.conf >> +++ b/meta-ti-bsp/conf/machine/j784s4-evm.conf >> @@ -27,3 +27,7 @@ KERNEL_DEVICETREE = " \ >> " >> >> UBOOT_MACHINE = "j784s4_evm_a72_defconfig" >> + >> +TI_CORE_INITRAMFS_KERNEL_MODULES = "kernel-module-cdns-pltfrm kernel-module- >> ti-j721e-ufs" >> +TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_6 = "" >> +TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_12 = "" > > Hello, > > The recent commit 135dfe3f2620 ("ti-bsp: Make ti-6_18 the default BSP") making > the ti-6_18 the default BSP appears to have broken our builds, which is what > this patch was trying to prevent. > > With ti-6_18 now default, all J784s4 based machines are forced into initramfs > build through TI_CORE_INITRAMFS_KERNEL_MODULES in j784s4.inc, regardless of > whether they actually need these EVM specific modules. > > Our Toradex Aquila AM69/TDA4 SoM doesn't use UFS storage and doens't need > initrafs. Our build now fail because the SoC include forces image levl > integration decisions. > > Can this patch be considered, or am I missing something about how to handle this > scenario? We are still looking into your patch (lots of moving parts over here). In the meantime, you can always just clear out the value of TI_CORE_INITRAMFS_KERNEL_MODULES in your machine file: TI_CORE_INITRAMFS_KERNEL_MODULES = "" That will turn off the initramfs code. > Thanks, > Vitor Soares
diff --git a/meta-ti-bsp/conf/machine/include/j784s4.inc b/meta-ti-bsp/conf/machine/include/j784s4.inc index 4dc3a71b4444..f29cd69bf9f5 100644 --- a/meta-ti-bsp/conf/machine/include/j784s4.inc +++ b/meta-ti-bsp/conf/machine/include/j784s4.inc @@ -14,6 +14,3 @@ OPTEEMACHINE = "k3-j784s4" MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS += "cadence-mhdp-fw cnm-wave-fw ti-eth-fw-j784s4" -TI_CORE_INITRAMFS_KERNEL_MODULES = "kernel-module-cdns-pltfrm kernel-module-ti-j721e-ufs" -TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_6 = "" -TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_12 = "" diff --git a/meta-ti-bsp/conf/machine/j784s4-evm.conf b/meta-ti-bsp/conf/machine/j784s4-evm.conf index bf53b07c33f4..333bd24d938a 100644 --- a/meta-ti-bsp/conf/machine/j784s4-evm.conf +++ b/meta-ti-bsp/conf/machine/j784s4-evm.conf @@ -27,3 +27,7 @@ KERNEL_DEVICETREE = " \ " UBOOT_MACHINE = "j784s4_evm_a72_defconfig" + +TI_CORE_INITRAMFS_KERNEL_MODULES = "kernel-module-cdns-pltfrm kernel-module-ti-j721e-ufs" +TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_6 = "" +TI_CORE_INITRAMFS_KERNEL_MODULES:bsp-ti-6_12 = ""