[PATCH v3 00/15] dt-bindings: remoteproc: qcom: split and reorganize PAS/PIL

Krzysztof Kozlowski posted 15 patches 1 year, 4 months ago
.../bindings/remoteproc/qcom,adsp.yaml        | 414 +-----------------
.../bindings/remoteproc/qcom,pas-common.yaml  |  97 ++++
.../bindings/remoteproc/qcom,qcs404-pas.yaml  |  86 ++++
.../bindings/remoteproc/qcom,sc7180-pas.yaml  | 125 ++++++
.../bindings/remoteproc/qcom,sc8180x-pas.yaml |  87 ++++
.../remoteproc/qcom,sc8280xp-pas.yaml         | 139 ++++++
.../bindings/remoteproc/qcom,sdx55-pas.yaml   | 101 +++++
.../bindings/remoteproc/qcom,sm6350-pas.yaml  | 159 +++++++
.../bindings/remoteproc/qcom,sm8150-pas.yaml  | 166 +++++++
.../bindings/remoteproc/qcom,sm8350-pas.yaml  | 174 ++++++++
arch/arm64/boot/dts/qcom/msm8996.dtsi         |   3 +-
arch/arm64/boot/dts/qcom/qcs404.dtsi          |  46 +-
arch/arm64/boot/dts/qcom/sc7180-idp.dts       |  18 +
arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi  |  18 +
arch/arm64/boot/dts/qcom/sc7180.dtsi          |  20 +-
.../dts/qcom/sc7280-herobrine-lte-sku.dtsi    |  17 +
arch/arm64/boot/dts/qcom/sc7280.dtsi          |  16 +-
17 files changed, 1221 insertions(+), 465 deletions(-)
create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,pas-common.yaml
create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,qcs404-pas.yaml
create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sc7180-pas.yaml
create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sc8180x-pas.yaml
create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml
create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sdx55-pas.yaml
create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sm6350-pas.yaml
create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sm8150-pas.yaml
create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sm8350-pas.yaml
[PATCH v3 00/15] dt-bindings: remoteproc: qcom: split and reorganize PAS/PIL
Posted by Krzysztof Kozlowski 1 year, 4 months ago
Changes since v2
================
1. Allow only one or two clocks, after dropping clocks related to PIL binding.
2. Drop if:then: for the clock and put it directly under properties
3. Merge two if:then: clauses for setting interrupts.
4. New patches: DTS fixes, qcom,adsp: drop resets and qcom,halt-regs,
   qcom,qcs404-pas, qcom,sc7180-pas and last msm8996-slpi-pil fix.

Changes since v1
================
1. Keep resets, reset-names, qcom,qmp and qcom,halt-regs in qcom,adsp, because
   they are not shared with most of PAS bindings.
2. Add firmware-name to examples.
3. New patches: qcom,sc8180x-pas and qcom,sdx55-pas.

Description
===========
The Qualcomm PAS/PIL (qcom,adsp.yaml) bindings grew considerably with huge
amount of if:then:else blocks and 40 compatibles (still growing).   These
if:then:else blocks constrain clocks, power-domains and interrupt per variants.
Adding new variants is now tricky - it's easy to forget to add variant to one
if:then: clause.

Split the bindings into reusable schema and per-soc bindings, groupping them
when applicable.

This part is finished, but more and more compatibles should be moved out of
qcom,adsp.yaml, eventually dropping the file.

Best regards,
Krzysztof

Krzysztof Kozlowski (15):
  arm64: dts: qcom: msm8996: drop address/size cells from smd-edge
  arm64: dts: qcom: qcs404: align CDSP PAS node with bindings
  arm64: dts: qcom: sc7180: align MPSS PAS node with bindings
  arm64: dts: qcom: sc7280: align MPSS PAS node with bindings
  dt-bindings: remoteproc: qcom,adsp: drop resets and qcom,halt-regs
  dt-bindings: remoteproc: qcom,adsp: split common part
  dt-bindings: remoteproc: qcom,sm8350-pas: split into separate file
  dt-bindings: remoteproc: qcom,sm8150-pas: split into separate file
  dt-bindings: remoteproc: qcom,sm6350-pas: split into separate file
  dt-bindings: remoteproc: qcom,sc8280xp-pas: split into separate file
  dt-bindings: remoteproc: qcom,sc8180x-pas: split into separate file
  dt-bindings: remoteproc: qcom,sdx55-pas: split into separate file
  dt-bindings: remoteproc: qcom,qcs404-pas: split into separate file
  dt-bindings: remoteproc: qcom,sc7180-pas: split into separate file
  dt-bindings: remoteproc: qcom,adsp: correct msm8996-slpi-pil clocks

 .../bindings/remoteproc/qcom,adsp.yaml        | 414 +-----------------
 .../bindings/remoteproc/qcom,pas-common.yaml  |  97 ++++
 .../bindings/remoteproc/qcom,qcs404-pas.yaml  |  86 ++++
 .../bindings/remoteproc/qcom,sc7180-pas.yaml  | 125 ++++++
 .../bindings/remoteproc/qcom,sc8180x-pas.yaml |  87 ++++
 .../remoteproc/qcom,sc8280xp-pas.yaml         | 139 ++++++
 .../bindings/remoteproc/qcom,sdx55-pas.yaml   | 101 +++++
 .../bindings/remoteproc/qcom,sm6350-pas.yaml  | 159 +++++++
 .../bindings/remoteproc/qcom,sm8150-pas.yaml  | 166 +++++++
 .../bindings/remoteproc/qcom,sm8350-pas.yaml  | 174 ++++++++
 arch/arm64/boot/dts/qcom/msm8996.dtsi         |   3 +-
 arch/arm64/boot/dts/qcom/qcs404.dtsi          |  46 +-
 arch/arm64/boot/dts/qcom/sc7180-idp.dts       |  18 +
 arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi  |  18 +
 arch/arm64/boot/dts/qcom/sc7180.dtsi          |  20 +-
 .../dts/qcom/sc7280-herobrine-lte-sku.dtsi    |  17 +
 arch/arm64/boot/dts/qcom/sc7280.dtsi          |  16 +-
 17 files changed, 1221 insertions(+), 465 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,pas-common.yaml
 create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,qcs404-pas.yaml
 create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sc7180-pas.yaml
 create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sc8180x-pas.yaml
 create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sc8280xp-pas.yaml
 create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sdx55-pas.yaml
 create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sm6350-pas.yaml
 create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sm8150-pas.yaml
 create mode 100644 Documentation/devicetree/bindings/remoteproc/qcom,sm8350-pas.yaml

-- 
2.34.1
Re: (subset) [PATCH v3 00/15] dt-bindings: remoteproc: qcom: split and reorganize PAS/PIL
Posted by Bjorn Andersson 1 year, 3 months ago
On Thu, 24 Nov 2022 19:43:18 +0100, Krzysztof Kozlowski wrote:
> Changes since v2
> ================
> 1. Allow only one or two clocks, after dropping clocks related to PIL binding.
> 2. Drop if:then: for the clock and put it directly under properties
> 3. Merge two if:then: clauses for setting interrupts.
> 4. New patches: DTS fixes, qcom,adsp: drop resets and qcom,halt-regs,
>    qcom,qcs404-pas, qcom,sc7180-pas and last msm8996-slpi-pil fix.
> 
> [...]

Applied, thanks!

[05/15] dt-bindings: remoteproc: qcom,adsp: drop resets and qcom,halt-regs
        commit: c97c4b480fcba2e6ebfacb3990d8b9092916d986
[06/15] dt-bindings: remoteproc: qcom,adsp: split common part
        commit: ea5a10ae295425c9f52577d91b2e45290291d242
[07/15] dt-bindings: remoteproc: qcom,sm8350-pas: split into separate file
        commit: 4e8b39a4d38ebb192ebff977d8a961fd97e7ce5f
[08/15] dt-bindings: remoteproc: qcom,sm8150-pas: split into separate file
        commit: db292776589f1fcbcf2fc6eaebe40a3a2abb4521
[09/15] dt-bindings: remoteproc: qcom,sm6350-pas: split into separate file
        commit: 41729b772c13105ee126ee6fe1bf2cd93c7bd258
[10/15] dt-bindings: remoteproc: qcom,sc8280xp-pas: split into separate file
        commit: 397c619cad8109f5904466ee76d5a1533d2f1590
[11/15] dt-bindings: remoteproc: qcom,sc8180x-pas: split into separate file
        commit: 4b4157518f1ab1276cd08dfab0e51b1409c22e40
[12/15] dt-bindings: remoteproc: qcom,sdx55-pas: split into separate file
        commit: b6f8410eab9270000b8b13b88bc038e9f27c2c45
[13/15] dt-bindings: remoteproc: qcom,qcs404-pas: split into separate file
        commit: 255d7a9581ed4506dddf993aad9dc27bff8296d1
[14/15] dt-bindings: remoteproc: qcom,sc7180-pas: split into separate file
        commit: 8bb92d6fd0b3788b4270eff547cb42cb64db1bff
[15/15] dt-bindings: remoteproc: qcom,adsp: correct msm8996-slpi-pil clocks
        commit: 569d3a7580bcbc463920b0d84ca5caf23e808f90

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>