mbox series

[00/10] qcom: convert to dtschema qcom,smd and qcom,rpmcc

Message ID 20220401201035.189106-1-krzysztof.kozlowski@linaro.org (mailing list archive)
Headers show
Series qcom: convert to dtschema qcom,smd and qcom,rpmcc | expand

Message

Krzysztof Kozlowski April 1, 2022, 8:10 p.m. UTC 
Hi,

Convert to DT schema clock/qcom,rpmcc and soc/qcom,smd. The SoC and clock
schema patches are independent, although Rob's bot might complain about
compatible without schema, so maybe let's take them via one tree?

The DTS patches are independent fixes/cleanups for issues pointed out by this
schema and can be taken independently.  Not tested, although no impact on
functionality is expected.

Best regards,
Krzysztof

Krzysztof Kozlowski (10):
  arm64: dts: qcom: msm8953: do not use underscore in node name
  arm64: dts: qcom: msm8994: remove SMD qcom,local-pid property
  arm64: dts: qcom: add RPM clock controller fallback compatible
  ARM: dts: qcom: msm8974-lge-nexus5: move gpio-keys out of soc
  ARM: dts: qcom: msm8974-samsung-klte: move gpio-keys out of soc
  ARM: dts: qcom: msm8974: override nodes by label
  ARM: dts: qcom: do not use underscore in node name
  dt-bindings: soc: qcom,smd: convert to dtschema
  dt-bindings: clock: qcom,rpmcc: convert to dtschema
  dt-bindings: clock: qcom,rpmcc: add clocks property

 .../devicetree/bindings/clock/qcom,rpmcc.txt  |   63 -
 .../devicetree/bindings/clock/qcom,rpmcc.yaml |   75 ++
 .../regulator/qcom,smd-rpm-regulator.yaml     |    2 +-
 .../bindings/remoteproc/qcom,q6v5.txt         |    2 +-
 .../bindings/remoteproc/qcom,wcnss-pil.txt    |    2 +-
 .../bindings/soc/qcom/qcom,smd-rpm.yaml       |    8 +-
 .../devicetree/bindings/soc/qcom/qcom,smd.txt |   98 --
 .../bindings/soc/qcom/qcom,smd.yaml           |  137 ++
 arch/arm/boot/dts/qcom-apq8064.dtsi           |    8 +-
 .../arm/boot/dts/qcom-apq8074-dragonboard.dts |  614 +++++----
 arch/arm/boot/dts/qcom-apq8084.dtsi           |    2 +-
 .../boot/dts/qcom-msm8974-fairphone-fp2.dts   |  581 +++++----
 .../qcom-msm8974-lge-nexus5-hammerhead.dts    | 1105 ++++++++--------
 .../boot/dts/qcom-msm8974-samsung-klte.dts    | 1127 ++++++++---------
 .../dts/qcom-msm8974-sony-xperia-amami.dts    |  569 ++++-----
 .../dts/qcom-msm8974-sony-xperia-castor.dts   |  908 +++++++------
 .../dts/qcom-msm8974-sony-xperia-honami.dts   |  636 +++++-----
 arch/arm/boot/dts/qcom-msm8974.dtsi           |   16 +-
 arch/arm64/boot/dts/qcom/msm8916.dtsi         |    2 +-
 arch/arm64/boot/dts/qcom/msm8953.dtsi         |    4 +-
 arch/arm64/boot/dts/qcom/msm8992.dtsi         |    2 +-
 arch/arm64/boot/dts/qcom/msm8994.dtsi         |    3 +-
 arch/arm64/boot/dts/qcom/msm8996.dtsi         |    2 +-
 arch/arm64/boot/dts/qcom/qcs404.dtsi          |    2 +-
 24 files changed, 3000 insertions(+), 2968 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/clock/qcom,rpmcc.txt
 create mode 100644 Documentation/devicetree/bindings/clock/qcom,rpmcc.yaml
 delete mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,smd.txt
 create mode 100644 Documentation/devicetree/bindings/soc/qcom/qcom,smd.yaml

Comments

Krzysztof Kozlowski April 18, 2022, 3:10 p.m. UTC | #1 
On 01/04/2022 22:10, Krzysztof Kozlowski wrote:
> Using node paths to extend or override a device tree node is error
> prone.  If there was a typo error, a new node will be created instead of
> extending the existing node.  This will lead to run-time errors that
> could be hard to detect.
> 
> A mistyped label on the other hand, will cause a dtc compile error
> (during build time).  This also reduces the indentation making the code
> easier to read.
> 
> Re-order the overrides by label name.  The pre/post DTBS are the same.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../arm/boot/dts/qcom-apq8074-dragonboard.dts |  614 +++++-----
>  .../boot/dts/qcom-msm8974-fairphone-fp2.dts   |  581 +++++----
>  .../qcom-msm8974-lge-nexus5-hammerhead.dts    | 1075 ++++++++--------
>  .../boot/dts/qcom-msm8974-samsung-klte.dts    | 1083 ++++++++---------
>  .../dts/qcom-msm8974-sony-xperia-amami.dts    |  569 +++++----
>  .../dts/qcom-msm8974-sony-xperia-castor.dts   |  908 +++++++-------
>  .../dts/qcom-msm8974-sony-xperia-honami.dts   |  636 +++++-----
>  arch/arm/boot/dts/qcom-msm8974.dtsi           |   16 +-
>  8 files changed, 2730 insertions(+), 2752 deletions(-)
> 

Hi Bjorn,

I see you applied other arm-dts patches from this set, except this one.
Any comments here? This is quite a big one, so maybe it had trouble
making through the discussion lists?


Best regards,
Krzysztof