Re: [meta-arago] [PATCH] recipes-core/images: Build ti-core-initramfs alongside tisdk images

[Moteen Shah <m-shah@ti.com>]

Hey Yogesh,

On 03/09/26 14:29, Yogesh Hegde wrote:
> Hi Moteen, Thanks for the patch
>
> On 18:58-20260902, Moteen Shah via lists.yoctoproject.org wrote:
>> Neither image sets TI_CORE_INITRAMFS_KERNEL_MODULES or
>> TI_CORE_INITRAMFS_EXTRA_INSTALL for j722s, so the machine-wide
>> TI_CORE_INITRAMFS_ENABLED gate in meta-ti-bsp stays off by default for
>> this machine. Force ti-core-initramfs to build via an explicit
>> do_build dependency in arago-image.inc, the common include every
>> tisdk-*-image recipe require.
>>
>> Signed-off-by: Moteen Shah <m-shah@ti.com>
> I am not sure why this patch is needed. Why does j722s need an
> empty initramfs to be built? Why does j722s need a empt initramfs?
> - If it geniunely needs to be built, this fix should belong to j722s machine config by
>    setting the appropriate variables, so it is limited to j722s only.
> - If it not to be built, then the gate is working correctly and there is
>    nothing to fix.
>
> -- Yogesh

Its not an empty initramfs actually, if you see the dependent series in 
meta-ti which I have linked it adds the required modules.

Regards,
Moteen

>
>> ---
>> This patch has a dependency on the following series:
>> https://lists.yoctoproject.org/g/meta-ti/message/20235
>>
>>   meta-arago-distro/recipes-core/images/arago-image.inc | 4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/meta-arago-distro/recipes-core/images/arago-image.inc b/meta-arago-distro/recipes-core/images/arago-image.inc
>> index 37b20850..398e299f 100644
>> --- a/meta-arago-distro/recipes-core/images/arago-image.inc
>> +++ b/meta-arago-distro/recipes-core/images/arago-image.inc
>> @@ -30,3 +30,7 @@ BAD_RECOMMENDATIONS += " \
>>   BAD_RECOMMENDATIONS += "${@oe.utils.conditional("INIT_MANAGER", "sysvinit", "", "busybox-syslog", d)}"
>>   
>>   inherit core-image remove-net-rules
>> +
>> +# Force ti-core-initramfs to build alongside every image based on this common include.
>> +
>> +do_build[depends] += "ti-core-initramfs:do_image_complete"