All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema
@ 2023-12-08 21:55 Krzysztof Kozlowski
  2023-12-08 21:55 ` [PATCH 02/10] dt-bindings: pinctrl: qcom,qdu1000-tlmm: restrict number of interrupts Krzysztof Kozlowski
                   ` (10 more replies)
  0 siblings, 11 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-08 21:55 UTC (permalink / raw)
  To: Bjorn Andersson, Andy Gross, Konrad Dybcio, Linus Walleij,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Srinivas Kandagatla, Stephan Gerhold, Shawn Guo, Melody Olvera,
	Bartosz Golaszewski, Richard Acayan, Vinod Koul, krishna Lanka,
	Rohit Agarwal, Iskren Chernev, Martin Botka, Danila Tikhonov,
	Abel Vesa, Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

Just like regular TLMM pin controllers in Qualcomm SoCs, the Low Power
Audio SubSystem (LPASS) Low Power Island (LPI) TLMM blocks share a lot
of properties, so common part can be moved to separate schema to reduce
code duplication and make reviewing easier.

Except the move of common part, this introduces effective changes:
1. To all LPASS LPI bindings: Reference pinmux-node.yaml in each pin
   muxing and configuration node, to bring definition of "function" and
   "pins" properties.

2. qcom,sc7280-lpass-lpi-pinctrl: Reference pinctrl.yaml in top leve.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../pinctrl/qcom,lpass-lpi-common.yaml        | 75 +++++++++++++++++++
 .../qcom,sc7280-lpass-lpi-pinctrl.yaml        | 49 ++----------
 .../qcom,sc8280xp-lpass-lpi-pinctrl.yaml      | 49 +-----------
 .../qcom,sm6115-lpass-lpi-pinctrl.yaml        | 48 +-----------
 .../qcom,sm8250-lpass-lpi-pinctrl.yaml        | 49 +-----------
 .../qcom,sm8350-lpass-lpi-pinctrl.yaml        | 49 +-----------
 .../qcom,sm8450-lpass-lpi-pinctrl.yaml        | 49 +-----------
 .../qcom,sm8550-lpass-lpi-pinctrl.yaml        | 49 +-----------
 .../qcom,sm8650-lpass-lpi-pinctrl.yaml        | 49 +-----------
 9 files changed, 109 insertions(+), 357 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/pinctrl/qcom,lpass-lpi-common.yaml

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,lpass-lpi-common.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,lpass-lpi-common.yaml
new file mode 100644
index 000000000000..3b5045730471
--- /dev/null
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,lpass-lpi-common.yaml
@@ -0,0 +1,75 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/pinctrl/qcom,lpass-lpi-common.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Qualcomm SoC LPASS LPI TLMM Common Properties
+
+maintainers:
+  - Bjorn Andersson <andersson@kernel.org>
+  - Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
+  - Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
+
+description:
+  Common properties for the Top Level Mode Multiplexer pin controllers in the
+  Low Power Audio SubSystem (LPASS) Low Power Island (LPI) of Qualcomm SoCs.
+
+properties:
+  gpio-controller: true
+
+  "#gpio-cells":
+    description:
+      Specifying the pin number and flags, as defined in
+      include/dt-bindings/gpio/gpio.h
+    const: 2
+
+  gpio-ranges:
+    maxItems: 1
+
+required:
+  - gpio-controller
+  - "#gpio-cells"
+  - gpio-ranges
+
+allOf:
+  - $ref: pinctrl.yaml#
+
+additionalProperties: true
+
+$defs:
+  qcom-tlmm-state:
+    properties:
+      drive-strength:
+        enum: [2, 4, 6, 8, 10, 12, 14, 16]
+        default: 2
+        description:
+          Selects the drive strength for the specified pins, in mA.
+
+      slew-rate:
+        enum: [0, 1, 2, 3]
+        default: 0
+        description: |
+          0: No adjustments
+          1: Higher Slew rate (faster edges)
+          2: Lower Slew rate (slower edges)
+          3: Reserved (No adjustments)
+
+      bias-bus-hold: true
+      bias-pull-down: true
+      bias-pull-up: true
+      bias-disable: true
+      input-enable: true
+      output-high: true
+      output-low: true
+
+    required:
+      - pins
+      - function
+
+    allOf:
+      - $ref: pincfg-node.yaml#
+      - $ref: pinmux-node.yaml#
+
+    additionalProperties: true
+
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-lpass-lpi-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-lpass-lpi-pinctrl.yaml
index 00c5a00e35fc..08801cc4e476 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-lpass-lpi-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-lpass-lpi-pinctrl.yaml
@@ -20,16 +20,6 @@ properties:
   reg:
     maxItems: 2
 
-  gpio-controller: true
-
-  "#gpio-cells":
-    description: Specifying the pin number and flags, as defined in
-      include/dt-bindings/gpio/gpio.h
-    const: 2
-
-  gpio-ranges:
-    maxItems: 1
-
 patternProperties:
   "-state$":
     oneOf:
@@ -45,7 +35,8 @@ $defs:
     description:
       Pinctrl node's client devices use subnodes for desired pin configuration.
       Client device subnodes use below standard properties.
-    $ref: /schemas/pinctrl/pincfg-node.yaml
+    $ref: qcom,lpass-lpi-common.yaml#/$defs/qcom-tlmm-state
+    unevaluatedProperties: false
 
     properties:
       pins:
@@ -68,42 +59,14 @@ $defs:
           Specify the alternative function to be configured for the specified
           pins.
 
-      drive-strength:
-        enum: [2, 4, 6, 8, 10, 12, 14, 16]
-        default: 2
-        description:
-          Selects the drive strength for the specified pins, in mA.
-
-      slew-rate:
-        enum: [0, 1, 2, 3]
-        default: 0
-        description: |
-            0: No adjustments
-            1: Higher Slew rate (faster edges)
-            2: Lower Slew rate (slower edges)
-            3: Reserved (No adjustments)
-
-      bias-pull-down: true
-      bias-pull-up: true
-      bias-bus-hold: true
-      bias-disable: true
-      output-high: true
-      output-low: true
-
-    required:
-      - pins
-      - function
-
-    additionalProperties: false
-
 required:
   - compatible
   - reg
-  - gpio-controller
-  - "#gpio-cells"
-  - gpio-ranges
 
-additionalProperties: false
+allOf:
+  - $ref: qcom,lpass-lpi-common.yaml#
+
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-lpass-lpi-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-lpass-lpi-pinctrl.yaml
index a9167dac9ab5..240e6d45cc95 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-lpass-lpi-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-lpass-lpi-pinctrl.yaml
@@ -32,16 +32,6 @@ properties:
       - const: core
       - const: audio
 
-  gpio-controller: true
-
-  "#gpio-cells":
-    description: Specifying the pin number and flags, as defined in
-      include/dt-bindings/gpio/gpio.h
-    const: 2
-
-  gpio-ranges:
-    maxItems: 1
-
 patternProperties:
   "-state$":
     oneOf:
@@ -57,7 +47,8 @@ $defs:
     description:
       Pinctrl node's client devices use subnodes for desired pin configuration.
       Client device subnodes use below standard properties.
-    $ref: /schemas/pinctrl/pincfg-node.yaml
+    $ref: qcom,lpass-lpi-common.yaml#/$defs/qcom-tlmm-state
+    unevaluatedProperties: false
 
     properties:
       pins:
@@ -79,48 +70,16 @@ $defs:
           Specify the alternative function to be configured for the specified
           pins.
 
-      drive-strength:
-        enum: [2, 4, 6, 8, 10, 12, 14, 16]
-        default: 2
-        description:
-          Selects the drive strength for the specified pins, in mA.
-
-      slew-rate:
-        enum: [0, 1, 2, 3]
-        default: 0
-        description: |
-          0: No adjustments
-          1: Higher Slew rate (faster edges)
-          2: Lower Slew rate (slower edges)
-          3: Reserved (No adjustments)
-
-      bias-bus-hold: true
-      bias-pull-down: true
-      bias-pull-up: true
-      bias-disable: true
-      input-enable: true
-      output-high: true
-      output-low: true
-
-    required:
-      - pins
-      - function
-
-    additionalProperties: false
-
 allOf:
-  - $ref: pinctrl.yaml#
+  - $ref: qcom,lpass-lpi-common.yaml#
 
 required:
   - compatible
   - reg
   - clocks
   - clock-names
-  - gpio-controller
-  - "#gpio-cells"
-  - gpio-ranges
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-lpass-lpi-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-lpass-lpi-pinctrl.yaml
index abac3311fc55..f4cf2ce86fcd 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-lpass-lpi-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-lpass-lpi-pinctrl.yaml
@@ -31,16 +31,6 @@ properties:
     items:
       - const: audio
 
-  gpio-controller: true
-
-  "#gpio-cells":
-    description: Specifying the pin number and flags, as defined in
-      include/dt-bindings/gpio/gpio.h
-    const: 2
-
-  gpio-ranges:
-    maxItems: 1
-
 patternProperties:
   "-state$":
     oneOf:
@@ -56,7 +46,8 @@ $defs:
     description:
       Pinctrl node's client devices use subnodes for desired pin configuration.
       Client device subnodes use below standard properties.
-    $ref: /schemas/pinctrl/pincfg-node.yaml
+    $ref: qcom,lpass-lpi-common.yaml#/$defs/qcom-tlmm-state
+    unevaluatedProperties: false
 
     properties:
       pins:
@@ -75,48 +66,17 @@ $defs:
           Specify the alternative function to be configured for the specified
           pins.
 
-      drive-strength:
-        enum: [2, 4, 6, 8, 10, 12, 14, 16]
-        default: 2
-        description:
-          Selects the drive strength for the specified pins, in mA.
-
-      slew-rate:
-        enum: [0, 1, 2, 3]
-        default: 0
-        description: |
-          0: No adjustments
-          1: Higher Slew rate (faster edges)
-          2: Lower Slew rate (slower edges)
-          3: Reserved (No adjustments)
-
-      bias-bus-hold: true
-      bias-pull-down: true
-      bias-pull-up: true
-      bias-disable: true
-      input-enable: true
-      output-high: true
-      output-low: true
-
-    required:
-      - pins
-      - function
-
-    additionalProperties: false
 
 allOf:
-  - $ref: pinctrl.yaml#
+  - $ref: qcom,lpass-lpi-common.yaml#
 
 required:
   - compatible
   - reg
   - clocks
   - clock-names
-  - gpio-controller
-  - "#gpio-cells"
-  - gpio-ranges
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-lpass-lpi-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-lpass-lpi-pinctrl.yaml
index 4b4be7efc150..750c996c10a7 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-lpass-lpi-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-lpass-lpi-pinctrl.yaml
@@ -30,16 +30,6 @@ properties:
       - const: core
       - const: audio
 
-  gpio-controller: true
-
-  "#gpio-cells":
-    description: Specifying the pin number and flags, as defined in
-      include/dt-bindings/gpio/gpio.h
-    const: 2
-
-  gpio-ranges:
-    maxItems: 1
-
 patternProperties:
   "-state$":
     oneOf:
@@ -55,7 +45,8 @@ $defs:
     description:
       Pinctrl node's client devices use subnodes for desired pin configuration.
       Client device subnodes use below standard properties.
-    $ref: /schemas/pinctrl/pincfg-node.yaml
+    $ref: qcom,lpass-lpi-common.yaml#/$defs/qcom-tlmm-state
+    unevaluatedProperties: false
 
     properties:
       pins:
@@ -78,48 +69,16 @@ $defs:
           Specify the alternative function to be configured for the specified
           pins.
 
-      drive-strength:
-        enum: [2, 4, 6, 8, 10, 12, 14, 16]
-        default: 2
-        description:
-          Selects the drive strength for the specified pins, in mA.
-
-      slew-rate:
-        enum: [0, 1, 2, 3]
-        default: 0
-        description: |
-            0: No adjustments
-            1: Higher Slew rate (faster edges)
-            2: Lower Slew rate (slower edges)
-            3: Reserved (No adjustments)
-
-      bias-pull-down: true
-      bias-pull-up: true
-      bias-bus-hold: true
-      bias-disable: true
-      input-enable: true
-      output-high: true
-      output-low: true
-
-    required:
-      - pins
-      - function
-
-    additionalProperties: false
-
 allOf:
-  - $ref: pinctrl.yaml#
+  - $ref: qcom,lpass-lpi-common.yaml#
 
 required:
   - compatible
   - reg
   - clocks
   - clock-names
-  - gpio-controller
-  - "#gpio-cells"
-  - gpio-ranges
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-lpass-lpi-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-lpass-lpi-pinctrl.yaml
index 2e65ae08dd21..9d782f910b31 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-lpass-lpi-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-lpass-lpi-pinctrl.yaml
@@ -33,16 +33,6 @@ properties:
       - const: core
       - const: audio
 
-  gpio-controller: true
-
-  "#gpio-cells":
-    description: Specifying the pin number and flags, as defined in
-      include/dt-bindings/gpio/gpio.h
-    const: 2
-
-  gpio-ranges:
-    maxItems: 1
-
 patternProperties:
   "-state$":
     oneOf:
@@ -58,7 +48,8 @@ $defs:
     description:
       Pinctrl node's client devices use subnodes for desired pin configuration.
       Client device subnodes use below standard properties.
-    $ref: /schemas/pinctrl/pincfg-node.yaml
+    $ref: qcom,lpass-lpi-common.yaml#/$defs/qcom-tlmm-state
+    unevaluatedProperties: false
 
     properties:
       pins:
@@ -81,48 +72,16 @@ $defs:
           Specify the alternative function to be configured for the specified
           pins.
 
-      drive-strength:
-        enum: [2, 4, 6, 8, 10, 12, 14, 16]
-        default: 2
-        description:
-          Selects the drive strength for the specified pins, in mA.
-
-      slew-rate:
-        enum: [0, 1, 2, 3]
-        default: 0
-        description: |
-          0: No adjustments
-          1: Higher Slew rate (faster edges)
-          2: Lower Slew rate (slower edges)
-          3: Reserved (No adjustments)
-
-      bias-bus-hold: true
-      bias-pull-down: true
-      bias-pull-up: true
-      bias-disable: true
-      input-enable: true
-      output-high: true
-      output-low: true
-
-    required:
-      - pins
-      - function
-
-    additionalProperties: false
-
 allOf:
-  - $ref: pinctrl.yaml#
+  - $ref: qcom,lpass-lpi-common.yaml#
 
 required:
   - compatible
   - reg
   - clocks
   - clock-names
-  - gpio-controller
-  - "#gpio-cells"
-  - gpio-ranges
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-lpass-lpi-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-lpass-lpi-pinctrl.yaml
index 1eefa9aa6a86..e7565592da86 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-lpass-lpi-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-lpass-lpi-pinctrl.yaml
@@ -32,16 +32,6 @@ properties:
       - const: core
       - const: audio
 
-  gpio-controller: true
-
-  "#gpio-cells":
-    description: Specifying the pin number and flags, as defined in
-      include/dt-bindings/gpio/gpio.h
-    const: 2
-
-  gpio-ranges:
-    maxItems: 1
-
 patternProperties:
   "-state$":
     oneOf:
@@ -57,7 +47,8 @@ $defs:
     description:
       Pinctrl node's client devices use subnodes for desired pin configuration.
       Client device subnodes use below standard properties.
-    $ref: /schemas/pinctrl/pincfg-node.yaml
+    $ref: qcom,lpass-lpi-common.yaml#/$defs/qcom-tlmm-state
+    unevaluatedProperties: false
 
     properties:
       pins:
@@ -81,48 +72,16 @@ $defs:
           Specify the alternative function to be configured for the specified
           pins.
 
-      drive-strength:
-        enum: [2, 4, 6, 8, 10, 12, 14, 16]
-        default: 2
-        description:
-          Selects the drive strength for the specified pins, in mA.
-
-      slew-rate:
-        enum: [0, 1, 2, 3]
-        default: 0
-        description: |
-          0: No adjustments
-          1: Higher Slew rate (faster edges)
-          2: Lower Slew rate (slower edges)
-          3: Reserved (No adjustments)
-
-      bias-bus-hold: true
-      bias-pull-down: true
-      bias-pull-up: true
-      bias-disable: true
-      input-enable: true
-      output-high: true
-      output-low: true
-
-    required:
-      - pins
-      - function
-
-    additionalProperties: false
-
 allOf:
-  - $ref: pinctrl.yaml#
+  - $ref: qcom,lpass-lpi-common.yaml#
 
 required:
   - compatible
   - reg
   - clocks
   - clock-names
-  - gpio-controller
-  - "#gpio-cells"
-  - gpio-ranges
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-lpass-lpi-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-lpass-lpi-pinctrl.yaml
index ad5e32130fd7..bf4a72facae1 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-lpass-lpi-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-lpass-lpi-pinctrl.yaml
@@ -37,16 +37,6 @@ properties:
       - const: core
       - const: audio
 
-  gpio-controller: true
-
-  "#gpio-cells":
-    description: Specifying the pin number and flags, as defined in
-      include/dt-bindings/gpio/gpio.h
-    const: 2
-
-  gpio-ranges:
-    maxItems: 1
-
 patternProperties:
   "-state$":
     oneOf:
@@ -62,7 +52,8 @@ $defs:
     description:
       Pinctrl node's client devices use subnodes for desired pin configuration.
       Client device subnodes use below standard properties.
-    $ref: /schemas/pinctrl/pincfg-node.yaml
+    $ref: qcom,lpass-lpi-common.yaml#/$defs/qcom-tlmm-state
+    unevaluatedProperties: false
 
     properties:
       pins:
@@ -85,48 +76,16 @@ $defs:
           Specify the alternative function to be configured for the specified
           pins.
 
-      drive-strength:
-        enum: [2, 4, 6, 8, 10, 12, 14, 16]
-        default: 2
-        description:
-          Selects the drive strength for the specified pins, in mA.
-
-      slew-rate:
-        enum: [0, 1, 2, 3]
-        default: 0
-        description: |
-          0: No adjustments
-          1: Higher Slew rate (faster edges)
-          2: Lower Slew rate (slower edges)
-          3: Reserved (No adjustments)
-
-      bias-bus-hold: true
-      bias-pull-down: true
-      bias-pull-up: true
-      bias-disable: true
-      input-enable: true
-      output-high: true
-      output-low: true
-
-    required:
-      - pins
-      - function
-
-    additionalProperties: false
-
 allOf:
-  - $ref: pinctrl.yaml#
+  - $ref: qcom,lpass-lpi-common.yaml#
 
 required:
   - compatible
   - reg
   - clocks
   - clock-names
-  - gpio-controller
-  - "#gpio-cells"
-  - gpio-ranges
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-lpass-lpi-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-lpass-lpi-pinctrl.yaml
index f5736ed140ee..db7214362301 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-lpass-lpi-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-lpass-lpi-pinctrl.yaml
@@ -32,16 +32,6 @@ properties:
       - const: core
       - const: audio
 
-  gpio-controller: true
-
-  "#gpio-cells":
-    description: Specifying the pin number and flags, as defined in
-      include/dt-bindings/gpio/gpio.h
-    const: 2
-
-  gpio-ranges:
-    maxItems: 1
-
 patternProperties:
   "-state$":
     oneOf:
@@ -57,7 +47,8 @@ $defs:
     description:
       Pinctrl node's client devices use subnodes for desired pin configuration.
       Client device subnodes use below standard properties.
-    $ref: /schemas/pinctrl/pincfg-node.yaml
+    $ref: qcom,lpass-lpi-common.yaml#/$defs/qcom-tlmm-state
+    unevaluatedProperties: false
 
     properties:
       pins:
@@ -80,48 +71,16 @@ $defs:
           Specify the alternative function to be configured for the specified
           pins.
 
-      drive-strength:
-        enum: [2, 4, 6, 8, 10, 12, 14, 16]
-        default: 2
-        description:
-          Selects the drive strength for the specified pins, in mA.
-
-      slew-rate:
-        enum: [0, 1, 2, 3]
-        default: 0
-        description: |
-          0: No adjustments
-          1: Higher Slew rate (faster edges)
-          2: Lower Slew rate (slower edges)
-          3: Reserved (No adjustments)
-
-      bias-bus-hold: true
-      bias-pull-down: true
-      bias-pull-up: true
-      bias-disable: true
-      input-enable: true
-      output-high: true
-      output-low: true
-
-    required:
-      - pins
-      - function
-
-    additionalProperties: false
-
 allOf:
-  - $ref: pinctrl.yaml#
+  - $ref: qcom,lpass-lpi-common.yaml#
 
 required:
   - compatible
   - reg
   - clocks
   - clock-names
-  - gpio-controller
-  - "#gpio-cells"
-  - gpio-ranges
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 02/10] dt-bindings: pinctrl: qcom,qdu1000-tlmm: restrict number of interrupts
  2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
@ 2023-12-08 21:55 ` Krzysztof Kozlowski
  2023-12-13 19:01   ` Rob Herring
  2023-12-08 21:55 ` [PATCH 03/10] dt-bindings: pinctrl: qcom,sa8775p-tlmm: " Krzysztof Kozlowski
                   ` (9 subsequent siblings)
  10 siblings, 1 reply; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-08 21:55 UTC (permalink / raw)
  To: Bjorn Andersson, Andy Gross, Konrad Dybcio, Linus Walleij,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Srinivas Kandagatla, Stephan Gerhold, Shawn Guo, Melody Olvera,
	Bartosz Golaszewski, Richard Acayan, Vinod Koul, krishna Lanka,
	Rohit Agarwal, Iskren Chernev, Martin Botka, Danila Tikhonov,
	Abel Vesa, Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

QDU1000 TLMM pin controller comes with only one interrupt, so narrow
the number of interrupts previously defined in common TLMM bindings.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml        | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml
index 237cac4f6ce1..47363c23f4ea 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml
@@ -23,7 +23,9 @@ properties:
   reg:
     maxItems: 1
 
-  interrupts: true
+  interrupts:
+    maxItems: 1
+
   interrupt-controller: true
   "#interrupt-cells": true
   gpio-controller: true
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 03/10] dt-bindings: pinctrl: qcom,sa8775p-tlmm: restrict number of interrupts
  2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
  2023-12-08 21:55 ` [PATCH 02/10] dt-bindings: pinctrl: qcom,qdu1000-tlmm: restrict number of interrupts Krzysztof Kozlowski
@ 2023-12-08 21:55 ` Krzysztof Kozlowski
  2023-12-13 18:40   ` Rob Herring
  2023-12-08 21:55 ` [PATCH 04/10] dt-bindings: pinctrl: qcom,sdx75-tlmm: " Krzysztof Kozlowski
                   ` (8 subsequent siblings)
  10 siblings, 1 reply; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-08 21:55 UTC (permalink / raw)
  To: Bjorn Andersson, Andy Gross, Konrad Dybcio, Linus Walleij,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Srinivas Kandagatla, Stephan Gerhold, Shawn Guo, Melody Olvera,
	Bartosz Golaszewski, Richard Acayan, Vinod Koul, krishna Lanka,
	Rohit Agarwal, Iskren Chernev, Martin Botka, Danila Tikhonov,
	Abel Vesa, Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

SA8775p TLMM pin controller comes with only one interrupt, so narrow
the number of interrupts previously defined in common TLMM bindings.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml        | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml
index 2173c5255638..d27933258db7 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml
@@ -22,7 +22,9 @@ properties:
   reg:
     maxItems: 1
 
-  interrupts: true
+  interrupts:
+    maxItems: 1
+
   interrupt-controller: true
   "#interrupt-cells": true
   gpio-controller: true
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 04/10] dt-bindings: pinctrl: qcom,sdx75-tlmm: restrict number of interrupts
  2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
  2023-12-08 21:55 ` [PATCH 02/10] dt-bindings: pinctrl: qcom,qdu1000-tlmm: restrict number of interrupts Krzysztof Kozlowski
  2023-12-08 21:55 ` [PATCH 03/10] dt-bindings: pinctrl: qcom,sa8775p-tlmm: " Krzysztof Kozlowski
@ 2023-12-08 21:55 ` Krzysztof Kozlowski
  2023-12-13 18:40   ` Rob Herring
  2023-12-08 21:55 ` [PATCH 05/10] dt-bindings: pinctrl: qcom,sm8550-tlmm: " Krzysztof Kozlowski
                   ` (7 subsequent siblings)
  10 siblings, 1 reply; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-08 21:55 UTC (permalink / raw)
  To: Bjorn Andersson, Andy Gross, Konrad Dybcio, Linus Walleij,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Srinivas Kandagatla, Stephan Gerhold, Shawn Guo, Melody Olvera,
	Bartosz Golaszewski, Richard Acayan, Vinod Koul, krishna Lanka,
	Rohit Agarwal, Iskren Chernev, Martin Botka, Danila Tikhonov,
	Abel Vesa, Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

SDX75 TLMM pin controller comes with only one interrupt, so narrow
the number of interrupts previously defined in common TLMM bindings.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml          | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml
index 7cb96aa75b08..62cd8f84af9c 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml
@@ -22,7 +22,9 @@ properties:
   reg:
     maxItems: 1
 
-  interrupts: true
+  interrupts:
+    maxItems: 1
+
   interrupt-controller: true
   "#interrupt-cells": true
   gpio-controller: true
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 05/10] dt-bindings: pinctrl: qcom,sm8550-tlmm: restrict number of interrupts
  2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
                   ` (2 preceding siblings ...)
  2023-12-08 21:55 ` [PATCH 04/10] dt-bindings: pinctrl: qcom,sdx75-tlmm: " Krzysztof Kozlowski
@ 2023-12-08 21:55 ` Krzysztof Kozlowski
  2023-12-13 18:40   ` Rob Herring
  2023-12-08 21:55 ` [PATCH 06/10] dt-bindings: pinctrl: qcom,sm8650-tlmm: " Krzysztof Kozlowski
                   ` (6 subsequent siblings)
  10 siblings, 1 reply; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-08 21:55 UTC (permalink / raw)
  To: Bjorn Andersson, Andy Gross, Konrad Dybcio, Linus Walleij,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Srinivas Kandagatla, Stephan Gerhold, Shawn Guo, Melody Olvera,
	Bartosz Golaszewski, Richard Acayan, Vinod Koul, krishna Lanka,
	Rohit Agarwal, Iskren Chernev, Martin Botka, Danila Tikhonov,
	Abel Vesa, Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

SM8550 TLMM pin controller comes with only one interrupt, so narrow
the number of interrupts previously defined in common TLMM bindings.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml         | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml
index f789c7753a92..567e44875c29 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml
@@ -22,7 +22,9 @@ properties:
   reg:
     maxItems: 1
 
-  interrupts: true
+  interrupts:
+    maxItems: 1
+
   interrupt-controller: true
   "#interrupt-cells": true
   gpio-controller: true
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 06/10] dt-bindings: pinctrl: qcom,sm8650-tlmm: restrict number of interrupts
  2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
                   ` (3 preceding siblings ...)
  2023-12-08 21:55 ` [PATCH 05/10] dt-bindings: pinctrl: qcom,sm8550-tlmm: " Krzysztof Kozlowski
@ 2023-12-08 21:55 ` Krzysztof Kozlowski
  2023-12-13 18:40   ` Rob Herring
  2023-12-08 21:55 ` [PATCH 07/10] dt-bindings: pinctrl: qcom,x1e80100-tlmm: " Krzysztof Kozlowski
                   ` (5 subsequent siblings)
  10 siblings, 1 reply; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-08 21:55 UTC (permalink / raw)
  To: Bjorn Andersson, Andy Gross, Konrad Dybcio, Linus Walleij,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Srinivas Kandagatla, Stephan Gerhold, Shawn Guo, Melody Olvera,
	Bartosz Golaszewski, Richard Acayan, Vinod Koul, krishna Lanka,
	Rohit Agarwal, Iskren Chernev, Martin Botka, Danila Tikhonov,
	Abel Vesa, Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

SM8650 TLMM pin controller comes with only one interrupt, so narrow
the number of interrupts previously defined in common TLMM bindings.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml         | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml
index 30432be8fb07..370789ce934b 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml
@@ -22,7 +22,9 @@ properties:
   reg:
     maxItems: 1
 
-  interrupts: true
+  interrupts:
+    maxItems: 1
+
   interrupt-controller: true
   "#interrupt-cells": true
   gpio-controller: true
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 07/10] dt-bindings: pinctrl: qcom,x1e80100-tlmm: restrict number of interrupts
  2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
                   ` (4 preceding siblings ...)
  2023-12-08 21:55 ` [PATCH 06/10] dt-bindings: pinctrl: qcom,sm8650-tlmm: " Krzysztof Kozlowski
@ 2023-12-08 21:55 ` Krzysztof Kozlowski
  2023-12-13 18:40   ` Rob Herring
  2023-12-08 21:55 ` [PATCH 08/10] dt-bindings: pinctrl: qcom,ipq5018-tlmm: use common TLMM bindings Krzysztof Kozlowski
                   ` (4 subsequent siblings)
  10 siblings, 1 reply; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-08 21:55 UTC (permalink / raw)
  To: Bjorn Andersson, Andy Gross, Konrad Dybcio, Linus Walleij,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Srinivas Kandagatla, Stephan Gerhold, Shawn Guo, Melody Olvera,
	Bartosz Golaszewski, Richard Acayan, Vinod Koul, krishna Lanka,
	Rohit Agarwal, Iskren Chernev, Martin Botka, Danila Tikhonov,
	Abel Vesa, Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

X1E80100 TLMM pin controller comes with only one interrupt, so narrow
the number of interrupts previously defined in common TLMM bindings.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml       | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml
index 2bde8845a31e..3249c63a29cf 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml
@@ -22,7 +22,9 @@ properties:
   reg:
     maxItems: 1
 
-  interrupts: true
+  interrupts:
+    maxItems: 1
+
   interrupt-controller: true
   "#interrupt-cells": true
   gpio-controller: true
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 08/10] dt-bindings: pinctrl: qcom,ipq5018-tlmm: use common TLMM bindings
  2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
                   ` (5 preceding siblings ...)
  2023-12-08 21:55 ` [PATCH 07/10] dt-bindings: pinctrl: qcom,x1e80100-tlmm: " Krzysztof Kozlowski
@ 2023-12-08 21:55 ` Krzysztof Kozlowski
  2023-12-13 18:41   ` Rob Herring
  2023-12-08 21:55 ` [PATCH 09/10] dt-bindings: pinctrl: qcom: drop common properties Krzysztof Kozlowski
                   ` (3 subsequent siblings)
  10 siblings, 1 reply; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-08 21:55 UTC (permalink / raw)
  To: Bjorn Andersson, Andy Gross, Konrad Dybcio, Linus Walleij,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Srinivas Kandagatla, Stephan Gerhold, Shawn Guo, Melody Olvera,
	Bartosz Golaszewski, Richard Acayan, Vinod Koul, krishna Lanka,
	Rohit Agarwal, Iskren Chernev, Martin Botka, Danila Tikhonov,
	Abel Vesa, Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

Reference common Qualcomm SoC TLMM bindings to drop commonly used
properties and also bring other schemas for common definitions.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../bindings/pinctrl/qcom,ipq5018-tlmm.yaml          | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,ipq5018-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,ipq5018-tlmm.yaml
index fad0118fd521..23300606547c 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,ipq5018-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,ipq5018-tlmm.yaml
@@ -23,13 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 24
@@ -95,7 +88,10 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+allOf:
+  - $ref: /schemas/pinctrl/qcom,tlmm-common.yaml#
+
+unevaluatedProperties: false
 
 examples:
   - |
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 09/10] dt-bindings: pinctrl: qcom: drop common properties
  2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
                   ` (6 preceding siblings ...)
  2023-12-08 21:55 ` [PATCH 08/10] dt-bindings: pinctrl: qcom,ipq5018-tlmm: use common TLMM bindings Krzysztof Kozlowski
@ 2023-12-08 21:55 ` Krzysztof Kozlowski
  2023-12-11 23:22   ` Richard Acayan
  2023-12-08 21:55 ` [PATCH 10/10] dt-bindings: pinctrl: qcom: drop common properties and allow wakeup-parent Krzysztof Kozlowski
                   ` (2 subsequent siblings)
  10 siblings, 1 reply; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-08 21:55 UTC (permalink / raw)
  To: Bjorn Andersson, Andy Gross, Konrad Dybcio, Linus Walleij,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Srinivas Kandagatla, Stephan Gerhold, Shawn Guo, Melody Olvera,
	Bartosz Golaszewski, Richard Acayan, Vinod Koul, krishna Lanka,
	Rohit Agarwal, Iskren Chernev, Martin Botka, Danila Tikhonov,
	Abel Vesa, Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

Drop common properties already defined in referenced common Qualcomm SoC
TLMM bindings and use "unevaluatedProperties: false".  This makes the
binding smaller and easier to review.

In few places move the "required:" block to bottom, to match convention.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../bindings/pinctrl/qcom,ipq5332-tlmm.yaml   |  9 +-----
 .../pinctrl/qcom,ipq8074-pinctrl.yaml         |  9 +-----
 .../bindings/pinctrl/qcom,ipq9574-tlmm.yaml   |  9 +-----
 .../bindings/pinctrl/qcom,mdm9607-tlmm.yaml   | 18 ++++--------
 .../pinctrl/qcom,msm8660-pinctrl.yaml         |  9 +-----
 .../bindings/pinctrl/qcom,msm8909-tlmm.yaml   | 18 ++++--------
 .../pinctrl/qcom,msm8916-pinctrl.yaml         |  9 +-----
 .../pinctrl/qcom,msm8960-pinctrl.yaml         |  9 +-----
 .../pinctrl/qcom,msm8974-pinctrl.yaml         |  9 +-----
 .../pinctrl/qcom,msm8976-pinctrl.yaml         |  9 +-----
 .../pinctrl/qcom,msm8994-pinctrl.yaml         |  9 +-----
 .../pinctrl/qcom,msm8996-pinctrl.yaml         |  9 +-----
 .../pinctrl/qcom,msm8998-pinctrl.yaml         |  9 +-----
 .../bindings/pinctrl/qcom,qcm2290-tlmm.yaml   |  9 +-----
 .../bindings/pinctrl/qcom,qcs404-pinctrl.yaml |  9 +-----
 .../bindings/pinctrl/qcom,qdu1000-tlmm.yaml   | 10 +------
 .../bindings/pinctrl/qcom,sa8775p-tlmm.yaml   | 19 ++++---------
 .../bindings/pinctrl/qcom,sc7180-pinctrl.yaml |  9 +-----
 .../bindings/pinctrl/qcom,sc7280-pinctrl.yaml | 28 +------------------
 .../bindings/pinctrl/qcom,sc8180x-tlmm.yaml   | 20 +++++--------
 .../bindings/pinctrl/qcom,sc8280xp-tlmm.yaml  | 18 ++++--------
 .../bindings/pinctrl/qcom,sdm630-pinctrl.yaml | 10 +------
 .../bindings/pinctrl/qcom,sdm670-tlmm.yaml    | 19 ++++---------
 .../bindings/pinctrl/qcom,sdm845-pinctrl.yaml | 10 +------
 .../bindings/pinctrl/qcom,sdx75-tlmm.yaml     | 10 +------
 .../bindings/pinctrl/qcom,sm6115-tlmm.yaml    |  8 +-----
 .../bindings/pinctrl/qcom,sm6125-tlmm.yaml    | 20 +++++--------
 .../bindings/pinctrl/qcom,sm6350-tlmm.yaml    | 20 ++++---------
 .../bindings/pinctrl/qcom,sm6375-tlmm.yaml    | 18 ++++--------
 .../bindings/pinctrl/qcom,sm7150-tlmm.yaml    |  9 +-----
 .../bindings/pinctrl/qcom,sm8150-pinctrl.yaml |  9 +-----
 .../bindings/pinctrl/qcom,sm8250-pinctrl.yaml |  9 +-----
 .../bindings/pinctrl/qcom,sm8350-tlmm.yaml    | 20 ++++---------
 .../bindings/pinctrl/qcom,sm8450-tlmm.yaml    | 20 ++++---------
 .../bindings/pinctrl/qcom,sm8550-tlmm.yaml    | 10 +------
 .../bindings/pinctrl/qcom,sm8650-tlmm.yaml    | 10 +------
 .../bindings/pinctrl/qcom,x1e80100-tlmm.yaml  | 10 +------
 37 files changed, 94 insertions(+), 375 deletions(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,ipq5332-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,ipq5332-tlmm.yaml
index 3d3086ae1ba6..e571cd64418f 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,ipq5332-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,ipq5332-tlmm.yaml
@@ -26,13 +26,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 27
@@ -100,7 +93,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,ipq8074-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,ipq8074-pinctrl.yaml
index e053fbd588b5..6f90dbbdbdcc 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,ipq8074-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,ipq8074-pinctrl.yaml
@@ -23,13 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 35
@@ -103,7 +96,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,ipq9574-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,ipq9574-tlmm.yaml
index e5e9962b2174..bca903b5da6d 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,ipq9574-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,ipq9574-tlmm.yaml
@@ -23,13 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 33
@@ -97,7 +90,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,mdm9607-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,mdm9607-tlmm.yaml
index 5ece3b9d676b..bd3cbb44c99a 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,mdm9607-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,mdm9607-tlmm.yaml
@@ -25,19 +25,7 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
   gpio-reserved-ranges: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
-required:
-  - compatible
-  - reg
-
-additionalProperties: false
 
 patternProperties:
   "-state$":
@@ -110,6 +98,12 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8660-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8660-pinctrl.yaml
index a05971611780..61f5be21f30c 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8660-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8660-pinctrl.yaml
@@ -23,13 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 86
@@ -92,7 +85,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8909-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8909-tlmm.yaml
index 5095e86fe9a2..295dd5fcf4c3 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8909-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8909-tlmm.yaml
@@ -25,19 +25,7 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
   gpio-reserved-ranges: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
-required:
-  - compatible
-  - reg
-
-additionalProperties: false
 
 patternProperties:
   "-state$":
@@ -108,6 +96,12 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8916-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8916-pinctrl.yaml
index 063d004967bb..904af87f9eaf 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8916-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8916-pinctrl.yaml
@@ -23,13 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 61
@@ -114,7 +107,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
index 9172b50f7a98..46618740bd31 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
@@ -23,13 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 76
@@ -108,7 +101,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8974-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8974-pinctrl.yaml
index 8a3be65c51ed..840fdaabde12 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8974-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8974-pinctrl.yaml
@@ -23,13 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 73
@@ -124,7 +117,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8976-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8976-pinctrl.yaml
index ca95de0b87a6..d4391c194ff7 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8976-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8976-pinctrl.yaml
@@ -23,13 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 73
@@ -104,7 +97,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8994-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8994-pinctrl.yaml
index 41525ecfa8e3..fa90981db40b 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8994-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8994-pinctrl.yaml
@@ -25,13 +25,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 73
@@ -114,7 +107,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8996-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8996-pinctrl.yaml
index 59d406b60957..c5010c175b23 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8996-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8996-pinctrl.yaml
@@ -23,13 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 75
@@ -133,7 +126,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8998-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8998-pinctrl.yaml
index bd6d7caf499a..bcaa231adaf7 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8998-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8998-pinctrl.yaml
@@ -23,13 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 75
@@ -118,7 +111,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,qcm2290-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,qcm2290-tlmm.yaml
index c323f6d495a4..e123beb33aef 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,qcm2290-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,qcm2290-tlmm.yaml
@@ -22,13 +22,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
 patternProperties:
   "-state$":
     oneOf:
@@ -92,7 +85,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,qcs404-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,qcs404-pinctrl.yaml
index b1b9cd319e50..4009501b3414 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,qcs404-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,qcs404-pinctrl.yaml
@@ -29,13 +29,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 60
@@ -130,7 +123,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml
index 47363c23f4ea..88afeae530c6 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml
@@ -26,10 +26,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 76
@@ -37,10 +33,6 @@ properties:
   gpio-line-names:
     maxItems: 151
 
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
 patternProperties:
   "-state$":
     oneOf:
@@ -103,7 +95,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml
index d27933258db7..e9abbf2c0689 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml
@@ -25,13 +25,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 74
@@ -39,12 +32,6 @@ properties:
   gpio-line-names:
     maxItems: 148
 
-required:
-  - compatible
-  - reg
-
-additionalProperties: false
-
 patternProperties:
   "-state$":
     oneOf:
@@ -110,6 +97,12 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sc7180-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sc7180-pinctrl.yaml
index 573e459b1c44..5606f2136ad1 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sc7180-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sc7180-pinctrl.yaml
@@ -29,13 +29,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 60
@@ -112,7 +105,7 @@ required:
   - reg
   - reg-names
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-pinctrl.yaml
index c8735ab97e40..5329fe2a4397 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-pinctrl.yaml
@@ -23,24 +23,6 @@ properties:
     description: Specifies the TLMM summary IRQ
     maxItems: 1
 
-  interrupt-controller: true
-
-  '#interrupt-cells':
-    description:
-      Specifies the PIN numbers and Flags, as defined in defined in
-      include/dt-bindings/interrupt-controller/irq.h
-    const: 2
-
-  gpio-controller: true
-
-  '#gpio-cells':
-    description: Specifying the pin number and flags, as defined in
-      include/dt-bindings/gpio/gpio.h
-    const: 2
-
-  gpio-ranges:
-    maxItems: 1
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 88
@@ -48,8 +30,6 @@ properties:
   gpio-line-names:
     maxItems: 175
 
-  wakeup-parent: true
-
 patternProperties:
   "-state$":
     oneOf:
@@ -124,14 +104,8 @@ allOf:
 required:
   - compatible
   - reg
-  - interrupts
-  - interrupt-controller
-  - '#interrupt-cells'
-  - gpio-controller
-  - '#gpio-cells'
-  - gpio-ranges
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sc8180x-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sc8180x-tlmm.yaml
index b086a5184235..c122bb849f0f 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sc8180x-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sc8180x-tlmm.yaml
@@ -31,20 +31,7 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  '#interrupt-cells': true
-  gpio-controller: true
   gpio-reserved-ranges: true
-  '#gpio-cells': true
-  gpio-ranges: true
-  wakeup-parent: true
-
-required:
-  - compatible
-  - reg
-  - reg-names
-
-additionalProperties: false
 
 patternProperties:
   "-state$":
@@ -106,6 +93,13 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+  - reg-names
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-tlmm.yaml
index 4bd6d7977d3e..ed344deaf8b9 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-tlmm.yaml
@@ -25,19 +25,7 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
   gpio-reserved-ranges: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
-required:
-  - compatible
-  - reg
-
-additionalProperties: false
 
 patternProperties:
   "-state$":
@@ -108,6 +96,12 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdm630-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdm630-pinctrl.yaml
index 508e0633b253..a00cb43df144 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sdm630-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdm630-pinctrl.yaml
@@ -34,10 +34,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 57
@@ -45,10 +41,6 @@ properties:
   gpio-line-names:
     maxItems: 114
 
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
 patternProperties:
   "-state$":
     oneOf:
@@ -130,7 +122,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdm670-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdm670-tlmm.yaml
index 84a15f77e710..b56e717aa28e 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sdm670-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdm670-tlmm.yaml
@@ -25,23 +25,10 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 75
 
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
-required:
-  - compatible
-  - reg
-
-additionalProperties: false
-
 patternProperties:
   "-state$":
     oneOf:
@@ -98,6 +85,12 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml
index d301881ddfa8..dfe5616b9b85 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml
@@ -26,10 +26,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 75
@@ -37,10 +33,6 @@ properties:
   gpio-line-names:
     maxItems: 150
 
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
 patternProperties:
   "-state$":
     oneOf:
@@ -110,7 +102,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml
index 62cd8f84af9c..cb1d978d02c9 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml
@@ -25,10 +25,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 67
@@ -36,10 +32,6 @@ properties:
   gpio-line-names:
     maxItems: 133
 
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
 patternProperties:
   "-state$":
     oneOf:
@@ -102,7 +94,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
index 871df54f69a2..7f36f9b93333 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
@@ -29,13 +29,7 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
   gpio-reserved-ranges: true
-  wakeup-parent: true
 
 patternProperties:
   "-state$":
@@ -97,7 +91,7 @@ required:
   - reg
   - reg-names
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6125-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6125-tlmm.yaml
index 8d77707b02b9..ddeaeaa9a450 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6125-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6125-tlmm.yaml
@@ -30,20 +30,7 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
   gpio-reserved-ranges: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
-required:
-  - compatible
-  - reg
-  - reg-names
-
-additionalProperties: false
 
 patternProperties:
   "-state$":
@@ -105,6 +92,13 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+  - reg-names
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6350-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6350-tlmm.yaml
index 27af379cf791..a4771f87d936 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6350-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6350-tlmm.yaml
@@ -26,10 +26,6 @@ properties:
     minItems: 9
     maxItems: 9
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 78
@@ -37,16 +33,6 @@ properties:
   gpio-line-names:
     maxItems: 156
 
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
-required:
-  - compatible
-  - reg
-
-additionalProperties: false
-
 patternProperties:
   "-state$":
     oneOf:
@@ -112,6 +98,12 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6375-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6375-tlmm.yaml
index 6e02ba24825f..047f82863f9b 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6375-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6375-tlmm.yaml
@@ -25,19 +25,7 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
   gpio-reserved-ranges: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
-required:
-  - compatible
-  - reg
-
-additionalProperties: false
 
 patternProperties:
   "-state$":
@@ -113,6 +101,12 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm7150-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm7150-tlmm.yaml
index ede0f3acad9c..7f23f939ad32 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm7150-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm7150-tlmm.yaml
@@ -32,13 +32,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 60
@@ -111,7 +104,7 @@ required:
   - reg
   - reg-names
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8150-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8150-pinctrl.yaml
index c6439626464e..bdb7ed4be026 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8150-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8150-pinctrl.yaml
@@ -30,13 +30,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 88
@@ -113,7 +106,7 @@ required:
   - reg
   - reg-names
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-pinctrl.yaml
index 021c54708524..b5d04347c064 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-pinctrl.yaml
@@ -28,13 +28,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 90
@@ -106,7 +99,7 @@ required:
   - reg
   - reg-names
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-tlmm.yaml
index 6e8f41ff0a76..ec5e09611d81 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-tlmm.yaml
@@ -25,10 +25,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 102
@@ -36,16 +32,6 @@ properties:
   gpio-line-names:
     maxItems: 203
 
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
-required:
-  - compatible
-  - reg
-
-additionalProperties: false
-
 patternProperties:
   "-state$":
     oneOf:
@@ -108,6 +94,12 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-tlmm.yaml
index 5163fe3f5365..16fd2c5e2339 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-tlmm.yaml
@@ -25,10 +25,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 105
@@ -36,16 +32,6 @@ properties:
   gpio-line-names:
     maxItems: 210
 
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
-required:
-  - compatible
-  - reg
-
-additionalProperties: false
-
 patternProperties:
   "-state$":
     oneOf:
@@ -107,6 +93,12 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml
index 567e44875c29..c2ae79df424f 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml
@@ -25,10 +25,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 105
@@ -36,10 +32,6 @@ properties:
   gpio-line-names:
     maxItems: 210
 
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
 patternProperties:
   "-state$":
     oneOf:
@@ -119,7 +111,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml
index 370789ce934b..c0a06abf851d 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml
@@ -25,10 +25,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 105
@@ -36,10 +32,6 @@ properties:
   gpio-line-names:
     maxItems: 210
 
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
 patternProperties:
   "-state$":
     oneOf:
@@ -112,7 +104,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml
index 3249c63a29cf..a1333e0743a9 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml
@@ -25,10 +25,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-
   gpio-reserved-ranges:
     minItems: 1
     maxItems: 119
@@ -36,10 +32,6 @@ properties:
   gpio-line-names:
     maxItems: 238
 
-  "#gpio-cells": true
-  gpio-ranges: true
-  wakeup-parent: true
-
 patternProperties:
   "-state$":
     oneOf:
@@ -108,7 +100,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 10/10] dt-bindings: pinctrl: qcom: drop common properties and allow wakeup-parent
  2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
                   ` (7 preceding siblings ...)
  2023-12-08 21:55 ` [PATCH 09/10] dt-bindings: pinctrl: qcom: drop common properties Krzysztof Kozlowski
@ 2023-12-08 21:55 ` Krzysztof Kozlowski
  2023-12-13 18:41   ` Rob Herring
  2023-12-13 18:40 ` [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Rob Herring
  2023-12-20 11:11 ` Linus Walleij
  10 siblings, 1 reply; 21+ messages in thread
From: Krzysztof Kozlowski @ 2023-12-08 21:55 UTC (permalink / raw)
  To: Bjorn Andersson, Andy Gross, Konrad Dybcio, Linus Walleij,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Srinivas Kandagatla, Stephan Gerhold, Shawn Guo, Melody Olvera,
	Bartosz Golaszewski, Richard Acayan, Vinod Koul, krishna Lanka,
	Rohit Agarwal, Iskren Chernev, Martin Botka, Danila Tikhonov,
	Abel Vesa, Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel
  Cc: Krzysztof Kozlowski

Drop common properties already defined in referenced common Qualcomm SoC
TLMM bindings and use "unevaluatedProperties: false".  This makes the
binding smaller and easier to review.  Additionally this allows now
"wakeup-parent" property coming from common TLMM bindings.

In few places move the "required:" block to bottom, to match convention.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../bindings/pinctrl/qcom,ipq6018-pinctrl.yaml |  8 +-------
 .../bindings/pinctrl/qcom,mdm9615-pinctrl.yaml | 18 ++++++------------
 .../bindings/pinctrl/qcom,msm8226-pinctrl.yaml |  8 +-------
 .../bindings/pinctrl/qcom,msm8953-pinctrl.yaml |  7 +------
 .../bindings/pinctrl/qcom,sdx55-pinctrl.yaml   |  8 +-------
 .../bindings/pinctrl/qcom,sdx65-tlmm.yaml      |  8 +-------
 6 files changed, 11 insertions(+), 46 deletions(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,ipq6018-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,ipq6018-pinctrl.yaml
index 7c3e5e043f07..ed00fbaec11b 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,ipq6018-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,ipq6018-pinctrl.yaml
@@ -22,12 +22,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-
 patternProperties:
   "-state$":
     oneOf:
@@ -100,7 +94,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,mdm9615-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,mdm9615-pinctrl.yaml
index 5885aee95c98..299e0b4b0ab4 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,mdm9615-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,mdm9615-pinctrl.yaml
@@ -23,18 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  '#interrupt-cells': true
-  gpio-controller: true
-  '#gpio-cells': true
-  gpio-ranges: true
-
-required:
-  - compatible
-  - reg
-
-additionalProperties: false
-
 patternProperties:
   "-state$":
     oneOf:
@@ -74,6 +62,12 @@ $defs:
     required:
       - pins
 
+required:
+  - compatible
+  - reg
+
+unevaluatedProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8226-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8226-pinctrl.yaml
index a602bf0d27fb..68d3fa2105b8 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8226-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8226-pinctrl.yaml
@@ -23,12 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-
   gpio-reserved-ranges:
     maxItems: 1
 
@@ -82,7 +76,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8953-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8953-pinctrl.yaml
index 798aac9e6e31..8a3a962f6c00 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8953-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8953-pinctrl.yaml
@@ -22,12 +22,7 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
   gpio-reserved-ranges: true
-  "#gpio-cells": true
-  gpio-ranges: true
 
 patternProperties:
   "-state$":
@@ -117,7 +112,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdx55-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdx55-pinctrl.yaml
index 67af99dd8f14..edbcff92bbf9 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sdx55-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdx55-pinctrl.yaml
@@ -23,12 +23,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-
   gpio-reserved-ranges:
     maxItems: 1
 
@@ -102,7 +96,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdx65-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdx65-tlmm.yaml
index 27319782d94b..a31b638c456d 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,sdx65-tlmm.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdx65-tlmm.yaml
@@ -22,12 +22,6 @@ properties:
   interrupts:
     maxItems: 1
 
-  interrupt-controller: true
-  "#interrupt-cells": true
-  gpio-controller: true
-  "#gpio-cells": true
-  gpio-ranges: true
-
   gpio-reserved-ranges:
     maxItems: 1
 
@@ -122,7 +116,7 @@ required:
   - compatible
   - reg
 
-additionalProperties: false
+unevaluatedProperties: false
 
 examples:
   - |
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* Re: [PATCH 09/10] dt-bindings: pinctrl: qcom: drop common properties
  2023-12-08 21:55 ` [PATCH 09/10] dt-bindings: pinctrl: qcom: drop common properties Krzysztof Kozlowski
@ 2023-12-11 23:22   ` Richard Acayan
  0 siblings, 0 replies; 21+ messages in thread
From: Richard Acayan @ 2023-12-11 23:22 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Bjorn Andersson, Andy Gross, Konrad Dybcio, Linus Walleij,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Srinivas Kandagatla, Stephan Gerhold, Shawn Guo, Melody Olvera,
	Bartosz Golaszewski, Vinod Koul, krishna Lanka, Rohit Agarwal,
	Iskren Chernev, Martin Botka, Danila Tikhonov, Abel Vesa,
	Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel

On Fri, Dec 08, 2023 at 10:55:33PM +0100, Krzysztof Kozlowski wrote:
> Drop common properties already defined in referenced common Qualcomm SoC
> TLMM bindings and use "unevaluatedProperties: false".  This makes the
> binding smaller and easier to review.
> 
> In few places move the "required:" block to bottom, to match convention.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---

Nice, acked for the SDM670 part.

>  .../bindings/pinctrl/qcom,ipq5332-tlmm.yaml   |  9 +-----
>  .../pinctrl/qcom,ipq8074-pinctrl.yaml         |  9 +-----
>  .../bindings/pinctrl/qcom,ipq9574-tlmm.yaml   |  9 +-----
>  .../bindings/pinctrl/qcom,mdm9607-tlmm.yaml   | 18 ++++--------
>  .../pinctrl/qcom,msm8660-pinctrl.yaml         |  9 +-----
>  .../bindings/pinctrl/qcom,msm8909-tlmm.yaml   | 18 ++++--------
>  .../pinctrl/qcom,msm8916-pinctrl.yaml         |  9 +-----
>  .../pinctrl/qcom,msm8960-pinctrl.yaml         |  9 +-----
>  .../pinctrl/qcom,msm8974-pinctrl.yaml         |  9 +-----
>  .../pinctrl/qcom,msm8976-pinctrl.yaml         |  9 +-----
>  .../pinctrl/qcom,msm8994-pinctrl.yaml         |  9 +-----
>  .../pinctrl/qcom,msm8996-pinctrl.yaml         |  9 +-----
>  .../pinctrl/qcom,msm8998-pinctrl.yaml         |  9 +-----
>  .../bindings/pinctrl/qcom,qcm2290-tlmm.yaml   |  9 +-----
>  .../bindings/pinctrl/qcom,qcs404-pinctrl.yaml |  9 +-----
>  .../bindings/pinctrl/qcom,qdu1000-tlmm.yaml   | 10 +------
>  .../bindings/pinctrl/qcom,sa8775p-tlmm.yaml   | 19 ++++---------
>  .../bindings/pinctrl/qcom,sc7180-pinctrl.yaml |  9 +-----
>  .../bindings/pinctrl/qcom,sc7280-pinctrl.yaml | 28 +------------------
>  .../bindings/pinctrl/qcom,sc8180x-tlmm.yaml   | 20 +++++--------
>  .../bindings/pinctrl/qcom,sc8280xp-tlmm.yaml  | 18 ++++--------
>  .../bindings/pinctrl/qcom,sdm630-pinctrl.yaml | 10 +------
>  .../bindings/pinctrl/qcom,sdm670-tlmm.yaml    | 19 ++++---------
>  .../bindings/pinctrl/qcom,sdm845-pinctrl.yaml | 10 +------
>  .../bindings/pinctrl/qcom,sdx75-tlmm.yaml     | 10 +------
>  .../bindings/pinctrl/qcom,sm6115-tlmm.yaml    |  8 +-----
>  .../bindings/pinctrl/qcom,sm6125-tlmm.yaml    | 20 +++++--------
>  .../bindings/pinctrl/qcom,sm6350-tlmm.yaml    | 20 ++++---------
>  .../bindings/pinctrl/qcom,sm6375-tlmm.yaml    | 18 ++++--------
>  .../bindings/pinctrl/qcom,sm7150-tlmm.yaml    |  9 +-----
>  .../bindings/pinctrl/qcom,sm8150-pinctrl.yaml |  9 +-----
>  .../bindings/pinctrl/qcom,sm8250-pinctrl.yaml |  9 +-----
>  .../bindings/pinctrl/qcom,sm8350-tlmm.yaml    | 20 ++++---------
>  .../bindings/pinctrl/qcom,sm8450-tlmm.yaml    | 20 ++++---------
>  .../bindings/pinctrl/qcom,sm8550-tlmm.yaml    | 10 +------
>  .../bindings/pinctrl/qcom,sm8650-tlmm.yaml    | 10 +------
>  .../bindings/pinctrl/qcom,x1e80100-tlmm.yaml  | 10 +------
>  37 files changed, 94 insertions(+), 375 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,ipq5332-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,ipq5332-tlmm.yaml
> index 3d3086ae1ba6..e571cd64418f 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,ipq5332-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,ipq5332-tlmm.yaml
> @@ -26,13 +26,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 27
> @@ -100,7 +93,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,ipq8074-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,ipq8074-pinctrl.yaml
> index e053fbd588b5..6f90dbbdbdcc 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,ipq8074-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,ipq8074-pinctrl.yaml
> @@ -23,13 +23,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 35
> @@ -103,7 +96,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,ipq9574-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,ipq9574-tlmm.yaml
> index e5e9962b2174..bca903b5da6d 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,ipq9574-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,ipq9574-tlmm.yaml
> @@ -23,13 +23,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 33
> @@ -97,7 +90,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,mdm9607-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,mdm9607-tlmm.yaml
> index 5ece3b9d676b..bd3cbb44c99a 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,mdm9607-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,mdm9607-tlmm.yaml
> @@ -25,19 +25,7 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
>    gpio-reserved-ranges: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
> -required:
> -  - compatible
> -  - reg
> -
> -additionalProperties: false
>  
>  patternProperties:
>    "-state$":
> @@ -110,6 +98,12 @@ $defs:
>      required:
>        - pins
>  
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8660-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8660-pinctrl.yaml
> index a05971611780..61f5be21f30c 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8660-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8660-pinctrl.yaml
> @@ -23,13 +23,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 86
> @@ -92,7 +85,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8909-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8909-tlmm.yaml
> index 5095e86fe9a2..295dd5fcf4c3 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8909-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8909-tlmm.yaml
> @@ -25,19 +25,7 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
>    gpio-reserved-ranges: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
> -required:
> -  - compatible
> -  - reg
> -
> -additionalProperties: false
>  
>  patternProperties:
>    "-state$":
> @@ -108,6 +96,12 @@ $defs:
>      required:
>        - pins
>  
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8916-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8916-pinctrl.yaml
> index 063d004967bb..904af87f9eaf 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8916-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8916-pinctrl.yaml
> @@ -23,13 +23,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 61
> @@ -114,7 +107,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
> index 9172b50f7a98..46618740bd31 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.yaml
> @@ -23,13 +23,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 76
> @@ -108,7 +101,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8974-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8974-pinctrl.yaml
> index 8a3be65c51ed..840fdaabde12 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8974-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8974-pinctrl.yaml
> @@ -23,13 +23,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 73
> @@ -124,7 +117,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8976-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8976-pinctrl.yaml
> index ca95de0b87a6..d4391c194ff7 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8976-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8976-pinctrl.yaml
> @@ -23,13 +23,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 73
> @@ -104,7 +97,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8994-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8994-pinctrl.yaml
> index 41525ecfa8e3..fa90981db40b 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8994-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8994-pinctrl.yaml
> @@ -25,13 +25,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 73
> @@ -114,7 +107,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8996-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8996-pinctrl.yaml
> index 59d406b60957..c5010c175b23 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8996-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8996-pinctrl.yaml
> @@ -23,13 +23,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 75
> @@ -133,7 +126,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,msm8998-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,msm8998-pinctrl.yaml
> index bd6d7caf499a..bcaa231adaf7 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,msm8998-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,msm8998-pinctrl.yaml
> @@ -23,13 +23,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 75
> @@ -118,7 +111,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,qcm2290-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,qcm2290-tlmm.yaml
> index c323f6d495a4..e123beb33aef 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,qcm2290-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,qcm2290-tlmm.yaml
> @@ -22,13 +22,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -92,7 +85,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,qcs404-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,qcs404-pinctrl.yaml
> index b1b9cd319e50..4009501b3414 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,qcs404-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,qcs404-pinctrl.yaml
> @@ -29,13 +29,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 60
> @@ -130,7 +123,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml
> index 47363c23f4ea..88afeae530c6 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml
> @@ -26,10 +26,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 76
> @@ -37,10 +33,6 @@ properties:
>    gpio-line-names:
>      maxItems: 151
>  
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -103,7 +95,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml
> index d27933258db7..e9abbf2c0689 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml
> @@ -25,13 +25,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 74
> @@ -39,12 +32,6 @@ properties:
>    gpio-line-names:
>      maxItems: 148
>  
> -required:
> -  - compatible
> -  - reg
> -
> -additionalProperties: false
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -110,6 +97,12 @@ $defs:
>      required:
>        - pins
>  
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sc7180-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sc7180-pinctrl.yaml
> index 573e459b1c44..5606f2136ad1 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sc7180-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sc7180-pinctrl.yaml
> @@ -29,13 +29,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 60
> @@ -112,7 +105,7 @@ required:
>    - reg
>    - reg-names
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-pinctrl.yaml
> index c8735ab97e40..5329fe2a4397 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sc7280-pinctrl.yaml
> @@ -23,24 +23,6 @@ properties:
>      description: Specifies the TLMM summary IRQ
>      maxItems: 1
>  
> -  interrupt-controller: true
> -
> -  '#interrupt-cells':
> -    description:
> -      Specifies the PIN numbers and Flags, as defined in defined in
> -      include/dt-bindings/interrupt-controller/irq.h
> -    const: 2
> -
> -  gpio-controller: true
> -
> -  '#gpio-cells':
> -    description: Specifying the pin number and flags, as defined in
> -      include/dt-bindings/gpio/gpio.h
> -    const: 2
> -
> -  gpio-ranges:
> -    maxItems: 1
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 88
> @@ -48,8 +30,6 @@ properties:
>    gpio-line-names:
>      maxItems: 175
>  
> -  wakeup-parent: true
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -124,14 +104,8 @@ allOf:
>  required:
>    - compatible
>    - reg
> -  - interrupts
> -  - interrupt-controller
> -  - '#interrupt-cells'
> -  - gpio-controller
> -  - '#gpio-cells'
> -  - gpio-ranges
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sc8180x-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sc8180x-tlmm.yaml
> index b086a5184235..c122bb849f0f 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sc8180x-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sc8180x-tlmm.yaml
> @@ -31,20 +31,7 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  '#interrupt-cells': true
> -  gpio-controller: true
>    gpio-reserved-ranges: true
> -  '#gpio-cells': true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
> -required:
> -  - compatible
> -  - reg
> -  - reg-names
> -
> -additionalProperties: false
>  
>  patternProperties:
>    "-state$":
> @@ -106,6 +93,13 @@ $defs:
>      required:
>        - pins
>  
> +required:
> +  - compatible
> +  - reg
> +  - reg-names
> +
> +unevaluatedProperties: false
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-tlmm.yaml
> index 4bd6d7977d3e..ed344deaf8b9 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sc8280xp-tlmm.yaml
> @@ -25,19 +25,7 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
>    gpio-reserved-ranges: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
> -required:
> -  - compatible
> -  - reg
> -
> -additionalProperties: false
>  
>  patternProperties:
>    "-state$":
> @@ -108,6 +96,12 @@ $defs:
>      required:
>        - pins
>  
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdm630-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdm630-pinctrl.yaml
> index 508e0633b253..a00cb43df144 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sdm630-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdm630-pinctrl.yaml
> @@ -34,10 +34,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 57
> @@ -45,10 +41,6 @@ properties:
>    gpio-line-names:
>      maxItems: 114
>  
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -130,7 +122,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdm670-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdm670-tlmm.yaml
> index 84a15f77e710..b56e717aa28e 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sdm670-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdm670-tlmm.yaml
> @@ -25,23 +25,10 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 75
>  
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
> -required:
> -  - compatible
> -  - reg
> -
> -additionalProperties: false
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -98,6 +85,12 @@ $defs:
>      required:
>        - pins
>  
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml
> index d301881ddfa8..dfe5616b9b85 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdm845-pinctrl.yaml
> @@ -26,10 +26,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 75
> @@ -37,10 +33,6 @@ properties:
>    gpio-line-names:
>      maxItems: 150
>  
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -110,7 +102,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml
> index 62cd8f84af9c..cb1d978d02c9 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml
> @@ -25,10 +25,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 67
> @@ -36,10 +32,6 @@ properties:
>    gpio-line-names:
>      maxItems: 133
>  
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -102,7 +94,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
> index 871df54f69a2..7f36f9b93333 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6115-tlmm.yaml
> @@ -29,13 +29,7 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
>    gpio-reserved-ranges: true
> -  wakeup-parent: true
>  
>  patternProperties:
>    "-state$":
> @@ -97,7 +91,7 @@ required:
>    - reg
>    - reg-names
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6125-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6125-tlmm.yaml
> index 8d77707b02b9..ddeaeaa9a450 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6125-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6125-tlmm.yaml
> @@ -30,20 +30,7 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
>    gpio-reserved-ranges: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
> -required:
> -  - compatible
> -  - reg
> -  - reg-names
> -
> -additionalProperties: false
>  
>  patternProperties:
>    "-state$":
> @@ -105,6 +92,13 @@ $defs:
>      required:
>        - pins
>  
> +required:
> +  - compatible
> +  - reg
> +  - reg-names
> +
> +unevaluatedProperties: false
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6350-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6350-tlmm.yaml
> index 27af379cf791..a4771f87d936 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6350-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6350-tlmm.yaml
> @@ -26,10 +26,6 @@ properties:
>      minItems: 9
>      maxItems: 9
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 78
> @@ -37,16 +33,6 @@ properties:
>    gpio-line-names:
>      maxItems: 156
>  
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
> -required:
> -  - compatible
> -  - reg
> -
> -additionalProperties: false
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -112,6 +98,12 @@ $defs:
>      required:
>        - pins
>  
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm6375-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm6375-tlmm.yaml
> index 6e02ba24825f..047f82863f9b 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm6375-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm6375-tlmm.yaml
> @@ -25,19 +25,7 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
>    gpio-reserved-ranges: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
> -required:
> -  - compatible
> -  - reg
> -
> -additionalProperties: false
>  
>  patternProperties:
>    "-state$":
> @@ -113,6 +101,12 @@ $defs:
>      required:
>        - pins
>  
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm7150-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm7150-tlmm.yaml
> index ede0f3acad9c..7f23f939ad32 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm7150-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm7150-tlmm.yaml
> @@ -32,13 +32,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 60
> @@ -111,7 +104,7 @@ required:
>    - reg
>    - reg-names
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8150-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8150-pinctrl.yaml
> index c6439626464e..bdb7ed4be026 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8150-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8150-pinctrl.yaml
> @@ -30,13 +30,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 88
> @@ -113,7 +106,7 @@ required:
>    - reg
>    - reg-names
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-pinctrl.yaml
> index 021c54708524..b5d04347c064 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8250-pinctrl.yaml
> @@ -28,13 +28,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 90
> @@ -106,7 +99,7 @@ required:
>    - reg
>    - reg-names
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-tlmm.yaml
> index 6e8f41ff0a76..ec5e09611d81 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8350-tlmm.yaml
> @@ -25,10 +25,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 102
> @@ -36,16 +32,6 @@ properties:
>    gpio-line-names:
>      maxItems: 203
>  
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
> -required:
> -  - compatible
> -  - reg
> -
> -additionalProperties: false
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -108,6 +94,12 @@ $defs:
>      required:
>        - pins
>  
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-tlmm.yaml
> index 5163fe3f5365..16fd2c5e2339 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8450-tlmm.yaml
> @@ -25,10 +25,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 105
> @@ -36,16 +32,6 @@ properties:
>    gpio-line-names:
>      maxItems: 210
>  
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
> -required:
> -  - compatible
> -  - reg
> -
> -additionalProperties: false
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -107,6 +93,12 @@ $defs:
>      required:
>        - pins
>  
> +required:
> +  - compatible
> +  - reg
> +
> +unevaluatedProperties: false
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/arm-gic.h>
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml
> index 567e44875c29..c2ae79df424f 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml
> @@ -25,10 +25,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 105
> @@ -36,10 +32,6 @@ properties:
>    gpio-line-names:
>      maxItems: 210
>  
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -119,7 +111,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml
> index 370789ce934b..c0a06abf851d 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml
> @@ -25,10 +25,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 105
> @@ -36,10 +32,6 @@ properties:
>    gpio-line-names:
>      maxItems: 210
>  
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -112,7 +104,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml
> index 3249c63a29cf..a1333e0743a9 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml
> @@ -25,10 +25,6 @@ properties:
>    interrupts:
>      maxItems: 1
>  
> -  interrupt-controller: true
> -  "#interrupt-cells": true
> -  gpio-controller: true
> -
>    gpio-reserved-ranges:
>      minItems: 1
>      maxItems: 119
> @@ -36,10 +32,6 @@ properties:
>    gpio-line-names:
>      maxItems: 238
>  
> -  "#gpio-cells": true
> -  gpio-ranges: true
> -  wakeup-parent: true
> -
>  patternProperties:
>    "-state$":
>      oneOf:
> @@ -108,7 +100,7 @@ required:
>    - compatible
>    - reg
>  
> -additionalProperties: false
> +unevaluatedProperties: false
>  
>  examples:
>    - |
> -- 
> 2.34.1
> 

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema
  2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
                   ` (8 preceding siblings ...)
  2023-12-08 21:55 ` [PATCH 10/10] dt-bindings: pinctrl: qcom: drop common properties and allow wakeup-parent Krzysztof Kozlowski
@ 2023-12-13 18:40 ` Rob Herring
  2023-12-20 11:11 ` Linus Walleij
  10 siblings, 0 replies; 21+ messages in thread
From: Rob Herring @ 2023-12-13 18:40 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Abel Vesa, Iskren Chernev, Shawn Guo, Rajendra Nayak,
	Srinivas Kandagatla, devicetree, Danila Tikhonov, linux-gpio,
	Konrad Dybcio, Rohit Agarwal, Vinod Koul, linux-kernel,
	Richard Acayan, Melody Olvera, Krzysztof Kozlowski,
	linux-arm-msm, Linus Walleij, Bjorn Andersson, Rob Herring,
	Martin Botka, Bartosz Golaszewski, krishna Lanka,
	Stephan Gerhold, Andy Gross, Conor Dooley


On Fri, 08 Dec 2023 22:55:25 +0100, Krzysztof Kozlowski wrote:
> Just like regular TLMM pin controllers in Qualcomm SoCs, the Low Power
> Audio SubSystem (LPASS) Low Power Island (LPI) TLMM blocks share a lot
> of properties, so common part can be moved to separate schema to reduce
> code duplication and make reviewing easier.
> 
> Except the move of common part, this introduces effective changes:
> 1. To all LPASS LPI bindings: Reference pinmux-node.yaml in each pin
>    muxing and configuration node, to bring definition of "function" and
>    "pins" properties.
> 
> 2. qcom,sc7280-lpass-lpi-pinctrl: Reference pinctrl.yaml in top leve.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../pinctrl/qcom,lpass-lpi-common.yaml        | 75 +++++++++++++++++++
>  .../qcom,sc7280-lpass-lpi-pinctrl.yaml        | 49 ++----------
>  .../qcom,sc8280xp-lpass-lpi-pinctrl.yaml      | 49 +-----------
>  .../qcom,sm6115-lpass-lpi-pinctrl.yaml        | 48 +-----------
>  .../qcom,sm8250-lpass-lpi-pinctrl.yaml        | 49 +-----------
>  .../qcom,sm8350-lpass-lpi-pinctrl.yaml        | 49 +-----------
>  .../qcom,sm8450-lpass-lpi-pinctrl.yaml        | 49 +-----------
>  .../qcom,sm8550-lpass-lpi-pinctrl.yaml        | 49 +-----------
>  .../qcom,sm8650-lpass-lpi-pinctrl.yaml        | 49 +-----------
>  9 files changed, 109 insertions(+), 357 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/pinctrl/qcom,lpass-lpi-common.yaml
> 

Reviewed-by: Rob Herring <robh@kernel.org>


^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 03/10] dt-bindings: pinctrl: qcom,sa8775p-tlmm: restrict number of interrupts
  2023-12-08 21:55 ` [PATCH 03/10] dt-bindings: pinctrl: qcom,sa8775p-tlmm: " Krzysztof Kozlowski
@ 2023-12-13 18:40   ` Rob Herring
  0 siblings, 0 replies; 21+ messages in thread
From: Rob Herring @ 2023-12-13 18:40 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Abel Vesa, Martin Botka, Rajendra Nayak, Rob Herring,
	Danila Tikhonov, Bartosz Golaszewski, Rohit Agarwal,
	Stephan Gerhold, linux-kernel, krishna Lanka, Vinod Koul,
	Andy Gross, Iskren Chernev, Srinivas Kandagatla, Richard Acayan,
	linux-arm-msm, linux-gpio, devicetree, Konrad Dybcio,
	Krzysztof Kozlowski, Linus Walleij, Melody Olvera, Conor Dooley,
	Shawn Guo, Bjorn Andersson


On Fri, 08 Dec 2023 22:55:27 +0100, Krzysztof Kozlowski wrote:
> SA8775p TLMM pin controller comes with only one interrupt, so narrow
> the number of interrupts previously defined in common TLMM bindings.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml        | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>


^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 04/10] dt-bindings: pinctrl: qcom,sdx75-tlmm: restrict number of interrupts
  2023-12-08 21:55 ` [PATCH 04/10] dt-bindings: pinctrl: qcom,sdx75-tlmm: " Krzysztof Kozlowski
@ 2023-12-13 18:40   ` Rob Herring
  0 siblings, 0 replies; 21+ messages in thread
From: Rob Herring @ 2023-12-13 18:40 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Conor Dooley, Srinivas Kandagatla, Rob Herring, Rohit Agarwal,
	Vinod Koul, Andy Gross, Linus Walleij, Iskren Chernev,
	Bartosz Golaszewski, Shawn Guo, devicetree, Melody Olvera,
	Stephan Gerhold, Krzysztof Kozlowski, Richard Acayan,
	krishna Lanka, Konrad Dybcio, linux-kernel, Danila Tikhonov,
	Martin Botka, Rajendra Nayak, linux-arm-msm, Bjorn Andersson,
	linux-gpio, Abel Vesa


On Fri, 08 Dec 2023 22:55:28 +0100, Krzysztof Kozlowski wrote:
> SDX75 TLMM pin controller comes with only one interrupt, so narrow
> the number of interrupts previously defined in common TLMM bindings.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml          | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>


^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 05/10] dt-bindings: pinctrl: qcom,sm8550-tlmm: restrict number of interrupts
  2023-12-08 21:55 ` [PATCH 05/10] dt-bindings: pinctrl: qcom,sm8550-tlmm: " Krzysztof Kozlowski
@ 2023-12-13 18:40   ` Rob Herring
  0 siblings, 0 replies; 21+ messages in thread
From: Rob Herring @ 2023-12-13 18:40 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Andy Gross, Rob Herring, Abel Vesa, Bartosz Golaszewski,
	devicetree, Shawn Guo, linux-gpio, Bjorn Andersson,
	Stephan Gerhold, Rajendra Nayak, Danila Tikhonov, linux-kernel,
	Krzysztof Kozlowski, Richard Acayan, Rohit Agarwal,
	linux-arm-msm, Conor Dooley, Konrad Dybcio, Iskren Chernev,
	Melody Olvera, Martin Botka, Srinivas Kandagatla, krishna Lanka,
	Vinod Koul, Linus Walleij


On Fri, 08 Dec 2023 22:55:29 +0100, Krzysztof Kozlowski wrote:
> SM8550 TLMM pin controller comes with only one interrupt, so narrow
> the number of interrupts previously defined in common TLMM bindings.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml         | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>


^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 06/10] dt-bindings: pinctrl: qcom,sm8650-tlmm: restrict number of interrupts
  2023-12-08 21:55 ` [PATCH 06/10] dt-bindings: pinctrl: qcom,sm8650-tlmm: " Krzysztof Kozlowski
@ 2023-12-13 18:40   ` Rob Herring
  0 siblings, 0 replies; 21+ messages in thread
From: Rob Herring @ 2023-12-13 18:40 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Andy Gross, Shawn Guo, Konrad Dybcio, Rob Herring,
	Stephan Gerhold, linux-kernel, Rajendra Nayak, linux-arm-msm,
	Rohit Agarwal, Vinod Koul, Richard Acayan, Abel Vesa,
	Krzysztof Kozlowski, Bjorn Andersson, krishna Lanka, devicetree,
	Danila Tikhonov, Bartosz Golaszewski, Srinivas Kandagatla,
	Linus Walleij, Conor Dooley, Martin Botka, Melody Olvera,
	Iskren Chernev, linux-gpio


On Fri, 08 Dec 2023 22:55:30 +0100, Krzysztof Kozlowski wrote:
> SM8650 TLMM pin controller comes with only one interrupt, so narrow
> the number of interrupts previously defined in common TLMM bindings.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml         | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>


^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 07/10] dt-bindings: pinctrl: qcom,x1e80100-tlmm: restrict number of interrupts
  2023-12-08 21:55 ` [PATCH 07/10] dt-bindings: pinctrl: qcom,x1e80100-tlmm: " Krzysztof Kozlowski
@ 2023-12-13 18:40   ` Rob Herring
  0 siblings, 0 replies; 21+ messages in thread
From: Rob Herring @ 2023-12-13 18:40 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Krzysztof Kozlowski, devicetree, Shawn Guo, Rajendra Nayak,
	Andy Gross, Srinivas Kandagatla, Martin Botka, Vinod Koul,
	Rohit Agarwal, Bjorn Andersson, linux-gpio, Danila Tikhonov,
	Abel Vesa, Melody Olvera, Stephan Gerhold, Rob Herring,
	linux-arm-msm, Konrad Dybcio, Linus Walleij, krishna Lanka,
	Iskren Chernev, Richard Acayan, Conor Dooley, linux-kernel,
	Bartosz Golaszewski


On Fri, 08 Dec 2023 22:55:31 +0100, Krzysztof Kozlowski wrote:
> X1E80100 TLMM pin controller comes with only one interrupt, so narrow
> the number of interrupts previously defined in common TLMM bindings.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml       | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>


^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 08/10] dt-bindings: pinctrl: qcom,ipq5018-tlmm: use common TLMM bindings
  2023-12-08 21:55 ` [PATCH 08/10] dt-bindings: pinctrl: qcom,ipq5018-tlmm: use common TLMM bindings Krzysztof Kozlowski
@ 2023-12-13 18:41   ` Rob Herring
  0 siblings, 0 replies; 21+ messages in thread
From: Rob Herring @ 2023-12-13 18:41 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Srinivas Kandagatla, Andy Gross, krishna Lanka, Rob Herring,
	devicetree, Danila Tikhonov, linux-arm-msm, Richard Acayan,
	linux-gpio, linux-kernel, Rajendra Nayak, Abel Vesa,
	Bartosz Golaszewski, Konrad Dybcio, Bjorn Andersson,
	Stephan Gerhold, Melody Olvera, Rohit Agarwal, Shawn Guo,
	Conor Dooley, Vinod Koul, Martin Botka, Krzysztof Kozlowski,
	Iskren Chernev, Linus Walleij


On Fri, 08 Dec 2023 22:55:32 +0100, Krzysztof Kozlowski wrote:
> Reference common Qualcomm SoC TLMM bindings to drop commonly used
> properties and also bring other schemas for common definitions.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../bindings/pinctrl/qcom,ipq5018-tlmm.yaml          | 12 ++++--------
>  1 file changed, 4 insertions(+), 8 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>


^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 10/10] dt-bindings: pinctrl: qcom: drop common properties and allow wakeup-parent
  2023-12-08 21:55 ` [PATCH 10/10] dt-bindings: pinctrl: qcom: drop common properties and allow wakeup-parent Krzysztof Kozlowski
@ 2023-12-13 18:41   ` Rob Herring
  0 siblings, 0 replies; 21+ messages in thread
From: Rob Herring @ 2023-12-13 18:41 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Linus Walleij, Abel Vesa, Melody Olvera, Iskren Chernev,
	Danila Tikhonov, krishna Lanka, Rob Herring, Srinivas Kandagatla,
	Konrad Dybcio, Bjorn Andersson, Martin Botka, linux-kernel,
	Shawn Guo, Rajendra Nayak, linux-gpio, Stephan Gerhold,
	devicetree, Vinod Koul, Conor Dooley, Richard Acayan, Andy Gross,
	Krzysztof Kozlowski, Rohit Agarwal, Bartosz Golaszewski,
	linux-arm-msm


On Fri, 08 Dec 2023 22:55:34 +0100, Krzysztof Kozlowski wrote:
> Drop common properties already defined in referenced common Qualcomm SoC
> TLMM bindings and use "unevaluatedProperties: false".  This makes the
> binding smaller and easier to review.  Additionally this allows now
> "wakeup-parent" property coming from common TLMM bindings.
> 
> In few places move the "required:" block to bottom, to match convention.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../bindings/pinctrl/qcom,ipq6018-pinctrl.yaml |  8 +-------
>  .../bindings/pinctrl/qcom,mdm9615-pinctrl.yaml | 18 ++++++------------
>  .../bindings/pinctrl/qcom,msm8226-pinctrl.yaml |  8 +-------
>  .../bindings/pinctrl/qcom,msm8953-pinctrl.yaml |  7 +------
>  .../bindings/pinctrl/qcom,sdx55-pinctrl.yaml   |  8 +-------
>  .../bindings/pinctrl/qcom,sdx65-tlmm.yaml      |  8 +-------
>  6 files changed, 11 insertions(+), 46 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>


^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 02/10] dt-bindings: pinctrl: qcom,qdu1000-tlmm: restrict number of interrupts
  2023-12-08 21:55 ` [PATCH 02/10] dt-bindings: pinctrl: qcom,qdu1000-tlmm: restrict number of interrupts Krzysztof Kozlowski
@ 2023-12-13 19:01   ` Rob Herring
  0 siblings, 0 replies; 21+ messages in thread
From: Rob Herring @ 2023-12-13 19:01 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Richard Acayan, Rob Herring, Stephan Gerhold, Melody Olvera,
	linux-gpio, Vinod Koul, Andy Gross, Iskren Chernev,
	linux-arm-msm, Abel Vesa, Krzysztof Kozlowski, Linus Walleij,
	Rohit Agarwal, Bjorn Andersson, Martin Botka, Danila Tikhonov,
	devicetree, Srinivas Kandagatla, Conor Dooley, linux-kernel,
	Bartosz Golaszewski, Rajendra Nayak, Konrad Dybcio, Shawn Guo,
	krishna Lanka


On Fri, 08 Dec 2023 22:55:26 +0100, Krzysztof Kozlowski wrote:
> QDU1000 TLMM pin controller comes with only one interrupt, so narrow
> the number of interrupts previously defined in common TLMM bindings.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml        | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>


^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema
  2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
                   ` (9 preceding siblings ...)
  2023-12-13 18:40 ` [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Rob Herring
@ 2023-12-20 11:11 ` Linus Walleij
  10 siblings, 0 replies; 21+ messages in thread
From: Linus Walleij @ 2023-12-20 11:11 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Bjorn Andersson, Andy Gross, Konrad Dybcio, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Srinivas Kandagatla,
	Stephan Gerhold, Shawn Guo, Melody Olvera, Bartosz Golaszewski,
	Richard Acayan, Vinod Koul, krishna Lanka, Rohit Agarwal,
	Iskren Chernev, Martin Botka, Danila Tikhonov, Abel Vesa,
	Rajendra Nayak, linux-arm-msm, linux-gpio, devicetree,
	linux-kernel

On Fri, Dec 8, 2023 at 10:55 PM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:

> Just like regular TLMM pin controllers in Qualcomm SoCs, the Low Power
> Audio SubSystem (LPASS) Low Power Island (LPI) TLMM blocks share a lot
> of properties, so common part can be moved to separate schema to reduce
> code duplication and make reviewing easier.
>
> Except the move of common part, this introduces effective changes:
> 1. To all LPASS LPI bindings: Reference pinmux-node.yaml in each pin
>    muxing and configuration node, to bring definition of "function" and
>    "pins" properties.
>
> 2. qcom,sc7280-lpass-lpi-pinctrl: Reference pinctrl.yaml in top leve.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Patch 9/10 was missing from my inbox (and presumably Rob's too since
he didn't ACK it) but there is absolutely nothing wrong with it so I just
grabbed the whole series with b4 and applied it.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 21+ messages in thread

end of thread, other threads:[~2023-12-20 11:11 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-08 21:55 [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Krzysztof Kozlowski
2023-12-08 21:55 ` [PATCH 02/10] dt-bindings: pinctrl: qcom,qdu1000-tlmm: restrict number of interrupts Krzysztof Kozlowski
2023-12-13 19:01   ` Rob Herring
2023-12-08 21:55 ` [PATCH 03/10] dt-bindings: pinctrl: qcom,sa8775p-tlmm: " Krzysztof Kozlowski
2023-12-13 18:40   ` Rob Herring
2023-12-08 21:55 ` [PATCH 04/10] dt-bindings: pinctrl: qcom,sdx75-tlmm: " Krzysztof Kozlowski
2023-12-13 18:40   ` Rob Herring
2023-12-08 21:55 ` [PATCH 05/10] dt-bindings: pinctrl: qcom,sm8550-tlmm: " Krzysztof Kozlowski
2023-12-13 18:40   ` Rob Herring
2023-12-08 21:55 ` [PATCH 06/10] dt-bindings: pinctrl: qcom,sm8650-tlmm: " Krzysztof Kozlowski
2023-12-13 18:40   ` Rob Herring
2023-12-08 21:55 ` [PATCH 07/10] dt-bindings: pinctrl: qcom,x1e80100-tlmm: " Krzysztof Kozlowski
2023-12-13 18:40   ` Rob Herring
2023-12-08 21:55 ` [PATCH 08/10] dt-bindings: pinctrl: qcom,ipq5018-tlmm: use common TLMM bindings Krzysztof Kozlowski
2023-12-13 18:41   ` Rob Herring
2023-12-08 21:55 ` [PATCH 09/10] dt-bindings: pinctrl: qcom: drop common properties Krzysztof Kozlowski
2023-12-11 23:22   ` Richard Acayan
2023-12-08 21:55 ` [PATCH 10/10] dt-bindings: pinctrl: qcom: drop common properties and allow wakeup-parent Krzysztof Kozlowski
2023-12-13 18:41   ` Rob Herring
2023-12-13 18:40 ` [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema Rob Herring
2023-12-20 11:11 ` Linus Walleij

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.