diff mbox series

[meta-ti,master,v2] j784s4: move TI_CORE_INITRAMFS_KERNEL_MODULES to j784s4-evm.conf

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

Commit Message

Vitor Soares Feb. 16, 2026, 11:04 p.m. UTC
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(-)

Comments

PRC Automation Feb. 16, 2026, 11:12 p.m. UTC | #1
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
Vitor Soares March 3, 2026, 7:20 p.m. UTC | #2
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
Ryan Eatmon March 3, 2026, 7:26 p.m. UTC | #3
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 mbox series

Patch

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 = ""