* 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-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)
` (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-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