diff mbox series

[meta-ti,master] ti-kernel-fitimage: Add ti-secdev

Message ID 20250715175445.28647-1-reatmon@ti.com
State Superseded
Delegated to: Ryan Eatmon
Headers show
Series [meta-ti,master] ti-kernel-fitimage: Add ti-secdev | expand

Commit Message

Ryan Eatmon July 15, 2025, 5:54 p.m. UTC
In order to turn the fitImage signing back on, we need to inherit the
ti-secdev class to setup all of the needed signing variables.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb | 2 ++
 1 file changed, 2 insertions(+)

Comments

Denys Dmytriyenko July 15, 2025, 6:11 p.m. UTC | #1
On Tue, Jul 15, 2025 at 12:54:45PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> In order to turn the fitImage signing back on, we need to inherit the
> ti-secdev class to setup all of the needed signing variables.

+ Andrew

I don't believe we need ti-secdev in the main kernel recipe any more now that 
fitImage got split out into its own recipe...

So this patch should rather be "move" ti-secdev instead of "add"?


> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb b/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb
> index dae76387..a5e6a280 100644
> --- a/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb
> +++ b/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb
> @@ -3,6 +3,8 @@ FIT_CONF_PREFIX ?= "conf-ti_"
>  
>  require recipes-kernel/linux/linux-yocto-fitimage.bb
>  
> +inherit ti-secdev
> +
>  include ${@ 'recipes-kernel/linux/ti-kernel-devicetree-prefix.inc' if d.getVar('KERNEL_DEVICETREE_PREFIX') else ''}
>  
>  do_compile[depends] += "virtual/kernel:do_shared_workdir"
> -- 
> 2.17.1
Ryan Eatmon July 16, 2025, 2:42 p.m. UTC | #2
On 7/15/2025 1:11 PM, Denys Dmytriyenko wrote:
> On Tue, Jul 15, 2025 at 12:54:45PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>> In order to turn the fitImage signing back on, we need to inherit the
>> ti-secdev class to setup all of the needed signing variables.
> 
> + Andrew
> 
> I don't believe we need ti-secdev in the main kernel recipe any more now that
> fitImage got split out into its own recipe...
> 
> So this patch should rather be "move" ti-secdev instead of "add"?

Good point.  I'll send a v2.


> 
>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>> ---
>>   meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb b/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb
>> index dae76387..a5e6a280 100644
>> --- a/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb
>> +++ b/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb
>> @@ -3,6 +3,8 @@ FIT_CONF_PREFIX ?= "conf-ti_"
>>   
>>   require recipes-kernel/linux/linux-yocto-fitimage.bb
>>   
>> +inherit ti-secdev
>> +
>>   include ${@ 'recipes-kernel/linux/ti-kernel-devicetree-prefix.inc' if d.getVar('KERNEL_DEVICETREE_PREFIX') else ''}
>>   
>>   do_compile[depends] += "virtual/kernel:do_shared_workdir"
>> -- 
>> 2.17.1
diff mbox series

Patch

diff --git a/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb b/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb
index dae76387..a5e6a280 100644
--- a/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb
+++ b/meta-ti-bsp/recipes-kernel/linux/ti-kernel-fitimage.bb
@@ -3,6 +3,8 @@  FIT_CONF_PREFIX ?= "conf-ti_"
 
 require recipes-kernel/linux/linux-yocto-fitimage.bb
 
+inherit ti-secdev
+
 include ${@ 'recipes-kernel/linux/ti-kernel-devicetree-prefix.inc' if d.getVar('KERNEL_DEVICETREE_PREFIX') else ''}
 
 do_compile[depends] += "virtual/kernel:do_shared_workdir"