2023-07-02 18:48:28

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH] dt-bindings: cleanup DTS example whitespaces

The DTS code coding style expects spaces around '=' sign.

Signed-off-by: Krzysztof Kozlowski <[email protected]>

---

Rob,

Maybe this could go via your tree? Rebased on your for-next:
v6.4-rc2-45-gf0ac35049606
---
.../bindings/arm/arm,coresight-cti.yaml | 18 +++++++++---------
.../bindings/arm/keystone/ti,sci.yaml | 8 ++++----
.../devicetree/bindings/display/msm/gmu.yaml | 2 +-
.../display/panel/samsung,s6e8aa0.yaml | 2 +-
.../display/rockchip/rockchip-vop.yaml | 4 ++--
.../bindings/iio/adc/ti,adc108s102.yaml | 2 +-
.../bindings/media/renesas,rzg2l-cru.yaml | 4 ++--
.../devicetree/bindings/media/renesas,vin.yaml | 4 ++--
.../devicetree/bindings/mtd/mtd-physmap.yaml | 2 +-
.../bindings/net/mediatek-dwmac.yaml | 2 +-
.../bindings/perf/amlogic,g12-ddr-pmu.yaml | 4 ++--
.../bindings/phy/mediatek,dsi-phy.yaml | 2 +-
.../remoteproc/amlogic,meson-mx-ao-arc.yaml | 2 +-
.../devicetree/bindings/usb/mediatek,mtu3.yaml | 2 +-
.../devicetree/bindings/usb/ti,am62-usb.yaml | 2 +-
15 files changed, 30 insertions(+), 30 deletions(-)

diff --git a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
index 0c5b875cb654..d6c84b6e7fe6 100644
--- a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
+++ b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
@@ -287,7 +287,7 @@ examples:
arm,trig-in-sigs = <0 1>;
arm,trig-in-types = <PE_DBGTRIGGER
PE_PMUIRQ>;
- arm,trig-out-sigs=<0 1 2 >;
+ arm,trig-out-sigs = <0 1 2 >;
arm,trig-out-types = <PE_EDBGREQ
PE_DBGRESTART
PE_CTIIRQ>;
@@ -309,24 +309,24 @@ examples:

trig-conns@0 {
reg = <0>;
- arm,trig-in-sigs=<0>;
- arm,trig-in-types=<GEN_INTREQ>;
- arm,trig-out-sigs=<0>;
- arm,trig-out-types=<GEN_HALTREQ>;
+ arm,trig-in-sigs = <0>;
+ arm,trig-in-types = <GEN_INTREQ>;
+ arm,trig-out-sigs = <0>;
+ arm,trig-out-types = <GEN_HALTREQ>;
arm,trig-conn-name = "sys_profiler";
};

trig-conns@1 {
reg = <1>;
- arm,trig-out-sigs=<2 3>;
- arm,trig-out-types=<GEN_HALTREQ GEN_RESTARTREQ>;
+ arm,trig-out-sigs = <2 3>;
+ arm,trig-out-types = <GEN_HALTREQ GEN_RESTARTREQ>;
arm,trig-conn-name = "watchdog";
};

trig-conns@2 {
reg = <2>;
- arm,trig-in-sigs=<1 6>;
- arm,trig-in-types=<GEN_HALTREQ GEN_RESTARTREQ>;
+ arm,trig-in-sigs = <1 6>;
+ arm,trig-in-types = <GEN_HALTREQ GEN_RESTARTREQ>;
arm,trig-conn-name = "g_counter";
};
};
diff --git a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
index 91b96065f7df..86b59de7707e 100644
--- a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
+++ b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
@@ -96,8 +96,8 @@ examples:
compatible = "ti,k2g-sci";
ti,system-reboot-controller;
mbox-names = "rx", "tx";
- mboxes= <&msgmgr 5 2>,
- <&msgmgr 0 0>;
+ mboxes = <&msgmgr 5 2>,
+ <&msgmgr 0 0>;
reg-names = "debug_messages";
reg = <0x02921800 0x800>;
};
@@ -107,8 +107,8 @@ examples:
compatible = "ti,k2g-sci";
ti,host-id = <12>;
mbox-names = "rx", "tx";
- mboxes= <&secure_proxy_main 11>,
- <&secure_proxy_main 13>;
+ mboxes = <&secure_proxy_main 11>,
+ <&secure_proxy_main 13>;
reg-names = "debug_messages";
reg = <0x44083000 0x1000>;

diff --git a/Documentation/devicetree/bindings/display/msm/gmu.yaml b/Documentation/devicetree/bindings/display/msm/gmu.yaml
index 029d72822d8b..65b02c7a1211 100644
--- a/Documentation/devicetree/bindings/display/msm/gmu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/gmu.yaml
@@ -225,7 +225,7 @@ examples:
#include <dt-bindings/interrupt-controller/arm-gic.h>

gmu: gmu@506a000 {
- compatible="qcom,adreno-gmu-630.2", "qcom,adreno-gmu";
+ compatible = "qcom,adreno-gmu-630.2", "qcom,adreno-gmu";

reg = <0x506a000 0x30000>,
<0xb280000 0x10000>,
diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
index 1cdc91b3439f..200fbf1c74a0 100644
--- a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
+++ b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
@@ -74,7 +74,7 @@ examples:
vdd3-supply = <&vcclcd_reg>;
vci-supply = <&vlcd_reg>;
reset-gpios = <&gpy4 5 0>;
- power-on-delay= <50>;
+ power-on-delay = <50>;
reset-delay = <100>;
init-delay = <100>;
panel-width-mm = <58>;
diff --git a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
index 6f43d885c9b3..df61cb5f5c54 100644
--- a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
+++ b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
@@ -121,11 +121,11 @@ examples:
#size-cells = <0>;
vopb_out_edp: endpoint@0 {
reg = <0>;
- remote-endpoint=<&edp_in_vopb>;
+ remote-endpoint = <&edp_in_vopb>;
};
vopb_out_hdmi: endpoint@1 {
reg = <1>;
- remote-endpoint=<&hdmi_in_vopb>;
+ remote-endpoint = <&hdmi_in_vopb>;
};
};
};
diff --git a/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml b/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
index 9b072b057f16..a60b1e100ee4 100644
--- a/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
@@ -35,7 +35,7 @@ unevaluatedProperties: false
examples:
- |
spi {
- #address-cells= <1>;
+ #address-cells = <1>;
#size-cells = <0>;

adc@0 {
diff --git a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
index 7dde7967c886..1e72b8808d24 100644
--- a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
+++ b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
@@ -137,7 +137,7 @@ examples:

cru_parallel_in: endpoint@0 {
reg = <0>;
- remote-endpoint= <&ov5642>;
+ remote-endpoint = <&ov5642>;
hsync-active = <1>;
vsync-active = <1>;
};
@@ -150,7 +150,7 @@ examples:

cru_csi_in: endpoint@0 {
reg = <0>;
- remote-endpoint= <&csi_cru_in>;
+ remote-endpoint = <&csi_cru_in>;
};
};
};
diff --git a/Documentation/devicetree/bindings/media/renesas,vin.yaml b/Documentation/devicetree/bindings/media/renesas,vin.yaml
index 91e8f368fb52..324703bfb1bd 100644
--- a/Documentation/devicetree/bindings/media/renesas,vin.yaml
+++ b/Documentation/devicetree/bindings/media/renesas,vin.yaml
@@ -303,11 +303,11 @@ examples:

vin0csi20: endpoint@0 {
reg = <0>;
- remote-endpoint= <&csi20vin0>;
+ remote-endpoint = <&csi20vin0>;
};
vin0csi40: endpoint@2 {
reg = <2>;
- remote-endpoint= <&csi40vin0>;
+ remote-endpoint = <&csi40vin0>;
};
};
};
diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
index f8c976898a95..18f6733408b4 100644
--- a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
+++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
@@ -164,7 +164,7 @@ examples:
reg = <0 0xf80000>;
};
firmware@f80000 {
- label ="firmware";
+ label = "firmware";
reg = <0xf80000 0x80000>;
read-only;
};
diff --git a/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml b/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
index 0fa2132fa4f4..400aedb58205 100644
--- a/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
+++ b/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
@@ -156,7 +156,7 @@ examples:
reg = <0x1101c000 0x1300>;
interrupts = <GIC_SPI 237 IRQ_TYPE_LEVEL_LOW>;
interrupt-names = "macirq";
- phy-mode ="rgmii-rxid";
+ phy-mode = "rgmii-rxid";
mac-address = [00 55 7b b5 7d f7];
clock-names = "axi",
"apb",
diff --git a/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml b/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
index 50f46a6898b1..4adab0149108 100644
--- a/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
+++ b/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
@@ -42,8 +42,8 @@ examples:
- |
#include <dt-bindings/interrupt-controller/arm-gic.h>
pmu {
- #address-cells=<2>;
- #size-cells=<2>;
+ #address-cells = <2>;
+ #size-cells = <2>;

pmu@ff638000 {
compatible = "amlogic,g12a-ddr-pmu";
diff --git a/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml b/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
index 26f2b887cfc1..b8d77165c4a1 100644
--- a/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
@@ -83,7 +83,7 @@ examples:
clocks = <&clk26m>;
clock-output-names = "mipi_tx0_pll";
drive-strength-microamp = <4000>;
- nvmem-cells= <&mipi_tx_calibration>;
+ nvmem-cells = <&mipi_tx_calibration>;
nvmem-cell-names = "calibration-data";
#clock-cells = <0>;
#phy-cells = <0>;
diff --git a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
index 3100cb870170..76e8ca44906a 100644
--- a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
@@ -75,7 +75,7 @@ additionalProperties: false
examples:
- |
remoteproc@1c {
- compatible= "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";
+ compatible = "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";
reg = <0x1c 0x8>, <0x38 0x8>;
reg-names = "remap", "cpu";
resets = <&media_cpu_reset>;
diff --git a/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml b/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
index 478214ab045e..a59d91243ac8 100644
--- a/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
+++ b/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
@@ -304,7 +304,7 @@ examples:
# Dual role switch with type-c
- |
usb@11201000 {
- compatible ="mediatek,mt8183-mtu3", "mediatek,mtu3";
+ compatible = "mediatek,mt8183-mtu3", "mediatek,mtu3";
reg = <0x11201000 0x2e00>, <0x11203e00 0x0100>;
reg-names = "mac", "ippc";
interrupts = <GIC_SPI 72 IRQ_TYPE_LEVEL_LOW>;
diff --git a/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml b/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
index d25fc708e32c..fec5651f5602 100644
--- a/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
+++ b/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
@@ -92,7 +92,7 @@ examples:

usb@31100000 {
compatible = "snps,dwc3";
- reg =<0x00 0x31100000 0x00 0x50000>;
+ reg = <0x00 0x31100000 0x00 0x50000>;
interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>, /* irq.0 */
<GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>; /* irq.0 */
interrupt-names = "host", "peripheral";
--
2.34.1



2023-07-03 09:50:23

by Mike Leach

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: cleanup DTS example whitespaces

On Sun, 2 Jul 2023 at 19:23, Krzysztof Kozlowski
<[email protected]> wrote:
>
> The DTS code coding style expects spaces around '=' sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Rob,
>
> Maybe this could go via your tree? Rebased on your for-next:
> v6.4-rc2-45-gf0ac35049606
> ---
> .../bindings/arm/arm,coresight-cti.yaml | 18 +++++++++---------
> .../bindings/arm/keystone/ti,sci.yaml | 8 ++++----
> .../devicetree/bindings/display/msm/gmu.yaml | 2 +-
> .../display/panel/samsung,s6e8aa0.yaml | 2 +-
> .../display/rockchip/rockchip-vop.yaml | 4 ++--
> .../bindings/iio/adc/ti,adc108s102.yaml | 2 +-
> .../bindings/media/renesas,rzg2l-cru.yaml | 4 ++--
> .../devicetree/bindings/media/renesas,vin.yaml | 4 ++--
> .../devicetree/bindings/mtd/mtd-physmap.yaml | 2 +-
> .../bindings/net/mediatek-dwmac.yaml | 2 +-
> .../bindings/perf/amlogic,g12-ddr-pmu.yaml | 4 ++--
> .../bindings/phy/mediatek,dsi-phy.yaml | 2 +-
> .../remoteproc/amlogic,meson-mx-ao-arc.yaml | 2 +-
> .../devicetree/bindings/usb/mediatek,mtu3.yaml | 2 +-
> .../devicetree/bindings/usb/ti,am62-usb.yaml | 2 +-
> 15 files changed, 30 insertions(+), 30 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> index 0c5b875cb654..d6c84b6e7fe6 100644
> --- a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> @@ -287,7 +287,7 @@ examples:
> arm,trig-in-sigs = <0 1>;
> arm,trig-in-types = <PE_DBGTRIGGER
> PE_PMUIRQ>;
> - arm,trig-out-sigs=<0 1 2 >;
> + arm,trig-out-sigs = <0 1 2 >;
> arm,trig-out-types = <PE_EDBGREQ
> PE_DBGRESTART
> PE_CTIIRQ>;
> @@ -309,24 +309,24 @@ examples:
>
> trig-conns@0 {
> reg = <0>;
> - arm,trig-in-sigs=<0>;
> - arm,trig-in-types=<GEN_INTREQ>;
> - arm,trig-out-sigs=<0>;
> - arm,trig-out-types=<GEN_HALTREQ>;
> + arm,trig-in-sigs = <0>;
> + arm,trig-in-types = <GEN_INTREQ>;
> + arm,trig-out-sigs = <0>;
> + arm,trig-out-types = <GEN_HALTREQ>;
> arm,trig-conn-name = "sys_profiler";
> };
>
> trig-conns@1 {
> reg = <1>;
> - arm,trig-out-sigs=<2 3>;
> - arm,trig-out-types=<GEN_HALTREQ GEN_RESTARTREQ>;
> + arm,trig-out-sigs = <2 3>;
> + arm,trig-out-types = <GEN_HALTREQ GEN_RESTARTREQ>;
> arm,trig-conn-name = "watchdog";
> };
>
> trig-conns@2 {
> reg = <2>;
> - arm,trig-in-sigs=<1 6>;
> - arm,trig-in-types=<GEN_HALTREQ GEN_RESTARTREQ>;
> + arm,trig-in-sigs = <1 6>;
> + arm,trig-in-types = <GEN_HALTREQ GEN_RESTARTREQ>;
> arm,trig-conn-name = "g_counter";
> };
> };

for above CTI chagnes

Acked-by: Mike Leach <[email protected]>


> diff --git a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> index 91b96065f7df..86b59de7707e 100644
> --- a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> +++ b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> @@ -96,8 +96,8 @@ examples:
> compatible = "ti,k2g-sci";
> ti,system-reboot-controller;
> mbox-names = "rx", "tx";
> - mboxes= <&msgmgr 5 2>,
> - <&msgmgr 0 0>;
> + mboxes = <&msgmgr 5 2>,
> + <&msgmgr 0 0>;
> reg-names = "debug_messages";
> reg = <0x02921800 0x800>;
> };
> @@ -107,8 +107,8 @@ examples:
> compatible = "ti,k2g-sci";
> ti,host-id = <12>;
> mbox-names = "rx", "tx";
> - mboxes= <&secure_proxy_main 11>,
> - <&secure_proxy_main 13>;
> + mboxes = <&secure_proxy_main 11>,
> + <&secure_proxy_main 13>;
> reg-names = "debug_messages";
> reg = <0x44083000 0x1000>;
>
> diff --git a/Documentation/devicetree/bindings/display/msm/gmu.yaml b/Documentation/devicetree/bindings/display/msm/gmu.yaml
> index 029d72822d8b..65b02c7a1211 100644
> --- a/Documentation/devicetree/bindings/display/msm/gmu.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/gmu.yaml
> @@ -225,7 +225,7 @@ examples:
> #include <dt-bindings/interrupt-controller/arm-gic.h>
>
> gmu: gmu@506a000 {
> - compatible="qcom,adreno-gmu-630.2", "qcom,adreno-gmu";
> + compatible = "qcom,adreno-gmu-630.2", "qcom,adreno-gmu";
>
> reg = <0x506a000 0x30000>,
> <0xb280000 0x10000>,
> diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> index 1cdc91b3439f..200fbf1c74a0 100644
> --- a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> @@ -74,7 +74,7 @@ examples:
> vdd3-supply = <&vcclcd_reg>;
> vci-supply = <&vlcd_reg>;
> reset-gpios = <&gpy4 5 0>;
> - power-on-delay= <50>;
> + power-on-delay = <50>;
> reset-delay = <100>;
> init-delay = <100>;
> panel-width-mm = <58>;
> diff --git a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> index 6f43d885c9b3..df61cb5f5c54 100644
> --- a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> +++ b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> @@ -121,11 +121,11 @@ examples:
> #size-cells = <0>;
> vopb_out_edp: endpoint@0 {
> reg = <0>;
> - remote-endpoint=<&edp_in_vopb>;
> + remote-endpoint = <&edp_in_vopb>;
> };
> vopb_out_hdmi: endpoint@1 {
> reg = <1>;
> - remote-endpoint=<&hdmi_in_vopb>;
> + remote-endpoint = <&hdmi_in_vopb>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml b/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> index 9b072b057f16..a60b1e100ee4 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> @@ -35,7 +35,7 @@ unevaluatedProperties: false
> examples:
> - |
> spi {
> - #address-cells= <1>;
> + #address-cells = <1>;
> #size-cells = <0>;
>
> adc@0 {
> diff --git a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> index 7dde7967c886..1e72b8808d24 100644
> --- a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> +++ b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> @@ -137,7 +137,7 @@ examples:
>
> cru_parallel_in: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&ov5642>;
> + remote-endpoint = <&ov5642>;
> hsync-active = <1>;
> vsync-active = <1>;
> };
> @@ -150,7 +150,7 @@ examples:
>
> cru_csi_in: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&csi_cru_in>;
> + remote-endpoint = <&csi_cru_in>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/media/renesas,vin.yaml b/Documentation/devicetree/bindings/media/renesas,vin.yaml
> index 91e8f368fb52..324703bfb1bd 100644
> --- a/Documentation/devicetree/bindings/media/renesas,vin.yaml
> +++ b/Documentation/devicetree/bindings/media/renesas,vin.yaml
> @@ -303,11 +303,11 @@ examples:
>
> vin0csi20: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&csi20vin0>;
> + remote-endpoint = <&csi20vin0>;
> };
> vin0csi40: endpoint@2 {
> reg = <2>;
> - remote-endpoint= <&csi40vin0>;
> + remote-endpoint = <&csi40vin0>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> index f8c976898a95..18f6733408b4 100644
> --- a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> +++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> @@ -164,7 +164,7 @@ examples:
> reg = <0 0xf80000>;
> };
> firmware@f80000 {
> - label ="firmware";
> + label = "firmware";
> reg = <0xf80000 0x80000>;
> read-only;
> };
> diff --git a/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml b/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> index 0fa2132fa4f4..400aedb58205 100644
> --- a/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> +++ b/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> @@ -156,7 +156,7 @@ examples:
> reg = <0x1101c000 0x1300>;
> interrupts = <GIC_SPI 237 IRQ_TYPE_LEVEL_LOW>;
> interrupt-names = "macirq";
> - phy-mode ="rgmii-rxid";
> + phy-mode = "rgmii-rxid";
> mac-address = [00 55 7b b5 7d f7];
> clock-names = "axi",
> "apb",
> diff --git a/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml b/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> index 50f46a6898b1..4adab0149108 100644
> --- a/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> +++ b/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> @@ -42,8 +42,8 @@ examples:
> - |
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> pmu {
> - #address-cells=<2>;
> - #size-cells=<2>;
> + #address-cells = <2>;
> + #size-cells = <2>;
>
> pmu@ff638000 {
> compatible = "amlogic,g12a-ddr-pmu";
> diff --git a/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml b/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> index 26f2b887cfc1..b8d77165c4a1 100644
> --- a/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> @@ -83,7 +83,7 @@ examples:
> clocks = <&clk26m>;
> clock-output-names = "mipi_tx0_pll";
> drive-strength-microamp = <4000>;
> - nvmem-cells= <&mipi_tx_calibration>;
> + nvmem-cells = <&mipi_tx_calibration>;
> nvmem-cell-names = "calibration-data";
> #clock-cells = <0>;
> #phy-cells = <0>;
> diff --git a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> index 3100cb870170..76e8ca44906a 100644
> --- a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> @@ -75,7 +75,7 @@ additionalProperties: false
> examples:
> - |
> remoteproc@1c {
> - compatible= "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";
> + compatible = "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";
> reg = <0x1c 0x8>, <0x38 0x8>;
> reg-names = "remap", "cpu";
> resets = <&media_cpu_reset>;
> diff --git a/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml b/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> index 478214ab045e..a59d91243ac8 100644
> --- a/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> +++ b/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> @@ -304,7 +304,7 @@ examples:
> # Dual role switch with type-c
> - |
> usb@11201000 {
> - compatible ="mediatek,mt8183-mtu3", "mediatek,mtu3";
> + compatible = "mediatek,mt8183-mtu3", "mediatek,mtu3";
> reg = <0x11201000 0x2e00>, <0x11203e00 0x0100>;
> reg-names = "mac", "ippc";
> interrupts = <GIC_SPI 72 IRQ_TYPE_LEVEL_LOW>;
> diff --git a/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml b/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> index d25fc708e32c..fec5651f5602 100644
> --- a/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> +++ b/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> @@ -92,7 +92,7 @@ examples:
>
> usb@31100000 {
> compatible = "snps,dwc3";
> - reg =<0x00 0x31100000 0x00 0x50000>;
> + reg = <0x00 0x31100000 0x00 0x50000>;
> interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>, /* irq.0 */
> <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>; /* irq.0 */
> interrupt-names = "host", "peripheral";
> --
> 2.34.1
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> [email protected]
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel



--
Mike Leach
Principal Engineer, ARM Ltd.
Manchester Design Centre. UK

2023-07-03 11:51:17

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: cleanup DTS example whitespaces

On Sun, 2 Jul 2023 20:23:08 +0200
Krzysztof Kozlowski <[email protected]> wrote:

> The DTS code coding style expects spaces around '=' sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
For the IIO one.
Acked-by: Jonathan Cameron <[email protected]>

Thanks for tidying these up.

Jonathan

> ---
>
> Rob,
>
> Maybe this could go via your tree? Rebased on your for-next:
> v6.4-rc2-45-gf0ac35049606
> ---
> .../bindings/arm/arm,coresight-cti.yaml | 18 +++++++++---------
> .../bindings/arm/keystone/ti,sci.yaml | 8 ++++----
> .../devicetree/bindings/display/msm/gmu.yaml | 2 +-
> .../display/panel/samsung,s6e8aa0.yaml | 2 +-
> .../display/rockchip/rockchip-vop.yaml | 4 ++--
> .../bindings/iio/adc/ti,adc108s102.yaml | 2 +-
> .../bindings/media/renesas,rzg2l-cru.yaml | 4 ++--
> .../devicetree/bindings/media/renesas,vin.yaml | 4 ++--
> .../devicetree/bindings/mtd/mtd-physmap.yaml | 2 +-
> .../bindings/net/mediatek-dwmac.yaml | 2 +-
> .../bindings/perf/amlogic,g12-ddr-pmu.yaml | 4 ++--
> .../bindings/phy/mediatek,dsi-phy.yaml | 2 +-
> .../remoteproc/amlogic,meson-mx-ao-arc.yaml | 2 +-
> .../devicetree/bindings/usb/mediatek,mtu3.yaml | 2 +-
> .../devicetree/bindings/usb/ti,am62-usb.yaml | 2 +-
> 15 files changed, 30 insertions(+), 30 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> index 0c5b875cb654..d6c84b6e7fe6 100644
> --- a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> @@ -287,7 +287,7 @@ examples:
> arm,trig-in-sigs = <0 1>;
> arm,trig-in-types = <PE_DBGTRIGGER
> PE_PMUIRQ>;
> - arm,trig-out-sigs=<0 1 2 >;
> + arm,trig-out-sigs = <0 1 2 >;
> arm,trig-out-types = <PE_EDBGREQ
> PE_DBGRESTART
> PE_CTIIRQ>;
> @@ -309,24 +309,24 @@ examples:
>
> trig-conns@0 {
> reg = <0>;
> - arm,trig-in-sigs=<0>;
> - arm,trig-in-types=<GEN_INTREQ>;
> - arm,trig-out-sigs=<0>;
> - arm,trig-out-types=<GEN_HALTREQ>;
> + arm,trig-in-sigs = <0>;
> + arm,trig-in-types = <GEN_INTREQ>;
> + arm,trig-out-sigs = <0>;
> + arm,trig-out-types = <GEN_HALTREQ>;
> arm,trig-conn-name = "sys_profiler";
> };
>
> trig-conns@1 {
> reg = <1>;
> - arm,trig-out-sigs=<2 3>;
> - arm,trig-out-types=<GEN_HALTREQ GEN_RESTARTREQ>;
> + arm,trig-out-sigs = <2 3>;
> + arm,trig-out-types = <GEN_HALTREQ GEN_RESTARTREQ>;
> arm,trig-conn-name = "watchdog";
> };
>
> trig-conns@2 {
> reg = <2>;
> - arm,trig-in-sigs=<1 6>;
> - arm,trig-in-types=<GEN_HALTREQ GEN_RESTARTREQ>;
> + arm,trig-in-sigs = <1 6>;
> + arm,trig-in-types = <GEN_HALTREQ GEN_RESTARTREQ>;
> arm,trig-conn-name = "g_counter";
> };
> };
> diff --git a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> index 91b96065f7df..86b59de7707e 100644
> --- a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> +++ b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> @@ -96,8 +96,8 @@ examples:
> compatible = "ti,k2g-sci";
> ti,system-reboot-controller;
> mbox-names = "rx", "tx";
> - mboxes= <&msgmgr 5 2>,
> - <&msgmgr 0 0>;
> + mboxes = <&msgmgr 5 2>,
> + <&msgmgr 0 0>;
> reg-names = "debug_messages";
> reg = <0x02921800 0x800>;
> };
> @@ -107,8 +107,8 @@ examples:
> compatible = "ti,k2g-sci";
> ti,host-id = <12>;
> mbox-names = "rx", "tx";
> - mboxes= <&secure_proxy_main 11>,
> - <&secure_proxy_main 13>;
> + mboxes = <&secure_proxy_main 11>,
> + <&secure_proxy_main 13>;
> reg-names = "debug_messages";
> reg = <0x44083000 0x1000>;
>
> diff --git a/Documentation/devicetree/bindings/display/msm/gmu.yaml b/Documentation/devicetree/bindings/display/msm/gmu.yaml
> index 029d72822d8b..65b02c7a1211 100644
> --- a/Documentation/devicetree/bindings/display/msm/gmu.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/gmu.yaml
> @@ -225,7 +225,7 @@ examples:
> #include <dt-bindings/interrupt-controller/arm-gic.h>
>
> gmu: gmu@506a000 {
> - compatible="qcom,adreno-gmu-630.2", "qcom,adreno-gmu";
> + compatible = "qcom,adreno-gmu-630.2", "qcom,adreno-gmu";
>
> reg = <0x506a000 0x30000>,
> <0xb280000 0x10000>,
> diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> index 1cdc91b3439f..200fbf1c74a0 100644
> --- a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> @@ -74,7 +74,7 @@ examples:
> vdd3-supply = <&vcclcd_reg>;
> vci-supply = <&vlcd_reg>;
> reset-gpios = <&gpy4 5 0>;
> - power-on-delay= <50>;
> + power-on-delay = <50>;
> reset-delay = <100>;
> init-delay = <100>;
> panel-width-mm = <58>;
> diff --git a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> index 6f43d885c9b3..df61cb5f5c54 100644
> --- a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> +++ b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> @@ -121,11 +121,11 @@ examples:
> #size-cells = <0>;
> vopb_out_edp: endpoint@0 {
> reg = <0>;
> - remote-endpoint=<&edp_in_vopb>;
> + remote-endpoint = <&edp_in_vopb>;
> };
> vopb_out_hdmi: endpoint@1 {
> reg = <1>;
> - remote-endpoint=<&hdmi_in_vopb>;
> + remote-endpoint = <&hdmi_in_vopb>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml b/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> index 9b072b057f16..a60b1e100ee4 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> @@ -35,7 +35,7 @@ unevaluatedProperties: false
> examples:
> - |
> spi {
> - #address-cells= <1>;
> + #address-cells = <1>;
> #size-cells = <0>;
>
> adc@0 {
> diff --git a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> index 7dde7967c886..1e72b8808d24 100644
> --- a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> +++ b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> @@ -137,7 +137,7 @@ examples:
>
> cru_parallel_in: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&ov5642>;
> + remote-endpoint = <&ov5642>;
> hsync-active = <1>;
> vsync-active = <1>;
> };
> @@ -150,7 +150,7 @@ examples:
>
> cru_csi_in: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&csi_cru_in>;
> + remote-endpoint = <&csi_cru_in>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/media/renesas,vin.yaml b/Documentation/devicetree/bindings/media/renesas,vin.yaml
> index 91e8f368fb52..324703bfb1bd 100644
> --- a/Documentation/devicetree/bindings/media/renesas,vin.yaml
> +++ b/Documentation/devicetree/bindings/media/renesas,vin.yaml
> @@ -303,11 +303,11 @@ examples:
>
> vin0csi20: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&csi20vin0>;
> + remote-endpoint = <&csi20vin0>;
> };
> vin0csi40: endpoint@2 {
> reg = <2>;
> - remote-endpoint= <&csi40vin0>;
> + remote-endpoint = <&csi40vin0>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> index f8c976898a95..18f6733408b4 100644
> --- a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> +++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> @@ -164,7 +164,7 @@ examples:
> reg = <0 0xf80000>;
> };
> firmware@f80000 {
> - label ="firmware";
> + label = "firmware";
> reg = <0xf80000 0x80000>;
> read-only;
> };
> diff --git a/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml b/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> index 0fa2132fa4f4..400aedb58205 100644
> --- a/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> +++ b/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> @@ -156,7 +156,7 @@ examples:
> reg = <0x1101c000 0x1300>;
> interrupts = <GIC_SPI 237 IRQ_TYPE_LEVEL_LOW>;
> interrupt-names = "macirq";
> - phy-mode ="rgmii-rxid";
> + phy-mode = "rgmii-rxid";
> mac-address = [00 55 7b b5 7d f7];
> clock-names = "axi",
> "apb",
> diff --git a/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml b/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> index 50f46a6898b1..4adab0149108 100644
> --- a/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> +++ b/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> @@ -42,8 +42,8 @@ examples:
> - |
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> pmu {
> - #address-cells=<2>;
> - #size-cells=<2>;
> + #address-cells = <2>;
> + #size-cells = <2>;
>
> pmu@ff638000 {
> compatible = "amlogic,g12a-ddr-pmu";
> diff --git a/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml b/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> index 26f2b887cfc1..b8d77165c4a1 100644
> --- a/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> @@ -83,7 +83,7 @@ examples:
> clocks = <&clk26m>;
> clock-output-names = "mipi_tx0_pll";
> drive-strength-microamp = <4000>;
> - nvmem-cells= <&mipi_tx_calibration>;
> + nvmem-cells = <&mipi_tx_calibration>;
> nvmem-cell-names = "calibration-data";
> #clock-cells = <0>;
> #phy-cells = <0>;
> diff --git a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> index 3100cb870170..76e8ca44906a 100644
> --- a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> @@ -75,7 +75,7 @@ additionalProperties: false
> examples:
> - |
> remoteproc@1c {
> - compatible= "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";
> + compatible = "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";
> reg = <0x1c 0x8>, <0x38 0x8>;
> reg-names = "remap", "cpu";
> resets = <&media_cpu_reset>;
> diff --git a/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml b/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> index 478214ab045e..a59d91243ac8 100644
> --- a/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> +++ b/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> @@ -304,7 +304,7 @@ examples:
> # Dual role switch with type-c
> - |
> usb@11201000 {
> - compatible ="mediatek,mt8183-mtu3", "mediatek,mtu3";
> + compatible = "mediatek,mt8183-mtu3", "mediatek,mtu3";
> reg = <0x11201000 0x2e00>, <0x11203e00 0x0100>;
> reg-names = "mac", "ippc";
> interrupts = <GIC_SPI 72 IRQ_TYPE_LEVEL_LOW>;
> diff --git a/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml b/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> index d25fc708e32c..fec5651f5602 100644
> --- a/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> +++ b/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> @@ -92,7 +92,7 @@ examples:
>
> usb@31100000 {
> compatible = "snps,dwc3";
> - reg =<0x00 0x31100000 0x00 0x50000>;
> + reg = <0x00 0x31100000 0x00 0x50000>;
> interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>, /* irq.0 */
> <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>; /* irq.0 */
> interrupt-names = "host", "peripheral";


2023-07-03 13:32:37

by Neil Armstrong

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: cleanup DTS example whitespaces

On 02/07/2023 20:23, Krzysztof Kozlowski wrote:
> The DTS code coding style expects spaces around '=' sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Rob,
>
> Maybe this could go via your tree? Rebased on your for-next:
> v6.4-rc2-45-gf0ac35049606
> ---
> .../bindings/arm/arm,coresight-cti.yaml | 18 +++++++++---------
> .../bindings/arm/keystone/ti,sci.yaml | 8 ++++----
> .../devicetree/bindings/display/msm/gmu.yaml | 2 +-
> .../display/panel/samsung,s6e8aa0.yaml | 2 +-
> .../display/rockchip/rockchip-vop.yaml | 4 ++--
> .../bindings/iio/adc/ti,adc108s102.yaml | 2 +-
> .../bindings/media/renesas,rzg2l-cru.yaml | 4 ++--
> .../devicetree/bindings/media/renesas,vin.yaml | 4 ++--
> .../devicetree/bindings/mtd/mtd-physmap.yaml | 2 +-
> .../bindings/net/mediatek-dwmac.yaml | 2 +-
> .../bindings/perf/amlogic,g12-ddr-pmu.yaml | 4 ++--
> .../bindings/phy/mediatek,dsi-phy.yaml | 2 +-
> .../remoteproc/amlogic,meson-mx-ao-arc.yaml | 2 +-

For amlogic ones:

Acked-by: Neil Armstrong <[email protected]>


> .../devicetree/bindings/usb/mediatek,mtu3.yaml | 2 +-
> .../devicetree/bindings/usb/ti,am62-usb.yaml | 2 +-
> 15 files changed, 30 insertions(+), 30 deletions(-)
>

<snip>


2023-07-03 16:19:26

by Mathieu Poirier

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: cleanup DTS example whitespaces

On Sun, Jul 02, 2023 at 08:23:08PM +0200, Krzysztof Kozlowski wrote:
> The DTS code coding style expects spaces around '=' sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Rob,
>
> Maybe this could go via your tree? Rebased on your for-next:
> v6.4-rc2-45-gf0ac35049606
> ---
> .../bindings/arm/arm,coresight-cti.yaml | 18 +++++++++---------
> .../bindings/arm/keystone/ti,sci.yaml | 8 ++++----
> .../devicetree/bindings/display/msm/gmu.yaml | 2 +-
> .../display/panel/samsung,s6e8aa0.yaml | 2 +-
> .../display/rockchip/rockchip-vop.yaml | 4 ++--
> .../bindings/iio/adc/ti,adc108s102.yaml | 2 +-
> .../bindings/media/renesas,rzg2l-cru.yaml | 4 ++--
> .../devicetree/bindings/media/renesas,vin.yaml | 4 ++--
> .../devicetree/bindings/mtd/mtd-physmap.yaml | 2 +-
> .../bindings/net/mediatek-dwmac.yaml | 2 +-
> .../bindings/perf/amlogic,g12-ddr-pmu.yaml | 4 ++--
> .../bindings/phy/mediatek,dsi-phy.yaml | 2 +-
> .../remoteproc/amlogic,meson-mx-ao-arc.yaml | 2 +-
> .../devicetree/bindings/usb/mediatek,mtu3.yaml | 2 +-
> .../devicetree/bindings/usb/ti,am62-usb.yaml | 2 +-
> 15 files changed, 30 insertions(+), 30 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> index 0c5b875cb654..d6c84b6e7fe6 100644
> --- a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> @@ -287,7 +287,7 @@ examples:
> arm,trig-in-sigs = <0 1>;
> arm,trig-in-types = <PE_DBGTRIGGER
> PE_PMUIRQ>;
> - arm,trig-out-sigs=<0 1 2 >;
> + arm,trig-out-sigs = <0 1 2 >;
> arm,trig-out-types = <PE_EDBGREQ
> PE_DBGRESTART
> PE_CTIIRQ>;
> @@ -309,24 +309,24 @@ examples:
>
> trig-conns@0 {
> reg = <0>;
> - arm,trig-in-sigs=<0>;
> - arm,trig-in-types=<GEN_INTREQ>;
> - arm,trig-out-sigs=<0>;
> - arm,trig-out-types=<GEN_HALTREQ>;
> + arm,trig-in-sigs = <0>;
> + arm,trig-in-types = <GEN_INTREQ>;
> + arm,trig-out-sigs = <0>;
> + arm,trig-out-types = <GEN_HALTREQ>;
> arm,trig-conn-name = "sys_profiler";
> };
>
> trig-conns@1 {
> reg = <1>;
> - arm,trig-out-sigs=<2 3>;
> - arm,trig-out-types=<GEN_HALTREQ GEN_RESTARTREQ>;
> + arm,trig-out-sigs = <2 3>;
> + arm,trig-out-types = <GEN_HALTREQ GEN_RESTARTREQ>;
> arm,trig-conn-name = "watchdog";
> };
>
> trig-conns@2 {
> reg = <2>;
> - arm,trig-in-sigs=<1 6>;
> - arm,trig-in-types=<GEN_HALTREQ GEN_RESTARTREQ>;
> + arm,trig-in-sigs = <1 6>;
> + arm,trig-in-types = <GEN_HALTREQ GEN_RESTARTREQ>;
> arm,trig-conn-name = "g_counter";
> };
> };
> diff --git a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> index 91b96065f7df..86b59de7707e 100644
> --- a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> +++ b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> @@ -96,8 +96,8 @@ examples:
> compatible = "ti,k2g-sci";
> ti,system-reboot-controller;
> mbox-names = "rx", "tx";
> - mboxes= <&msgmgr 5 2>,
> - <&msgmgr 0 0>;
> + mboxes = <&msgmgr 5 2>,
> + <&msgmgr 0 0>;
> reg-names = "debug_messages";
> reg = <0x02921800 0x800>;
> };
> @@ -107,8 +107,8 @@ examples:
> compatible = "ti,k2g-sci";
> ti,host-id = <12>;
> mbox-names = "rx", "tx";
> - mboxes= <&secure_proxy_main 11>,
> - <&secure_proxy_main 13>;
> + mboxes = <&secure_proxy_main 11>,
> + <&secure_proxy_main 13>;
> reg-names = "debug_messages";
> reg = <0x44083000 0x1000>;
>
> diff --git a/Documentation/devicetree/bindings/display/msm/gmu.yaml b/Documentation/devicetree/bindings/display/msm/gmu.yaml
> index 029d72822d8b..65b02c7a1211 100644
> --- a/Documentation/devicetree/bindings/display/msm/gmu.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/gmu.yaml
> @@ -225,7 +225,7 @@ examples:
> #include <dt-bindings/interrupt-controller/arm-gic.h>
>
> gmu: gmu@506a000 {
> - compatible="qcom,adreno-gmu-630.2", "qcom,adreno-gmu";
> + compatible = "qcom,adreno-gmu-630.2", "qcom,adreno-gmu";
>
> reg = <0x506a000 0x30000>,
> <0xb280000 0x10000>,
> diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> index 1cdc91b3439f..200fbf1c74a0 100644
> --- a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> @@ -74,7 +74,7 @@ examples:
> vdd3-supply = <&vcclcd_reg>;
> vci-supply = <&vlcd_reg>;
> reset-gpios = <&gpy4 5 0>;
> - power-on-delay= <50>;
> + power-on-delay = <50>;
> reset-delay = <100>;
> init-delay = <100>;
> panel-width-mm = <58>;
> diff --git a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> index 6f43d885c9b3..df61cb5f5c54 100644
> --- a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> +++ b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> @@ -121,11 +121,11 @@ examples:
> #size-cells = <0>;
> vopb_out_edp: endpoint@0 {
> reg = <0>;
> - remote-endpoint=<&edp_in_vopb>;
> + remote-endpoint = <&edp_in_vopb>;
> };
> vopb_out_hdmi: endpoint@1 {
> reg = <1>;
> - remote-endpoint=<&hdmi_in_vopb>;
> + remote-endpoint = <&hdmi_in_vopb>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml b/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> index 9b072b057f16..a60b1e100ee4 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> @@ -35,7 +35,7 @@ unevaluatedProperties: false
> examples:
> - |
> spi {
> - #address-cells= <1>;
> + #address-cells = <1>;
> #size-cells = <0>;
>
> adc@0 {
> diff --git a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> index 7dde7967c886..1e72b8808d24 100644
> --- a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> +++ b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> @@ -137,7 +137,7 @@ examples:
>
> cru_parallel_in: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&ov5642>;
> + remote-endpoint = <&ov5642>;
> hsync-active = <1>;
> vsync-active = <1>;
> };
> @@ -150,7 +150,7 @@ examples:
>
> cru_csi_in: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&csi_cru_in>;
> + remote-endpoint = <&csi_cru_in>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/media/renesas,vin.yaml b/Documentation/devicetree/bindings/media/renesas,vin.yaml
> index 91e8f368fb52..324703bfb1bd 100644
> --- a/Documentation/devicetree/bindings/media/renesas,vin.yaml
> +++ b/Documentation/devicetree/bindings/media/renesas,vin.yaml
> @@ -303,11 +303,11 @@ examples:
>
> vin0csi20: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&csi20vin0>;
> + remote-endpoint = <&csi20vin0>;
> };
> vin0csi40: endpoint@2 {
> reg = <2>;
> - remote-endpoint= <&csi40vin0>;
> + remote-endpoint = <&csi40vin0>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> index f8c976898a95..18f6733408b4 100644
> --- a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> +++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> @@ -164,7 +164,7 @@ examples:
> reg = <0 0xf80000>;
> };
> firmware@f80000 {
> - label ="firmware";
> + label = "firmware";
> reg = <0xf80000 0x80000>;
> read-only;
> };
> diff --git a/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml b/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> index 0fa2132fa4f4..400aedb58205 100644
> --- a/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> +++ b/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> @@ -156,7 +156,7 @@ examples:
> reg = <0x1101c000 0x1300>;
> interrupts = <GIC_SPI 237 IRQ_TYPE_LEVEL_LOW>;
> interrupt-names = "macirq";
> - phy-mode ="rgmii-rxid";
> + phy-mode = "rgmii-rxid";
> mac-address = [00 55 7b b5 7d f7];
> clock-names = "axi",
> "apb",
> diff --git a/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml b/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> index 50f46a6898b1..4adab0149108 100644
> --- a/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> +++ b/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> @@ -42,8 +42,8 @@ examples:
> - |
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> pmu {
> - #address-cells=<2>;
> - #size-cells=<2>;
> + #address-cells = <2>;
> + #size-cells = <2>;
>
> pmu@ff638000 {
> compatible = "amlogic,g12a-ddr-pmu";
> diff --git a/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml b/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> index 26f2b887cfc1..b8d77165c4a1 100644
> --- a/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> @@ -83,7 +83,7 @@ examples:
> clocks = <&clk26m>;
> clock-output-names = "mipi_tx0_pll";
> drive-strength-microamp = <4000>;
> - nvmem-cells= <&mipi_tx_calibration>;
> + nvmem-cells = <&mipi_tx_calibration>;
> nvmem-cell-names = "calibration-data";
> #clock-cells = <0>;
> #phy-cells = <0>;
> diff --git a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> index 3100cb870170..76e8ca44906a 100644
> --- a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> @@ -75,7 +75,7 @@ additionalProperties: false
> examples:
> - |
> remoteproc@1c {
> - compatible= "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";
> + compatible = "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";

Reviewed-by: Mathieu Poirier <[email protected]>

> reg = <0x1c 0x8>, <0x38 0x8>;
> reg-names = "remap", "cpu";
> resets = <&media_cpu_reset>;
> diff --git a/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml b/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> index 478214ab045e..a59d91243ac8 100644
> --- a/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> +++ b/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> @@ -304,7 +304,7 @@ examples:
> # Dual role switch with type-c
> - |
> usb@11201000 {
> - compatible ="mediatek,mt8183-mtu3", "mediatek,mtu3";
> + compatible = "mediatek,mt8183-mtu3", "mediatek,mtu3";
> reg = <0x11201000 0x2e00>, <0x11203e00 0x0100>;
> reg-names = "mac", "ippc";
> interrupts = <GIC_SPI 72 IRQ_TYPE_LEVEL_LOW>;
> diff --git a/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml b/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> index d25fc708e32c..fec5651f5602 100644
> --- a/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> +++ b/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> @@ -92,7 +92,7 @@ examples:
>
> usb@31100000 {
> compatible = "snps,dwc3";
> - reg =<0x00 0x31100000 0x00 0x50000>;
> + reg = <0x00 0x31100000 0x00 0x50000>;
> interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>, /* irq.0 */
> <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>; /* irq.0 */
> interrupt-names = "host", "peripheral";
> --
> 2.34.1
>

2023-07-03 16:58:58

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: cleanup DTS example whitespaces

On Sun, Jul 02, 2023 at 08:23:08PM +0200, Krzysztof Kozlowski wrote:
> The DTS code coding style expects spaces around '=' sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Reviewed-by: Conor Dooley <[email protected]>

Cheers,
Conor.


Attachments:
(No filename) (283.00 B)
signature.asc (235.00 B)
Download all attachments

2023-07-04 11:34:41

by Vinod Koul

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: cleanup DTS example whitespaces

On 02-07-23, 20:23, Krzysztof Kozlowski wrote:
> The DTS code coding style expects spaces around '=' sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Rob,
>
> Maybe this could go via your tree? Rebased on your for-next:
> v6.4-rc2-45-gf0ac35049606
> ---
> .../bindings/arm/arm,coresight-cti.yaml | 18 +++++++++---------
> .../bindings/arm/keystone/ti,sci.yaml | 8 ++++----
> .../devicetree/bindings/display/msm/gmu.yaml | 2 +-
> .../display/panel/samsung,s6e8aa0.yaml | 2 +-
> .../display/rockchip/rockchip-vop.yaml | 4 ++--
> .../bindings/iio/adc/ti,adc108s102.yaml | 2 +-
> .../bindings/media/renesas,rzg2l-cru.yaml | 4 ++--
> .../devicetree/bindings/media/renesas,vin.yaml | 4 ++--
> .../devicetree/bindings/mtd/mtd-physmap.yaml | 2 +-
> .../bindings/net/mediatek-dwmac.yaml | 2 +-
> .../bindings/perf/amlogic,g12-ddr-pmu.yaml | 4 ++--
> .../bindings/phy/mediatek,dsi-phy.yaml | 2 +-

Acked-by: Vinod Koul <[email protected]>

--
~Vinod

2023-07-04 13:04:18

by Matthias Brugger

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: cleanup DTS example whitespaces



On 02/07/2023 20:23, Krzysztof Kozlowski wrote:
> The DTS code coding style expects spaces around '=' sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Rob,
>
> Maybe this could go via your tree? Rebased on your for-next:
> v6.4-rc2-45-gf0ac35049606
> ---
[...]
> .../bindings/net/mediatek-dwmac.yaml | 2 +-
[...]
> .../bindings/phy/mediatek,dsi-phy.yaml | 2 +-
[...]
> .../devicetree/bindings/usb/mediatek,mtu3.yaml | 2 +-
> .../devicetree/bindings/usb/ti,am62-usb.yaml | 2 +-

For MediaTek parts:
Reviewed-by: Matthias Brugger <[email protected]>

> 15 files changed, 30 insertions(+), 30 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> index 0c5b875cb654..d6c84b6e7fe6 100644
> --- a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> @@ -287,7 +287,7 @@ examples:
> arm,trig-in-sigs = <0 1>;
> arm,trig-in-types = <PE_DBGTRIGGER
> PE_PMUIRQ>;
> - arm,trig-out-sigs=<0 1 2 >;
> + arm,trig-out-sigs = <0 1 2 >;
> arm,trig-out-types = <PE_EDBGREQ
> PE_DBGRESTART
> PE_CTIIRQ>;
> @@ -309,24 +309,24 @@ examples:
>
> trig-conns@0 {
> reg = <0>;
> - arm,trig-in-sigs=<0>;
> - arm,trig-in-types=<GEN_INTREQ>;
> - arm,trig-out-sigs=<0>;
> - arm,trig-out-types=<GEN_HALTREQ>;
> + arm,trig-in-sigs = <0>;
> + arm,trig-in-types = <GEN_INTREQ>;
> + arm,trig-out-sigs = <0>;
> + arm,trig-out-types = <GEN_HALTREQ>;
> arm,trig-conn-name = "sys_profiler";
> };
>
> trig-conns@1 {
> reg = <1>;
> - arm,trig-out-sigs=<2 3>;
> - arm,trig-out-types=<GEN_HALTREQ GEN_RESTARTREQ>;
> + arm,trig-out-sigs = <2 3>;
> + arm,trig-out-types = <GEN_HALTREQ GEN_RESTARTREQ>;
> arm,trig-conn-name = "watchdog";
> };
>
> trig-conns@2 {
> reg = <2>;
> - arm,trig-in-sigs=<1 6>;
> - arm,trig-in-types=<GEN_HALTREQ GEN_RESTARTREQ>;
> + arm,trig-in-sigs = <1 6>;
> + arm,trig-in-types = <GEN_HALTREQ GEN_RESTARTREQ>;
> arm,trig-conn-name = "g_counter";
> };
> };
> diff --git a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> index 91b96065f7df..86b59de7707e 100644
> --- a/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> +++ b/Documentation/devicetree/bindings/arm/keystone/ti,sci.yaml
> @@ -96,8 +96,8 @@ examples:
> compatible = "ti,k2g-sci";
> ti,system-reboot-controller;
> mbox-names = "rx", "tx";
> - mboxes= <&msgmgr 5 2>,
> - <&msgmgr 0 0>;
> + mboxes = <&msgmgr 5 2>,
> + <&msgmgr 0 0>;
> reg-names = "debug_messages";
> reg = <0x02921800 0x800>;
> };
> @@ -107,8 +107,8 @@ examples:
> compatible = "ti,k2g-sci";
> ti,host-id = <12>;
> mbox-names = "rx", "tx";
> - mboxes= <&secure_proxy_main 11>,
> - <&secure_proxy_main 13>;
> + mboxes = <&secure_proxy_main 11>,
> + <&secure_proxy_main 13>;
> reg-names = "debug_messages";
> reg = <0x44083000 0x1000>;
>
> diff --git a/Documentation/devicetree/bindings/display/msm/gmu.yaml b/Documentation/devicetree/bindings/display/msm/gmu.yaml
> index 029d72822d8b..65b02c7a1211 100644
> --- a/Documentation/devicetree/bindings/display/msm/gmu.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/gmu.yaml
> @@ -225,7 +225,7 @@ examples:
> #include <dt-bindings/interrupt-controller/arm-gic.h>
>
> gmu: gmu@506a000 {
> - compatible="qcom,adreno-gmu-630.2", "qcom,adreno-gmu";
> + compatible = "qcom,adreno-gmu-630.2", "qcom,adreno-gmu";
>
> reg = <0x506a000 0x30000>,
> <0xb280000 0x10000>,
> diff --git a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> index 1cdc91b3439f..200fbf1c74a0 100644
> --- a/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/samsung,s6e8aa0.yaml
> @@ -74,7 +74,7 @@ examples:
> vdd3-supply = <&vcclcd_reg>;
> vci-supply = <&vlcd_reg>;
> reset-gpios = <&gpy4 5 0>;
> - power-on-delay= <50>;
> + power-on-delay = <50>;
> reset-delay = <100>;
> init-delay = <100>;
> panel-width-mm = <58>;
> diff --git a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> index 6f43d885c9b3..df61cb5f5c54 100644
> --- a/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> +++ b/Documentation/devicetree/bindings/display/rockchip/rockchip-vop.yaml
> @@ -121,11 +121,11 @@ examples:
> #size-cells = <0>;
> vopb_out_edp: endpoint@0 {
> reg = <0>;
> - remote-endpoint=<&edp_in_vopb>;
> + remote-endpoint = <&edp_in_vopb>;
> };
> vopb_out_hdmi: endpoint@1 {
> reg = <1>;
> - remote-endpoint=<&hdmi_in_vopb>;
> + remote-endpoint = <&hdmi_in_vopb>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml b/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> index 9b072b057f16..a60b1e100ee4 100644
> --- a/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/ti,adc108s102.yaml
> @@ -35,7 +35,7 @@ unevaluatedProperties: false
> examples:
> - |
> spi {
> - #address-cells= <1>;
> + #address-cells = <1>;
> #size-cells = <0>;
>
> adc@0 {
> diff --git a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> index 7dde7967c886..1e72b8808d24 100644
> --- a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> +++ b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml
> @@ -137,7 +137,7 @@ examples:
>
> cru_parallel_in: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&ov5642>;
> + remote-endpoint = <&ov5642>;
> hsync-active = <1>;
> vsync-active = <1>;
> };
> @@ -150,7 +150,7 @@ examples:
>
> cru_csi_in: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&csi_cru_in>;
> + remote-endpoint = <&csi_cru_in>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/media/renesas,vin.yaml b/Documentation/devicetree/bindings/media/renesas,vin.yaml
> index 91e8f368fb52..324703bfb1bd 100644
> --- a/Documentation/devicetree/bindings/media/renesas,vin.yaml
> +++ b/Documentation/devicetree/bindings/media/renesas,vin.yaml
> @@ -303,11 +303,11 @@ examples:
>
> vin0csi20: endpoint@0 {
> reg = <0>;
> - remote-endpoint= <&csi20vin0>;
> + remote-endpoint = <&csi20vin0>;
> };
> vin0csi40: endpoint@2 {
> reg = <2>;
> - remote-endpoint= <&csi40vin0>;
> + remote-endpoint = <&csi40vin0>;
> };
> };
> };
> diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> index f8c976898a95..18f6733408b4 100644
> --- a/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> +++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.yaml
> @@ -164,7 +164,7 @@ examples:
> reg = <0 0xf80000>;
> };
> firmware@f80000 {
> - label ="firmware";
> + label = "firmware";
> reg = <0xf80000 0x80000>;
> read-only;
> };
> diff --git a/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml b/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> index 0fa2132fa4f4..400aedb58205 100644
> --- a/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> +++ b/Documentation/devicetree/bindings/net/mediatek-dwmac.yaml
> @@ -156,7 +156,7 @@ examples:
> reg = <0x1101c000 0x1300>;
> interrupts = <GIC_SPI 237 IRQ_TYPE_LEVEL_LOW>;
> interrupt-names = "macirq";
> - phy-mode ="rgmii-rxid";
> + phy-mode = "rgmii-rxid";
> mac-address = [00 55 7b b5 7d f7];
> clock-names = "axi",
> "apb",
> diff --git a/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml b/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> index 50f46a6898b1..4adab0149108 100644
> --- a/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> +++ b/Documentation/devicetree/bindings/perf/amlogic,g12-ddr-pmu.yaml
> @@ -42,8 +42,8 @@ examples:
> - |
> #include <dt-bindings/interrupt-controller/arm-gic.h>
> pmu {
> - #address-cells=<2>;
> - #size-cells=<2>;
> + #address-cells = <2>;
> + #size-cells = <2>;
>
> pmu@ff638000 {
> compatible = "amlogic,g12a-ddr-pmu";
> diff --git a/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml b/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> index 26f2b887cfc1..b8d77165c4a1 100644
> --- a/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/mediatek,dsi-phy.yaml
> @@ -83,7 +83,7 @@ examples:
> clocks = <&clk26m>;
> clock-output-names = "mipi_tx0_pll";
> drive-strength-microamp = <4000>;
> - nvmem-cells= <&mipi_tx_calibration>;
> + nvmem-cells = <&mipi_tx_calibration>;
> nvmem-cell-names = "calibration-data";
> #clock-cells = <0>;
> #phy-cells = <0>;
> diff --git a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> index 3100cb870170..76e8ca44906a 100644
> --- a/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/amlogic,meson-mx-ao-arc.yaml
> @@ -75,7 +75,7 @@ additionalProperties: false
> examples:
> - |
> remoteproc@1c {
> - compatible= "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";
> + compatible = "amlogic,meson8-ao-arc", "amlogic,meson-mx-ao-arc";
> reg = <0x1c 0x8>, <0x38 0x8>;
> reg-names = "remap", "cpu";
> resets = <&media_cpu_reset>;
> diff --git a/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml b/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> index 478214ab045e..a59d91243ac8 100644
> --- a/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> +++ b/Documentation/devicetree/bindings/usb/mediatek,mtu3.yaml
> @@ -304,7 +304,7 @@ examples:
> # Dual role switch with type-c
> - |
> usb@11201000 {
> - compatible ="mediatek,mt8183-mtu3", "mediatek,mtu3";
> + compatible = "mediatek,mt8183-mtu3", "mediatek,mtu3";
> reg = <0x11201000 0x2e00>, <0x11203e00 0x0100>;
> reg-names = "mac", "ippc";
> interrupts = <GIC_SPI 72 IRQ_TYPE_LEVEL_LOW>;
> diff --git a/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml b/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> index d25fc708e32c..fec5651f5602 100644
> --- a/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> +++ b/Documentation/devicetree/bindings/usb/ti,am62-usb.yaml
> @@ -92,7 +92,7 @@ examples:
>
> usb@31100000 {
> compatible = "snps,dwc3";
> - reg =<0x00 0x31100000 0x00 0x50000>;
> + reg = <0x00 0x31100000 0x00 0x50000>;
> interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>, /* irq.0 */
> <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>; /* irq.0 */
> interrupt-names = "host", "peripheral";

2023-07-04 13:55:30

by Dmitry Baryshkov

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: cleanup DTS example whitespaces

On Sun, 2 Jul 2023 at 21:23, Krzysztof Kozlowski
<[email protected]> wrote:
>
> The DTS code coding style expects spaces around '=' sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Rob,
>
> Maybe this could go via your tree? Rebased on your for-next:
> v6.4-rc2-45-gf0ac35049606
> ---
> .../bindings/arm/arm,coresight-cti.yaml | 18 +++++++++---------
> .../bindings/arm/keystone/ti,sci.yaml | 8 ++++----
> .../devicetree/bindings/display/msm/gmu.yaml | 2 +-

Acked-by: Dmitry Baryshkov <[email protected]> #display/msm

> .../display/panel/samsung,s6e8aa0.yaml | 2 +-
> .../display/rockchip/rockchip-vop.yaml | 4 ++--
> .../bindings/iio/adc/ti,adc108s102.yaml | 2 +-
> .../bindings/media/renesas,rzg2l-cru.yaml | 4 ++--
> .../devicetree/bindings/media/renesas,vin.yaml | 4 ++--
> .../devicetree/bindings/mtd/mtd-physmap.yaml | 2 +-
> .../bindings/net/mediatek-dwmac.yaml | 2 +-
> .../bindings/perf/amlogic,g12-ddr-pmu.yaml | 4 ++--
> .../bindings/phy/mediatek,dsi-phy.yaml | 2 +-
> .../remoteproc/amlogic,meson-mx-ao-arc.yaml | 2 +-
> .../devicetree/bindings/usb/mediatek,mtu3.yaml | 2 +-
> .../devicetree/bindings/usb/ti,am62-usb.yaml | 2 +-
> 15 files changed, 30 insertions(+), 30 deletions(-)


--
With best wishes
Dmitry

2023-07-04 15:00:20

by Jonas Gorski

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: cleanup DTS example whitespaces

On Sun, 2 Jul 2023 at 20:24, Krzysztof Kozlowski
<[email protected]> wrote:
>
> The DTS code coding style expects spaces around '=' sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Rob,
>
> Maybe this could go via your tree? Rebased on your for-next:
> v6.4-rc2-45-gf0ac35049606
> ---
> .../bindings/arm/arm,coresight-cti.yaml | 18 +++++++++---------
> .../bindings/arm/keystone/ti,sci.yaml | 8 ++++----
> .../devicetree/bindings/display/msm/gmu.yaml | 2 +-
> .../display/panel/samsung,s6e8aa0.yaml | 2 +-
> .../display/rockchip/rockchip-vop.yaml | 4 ++--
> .../bindings/iio/adc/ti,adc108s102.yaml | 2 +-
> .../bindings/media/renesas,rzg2l-cru.yaml | 4 ++--
> .../devicetree/bindings/media/renesas,vin.yaml | 4 ++--
> .../devicetree/bindings/mtd/mtd-physmap.yaml | 2 +-
> .../bindings/net/mediatek-dwmac.yaml | 2 +-
> .../bindings/perf/amlogic,g12-ddr-pmu.yaml | 4 ++--
> .../bindings/phy/mediatek,dsi-phy.yaml | 2 +-
> .../remoteproc/amlogic,meson-mx-ao-arc.yaml | 2 +-
> .../devicetree/bindings/usb/mediatek,mtu3.yaml | 2 +-
> .../devicetree/bindings/usb/ti,am62-usb.yaml | 2 +-
> 15 files changed, 30 insertions(+), 30 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> index 0c5b875cb654..d6c84b6e7fe6 100644
> --- a/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> +++ b/Documentation/devicetree/bindings/arm/arm,coresight-cti.yaml
> @@ -287,7 +287,7 @@ examples:
> arm,trig-in-sigs = <0 1>;
> arm,trig-in-types = <PE_DBGTRIGGER
> PE_PMUIRQ>;
> - arm,trig-out-sigs=<0 1 2 >;
> + arm,trig-out-sigs = <0 1 2 >;

There's a space before the closing > you could drop as well.

Best Regards,
Jonas

2023-07-05 16:16:44

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: cleanup DTS example whitespaces


On Sun, 02 Jul 2023 20:23:08 +0200, Krzysztof Kozlowski wrote:
> The DTS code coding style expects spaces around '=' sign.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Rob,
>
> Maybe this could go via your tree? Rebased on your for-next:
> v6.4-rc2-45-gf0ac35049606
> ---
> .../bindings/arm/arm,coresight-cti.yaml | 18 +++++++++---------
> .../bindings/arm/keystone/ti,sci.yaml | 8 ++++----
> .../devicetree/bindings/display/msm/gmu.yaml | 2 +-
> .../display/panel/samsung,s6e8aa0.yaml | 2 +-
> .../display/rockchip/rockchip-vop.yaml | 4 ++--
> .../bindings/iio/adc/ti,adc108s102.yaml | 2 +-
> .../bindings/media/renesas,rzg2l-cru.yaml | 4 ++--
> .../devicetree/bindings/media/renesas,vin.yaml | 4 ++--
> .../devicetree/bindings/mtd/mtd-physmap.yaml | 2 +-
> .../bindings/net/mediatek-dwmac.yaml | 2 +-
> .../bindings/perf/amlogic,g12-ddr-pmu.yaml | 4 ++--
> .../bindings/phy/mediatek,dsi-phy.yaml | 2 +-
> .../remoteproc/amlogic,meson-mx-ao-arc.yaml | 2 +-
> .../devicetree/bindings/usb/mediatek,mtu3.yaml | 2 +-
> .../devicetree/bindings/usb/ti,am62-usb.yaml | 2 +-
> 15 files changed, 30 insertions(+), 30 deletions(-)
>

Applied, thanks!