Re: [PATCH v2 0/5] ti: fix dtbs_check for some syscon bindings and DTS node

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



+ Andrew and Bryan

Hi Krzysztof

On 26/06/24 16:53, Krzysztof Kozlowski wrote:
> Hi,
> 
> Changes in v2:
> - Add acks/tags
> - Correct Kishon Vijay Abraham email
> - Drop R/W from title of bindings
> - Link to v1: https://lore.kernel.org/r/20240518-dt-bindings-ti-soc-mfd-v1-0-b3952f104c9a@xxxxxxxxxx
> 
> I am in the process of fixing syscon/simple-mfd bindings (to be posted
> separately) and found set of issues in TI looking independent of my
> series.  Please apply via TI SoC.
> 
> Best regards,
> Krzysztof
> 
> ---
> Krzysztof Kozlowski (5):
>       dt-bindings: soc: ti: am62-system-controller: add AM62 syscon
>       dt-bindings: soc: ti: am645-system-controller: add AM654 syscon
>       arm64: dts: ti: k3-am62: add dedicated wakeup controller compatible
>       arm64: dts: ti: k3-am65-mcu: add dedicated wakeup controller compatible
>       arm64: dts: ti: k3-am62a: use a specific MCU controller compatible


Thanks for the patches!

Andrew's patches [0] remove "syscon", "simple-mfd" from all the below
files except for k3-am62-wakeup.dtsi. But I think k3-am62-wakeup.dtsi
can be fixed up in the similar way and not really need a custom
system-controller binding. Last user is "ti-cpufreq" driver which is
being adapted over to use granular syscon nodes [1].

I believe add new compatible for system-controllers nodes is unneeded?

AM62 may need one more merge window to be resolved owing to dependencies
but I am queuing [0] to get rid of the warning for rest of the dts.

Bryan,

Could you repsin [1] with cleanups to am62-wakeup.dtsi as well?

[0] lore.kernel.org/r/20240628151518.40100-1-afd@xxxxxx
[1] lore.kernel.org/r/20240621-ti-opp-updates-v3-0-d857be6dac8b@xxxxxx


> 
>  .../bindings/soc/ti/ti,am62-system-controller.yaml | 77 ++++++++++++++++++++++
>  .../soc/ti/ti,am654-system-controller.yaml         | 60 +++++++++++++++++
>  arch/arm64/boot/dts/ti/k3-am62-wakeup.dtsi         |  2 +-
>  arch/arm64/boot/dts/ti/k3-am62a-wakeup.dtsi        |  2 +-
>  arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi            |  2 +-
>  5 files changed, 140 insertions(+), 3 deletions(-)
> ---
> base-commit: e032bb82c315d2317a80506195d16ce4308d8cf7
> change-id: 20240518-dt-bindings-ti-soc-mfd-ac211578d7a5
> 
> Best regards,

-- 
Regards
Vignesh




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux