[PATCH v1 0/4] J721E DTS Sync with Kernel v6.5-rc1

[Neha Malcom Francis]

Hi Nishanth

On 28/08/23 23:03, Nishanth Menon wrote:
> On 17:01-20230828, Neha Malcom Francis wrote:
>> This series aims to sync kernel.org v6.5-rc1 DTS with that of U-Boot. It
>> also includes cleanups where necessary along with certain changes to
>> ensure boot is unaffected.
>>
>> While it's mainly synced to the tag v6.5-rc1, one extra commit has been
>> added that cleans up duplicating wkup_i2c0 nodes [1] in Kernel.
>>
>> Same as with other board series that have taken up this effort, cleanup
>> of mcu_ringacc and mcu_udmap are dependent on MCU Ringacc [2] and DMA
>> [3] fixes. This will be taken up after their merge to Linux.
>>
>> [1] https://lore.kernel.org/all/20230721082344.1534094-1-u-kumar1@ti.com/
>> [2] https://lore.kernel.org/all/20230809175932.2553156-1-vigneshr@ti.com/
>> [3] https://lore.kernel.org/all/20230810174356.3322583-1-vigneshr@ti.com/
>>
>> Boot logs:
>> https://gist.github.com/nehamalcom/231426b784e0ba96ef3a0da84346e038
>>
>> Neha Malcom Francis (4):
>>    arm: dts: k3-j721e-r5: Clean up inclusion hierarchy
>>    arm: dts: k3-j721e: Sync with v6.5-rc1
>>    drivers: misc: k3_avs: Add linux compatible to maintain sync
>>    configs: j721e: Remove HBMC_AM654 config
> please change the order of patches:
> a) introduce the driver change for vtm
> b) cleanup config and drop the hbmc node stuff
> c) rest of dts sync
> 
> NOTE:
> * J721e does'nt use SYSFW - it uses TIFS
> * for properties that dont use bootph or u-boot, properties, we
>    should'nt be including property changes in u-boot.dtsi or r5.dts
>    without explanation as to what.
> * There is a bunch of /delete-property/ plugged in for no clear reason.
>    I see that even for timer - but timer data could have been included in
>    arch/arm/mach-k3/j721e/dev-data.c and so could have been others too.
>    This needs clear explanation.
> 

All valid points, will factor these in. However regarding the timer, we have 
already tried to do the same for J721E by adding in the data in dev-data.c 
however this does not work since TIFS is in sysfw.itb which has not been loaded 
yet. Will take action to debug and root cause that further as well as try adding 
data for all the other existing /delete-property/ using nodes.

>>
>>   .../k3-j721e-common-proc-board-u-boot.dtsi    | 139 ++-
>>   arch/arm/dts/k3-j721e-common-proc-board.dts   | 483 ++++++---
>>   arch/arm/dts/k3-j721e-main.dtsi               | 974 ++++++++++++++++--
>>   arch/arm/dts/k3-j721e-mcu-wakeup.dtsi         | 280 ++++-
>>   .../k3-j721e-r5-common-proc-board-u-boot.dtsi |  29 -
>>   .../arm/dts/k3-j721e-r5-common-proc-board.dts | 308 +-----
>>   arch/arm/dts/k3-j721e-r5-sk-u-boot.dtsi       |  31 -
>>   arch/arm/dts/k3-j721e-r5-sk.dts               | 536 +---------
>>   arch/arm/dts/k3-j721e-sk-u-boot.dtsi          | 158 +--
>>   arch/arm/dts/k3-j721e-sk.dts                  | 663 +++++++++---
>>   arch/arm/dts/k3-j721e-som-p0.dtsi             | 213 ++--
>>   arch/arm/dts/k3-j721e-thermal.dtsi            |  75 ++
>>   arch/arm/dts/k3-j721e.dtsi                    |  32 +-
>>   configs/j721e_evm_a72_defconfig               |   1 -
>>   configs/j721e_evm_r5_defconfig                |   1 -
>>   drivers/misc/k3_avs.c                         |   1 +
>>   16 files changed, 2428 insertions(+), 1496 deletions(-)
>>   delete mode 100644 arch/arm/dts/k3-j721e-r5-common-proc-board-u-boot.dtsi
>>   delete mode 100644 arch/arm/dts/k3-j721e-r5-sk-u-boot.dtsi
>>   create mode 100644 arch/arm/dts/k3-j721e-thermal.dtsi
>>
>> -- 
>> 2.34.1
>>
> 

-- 
Thanking You
Neha Malcom Francis