[PATCH] dt-bindings: Fix inconsistent quoting

linux-iio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed

* [PATCH] dt-bindings: Fix inconsistent quoting
@ 2025-10-15 23:16 Rob Herring (Arm)
  2025-10-16  0:26 ` Rob Herring (Arm)
                   ` (10 more replies)
  0 siblings, 11 replies; 15+ messages in thread
From: Rob Herring (Arm) @ 2025-10-15 23:16 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Linus Walleij, Bartosz Golaszewski, Shawn Guo, Fabio Estevam,
	Nuno Sá, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Andy Shevchenko, Jassi Brar,
	Mauro Carvalho Chehab, Lee Jones, Joel Stanley, Andrew Jeffery,
	Andrew Lunn, Vladimir Oltean, David S. Miller, Eric Dumazet,
	Jakub Kicinski, Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Alexandre Belloni,
	Claudiu Beznea, Florian Fainelli, Tony Lindgren
  Cc: devicetree, linux-kernel, linux-clk, linux-gpio, linux-arm-kernel,
	linux-iio, linux-media, netdev, linux-pci, linux-phy

yamllint has gained a new check which checks for inconsistent quoting
(mixed " and ' quotes within a file). Fix all the cases yamllint found
so we can enable the check (once the check is in a release). Use
whichever quoting is dominate in the file.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 .../arm/altera/socfpga-clk-manager.yaml       |  4 ++--
 .../bindings/clock/nvidia,tegra124-car.yaml   |  8 ++++----
 .../bindings/clock/nvidia,tegra20-car.yaml    |  6 +++---
 .../devicetree/bindings/gpio/gpio-mxs.yaml    |  9 +++++----
 .../bindings/gpio/snps,dw-apb-gpio.yaml       |  4 ++--
 .../bindings/iio/temperature/adi,ltc2983.yaml | 20 +++++++++----------
 .../mailbox/qcom,apcs-kpss-global.yaml        | 16 +++++++--------
 .../mailbox/xlnx,zynqmp-ipi-mailbox.yaml      |  2 +-
 .../bindings/media/fsl,imx6q-vdoa.yaml        |  2 +-
 .../devicetree/bindings/mfd/aspeed-lpc.yaml   |  4 ++--
 .../devicetree/bindings/mfd/ti,twl.yaml       |  4 ++--
 .../bindings/net/ethernet-switch.yaml         |  2 +-
 .../pci/plda,xpressrich3-axi-common.yaml      |  2 +-
 .../bindings/phy/motorola,cpcap-usb-phy.yaml  |  4 ++--
 .../pinctrl/microchip,sparx5-sgpio.yaml       | 12 +++++------
 .../bindings/pinctrl/qcom,pmic-gpio.yaml      | 10 +++++-----
 .../bindings/pinctrl/qcom,pmic-mpp.yaml       |  6 +++---
 .../bindings/pinctrl/renesas,pfc.yaml         |  4 ++--
 .../bindings/pinctrl/renesas,rza1-ports.yaml  |  2 +-
 .../pinctrl/renesas,rzg2l-pinctrl.yaml        |  2 +-
 .../pinctrl/renesas,rzv2m-pinctrl.yaml        |  2 +-
 .../bindings/power/renesas,sysc-rmobile.yaml  |  4 ++--
 .../soc/microchip/atmel,at91rm9200-tcb.yaml   |  8 ++++----
 .../soc/tegra/nvidia,tegra20-pmc.yaml         | 12 +++++------
 24 files changed, 75 insertions(+), 74 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml b/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
index a758f4bb2bb3..275554bfedce 100644
--- a/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
+++ b/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
@@ -39,7 +39,7 @@ properties:
 
       "^[a-z0-9,_]+(clk|pll|clk_gate|clk_divided)(@[a-f0-9]+)?$":
         type: object
-        $ref: '#/$defs/clock-props'
+        $ref: "#/$defs/clock-props"
         unevaluatedProperties: false
 
         properties:
@@ -67,7 +67,7 @@ properties:
         patternProperties:
           "^[a-z0-9,_]+(clk|pll)(@[a-f0-9]+)?$":
             type: object
-            $ref: '#/$defs/clock-props'
+            $ref: "#/$defs/clock-props"
             unevaluatedProperties: false
 
             properties:
diff --git a/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml b/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml
index a9ba21144a56..13bb616249a1 100644
--- a/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml
+++ b/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml
@@ -37,7 +37,7 @@ properties:
   '#clock-cells':
     const: 1
 
-  "#reset-cells":
+  '#reset-cells':
     const: 1
 
   nvidia,external-memory-controller:
@@ -46,7 +46,7 @@ properties:
       phandle of the external memory controller node
 
 patternProperties:
-  "^emc-timings-[0-9]+$":
+  '^emc-timings-[0-9]+$':
     type: object
     properties:
       nvidia,ram-code:
@@ -56,7 +56,7 @@ patternProperties:
           this timing set is used for
 
     patternProperties:
-      "^timing-[0-9]+$":
+      '^timing-[0-9]+$':
         type: object
         properties:
           clock-frequency:
@@ -94,7 +94,7 @@ required:
   - compatible
   - reg
   - '#clock-cells'
-  - "#reset-cells"
+  - '#reset-cells'
 
 additionalProperties: false
 
diff --git a/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml b/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml
index bee2dd4b29bf..73cccc0df424 100644
--- a/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml
+++ b/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml
@@ -39,11 +39,11 @@ properties:
   '#clock-cells':
     const: 1
 
-  "#reset-cells":
+  '#reset-cells':
     const: 1
 
 patternProperties:
-  "^(sclk)|(pll-[cem])$":
+  '^(sclk)|(pll-[cem])$':
     type: object
     properties:
       compatible:
@@ -76,7 +76,7 @@ required:
   - compatible
   - reg
   - '#clock-cells'
-  - "#reset-cells"
+  - '#reset-cells'
 
 additionalProperties: false
 
diff --git a/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml b/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml
index aaf97124803f..4b5b8e794613 100644
--- a/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml
+++ b/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml
@@ -26,9 +26,10 @@ properties:
       # Devices. Keep it as it to be compatible existed dts files.
       - const: simple-bus
 
-  '#address-cells':
+  "#address-cells":
     const: 1
-  '#size-cells':
+
+  "#size-cells":
     const: 0
 
   reg:
@@ -132,8 +133,8 @@ patternProperties:
 required:
   - compatible
   - reg
-  - '#address-cells'
-  - '#size-cells'
+  - "#address-cells"
+  - "#size-cells"
 
 additionalProperties: false
 
diff --git a/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml b/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
index ab2afc0e4153..bba6f5b6606f 100644
--- a/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
+++ b/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
@@ -111,8 +111,8 @@ additionalProperties: false
 required:
   - compatible
   - reg
-  - "#address-cells"
-  - "#size-cells"
+  - '#address-cells'
+  - '#size-cells'
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml b/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
index 312febeeb3bb..ee0b558bb866 100644
--- a/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
+++ b/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
@@ -88,7 +88,7 @@ properties:
     const: 0
 
 patternProperties:
-  "^thermocouple@":
+  '^thermocouple@':
     $ref: '#/$defs/sensor-node'
     unevaluatedProperties: false
 
@@ -146,7 +146,7 @@ patternProperties:
           required:
             - adi,custom-thermocouple
 
-  "^diode@":
+  '^diode@':
     $ref: '#/$defs/sensor-node'
     unevaluatedProperties: false
 
@@ -191,7 +191,7 @@ patternProperties:
         $ref: /schemas/types.yaml#/definitions/uint32
         default: 0
 
-  "^rtd@":
+  '^rtd@':
     $ref: '#/$defs/sensor-node'
     unevaluatedProperties: false
     description: RTD sensor.
@@ -280,7 +280,7 @@ patternProperties:
               type: boolean
 
           dependencies:
-            adi,current-rotate: [ "adi,rsense-share" ]
+            adi,current-rotate: [ 'adi,rsense-share' ]
 
       - if:
           properties:
@@ -290,7 +290,7 @@ patternProperties:
           required:
             - adi,custom-rtd
 
-  "^thermistor@":
+  '^thermistor@':
     $ref: '#/$defs/sensor-node'
     unevaluatedProperties: false
     description: Thermistor sensor.
@@ -364,7 +364,7 @@ patternProperties:
       - adi,rsense-handle
 
     dependencies:
-      adi,current-rotate: [ "adi,rsense-share" ]
+      adi,current-rotate: [ 'adi,rsense-share' ]
 
     allOf:
       - if:
@@ -392,7 +392,7 @@ patternProperties:
           required:
             - adi,custom-thermistor
 
-  "^adc@":
+  '^adc@':
     $ref: '#/$defs/sensor-node'
     unevaluatedProperties: false
     description: Direct ADC sensor.
@@ -407,7 +407,7 @@ patternProperties:
         description: Whether the sensor is single-ended.
         type: boolean
 
-  "^temp@":
+  '^temp@':
     $ref: '#/$defs/sensor-node'
     unevaluatedProperties: false
     description: Active analog temperature sensor.
@@ -437,7 +437,7 @@ patternProperties:
     required:
       - adi,custom-temp
 
-  "^rsense@":
+  '^rsense@':
     $ref: '#/$defs/sensor-node'
     unevaluatedProperties: false
     description: Sense resistor sensor.
@@ -476,7 +476,7 @@ allOf:
               - adi,ltc2984
     then:
       patternProperties:
-        "^temp@": false
+        '^temp@': false
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
index 615ed103b7e6..f40dc9048327 100644
--- a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
+++ b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
@@ -187,10 +187,10 @@ allOf:
             enum:
               - qcom,msm8916-apcs-kpss-global
     then:
-      $ref: "#/$defs/msm8916-apcs-clock-controller"
+      $ref: '#/$defs/msm8916-apcs-clock-controller'
       properties:
         clock-controller:
-          $ref: "#/$defs/msm8916-apcs-clock-controller"
+          $ref: '#/$defs/msm8916-apcs-clock-controller'
 
   - if:
       properties:
@@ -199,10 +199,10 @@ allOf:
             enum:
               - qcom,msm8939-apcs-kpss-global
     then:
-      $ref: "#/$defs/msm8939-apcs-clock-controller"
+      $ref: '#/$defs/msm8939-apcs-clock-controller'
       properties:
         clock-controller:
-          $ref: "#/$defs/msm8939-apcs-clock-controller"
+          $ref: '#/$defs/msm8939-apcs-clock-controller'
 
   - if:
       properties:
@@ -211,10 +211,10 @@ allOf:
             enum:
               - qcom,sdx55-apcs-gcc
     then:
-      $ref: "#/$defs/sdx55-apcs-clock-controller"
+      $ref: '#/$defs/sdx55-apcs-clock-controller'
       properties:
         clock-controller:
-          $ref: "#/$defs/sdx55-apcs-clock-controller"
+          $ref: '#/$defs/sdx55-apcs-clock-controller'
 
   - if:
       properties:
@@ -223,10 +223,10 @@ allOf:
             enum:
               - qcom,ipq6018-apcs-apps-global
     then:
-      $ref: "#/$defs/ipq6018-apcs-clock-controller"
+      $ref: '#/$defs/ipq6018-apcs-clock-controller'
       properties:
         clock-controller:
-          $ref: "#/$defs/ipq6018-apcs-clock-controller"
+          $ref: '#/$defs/ipq6018-apcs-clock-controller'
 
   - if:
       properties:
diff --git a/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml
index fe83b5cb1278..04d6473d666f 100644
--- a/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml
+++ b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml
@@ -142,7 +142,7 @@ patternProperties:
       - compatible
       - reg
       - reg-names
-      - "#mbox-cells"
+      - '#mbox-cells'
       - xlnx,ipi-id
 
 required:
diff --git a/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml b/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml
index 511ac0d67a7f..988a5b3a62bd 100644
--- a/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml
+++ b/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml
@@ -16,7 +16,7 @@ maintainers:
 
 properties:
   compatible:
-    const: "fsl,imx6q-vdoa"
+    const: fsl,imx6q-vdoa
 
   reg:
     maxItems: 1
diff --git a/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml b/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml
index f329223cec07..0adfeef149e7 100644
--- a/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml
+++ b/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml
@@ -111,12 +111,12 @@ patternProperties:
       reg:
         maxItems: 1
 
-      '#reset-cells':
+      "#reset-cells":
         const: 1
 
     required:
       - compatible
-      - '#reset-cells'
+      - "#reset-cells"
 
   "^lpc-snoop@[0-9a-f]+$":
     type: object
diff --git a/Documentation/devicetree/bindings/mfd/ti,twl.yaml b/Documentation/devicetree/bindings/mfd/ti,twl.yaml
index 776b04e182cb..1611b1581a8e 100644
--- a/Documentation/devicetree/bindings/mfd/ti,twl.yaml
+++ b/Documentation/devicetree/bindings/mfd/ti,twl.yaml
@@ -400,7 +400,7 @@ properties:
       - '#pwm-cells'
 
 patternProperties:
-  "^regulator-":
+  '^regulator-':
     type: object
     unevaluatedProperties: false
     $ref: /schemas/regulator/regulator.yaml
@@ -429,7 +429,7 @@ required:
   - reg
   - interrupts
   - interrupt-controller
-  - "#interrupt-cells"
+  - '#interrupt-cells'
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/net/ethernet-switch.yaml b/Documentation/devicetree/bindings/net/ethernet-switch.yaml
index b3b7e1a1b127..03001ba40e0f 100644
--- a/Documentation/devicetree/bindings/net/ethernet-switch.yaml
+++ b/Documentation/devicetree/bindings/net/ethernet-switch.yaml
@@ -72,7 +72,7 @@ additionalProperties: true
 $defs:
   ethernet-ports:
     description: An ethernet switch without any extra port properties
-    $ref: '#'
+    $ref: "#"
 
     patternProperties:
       "^(ethernet-)?ports$":
diff --git a/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml b/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml
index 039eecdbd6aa..fe2e8beb5bab 100644
--- a/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml
+++ b/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml
@@ -72,7 +72,7 @@ required:
   - reg-names
   - interrupts
   - msi-controller
-  - "#interrupt-cells"
+  - '#interrupt-cells'
   - interrupt-map-mask
   - interrupt-map
 
diff --git a/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml b/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml
index 0febd04a61f4..dd345cbd0a0b 100644
--- a/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml
@@ -67,8 +67,8 @@ properties:
   mode-gpios:
     description: Optional GPIOs for configuring alternate modes
     items:
-      - description: "mode selection GPIO #0"
-      - description: "mode selection GPIO #1"
+      - description: mode selection GPIO#0
+      - description: mode selection GPIO#1
 
 required:
   - compatible
diff --git a/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml b/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml
index 0df4e114fdd6..fa47732d7cef 100644
--- a/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml
@@ -18,7 +18,7 @@ description: |
 
 properties:
   $nodename:
-    pattern: "^gpio@[0-9a-f]+$"
+    pattern: '^gpio@[0-9a-f]+$'
 
   compatible:
     enum:
@@ -26,10 +26,10 @@ properties:
       - mscc,ocelot-sgpio
       - mscc,luton-sgpio
 
-  "#address-cells":
+  '#address-cells':
     const: 1
 
-  "#size-cells":
+  '#size-cells':
     const: 0
 
   reg:
@@ -76,7 +76,7 @@ properties:
       - const: switch
 
 patternProperties:
-  "^gpio@[0-1]$":
+  '^gpio@[0-1]$':
     type: object
     properties:
       compatible:
@@ -132,8 +132,8 @@ required:
   - reg
   - clocks
   - microchip,sgpio-port-ranges
-  - "#address-cells"
-  - "#size-cells"
+  - '#address-cells'
+  - '#size-cells'
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml
index 5e6dfcc3fe9b..6632bcd037ba 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml
@@ -424,13 +424,13 @@ allOf:
 patternProperties:
   '-state$':
     oneOf:
-      - $ref: "#/$defs/qcom-pmic-gpio-state"
+      - $ref: '#/$defs/qcom-pmic-gpio-state'
       - patternProperties:
-          "(pinconf|-pins)$":
-            $ref: "#/$defs/qcom-pmic-gpio-state"
+          '(pinconf|-pins)$':
+            $ref: '#/$defs/qcom-pmic-gpio-state'
         additionalProperties: false
 
-  "-hog(-[0-9]+)?$":
+  '-hog(-[0-9]+)?$':
     type: object
     required:
       - gpio-hog
@@ -503,7 +503,7 @@ $defs:
                  - gpio1-gpio12 for pmxr2230
 
         items:
-          pattern: "^gpio([0-9]+)$"
+          pattern: '^gpio([0-9]+)$'
 
       function:
         items:
diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml
index 9364ae05f3e6..daf4c1c03712 100644
--- a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml
@@ -74,10 +74,10 @@ required:
 patternProperties:
   '-state$':
     oneOf:
-      - $ref: "#/$defs/qcom-pmic-mpp-state"
+      - $ref: '#/$defs/qcom-pmic-mpp-state'
       - patternProperties:
           '-pins$':
-            $ref: "#/$defs/qcom-pmic-mpp-state"
+            $ref: '#/$defs/qcom-pmic-mpp-state'
         additionalProperties: false
 
 $defs:
@@ -100,7 +100,7 @@ $defs:
                  - mpp1-mpp4 for pma8084
 
         items:
-          pattern: "^mpp([0-9]+)$"
+          pattern: '^mpp([0-9]+)$'
 
       function:
         items:
diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
index cfe004573366..ab1cfe9dcde5 100644
--- a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
@@ -129,7 +129,7 @@ additionalProperties:
 
     - type: object
       additionalProperties:
-        $ref: "#/additionalProperties/anyOf/0"
+        $ref: '#/additionalProperties/anyOf/0'
 
 examples:
   - |
@@ -190,7 +190,7 @@ examples:
 
             sdhi0_pins: sd0 {
                     groups = "sdhi0_data4", "sdhi0_ctrl";
-                    function = "sdhi0";
+                    function = "sdhi0';
                     power-source = <3300>;
             };
     };
diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml
index 2bd7d47d0fdb..737eb4e14090 100644
--- a/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml
@@ -118,7 +118,7 @@ additionalProperties:
 
     - type: object
       additionalProperties:
-        $ref: "#/additionalProperties/anyOf/0"
+        $ref: '#/additionalProperties/anyOf/0'
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
index 5156d54b240b..00c05243b9a4 100644
--- a/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
@@ -135,7 +135,7 @@ additionalProperties:
 
     - type: object
       additionalProperties:
-        $ref: "#/additionalProperties/anyOf/0"
+        $ref: '#/additionalProperties/anyOf/0'
 
 allOf:
   - $ref: pinctrl.yaml#
diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml
index 5fa5d31f8866..88b2fa5e684d 100644
--- a/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml
@@ -88,7 +88,7 @@ additionalProperties:
 
     - type: object
       additionalProperties:
-        $ref: "#/additionalProperties/anyOf/0"
+        $ref: '#/additionalProperties/anyOf/0'
 
 allOf:
   - $ref: pinctrl.yaml#
diff --git a/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml b/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml
index fba6914ec40d..948a9da111df 100644
--- a/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml
+++ b/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml
@@ -45,7 +45,7 @@ properties:
         const: 0
 
     additionalProperties:
-      $ref: "#/$defs/pd-node"
+      $ref: '#/$defs/pd-node'
 
 required:
   - compatible
@@ -83,7 +83,7 @@ $defs:
       - '#power-domain-cells'
 
     additionalProperties:
-      $ref: "#/$defs/pd-node"
+      $ref: '#/$defs/pd-node'
 
 examples:
   - |
diff --git a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
index 2c7275c4503b..abf1adca0773 100644
--- a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
+++ b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
@@ -57,7 +57,7 @@ properties:
     const: 0
 
 patternProperties:
-  "^timer@[0-2]$":
+  '^timer@[0-2]$':
     description: The timer block channels that are used as timers or counters.
     type: object
     additionalProperties: false
@@ -80,7 +80,7 @@ patternProperties:
       - compatible
       - reg
 
-  "^pwm@[0-2]$":
+  '^pwm@[0-2]$':
     description: The timer block channels that are used as PWMs.
     $ref: /schemas/pwm/pwm.yaml#
     type: object
@@ -92,7 +92,7 @@ patternProperties:
           TCB channel to use for this PWM.
         enum: [ 0, 1, 2 ]
 
-      "#pwm-cells":
+      '#pwm-cells':
         description:
           The only third cell flag supported by this binding is
           PWM_POLARITY_INVERTED.
@@ -101,7 +101,7 @@ patternProperties:
     required:
       - compatible
       - reg
-      - "#pwm-cells"
+      - '#pwm-cells'
 
     additionalProperties: false
 
diff --git a/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml b/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml
index 7140c312d898..f516960dbbef 100644
--- a/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml
+++ b/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml
@@ -133,12 +133,12 @@ properties:
           property. The supported-hw is a bitfield indicating SoC speedo or
           process ID mask.
 
-      "#power-domain-cells":
+      '#power-domain-cells':
         const: 0
 
     required:
       - operating-points-v2
-      - "#power-domain-cells"
+      - '#power-domain-cells'
 
   i2c-thermtrip:
     type: object
@@ -220,7 +220,7 @@ properties:
         xusbc    USB Partition C               Tegra114/124/210
 
     patternProperties:
-      "^[a-z0-9]+$":
+      '^[a-z0-9]+$':
         type: object
         additionalProperties: false
         properties:
@@ -365,9 +365,9 @@ allOf:
 additionalProperties: false
 
 dependencies:
-  nvidia,suspend-mode: ["nvidia,core-pwr-off-time", "nvidia,cpu-pwr-off-time"]
-  nvidia,core-pwr-off-time: ["nvidia,core-pwr-good-time"]
-  nvidia,cpu-pwr-off-time: ["nvidia,cpu-pwr-good-time"]
+  nvidia,suspend-mode: ['nvidia,core-pwr-off-time', 'nvidia,cpu-pwr-off-time']
+  nvidia,core-pwr-off-time: ['nvidia,core-pwr-good-time']
+  nvidia,cpu-pwr-off-time: ['nvidia,cpu-pwr-good-time']
 
 examples:
   - |
-- 
2.51.0


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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
@ 2025-10-16  0:26 ` Rob Herring (Arm)
  2025-10-16  6:49 ` Alexandre Belloni
                   ` (9 subsequent siblings)
  10 siblings, 0 replies; 15+ messages in thread
From: Rob Herring (Arm) @ 2025-10-16  0:26 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Vladimir Oltean, Nicolas Ferre, devicetree,
	Jakub Kicinski, Andy Shevchenko, Joel Stanley, Bjorn Helgaas,
	Andrew Jeffery, Lee Jones, Daire McNamara, Mauro Carvalho Chehab,
	Eric Dumazet, linux-iio, Manivannan Sadhasivam, Andrew Lunn,
	Michael Hennerich, Lorenzo Pieralisi, Vinod Koul,
	Jonathan Cameron, linux-phy, Florian Fainelli, netdev,
	Paolo Abeni, Kishon Vijay Abraham I, Stephen Boyd, Jassi Brar,
	linux-gpio, Bartosz Golaszewski, linux-media, David S. Miller,
	Lars-Peter Clausen, Alexandre Belloni, linux-clk, Conor Dooley,
	linux-pci, Shawn Guo, Claudiu Beznea, linux-kernel,
	Geert Uytterhoeven, Thierry Reding, Fabio Estevam,
	linux-arm-kernel, Linus Walleij, Krzysztof Wilczyński,
	Bjorn Andersson, Nuno Sá, Tony Lindgren


On Wed, 15 Oct 2025 18:16:24 -0500, Rob Herring (Arm) wrote:
> yamllint has gained a new check which checks for inconsistent quoting
> (mixed " and ' quotes within a file). Fix all the cases yamllint found
> so we can enable the check (once the check is in a release). Use
> whichever quoting is dominate in the file.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../arm/altera/socfpga-clk-manager.yaml       |  4 ++--
>  .../bindings/clock/nvidia,tegra124-car.yaml   |  8 ++++----
>  .../bindings/clock/nvidia,tegra20-car.yaml    |  6 +++---
>  .../devicetree/bindings/gpio/gpio-mxs.yaml    |  9 +++++----
>  .../bindings/gpio/snps,dw-apb-gpio.yaml       |  4 ++--
>  .../bindings/iio/temperature/adi,ltc2983.yaml | 20 +++++++++----------
>  .../mailbox/qcom,apcs-kpss-global.yaml        | 16 +++++++--------
>  .../mailbox/xlnx,zynqmp-ipi-mailbox.yaml      |  2 +-
>  .../bindings/media/fsl,imx6q-vdoa.yaml        |  2 +-
>  .../devicetree/bindings/mfd/aspeed-lpc.yaml   |  4 ++--
>  .../devicetree/bindings/mfd/ti,twl.yaml       |  4 ++--
>  .../bindings/net/ethernet-switch.yaml         |  2 +-
>  .../pci/plda,xpressrich3-axi-common.yaml      |  2 +-
>  .../bindings/phy/motorola,cpcap-usb-phy.yaml  |  4 ++--
>  .../pinctrl/microchip,sparx5-sgpio.yaml       | 12 +++++------
>  .../bindings/pinctrl/qcom,pmic-gpio.yaml      | 10 +++++-----
>  .../bindings/pinctrl/qcom,pmic-mpp.yaml       |  6 +++---
>  .../bindings/pinctrl/renesas,pfc.yaml         |  4 ++--
>  .../bindings/pinctrl/renesas,rza1-ports.yaml  |  2 +-
>  .../pinctrl/renesas,rzg2l-pinctrl.yaml        |  2 +-
>  .../pinctrl/renesas,rzv2m-pinctrl.yaml        |  2 +-
>  .../bindings/power/renesas,sysc-rmobile.yaml  |  4 ++--
>  .../soc/microchip/atmel,at91rm9200-tcb.yaml   |  8 ++++----
>  .../soc/tegra/nvidia,tegra20-pmc.yaml         | 12 +++++------
>  24 files changed, 75 insertions(+), 74 deletions(-)
> 

My bot found errors running 'make dt_binding_check' on your patch:

yamllint warnings/errors:

dtschema/dtc warnings/errors:
Error: Documentation/devicetree/bindings/pinctrl/renesas,pfc.example.dts:90.36-37 syntax error
FATAL ERROR: Unable to parse input tree
make[2]: *** [scripts/Makefile.dtbs:132: Documentation/devicetree/bindings/pinctrl/renesas,pfc.example.dtb] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [/builds/robherring/dt-review-ci/linux/Makefile:1525: dt_binding_check] Error 2
make: *** [Makefile:248: __sub-make] Error 2

doc reference errors (make refcheckdocs):

See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20251015232015.846282-1-robh@kernel.org

The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.

If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:

pip3 install dtschema --upgrade

Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.


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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
  2025-10-16  0:26 ` Rob Herring (Arm)
@ 2025-10-16  6:49 ` Alexandre Belloni
  2025-10-16  6:57 ` Krzysztof Kozlowski
                   ` (8 subsequent siblings)
  10 siblings, 0 replies; 15+ messages in thread
From: Alexandre Belloni @ 2025-10-16  6:49 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Linus Walleij, Bartosz Golaszewski, Shawn Guo, Fabio Estevam,
	Nuno Sá, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Andy Shevchenko, Jassi Brar,
	Mauro Carvalho Chehab, Lee Jones, Joel Stanley, Andrew Jeffery,
	Andrew Lunn, Vladimir Oltean, David S. Miller, Eric Dumazet,
	Jakub Kicinski, Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Claudiu Beznea,
	Florian Fainelli, Tony Lindgren, devicetree, linux-kernel,
	linux-clk, linux-gpio, linux-arm-kernel, linux-iio, linux-media,
	netdev, linux-pci, linux-phy

On 15/10/2025 18:16:24-0500, Rob Herring (Arm) wrote:
> yamllint has gained a new check which checks for inconsistent quoting
> (mixed " and ' quotes within a file). Fix all the cases yamllint found
> so we can enable the check (once the check is in a release). Use
> whichever quoting is dominate in the file.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>


> ---
>  .../arm/altera/socfpga-clk-manager.yaml       |  4 ++--
>  .../bindings/clock/nvidia,tegra124-car.yaml   |  8 ++++----
>  .../bindings/clock/nvidia,tegra20-car.yaml    |  6 +++---
>  .../devicetree/bindings/gpio/gpio-mxs.yaml    |  9 +++++----
>  .../bindings/gpio/snps,dw-apb-gpio.yaml       |  4 ++--
>  .../bindings/iio/temperature/adi,ltc2983.yaml | 20 +++++++++----------
>  .../mailbox/qcom,apcs-kpss-global.yaml        | 16 +++++++--------
>  .../mailbox/xlnx,zynqmp-ipi-mailbox.yaml      |  2 +-
>  .../bindings/media/fsl,imx6q-vdoa.yaml        |  2 +-
>  .../devicetree/bindings/mfd/aspeed-lpc.yaml   |  4 ++--
>  .../devicetree/bindings/mfd/ti,twl.yaml       |  4 ++--
>  .../bindings/net/ethernet-switch.yaml         |  2 +-
>  .../pci/plda,xpressrich3-axi-common.yaml      |  2 +-
>  .../bindings/phy/motorola,cpcap-usb-phy.yaml  |  4 ++--
>  .../pinctrl/microchip,sparx5-sgpio.yaml       | 12 +++++------
>  .../bindings/pinctrl/qcom,pmic-gpio.yaml      | 10 +++++-----
>  .../bindings/pinctrl/qcom,pmic-mpp.yaml       |  6 +++---
>  .../bindings/pinctrl/renesas,pfc.yaml         |  4 ++--
>  .../bindings/pinctrl/renesas,rza1-ports.yaml  |  2 +-
>  .../pinctrl/renesas,rzg2l-pinctrl.yaml        |  2 +-
>  .../pinctrl/renesas,rzv2m-pinctrl.yaml        |  2 +-
>  .../bindings/power/renesas,sysc-rmobile.yaml  |  4 ++--
>  .../soc/microchip/atmel,at91rm9200-tcb.yaml   |  8 ++++----
>  .../soc/tegra/nvidia,tegra20-pmc.yaml         | 12 +++++------
>  24 files changed, 75 insertions(+), 74 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml b/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
> index a758f4bb2bb3..275554bfedce 100644
> --- a/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
> +++ b/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
> @@ -39,7 +39,7 @@ properties:
>  
>        "^[a-z0-9,_]+(clk|pll|clk_gate|clk_divided)(@[a-f0-9]+)?$":
>          type: object
> -        $ref: '#/$defs/clock-props'
> +        $ref: "#/$defs/clock-props"
>          unevaluatedProperties: false
>  
>          properties:
> @@ -67,7 +67,7 @@ properties:
>          patternProperties:
>            "^[a-z0-9,_]+(clk|pll)(@[a-f0-9]+)?$":
>              type: object
> -            $ref: '#/$defs/clock-props'
> +            $ref: "#/$defs/clock-props"
>              unevaluatedProperties: false
>  
>              properties:
> diff --git a/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml b/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml
> index a9ba21144a56..13bb616249a1 100644
> --- a/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml
> +++ b/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml
> @@ -37,7 +37,7 @@ properties:
>    '#clock-cells':
>      const: 1
>  
> -  "#reset-cells":
> +  '#reset-cells':
>      const: 1
>  
>    nvidia,external-memory-controller:
> @@ -46,7 +46,7 @@ properties:
>        phandle of the external memory controller node
>  
>  patternProperties:
> -  "^emc-timings-[0-9]+$":
> +  '^emc-timings-[0-9]+$':
>      type: object
>      properties:
>        nvidia,ram-code:
> @@ -56,7 +56,7 @@ patternProperties:
>            this timing set is used for
>  
>      patternProperties:
> -      "^timing-[0-9]+$":
> +      '^timing-[0-9]+$':
>          type: object
>          properties:
>            clock-frequency:
> @@ -94,7 +94,7 @@ required:
>    - compatible
>    - reg
>    - '#clock-cells'
> -  - "#reset-cells"
> +  - '#reset-cells'
>  
>  additionalProperties: false
>  
> diff --git a/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml b/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml
> index bee2dd4b29bf..73cccc0df424 100644
> --- a/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml
> +++ b/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml
> @@ -39,11 +39,11 @@ properties:
>    '#clock-cells':
>      const: 1
>  
> -  "#reset-cells":
> +  '#reset-cells':
>      const: 1
>  
>  patternProperties:
> -  "^(sclk)|(pll-[cem])$":
> +  '^(sclk)|(pll-[cem])$':
>      type: object
>      properties:
>        compatible:
> @@ -76,7 +76,7 @@ required:
>    - compatible
>    - reg
>    - '#clock-cells'
> -  - "#reset-cells"
> +  - '#reset-cells'
>  
>  additionalProperties: false
>  
> diff --git a/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml b/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml
> index aaf97124803f..4b5b8e794613 100644
> --- a/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml
> +++ b/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml
> @@ -26,9 +26,10 @@ properties:
>        # Devices. Keep it as it to be compatible existed dts files.
>        - const: simple-bus
>  
> -  '#address-cells':
> +  "#address-cells":
>      const: 1
> -  '#size-cells':
> +
> +  "#size-cells":
>      const: 0
>  
>    reg:
> @@ -132,8 +133,8 @@ patternProperties:
>  required:
>    - compatible
>    - reg
> -  - '#address-cells'
> -  - '#size-cells'
> +  - "#address-cells"
> +  - "#size-cells"
>  
>  additionalProperties: false
>  
> diff --git a/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml b/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
> index ab2afc0e4153..bba6f5b6606f 100644
> --- a/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
> +++ b/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
> @@ -111,8 +111,8 @@ additionalProperties: false
>  required:
>    - compatible
>    - reg
> -  - "#address-cells"
> -  - "#size-cells"
> +  - '#address-cells'
> +  - '#size-cells'
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml b/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
> index 312febeeb3bb..ee0b558bb866 100644
> --- a/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
> +++ b/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
> @@ -88,7 +88,7 @@ properties:
>      const: 0
>  
>  patternProperties:
> -  "^thermocouple@":
> +  '^thermocouple@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>  
> @@ -146,7 +146,7 @@ patternProperties:
>            required:
>              - adi,custom-thermocouple
>  
> -  "^diode@":
> +  '^diode@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>  
> @@ -191,7 +191,7 @@ patternProperties:
>          $ref: /schemas/types.yaml#/definitions/uint32
>          default: 0
>  
> -  "^rtd@":
> +  '^rtd@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>      description: RTD sensor.
> @@ -280,7 +280,7 @@ patternProperties:
>                type: boolean
>  
>            dependencies:
> -            adi,current-rotate: [ "adi,rsense-share" ]
> +            adi,current-rotate: [ 'adi,rsense-share' ]
>  
>        - if:
>            properties:
> @@ -290,7 +290,7 @@ patternProperties:
>            required:
>              - adi,custom-rtd
>  
> -  "^thermistor@":
> +  '^thermistor@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>      description: Thermistor sensor.
> @@ -364,7 +364,7 @@ patternProperties:
>        - adi,rsense-handle
>  
>      dependencies:
> -      adi,current-rotate: [ "adi,rsense-share" ]
> +      adi,current-rotate: [ 'adi,rsense-share' ]
>  
>      allOf:
>        - if:
> @@ -392,7 +392,7 @@ patternProperties:
>            required:
>              - adi,custom-thermistor
>  
> -  "^adc@":
> +  '^adc@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>      description: Direct ADC sensor.
> @@ -407,7 +407,7 @@ patternProperties:
>          description: Whether the sensor is single-ended.
>          type: boolean
>  
> -  "^temp@":
> +  '^temp@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>      description: Active analog temperature sensor.
> @@ -437,7 +437,7 @@ patternProperties:
>      required:
>        - adi,custom-temp
>  
> -  "^rsense@":
> +  '^rsense@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>      description: Sense resistor sensor.
> @@ -476,7 +476,7 @@ allOf:
>                - adi,ltc2984
>      then:
>        patternProperties:
> -        "^temp@": false
> +        '^temp@': false
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
> index 615ed103b7e6..f40dc9048327 100644
> --- a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
> +++ b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
> @@ -187,10 +187,10 @@ allOf:
>              enum:
>                - qcom,msm8916-apcs-kpss-global
>      then:
> -      $ref: "#/$defs/msm8916-apcs-clock-controller"
> +      $ref: '#/$defs/msm8916-apcs-clock-controller'
>        properties:
>          clock-controller:
> -          $ref: "#/$defs/msm8916-apcs-clock-controller"
> +          $ref: '#/$defs/msm8916-apcs-clock-controller'
>  
>    - if:
>        properties:
> @@ -199,10 +199,10 @@ allOf:
>              enum:
>                - qcom,msm8939-apcs-kpss-global
>      then:
> -      $ref: "#/$defs/msm8939-apcs-clock-controller"
> +      $ref: '#/$defs/msm8939-apcs-clock-controller'
>        properties:
>          clock-controller:
> -          $ref: "#/$defs/msm8939-apcs-clock-controller"
> +          $ref: '#/$defs/msm8939-apcs-clock-controller'
>  
>    - if:
>        properties:
> @@ -211,10 +211,10 @@ allOf:
>              enum:
>                - qcom,sdx55-apcs-gcc
>      then:
> -      $ref: "#/$defs/sdx55-apcs-clock-controller"
> +      $ref: '#/$defs/sdx55-apcs-clock-controller'
>        properties:
>          clock-controller:
> -          $ref: "#/$defs/sdx55-apcs-clock-controller"
> +          $ref: '#/$defs/sdx55-apcs-clock-controller'
>  
>    - if:
>        properties:
> @@ -223,10 +223,10 @@ allOf:
>              enum:
>                - qcom,ipq6018-apcs-apps-global
>      then:
> -      $ref: "#/$defs/ipq6018-apcs-clock-controller"
> +      $ref: '#/$defs/ipq6018-apcs-clock-controller'
>        properties:
>          clock-controller:
> -          $ref: "#/$defs/ipq6018-apcs-clock-controller"
> +          $ref: '#/$defs/ipq6018-apcs-clock-controller'
>  
>    - if:
>        properties:
> diff --git a/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml
> index fe83b5cb1278..04d6473d666f 100644
> --- a/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml
> +++ b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml
> @@ -142,7 +142,7 @@ patternProperties:
>        - compatible
>        - reg
>        - reg-names
> -      - "#mbox-cells"
> +      - '#mbox-cells'
>        - xlnx,ipi-id
>  
>  required:
> diff --git a/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml b/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml
> index 511ac0d67a7f..988a5b3a62bd 100644
> --- a/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml
> +++ b/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml
> @@ -16,7 +16,7 @@ maintainers:
>  
>  properties:
>    compatible:
> -    const: "fsl,imx6q-vdoa"
> +    const: fsl,imx6q-vdoa
>  
>    reg:
>      maxItems: 1
> diff --git a/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml b/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml
> index f329223cec07..0adfeef149e7 100644
> --- a/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml
> +++ b/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml
> @@ -111,12 +111,12 @@ patternProperties:
>        reg:
>          maxItems: 1
>  
> -      '#reset-cells':
> +      "#reset-cells":
>          const: 1
>  
>      required:
>        - compatible
> -      - '#reset-cells'
> +      - "#reset-cells"
>  
>    "^lpc-snoop@[0-9a-f]+$":
>      type: object
> diff --git a/Documentation/devicetree/bindings/mfd/ti,twl.yaml b/Documentation/devicetree/bindings/mfd/ti,twl.yaml
> index 776b04e182cb..1611b1581a8e 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,twl.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,twl.yaml
> @@ -400,7 +400,7 @@ properties:
>        - '#pwm-cells'
>  
>  patternProperties:
> -  "^regulator-":
> +  '^regulator-':
>      type: object
>      unevaluatedProperties: false
>      $ref: /schemas/regulator/regulator.yaml
> @@ -429,7 +429,7 @@ required:
>    - reg
>    - interrupts
>    - interrupt-controller
> -  - "#interrupt-cells"
> +  - '#interrupt-cells'
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/net/ethernet-switch.yaml b/Documentation/devicetree/bindings/net/ethernet-switch.yaml
> index b3b7e1a1b127..03001ba40e0f 100644
> --- a/Documentation/devicetree/bindings/net/ethernet-switch.yaml
> +++ b/Documentation/devicetree/bindings/net/ethernet-switch.yaml
> @@ -72,7 +72,7 @@ additionalProperties: true
>  $defs:
>    ethernet-ports:
>      description: An ethernet switch without any extra port properties
> -    $ref: '#'
> +    $ref: "#"
>  
>      patternProperties:
>        "^(ethernet-)?ports$":
> diff --git a/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml b/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml
> index 039eecdbd6aa..fe2e8beb5bab 100644
> --- a/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml
> +++ b/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml
> @@ -72,7 +72,7 @@ required:
>    - reg-names
>    - interrupts
>    - msi-controller
> -  - "#interrupt-cells"
> +  - '#interrupt-cells'
>    - interrupt-map-mask
>    - interrupt-map
>  
> diff --git a/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml b/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml
> index 0febd04a61f4..dd345cbd0a0b 100644
> --- a/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml
> @@ -67,8 +67,8 @@ properties:
>    mode-gpios:
>      description: Optional GPIOs for configuring alternate modes
>      items:
> -      - description: "mode selection GPIO #0"
> -      - description: "mode selection GPIO #1"
> +      - description: mode selection GPIO#0
> +      - description: mode selection GPIO#1
>  
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml b/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml
> index 0df4e114fdd6..fa47732d7cef 100644
> --- a/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml
> @@ -18,7 +18,7 @@ description: |
>  
>  properties:
>    $nodename:
> -    pattern: "^gpio@[0-9a-f]+$"
> +    pattern: '^gpio@[0-9a-f]+$'
>  
>    compatible:
>      enum:
> @@ -26,10 +26,10 @@ properties:
>        - mscc,ocelot-sgpio
>        - mscc,luton-sgpio
>  
> -  "#address-cells":
> +  '#address-cells':
>      const: 1
>  
> -  "#size-cells":
> +  '#size-cells':
>      const: 0
>  
>    reg:
> @@ -76,7 +76,7 @@ properties:
>        - const: switch
>  
>  patternProperties:
> -  "^gpio@[0-1]$":
> +  '^gpio@[0-1]$':
>      type: object
>      properties:
>        compatible:
> @@ -132,8 +132,8 @@ required:
>    - reg
>    - clocks
>    - microchip,sgpio-port-ranges
> -  - "#address-cells"
> -  - "#size-cells"
> +  - '#address-cells'
> +  - '#size-cells'
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml
> index 5e6dfcc3fe9b..6632bcd037ba 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml
> @@ -424,13 +424,13 @@ allOf:
>  patternProperties:
>    '-state$':
>      oneOf:
> -      - $ref: "#/$defs/qcom-pmic-gpio-state"
> +      - $ref: '#/$defs/qcom-pmic-gpio-state'
>        - patternProperties:
> -          "(pinconf|-pins)$":
> -            $ref: "#/$defs/qcom-pmic-gpio-state"
> +          '(pinconf|-pins)$':
> +            $ref: '#/$defs/qcom-pmic-gpio-state'
>          additionalProperties: false
>  
> -  "-hog(-[0-9]+)?$":
> +  '-hog(-[0-9]+)?$':
>      type: object
>      required:
>        - gpio-hog
> @@ -503,7 +503,7 @@ $defs:
>                   - gpio1-gpio12 for pmxr2230
>  
>          items:
> -          pattern: "^gpio([0-9]+)$"
> +          pattern: '^gpio([0-9]+)$'
>  
>        function:
>          items:
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml
> index 9364ae05f3e6..daf4c1c03712 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml
> @@ -74,10 +74,10 @@ required:
>  patternProperties:
>    '-state$':
>      oneOf:
> -      - $ref: "#/$defs/qcom-pmic-mpp-state"
> +      - $ref: '#/$defs/qcom-pmic-mpp-state'
>        - patternProperties:
>            '-pins$':
> -            $ref: "#/$defs/qcom-pmic-mpp-state"
> +            $ref: '#/$defs/qcom-pmic-mpp-state'
>          additionalProperties: false
>  
>  $defs:
> @@ -100,7 +100,7 @@ $defs:
>                   - mpp1-mpp4 for pma8084
>  
>          items:
> -          pattern: "^mpp([0-9]+)$"
> +          pattern: '^mpp([0-9]+)$'
>  
>        function:
>          items:
> diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> index cfe004573366..ab1cfe9dcde5 100644
> --- a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> @@ -129,7 +129,7 @@ additionalProperties:
>  
>      - type: object
>        additionalProperties:
> -        $ref: "#/additionalProperties/anyOf/0"
> +        $ref: '#/additionalProperties/anyOf/0'
>  
>  examples:
>    - |
> @@ -190,7 +190,7 @@ examples:
>  
>              sdhi0_pins: sd0 {
>                      groups = "sdhi0_data4", "sdhi0_ctrl";
> -                    function = "sdhi0";
> +                    function = "sdhi0';
>                      power-source = <3300>;
>              };
>      };
> diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml
> index 2bd7d47d0fdb..737eb4e14090 100644
> --- a/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml
> @@ -118,7 +118,7 @@ additionalProperties:
>  
>      - type: object
>        additionalProperties:
> -        $ref: "#/additionalProperties/anyOf/0"
> +        $ref: '#/additionalProperties/anyOf/0'
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> index 5156d54b240b..00c05243b9a4 100644
> --- a/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> @@ -135,7 +135,7 @@ additionalProperties:
>  
>      - type: object
>        additionalProperties:
> -        $ref: "#/additionalProperties/anyOf/0"
> +        $ref: '#/additionalProperties/anyOf/0'
>  
>  allOf:
>    - $ref: pinctrl.yaml#
> diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml
> index 5fa5d31f8866..88b2fa5e684d 100644
> --- a/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml
> @@ -88,7 +88,7 @@ additionalProperties:
>  
>      - type: object
>        additionalProperties:
> -        $ref: "#/additionalProperties/anyOf/0"
> +        $ref: '#/additionalProperties/anyOf/0'
>  
>  allOf:
>    - $ref: pinctrl.yaml#
> diff --git a/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml b/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml
> index fba6914ec40d..948a9da111df 100644
> --- a/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml
> +++ b/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml
> @@ -45,7 +45,7 @@ properties:
>          const: 0
>  
>      additionalProperties:
> -      $ref: "#/$defs/pd-node"
> +      $ref: '#/$defs/pd-node'
>  
>  required:
>    - compatible
> @@ -83,7 +83,7 @@ $defs:
>        - '#power-domain-cells'
>  
>      additionalProperties:
> -      $ref: "#/$defs/pd-node"
> +      $ref: '#/$defs/pd-node'
>  
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> index 2c7275c4503b..abf1adca0773 100644
> --- a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> +++ b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> @@ -57,7 +57,7 @@ properties:
>      const: 0
>  
>  patternProperties:
> -  "^timer@[0-2]$":
> +  '^timer@[0-2]$':
>      description: The timer block channels that are used as timers or counters.
>      type: object
>      additionalProperties: false
> @@ -80,7 +80,7 @@ patternProperties:
>        - compatible
>        - reg
>  
> -  "^pwm@[0-2]$":
> +  '^pwm@[0-2]$':
>      description: The timer block channels that are used as PWMs.
>      $ref: /schemas/pwm/pwm.yaml#
>      type: object
> @@ -92,7 +92,7 @@ patternProperties:
>            TCB channel to use for this PWM.
>          enum: [ 0, 1, 2 ]
>  
> -      "#pwm-cells":
> +      '#pwm-cells':
>          description:
>            The only third cell flag supported by this binding is
>            PWM_POLARITY_INVERTED.
> @@ -101,7 +101,7 @@ patternProperties:
>      required:
>        - compatible
>        - reg
> -      - "#pwm-cells"
> +      - '#pwm-cells'
>  
>      additionalProperties: false
>  
> diff --git a/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml b/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml
> index 7140c312d898..f516960dbbef 100644
> --- a/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml
> +++ b/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml
> @@ -133,12 +133,12 @@ properties:
>            property. The supported-hw is a bitfield indicating SoC speedo or
>            process ID mask.
>  
> -      "#power-domain-cells":
> +      '#power-domain-cells':
>          const: 0
>  
>      required:
>        - operating-points-v2
> -      - "#power-domain-cells"
> +      - '#power-domain-cells'
>  
>    i2c-thermtrip:
>      type: object
> @@ -220,7 +220,7 @@ properties:
>          xusbc    USB Partition C               Tegra114/124/210
>  
>      patternProperties:
> -      "^[a-z0-9]+$":
> +      '^[a-z0-9]+$':
>          type: object
>          additionalProperties: false
>          properties:
> @@ -365,9 +365,9 @@ allOf:
>  additionalProperties: false
>  
>  dependencies:
> -  nvidia,suspend-mode: ["nvidia,core-pwr-off-time", "nvidia,cpu-pwr-off-time"]
> -  nvidia,core-pwr-off-time: ["nvidia,core-pwr-good-time"]
> -  nvidia,cpu-pwr-off-time: ["nvidia,cpu-pwr-good-time"]
> +  nvidia,suspend-mode: ['nvidia,core-pwr-off-time', 'nvidia,cpu-pwr-off-time']
> +  nvidia,core-pwr-off-time: ['nvidia,core-pwr-good-time']
> +  nvidia,cpu-pwr-off-time: ['nvidia,cpu-pwr-good-time']
>  
>  examples:
>    - |
> -- 
> 2.51.0
> 

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
  2025-10-16  0:26 ` Rob Herring (Arm)
  2025-10-16  6:49 ` Alexandre Belloni
@ 2025-10-16  6:57 ` Krzysztof Kozlowski
  2025-10-16  7:56 ` Geert Uytterhoeven
                   ` (7 subsequent siblings)
  10 siblings, 0 replies; 15+ messages in thread
From: Krzysztof Kozlowski @ 2025-10-16  6:57 UTC (permalink / raw)
  To: Rob Herring (Arm), Krzysztof Kozlowski, Conor Dooley,
	Stephen Boyd, Thierry Reding, Linus Walleij, Bartosz Golaszewski,
	Shawn Guo, Fabio Estevam, Nuno Sá, Lars-Peter Clausen,
	Michael Hennerich, Jonathan Cameron, Andy Shevchenko, Jassi Brar,
	Mauro Carvalho Chehab, Lee Jones, Joel Stanley, Andrew Jeffery,
	Andrew Lunn, Vladimir Oltean, David S. Miller, Eric Dumazet,
	Jakub Kicinski, Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Alexandre Belloni,
	Claudiu Beznea, Florian Fainelli, Tony Lindgren
  Cc: devicetree, linux-kernel, linux-clk, linux-gpio, linux-arm-kernel,
	linux-iio, linux-media, netdev, linux-pci, linux-phy

On 16/10/2025 01:16, Rob Herring (Arm) wrote:
> yamllint has gained a new check which checks for inconsistent quoting
> (mixed " and ' quotes within a file). Fix all the cases yamllint found
> so we can enable the check (once the check is in a release). Use
> whichever quoting is dominate in the file.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof

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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
                   ` (2 preceding siblings ...)
  2025-10-16  6:57 ` Krzysztof Kozlowski
@ 2025-10-16  7:56 ` Geert Uytterhoeven
  2025-10-16 11:46   ` Rob Herring
  2025-10-16 10:53 ` Jonathan Cameron
                   ` (6 subsequent siblings)
  10 siblings, 1 reply; 15+ messages in thread
From: Geert Uytterhoeven @ 2025-10-16  7:56 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Linus Walleij, Bartosz Golaszewski, Shawn Guo, Fabio Estevam,
	Nuno Sá, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Andy Shevchenko, Jassi Brar,
	Mauro Carvalho Chehab, Lee Jones, Joel Stanley, Andrew Jeffery,
	Andrew Lunn, Vladimir Oltean, David S. Miller, Eric Dumazet,
	Jakub Kicinski, Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Alexandre Belloni,
	Claudiu Beznea, Florian Fainelli, Tony Lindgren, devicetree,
	linux-kernel, linux-clk, linux-gpio, linux-arm-kernel, linux-iio,
	linux-media, netdev, linux-pci, linux-phy

Hi Rob,

On Thu, 16 Oct 2025 at 01:20, Rob Herring (Arm) <robh@kernel.org> wrote:
> yamllint has gained a new check which checks for inconsistent quoting
> (mixed " and ' quotes within a file). Fix all the cases yamllint found
> so we can enable the check (once the check is in a release). Use
> whichever quoting is dominate in the file.
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Thanks for your patch!

Since you are mentioning mixed quotes, is one or the other preferred?
Shouldn't we try to be consistent across all files?

> --- a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> @@ -129,7 +129,7 @@ additionalProperties:
>
>      - type: object
>        additionalProperties:
> -        $ref: "#/additionalProperties/anyOf/0"
> +        $ref: '#/additionalProperties/anyOf/0'
>
>  examples:
>    - |
> @@ -190,7 +190,7 @@ examples:
>
>              sdhi0_pins: sd0 {
>                      groups = "sdhi0_data4", "sdhi0_ctrl";
> -                    function = "sdhi0";
> +                    function = "sdhi0';

Huh?

>                      power-source = <3300>;
>              };
>      };

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
                   ` (3 preceding siblings ...)
  2025-10-16  7:56 ` Geert Uytterhoeven
@ 2025-10-16 10:53 ` Jonathan Cameron
  2025-10-16 15:38 ` Frank Li
                   ` (5 subsequent siblings)
  10 siblings, 0 replies; 15+ messages in thread
From: Jonathan Cameron @ 2025-10-16 10:53 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Linus Walleij, Bartosz Golaszewski, Shawn Guo, Fabio Estevam,
	Nuno Sá, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Andy Shevchenko, Jassi Brar,
	Mauro Carvalho Chehab, Lee Jones, Joel Stanley, Andrew Jeffery,
	Andrew Lunn, Vladimir Oltean, David S. Miller, Eric Dumazet,
	Jakub Kicinski, Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Alexandre Belloni,
	Claudiu Beznea, Florian Fainelli, Tony Lindgren, devicetree,
	linux-kernel, linux-clk, linux-gpio, linux-arm-kernel, linux-iio,
	linux-media, netdev, linux-pci, linux-phy

On Wed, 15 Oct 2025 18:16:24 -0500
"Rob Herring (Arm)" <robh@kernel.org> wrote:

> yamllint has gained a new check which checks for inconsistent quoting
> (mixed " and ' quotes within a file). Fix all the cases yamllint found
> so we can enable the check (once the check is in a release). Use
> whichever quoting is dominate in the file.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../arm/altera/socfpga-clk-manager.yaml       |  4 ++--
>  .../bindings/clock/nvidia,tegra124-car.yaml   |  8 ++++----
>  .../bindings/clock/nvidia,tegra20-car.yaml    |  6 +++---
>  .../devicetree/bindings/gpio/gpio-mxs.yaml    |  9 +++++----
>  .../bindings/gpio/snps,dw-apb-gpio.yaml       |  4 ++--
>  .../bindings/iio/temperature/adi,ltc2983.yaml | 20 +++++++++----------
For this one
Acked-by: Jonathan Cameron <jonathan.cameron@huawei.com>


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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-16  7:56 ` Geert Uytterhoeven
@ 2025-10-16 11:46   ` Rob Herring
  2025-10-16 12:06     ` Geert Uytterhoeven
  0 siblings, 1 reply; 15+ messages in thread
From: Rob Herring @ 2025-10-16 11:46 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Linus Walleij, Bartosz Golaszewski, Shawn Guo, Fabio Estevam,
	Nuno Sá, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Andy Shevchenko, Jassi Brar,
	Mauro Carvalho Chehab, Lee Jones, Joel Stanley, Andrew Jeffery,
	Andrew Lunn, Vladimir Oltean, David S. Miller, Eric Dumazet,
	Jakub Kicinski, Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Alexandre Belloni,
	Claudiu Beznea, Florian Fainelli, Tony Lindgren, devicetree,
	linux-kernel, linux-clk, linux-gpio, linux-arm-kernel, linux-iio,
	linux-media, netdev, linux-pci, linux-phy

On Thu, Oct 16, 2025 at 2:57 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>
> Hi Rob,
>
> On Thu, 16 Oct 2025 at 01:20, Rob Herring (Arm) <robh@kernel.org> wrote:
> > yamllint has gained a new check which checks for inconsistent quoting
> > (mixed " and ' quotes within a file). Fix all the cases yamllint found
> > so we can enable the check (once the check is in a release). Use
> > whichever quoting is dominate in the file.
> >
> > Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
>
> Thanks for your patch!
>
> Since you are mentioning mixed quotes, is one or the other preferred?

I have a slight preference for single quotes.

> Shouldn't we try to be consistent across all files?

I don't particularly care to change 915 files. And if the tools don't
enforce it, I'm not going to do so in reviews.

> > --- a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> > +++ b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> > @@ -129,7 +129,7 @@ additionalProperties:
> >
> >      - type: object
> >        additionalProperties:
> > -        $ref: "#/additionalProperties/anyOf/0"
> > +        $ref: '#/additionalProperties/anyOf/0'
> >
> >  examples:
> >    - |
> > @@ -190,7 +190,7 @@ examples:
> >
> >              sdhi0_pins: sd0 {
> >                      groups = "sdhi0_data4", "sdhi0_ctrl";
> > -                    function = "sdhi0";
> > +                    function = "sdhi0';
>
> Huh?

One too many search and replace. Fixed.

Rob

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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-16 11:46   ` Rob Herring
@ 2025-10-16 12:06     ` Geert Uytterhoeven
  0 siblings, 0 replies; 15+ messages in thread
From: Geert Uytterhoeven @ 2025-10-16 12:06 UTC (permalink / raw)
  To: Rob Herring
  Cc: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Linus Walleij, Bartosz Golaszewski, Shawn Guo, Fabio Estevam,
	Nuno Sá, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Andy Shevchenko, Jassi Brar,
	Mauro Carvalho Chehab, Lee Jones, Joel Stanley, Andrew Jeffery,
	Andrew Lunn, Vladimir Oltean, David S. Miller, Eric Dumazet,
	Jakub Kicinski, Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Alexandre Belloni,
	Claudiu Beznea, Florian Fainelli, Tony Lindgren, devicetree,
	linux-kernel, linux-clk, linux-gpio, linux-arm-kernel, linux-iio,
	linux-media, netdev, linux-pci, linux-phy

Hi Rob,

On Thu, 16 Oct 2025 at 13:46, Rob Herring <robh@kernel.org> wrote:
> On Thu, Oct 16, 2025 at 2:57 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > On Thu, 16 Oct 2025 at 01:20, Rob Herring (Arm) <robh@kernel.org> wrote:
> > > yamllint has gained a new check which checks for inconsistent quoting
> > > (mixed " and ' quotes within a file). Fix all the cases yamllint found
> > > so we can enable the check (once the check is in a release). Use
> > > whichever quoting is dominate in the file.
> > >
> > > Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> >
> > Thanks for your patch!
> >
> > Since you are mentioning mixed quotes, is one or the other preferred?
>
> I have a slight preference for single quotes.

OK, so outside human-readable descriptions, there should only be double
quotes in property values, i.e. on lines ending with a comma or a
semicolon.  Sounds like that can be scripted, or validated by scripting.

> > Shouldn't we try to be consistent across all files?
>
> I don't particularly care to change 915 files. And if the tools don't
> enforce it, I'm not going to do so in reviews.

Fair enough.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
                   ` (4 preceding siblings ...)
  2025-10-16 10:53 ` Jonathan Cameron
@ 2025-10-16 15:38 ` Frank Li
  2025-10-16 16:06   ` Rob Herring
  2025-10-17  6:26 ` Andrew Jeffery
                   ` (4 subsequent siblings)
  10 siblings, 1 reply; 15+ messages in thread
From: Frank Li @ 2025-10-16 15:38 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Linus Walleij, Bartosz Golaszewski, Shawn Guo, Fabio Estevam,
	Nuno Sá, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Andy Shevchenko, Jassi Brar,
	Mauro Carvalho Chehab, Lee Jones, Joel Stanley, Andrew Jeffery,
	Andrew Lunn, Vladimir Oltean, David S. Miller, Eric Dumazet,
	Jakub Kicinski, Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Alexandre Belloni,
	Claudiu Beznea, Florian Fainelli, Tony Lindgren, devicetree,
	linux-kernel, linux-clk, linux-gpio, linux-arm-kernel, linux-iio,
	linux-media, netdev, linux-pci, linux-phy

On Wed, Oct 15, 2025 at 06:16:24PM -0500, Rob Herring (Arm) wrote:
> yamllint has gained a new check which checks for inconsistent quoting
> (mixed " and ' quotes within a file). Fix all the cases yamllint found
> so we can enable the check (once the check is in a release). Use
> whichever quoting is dominate in the file.

Can we simple require only use one of " or ' to let everyone follow easily?
support both " or ' is unneccessary options.

like hexvalue in dts should be lowcase.

Frank

>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../arm/altera/socfpga-clk-manager.yaml       |  4 ++--
>  .../bindings/clock/nvidia,tegra124-car.yaml   |  8 ++++----
>  .../bindings/clock/nvidia,tegra20-car.yaml    |  6 +++---
>  .../devicetree/bindings/gpio/gpio-mxs.yaml    |  9 +++++----
>  .../bindings/gpio/snps,dw-apb-gpio.yaml       |  4 ++--
>  .../bindings/iio/temperature/adi,ltc2983.yaml | 20 +++++++++----------
>  .../mailbox/qcom,apcs-kpss-global.yaml        | 16 +++++++--------
>  .../mailbox/xlnx,zynqmp-ipi-mailbox.yaml      |  2 +-
>  .../bindings/media/fsl,imx6q-vdoa.yaml        |  2 +-
>  .../devicetree/bindings/mfd/aspeed-lpc.yaml   |  4 ++--
>  .../devicetree/bindings/mfd/ti,twl.yaml       |  4 ++--
>  .../bindings/net/ethernet-switch.yaml         |  2 +-
>  .../pci/plda,xpressrich3-axi-common.yaml      |  2 +-
>  .../bindings/phy/motorola,cpcap-usb-phy.yaml  |  4 ++--
>  .../pinctrl/microchip,sparx5-sgpio.yaml       | 12 +++++------
>  .../bindings/pinctrl/qcom,pmic-gpio.yaml      | 10 +++++-----
>  .../bindings/pinctrl/qcom,pmic-mpp.yaml       |  6 +++---
>  .../bindings/pinctrl/renesas,pfc.yaml         |  4 ++--
>  .../bindings/pinctrl/renesas,rza1-ports.yaml  |  2 +-
>  .../pinctrl/renesas,rzg2l-pinctrl.yaml        |  2 +-
>  .../pinctrl/renesas,rzv2m-pinctrl.yaml        |  2 +-
>  .../bindings/power/renesas,sysc-rmobile.yaml  |  4 ++--
>  .../soc/microchip/atmel,at91rm9200-tcb.yaml   |  8 ++++----
>  .../soc/tegra/nvidia,tegra20-pmc.yaml         | 12 +++++------
>  24 files changed, 75 insertions(+), 74 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml b/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
> index a758f4bb2bb3..275554bfedce 100644
> --- a/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
> +++ b/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
> @@ -39,7 +39,7 @@ properties:
>
>        "^[a-z0-9,_]+(clk|pll|clk_gate|clk_divided)(@[a-f0-9]+)?$":
>          type: object
> -        $ref: '#/$defs/clock-props'
> +        $ref: "#/$defs/clock-props"
>          unevaluatedProperties: false
>
>          properties:
> @@ -67,7 +67,7 @@ properties:
>          patternProperties:
>            "^[a-z0-9,_]+(clk|pll)(@[a-f0-9]+)?$":
>              type: object
> -            $ref: '#/$defs/clock-props'
> +            $ref: "#/$defs/clock-props"
>              unevaluatedProperties: false
>
>              properties:
> diff --git a/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml b/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml
> index a9ba21144a56..13bb616249a1 100644
> --- a/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml
> +++ b/Documentation/devicetree/bindings/clock/nvidia,tegra124-car.yaml
> @@ -37,7 +37,7 @@ properties:
>    '#clock-cells':
>      const: 1
>
> -  "#reset-cells":
> +  '#reset-cells':
>      const: 1
>
>    nvidia,external-memory-controller:
> @@ -46,7 +46,7 @@ properties:
>        phandle of the external memory controller node
>
>  patternProperties:
> -  "^emc-timings-[0-9]+$":
> +  '^emc-timings-[0-9]+$':
>      type: object
>      properties:
>        nvidia,ram-code:
> @@ -56,7 +56,7 @@ patternProperties:
>            this timing set is used for
>
>      patternProperties:
> -      "^timing-[0-9]+$":
> +      '^timing-[0-9]+$':
>          type: object
>          properties:
>            clock-frequency:
> @@ -94,7 +94,7 @@ required:
>    - compatible
>    - reg
>    - '#clock-cells'
> -  - "#reset-cells"
> +  - '#reset-cells'
>
>  additionalProperties: false
>
> diff --git a/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml b/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml
> index bee2dd4b29bf..73cccc0df424 100644
> --- a/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml
> +++ b/Documentation/devicetree/bindings/clock/nvidia,tegra20-car.yaml
> @@ -39,11 +39,11 @@ properties:
>    '#clock-cells':
>      const: 1
>
> -  "#reset-cells":
> +  '#reset-cells':
>      const: 1
>
>  patternProperties:
> -  "^(sclk)|(pll-[cem])$":
> +  '^(sclk)|(pll-[cem])$':
>      type: object
>      properties:
>        compatible:
> @@ -76,7 +76,7 @@ required:
>    - compatible
>    - reg
>    - '#clock-cells'
> -  - "#reset-cells"
> +  - '#reset-cells'
>
>  additionalProperties: false
>
> diff --git a/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml b/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml
> index aaf97124803f..4b5b8e794613 100644
> --- a/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml
> +++ b/Documentation/devicetree/bindings/gpio/gpio-mxs.yaml
> @@ -26,9 +26,10 @@ properties:
>        # Devices. Keep it as it to be compatible existed dts files.
>        - const: simple-bus
>
> -  '#address-cells':
> +  "#address-cells":
>      const: 1
> -  '#size-cells':
> +
> +  "#size-cells":
>      const: 0
>
>    reg:
> @@ -132,8 +133,8 @@ patternProperties:
>  required:
>    - compatible
>    - reg
> -  - '#address-cells'
> -  - '#size-cells'
> +  - "#address-cells"
> +  - "#size-cells"
>
>  additionalProperties: false
>
> diff --git a/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml b/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
> index ab2afc0e4153..bba6f5b6606f 100644
> --- a/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
> +++ b/Documentation/devicetree/bindings/gpio/snps,dw-apb-gpio.yaml
> @@ -111,8 +111,8 @@ additionalProperties: false
>  required:
>    - compatible
>    - reg
> -  - "#address-cells"
> -  - "#size-cells"
> +  - '#address-cells'
> +  - '#size-cells'
>
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml b/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
> index 312febeeb3bb..ee0b558bb866 100644
> --- a/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
> +++ b/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
> @@ -88,7 +88,7 @@ properties:
>      const: 0
>
>  patternProperties:
> -  "^thermocouple@":
> +  '^thermocouple@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>
> @@ -146,7 +146,7 @@ patternProperties:
>            required:
>              - adi,custom-thermocouple
>
> -  "^diode@":
> +  '^diode@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>
> @@ -191,7 +191,7 @@ patternProperties:
>          $ref: /schemas/types.yaml#/definitions/uint32
>          default: 0
>
> -  "^rtd@":
> +  '^rtd@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>      description: RTD sensor.
> @@ -280,7 +280,7 @@ patternProperties:
>                type: boolean
>
>            dependencies:
> -            adi,current-rotate: [ "adi,rsense-share" ]
> +            adi,current-rotate: [ 'adi,rsense-share' ]
>
>        - if:
>            properties:
> @@ -290,7 +290,7 @@ patternProperties:
>            required:
>              - adi,custom-rtd
>
> -  "^thermistor@":
> +  '^thermistor@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>      description: Thermistor sensor.
> @@ -364,7 +364,7 @@ patternProperties:
>        - adi,rsense-handle
>
>      dependencies:
> -      adi,current-rotate: [ "adi,rsense-share" ]
> +      adi,current-rotate: [ 'adi,rsense-share' ]
>
>      allOf:
>        - if:
> @@ -392,7 +392,7 @@ patternProperties:
>            required:
>              - adi,custom-thermistor
>
> -  "^adc@":
> +  '^adc@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>      description: Direct ADC sensor.
> @@ -407,7 +407,7 @@ patternProperties:
>          description: Whether the sensor is single-ended.
>          type: boolean
>
> -  "^temp@":
> +  '^temp@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>      description: Active analog temperature sensor.
> @@ -437,7 +437,7 @@ patternProperties:
>      required:
>        - adi,custom-temp
>
> -  "^rsense@":
> +  '^rsense@':
>      $ref: '#/$defs/sensor-node'
>      unevaluatedProperties: false
>      description: Sense resistor sensor.
> @@ -476,7 +476,7 @@ allOf:
>                - adi,ltc2984
>      then:
>        patternProperties:
> -        "^temp@": false
> +        '^temp@': false
>
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
> index 615ed103b7e6..f40dc9048327 100644
> --- a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
> +++ b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml
> @@ -187,10 +187,10 @@ allOf:
>              enum:
>                - qcom,msm8916-apcs-kpss-global
>      then:
> -      $ref: "#/$defs/msm8916-apcs-clock-controller"
> +      $ref: '#/$defs/msm8916-apcs-clock-controller'
>        properties:
>          clock-controller:
> -          $ref: "#/$defs/msm8916-apcs-clock-controller"
> +          $ref: '#/$defs/msm8916-apcs-clock-controller'
>
>    - if:
>        properties:
> @@ -199,10 +199,10 @@ allOf:
>              enum:
>                - qcom,msm8939-apcs-kpss-global
>      then:
> -      $ref: "#/$defs/msm8939-apcs-clock-controller"
> +      $ref: '#/$defs/msm8939-apcs-clock-controller'
>        properties:
>          clock-controller:
> -          $ref: "#/$defs/msm8939-apcs-clock-controller"
> +          $ref: '#/$defs/msm8939-apcs-clock-controller'
>
>    - if:
>        properties:
> @@ -211,10 +211,10 @@ allOf:
>              enum:
>                - qcom,sdx55-apcs-gcc
>      then:
> -      $ref: "#/$defs/sdx55-apcs-clock-controller"
> +      $ref: '#/$defs/sdx55-apcs-clock-controller'
>        properties:
>          clock-controller:
> -          $ref: "#/$defs/sdx55-apcs-clock-controller"
> +          $ref: '#/$defs/sdx55-apcs-clock-controller'
>
>    - if:
>        properties:
> @@ -223,10 +223,10 @@ allOf:
>              enum:
>                - qcom,ipq6018-apcs-apps-global
>      then:
> -      $ref: "#/$defs/ipq6018-apcs-clock-controller"
> +      $ref: '#/$defs/ipq6018-apcs-clock-controller'
>        properties:
>          clock-controller:
> -          $ref: "#/$defs/ipq6018-apcs-clock-controller"
> +          $ref: '#/$defs/ipq6018-apcs-clock-controller'
>
>    - if:
>        properties:
> diff --git a/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml
> index fe83b5cb1278..04d6473d666f 100644
> --- a/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml
> +++ b/Documentation/devicetree/bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml
> @@ -142,7 +142,7 @@ patternProperties:
>        - compatible
>        - reg
>        - reg-names
> -      - "#mbox-cells"
> +      - '#mbox-cells'
>        - xlnx,ipi-id
>
>  required:
> diff --git a/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml b/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml
> index 511ac0d67a7f..988a5b3a62bd 100644
> --- a/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml
> +++ b/Documentation/devicetree/bindings/media/fsl,imx6q-vdoa.yaml
> @@ -16,7 +16,7 @@ maintainers:
>
>  properties:
>    compatible:
> -    const: "fsl,imx6q-vdoa"
> +    const: fsl,imx6q-vdoa
>
>    reg:
>      maxItems: 1
> diff --git a/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml b/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml
> index f329223cec07..0adfeef149e7 100644
> --- a/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml
> +++ b/Documentation/devicetree/bindings/mfd/aspeed-lpc.yaml
> @@ -111,12 +111,12 @@ patternProperties:
>        reg:
>          maxItems: 1
>
> -      '#reset-cells':
> +      "#reset-cells":
>          const: 1
>
>      required:
>        - compatible
> -      - '#reset-cells'
> +      - "#reset-cells"
>
>    "^lpc-snoop@[0-9a-f]+$":
>      type: object
> diff --git a/Documentation/devicetree/bindings/mfd/ti,twl.yaml b/Documentation/devicetree/bindings/mfd/ti,twl.yaml
> index 776b04e182cb..1611b1581a8e 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,twl.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,twl.yaml
> @@ -400,7 +400,7 @@ properties:
>        - '#pwm-cells'
>
>  patternProperties:
> -  "^regulator-":
> +  '^regulator-':
>      type: object
>      unevaluatedProperties: false
>      $ref: /schemas/regulator/regulator.yaml
> @@ -429,7 +429,7 @@ required:
>    - reg
>    - interrupts
>    - interrupt-controller
> -  - "#interrupt-cells"
> +  - '#interrupt-cells'
>
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/net/ethernet-switch.yaml b/Documentation/devicetree/bindings/net/ethernet-switch.yaml
> index b3b7e1a1b127..03001ba40e0f 100644
> --- a/Documentation/devicetree/bindings/net/ethernet-switch.yaml
> +++ b/Documentation/devicetree/bindings/net/ethernet-switch.yaml
> @@ -72,7 +72,7 @@ additionalProperties: true
>  $defs:
>    ethernet-ports:
>      description: An ethernet switch without any extra port properties
> -    $ref: '#'
> +    $ref: "#"
>
>      patternProperties:
>        "^(ethernet-)?ports$":
> diff --git a/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml b/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml
> index 039eecdbd6aa..fe2e8beb5bab 100644
> --- a/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml
> +++ b/Documentation/devicetree/bindings/pci/plda,xpressrich3-axi-common.yaml
> @@ -72,7 +72,7 @@ required:
>    - reg-names
>    - interrupts
>    - msi-controller
> -  - "#interrupt-cells"
> +  - '#interrupt-cells'
>    - interrupt-map-mask
>    - interrupt-map
>
> diff --git a/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml b/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml
> index 0febd04a61f4..dd345cbd0a0b 100644
> --- a/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/motorola,cpcap-usb-phy.yaml
> @@ -67,8 +67,8 @@ properties:
>    mode-gpios:
>      description: Optional GPIOs for configuring alternate modes
>      items:
> -      - description: "mode selection GPIO #0"
> -      - description: "mode selection GPIO #1"
> +      - description: mode selection GPIO#0
> +      - description: mode selection GPIO#1
>
>  required:
>    - compatible
> diff --git a/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml b/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml
> index 0df4e114fdd6..fa47732d7cef 100644
> --- a/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/microchip,sparx5-sgpio.yaml
> @@ -18,7 +18,7 @@ description: |
>
>  properties:
>    $nodename:
> -    pattern: "^gpio@[0-9a-f]+$"
> +    pattern: '^gpio@[0-9a-f]+$'
>
>    compatible:
>      enum:
> @@ -26,10 +26,10 @@ properties:
>        - mscc,ocelot-sgpio
>        - mscc,luton-sgpio
>
> -  "#address-cells":
> +  '#address-cells':
>      const: 1
>
> -  "#size-cells":
> +  '#size-cells':
>      const: 0
>
>    reg:
> @@ -76,7 +76,7 @@ properties:
>        - const: switch
>
>  patternProperties:
> -  "^gpio@[0-1]$":
> +  '^gpio@[0-1]$':
>      type: object
>      properties:
>        compatible:
> @@ -132,8 +132,8 @@ required:
>    - reg
>    - clocks
>    - microchip,sgpio-port-ranges
> -  - "#address-cells"
> -  - "#size-cells"
> +  - '#address-cells'
> +  - '#size-cells'
>
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml
> index 5e6dfcc3fe9b..6632bcd037ba 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.yaml
> @@ -424,13 +424,13 @@ allOf:
>  patternProperties:
>    '-state$':
>      oneOf:
> -      - $ref: "#/$defs/qcom-pmic-gpio-state"
> +      - $ref: '#/$defs/qcom-pmic-gpio-state'
>        - patternProperties:
> -          "(pinconf|-pins)$":
> -            $ref: "#/$defs/qcom-pmic-gpio-state"
> +          '(pinconf|-pins)$':
> +            $ref: '#/$defs/qcom-pmic-gpio-state'
>          additionalProperties: false
>
> -  "-hog(-[0-9]+)?$":
> +  '-hog(-[0-9]+)?$':
>      type: object
>      required:
>        - gpio-hog
> @@ -503,7 +503,7 @@ $defs:
>                   - gpio1-gpio12 for pmxr2230
>
>          items:
> -          pattern: "^gpio([0-9]+)$"
> +          pattern: '^gpio([0-9]+)$'
>
>        function:
>          items:
> diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml
> index 9364ae05f3e6..daf4c1c03712 100644
> --- a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-mpp.yaml
> @@ -74,10 +74,10 @@ required:
>  patternProperties:
>    '-state$':
>      oneOf:
> -      - $ref: "#/$defs/qcom-pmic-mpp-state"
> +      - $ref: '#/$defs/qcom-pmic-mpp-state'
>        - patternProperties:
>            '-pins$':
> -            $ref: "#/$defs/qcom-pmic-mpp-state"
> +            $ref: '#/$defs/qcom-pmic-mpp-state'
>          additionalProperties: false
>
>  $defs:
> @@ -100,7 +100,7 @@ $defs:
>                   - mpp1-mpp4 for pma8084
>
>          items:
> -          pattern: "^mpp([0-9]+)$"
> +          pattern: '^mpp([0-9]+)$'
>
>        function:
>          items:
> diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> index cfe004573366..ab1cfe9dcde5 100644
> --- a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> @@ -129,7 +129,7 @@ additionalProperties:
>
>      - type: object
>        additionalProperties:
> -        $ref: "#/additionalProperties/anyOf/0"
> +        $ref: '#/additionalProperties/anyOf/0'
>
>  examples:
>    - |
> @@ -190,7 +190,7 @@ examples:
>
>              sdhi0_pins: sd0 {
>                      groups = "sdhi0_data4", "sdhi0_ctrl";
> -                    function = "sdhi0";
> +                    function = "sdhi0';
>                      power-source = <3300>;
>              };
>      };
> diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml
> index 2bd7d47d0fdb..737eb4e14090 100644
> --- a/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rza1-ports.yaml
> @@ -118,7 +118,7 @@ additionalProperties:
>
>      - type: object
>        additionalProperties:
> -        $ref: "#/additionalProperties/anyOf/0"
> +        $ref: '#/additionalProperties/anyOf/0'
>
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> index 5156d54b240b..00c05243b9a4 100644
> --- a/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> @@ -135,7 +135,7 @@ additionalProperties:
>
>      - type: object
>        additionalProperties:
> -        $ref: "#/additionalProperties/anyOf/0"
> +        $ref: '#/additionalProperties/anyOf/0'
>
>  allOf:
>    - $ref: pinctrl.yaml#
> diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml
> index 5fa5d31f8866..88b2fa5e684d 100644
> --- a/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rzv2m-pinctrl.yaml
> @@ -88,7 +88,7 @@ additionalProperties:
>
>      - type: object
>        additionalProperties:
> -        $ref: "#/additionalProperties/anyOf/0"
> +        $ref: '#/additionalProperties/anyOf/0'
>
>  allOf:
>    - $ref: pinctrl.yaml#
> diff --git a/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml b/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml
> index fba6914ec40d..948a9da111df 100644
> --- a/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml
> +++ b/Documentation/devicetree/bindings/power/renesas,sysc-rmobile.yaml
> @@ -45,7 +45,7 @@ properties:
>          const: 0
>
>      additionalProperties:
> -      $ref: "#/$defs/pd-node"
> +      $ref: '#/$defs/pd-node'
>
>  required:
>    - compatible
> @@ -83,7 +83,7 @@ $defs:
>        - '#power-domain-cells'
>
>      additionalProperties:
> -      $ref: "#/$defs/pd-node"
> +      $ref: '#/$defs/pd-node'
>
>  examples:
>    - |
> diff --git a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> index 2c7275c4503b..abf1adca0773 100644
> --- a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> +++ b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> @@ -57,7 +57,7 @@ properties:
>      const: 0
>
>  patternProperties:
> -  "^timer@[0-2]$":
> +  '^timer@[0-2]$':
>      description: The timer block channels that are used as timers or counters.
>      type: object
>      additionalProperties: false
> @@ -80,7 +80,7 @@ patternProperties:
>        - compatible
>        - reg
>
> -  "^pwm@[0-2]$":
> +  '^pwm@[0-2]$':
>      description: The timer block channels that are used as PWMs.
>      $ref: /schemas/pwm/pwm.yaml#
>      type: object
> @@ -92,7 +92,7 @@ patternProperties:
>            TCB channel to use for this PWM.
>          enum: [ 0, 1, 2 ]
>
> -      "#pwm-cells":
> +      '#pwm-cells':
>          description:
>            The only third cell flag supported by this binding is
>            PWM_POLARITY_INVERTED.
> @@ -101,7 +101,7 @@ patternProperties:
>      required:
>        - compatible
>        - reg
> -      - "#pwm-cells"
> +      - '#pwm-cells'
>
>      additionalProperties: false
>
> diff --git a/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml b/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml
> index 7140c312d898..f516960dbbef 100644
> --- a/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml
> +++ b/Documentation/devicetree/bindings/soc/tegra/nvidia,tegra20-pmc.yaml
> @@ -133,12 +133,12 @@ properties:
>            property. The supported-hw is a bitfield indicating SoC speedo or
>            process ID mask.
>
> -      "#power-domain-cells":
> +      '#power-domain-cells':
>          const: 0
>
>      required:
>        - operating-points-v2
> -      - "#power-domain-cells"
> +      - '#power-domain-cells'
>
>    i2c-thermtrip:
>      type: object
> @@ -220,7 +220,7 @@ properties:
>          xusbc    USB Partition C               Tegra114/124/210
>
>      patternProperties:
> -      "^[a-z0-9]+$":
> +      '^[a-z0-9]+$':
>          type: object
>          additionalProperties: false
>          properties:
> @@ -365,9 +365,9 @@ allOf:
>  additionalProperties: false
>
>  dependencies:
> -  nvidia,suspend-mode: ["nvidia,core-pwr-off-time", "nvidia,cpu-pwr-off-time"]
> -  nvidia,core-pwr-off-time: ["nvidia,core-pwr-good-time"]
> -  nvidia,cpu-pwr-off-time: ["nvidia,cpu-pwr-good-time"]
> +  nvidia,suspend-mode: ['nvidia,core-pwr-off-time', 'nvidia,cpu-pwr-off-time']
> +  nvidia,core-pwr-off-time: ['nvidia,core-pwr-good-time']
> +  nvidia,cpu-pwr-off-time: ['nvidia,cpu-pwr-good-time']
>
>  examples:
>    - |
> --
> 2.51.0
>

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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-16 15:38 ` Frank Li
@ 2025-10-16 16:06   ` Rob Herring
  0 siblings, 0 replies; 15+ messages in thread
From: Rob Herring @ 2025-10-16 16:06 UTC (permalink / raw)
  To: Frank Li
  Cc: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Linus Walleij, Bartosz Golaszewski, Shawn Guo, Fabio Estevam,
	Nuno Sá, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Andy Shevchenko, Jassi Brar,
	Mauro Carvalho Chehab, Lee Jones, Joel Stanley, Andrew Jeffery,
	Andrew Lunn, Vladimir Oltean, David S. Miller, Eric Dumazet,
	Jakub Kicinski, Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Alexandre Belloni,
	Claudiu Beznea, Florian Fainelli, Tony Lindgren, devicetree,
	linux-kernel, linux-clk, linux-gpio, linux-arm-kernel, linux-iio,
	linux-media, netdev, linux-pci, linux-phy

On Thu, Oct 16, 2025 at 10:38 AM Frank Li <Frank.li@nxp.com> wrote:
>
> On Wed, Oct 15, 2025 at 06:16:24PM -0500, Rob Herring (Arm) wrote:
> > yamllint has gained a new check which checks for inconsistent quoting
> > (mixed " and ' quotes within a file). Fix all the cases yamllint found
> > so we can enable the check (once the check is in a release). Use
> > whichever quoting is dominate in the file.
>
> Can we simple require only use one of " or ' to let everyone follow easily?
> support both " or ' is unneccessary options.

I don't really care to fix 915 files. And don't send 100s of patches
for me to review either. Given we've got 5200 total, it's a good
chance folks will copy the preferred style.

Rob

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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
                   ` (5 preceding siblings ...)
  2025-10-16 15:38 ` Frank Li
@ 2025-10-17  6:26 ` Andrew Jeffery
  2025-10-18 19:05 ` Jassi Brar
                   ` (3 subsequent siblings)
  10 siblings, 0 replies; 15+ messages in thread
From: Andrew Jeffery @ 2025-10-17  6:26 UTC (permalink / raw)
  To: Rob Herring (Arm), Krzysztof Kozlowski, Conor Dooley,
	Stephen Boyd, Thierry Reding, Linus Walleij, Bartosz Golaszewski,
	Shawn Guo, Fabio Estevam, Nuno Sá, Lars-Peter Clausen,
	Michael Hennerich, Jonathan Cameron, Andy Shevchenko, Jassi Brar,
	Mauro Carvalho Chehab, Lee Jones, Joel Stanley, Andrew Lunn,
	Vladimir Oltean, David S. Miller, Eric Dumazet, Jakub Kicinski,
	Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Alexandre Belloni,
	Claudiu Beznea, Florian Fainelli, Tony Lindgren
  Cc: devicetree, linux-kernel, linux-clk, linux-gpio, linux-arm-kernel,
	linux-iio, linux-media, netdev, linux-pci, linux-phy

On Wed, 2025-10-15 at 18:16 -0500, Rob Herring (Arm) wrote:
> yamllint has gained a new check which checks for inconsistent quoting
> (mixed " and ' quotes within a file). Fix all the cases yamllint found
> so we can enable the check (once the check is in a release). Use
> whichever quoting is dominate in the file.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../arm/altera/socfpga-clk-manager.yaml       |  4 ++--
>  .../bindings/clock/nvidia,tegra124-car.yaml   |  8 ++++----
>  .../bindings/clock/nvidia,tegra20-car.yaml    |  6 +++---
>  .../devicetree/bindings/gpio/gpio-mxs.yaml    |  9 +++++----
>  .../bindings/gpio/snps,dw-apb-gpio.yaml       |  4 ++--
>  .../bindings/iio/temperature/adi,ltc2983.yaml | 20 +++++++++----------
>  .../mailbox/qcom,apcs-kpss-global.yaml        | 16 +++++++--------
>  .../mailbox/xlnx,zynqmp-ipi-mailbox.yaml      |  2 +-
>  .../bindings/media/fsl,imx6q-vdoa.yaml        |  2 +-
>  .../devicetree/bindings/mfd/aspeed-lpc.yaml   |  4 ++--

For aspeed-lpc.yaml:

Acked-by: Andrew Jeffery <andrew@codeconstruct.com.au>

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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
                   ` (6 preceding siblings ...)
  2025-10-17  6:26 ` Andrew Jeffery
@ 2025-10-18 19:05 ` Jassi Brar
  2025-10-20 21:35 ` Linus Walleij
                   ` (2 subsequent siblings)
  10 siblings, 0 replies; 15+ messages in thread
From: Jassi Brar @ 2025-10-18 19:05 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Linus Walleij, Bartosz Golaszewski, Shawn Guo, Fabio Estevam,
	Nuno Sá, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Andy Shevchenko, Mauro Carvalho Chehab,
	Lee Jones, Joel Stanley, Andrew Jeffery, Andrew Lunn,
	Vladimir Oltean, David S. Miller, Eric Dumazet, Jakub Kicinski,
	Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Alexandre Belloni,
	Claudiu Beznea, Florian Fainelli, Tony Lindgren, devicetree,
	linux-kernel, linux-clk, linux-gpio, linux-arm-kernel, linux-iio,
	linux-media, netdev, linux-pci, linux-phy

On Wed, Oct 15, 2025 at 6:20 PM Rob Herring (Arm) <robh@kernel.org> wrote:
>
> yamllint has gained a new check which checks for inconsistent quoting
> (mixed " and ' quotes within a file). Fix all the cases yamllint found
> so we can enable the check (once the check is in a release). Use
> whichever quoting is dominate in the file.
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../arm/altera/socfpga-clk-manager.yaml       |  4 ++--
>  .../bindings/clock/nvidia,tegra124-car.yaml   |  8 ++++----
>  .../bindings/clock/nvidia,tegra20-car.yaml    |  6 +++---
>  .../devicetree/bindings/gpio/gpio-mxs.yaml    |  9 +++++----
>  .../bindings/gpio/snps,dw-apb-gpio.yaml       |  4 ++--
>  .../bindings/iio/temperature/adi,ltc2983.yaml | 20 +++++++++----------
>  .../mailbox/qcom,apcs-kpss-global.yaml        | 16 +++++++--------
>  .../mailbox/xlnx,zynqmp-ipi-mailbox.yaml      |  2 +-
>  .../bindings/media/fsl,imx6q-vdoa.yaml        |  2 +-
>  .../devicetree/bindings/mfd/aspeed-lpc.yaml   |  4 ++--
>  .../devicetree/bindings/mfd/ti,twl.yaml       |  4 ++--
>  .../bindings/net/ethernet-switch.yaml         |  2 +-
>  .../pci/plda,xpressrich3-axi-common.yaml      |  2 +-
>  .../bindings/phy/motorola,cpcap-usb-phy.yaml  |  4 ++--
>  .../pinctrl/microchip,sparx5-sgpio.yaml       | 12 +++++------
>  .../bindings/pinctrl/qcom,pmic-gpio.yaml      | 10 +++++-----
>  .../bindings/pinctrl/qcom,pmic-mpp.yaml       |  6 +++---
>  .../bindings/pinctrl/renesas,pfc.yaml         |  4 ++--
>  .../bindings/pinctrl/renesas,rza1-ports.yaml  |  2 +-
>  .../pinctrl/renesas,rzg2l-pinctrl.yaml        |  2 +-
>  .../pinctrl/renesas,rzv2m-pinctrl.yaml        |  2 +-
>  .../bindings/power/renesas,sysc-rmobile.yaml  |  4 ++--
>  .../soc/microchip/atmel,at91rm9200-tcb.yaml   |  8 ++++----
>  .../soc/tegra/nvidia,tegra20-pmc.yaml         | 12 +++++------
>  24 files changed, 75 insertions(+), 74 deletions(-)
>
For the mailbox changes
  Acked-by: Jassi Brar <jassisinghbrar@gmail.com>

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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
                   ` (7 preceding siblings ...)
  2025-10-18 19:05 ` Jassi Brar
@ 2025-10-20 21:35 ` Linus Walleij
  2025-10-21 10:37 ` Lee Jones
  2025-10-21 12:40 ` Bartosz Golaszewski
  10 siblings, 0 replies; 15+ messages in thread
From: Linus Walleij @ 2025-10-20 21:35 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Bartosz Golaszewski, Shawn Guo, Fabio Estevam, Nuno Sá,
	Lars-Peter Clausen, Michael Hennerich, Jonathan Cameron,
	Andy Shevchenko, Jassi Brar, Mauro Carvalho Chehab, Lee Jones,
	Joel Stanley, Andrew Jeffery, Andrew Lunn, Vladimir Oltean,
	David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni,
	Daire McNamara, Lorenzo Pieralisi, Krzysztof Wilczyński,
	Manivannan Sadhasivam, Bjorn Helgaas, Vinod Koul,
	Kishon Vijay Abraham I, Bjorn Andersson, Geert Uytterhoeven,
	Nicolas Ferre, Alexandre Belloni, Claudiu Beznea,
	Florian Fainelli, Tony Lindgren, devicetree, linux-kernel,
	linux-clk, linux-gpio, linux-arm-kernel, linux-iio, linux-media,
	netdev, linux-pci, linux-phy

On Thu, Oct 16, 2025 at 1:20 AM Rob Herring (Arm) <robh@kernel.org> wrote:

> yamllint has gained a new check which checks for inconsistent quoting
> (mixed " and ' quotes within a file). Fix all the cases yamllint found
> so we can enable the check (once the check is in a release). Use
> whichever quoting is dominate in the file.
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij

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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
                   ` (8 preceding siblings ...)
  2025-10-20 21:35 ` Linus Walleij
@ 2025-10-21 10:37 ` Lee Jones
  2025-10-21 12:40 ` Bartosz Golaszewski
  10 siblings, 0 replies; 15+ messages in thread
From: Lee Jones @ 2025-10-21 10:37 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Linus Walleij, Bartosz Golaszewski, Shawn Guo, Fabio Estevam,
	Nuno Sá, Lars-Peter Clausen, Michael Hennerich,
	Jonathan Cameron, Andy Shevchenko, Jassi Brar,
	Mauro Carvalho Chehab, Joel Stanley, Andrew Jeffery, Andrew Lunn,
	Vladimir Oltean, David S. Miller, Eric Dumazet, Jakub Kicinski,
	Paolo Abeni, Daire McNamara, Lorenzo Pieralisi,
	Krzysztof Wilczyński, Manivannan Sadhasivam, Bjorn Helgaas,
	Vinod Koul, Kishon Vijay Abraham I, Bjorn Andersson,
	Geert Uytterhoeven, Nicolas Ferre, Alexandre Belloni,
	Claudiu Beznea, Florian Fainelli, Tony Lindgren, devicetree,
	linux-kernel, linux-clk, linux-gpio, linux-arm-kernel, linux-iio,
	linux-media, netdev, linux-pci, linux-phy

On Wed, 15 Oct 2025, Rob Herring (Arm) wrote:

> yamllint has gained a new check which checks for inconsistent quoting
> (mixed " and ' quotes within a file). Fix all the cases yamllint found
> so we can enable the check (once the check is in a release). Use
> whichever quoting is dominate in the file.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../arm/altera/socfpga-clk-manager.yaml       |  4 ++--
>  .../bindings/clock/nvidia,tegra124-car.yaml   |  8 ++++----
>  .../bindings/clock/nvidia,tegra20-car.yaml    |  6 +++---
>  .../devicetree/bindings/gpio/gpio-mxs.yaml    |  9 +++++----
>  .../bindings/gpio/snps,dw-apb-gpio.yaml       |  4 ++--
>  .../bindings/iio/temperature/adi,ltc2983.yaml | 20 +++++++++----------
>  .../mailbox/qcom,apcs-kpss-global.yaml        | 16 +++++++--------
>  .../mailbox/xlnx,zynqmp-ipi-mailbox.yaml      |  2 +-
>  .../bindings/media/fsl,imx6q-vdoa.yaml        |  2 +-

>  .../devicetree/bindings/mfd/aspeed-lpc.yaml   |  4 ++--
>  .../devicetree/bindings/mfd/ti,twl.yaml       |  4 ++--

Acked-by: Lee Jones <lee@kernel.org>

>  .../bindings/net/ethernet-switch.yaml         |  2 +-
>  .../pci/plda,xpressrich3-axi-common.yaml      |  2 +-
>  .../bindings/phy/motorola,cpcap-usb-phy.yaml  |  4 ++--
>  .../pinctrl/microchip,sparx5-sgpio.yaml       | 12 +++++------
>  .../bindings/pinctrl/qcom,pmic-gpio.yaml      | 10 +++++-----
>  .../bindings/pinctrl/qcom,pmic-mpp.yaml       |  6 +++---
>  .../bindings/pinctrl/renesas,pfc.yaml         |  4 ++--
>  .../bindings/pinctrl/renesas,rza1-ports.yaml  |  2 +-
>  .../pinctrl/renesas,rzg2l-pinctrl.yaml        |  2 +-
>  .../pinctrl/renesas,rzv2m-pinctrl.yaml        |  2 +-
>  .../bindings/power/renesas,sysc-rmobile.yaml  |  4 ++--
>  .../soc/microchip/atmel,at91rm9200-tcb.yaml   |  8 ++++----
>  .../soc/tegra/nvidia,tegra20-pmc.yaml         | 12 +++++------
>  24 files changed, 75 insertions(+), 74 deletions(-)

[...]

-- 
Lee Jones [李琼斯]

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

* Re: [PATCH] dt-bindings: Fix inconsistent quoting
  2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
                   ` (9 preceding siblings ...)
  2025-10-21 10:37 ` Lee Jones
@ 2025-10-21 12:40 ` Bartosz Golaszewski
  10 siblings, 0 replies; 15+ messages in thread
From: Bartosz Golaszewski @ 2025-10-21 12:40 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Conor Dooley, Stephen Boyd, Thierry Reding,
	Linus Walleij, Shawn Guo, Fabio Estevam, Nuno Sá,
	Lars-Peter Clausen, Michael Hennerich, Jonathan Cameron,
	Andy Shevchenko, Jassi Brar, Mauro Carvalho Chehab, Lee Jones,
	Joel Stanley, Andrew Jeffery, Andrew Lunn, Vladimir Oltean,
	David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni,
	Daire McNamara, Lorenzo Pieralisi, Krzysztof Wilczyński,
	Manivannan Sadhasivam, Bjorn Helgaas, Vinod Koul,
	Kishon Vijay Abraham I, Bjorn Andersson, Geert Uytterhoeven,
	Nicolas Ferre, Alexandre Belloni, Claudiu Beznea,
	Florian Fainelli, Tony Lindgren, devicetree, linux-kernel,
	linux-clk, linux-gpio, linux-arm-kernel, linux-iio, linux-media,
	netdev, linux-pci, linux-phy

On Thu, Oct 16, 2025 at 1:20 AM Rob Herring (Arm) <robh@kernel.org> wrote:
>
> yamllint has gained a new check which checks for inconsistent quoting
> (mixed " and ' quotes within a file). Fix all the cases yamllint found
> so we can enable the check (once the check is in a release). Use
> whichever quoting is dominate in the file.
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  .../arm/altera/socfpga-clk-manager.yaml       |  4 ++--
>  .../bindings/clock/nvidia,tegra124-car.yaml   |  8 ++++----
>  .../bindings/clock/nvidia,tegra20-car.yaml    |  6 +++---
>  .../devicetree/bindings/gpio/gpio-mxs.yaml    |  9 +++++----
>  .../bindings/gpio/snps,dw-apb-gpio.yaml       |  4 ++--
>  .../bindings/iio/temperature/adi,ltc2983.yaml | 20 +++++++++----------
>  .../mailbox/qcom,apcs-kpss-global.yaml        | 16 +++++++--------
>  .../mailbox/xlnx,zynqmp-ipi-mailbox.yaml      |  2 +-
>  .../bindings/media/fsl,imx6q-vdoa.yaml        |  2 +-
>  .../devicetree/bindings/mfd/aspeed-lpc.yaml   |  4 ++--
>  .../devicetree/bindings/mfd/ti,twl.yaml       |  4 ++--
>  .../bindings/net/ethernet-switch.yaml         |  2 +-
>  .../pci/plda,xpressrich3-axi-common.yaml      |  2 +-
>  .../bindings/phy/motorola,cpcap-usb-phy.yaml  |  4 ++--
>  .../pinctrl/microchip,sparx5-sgpio.yaml       | 12 +++++------
>  .../bindings/pinctrl/qcom,pmic-gpio.yaml      | 10 +++++-----
>  .../bindings/pinctrl/qcom,pmic-mpp.yaml       |  6 +++---
>  .../bindings/pinctrl/renesas,pfc.yaml         |  4 ++--
>  .../bindings/pinctrl/renesas,rza1-ports.yaml  |  2 +-
>  .../pinctrl/renesas,rzg2l-pinctrl.yaml        |  2 +-
>  .../pinctrl/renesas,rzv2m-pinctrl.yaml        |  2 +-
>  .../bindings/power/renesas,sysc-rmobile.yaml  |  4 ++--
>  .../soc/microchip/atmel,at91rm9200-tcb.yaml   |  8 ++++----
>  .../soc/tegra/nvidia,tegra20-pmc.yaml         | 12 +++++------

For GPIO:

Acked-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

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

end of thread, other threads:[~2025-10-21 12:41 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-10-15 23:16 [PATCH] dt-bindings: Fix inconsistent quoting Rob Herring (Arm)
2025-10-16  0:26 ` Rob Herring (Arm)
2025-10-16  6:49 ` Alexandre Belloni
2025-10-16  6:57 ` Krzysztof Kozlowski
2025-10-16  7:56 ` Geert Uytterhoeven
2025-10-16 11:46   ` Rob Herring
2025-10-16 12:06     ` Geert Uytterhoeven
2025-10-16 10:53 ` Jonathan Cameron
2025-10-16 15:38 ` Frank Li
2025-10-16 16:06   ` Rob Herring
2025-10-17  6:26 ` Andrew Jeffery
2025-10-18 19:05 ` Jassi Brar
2025-10-20 21:35 ` Linus Walleij
2025-10-21 10:37 ` Lee Jones
2025-10-21 12:40 ` Bartosz Golaszewski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).