diff mbox series

[meta-ti,master/scarthgap] meta-ti-bsp: am62lxx: Do not deploy extra tiboot3.bin copy

Message ID 20250522210712.2106045-1-afd@ti.com
State New
Headers show
Series [meta-ti,master/scarthgap] meta-ti-bsp: am62lxx: Do not deploy extra tiboot3.bin copy | expand

Commit Message

Andrew Davis May 22, 2025, 9:07 p.m. UTC
For AM62L there is no GP and both HS sub-types are supported by the
same TIFS. This means we do not need multiple versions of tiboot3.bin.
So no need to deploy an additional version which is identical to the
default tiboot3.bin already deployed.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 meta-ti-bsp/conf/machine/include/am62lxx.inc | 3 ---
 1 file changed, 3 deletions(-)

Comments

Chirag Shilwant May 23, 2025, 9 a.m. UTC | #1
On 23/05/25 02:37, Andrew Davis via lists.yoctoproject.org wrote:
> For AM62L there is no GP and both HS sub-types are supported by the
> same TIFS. This means we do not need multiple versions of tiboot3.bin.
> So no need to deploy an additional version which is identical to the
> default tiboot3.bin already deployed.
>
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>   meta-ti-bsp/conf/machine/include/am62lxx.inc | 3 ---
>   1 file changed, 3 deletions(-)
>
> diff --git a/meta-ti-bsp/conf/machine/include/am62lxx.inc b/meta-ti-bsp/conf/machine/include/am62lxx.inc
> index d3bc81df..b4f0b8c9 100644
> --- a/meta-ti-bsp/conf/machine/include/am62lxx.inc
> +++ b/meta-ti-bsp/conf/machine/include/am62lxx.inc
> @@ -3,9 +3,6 @@ SOC_FAMILY:append = ":am62lxx"
>   
>   MACHINE_FEATURES += "screen"
>   
> -# Default tiboot3.bin on AM62L is for HS-FS
> -IMAGE_BOOT_FILES += "tiboot3-am62lx-hs-fs-evm.bin"


I still see the current ti-u-boot-2025.01-next shipping the 
|tiboot3-am62lx-hs-fs-evm.bin|[0]
Are we planning to drop that entry from 
arch/arm/dts/k3-am62l3-evm-binman.dtsi as well ?


[0]: 
https://git.ti.com/cgit/ti-u-boot/ti-u-boot/tree/arch/arm/dts/k3-am62l3-evm-binman.dtsi?h=ti-u-boot-2025.01-next&id=3d024809613c2a8f1d69258554ab8954d26537f5#n54


--
Chirag


> -
>   TFA_BOARD = "am62l"
>   TFA_K3_SYSTEM_SUSPEND = "1"
>   
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#18588): https://lists.yoctoproject.org/g/meta-ti/message/18588
> Mute This Topic: https://lists.yoctoproject.org/mt/113254992/7030289
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [c-shilwant@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Bajjuri, Praneeth May 23, 2025, 2:12 p.m. UTC | #2
On 5/23/2025 4:00 AM, Chirag Shilwant via lists.yoctoproject.org wrote:
> On 23/05/25 02:37, Andrew Davis via lists.yoctoproject.org wrote:
>> For AM62L there is no GP and both HS sub-types are supported by the
>> same TIFS. This means we do not need multiple versions of tiboot3.bin.
>> So no need to deploy an additional version which is identical to the
>> default tiboot3.bin already deployed.
>>
>> Signed-off-by: Andrew Davis <afd@ti.com>
>> ---
>>   meta-ti-bsp/conf/machine/include/am62lxx.inc | 3 ---
>>   1 file changed, 3 deletions(-)
>>
>> diff --git a/meta-ti-bsp/conf/machine/include/am62lxx.inc 
>> b/meta-ti-bsp/conf/machine/include/am62lxx.inc
>> index d3bc81df..b4f0b8c9 100644
>> --- a/meta-ti-bsp/conf/machine/include/am62lxx.inc
>> +++ b/meta-ti-bsp/conf/machine/include/am62lxx.inc
>> @@ -3,9 +3,6 @@ SOC_FAMILY:append = ":am62lxx"
>>   MACHINE_FEATURES += "screen"
>> -# Default tiboot3.bin on AM62L is for HS-FS
>> -IMAGE_BOOT_FILES += "tiboot3-am62lx-hs-fs-evm.bin"
> 
> 
> I still see the current ti-u-boot-2025.01-next shipping the 
> |tiboot3-am62lx-hs-fs-evm.bin|[0]
> Are we planning to drop that entry from 
> arch/arm/dts/k3-am62l3-evm-binman.dtsi as well ?
> 
> 
> [0]: 
> https://git.ti.com/cgit/ti-u-boot/ti-u-boot/tree/arch/arm/dts/k3-am62l3-evm-binman.dtsi?h=ti-u-boot-2025.01-next&id=3d024809613c2a8f1d69258554ab8954d26537f5#n54

Part of
https://git.ti.com/cgit/ti-u-boot/ti-u-boot/commit/?h=ti-u-boot-2025.01-next&id=3f6b151073dcf1358cdafe5cb64a1329c60282ea 



> 
> 
> -- 
> Chirag
> 
> 
>> -
>>   TFA_BOARD = "am62l"
>>   TFA_K3_SYSTEM_SUSPEND = "1"
>>
>>
>>
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#18589): https://lists.yoctoproject.org/g/meta-ti/message/18589
> Mute This Topic: https://lists.yoctoproject.org/mt/113254992/2167429
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [praneeth@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Chirag Shilwant May 23, 2025, 2:18 p.m. UTC | #3
Hi Praneeth & Andrew,
On 23/05/25 19:42, Bajjuri, Praneeth wrote:
>
>
> On 5/23/2025 4:00 AM, Chirag Shilwant via lists.yoctoproject.org wrote:
>> On 23/05/25 02:37, Andrew Davis via lists.yoctoproject.org wrote:
>>> For AM62L there is no GP and both HS sub-types are supported by the
>>> same TIFS. This means we do not need multiple versions of tiboot3.bin.
>>> So no need to deploy an additional version which is identical to the
>>> default tiboot3.bin already deployed.
>>>
>>> Signed-off-by: Andrew Davis <afd@ti.com>


Acked-by: Chirag Shilwant <c-shilwant@ti.com>


>>>
>>> ---
>>>   meta-ti-bsp/conf/machine/include/am62lxx.inc | 3 ---
>>>   1 file changed, 3 deletions(-)
>>>
>>> diff --git a/meta-ti-bsp/conf/machine/include/am62lxx.inc 
>>> b/meta-ti-bsp/conf/machine/include/am62lxx.inc
>>> index d3bc81df..b4f0b8c9 100644
>>> --- a/meta-ti-bsp/conf/machine/include/am62lxx.inc
>>> +++ b/meta-ti-bsp/conf/machine/include/am62lxx.inc
>>> @@ -3,9 +3,6 @@ SOC_FAMILY:append = ":am62lxx"
>>>   MACHINE_FEATURES += "screen"
>>> -# Default tiboot3.bin on AM62L is for HS-FS
>>> -IMAGE_BOOT_FILES += "tiboot3-am62lx-hs-fs-evm.bin"
>>
>>
>> I still see the current ti-u-boot-2025.01-next shipping the 
>> |tiboot3-am62lx-hs-fs-evm.bin|[0]
>> Are we planning to drop that entry from 
>> arch/arm/dts/k3-am62l3-evm-binman.dtsi as well ?
>>
>>
>> [0]: 
>> https://git.ti.com/cgit/ti-u-boot/ti-u-boot/tree/arch/arm/dts/k3-am62l3-evm-binman.dtsi?h=ti-u-boot-2025.01-next&id=3d024809613c2a8f1d69258554ab8954d26537f5#n54
>
> Part of
> https://git.ti.com/cgit/ti-u-boot/ti-u-boot/commit/?h=ti-u-boot-2025.01-next&id=3f6b151073dcf1358cdafe5cb64a1329c60282ea 



Thanks for the information!
Ack from my end.


--
Chirag


>
>
>
>>
>>
>> -- 
>> Chirag
>>
>>
>>> -
>>>   TFA_BOARD = "am62l"
>>>   TFA_K3_SYSTEM_SUSPEND = "1"
>>>
>>>
>>>
>>
>>
>> -=-=-=-=-=-=-=-=-=-=-=-
>> Links: You receive all messages sent to this group.
>> View/Reply Online (#18589): 
>> https://lists.yoctoproject.org/g/meta-ti/message/18589
>> Mute This Topic: https://lists.yoctoproject.org/mt/113254992/2167429
>> Group Owner: meta-ti+owner@lists.yoctoproject.org
>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub 
>> [praneeth@ti.com]
>> -=-=-=-=-=-=-=-=-=-=-=-
>>
diff mbox series

Patch

diff --git a/meta-ti-bsp/conf/machine/include/am62lxx.inc b/meta-ti-bsp/conf/machine/include/am62lxx.inc
index d3bc81df..b4f0b8c9 100644
--- a/meta-ti-bsp/conf/machine/include/am62lxx.inc
+++ b/meta-ti-bsp/conf/machine/include/am62lxx.inc
@@ -3,9 +3,6 @@  SOC_FAMILY:append = ":am62lxx"
 
 MACHINE_FEATURES += "screen"
 
-# Default tiboot3.bin on AM62L is for HS-FS
-IMAGE_BOOT_FILES += "tiboot3-am62lx-hs-fs-evm.bin"
-
 TFA_BOARD = "am62l"
 TFA_K3_SYSTEM_SUSPEND = "1"