[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAJ9a7ViDdBeom-pBEOEySN7e78GYjPkqSXvS9Has1aA-egQdLA@mail.gmail.com>
Date: Mon, 3 Jul 2023 10:45:50 +0100
From: Mike Leach <mike.leach@...aro.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Cc: Leo Yan <leo.yan@...aro.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>, coresight@...ts.linaro.org,
linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org,
dri-devel@...ts.freedesktop.org, freedreno@...ts.freedesktop.org,
linux-rockchip@...ts.infradead.org, linux-iio@...r.kernel.org,
linux-media@...r.kernel.org, linux-renesas-soc@...r.kernel.org,
linux-mtd@...ts.infradead.org, linux-amlogic@...ts.infradead.org,
linux-mediatek@...ts.infradead.org, linux-phy@...ts.infradead.org,
linux-remoteproc@...r.kernel.org, linux-usb@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: cleanup DTS example whitespaces
On Sun, 2 Jul 2023 at 19:23, Krzysztof Kozlowski
<krzysztof.kozlowski@...aro.org> wrote:
>
> The DTS code coding style expects spaces around '=' sign.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>
> ---
>
> 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 <mike.leach@...aro.org>
> 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@...a000 {
> - 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@...000 {
> - 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@...38000 {
> 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@...01000 {
> - 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@...00000 {
> 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
> linux-arm-kernel@...ts.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Mike Leach
Principal Engineer, ARM Ltd.
Manchester Design Centre. UK
Powered by blists - more mailing lists