2022-05-27 03:20:35

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 1/2] ARM: dts: sti: adjust whitespace around '='

Fix whitespace coding style: use single space instead of tabs or
multiple spaces around '=' sign in property assignment. No functional
changes (same DTB).

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

---

Output compared with dtx_diff and fdtdump.
---
arch/arm/boot/dts/stih407-family.dtsi | 174 +++++++++++++-------------
arch/arm/boot/dts/stih407.dtsi | 4 +-
arch/arm/boot/dts/stih410.dtsi | 4 +-
arch/arm/boot/dts/stihxxx-b2120.dtsi | 8 +-
4 files changed, 95 insertions(+), 95 deletions(-)

diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi
index 1713f7878117..33cf54cb46cc 100644
--- a/arch/arm/boot/dts/stih407-family.dtsi
+++ b/arch/arm/boot/dts/stih407-family.dtsi
@@ -137,8 +137,8 @@ picophyreset: picophyreset-controller {
};

irq-syscfg {
- compatible = "st,stih407-irq-syscfg";
- st,syscfg = <&syscfg_core>;
+ compatible = "st,stih407-irq-syscfg";
+ st,syscfg = <&syscfg_core>;
st,irq-device = <ST_IRQ_SYSCFG_PMU_0>,
<ST_IRQ_SYSCFG_PMU_1>;
st,fiq-device = <ST_IRQ_SYSCFG_DISABLED>,
@@ -157,8 +157,8 @@ usb2_picophy0: phy1 {
miphy28lp_phy: miphy28lp {
compatible = "st,miphy28lp-phy";
st,syscfg = <&syscfg_core>;
- #address-cells = <1>;
- #size-cells = <1>;
+ #address-cells = <1>;
+ #size-cells = <1>;
ranges;

phy_port0: port@9b22000 {
@@ -208,26 +208,26 @@ phy_port2: port@8f95000 {
};

st231_gp0: st231-gp0 {
- compatible = "st,st231-rproc";
- memory-region = <&gp0_reserved>;
- resets = <&softreset STIH407_ST231_GP0_SOFTRESET>;
- reset-names = "sw_reset";
- clocks = <&clk_s_c0_flexgen CLK_ST231_GP_0>;
- clock-frequency = <600000000>;
- st,syscfg = <&syscfg_core 0x22c>;
+ compatible = "st,st231-rproc";
+ memory-region = <&gp0_reserved>;
+ resets = <&softreset STIH407_ST231_GP0_SOFTRESET>;
+ reset-names = "sw_reset";
+ clocks = <&clk_s_c0_flexgen CLK_ST231_GP_0>;
+ clock-frequency = <600000000>;
+ st,syscfg = <&syscfg_core 0x22c>;
#mbox-cells = <1>;
mbox-names = "vq0_rx", "vq0_tx", "vq1_rx", "vq1_tx";
mboxes = <&mailbox0 0 2>, <&mailbox2 0 1>, <&mailbox0 0 3>, <&mailbox2 0 0>;
};

st231_delta: st231-delta {
- compatible = "st,st231-rproc";
- memory-region = <&delta_reserved>;
- resets = <&softreset STIH407_ST231_DMU_SOFTRESET>;
- reset-names = "sw_reset";
- clocks = <&clk_s_c0_flexgen CLK_ST231_DMU>;
- clock-frequency = <600000000>;
- st,syscfg = <&syscfg_core 0x224>;
+ compatible = "st,st231-rproc";
+ memory-region = <&delta_reserved>;
+ resets = <&softreset STIH407_ST231_DMU_SOFTRESET>;
+ reset-names = "sw_reset";
+ clocks = <&clk_s_c0_flexgen CLK_ST231_DMU>;
+ clock-frequency = <600000000>;
+ st,syscfg = <&syscfg_core 0x224>;
#mbox-cells = <1>;
mbox-names = "vq0_rx", "vq0_tx", "vq1_rx", "vq1_tx";
mboxes = <&mailbox0 0 0>, <&mailbox3 0 1>, <&mailbox0 0 1>, <&mailbox3 0 0>;
@@ -710,78 +710,78 @@ sata1: sata@9b28000 {


st_dwc3: dwc3@8f94000 {
- compatible = "st,stih407-dwc3";
- reg = <0x08f94000 0x1000>, <0x110 0x4>;
- reg-names = "reg-glue", "syscfg-reg";
- st,syscfg = <&syscfg_core>;
- resets = <&powerdown STIH407_USB3_POWERDOWN>,
- <&softreset STIH407_MIPHY2_SOFTRESET>;
- reset-names = "powerdown", "softreset";
- #address-cells = <1>;
- #size-cells = <1>;
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_usb3>;
+ compatible = "st,stih407-dwc3";
+ reg = <0x08f94000 0x1000>, <0x110 0x4>;
+ reg-names = "reg-glue", "syscfg-reg";
+ st,syscfg = <&syscfg_core>;
+ resets = <&powerdown STIH407_USB3_POWERDOWN>,
+ <&softreset STIH407_MIPHY2_SOFTRESET>;
+ reset-names = "powerdown", "softreset";
+ #address-cells = <1>;
+ #size-cells = <1>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_usb3>;
ranges;

status = "disabled";

dwc3: dwc3@9900000 {
- compatible = "snps,dwc3";
- reg = <0x09900000 0x100000>;
- interrupts = <GIC_SPI 155 IRQ_TYPE_LEVEL_HIGH>;
- dr_mode = "host";
- phy-names = "usb2-phy", "usb3-phy";
- phys = <&usb2_picophy0>,
- <&phy_port2 PHY_TYPE_USB3>;
+ compatible = "snps,dwc3";
+ reg = <0x09900000 0x100000>;
+ interrupts = <GIC_SPI 155 IRQ_TYPE_LEVEL_HIGH>;
+ dr_mode = "host";
+ phy-names = "usb2-phy", "usb3-phy";
+ phys = <&usb2_picophy0>,
+ <&phy_port2 PHY_TYPE_USB3>;
snps,dis_u3_susphy_quirk;
};
};

/* COMMS PWM Module */
pwm0: pwm@9810000 {
- compatible = "st,sti-pwm";
- #pwm-cells = <2>;
- reg = <0x9810000 0x68>;
- interrupts = <GIC_SPI 128 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_pwm0_chan0_default>;
- clock-names = "pwm";
- clocks = <&clk_sysin>;
+ compatible = "st,sti-pwm";
+ #pwm-cells = <2>;
+ reg = <0x9810000 0x68>;
+ interrupts = <GIC_SPI 128 IRQ_TYPE_LEVEL_HIGH>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_pwm0_chan0_default>;
+ clock-names = "pwm";
+ clocks = <&clk_sysin>;
st,pwm-num-chan = <1>;

- status = "disabled";
+ status = "disabled";
};

/* SBC PWM Module */
pwm1: pwm@9510000 {
- compatible = "st,sti-pwm";
- #pwm-cells = <2>;
- reg = <0x9510000 0x68>;
- interrupts = <GIC_SPI 131 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
- pinctrl-0 = <&pinctrl_pwm1_chan0_default
- &pinctrl_pwm1_chan1_default
- &pinctrl_pwm1_chan2_default
- &pinctrl_pwm1_chan3_default>;
- clock-names = "pwm";
- clocks = <&clk_sysin>;
+ compatible = "st,sti-pwm";
+ #pwm-cells = <2>;
+ reg = <0x9510000 0x68>;
+ interrupts = <GIC_SPI 131 IRQ_TYPE_LEVEL_HIGH>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pinctrl_pwm1_chan0_default
+ &pinctrl_pwm1_chan1_default
+ &pinctrl_pwm1_chan2_default
+ &pinctrl_pwm1_chan3_default>;
+ clock-names = "pwm";
+ clocks = <&clk_sysin>;
st,pwm-num-chan = <4>;

- status = "disabled";
+ status = "disabled";
};

rng10: rng@8a89000 {
- compatible = "st,rng";
- reg = <0x08a89000 0x1000>;
- clocks = <&clk_sysin>;
- status = "okay";
+ compatible = "st,rng";
+ reg = <0x08a89000 0x1000>;
+ clocks = <&clk_sysin>;
+ status = "okay";
};

rng11: rng@8a8a000 {
- compatible = "st,rng";
- reg = <0x08a8a000 0x1000>;
- clocks = <&clk_sysin>;
- status = "okay";
+ compatible = "st,rng";
+ reg = <0x08a8a000 0x1000>;
+ clocks = <&clk_sysin>;
+ status = "okay";
};

ethernet0: dwmac@9630000 {
@@ -812,36 +812,36 @@ ethernet0: dwmac@9630000 {
};

mailbox0: mailbox@8f00000 {
- compatible = "st,stih407-mailbox";
- reg = <0x8f00000 0x1000>;
- interrupts = <GIC_SPI 1 IRQ_TYPE_LEVEL_HIGH>;
- #mbox-cells = <2>;
- mbox-name = "a9";
- status = "okay";
+ compatible = "st,stih407-mailbox";
+ reg = <0x8f00000 0x1000>;
+ interrupts = <GIC_SPI 1 IRQ_TYPE_LEVEL_HIGH>;
+ #mbox-cells = <2>;
+ mbox-name = "a9";
+ status = "okay";
};

mailbox1: mailbox@8f01000 {
- compatible = "st,stih407-mailbox";
- reg = <0x8f01000 0x1000>;
- #mbox-cells = <2>;
- mbox-name = "st231_gp_1";
- status = "okay";
+ compatible = "st,stih407-mailbox";
+ reg = <0x8f01000 0x1000>;
+ #mbox-cells = <2>;
+ mbox-name = "st231_gp_1";
+ status = "okay";
};

mailbox2: mailbox@8f02000 {
- compatible = "st,stih407-mailbox";
- reg = <0x8f02000 0x1000>;
- #mbox-cells = <2>;
- mbox-name = "st231_gp_0";
- status = "okay";
+ compatible = "st,stih407-mailbox";
+ reg = <0x8f02000 0x1000>;
+ #mbox-cells = <2>;
+ mbox-name = "st231_gp_0";
+ status = "okay";
};

mailbox3: mailbox@8f03000 {
- compatible = "st,stih407-mailbox";
- reg = <0x8f03000 0x1000>;
- #mbox-cells = <2>;
- mbox-name = "st231_audio_video";
- status = "okay";
+ compatible = "st,stih407-mailbox";
+ reg = <0x8f03000 0x1000>;
+ #mbox-cells = <2>;
+ mbox-name = "st231_audio_video";
+ status = "okay";
};

/* fdma audio */
@@ -913,7 +913,7 @@ sti_uni_player0: sti-uni-player@8d80000 {
dmas = <&fdma0 2 0 1>;
dma-names = "tx";

- status = "disabled";
+ status = "disabled";
};

sti_uni_player1: sti-uni-player@8d81000 {
diff --git a/arch/arm/boot/dts/stih407.dtsi b/arch/arm/boot/dts/stih407.dtsi
index 9e212b0af89d..aca43d2bdaad 100644
--- a/arch/arm/boot/dts/stih407.dtsi
+++ b/arch/arm/boot/dts/stih407.dtsi
@@ -13,7 +13,7 @@ sti-display-subsystem@0 {
#address-cells = <1>;
#size-cells = <1>;
reg = <0 0>;
- assigned-clocks = <&clk_s_d2_quadfs 0>,
+ assigned-clocks = <&clk_s_d2_quadfs 0>,
<&clk_s_d2_quadfs 1>,
<&clk_s_c0_pll1 0>,
<&clk_s_c0_flexgen CLK_COMPO_DVP>,
@@ -106,7 +106,7 @@ sti_hdmi: sti-hdmi@8d04000 {
reg-names = "hdmi-reg";
#sound-dai-cells = <0>;
interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>;
- interrupt-names = "irq";
+ interrupt-names = "irq";
clock-names = "pix",
"tmds",
"phy",
diff --git a/arch/arm/boot/dts/stih410.dtsi b/arch/arm/boot/dts/stih410.dtsi
index ce2f62cf129b..a39dd5f7bcae 100644
--- a/arch/arm/boot/dts/stih410.dtsi
+++ b/arch/arm/boot/dts/stih410.dtsi
@@ -105,7 +105,7 @@ sti-display-subsystem@0 {
#size-cells = <1>;

reg = <0 0>;
- assigned-clocks = <&clk_s_d2_quadfs 0>,
+ assigned-clocks = <&clk_s_d2_quadfs 0>,
<&clk_s_d2_quadfs 1>,
<&clk_s_c0_pll1 0>,
<&clk_s_c0_flexgen CLK_COMPO_DVP>,
@@ -198,7 +198,7 @@ sti_hdmi: sti-hdmi@8d04000 {
reg-names = "hdmi-reg";
#sound-dai-cells = <0>;
interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>;
- interrupt-names = "irq";
+ interrupt-names = "irq";
clock-names = "pix",
"tmds",
"phy",
diff --git a/arch/arm/boot/dts/stihxxx-b2120.dtsi b/arch/arm/boot/dts/stihxxx-b2120.dtsi
index 4c72dedcd1be..2aa94605d3d4 100644
--- a/arch/arm/boot/dts/stihxxx-b2120.dtsi
+++ b/arch/arm/boot/dts/stihxxx-b2120.dtsi
@@ -175,11 +175,11 @@ demux@8a20000 {

/* tsin0 is TSA on NIMA */
tsin0: port {
- tsin-num = <0>;
+ tsin-num = <0>;
serial-not-parallel;
- i2c-bus = <&ssc2>;
- reset-gpios = <&pio15 4 GPIO_ACTIVE_HIGH>;
- dvb-card = <STV0367_TDA18212_NIMA_1>;
+ i2c-bus = <&ssc2>;
+ reset-gpios = <&pio15 4 GPIO_ACTIVE_HIGH>;
+ dvb-card = <STV0367_TDA18212_NIMA_1>;
};
};

--
2.34.1



2022-05-28 18:50:33

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 2/2] ARM: dts: stm: adjust whitespace around '='

Fix whitespace coding style: use single space instead of tabs or
multiple spaces around '=' sign in property assignment. No functional
changes (same DTB).

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

---

Output compared with dtx_diff and fdtdump.
---
arch/arm/boot/dts/stm32429i-eval.dts | 8 ++++----
arch/arm/boot/dts/stm32h743.dtsi | 4 ++--
arch/arm/boot/dts/stm32h743i-disco.dts | 8 ++++----
arch/arm/boot/dts/stm32h743i-eval.dts | 8 ++++----
arch/arm/boot/dts/stm32h750i-art-pi.dts | 8 ++++----
5 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/arch/arm/boot/dts/stm32429i-eval.dts b/arch/arm/boot/dts/stm32429i-eval.dts
index 0d98aca01736..3de0e9dbe030 100644
--- a/arch/arm/boot/dts/stm32429i-eval.dts
+++ b/arch/arm/boot/dts/stm32429i-eval.dts
@@ -251,10 +251,10 @@ ltdc_out_rgb: endpoint {

&mac {
status = "okay";
- pinctrl-0 = <&ethernet_mii>;
- pinctrl-names = "default";
- phy-mode = "mii";
- phy-handle = <&phy1>;
+ pinctrl-0 = <&ethernet_mii>;
+ pinctrl-names = "default";
+ phy-mode = "mii";
+ phy-handle = <&phy1>;
mdio0 {
#address-cells = <1>;
#size-cells = <0>;
diff --git a/arch/arm/boot/dts/stm32h743.dtsi b/arch/arm/boot/dts/stm32h743.dtsi
index 91dde07a38ba..2059593da21d 100644
--- a/arch/arm/boot/dts/stm32h743.dtsi
+++ b/arch/arm/boot/dts/stm32h743.dtsi
@@ -375,7 +375,7 @@ sdmmc1: mmc@52007000 {
arm,primecell-periphid = <0x10153180>;
reg = <0x52007000 0x1000>;
interrupts = <49>;
- interrupt-names = "cmd_irq";
+ interrupt-names = "cmd_irq";
clocks = <&rcc SDMMC1_CK>;
clock-names = "apb_pclk";
resets = <&rcc STM32H7_AHB3_RESET(SDMMC1)>;
@@ -389,7 +389,7 @@ sdmmc2: mmc@48022400 {
arm,primecell-periphid = <0x10153180>;
reg = <0x48022400 0x400>;
interrupts = <124>;
- interrupt-names = "cmd_irq";
+ interrupt-names = "cmd_irq";
clocks = <&rcc SDMMC2_CK>;
clock-names = "apb_pclk";
resets = <&rcc STM32H7_AHB2_RESET(SDMMC2)>;
diff --git a/arch/arm/boot/dts/stm32h743i-disco.dts b/arch/arm/boot/dts/stm32h743i-disco.dts
index 59e01ce10318..2b452883a708 100644
--- a/arch/arm/boot/dts/stm32h743i-disco.dts
+++ b/arch/arm/boot/dts/stm32h743i-disco.dts
@@ -77,10 +77,10 @@ &clk_hse {

&mac {
status = "disabled";
- pinctrl-0 = <&ethernet_rmii>;
- pinctrl-names = "default";
- phy-mode = "rmii";
- phy-handle = <&phy0>;
+ pinctrl-0 = <&ethernet_rmii>;
+ pinctrl-names = "default";
+ phy-mode = "rmii";
+ phy-handle = <&phy0>;

mdio0 {
#address-cells = <1>;
diff --git a/arch/arm/boot/dts/stm32h743i-eval.dts b/arch/arm/boot/dts/stm32h743i-eval.dts
index 38cc7faf6884..5c5d8059bdc7 100644
--- a/arch/arm/boot/dts/stm32h743i-eval.dts
+++ b/arch/arm/boot/dts/stm32h743i-eval.dts
@@ -115,10 +115,10 @@ &rtc {

&mac {
status = "disabled";
- pinctrl-0 = <&ethernet_rmii>;
- pinctrl-names = "default";
- phy-mode = "rmii";
- phy-handle = <&phy0>;
+ pinctrl-0 = <&ethernet_rmii>;
+ pinctrl-names = "default";
+ phy-mode = "rmii";
+ phy-handle = <&phy0>;

mdio0 {
#address-cells = <1>;
diff --git a/arch/arm/boot/dts/stm32h750i-art-pi.dts b/arch/arm/boot/dts/stm32h750i-art-pi.dts
index 9bb73bb61901..f3e70d3b65ac 100644
--- a/arch/arm/boot/dts/stm32h750i-art-pi.dts
+++ b/arch/arm/boot/dts/stm32h750i-art-pi.dts
@@ -126,10 +126,10 @@ &dma2 {

&mac {
status = "disabled";
- pinctrl-0 = <&ethernet_rmii>;
- pinctrl-names = "default";
- phy-mode = "rmii";
- phy-handle = <&phy0>;
+ pinctrl-0 = <&ethernet_rmii>;
+ pinctrl-names = "default";
+ phy-mode = "rmii";
+ phy-handle = <&phy0>;

mdio0 {
#address-cells = <1>;
--
2.34.1


2022-05-30 07:53:12

by Patrice CHOTARD

[permalink] [raw]
Subject: Re: [PATCH 1/2] ARM: dts: sti: adjust whitespace around '='

Hi Krzysztof

On 5/26/22 22:36, Krzysztof Kozlowski wrote:
> Fix whitespace coding style: use single space instead of tabs or
> multiple spaces around '=' sign in property assignment. No functional
> changes (same DTB).
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---

Reviewed-by: Patrice Chotard <[email protected]>

Thanks


>
> Output compared with dtx_diff and fdtdump.
> ---
> arch/arm/boot/dts/stih407-family.dtsi | 174 +++++++++++++-------------
> arch/arm/boot/dts/stih407.dtsi | 4 +-
> arch/arm/boot/dts/stih410.dtsi | 4 +-
> arch/arm/boot/dts/stihxxx-b2120.dtsi | 8 +-
> 4 files changed, 95 insertions(+), 95 deletions(-)
>
> diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi
> index 1713f7878117..33cf54cb46cc 100644
> --- a/arch/arm/boot/dts/stih407-family.dtsi
> +++ b/arch/arm/boot/dts/stih407-family.dtsi
> @@ -137,8 +137,8 @@ picophyreset: picophyreset-controller {
> };
>
> irq-syscfg {
> - compatible = "st,stih407-irq-syscfg";
> - st,syscfg = <&syscfg_core>;
> + compatible = "st,stih407-irq-syscfg";
> + st,syscfg = <&syscfg_core>;
> st,irq-device = <ST_IRQ_SYSCFG_PMU_0>,
> <ST_IRQ_SYSCFG_PMU_1>;
> st,fiq-device = <ST_IRQ_SYSCFG_DISABLED>,
> @@ -157,8 +157,8 @@ usb2_picophy0: phy1 {
> miphy28lp_phy: miphy28lp {
> compatible = "st,miphy28lp-phy";
> st,syscfg = <&syscfg_core>;
> - #address-cells = <1>;
> - #size-cells = <1>;
> + #address-cells = <1>;
> + #size-cells = <1>;
> ranges;
>
> phy_port0: port@9b22000 {
> @@ -208,26 +208,26 @@ phy_port2: port@8f95000 {
> };
>
> st231_gp0: st231-gp0 {
> - compatible = "st,st231-rproc";
> - memory-region = <&gp0_reserved>;
> - resets = <&softreset STIH407_ST231_GP0_SOFTRESET>;
> - reset-names = "sw_reset";
> - clocks = <&clk_s_c0_flexgen CLK_ST231_GP_0>;
> - clock-frequency = <600000000>;
> - st,syscfg = <&syscfg_core 0x22c>;
> + compatible = "st,st231-rproc";
> + memory-region = <&gp0_reserved>;
> + resets = <&softreset STIH407_ST231_GP0_SOFTRESET>;
> + reset-names = "sw_reset";
> + clocks = <&clk_s_c0_flexgen CLK_ST231_GP_0>;
> + clock-frequency = <600000000>;
> + st,syscfg = <&syscfg_core 0x22c>;
> #mbox-cells = <1>;
> mbox-names = "vq0_rx", "vq0_tx", "vq1_rx", "vq1_tx";
> mboxes = <&mailbox0 0 2>, <&mailbox2 0 1>, <&mailbox0 0 3>, <&mailbox2 0 0>;
> };
>
> st231_delta: st231-delta {
> - compatible = "st,st231-rproc";
> - memory-region = <&delta_reserved>;
> - resets = <&softreset STIH407_ST231_DMU_SOFTRESET>;
> - reset-names = "sw_reset";
> - clocks = <&clk_s_c0_flexgen CLK_ST231_DMU>;
> - clock-frequency = <600000000>;
> - st,syscfg = <&syscfg_core 0x224>;
> + compatible = "st,st231-rproc";
> + memory-region = <&delta_reserved>;
> + resets = <&softreset STIH407_ST231_DMU_SOFTRESET>;
> + reset-names = "sw_reset";
> + clocks = <&clk_s_c0_flexgen CLK_ST231_DMU>;
> + clock-frequency = <600000000>;
> + st,syscfg = <&syscfg_core 0x224>;
> #mbox-cells = <1>;
> mbox-names = "vq0_rx", "vq0_tx", "vq1_rx", "vq1_tx";
> mboxes = <&mailbox0 0 0>, <&mailbox3 0 1>, <&mailbox0 0 1>, <&mailbox3 0 0>;
> @@ -710,78 +710,78 @@ sata1: sata@9b28000 {
>
>
> st_dwc3: dwc3@8f94000 {
> - compatible = "st,stih407-dwc3";
> - reg = <0x08f94000 0x1000>, <0x110 0x4>;
> - reg-names = "reg-glue", "syscfg-reg";
> - st,syscfg = <&syscfg_core>;
> - resets = <&powerdown STIH407_USB3_POWERDOWN>,
> - <&softreset STIH407_MIPHY2_SOFTRESET>;
> - reset-names = "powerdown", "softreset";
> - #address-cells = <1>;
> - #size-cells = <1>;
> - pinctrl-names = "default";
> - pinctrl-0 = <&pinctrl_usb3>;
> + compatible = "st,stih407-dwc3";
> + reg = <0x08f94000 0x1000>, <0x110 0x4>;
> + reg-names = "reg-glue", "syscfg-reg";
> + st,syscfg = <&syscfg_core>;
> + resets = <&powerdown STIH407_USB3_POWERDOWN>,
> + <&softreset STIH407_MIPHY2_SOFTRESET>;
> + reset-names = "powerdown", "softreset";
> + #address-cells = <1>;
> + #size-cells = <1>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_usb3>;
> ranges;
>
> status = "disabled";
>
> dwc3: dwc3@9900000 {
> - compatible = "snps,dwc3";
> - reg = <0x09900000 0x100000>;
> - interrupts = <GIC_SPI 155 IRQ_TYPE_LEVEL_HIGH>;
> - dr_mode = "host";
> - phy-names = "usb2-phy", "usb3-phy";
> - phys = <&usb2_picophy0>,
> - <&phy_port2 PHY_TYPE_USB3>;
> + compatible = "snps,dwc3";
> + reg = <0x09900000 0x100000>;
> + interrupts = <GIC_SPI 155 IRQ_TYPE_LEVEL_HIGH>;
> + dr_mode = "host";
> + phy-names = "usb2-phy", "usb3-phy";
> + phys = <&usb2_picophy0>,
> + <&phy_port2 PHY_TYPE_USB3>;
> snps,dis_u3_susphy_quirk;
> };
> };
>
> /* COMMS PWM Module */
> pwm0: pwm@9810000 {
> - compatible = "st,sti-pwm";
> - #pwm-cells = <2>;
> - reg = <0x9810000 0x68>;
> - interrupts = <GIC_SPI 128 IRQ_TYPE_LEVEL_HIGH>;
> - pinctrl-names = "default";
> - pinctrl-0 = <&pinctrl_pwm0_chan0_default>;
> - clock-names = "pwm";
> - clocks = <&clk_sysin>;
> + compatible = "st,sti-pwm";
> + #pwm-cells = <2>;
> + reg = <0x9810000 0x68>;
> + interrupts = <GIC_SPI 128 IRQ_TYPE_LEVEL_HIGH>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_pwm0_chan0_default>;
> + clock-names = "pwm";
> + clocks = <&clk_sysin>;
> st,pwm-num-chan = <1>;
>
> - status = "disabled";
> + status = "disabled";
> };
>
> /* SBC PWM Module */
> pwm1: pwm@9510000 {
> - compatible = "st,sti-pwm";
> - #pwm-cells = <2>;
> - reg = <0x9510000 0x68>;
> - interrupts = <GIC_SPI 131 IRQ_TYPE_LEVEL_HIGH>;
> - pinctrl-names = "default";
> - pinctrl-0 = <&pinctrl_pwm1_chan0_default
> - &pinctrl_pwm1_chan1_default
> - &pinctrl_pwm1_chan2_default
> - &pinctrl_pwm1_chan3_default>;
> - clock-names = "pwm";
> - clocks = <&clk_sysin>;
> + compatible = "st,sti-pwm";
> + #pwm-cells = <2>;
> + reg = <0x9510000 0x68>;
> + interrupts = <GIC_SPI 131 IRQ_TYPE_LEVEL_HIGH>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_pwm1_chan0_default
> + &pinctrl_pwm1_chan1_default
> + &pinctrl_pwm1_chan2_default
> + &pinctrl_pwm1_chan3_default>;
> + clock-names = "pwm";
> + clocks = <&clk_sysin>;
> st,pwm-num-chan = <4>;
>
> - status = "disabled";
> + status = "disabled";
> };
>
> rng10: rng@8a89000 {
> - compatible = "st,rng";
> - reg = <0x08a89000 0x1000>;
> - clocks = <&clk_sysin>;
> - status = "okay";
> + compatible = "st,rng";
> + reg = <0x08a89000 0x1000>;
> + clocks = <&clk_sysin>;
> + status = "okay";
> };
>
> rng11: rng@8a8a000 {
> - compatible = "st,rng";
> - reg = <0x08a8a000 0x1000>;
> - clocks = <&clk_sysin>;
> - status = "okay";
> + compatible = "st,rng";
> + reg = <0x08a8a000 0x1000>;
> + clocks = <&clk_sysin>;
> + status = "okay";
> };
>
> ethernet0: dwmac@9630000 {
> @@ -812,36 +812,36 @@ ethernet0: dwmac@9630000 {
> };
>
> mailbox0: mailbox@8f00000 {
> - compatible = "st,stih407-mailbox";
> - reg = <0x8f00000 0x1000>;
> - interrupts = <GIC_SPI 1 IRQ_TYPE_LEVEL_HIGH>;
> - #mbox-cells = <2>;
> - mbox-name = "a9";
> - status = "okay";
> + compatible = "st,stih407-mailbox";
> + reg = <0x8f00000 0x1000>;
> + interrupts = <GIC_SPI 1 IRQ_TYPE_LEVEL_HIGH>;
> + #mbox-cells = <2>;
> + mbox-name = "a9";
> + status = "okay";
> };
>
> mailbox1: mailbox@8f01000 {
> - compatible = "st,stih407-mailbox";
> - reg = <0x8f01000 0x1000>;
> - #mbox-cells = <2>;
> - mbox-name = "st231_gp_1";
> - status = "okay";
> + compatible = "st,stih407-mailbox";
> + reg = <0x8f01000 0x1000>;
> + #mbox-cells = <2>;
> + mbox-name = "st231_gp_1";
> + status = "okay";
> };
>
> mailbox2: mailbox@8f02000 {
> - compatible = "st,stih407-mailbox";
> - reg = <0x8f02000 0x1000>;
> - #mbox-cells = <2>;
> - mbox-name = "st231_gp_0";
> - status = "okay";
> + compatible = "st,stih407-mailbox";
> + reg = <0x8f02000 0x1000>;
> + #mbox-cells = <2>;
> + mbox-name = "st231_gp_0";
> + status = "okay";
> };
>
> mailbox3: mailbox@8f03000 {
> - compatible = "st,stih407-mailbox";
> - reg = <0x8f03000 0x1000>;
> - #mbox-cells = <2>;
> - mbox-name = "st231_audio_video";
> - status = "okay";
> + compatible = "st,stih407-mailbox";
> + reg = <0x8f03000 0x1000>;
> + #mbox-cells = <2>;
> + mbox-name = "st231_audio_video";
> + status = "okay";
> };
>
> /* fdma audio */
> @@ -913,7 +913,7 @@ sti_uni_player0: sti-uni-player@8d80000 {
> dmas = <&fdma0 2 0 1>;
> dma-names = "tx";
>
> - status = "disabled";
> + status = "disabled";
> };
>
> sti_uni_player1: sti-uni-player@8d81000 {
> diff --git a/arch/arm/boot/dts/stih407.dtsi b/arch/arm/boot/dts/stih407.dtsi
> index 9e212b0af89d..aca43d2bdaad 100644
> --- a/arch/arm/boot/dts/stih407.dtsi
> +++ b/arch/arm/boot/dts/stih407.dtsi
> @@ -13,7 +13,7 @@ sti-display-subsystem@0 {
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0 0>;
> - assigned-clocks = <&clk_s_d2_quadfs 0>,
> + assigned-clocks = <&clk_s_d2_quadfs 0>,
> <&clk_s_d2_quadfs 1>,
> <&clk_s_c0_pll1 0>,
> <&clk_s_c0_flexgen CLK_COMPO_DVP>,
> @@ -106,7 +106,7 @@ sti_hdmi: sti-hdmi@8d04000 {
> reg-names = "hdmi-reg";
> #sound-dai-cells = <0>;
> interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>;
> - interrupt-names = "irq";
> + interrupt-names = "irq";
> clock-names = "pix",
> "tmds",
> "phy",
> diff --git a/arch/arm/boot/dts/stih410.dtsi b/arch/arm/boot/dts/stih410.dtsi
> index ce2f62cf129b..a39dd5f7bcae 100644
> --- a/arch/arm/boot/dts/stih410.dtsi
> +++ b/arch/arm/boot/dts/stih410.dtsi
> @@ -105,7 +105,7 @@ sti-display-subsystem@0 {
> #size-cells = <1>;
>
> reg = <0 0>;
> - assigned-clocks = <&clk_s_d2_quadfs 0>,
> + assigned-clocks = <&clk_s_d2_quadfs 0>,
> <&clk_s_d2_quadfs 1>,
> <&clk_s_c0_pll1 0>,
> <&clk_s_c0_flexgen CLK_COMPO_DVP>,
> @@ -198,7 +198,7 @@ sti_hdmi: sti-hdmi@8d04000 {
> reg-names = "hdmi-reg";
> #sound-dai-cells = <0>;
> interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>;
> - interrupt-names = "irq";
> + interrupt-names = "irq";
> clock-names = "pix",
> "tmds",
> "phy",
> diff --git a/arch/arm/boot/dts/stihxxx-b2120.dtsi b/arch/arm/boot/dts/stihxxx-b2120.dtsi
> index 4c72dedcd1be..2aa94605d3d4 100644
> --- a/arch/arm/boot/dts/stihxxx-b2120.dtsi
> +++ b/arch/arm/boot/dts/stihxxx-b2120.dtsi
> @@ -175,11 +175,11 @@ demux@8a20000 {
>
> /* tsin0 is TSA on NIMA */
> tsin0: port {
> - tsin-num = <0>;
> + tsin-num = <0>;
> serial-not-parallel;
> - i2c-bus = <&ssc2>;
> - reset-gpios = <&pio15 4 GPIO_ACTIVE_HIGH>;
> - dvb-card = <STV0367_TDA18212_NIMA_1>;
> + i2c-bus = <&ssc2>;
> + reset-gpios = <&pio15 4 GPIO_ACTIVE_HIGH>;
> + dvb-card = <STV0367_TDA18212_NIMA_1>;
> };
> };
>

2022-06-13 08:13:51

by Alexandre TORGUE

[permalink] [raw]
Subject: Re: [PATCH 2/2] ARM: dts: stm: adjust whitespace around '='

Hi Krzysztof

On 5/26/22 22:36, Krzysztof Kozlowski wrote:
> Fix whitespace coding style: use single space instead of tabs or
> multiple spaces around '=' sign in property assignment. No functional
> changes (same DTB).
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>
> ---
>
> Output compared with dtx_diff and fdtdump.
> ---
> arch/arm/boot/dts/stm32429i-eval.dts | 8 ++++----
> arch/arm/boot/dts/stm32h743.dtsi | 4 ++--
> arch/arm/boot/dts/stm32h743i-disco.dts | 8 ++++----
> arch/arm/boot/dts/stm32h743i-eval.dts | 8 ++++----
> arch/arm/boot/dts/stm32h750i-art-pi.dts | 8 ++++----
> 5 files changed, 18 insertions(+), 18 deletions(-)
>
> diff --git a/arch/arm/boot/dts/stm32429i-eval.dts b/arch/arm/boot/dts/stm32429i-eval.dts
> index 0d98aca01736..3de0e9dbe030 100644
> --- a/arch/arm/boot/dts/stm32429i-eval.dts
> +++ b/arch/arm/boot/dts/stm32429i-eval.dts
> @@ -251,10 +251,10 @@ ltdc_out_rgb: endpoint {
>

Applied on stm32-next. I just slightly updated the commit title.

Thanks.
Alex

2022-06-16 20:49:13

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: (subset) [PATCH 1/2] ARM: dts: sti: adjust whitespace around '='

On Thu, 26 May 2022 22:36:31 +0200, Krzysztof Kozlowski wrote:
> Fix whitespace coding style: use single space instead of tabs or
> multiple spaces around '=' sign in property assignment. No functional
> changes (same DTB).
>
>

Applied, thanks!

[1/2] ARM: dts: sti: adjust whitespace around '='
https://git.kernel.org/krzk/linux/c/53f2ade4b0a36a38c3c29134586857d6db6f6dcb

Best regards,
--
Krzysztof Kozlowski <[email protected]>