| Message ID | 20251201100941.95152-1-m-shah@ti.com |
|---|---|
| State | Superseded |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | [master] meta-ti-bsp: include: am62axx.inc: Add TPS6594 regulator and core modules to initramfs image | expand |
meta-ti / na / 20251201100941.95152-1-m-shah
PRC Results: FAIL
=========================================================
check-yocto-patches: FAIL
=========================================================
Patches
----------------------------------------
FAIL - [master][PATCH] meta-ti-bsp: include: am62axx.inc: Add TPS6594 regulator and core modules to initramfs image
ERROR: Commit message too long (> 80 chars): meta-ti-bsp: include: am62axx.inc: Add TPS6594 regulator and core modules to initramfs image. (COMMIT-MESSAGE-1)
patch:68
Subject: [master][PATCH] meta-ti-bsp: include: am62axx.inc: Add TPS6594 regulator and core modules to initramfs image
WARN: Commit message should not include directory path to recipe: meta-ti-bsp: include: am62axx.inc: Add TPS6594 regulator and core modules to initramfs image.
meta-ti-bsp
include (COMMIT-MESSAGE-3)
patch
For details on the above errors/warnings visit: https://lists.yoctoproject.org/g/meta-ti/wiki/40887
Hi, On 01/12/25 15:39, Moteen Shah via lists.yoctoproject.org wrote: > Add TPS6594 core and regulator driver modules in the initramfs image so > that the SD card can be probed for am62axx as it is dependent on regulator > to get the SD card probing and eventually boot from it. > > Signed-off-by: Moteen Shah <m-shah@ti.com> > --- > Bootlogs with this patch applied: http://serenity.dal.design.ti.com:7777/ecagupigod.yaml > Bootlogs without this patch applied: http://serenity.dal.design.ti.com:7777/kiweziviha.yaml meta-ti is a public mailing list [0] Please avoid internal serenity links. Use gist.github [1] for attaching logs Chirag -- [0] - https://lists.yoctoproject.org/g/meta-ti [1] - https://gist.github.com/ > > meta-ti-bsp/conf/machine/include/am62axx.inc | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/meta-ti-bsp/conf/machine/include/am62axx.inc b/meta-ti-bsp/conf/machine/include/am62axx.inc > index b66a2227..08111f82 100644 > --- a/meta-ti-bsp/conf/machine/include/am62axx.inc > +++ b/meta-ti-bsp/conf/machine/include/am62axx.inc > @@ -12,5 +12,5 @@ OPTEEMACHINE = "k3-am62x" > > MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS += "cnm-wave-fw" > > -TI_INITRAMFS_KERNEL_MODULES:append:bsp-mainline = " kernel-module-tps6594-i2c" > -TI_INITRAMFS_KERNEL_MODULES:append:bsp-next = " kernel-module-tps6594-i2c" > +TI_INITRAMFS_KERNEL_MODULES:append:bsp-mainline = " kernel-module-tps6594-i2c kernel-module-tps6594-regulator kernel-module-tps6594-core" > +TI_INITRAMFS_KERNEL_MODULES:append:bsp-next = " kernel-module-tps6594-i2c kernel-module-tps6594-regulator kernel-module-tps6594-core" > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#19308): https://lists.yoctoproject.org/g/meta-ti/message/19308 > Mute This Topic: https://lists.yoctoproject.org/mt/116553566/7030289 > Group Owner: meta-ti+owner@lists.yoctoproject.org > Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [c-shilwant@ti.com] > -=-=-=-=-=-=-=-=-=-=-=- >
On 01/12/25 3:39 pm, Moteen Shah wrote: > Add TPS6594 core and regulator driver modules in the initramfs image so > that the SD card can be probed for am62axx as it is dependent on regulator > to get the SD card probing and eventually boot from it. > > Signed-off-by: Moteen Shah <m-shah@ti.com> The subject line can be just "conf: machine: am62axx: ..." > --- > Bootlogs with this patch applied: http://serenity.dal.design.ti.com:7777/ecagupigod.yaml > Bootlogs without this patch applied: http://serenity.dal.design.ti.com:7777/kiweziviha.yaml > > meta-ti-bsp/conf/machine/include/am62axx.inc | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/meta-ti-bsp/conf/machine/include/am62axx.inc b/meta-ti-bsp/conf/machine/include/am62axx.inc > index b66a2227..08111f82 100644 > --- a/meta-ti-bsp/conf/machine/include/am62axx.inc > +++ b/meta-ti-bsp/conf/machine/include/am62axx.inc > @@ -12,5 +12,5 @@ OPTEEMACHINE = "k3-am62x" > > MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS += "cnm-wave-fw" > > -TI_INITRAMFS_KERNEL_MODULES:append:bsp-mainline = " kernel-module-tps6594-i2c" > -TI_INITRAMFS_KERNEL_MODULES:append:bsp-next = " kernel-module-tps6594-i2c" > +TI_INITRAMFS_KERNEL_MODULES:append:bsp-mainline = " kernel-module-tps6594-i2c kernel-module-tps6594-regulator kernel-module-tps6594-core" > +TI_INITRAMFS_KERNEL_MODULES:append:bsp-next = " kernel-module-tps6594-i2c kernel-module-tps6594-regulator kernel-module-tps6594-core"
Hello Chirag, On 01/12/25 15:46, Chirag Shilwant wrote: > Hi, > On 01/12/25 15:39, Moteen Shah via lists.yoctoproject.org wrote: >> Add TPS6594 core and regulator driver modules in the initramfs image so >> that the SD card can be probed for am62axx as it is dependent on >> regulator >> to get the SD card probing and eventually boot from it. >> >> Signed-off-by: Moteen Shah <m-shah@ti.com> >> --- >> Bootlogs with this patch applied: >> http://serenity.dal.design.ti.com:7777/ecagupigod.yaml >> Bootlogs without this patch applied: >> http://serenity.dal.design.ti.com:7777/kiweziviha.yaml > > > meta-ti is a public mailing list [0] > Please avoid internal serenity links. > > Use gist.github [1] for attaching logs > > Chirag > -- > > [0] - https://lists.yoctoproject.org/g/meta-ti > > [1] - https://gist.github.com/ My bad, missed using the public link, will send a respin after some initial reviews, thanks for pointing out. Regards, Moteen > > >> >> meta-ti-bsp/conf/machine/include/am62axx.inc | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/meta-ti-bsp/conf/machine/include/am62axx.inc >> b/meta-ti-bsp/conf/machine/include/am62axx.inc >> index b66a2227..08111f82 100644 >> --- a/meta-ti-bsp/conf/machine/include/am62axx.inc >> +++ b/meta-ti-bsp/conf/machine/include/am62axx.inc >> @@ -12,5 +12,5 @@ OPTEEMACHINE = "k3-am62x" >> MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS += "cnm-wave-fw" >> -TI_INITRAMFS_KERNEL_MODULES:append:bsp-mainline = " >> kernel-module-tps6594-i2c" >> -TI_INITRAMFS_KERNEL_MODULES:append:bsp-next = " >> kernel-module-tps6594-i2c" >> +TI_INITRAMFS_KERNEL_MODULES:append:bsp-mainline = " >> kernel-module-tps6594-i2c kernel-module-tps6594-regulator >> kernel-module-tps6594-core" >> +TI_INITRAMFS_KERNEL_MODULES:append:bsp-next = " >> kernel-module-tps6594-i2c kernel-module-tps6594-regulator >> kernel-module-tps6594-core" >> >> -=-=-=-=-=-=-=-=-=-=-=- >> Links: You receive all messages sent to this group. >> View/Reply Online (#19308): >> https://lists.yoctoproject.org/g/meta-ti/message/19308 >> Mute This Topic: https://lists.yoctoproject.org/mt/116553566/7030289 >> Group Owner: meta-ti+owner@lists.yoctoproject.org >> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub >> [c-shilwant@ti.com] >> -=-=-=-=-=-=-=-=-=-=-=- >>
diff --git a/meta-ti-bsp/conf/machine/include/am62axx.inc b/meta-ti-bsp/conf/machine/include/am62axx.inc index b66a2227..08111f82 100644 --- a/meta-ti-bsp/conf/machine/include/am62axx.inc +++ b/meta-ti-bsp/conf/machine/include/am62axx.inc @@ -12,5 +12,5 @@ OPTEEMACHINE = "k3-am62x" MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS += "cnm-wave-fw" -TI_INITRAMFS_KERNEL_MODULES:append:bsp-mainline = " kernel-module-tps6594-i2c" -TI_INITRAMFS_KERNEL_MODULES:append:bsp-next = " kernel-module-tps6594-i2c" +TI_INITRAMFS_KERNEL_MODULES:append:bsp-mainline = " kernel-module-tps6594-i2c kernel-module-tps6594-regulator kernel-module-tps6594-core" +TI_INITRAMFS_KERNEL_MODULES:append:bsp-next = " kernel-module-tps6594-i2c kernel-module-tps6594-regulator kernel-module-tps6594-core"
Add TPS6594 core and regulator driver modules in the initramfs image so that the SD card can be probed for am62axx as it is dependent on regulator to get the SD card probing and eventually boot from it. Signed-off-by: Moteen Shah <m-shah@ti.com> --- Bootlogs with this patch applied: http://serenity.dal.design.ti.com:7777/ecagupigod.yaml Bootlogs without this patch applied: http://serenity.dal.design.ti.com:7777/kiweziviha.yaml meta-ti-bsp/conf/machine/include/am62axx.inc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)