| Message ID | 20260728141051.2851131-1-reatmon@ti.com |
|---|---|
| State | New |
| Headers | show |
| Series | [meta-ti,master/wrynose,1/2] packagegroup-ti-core-initramfs: Move the inclusion of initramfs-module-netsetup | expand |
meta-ti / na / 20260728141051.2851131-1-reatmon
PRC Results: FAIL
=========================================================
check-yocto-patches: FAIL
=========================================================
Patches
----------------------------------------
FAIL - [meta-ti][master/wrynose][PATCH 1/2] packagegroup-ti-core-initramfs: Move the inclusion of initramfs-module-netsetup
ERROR: Commit message too long (> 80 chars): =?UTF-8?q?packagegroup-ti-co?= =?UTF-8?q?re-initramfs:=C2=A0Move=20the=20inclusion=20of=20initramfs-modu?= =?UTF-8?q?le-netsetup?=. (COMMIT-MESSAGE-1)
patch:90
Subject: [meta-ti][master/wrynose][PATCH 1/2] =?UTF-8?q?packagegroup-ti-co?= =?UTF-8?q?re-initramfs:=C2=A0Move=20the=20inclusion=20of=20initramfs-modu?= =?UTF-8?q?le-netsetup?=
WARN: Commit message does not include file/recipe name: =?UTF-8?q?packagegroup-ti-co?= =?UTF-8?q?re-initramfs:=C2=A0Move=20the=20inclusion=20of=20initramfs-modu?= =?UTF-8?q?le-netsetup?=. (COMMIT-MESSAGE-2)
patch
For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887
On Tue, Jul 28, 2026 at 09:16:08AM -0500, PRC Automation via lists.yoctoproject.org wrote: > meta-ti / na / 20260728141051.2851131-1-reatmon > > PRC Results: FAIL > > ========================================================= > check-yocto-patches: FAIL > ========================================================= > Patches > ---------------------------------------- > FAIL - [meta-ti][master/wrynose][PATCH 1/2] packagegroup-ti-core-initramfs: Move the inclusion of initramfs-module-netsetup > ERROR: Commit message too long (> 80 chars): =?UTF-8?q?packagegroup-ti-co?= =?UTF-8?q?re-initramfs:=C2=A0Move=20the=20inclusion=20of=20initramfs-modu?= =?UTF-8?q?le-netsetup?=. (COMMIT-MESSAGE-1) Moreover, there's non-ASCII character right before "Move", hence UTF-8 encoding - instead of a regular =20 whitespace, it's =C2=A0 which is a non-breaking space. > patch:90 > Subject: [meta-ti][master/wrynose][PATCH 1/2] =?UTF-8?q?packagegroup-ti-co?= =?UTF-8?q?re-initramfs:=C2=A0Move=20the=20inclusion=20of=20initramfs-modu?= =?UTF-8?q?le-netsetup?= > > WARN: Commit message does not include file/recipe name: =?UTF-8?q?packagegroup-ti-co?= =?UTF-8?q?re-initramfs:=C2=A0Move=20the=20inclusion=20of=20initramfs-modu?= =?UTF-8?q?le-netsetup?=. (COMMIT-MESSAGE-2) > patch > > For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887
On 7/28/2026 9:39 AM, Denys Dmytriyenko wrote: > On Tue, Jul 28, 2026 at 09:16:08AM -0500, PRC Automation via lists.yoctoproject.org wrote: >> meta-ti / na / 20260728141051.2851131-1-reatmon >> >> PRC Results: FAIL >> >> ========================================================= >> check-yocto-patches: FAIL >> ========================================================= >> Patches >> ---------------------------------------- >> FAIL - [meta-ti][master/wrynose][PATCH 1/2] packagegroup-ti-core-initramfs: Move the inclusion of initramfs-module-netsetup >> ERROR: Commit message too long (> 80 chars): =?UTF-8?q?packagegroup-ti-co?= =?UTF-8?q?re-initramfs:=C2=A0Move=20the=20inclusion=20of=20initramfs-modu?= =?UTF-8?q?le-netsetup?=. (COMMIT-MESSAGE-1) > > Moreover, there's non-ASCII character right before "Move", hence UTF-8 > encoding - instead of a regular =20 whitespace, it's =C2=A0 which is a > non-breaking space. Well.. that's just odd. That's what git format-patch produced... What the heck... And the actual string in git is 79 chars. So this is all a product of the email encoding. > >> patch:90 >> Subject: [meta-ti][master/wrynose][PATCH 1/2] =?UTF-8?q?packagegroup-ti-co?= =?UTF-8?q?re-initramfs:=C2=A0Move=20the=20inclusion=20of=20initramfs-modu?= =?UTF-8?q?le-netsetup?= >> >> WARN: Commit message does not include file/recipe name: =?UTF-8?q?packagegroup-ti-co?= =?UTF-8?q?re-initramfs:=C2=A0Move=20the=20inclusion=20of=20initramfs-modu?= =?UTF-8?q?le-netsetup?=. (COMMIT-MESSAGE-2) >> patch >> >> For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887
diff --git a/meta-ti-bsp/conf/machine/include/k3.inc b/meta-ti-bsp/conf/machine/include/k3.inc index 6833789c..2ebbfb9e 100644 --- a/meta-ti-bsp/conf/machine/include/k3.inc +++ b/meta-ti-bsp/conf/machine/include/k3.inc @@ -31,6 +31,9 @@ EXTRA_IMAGEDEPENDS += "virtual/bootloader" MACHINE_ESSENTIAL_EXTRA_RDEPENDS += "kernel-image-image ti-kernel-fitimage" +TI_CORE_INITRAMFS_EXTRA_INSTALL:append:bsp-mainline = " initramfs-module-netsetup" +TI_CORE_INITRAMFS_EXTRA_INSTALL:append:bsp-next = " initramfs-module-netsetup" + TFA_PLATFORM = "k3" # Use the expected value of the ubifs filesystem's volume name in the kernel diff --git a/meta-ti-bsp/recipes-ti/initramfs/packagegroup-ti-core-initramfs.bb b/meta-ti-bsp/recipes-ti/initramfs/packagegroup-ti-core-initramfs.bb index b4e6adff..a9eff847 100644 --- a/meta-ti-bsp/recipes-ti/initramfs/packagegroup-ti-core-initramfs.bb +++ b/meta-ti-bsp/recipes-ti/initramfs/packagegroup-ti-core-initramfs.bb @@ -23,5 +23,3 @@ RDEPENDS:${PN} += "\ initramfs-module-nfsrootfs \ nfs-utils-mount \ " - -RDEPENDS:${PN}:append:bsp-next = " initramfs-module-netsetup"
The list of what modules are required for the initramfs has changed as of 7.1. The CPSW driver is no longer built into the kernel and is required to be built as a module and included in the initramfs for all K3 boards. While the existing solution for including initramfs-module-netsetup was fine, it did not trigger the building of initramfs for all k3 platforms. It would just include it if the ti-core-initramfs was built. This change tells the system to build and include the needed packages for all k3 and makes sure that the ti-core-initramfs is built for all of them. Signed-off-by: Ryan Eatmon <reatmon@ti.com> --- meta-ti-bsp/conf/machine/include/k3.inc | 3 +++ .../recipes-ti/initramfs/packagegroup-ti-core-initramfs.bb | 2 -- 2 files changed, 3 insertions(+), 2 deletions(-)