From: Henrik Grimler <henrik.grimler@axis.com>
To: Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Peter Griffin <peter.griffin@linaro.org>,
Alim Akhtar <alim.akhtar@samsung.com>
Cc: devicetree@vger.kernel.org, waqar.hameed@axis.com,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
Henrik Grimler <henrik.grimler@axis.com>
Subject: [PATCH v2 03/17] ARM: dts: samsung: exynos4: sort all properties
Date: Tue, 06 Oct 2026 13:57:17 +0200 [thread overview]
Message-ID: <20261006-exynos-arm-cleanup-v2-3-e8e86a3c8c1f@axis.com> (raw)
In-Reply-To: <20261006-exynos-arm-cleanup-v2-0-e8e86a3c8c1f@axis.com>
Fixes dt-check-style in strict mode property-order warnings like:
samsung/exynos4.dtsi:226: [property-order] property 'iommus' out of canonical order (should sort before 'samsung,sysreg')
Signed-off-by: Henrik Grimler <henrik.grimler@axis.com>
---
arch/arm/boot/dts/samsung/exynos4.dtsi | 89 +++++++++++++++++-----------------
1 file changed, 45 insertions(+), 44 deletions(-)
diff --git a/arch/arm/boot/dts/samsung/exynos4.dtsi b/arch/arm/boot/dts/samsung/exynos4.dtsi
index f69b01423ae0..43cbbedbfe41 100644
--- a/arch/arm/boot/dts/samsung/exynos4.dtsi
+++ b/arch/arm/boot/dts/samsung/exynos4.dtsi
@@ -60,9 +60,9 @@ pmu: pmu {
soc: soc {
compatible = "simple-bus";
+ ranges;
#address-cells = <1>;
#size-cells = <1>;
- ranges;
clock_audss: clock-controller@3810000 {
compatible = "samsung,exynos4210-audss-clock";
@@ -86,8 +86,8 @@ i2s0: i2s@3830000 {
clock-output-names = "i2s_cdclk0";
dmas = <&pdma0 12>, <&pdma0 11>, <&pdma0 10>;
dma-names = "tx", "rx", "tx-sec";
- samsung,idma-addr = <0x03000000>;
#sound-dai-cells = <1>;
+ samsung,idma-addr = <0x03000000>;
status = "disabled";
};
@@ -159,16 +159,16 @@ pd_gps_alive: power-domain@10023d00 {
gic: interrupt-controller@10490000 {
compatible = "arm,cortex-a9-gic";
+ reg = <0x10490000 0x10000>, <0x10480000 0x10000>;
#interrupt-cells = <3>;
interrupt-controller;
- reg = <0x10490000 0x10000>, <0x10480000 0x10000>;
};
combiner: interrupt-controller@10440000 {
compatible = "samsung,exynos4210-combiner";
+ reg = <0x10440000 0x1000>;
#interrupt-cells = <2>;
interrupt-controller;
- reg = <0x10440000 0x1000>;
};
sys_reg: syscon@10010000 {
@@ -198,22 +198,23 @@ dsi_0: dsi@11c80000 {
phy-names = "dsim";
clocks = <&clock CLK_DSIM0>, <&clock CLK_SCLK_MIPI0>;
clock-names = "bus_clk", "sclk_mipi";
-
- status = "disabled";
#address-cells = <1>;
#size-cells = <0>;
+
+ status = "disabled";
};
camera: camera@11800000 {
compatible = "samsung,fimc";
ranges = <0x0 0x11800000 0xa0000>;
- status = "disabled";
#address-cells = <1>;
#size-cells = <1>;
#clock-cells = <1>;
clock-output-names = "cam_a_clkout", "cam_b_clkout";
+ status = "disabled";
+
fimc_0: fimc@0 {
compatible = "samsung,exynos4210-fimc";
reg = <0x0 0x1000>;
@@ -221,9 +222,9 @@ fimc_0: fimc@0 {
clocks = <&clock CLK_FIMC0>,
<&clock CLK_SCLK_FIMC0>;
clock-names = "fimc", "sclk_fimc";
+ iommus = <&sysmmu_fimc0>;
power-domains = <&pd_cam>;
samsung,sysreg = <&sys_reg>;
- iommus = <&sysmmu_fimc0>;
status = "disabled";
};
@@ -235,9 +236,9 @@ fimc_1: fimc@10000 {
clocks = <&clock CLK_FIMC1>,
<&clock CLK_SCLK_FIMC1>;
clock-names = "fimc", "sclk_fimc";
+ iommus = <&sysmmu_fimc1>;
power-domains = <&pd_cam>;
samsung,sysreg = <&sys_reg>;
- iommus = <&sysmmu_fimc1>;
status = "disabled";
};
@@ -249,9 +250,9 @@ fimc_2: fimc@20000 {
clocks = <&clock CLK_FIMC2>,
<&clock CLK_SCLK_FIMC2>;
clock-names = "fimc", "sclk_fimc";
+ iommus = <&sysmmu_fimc2>;
power-domains = <&pd_cam>;
samsung,sysreg = <&sys_reg>;
- iommus = <&sysmmu_fimc2>;
status = "disabled";
};
@@ -263,9 +264,9 @@ fimc_3: fimc@30000 {
clocks = <&clock CLK_FIMC3>,
<&clock CLK_SCLK_FIMC3>;
clock-names = "fimc", "sclk_fimc";
+ iommus = <&sysmmu_fimc3>;
power-domains = <&pd_cam>;
samsung,sysreg = <&sys_reg>;
- iommus = <&sysmmu_fimc3>;
status = "disabled";
};
@@ -281,10 +282,10 @@ csis_0: csis@80000 {
power-domains = <&pd_cam>;
phys = <&mipi_phy 0>;
phy-names = "csis";
-
- status = "disabled";
#address-cells = <1>;
#size-cells = <0>;
+
+ status = "disabled";
};
csis_1: csis@90000 {
@@ -298,10 +299,10 @@ csis_1: csis@90000 {
power-domains = <&pd_cam>;
phys = <&mipi_phy 2>;
phy-names = "csis";
-
- status = "disabled";
#address-cells = <1>;
#size-cells = <0>;
+
+ status = "disabled";
};
};
@@ -370,10 +371,10 @@ sdhci_3: mmc@12540000 {
exynos_usbphy: usb-phy@125b0000 {
compatible = "samsung,exynos4210-usb2-phy";
reg = <0x125b0000 0x100>;
- samsung,pmureg-phandle = <&pmu_system_controller>;
clocks = <&clock CLK_USB_DEVICE>, <&clock CLK_XUSBXTI>;
clock-names = "phy", "ref";
#phy-cells = <1>;
+ samsung,pmureg-phandle = <&pmu_system_controller>;
status = "disabled";
};
@@ -396,10 +397,10 @@ ehci: usb@12580000 {
interrupts = <GIC_SPI 70 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_USB_HOST>;
clock-names = "usbhost";
-
- status = "disabled";
phys = <&exynos_usbphy 1>, <&exynos_usbphy 2>, <&exynos_usbphy 3>;
phy-names = "host", "hsic0", "hsic1";
+
+ status = "disabled";
};
ohci: usb@12590000 {
@@ -408,10 +409,10 @@ ohci: usb@12590000 {
interrupts = <GIC_SPI 70 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_USB_HOST>;
clock-names = "usbhost";
-
- status = "disabled";
phys = <&exynos_usbphy 1>;
phy-names = "host";
+
+ status = "disabled";
};
gpu: gpu@13000000 {
@@ -517,10 +518,10 @@ serial_3: serial@13830000 {
};
i2c_0: i2c@13860000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "samsung,s3c2440-i2c";
reg = <0x13860000 0x100>;
+ #address-cells = <1>;
+ #size-cells = <0>;
interrupts = <GIC_SPI 58 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_I2C0>;
clock-names = "i2c";
@@ -531,10 +532,10 @@ i2c_0: i2c@13860000 {
};
i2c_1: i2c@13870000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "samsung,s3c2440-i2c";
reg = <0x13870000 0x100>;
+ #address-cells = <1>;
+ #size-cells = <0>;
interrupts = <GIC_SPI 59 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_I2C1>;
clock-names = "i2c";
@@ -545,10 +546,10 @@ i2c_1: i2c@13870000 {
};
i2c_2: i2c@13880000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "samsung,s3c2440-i2c";
reg = <0x13880000 0x100>;
+ #address-cells = <1>;
+ #size-cells = <0>;
interrupts = <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_I2C2>;
clock-names = "i2c";
@@ -559,10 +560,10 @@ i2c_2: i2c@13880000 {
};
i2c_3: i2c@13890000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "samsung,s3c2440-i2c";
reg = <0x13890000 0x100>;
+ #address-cells = <1>;
+ #size-cells = <0>;
interrupts = <GIC_SPI 61 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_I2C3>;
clock-names = "i2c";
@@ -573,10 +574,10 @@ i2c_3: i2c@13890000 {
};
i2c_4: i2c@138a0000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "samsung,s3c2440-i2c";
reg = <0x138a0000 0x100>;
+ #address-cells = <1>;
+ #size-cells = <0>;
interrupts = <GIC_SPI 62 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_I2C4>;
clock-names = "i2c";
@@ -587,10 +588,10 @@ i2c_4: i2c@138a0000 {
};
i2c_5: i2c@138b0000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "samsung,s3c2440-i2c";
reg = <0x138b0000 0x100>;
+ #address-cells = <1>;
+ #size-cells = <0>;
interrupts = <GIC_SPI 63 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_I2C5>;
clock-names = "i2c";
@@ -601,10 +602,10 @@ i2c_5: i2c@138b0000 {
};
i2c_6: i2c@138c0000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "samsung,s3c2440-i2c";
reg = <0x138c0000 0x100>;
+ #address-cells = <1>;
+ #size-cells = <0>;
interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_I2C6>;
clock-names = "i2c";
@@ -615,10 +616,10 @@ i2c_6: i2c@138c0000 {
};
i2c_7: i2c@138d0000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "samsung,s3c2440-i2c";
reg = <0x138d0000 0x100>;
+ #address-cells = <1>;
+ #size-cells = <0>;
interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_I2C7>;
clock-names = "i2c";
@@ -629,10 +630,10 @@ i2c_7: i2c@138d0000 {
};
i2c_8: i2c@138e0000 {
- #address-cells = <1>;
- #size-cells = <0>;
compatible = "samsung,s3c2440-hdmiphy-i2c";
reg = <0x138e0000 0x100>;
+ #address-cells = <1>;
+ #size-cells = <0>;
interrupts = <GIC_SPI 93 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_I2C_HDMI>;
clock-names = "i2c";
@@ -740,8 +741,8 @@ mdma1: dma-controller@12850000 {
fimd: fimd@11c00000 {
compatible = "samsung,exynos4210-fimd";
- interrupt-parent = <&combiner>;
reg = <0x11c00000 0x20000>;
+ interrupt-parent = <&combiner>;
interrupts = <11 0>, <11 1>, <11 2>;
interrupt-names = "fifo", "vsync", "lcd_sys";
clocks = <&clock CLK_SCLK_FIMD0>, <&clock CLK_FIMD0>;
@@ -754,12 +755,12 @@ fimd: fimd@11c00000 {
};
tmu: tmu@100c0000 {
- interrupt-parent = <&combiner>;
reg = <0x100c0000 0x100>;
+ interrupt-parent = <&combiner>;
interrupts = <2 4>;
+ #thermal-sensor-cells = <0>;
status = "disabled";
- #thermal-sensor-cells = <0>;
};
jpeg_codec: jpeg-codec@11840000 {
@@ -793,8 +794,8 @@ hdmi: hdmi@12d00000 {
"sclk_hdmiphy", "mout_hdmi";
phy = <&hdmi_i2c_phy>;
power-domains = <&pd_tv>;
- samsung,syscon-phandle = <&pmu_system_controller>;
#sound-dai-cells = <0>;
+ samsung,syscon-phandle = <&pmu_system_controller>;
status = "disabled";
};
@@ -805,18 +806,18 @@ hdmicec: cec@100b0000 {
interrupts = <GIC_SPI 114 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clock CLK_HDMI_CEC>;
clock-names = "hdmicec";
- samsung,syscon-phandle = <&pmu_system_controller>;
hdmi-phandle = <&hdmi>;
pinctrl-0 = <&hdmi_cec>;
pinctrl-names = "default";
+ samsung,syscon-phandle = <&pmu_system_controller>;
status = "disabled";
};
mixer: mixer@12c10000 {
compatible = "samsung,exynos4210-mixer";
- interrupts = <GIC_SPI 91 IRQ_TYPE_LEVEL_HIGH>;
reg = <0x12c10000 0x2100>, <0x12c00000 0x300>;
+ interrupts = <GIC_SPI 91 IRQ_TYPE_LEVEL_HIGH>;
power-domains = <&pd_tv>;
iommus = <&sysmmu_tv>;
--
2.43.0
next prev parent reply other threads:[~2026-10-06 11:58 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-06 11:57 [PATCH v2 00/17] ARM: dts: samsung: cleanup Exynos 4210 dts'es Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 01/17] ARM: dts: samsung: exynos4: add or remove empty lines Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 02/17] ARM: dts: samsung: exynos4: order *-names properties Henrik Grimler
2026-10-06 11:57 ` Henrik Grimler [this message]
2026-10-06 11:57 ` [PATCH v2 04/17] ARM: dts: samsung: exynos4-cpu-thermal: fix indentation and sort Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 05/17] ARM: dts: samsung: exynos4210: sort nodes by address Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 06/17] ARM: dts: samsung: exynos4210: add or remove empty lines Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 07/17] ARM: dts: samsung: exynos4210: sort all properties Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 08/17] ARM: dts: samsung: exynos4210: use operating-points-v2 Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 09/17] ARM: dts: samsung: exynos4210: align all multiline properties Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 10/17] ARM: dts: samsung: exynos4210-pinctrl: align all continued lines Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 11/17] ARM: dts: samsung: exynos4210-i9100: sort properties Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 12/17] ARM: dts: samsung: exynos4210-trats: " Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 13/17] ARM: dts: samsung: exynos4210-origen: " Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 14/17] ARM: dts: samsung: exynos4210-universal_c210: " Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 15/17] ARM: dts: samsung: exynos4210-smdkv310: " Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 16/17] ARM: dts: samsung: exynos4210-smdkv310: sort child nodes Henrik Grimler
2026-10-06 11:57 ` [PATCH v2 17/17] ARM: dts: samsung: exynos4210-universal_c210: fix alignments Henrik Grimler
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20261006-exynos-arm-cleanup-v2-3-e8e86a3c8c1f@axis.com \
--to=henrik.grimler@axis.com \
--cc=alim.akhtar@samsung.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=peter.griffin@linaro.org \
--cc=robh@kernel.org \
--cc=waqar.hameed@axis.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®