[PATCH 2/2] arm: dts: k3: Remove unneeded ti, sci-sysreset binding and nodes

[Andrew Davis]

On 4/3/24 1:37 AM, Neha Malcom Francis wrote:
> Hi Andrew,
> 
> On 02/04/24 21:39, Andrew Davis wrote:
>> This extra binding is non-standard and now unneeded as we bind the
>> sysreset driver automatically. This matches what is done in Linux
>> and allows us to more closely match the DTBs. Remove the binding
>> and all users.
>>
>> Signed-off-by: Andrew Davis <afd at ti.com>
>> ---
>>   arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi  |  7 -----
>>   .../k3-am625-phyboard-lyra-rdk-u-boot.dtsi    |  7 -----
>>   .../dts/k3-am625-verdin-wifi-dev-u-boot.dtsi  |  7 -----
>>   arch/arm/dts/k3-am62a7-sk-u-boot.dtsi         |  4 ---
>>   arch/arm/dts/k3-am62p5-sk-u-boot.dtsi         |  5 ----
>>   arch/arm/dts/k3-am642-evm-u-boot.dtsi         |  7 -----
>>   .../k3-am642-phyboard-electra-rdk-u-boot.dtsi |  4 ---
>>   arch/arm/dts/k3-am642-sk-u-boot.dtsi          |  7 -----
>>   .../dts/k3-am65-iot2050-common-u-boot.dtsi    |  4 ---
>>   .../arm/dts/k3-am68-sk-base-board-u-boot.dtsi |  4 ---
>>   arch/arm/dts/k3-am69-sk-u-boot.dtsi           |  7 -----
>>   .../k3-j7200-common-proc-board-u-boot.dtsi    |  4 ---
>>   .../dts/k3-j721e-beagleboneai64-u-boot.dtsi   |  4 ---
>>   .../k3-j721e-common-proc-board-u-boot.dtsi    |  4 ---
>>   arch/arm/dts/k3-j721e-sk-u-boot.dtsi          |  4 ---
>>   .../k3-j721s2-common-proc-board-u-boot.dtsi   |  4 ---
>>   arch/arm/dts/k3-j784s4-evm-u-boot.dtsi        |  7 -----
>>   .../sysreset/ti,sci-sysreset.txt              | 29 -------------------
>>   18 files changed, 119 deletions(-)
>>   delete mode 100644 doc/device-tree-bindings/sysreset/ti,sci-sysreset.txt
>>
> 
> [...]
> 
> Reviewed-by: Neha Malcom Francis <n-francis at ti.com>
> 
> Why not have Kconfig set this as default y if TI_SCI_PROTOCOL set using same argument?
> 

Good idea, and the same could probably be said for most TI-SCI based drivers.

We might also look into adding more of these as "imply" under config ARCH_K3,
all K3 boards will want/need these. Should help simplify our defconfig.

Andrew