2023-12-08 21:55:55

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema

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 <[email protected]>
---
.../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 <[email protected]>
+ - Srinivas Kandagatla <[email protected]>
+ - Krzysztof Kozlowski <[email protected]>
+
+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


2023-12-08 21:55:59

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 02/10] dt-bindings: pinctrl: qcom,qdu1000-tlmm: restrict number of interrupts

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 <[email protected]>
---
.../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

2023-12-08 21:56:06

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 03/10] dt-bindings: pinctrl: qcom,sa8775p-tlmm: restrict number of interrupts

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 <[email protected]>
---
.../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

2023-12-08 21:56:16

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 05/10] dt-bindings: pinctrl: qcom,sm8550-tlmm: restrict number of interrupts

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 <[email protected]>
---
.../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

2023-12-08 21:57:21

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 06/10] dt-bindings: pinctrl: qcom,sm8650-tlmm: restrict number of interrupts

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 <[email protected]>
---
.../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

2023-12-08 21:57:30

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 04/10] dt-bindings: pinctrl: qcom,sdx75-tlmm: restrict number of interrupts

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 <[email protected]>
---
.../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

2023-12-08 21:57:31

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 10/10] dt-bindings: pinctrl: qcom: drop common properties and allow wakeup-parent

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 <[email protected]>
---
.../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

2023-12-08 21:57:37

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 08/10] dt-bindings: pinctrl: qcom,ipq5018-tlmm: use common TLMM bindings

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

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
.../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

2023-12-08 21:57:47

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 07/10] dt-bindings: pinctrl: qcom,x1e80100-tlmm: restrict number of interrupts

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 <[email protected]>
---
.../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

2023-12-08 21:57:48

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 09/10] dt-bindings: pinctrl: qcom: drop common properties

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 <[email protected]>
---
.../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

2023-12-11 23:22:37

by Richard Acayan

[permalink] [raw]
Subject: Re: [PATCH 09/10] dt-bindings: pinctrl: qcom: drop common properties

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 <[email protected]>
> ---

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
>

2023-12-13 18:40:59

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema


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 <[email protected]>
> ---
> .../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 <[email protected]>

2023-12-13 18:41:24

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 05/10] dt-bindings: pinctrl: qcom,sm8550-tlmm: restrict number of interrupts


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 <[email protected]>
> ---
> .../devicetree/bindings/pinctrl/qcom,sm8550-tlmm.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>

Reviewed-by: Rob Herring <[email protected]>

2023-12-13 18:41:28

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 03/10] dt-bindings: pinctrl: qcom,sa8775p-tlmm: restrict number of interrupts


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 <[email protected]>
> ---
> .../devicetree/bindings/pinctrl/qcom,sa8775p-tlmm.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>

Reviewed-by: Rob Herring <[email protected]>

2023-12-13 18:41:35

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 08/10] dt-bindings: pinctrl: qcom,ipq5018-tlmm: use common TLMM bindings


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 <[email protected]>
> ---
> .../bindings/pinctrl/qcom,ipq5018-tlmm.yaml | 12 ++++--------
> 1 file changed, 4 insertions(+), 8 deletions(-)
>

Reviewed-by: Rob Herring <[email protected]>

2023-12-13 18:41:46

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 04/10] dt-bindings: pinctrl: qcom,sdx75-tlmm: restrict number of interrupts


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 <[email protected]>
> ---
> .../devicetree/bindings/pinctrl/qcom,sdx75-tlmm.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>

Reviewed-by: Rob Herring <[email protected]>

2023-12-13 18:41:49

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 07/10] dt-bindings: pinctrl: qcom,x1e80100-tlmm: restrict number of interrupts


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 <[email protected]>
> ---
> .../devicetree/bindings/pinctrl/qcom,x1e80100-tlmm.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>

Reviewed-by: Rob Herring <[email protected]>

2023-12-13 18:42:20

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 10/10] dt-bindings: pinctrl: qcom: drop common properties and allow wakeup-parent


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 <[email protected]>
> ---
> .../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 <[email protected]>

2023-12-13 19:01:46

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 02/10] dt-bindings: pinctrl: qcom,qdu1000-tlmm: restrict number of interrupts


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 <[email protected]>
> ---
> .../devicetree/bindings/pinctrl/qcom,qdu1000-tlmm.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>

Reviewed-by: Rob Herring <[email protected]>

2023-12-13 19:43:01

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 06/10] dt-bindings: pinctrl: qcom,sm8650-tlmm: restrict number of interrupts


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 <[email protected]>
> ---
> .../devicetree/bindings/pinctrl/qcom,sm8650-tlmm.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>

Reviewed-by: Rob Herring <[email protected]>

2023-12-20 11:11:32

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH 01/10] dt-bindings: pinctrl: qcom: create common LPASS LPI schema

On Fri, Dec 8, 2023 at 10:55 PM Krzysztof Kozlowski
<[email protected]> 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 <[email protected]>

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