* [PATCH 1/6] arm64: dts: broadcom: Add missing space around status property
2022-06-12 22:31 [PATCH 0/6] arm64: dts: Add missing space around properties Niklas Söderlund
@ 2022-06-12 22:31 ` Niklas Söderlund
2022-06-13 16:49 ` Florian Fainelli
2022-06-12 22:31 ` [PATCH 2/6] arm64: dts: rockchip: Add missing space around regulator-name property Niklas Söderlund
` (6 subsequent siblings)
7 siblings, 1 reply; 14+ messages in thread
From: Niklas Söderlund @ 2022-06-12 22:31 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip, Niklas Söderlund
Add the missing space around the status property before the typo spreads
to other files.
Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
---
arch/arm64/boot/dts/broadcom/stingray/stingray.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/broadcom/stingray/stingray.dtsi b/arch/arm64/boot/dts/broadcom/stingray/stingray.dtsi
index 09d4aa8ae1d6ee89..8f8c25e51194de83 100644
--- a/arch/arm64/boot/dts/broadcom/stingray/stingray.dtsi
+++ b/arch/arm64/boot/dts/broadcom/stingray/stingray.dtsi
@@ -567,7 +567,7 @@ enet: ethernet@340000{
reg-names = "amac_base";
dma-coherent;
interrupts = <GIC_SPI 213 IRQ_TYPE_LEVEL_HIGH>;
- status= "disabled";
+ status = "disabled";
};
nand: nand@360000 {
--
2.36.1
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: [PATCH 1/6] arm64: dts: broadcom: Add missing space around status property
2022-06-12 22:31 ` [PATCH 1/6] arm64: dts: broadcom: Add missing space around status property Niklas Söderlund
@ 2022-06-13 16:49 ` Florian Fainelli
0 siblings, 0 replies; 14+ messages in thread
From: Florian Fainelli @ 2022-06-13 16:49 UTC (permalink / raw)
To: Niklas Söderlund, Rob Herring, Krzysztof Kozlowski,
devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip
On 6/12/22 15:31, Niklas Söderlund wrote:
> Add the missing space around the status property before the typo spreads
> to other files.
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Already carrying an identical patch that should land in v4.20:
https://github.com/Broadcom/stblinux/commit/b65b9477d9118e3cd359245f59af556597bc5b71
--
Florian
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 2/6] arm64: dts: rockchip: Add missing space around regulator-name property
2022-06-12 22:31 [PATCH 0/6] arm64: dts: Add missing space around properties Niklas Söderlund
2022-06-12 22:31 ` [PATCH 1/6] arm64: dts: broadcom: Add missing space around status property Niklas Söderlund
@ 2022-06-12 22:31 ` Niklas Söderlund
2022-06-12 22:31 ` [PATCH 3/6] arm64: dts: freescale: Add missing space around properties Niklas Söderlund
` (5 subsequent siblings)
7 siblings, 0 replies; 14+ messages in thread
From: Niklas Söderlund @ 2022-06-12 22:31 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip, Niklas Söderlund
Add the missing space around the regulator-name property before the typo
spreads to other files.
Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
---
arch/arm64/boot/dts/rockchip/rk3368-orion-r68-meta.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/rockchip/rk3368-orion-r68-meta.dts b/arch/arm64/boot/dts/rockchip/rk3368-orion-r68-meta.dts
index 3ebe15e03cf4dea5..05c50e584e2e62d9 100644
--- a/arch/arm64/boot/dts/rockchip/rk3368-orion-r68-meta.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3368-orion-r68-meta.dts
@@ -134,7 +134,7 @@ vcc_sys: vcc-sys-regulator {
vccio_sd: vcc-io-sd-regulator {
compatible = "regulator-fixed";
- regulator-name= "vccio_sd";
+ regulator-name = "vccio_sd";
regulator-min-microvolt = <1800000>;
regulator-max-microvolt = <3300000>;
regulator-always-on;
--
2.36.1
^ permalink raw reply related [flat|nested] 14+ messages in thread* [PATCH 3/6] arm64: dts: freescale: Add missing space around properties
2022-06-12 22:31 [PATCH 0/6] arm64: dts: Add missing space around properties Niklas Söderlund
2022-06-12 22:31 ` [PATCH 1/6] arm64: dts: broadcom: Add missing space around status property Niklas Söderlund
2022-06-12 22:31 ` [PATCH 2/6] arm64: dts: rockchip: Add missing space around regulator-name property Niklas Söderlund
@ 2022-06-12 22:31 ` Niklas Söderlund
2022-06-16 0:15 ` Krzysztof Kozlowski
2022-06-12 22:31 ` [PATCH 4/6] arm64: dts: hisilicon: " Niklas Söderlund
` (4 subsequent siblings)
7 siblings, 1 reply; 14+ messages in thread
From: Niklas Söderlund @ 2022-06-12 22:31 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip, Niklas Söderlund
Add the missing space around properties before the typo spreads to other
files.
Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
---
arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 6 +++---
arch/arm64/boot/dts/freescale/imx8-ss-conn.dtsi | 2 +-
arch/arm64/boot/dts/freescale/imx8mm.dtsi | 8 ++++----
arch/arm64/boot/dts/freescale/imx8mn.dtsi | 8 ++++----
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 8 ++++----
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 +-
.../arm64/boot/dts/freescale/imx8qxp-colibri-eval-v3.dtsi | 6 +++---
arch/arm64/boot/dts/freescale/imx8ulp.dtsi | 6 +++---
8 files changed, 23 insertions(+), 23 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
index 92465f7776035a14..d4b719c8a0b93faf 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
@@ -120,13 +120,13 @@ pmu {
};
gic: interrupt-controller@6000000 {
- compatible= "arm,gic-v3";
+ compatible = "arm,gic-v3";
#address-cells = <2>;
#size-cells = <2>;
ranges;
- reg= <0x0 0x06000000 0 0x10000>, /* GIC Dist */
+ reg = <0x0 0x06000000 0 0x10000>, /* GIC Dist */
<0x0 0x06040000 0 0x40000>; /* GIC Redistributor */
- #interrupt-cells= <3>;
+ #interrupt-cells = <3>;
interrupt-controller;
interrupts = <GIC_PPI 9 (GIC_CPU_MASK_RAW(0xf) |
IRQ_TYPE_LEVEL_LOW)>;
diff --git a/arch/arm64/boot/dts/freescale/imx8-ss-conn.dtsi b/arch/arm64/boot/dts/freescale/imx8-ss-conn.dtsi
index a79f42a9618ec55f..3e335b4664980189 100644
--- a/arch/arm64/boot/dts/freescale/imx8-ss-conn.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8-ss-conn.dtsi
@@ -54,7 +54,7 @@ usdhc2: mmc@5b020000 {
clock-names = "ipg", "per", "ahb";
power-domains = <&pd IMX_SC_R_SDHC_1>;
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
status = "disabled";
};
diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
index 1bf070473829244d..eb02244fa55441be 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
@@ -212,7 +212,7 @@ clk_ext3: clock-ext3 {
clk_ext4: clock-ext4 {
compatible = "fixed-clock";
#clock-cells = <0>;
- clock-frequency= <133000000>;
+ clock-frequency = <133000000>;
clock-output-names = "clk_ext4";
};
@@ -995,7 +995,7 @@ usdhc1: mmc@30b40000 {
<&clk IMX8MM_CLK_USDHC1_ROOT>;
clock-names = "ipg", "ahb", "per";
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
@@ -1009,7 +1009,7 @@ usdhc2: mmc@30b50000 {
<&clk IMX8MM_CLK_USDHC2_ROOT>;
clock-names = "ipg", "ahb", "per";
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
@@ -1023,7 +1023,7 @@ usdhc3: mmc@30b60000 {
<&clk IMX8MM_CLK_USDHC3_ROOT>;
clock-names = "ipg", "ahb", "per";
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
diff --git a/arch/arm64/boot/dts/freescale/imx8mn.dtsi b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
index e41e1d56f980d511..130c6d90dd79bb4d 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
@@ -212,7 +212,7 @@ clk_ext3: clock-ext3 {
clk_ext4: clock-ext4 {
compatible = "fixed-clock";
#clock-cells = <0>;
- clock-frequency= <133000000>;
+ clock-frequency = <133000000>;
clock-output-names = "clk_ext4";
};
@@ -941,7 +941,7 @@ usdhc1: mmc@30b40000 {
<&clk IMX8MN_CLK_USDHC1_ROOT>;
clock-names = "ipg", "ahb", "per";
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
@@ -955,7 +955,7 @@ usdhc2: mmc@30b50000 {
<&clk IMX8MN_CLK_USDHC2_ROOT>;
clock-names = "ipg", "ahb", "per";
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
@@ -969,7 +969,7 @@ usdhc3: mmc@30b60000 {
<&clk IMX8MN_CLK_USDHC3_ROOT>;
clock-names = "ipg", "ahb", "per";
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
index d9542dfff83fba78..b2f4a5f1f73cbe15 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
@@ -195,7 +195,7 @@ clk_ext3: clock-ext3 {
clk_ext4: clock-ext4 {
compatible = "fixed-clock";
#clock-cells = <0>;
- clock-frequency= <133000000>;
+ clock-frequency = <133000000>;
clock-output-names = "clk_ext4";
};
@@ -903,7 +903,7 @@ usdhc1: mmc@30b40000 {
<&clk IMX8MP_CLK_USDHC1_ROOT>;
clock-names = "ipg", "ahb", "per";
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
@@ -917,7 +917,7 @@ usdhc2: mmc@30b50000 {
<&clk IMX8MP_CLK_USDHC2_ROOT>;
clock-names = "ipg", "ahb", "per";
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
@@ -931,7 +931,7 @@ usdhc3: mmc@30b60000 {
<&clk IMX8MP_CLK_USDHC3_ROOT>;
clock-names = "ipg", "ahb", "per";
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index 49eadb081b19860e..0c74c2a4a40f7615 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -94,7 +94,7 @@ clk_ext3: clock-ext3 {
clk_ext4: clock-ext4 {
compatible = "fixed-clock";
#clock-cells = <0>;
- clock-frequency= <133000000>;
+ clock-frequency = <133000000>;
clock-output-names = "clk_ext4";
};
diff --git a/arch/arm64/boot/dts/freescale/imx8qxp-colibri-eval-v3.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp-colibri-eval-v3.dtsi
index 144fc9e82da751c6..e3f161a10448e007 100644
--- a/arch/arm64/boot/dts/freescale/imx8qxp-colibri-eval-v3.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8qxp-colibri-eval-v3.dtsi
@@ -38,17 +38,17 @@ rtc_i2c: rtc@68 {
/* Colibri UART_B */
&lpuart0 {
- status= "okay";
+ status = "okay";
};
/* Colibri UART_C */
&lpuart2 {
- status= "okay";
+ status = "okay";
};
/* Colibri UART_A */
&lpuart3 {
- status= "okay";
+ status = "okay";
};
/* Colibri FastEthernet */
diff --git a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
index 09f7364dd1d05a11..ba281e689cf6ed26 100644
--- a/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8ulp.dtsi
@@ -331,7 +331,7 @@ usdhc0: mmc@298d0000 {
clock-names = "ipg", "ahb", "per";
power-domains = <&scmi_devpd IMX8ULP_PD_USDHC0>;
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
@@ -346,7 +346,7 @@ usdhc1: mmc@298e0000 {
clock-names = "ipg", "ahb", "per";
power-domains = <&scmi_devpd IMX8ULP_PD_USDHC1>;
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
@@ -361,7 +361,7 @@ usdhc2: mmc@298f0000 {
clock-names = "ipg", "ahb", "per";
power-domains = <&scmi_devpd IMX8ULP_PD_USDHC2_USB1>;
fsl,tuning-start-tap = <20>;
- fsl,tuning-step= <2>;
+ fsl,tuning-step = <2>;
bus-width = <4>;
status = "disabled";
};
--
2.36.1
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: [PATCH 3/6] arm64: dts: freescale: Add missing space around properties
2022-06-12 22:31 ` [PATCH 3/6] arm64: dts: freescale: Add missing space around properties Niklas Söderlund
@ 2022-06-16 0:15 ` Krzysztof Kozlowski
0 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2022-06-16 0:15 UTC (permalink / raw)
To: Niklas Söderlund, Rob Herring, Krzysztof Kozlowski,
devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip
On 12/06/2022 15:31, Niklas Söderlund wrote:
> Add the missing space around properties before the typo spreads to other
> files.
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> ---
> arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 6 +++---
> arch/arm64/boot/dts/freescale/imx8-ss-conn.dtsi | 2 +-
> arch/arm64/boot/dts/freescale/imx8mm.dtsi | 8 ++++----
> arch/arm64/boot/dts/freescale/imx8mn.dtsi | 8 ++++----
> arch/arm64/boot/dts/freescale/imx8mp.dtsi | 8 ++++----
> arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 +-
> .../arm64/boot/dts/freescale/imx8qxp-colibri-eval-v3.dtsi | 6 +++---
> arch/arm64/boot/dts/freescale/imx8ulp.dtsi | 6 +++---
> 8 files changed, 23 insertions(+), 23 deletions(-)
Please rebase on top of earlier patch:
https://lore.kernel.org/all/20220526204258.832179-1-krzysztof.kozlowski@linaro.org/
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 4/6] arm64: dts: hisilicon: Add missing space around properties
2022-06-12 22:31 [PATCH 0/6] arm64: dts: Add missing space around properties Niklas Söderlund
` (2 preceding siblings ...)
2022-06-12 22:31 ` [PATCH 3/6] arm64: dts: freescale: Add missing space around properties Niklas Söderlund
@ 2022-06-12 22:31 ` Niklas Söderlund
2022-06-16 0:15 ` Krzysztof Kozlowski
2022-06-12 22:32 ` [PATCH 5/6] arm64: dts: qcom: " Niklas Söderlund
` (3 subsequent siblings)
7 siblings, 1 reply; 14+ messages in thread
From: Niklas Söderlund @ 2022-06-12 22:31 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip, Niklas Söderlund
Add the missing space around properties before the typo spreads to other
files.
Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
---
arch/arm64/boot/dts/hisilicon/hip06.dtsi | 6 +++---
arch/arm64/boot/dts/hisilicon/hip07.dtsi | 6 +++---
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/arch/arm64/boot/dts/hisilicon/hip06.dtsi b/arch/arm64/boot/dts/hisilicon/hip06.dtsi
index 70d7732dd348552d..2f8b03b0d36537ca 100644
--- a/arch/arm64/boot/dts/hisilicon/hip06.dtsi
+++ b/arch/arm64/boot/dts/hisilicon/hip06.dtsi
@@ -537,7 +537,7 @@ port@0 {
port@1 {
reg = <1>;
- serdes-syscon= <&serdes_ctrl>;
+ serdes-syscon = <&serdes_ctrl>;
port-rst-offset = <1>;
port-mode-offset = <1>;
media-type = "fiber";
@@ -546,7 +546,7 @@ port@1 {
port@4 {
reg = <4>;
phy-handle = <&phy0>;
- serdes-syscon= <&serdes_ctrl>;
+ serdes-syscon = <&serdes_ctrl>;
port-rst-offset = <4>;
port-mode-offset = <2>;
media-type = "copper";
@@ -555,7 +555,7 @@ port@4 {
port@5 {
reg = <5>;
phy-handle = <&phy1>;
- serdes-syscon= <&serdes_ctrl>;
+ serdes-syscon = <&serdes_ctrl>;
port-rst-offset = <5>;
port-mode-offset = <3>;
media-type = "copper";
diff --git a/arch/arm64/boot/dts/hisilicon/hip07.dtsi b/arch/arm64/boot/dts/hisilicon/hip07.dtsi
index 6baf6a686450ee28..1a16662f8867ba1e 100644
--- a/arch/arm64/boot/dts/hisilicon/hip07.dtsi
+++ b/arch/arm64/boot/dts/hisilicon/hip07.dtsi
@@ -1422,7 +1422,7 @@ port@0 {
port@1 {
reg = <1>;
- serdes-syscon= <&serdes_ctrl>;
+ serdes-syscon = <&serdes_ctrl>;
cpld-syscon = <&dsa_cpld 0x4>;
port-rst-offset = <1>;
port-mode-offset = <1>;
@@ -1433,7 +1433,7 @@ port@1 {
port@4 {
reg = <4>;
phy-handle = <&phy0>;
- serdes-syscon= <&serdes_ctrl>;
+ serdes-syscon = <&serdes_ctrl>;
port-rst-offset = <4>;
port-mode-offset = <2>;
mc-mac-mask = [ff f0 00 00 00 00];
@@ -1443,7 +1443,7 @@ port@4 {
port@5 {
reg = <5>;
phy-handle = <&phy1>;
- serdes-syscon= <&serdes_ctrl>;
+ serdes-syscon = <&serdes_ctrl>;
port-rst-offset = <5>;
port-mode-offset = <3>;
mc-mac-mask = [ff f0 00 00 00 00];
--
2.36.1
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: [PATCH 4/6] arm64: dts: hisilicon: Add missing space around properties
2022-06-12 22:31 ` [PATCH 4/6] arm64: dts: hisilicon: " Niklas Söderlund
@ 2022-06-16 0:15 ` Krzysztof Kozlowski
0 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2022-06-16 0:15 UTC (permalink / raw)
To: Niklas Söderlund, Rob Herring, Krzysztof Kozlowski,
devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip
On 12/06/2022 15:31, Niklas Söderlund wrote:
> Add the missing space around properties before the typo spreads to other
> files.
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> ---
> arch/arm64/boot/dts/hisilicon/hip06.dtsi | 6 +++---
> arch/arm64/boot/dts/hisilicon/hip07.dtsi | 6 +++---
> 2 files changed, 6 insertions(+), 6 deletions(-)
This (and other similar issues) is fixed here:
https://lore.kernel.org/all/20220526204453.832681-1-krzysztof.kozlowski@linaro.org/
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 5/6] arm64: dts: qcom: Add missing space around properties
2022-06-12 22:31 [PATCH 0/6] arm64: dts: Add missing space around properties Niklas Söderlund
` (3 preceding siblings ...)
2022-06-12 22:31 ` [PATCH 4/6] arm64: dts: hisilicon: " Niklas Söderlund
@ 2022-06-12 22:32 ` Niklas Söderlund
2022-06-16 0:16 ` Krzysztof Kozlowski
2022-06-12 22:32 ` [PATCH 6/6] arm64: dts: ti: " Niklas Söderlund
` (2 subsequent siblings)
7 siblings, 1 reply; 14+ messages in thread
From: Niklas Söderlund @ 2022-06-12 22:32 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip, Niklas Söderlund
Add the missing space around properties before the typo spreads to other
files
Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
---
arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile.dtsi | 2 +-
arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile.dtsi b/arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile.dtsi
index 42af1fade461bdfa..03c18c74da3081a3 100644
--- a/arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile.dtsi
@@ -34,7 +34,7 @@ framebuffer0: framebuffer@9d400000 {
height = <1920>;
stride = <(1080 * 4)>;
format = "a8r8g8b8";
- status= "okay";
+ status = "okay";
};
};
diff --git a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
index f1619b3f97ef80d0..eae185ced74a598c 100644
--- a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
+++ b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
@@ -817,5 +817,5 @@ &wifi {
&crypto {
/* FIXME: qce_start triggers an SError */
- status= "disable";
+ status = "disable";
};
--
2.36.1
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: [PATCH 5/6] arm64: dts: qcom: Add missing space around properties
2022-06-12 22:32 ` [PATCH 5/6] arm64: dts: qcom: " Niklas Söderlund
@ 2022-06-16 0:16 ` Krzysztof Kozlowski
0 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2022-06-16 0:16 UTC (permalink / raw)
To: Niklas Söderlund, Rob Herring, Krzysztof Kozlowski,
devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip
On 12/06/2022 15:32, Niklas Söderlund wrote:
> Add the missing space around properties before the typo spreads to other
> files
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> ---
> arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
https://lore.kernel.org/all/20220526204248.832139-1-krzysztof.kozlowski@linaro.org/
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 6/6] arm64: dts: ti: Add missing space around properties
2022-06-12 22:31 [PATCH 0/6] arm64: dts: Add missing space around properties Niklas Söderlund
` (4 preceding siblings ...)
2022-06-12 22:32 ` [PATCH 5/6] arm64: dts: qcom: " Niklas Söderlund
@ 2022-06-12 22:32 ` Niklas Söderlund
2022-06-16 0:16 ` Krzysztof Kozlowski
2022-06-12 22:58 ` (subset) [PATCH 0/6] arm64: dts: " Heiko Stuebner
2022-06-16 0:14 ` Krzysztof Kozlowski
7 siblings, 1 reply; 14+ messages in thread
From: Niklas Söderlund @ 2022-06-12 22:32 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip, Niklas Söderlund
Add the missing space around properties before the typo spreads to other
files.
Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
---
arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 4 ++--
arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 4 ++--
arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 2 +-
arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 4 ++--
arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 2 +-
arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 4 ++--
arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 4 ++--
arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi | 4 ++--
8 files changed, 14 insertions(+), 14 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
index d08abad0bcf4e58c..df3b9883e88753d5 100644
--- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
@@ -144,8 +144,8 @@ dmsc: system-controller@44043000 {
compatible = "ti,k2g-sci";
ti,host-id = <12>;
mbox-names = "rx", "tx";
- mboxes= <&secure_proxy_main 12>,
- <&secure_proxy_main 13>;
+ mboxes = <&secure_proxy_main 12>,
+ <&secure_proxy_main 13>;
reg-names = "debug_messages";
reg = <0x00 0x44043000 0x00 0xfe0>;
diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
index f64b368c6c371757..7bfddd35eb93e4e8 100644
--- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
@@ -174,8 +174,8 @@ dmsc: system-controller@44043000 {
compatible = "ti,k2g-sci";
ti,host-id = <12>;
mbox-names = "rx", "tx";
- mboxes= <&secure_proxy_main 12>,
- <&secure_proxy_main 13>;
+ mboxes = <&secure_proxy_main 12>,
+ <&secure_proxy_main 13>;
reg-names = "debug_messages";
reg = <0x00 0x44043000 0x00 0xfe0>;
diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
index 6e41f2fa044af9f3..d050c410c6917a8f 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
@@ -574,7 +574,7 @@ &mcu_spi0 {
pinctrl-0 = <&mcu_spi0_pins_default>;
#address-cells = <1>;
- #size-cells= <0>;
+ #size-cells = <0>;
ti,pindir-d0-out-d1-in;
};
diff --git a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
index 9c69d0917f69ac9c..fa11d7142006a72a 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi
@@ -12,8 +12,8 @@ dmsc: system-controller@44083000 {
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/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
index 57497cb1ed68ecb3..048228c89d5b58d7 100644
--- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
@@ -330,7 +330,7 @@ &main_spi0 {
pinctrl-names = "default";
pinctrl-0 = <&main_spi0_pins_default>;
#address-cells = <1>;
- #size-cells= <0>;
+ #size-cells = <0>;
ti,pindir-d0-out-d1-in;
flash@0 {
diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
index 1044ec6c4b0d4b10..ff13bbeed30c91e6 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
@@ -12,8 +12,8 @@ dmsc: system-controller@44083000 {
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 = <0x00 0x44083000 0x00 0x1000>;
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
index b4972dfb7da815a0..df08724bbf1c51c3 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
@@ -12,8 +12,8 @@ dmsc: system-controller@44083000 {
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 = <0x00 0x44083000 0x0 0x1000>;
diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
index 6c5c02edb375db32..4d1bfabd1313a560 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
@@ -12,8 +12,8 @@ sms: system-controller@44083000 {
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 = <0x00 0x44083000 0x00 0x1000>;
--
2.36.1
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: [PATCH 6/6] arm64: dts: ti: Add missing space around properties
2022-06-12 22:32 ` [PATCH 6/6] arm64: dts: ti: " Niklas Söderlund
@ 2022-06-16 0:16 ` Krzysztof Kozlowski
0 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2022-06-16 0:16 UTC (permalink / raw)
To: Niklas Söderlund, Rob Herring, Krzysztof Kozlowski,
devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip
On 12/06/2022 15:32, Niklas Söderlund wrote:
> Add the missing space around properties before the typo spreads to other
> files.
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> ---
> arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 4 ++--
> arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 4 ++--
> arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 2 +-
> arch/arm64/boot/dts/ti/k3-am65-wakeup.dtsi | 4 ++--
> arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 2 +-
> arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 4 ++--
> arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 4 ++--
> arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi | 4 ++--
> 8 files changed, 14 insertions(+), 14 deletions(-)
https://lore.kernel.org/all/20220526204139.831895-1-krzysztof.kozlowski@linaro.org/
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: (subset) [PATCH 0/6] arm64: dts: Add missing space around properties
2022-06-12 22:31 [PATCH 0/6] arm64: dts: Add missing space around properties Niklas Söderlund
` (5 preceding siblings ...)
2022-06-12 22:32 ` [PATCH 6/6] arm64: dts: ti: " Niklas Söderlund
@ 2022-06-12 22:58 ` Heiko Stuebner
2022-06-16 0:14 ` Krzysztof Kozlowski
7 siblings, 0 replies; 14+ messages in thread
From: Heiko Stuebner @ 2022-06-12 22:58 UTC (permalink / raw)
To: devicetree, Niklas Söderlund, Rob Herring,
Krzysztof Kozlowski
Cc: Heiko Stuebner, Nishanth Menon, Ray Jui, Fabio Estevam,
Sascha Hauer, linux-imx, Scott Branden, kernel,
bcm-kernel-feedback-list, linux-rockchip, Tero Kristo,
linux-arm-msm, Wei Xu, Bjorn Andersson, Shawn Guo,
linux-arm-kernel, Andy Gross, Li Yang, Vignesh Raghavendra
On Mon, 13 Jun 2022 00:31:55 +0200, Niklas Söderlund wrote:
> While debugging an issue on a Renesas board I found a typo with missing
> spaces around properties that had been copied around [1]. This extends
> this fix for all arm64 files.
>
> 1. https://lore.kernel.org/linux-renesas-soc/20220608175728.1012550-1-niklas.soderlund+renesas@ragnatech.se/
>
> Niklas Söderlund (6):
> arm64: dts: broadcom: Add missing space around status property
> arm64: dts: rockchip: Add missing space around regulator-name property
> arm64: dts: freescale: Add missing space around properties
> arm64: dts: hisilicon: Add missing space around properties
> arm64: dts: qcom: Add missing space around properties
> arm64: dts: ti: Add missing space around properties
>
> [...]
Applied, thanks!
[2/6] arm64: dts: rockchip: Add missing space around regulator-name property
commit: d040e81bec407057b6dea9dadfe5e15b9845b010
Best regards,
--
Heiko Stuebner <heiko@sntech.de>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH 0/6] arm64: dts: Add missing space around properties
2022-06-12 22:31 [PATCH 0/6] arm64: dts: Add missing space around properties Niklas Söderlund
` (6 preceding siblings ...)
2022-06-12 22:58 ` (subset) [PATCH 0/6] arm64: dts: " Heiko Stuebner
@ 2022-06-16 0:14 ` Krzysztof Kozlowski
7 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2022-06-16 0:14 UTC (permalink / raw)
To: Niklas Söderlund, Rob Herring, Krzysztof Kozlowski,
devicetree
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Shawn Guo,
Li Yang, Sascha Hauer, kernel, Fabio Estevam, linux-imx, Wei Xu,
Andy Gross, Bjorn Andersson, Heiko Stuebner, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo, linux-arm-kernel, linux-arm-msm,
linux-rockchip
On 12/06/2022 15:31, Niklas Söderlund wrote:
> Hello,
>
> While debugging an issue on a Renesas board I found a typo with missing
> spaces around properties that had been copied around [1]. This extends
> this fix for all arm64 files.
>
> 1. https://lore.kernel.org/linux-renesas-soc/20220608175728.1012550-1-niklas.soderlund+renesas@ragnatech.se/
>
> Niklas Söderlund (6):
> arm64: dts: broadcom: Add missing space around status property
> arm64: dts: rockchip: Add missing space around regulator-name property
> arm64: dts: freescale: Add missing space around properties
> arm64: dts: hisilicon: Add missing space around properties
> arm64: dts: qcom: Add missing space around properties
> arm64: dts: ti: Add missing space around properties
I think you just did quite the same work already posted:
https://lore.kernel.org/all/?q=f%3Akrzysztof+%22adjust+whitespace+around%22
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread