Re: [PATCH] dt-bindings: correct white-spaces in examples

From: Mathieu Poirier
Date: Mon Nov 27 2023 - 12:35:28 EST


On Fri, Nov 24, 2023 at 10:21:21AM +0100, Krzysztof Kozlowski wrote:
> Use only one and exactly one space around '=' in DTS example.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>
> ---
>
> Merging idea: Rob's DT.
> Should apply cleanly on Rob's for-next.
> ---
> .../devicetree/bindings/auxdisplay/hit,hd44780.yaml | 2 +-
> .../devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml | 2 +-
> Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml | 6 +++---
> .../devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml | 2 +-
> .../devicetree/bindings/iio/adc/qcom,spmi-rradc.yaml | 2 +-
> .../interrupt-controller/st,stih407-irq-syscfg.yaml | 4 ++--
> Documentation/devicetree/bindings/mmc/arm,pl18x.yaml | 2 +-
> Documentation/devicetree/bindings/net/sff,sfp.yaml | 2 +-
> .../devicetree/bindings/pci/toshiba,visconti-pcie.yaml | 2 +-
> .../bindings/pinctrl/renesas,rzg2l-pinctrl.yaml | 6 +++---
> .../devicetree/bindings/power/supply/richtek,rt9455.yaml | 8 ++++----
> .../devicetree/bindings/regulator/mps,mp5416.yaml | 4 ++--
> .../devicetree/bindings/regulator/mps,mpq7920.yaml | 4 ++--
> .../devicetree/bindings/remoteproc/fsl,imx-rproc.yaml | 8 ++++----

Reviewed-by: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx>

> 14 files changed, 27 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/auxdisplay/hit,hd44780.yaml b/Documentation/devicetree/bindings/auxdisplay/hit,hd44780.yaml
> index fde07e4b119d..406a922a714e 100644
> --- a/Documentation/devicetree/bindings/auxdisplay/hit,hd44780.yaml
> +++ b/Documentation/devicetree/bindings/auxdisplay/hit,hd44780.yaml
> @@ -113,7 +113,7 @@ examples:
> hd44780 {
> compatible = "hit,hd44780";
> display-height-chars = <2>;
> - display-width-chars = <16>;
> + display-width-chars = <16>;
> data-gpios = <&pcf8574 4 0>,
> <&pcf8574 5 0>,
> <&pcf8574 6 0>,
> diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> index 624984d51c10..7f8d98226437 100644
> --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> @@ -125,7 +125,7 @@ examples:
> clk25m: clock-oscillator-25m {
> compatible = "fixed-clock";
> #clock-cells = <0>;
> - clock-frequency = <25000000>;
> + clock-frequency = <25000000>;
> clock-output-names = "clk25m";
> };
> ...
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> index 5fcc8dd012f1..be2616ff9af6 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml
> @@ -80,9 +80,9 @@ examples:
> compatible = "adi,ad7780";
> reg = <0>;
>
> - avdd-supply = <&vdd_supply>;
> - powerdown-gpios = <&gpio0 12 GPIO_ACTIVE_HIGH>;
> - adi,gain-gpios = <&gpio1 5 GPIO_ACTIVE_LOW>;
> + avdd-supply = <&vdd_supply>;
> + powerdown-gpios = <&gpio0 12 GPIO_ACTIVE_HIGH>;
> + adi,gain-gpios = <&gpio1 5 GPIO_ACTIVE_LOW>;
> adi,filter-gpios = <&gpio2 15 GPIO_ACTIVE_LOW>;
> };
> };
> diff --git a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml
> index 73def67fbe01..b6a233cd5f6b 100644
> --- a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-iadc.yaml
> @@ -58,7 +58,7 @@ examples:
> reg = <0x3600>;
> interrupts = <0x0 0x36 0x0 IRQ_TYPE_EDGE_RISING>;
> qcom,external-resistor-micro-ohms = <10000>;
> - #io-channel-cells = <1>;
> + #io-channel-cells = <1>;
> };
> };
> ...
> diff --git a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-rradc.yaml b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-rradc.yaml
> index b3a626389870..64abe9a4cd9e 100644
> --- a/Documentation/devicetree/bindings/iio/adc/qcom,spmi-rradc.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/qcom,spmi-rradc.yaml
> @@ -46,6 +46,6 @@ examples:
> pmic_rradc: adc@4500 {
> compatible = "qcom,pmi8998-rradc";
> reg = <0x4500>;
> - #io-channel-cells = <1>;
> + #io-channel-cells = <1>;
> };
> };
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/st,stih407-irq-syscfg.yaml b/Documentation/devicetree/bindings/interrupt-controller/st,stih407-irq-syscfg.yaml
> index 2b153d7c5421..e44e4e5708a7 100644
> --- a/Documentation/devicetree/bindings/interrupt-controller/st,stih407-irq-syscfg.yaml
> +++ b/Documentation/devicetree/bindings/interrupt-controller/st,stih407-irq-syscfg.yaml
> @@ -55,8 +55,8 @@ examples:
> - |
> #include <dt-bindings/interrupt-controller/irq-st.h>
> irq-syscfg {
> - compatible = "st,stih407-irq-syscfg";
> - st,syscfg = <&syscfg_cpu>;
> + compatible = "st,stih407-irq-syscfg";
> + st,syscfg = <&syscfg_cpu>;
> st,irq-device = <ST_IRQ_SYSCFG_PMU_0>,
> <ST_IRQ_SYSCFG_PMU_1>;
> st,fiq-device = <ST_IRQ_SYSCFG_DISABLED>,
> diff --git a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml
> index 2459a55ed540..940b12688167 100644
> --- a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml
> +++ b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml
> @@ -203,7 +203,7 @@ examples:
> bus-width = <4>;
> cap-sd-highspeed;
> cap-mmc-highspeed;
> - cd-gpios = <&gpio2 31 0x4>;
> + cd-gpios = <&gpio2 31 0x4>;
> st,sig-dir-dat0;
> st,sig-dir-dat2;
> st,sig-dir-cmd;
> diff --git a/Documentation/devicetree/bindings/net/sff,sfp.yaml b/Documentation/devicetree/bindings/net/sff,sfp.yaml
> index 973e478a399d..bf6cbc7c2ba3 100644
> --- a/Documentation/devicetree/bindings/net/sff,sfp.yaml
> +++ b/Documentation/devicetree/bindings/net/sff,sfp.yaml
> @@ -120,7 +120,7 @@ examples:
> pinctrl-names = "default";
> pinctrl-0 = <&cps_sfpp0_pins>;
> tx-disable-gpios = <&cps_gpio1 29 GPIO_ACTIVE_HIGH>;
> - tx-fault-gpios = <&cps_gpio1 26 GPIO_ACTIVE_HIGH>;
> + tx-fault-gpios = <&cps_gpio1 26 GPIO_ACTIVE_HIGH>;
> };
>
> mdio {
> diff --git a/Documentation/devicetree/bindings/pci/toshiba,visconti-pcie.yaml b/Documentation/devicetree/bindings/pci/toshiba,visconti-pcie.yaml
> index 53da2edd7c9a..120e3bb1e545 100644
> --- a/Documentation/devicetree/bindings/pci/toshiba,visconti-pcie.yaml
> +++ b/Documentation/devicetree/bindings/pci/toshiba,visconti-pcie.yaml
> @@ -83,7 +83,7 @@ examples:
> <0x0 0x28050000 0x0 0x00010000>,
> <0x0 0x24200000 0x0 0x00002000>,
> <0x0 0x24162000 0x0 0x00001000>;
> - reg-names = "dbi", "config", "ulreg", "smu", "mpu";
> + reg-names = "dbi", "config", "ulreg", "smu", "mpu";
> device_type = "pci";
> bus-range = <0x00 0xff>;
> num-lanes = <2>;
> diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> index b5ca40d0e251..d476de82e5c3 100644
> --- a/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml
> @@ -185,17 +185,17 @@ examples:
> sd1_mux {
> pinmux = <RZG2L_PORT_PINMUX(19, 0, 1)>, /* CD */
> <RZG2L_PORT_PINMUX(19, 1, 1)>; /* WP */
> - power-source = <3300>;
> + power-source = <3300>;
> };
>
> sd1_data {
> pins = "SD1_DATA0", "SD1_DATA1", "SD1_DATA2", "SD1_DATA3";
> - power-source = <3300>;
> + power-source = <3300>;
> };
>
> sd1_ctrl {
> pins = "SD1_CLK", "SD1_CMD";
> - power-source = <3300>;
> + power-source = <3300>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/power/supply/richtek,rt9455.yaml b/Documentation/devicetree/bindings/power/supply/richtek,rt9455.yaml
> index 07e38be39f1b..89f9603499b4 100644
> --- a/Documentation/devicetree/bindings/power/supply/richtek,rt9455.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/richtek,rt9455.yaml
> @@ -79,10 +79,10 @@ examples:
> interrupt-parent = <&gpio1>;
> interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
>
> - richtek,output-charge-current = <500000>;
> - richtek,end-of-charge-percentage = <10>;
> - richtek,battery-regulation-voltage = <4200000>;
> - richtek,boost-output-voltage = <5050000>;
> + richtek,output-charge-current = <500000>;
> + richtek,end-of-charge-percentage = <10>;
> + richtek,battery-regulation-voltage = <4200000>;
> + richtek,boost-output-voltage = <5050000>;
>
> richtek,min-input-voltage-regulation = <4500000>;
> richtek,avg-input-current-regulation = <500000>;
> diff --git a/Documentation/devicetree/bindings/regulator/mps,mp5416.yaml b/Documentation/devicetree/bindings/regulator/mps,mp5416.yaml
> index 0221397eb51e..f825ee9efd81 100644
> --- a/Documentation/devicetree/bindings/regulator/mps,mp5416.yaml
> +++ b/Documentation/devicetree/bindings/regulator/mps,mp5416.yaml
> @@ -62,8 +62,8 @@ examples:
> regulator-name = "buck1";
> regulator-min-microvolt = <600000>;
> regulator-max-microvolt = <2187500>;
> - regulator-min-microamp = <3800000>;
> - regulator-max-microamp = <6800000>;
> + regulator-min-microamp = <3800000>;
> + regulator-max-microamp = <6800000>;
> regulator-boot-on;
> };
>
> diff --git a/Documentation/devicetree/bindings/regulator/mps,mpq7920.yaml b/Documentation/devicetree/bindings/regulator/mps,mpq7920.yaml
> index 6de5b027f990..0d34af98403f 100644
> --- a/Documentation/devicetree/bindings/regulator/mps,mpq7920.yaml
> +++ b/Documentation/devicetree/bindings/regulator/mps,mpq7920.yaml
> @@ -98,8 +98,8 @@ examples:
> regulator-name = "buck1";
> regulator-min-microvolt = <400000>;
> regulator-max-microvolt = <3587500>;
> - regulator-min-microamp = <460000>;
> - regulator-max-microamp = <7600000>;
> + regulator-min-microamp = <460000>;
> + regulator-max-microamp = <7600000>;
> regulator-boot-on;
> mps,buck-ovp-disable;
> mps,buck-phase-delay = /bits/ 8 <2>;
> diff --git a/Documentation/devicetree/bindings/remoteproc/fsl,imx-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/fsl,imx-rproc.yaml
> index 30632efdad8b..df36e29d974c 100644
> --- a/Documentation/devicetree/bindings/remoteproc/fsl,imx-rproc.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/fsl,imx-rproc.yaml
> @@ -113,10 +113,10 @@ examples:
> };
>
> imx7d-cm4 {
> - compatible = "fsl,imx7d-cm4";
> - memory-region = <&m4_reserved_sysmem1>, <&m4_reserved_sysmem2>;
> - syscon = <&src>;
> - clocks = <&clks IMX7D_ARM_M4_ROOT_CLK>;
> + compatible = "fsl,imx7d-cm4";
> + memory-region = <&m4_reserved_sysmem1>, <&m4_reserved_sysmem2>;
> + syscon = <&src>;
> + clocks = <&clks IMX7D_ARM_M4_ROOT_CLK>;
> };
>
> - |
> --
> 2.34.1
>
>