[meta-arago,master/dunfell] conf: machine: j721s2: add new dtb for GESI

Message ID 20220701161237.10497-1-reatmon@ti.com
State Rejected
Delegated to: Ryan Eatmon
Headers show
Series [meta-arago,master/dunfell] conf: machine: j721s2: add new dtb for GESI | expand

Commit Message

Ryan Eatmon July 1, 2022, 4:12 p.m. UTC
The following commits in ti-linux-5.10.y kernel has introduced a new dtb.

commit: 9b05b2a89ff ("arm64: dts: ti: Add overlay for GESI to enable main CPSW2G in J721S2")

This patch on j721s2 conf is to include the new dtb.

Signed-off-by: Ryan Eatmon <reatmon@ti.com>
---
 conf/machine/j721s2-evm.conf | 1 +
 1 file changed, 1 insertion(+)

Comments

Denys Dmytriyenko July 1, 2022, 4:30 p.m. UTC | #1
1. wrong list - should go to meta-ti
2. kernel commit says "overlay" - should this be .dtbo, not .dtb?


On Fri, Jul 01, 2022 at 11:12:37AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> The following commits in ti-linux-5.10.y kernel has introduced a new dtb.
> 
> commit: 9b05b2a89ff ("arm64: dts: ti: Add overlay for GESI to enable main CPSW2G in J721S2")
> 
> This patch on j721s2 conf is to include the new dtb.
> 
> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  conf/machine/j721s2-evm.conf | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/conf/machine/j721s2-evm.conf b/conf/machine/j721s2-evm.conf
> index e5ed0b73..b224ce7b 100644
> --- a/conf/machine/j721s2-evm.conf
> +++ b/conf/machine/j721s2-evm.conf
> @@ -13,6 +13,7 @@ OPTEE_K3_USART = "0x8"
>  
>  KERNEL_DEVICETREE = " \
>      ti/k3-j721s2-common-proc-board.dtb \
> +    ti/k3-j721s2-gesi-exp-board.dtb \
>  "
>  
>  UBOOT_MACHINE = "j721s2_evm_a72_defconfig"
> -- 
> 2.17.1
Ryan Eatmon July 1, 2022, 5:34 p.m. UTC | #2
You are correct on both counts.  I will fix.


On 7/1/2022 11:30, Denys Dmytriyenko wrote:
> 1. wrong list - should go to meta-ti
> 2. kernel commit says "overlay" - should this be .dtbo, not .dtb?
> 
> 
> On Fri, Jul 01, 2022 at 11:12:37AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>> The following commits in ti-linux-5.10.y kernel has introduced a new dtb.
>>
>> commit: 9b05b2a89ff ("arm64: dts: ti: Add overlay for GESI to enable main CPSW2G in J721S2")
>>
>> This patch on j721s2 conf is to include the new dtb.
>>
>> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
>> ---
>>   conf/machine/j721s2-evm.conf | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/conf/machine/j721s2-evm.conf b/conf/machine/j721s2-evm.conf
>> index e5ed0b73..b224ce7b 100644
>> --- a/conf/machine/j721s2-evm.conf
>> +++ b/conf/machine/j721s2-evm.conf
>> @@ -13,6 +13,7 @@ OPTEE_K3_USART = "0x8"
>>   
>>   KERNEL_DEVICETREE = " \
>>       ti/k3-j721s2-common-proc-board.dtb \
>> +    ti/k3-j721s2-gesi-exp-board.dtb \
>>   "
>>   
>>   UBOOT_MACHINE = "j721s2_evm_a72_defconfig"
>> -- 
>> 2.17.1

Patch

diff --git a/conf/machine/j721s2-evm.conf b/conf/machine/j721s2-evm.conf
index e5ed0b73..b224ce7b 100644
--- a/conf/machine/j721s2-evm.conf
+++ b/conf/machine/j721s2-evm.conf
@@ -13,6 +13,7 @@  OPTEE_K3_USART = "0x8"
 
 KERNEL_DEVICETREE = " \
     ti/k3-j721s2-common-proc-board.dtb \
+    ti/k3-j721s2-gesi-exp-board.dtb \
 "
 
 UBOOT_MACHINE = "j721s2_evm_a72_defconfig"