mbox series

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

Message ID 20240626-dt-bindings-ti-soc-mfd-v2-0-ee4d69017326@linaro.org (mailing list archive)
Headers show
Series ti: fix dtbs_check for some syscon bindings and DTS node | expand

Message

Krzysztof Kozlowski June 26, 2024, 11:23 a.m. UTC
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@linaro.org

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

 .../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,

Comments

Vignesh Raghavendra June 30, 2024, 6:59 a.m. UTC | #1
+ 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@linaro.org
> 
> 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@ti.com
[1] lore.kernel.org/r/20240621-ti-opp-updates-v3-0-d857be6dac8b@ti.com


> 
>  .../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,