Re: [RFC PATCH v2 4/5] arm: dts: Introduce AM62D2-EVM dtbs

[Paresh Bhagat <p-bhagat@ti.com>]

Hi Anshul,


On 10/09/25 10:08, Anshul Dalal wrote:
>
> On Tue Sep 9, 2025 at 11:04 AM IST, Paresh Bhagat wrote:
>> Introduce the base dts files needed for u-boot or to augment the linux
>> dtbs for use in the u-boot-spl and u-boot binaries. Support added for
>> HS-SE and HS-FS boot binaries for AM62dx via binman.
>>
>> Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
>> ---
>>   arch/arm/dts/Makefile                        |    2 +
>>   arch/arm/dts/k3-am62d-ddr-1866mhz-32bit.dtsi | 2802 ++++++++++++++++++
>>   arch/arm/dts/k3-am62d-evm-binman.dtsi        |  262 ++
>>   arch/arm/dts/k3-am62d2-evm-u-boot.dtsi       |   14 +
>>   arch/arm/dts/k3-am62d2-r5-evm.dts            |   82 +
>>   5 files changed, 3162 insertions(+)
>>   create mode 100644 arch/arm/dts/k3-am62d-ddr-1866mhz-32bit.dtsi
>>   create mode 100644 arch/arm/dts/k3-am62d-evm-binman.dtsi
>>   create mode 100644 arch/arm/dts/k3-am62d2-evm-u-boot.dtsi
>>   create mode 100644 arch/arm/dts/k3-am62d2-r5-evm.dts
>>
> [snip]
>> diff --git a/arch/arm/dts/k3-am62d2-evm-u-boot.dtsi b/arch/arm/dts/k3-am62d2-evm-u-boot.dtsi
>> new file mode 100644
>> index 00000000000..d1b17e8f83f
>> --- /dev/null
>> +++ b/arch/arm/dts/k3-am62d2-evm-u-boot.dtsi
>> @@ -0,0 +1,14 @@
>> +// SPDX-License-Identifier: GPL-2.0
>> +/*
>> + * Common AM62D EVM dts file for SPLs
>> + * Copyright (C) 2025 Texas Instruments Incorporated - https://www.ti.com/
>> + */
>> +
>> +#include "k3-am62d-evm-binman.dtsi"
>> +
>> +/ {
>> +	chosen {
>> +		stdout-path = "serial2:115200n8";
>> +		tick-timer = &main_timer0;
>> +	};
>> +};
Since serial_present is enabled, it checks for stdout in the specified 
format.
Also we need to define timer property for r5, so does not make sense to 
move it to kernel dts.


> The chosen node is already part of the kernel dt for the AM62d, why not
> move this addition to the kernel dt directly?
>
> Regards,
> Anshul