[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20250928-exynos7870-dt-fixes-v1-1-a40e77a73f16@disroot.org>
Date: Sun, 28 Sep 2025 22:13:52 +0530
From: Kaustabh Chakraborty <kauschluss@...root.org>
To: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Alim Akhtar <alim.akhtar@...sung.com>
Cc: devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-samsung-soc@...r.kernel.org, linux-kernel@...r.kernel.org,
Kaustabh Chakraborty <kauschluss@...root.org>
Subject: [PATCH 1/7] arm64: dts: exynos7870: relocate ${x}-names property
after ${x}
All ${x}-names properties are conventionally placed after their
corresponding ${x} properties. For instance, 'clock-names' must follow
'clocks', 'interrupt-names' must follow 'interrupts'. Make necessary
changes to follow said convention. No functional changes made.
Signed-off-by: Kaustabh Chakraborty <kauschluss@...root.org>
---
arch/arm64/boot/dts/exynos/exynos7870.dtsi | 108 ++++++++++++++---------------
1 file changed, 54 insertions(+), 54 deletions(-)
diff --git a/arch/arm64/boot/dts/exynos/exynos7870.dtsi b/arch/arm64/boot/dts/exynos/exynos7870.dtsi
index d5d347623b9038b71da55dccdc9084aeaf71618c..48ace70fb958edd28d758f4afea52bc4c7192001 100644
--- a/arch/arm64/boot/dts/exynos/exynos7870.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynos7870.dtsi
@@ -150,8 +150,6 @@ cmu_peri: clock-controller@...f0000 {
reg = <0x101f0000 0x1000>;
#clock-cells = <1>;
- clock-names = "oscclk", "bus", "spi0", "spi1", "spi2",
- "spi3", "spi4", "uart0", "uart1", "uart2";
clocks = <&oscclk>,
<&cmu_mif CLK_GOUT_MIF_CMU_PERI_BUS>,
<&cmu_mif CLK_GOUT_MIF_CMU_PERI_SPI0>,
@@ -162,6 +160,8 @@ cmu_peri: clock-controller@...f0000 {
<&cmu_mif CLK_GOUT_MIF_CMU_PERI_UART0>,
<&cmu_mif CLK_GOUT_MIF_CMU_PERI_UART1>,
<&cmu_mif CLK_GOUT_MIF_CMU_PERI_UART2>;
+ clock-names = "oscclk", "bus", "spi0", "spi1", "spi2",
+ "spi3", "spi4", "uart0", "uart1", "uart2";
};
cmu_mif: clock-controller@...60000 {
@@ -169,8 +169,8 @@ cmu_mif: clock-controller@...60000 {
reg = <0x10460000 0x1000>;
#clock-cells = <1>;
- clock-names = "oscclk";
clocks = <&oscclk>;
+ clock-names = "oscclk";
};
pmu_system_controller: system-controller@...80000 {
@@ -206,11 +206,11 @@ hsi2c0: i2c@...10000 {
reg = <0x10510000 0x2000>;
interrupts = <GIC_SPI 371 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&hsi2c0_bus>;
+ pinctrl-names = "default";
- clock-names = "hsi2c";
clocks = <&cmu_mif CLK_GOUT_MIF_HSI2C_IPCLK>;
+ clock-names = "hsi2c";
status = "disabled";
};
@@ -224,14 +224,14 @@ pinctrl_mif: pinctrl@...30000 {
gpu: gpu@...00000 {
compatible = "samsung,exynos7870-mali", "arm,mali-t830";
reg = <0x11400000 0x5000>;
- interrupt-names = "job", "mmu", "gpu";
interrupts = <GIC_SPI 282 IRQ_TYPE_LEVEL_HIGH>,
<GIC_SPI 283 IRQ_TYPE_LEVEL_HIGH>,
<GIC_SPI 281 IRQ_TYPE_LEVEL_HIGH>;
+ interrupt-names = "job", "mmu", "gpu";
- clock-names = "core", "bus";
clocks = <&cmu_g3d CLK_GOUT_G3D_CLK>,
<&cmu_g3d CLK_GOUT_G3D_ASYNCS_D0_CLK>;
+ clock-names = "core", "bus";
status = "disabled";
};
@@ -241,9 +241,9 @@ cmu_g3d: clock-controller@...60000 {
reg = <0x11460000 0x1000>;
#clock-cells = <1>;
- clock-names = "oscclk", "switch";
clocks = <&oscclk>,
<&cmu_mif CLK_GOUT_MIF_CMU_G3D_SWITCH>;
+ clock-names = "oscclk", "switch";
};
cmu_mfcmscl: clock-controller@...b0000 {
@@ -251,10 +251,10 @@ cmu_mfcmscl: clock-controller@...b0000 {
reg = <0x12cb0000 0x1000>;
#clock-cells = <1>;
- clock-names = "oscclk", "mfc", "mscl";
clocks = <&oscclk>,
<&cmu_mif CLK_GOUT_MIF_CMU_MFCMSCL_MFC>,
<&cmu_mif CLK_GOUT_MIF_CMU_MFCMSCL_MSCL>;
+ clock-names = "oscclk", "mfc", "mscl";
};
mmc0: mmc@...40000 {
@@ -262,9 +262,9 @@ mmc0: mmc@...40000 {
reg = <0x13540000 0x2000>;
interrupts = <GIC_SPI 245 IRQ_TYPE_LEVEL_HIGH>;
- clock-names = "biu", "ciu";
clocks = <&cmu_fsys CLK_GOUT_FSYS_MMC0_ACLK>,
<&cmu_mif CLK_GOUT_MIF_CMU_FSYS_MMC0>;
+ clock-names = "biu", "ciu";
status = "disabled";
};
@@ -274,9 +274,9 @@ mmc1: mmc@...50000 {
reg = <0x13550000 0x2000>;
interrupts = <GIC_SPI 246 IRQ_TYPE_LEVEL_HIGH>;
- clock-names = "biu", "ciu";
clocks = <&cmu_fsys CLK_GOUT_FSYS_MMC1_ACLK>,
<&cmu_mif CLK_GOUT_MIF_CMU_FSYS_MMC1>;
+ clock-names = "biu", "ciu";
status = "disabled";
};
@@ -286,9 +286,9 @@ mmc2: mmc@...60000 {
reg = <0x13560000 0x2000>;
interrupts = <GIC_SPI 247 IRQ_TYPE_LEVEL_HIGH>;
- clock-names = "biu", "ciu";
clocks = <&cmu_fsys CLK_GOUT_FSYS_MMC2_ACLK>,
<&cmu_mif CLK_GOUT_MIF_CMU_FSYS_MMC2>;
+ clock-names = "biu", "ciu";
status = "disabled";
};
@@ -298,9 +298,9 @@ usbdrd_phy: phy@...c0000 {
reg = <0x135c0000 0x100>;
#phy-cells = <1>;
- clock-names = "phy", "ref";
clocks = <&cmu_fsys CLK_GOUT_FSYS_MUX_USB20DRD_PHYCLOCK_USER>,
<&cmu_fsys CLK_GOUT_FSYS_MUX_USB_PLL>;
+ clock-names = "phy", "ref";
samsung,pmu-syscon = <&pmu_system_controller>;
};
@@ -311,10 +311,10 @@ usbdrd: usb@...00000 {
#address-cells = <1>;
#size-cells = <1>;
- clock-names = "bus_early", "ref", "ctrl";
clocks = <&cmu_fsys CLK_GOUT_FSYS_USB20DRD_ACLK_HSDRD>,
<&cmu_fsys CLK_GOUT_FSYS_USB20DRD_HSDRD_REF_CLK>,
<&cmu_fsys CLK_GOUT_FSYS_USB20DRD_HCLK_USB20_CTRL>;
+ clock-names = "bus_early", "ref", "ctrl";
status = "disabled";
@@ -323,8 +323,8 @@ usb@0 {
reg = <0x0 0x10000>;
interrupts = <GIC_SPI 230 IRQ_TYPE_LEVEL_HIGH>;
- phy-names = "usb2-phy";
phys = <&usbdrd_phy 0>;
+ phy-names = "usb2-phy";
usb-role-switch;
snps,usb2-gadget-lpm-disable;
@@ -336,10 +336,10 @@ cmu_fsys: clock-controller@...30000 {
reg = <0x13730000 0x1000>;
#clock-cells = <1>;
- clock-names = "oscclk", "bus", "usb20drd";
clocks = <&oscclk>,
<&cmu_mif CLK_GOUT_MIF_CMU_FSYS_BUS>,
<&cmu_mif CLK_GOUT_MIF_CMU_FSYS_USB20DRD_REFCLK>;
+ clock-names = "oscclk", "bus", "usb20drd";
};
pinctrl_fsys: pinctrl@...50000 {
@@ -354,12 +354,12 @@ serial0: serial@...00000 {
reg = <0x13800000 0x100>;
interrupts = <GIC_SPI 421 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&uart0_bus>;
+ pinctrl-names = "default";
- clock-names = "uart", "clk_uart_baud0";
clocks = <&cmu_peri CLK_GOUT_PERI_UART0_PCLK>,
<&cmu_peri CLK_GOUT_PERI_UART0_EXT_UCLK>;
+ clock-names = "uart", "clk_uart_baud0";
samsung,uart-fifosize = <16>;
@@ -372,12 +372,12 @@ serial1: serial@...10000 {
reg = <0x13810000 0x100>;
interrupts = <GIC_SPI 422 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&uart1_bus>;
+ pinctrl-names = "default";
- clock-names = "uart", "clk_uart_baud0";
clocks = <&cmu_peri CLK_GOUT_PERI_UART1_PCLK>,
<&cmu_peri CLK_GOUT_PERI_UART1_EXT_UCLK>;
+ clock-names = "uart", "clk_uart_baud0";
samsung,uart-fifosize = <256>;
@@ -390,12 +390,12 @@ serial2: serial@...20000 {
reg = <0x13820000 0x100>;
interrupts = <GIC_SPI 423 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&uart2_bus>;
+ pinctrl-names = "default";
- clock-names = "uart", "clk_uart_baud0";
clocks = <&cmu_peri CLK_GOUT_PERI_UART2_PCLK>,
<&cmu_peri CLK_GOUT_PERI_UART2_EXT_UCLK>;
+ clock-names = "uart", "clk_uart_baud0";
samsung,uart-fifosize = <256>;
@@ -408,11 +408,11 @@ i2c0: i2c@...30000 {
reg = <0x13830000 0x100>;
interrupts = <GIC_SPI 424 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&i2c0_bus>;
+ pinctrl-names = "default";
- clock-names = "i2c";
clocks = <&cmu_peri CLK_GOUT_PERI_I2C0_PCLK>;
+ clock-names = "i2c";
status = "disabled";
};
@@ -423,11 +423,11 @@ i2c1: i2c@...40000 {
reg = <0x13840000 0x100>;
interrupts = <GIC_SPI 425 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&i2c1_bus>;
+ pinctrl-names = "default";
- clock-names = "i2c";
clocks = <&cmu_peri CLK_GOUT_PERI_I2C1_PCLK>;
+ clock-names = "i2c";
status = "disabled";
};
@@ -438,11 +438,11 @@ i2c2: i2c@...50000 {
reg = <0x13850000 0x100>;
interrupts = <GIC_SPI 426 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&i2c2_bus>;
+ pinctrl-names = "default";
- clock-names = "i2c";
clocks = <&cmu_peri CLK_GOUT_PERI_I2C2_PCLK>;
+ clock-names = "i2c";
status = "disabled";
};
@@ -453,11 +453,11 @@ i2c3: i2c@...60000 {
reg = <0x13860000 0x100>;
interrupts = <GIC_SPI 427 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&i2c3_bus>;
+ pinctrl-names = "default";
- clock-names = "i2c";
clocks = <&cmu_peri CLK_GOUT_PERI_I2C3_PCLK>;
+ clock-names = "i2c";
status = "disabled";
};
@@ -468,11 +468,11 @@ i2c4: i2c@...70000 {
reg = <0x13870000 0x100>;
interrupts = <GIC_SPI 428 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&i2c4_bus>;
+ pinctrl-names = "default";
- clock-names = "i2c";
clocks = <&cmu_peri CLK_GOUT_PERI_I2C4_PCLK>;
+ clock-names = "i2c";
status = "disabled";
};
@@ -483,11 +483,11 @@ i2c5: i2c@...80000 {
reg = <0x13880000 0x100>;
interrupts = <GIC_SPI 429 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&i2c5_bus>;
+ pinctrl-names = "default";
- clock-names = "i2c";
clocks = <&cmu_peri CLK_GOUT_PERI_I2C5_PCLK>;
+ clock-names = "i2c";
status = "disabled";
};
@@ -498,11 +498,11 @@ i2c6: i2c@...90000 {
reg = <0x13890000 0x100>;
interrupts = <GIC_SPI 430 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&i2c6_bus>;
+ pinctrl-names = "default";
- clock-names = "i2c";
clocks = <&cmu_peri CLK_GOUT_PERI_I2C6_PCLK>;
+ clock-names = "i2c";
status = "disabled";
};
@@ -513,11 +513,11 @@ hsi2c1: i2c@...a0000 {
reg = <0x138a0000 0x1000>;
interrupts = <GIC_SPI 449 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&hsi2c1_bus>;
+ pinctrl-names = "default";
- clock-names = "hsi2c";
clocks = <&cmu_peri CLK_GOUT_PERI_HSI2C1_IPCLK>;
+ clock-names = "hsi2c";
status = "disabled";
};
@@ -528,11 +528,11 @@ hsi2c2: i2c@...b0000 {
reg = <0x138b0000 0x1000>;
interrupts = <GIC_SPI 450 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&hsi2c2_bus>;
+ pinctrl-names = "default";
- clock-names = "hsi2c";
clocks = <&cmu_peri CLK_GOUT_PERI_HSI2C2_IPCLK>;
+ clock-names = "hsi2c";
status = "disabled";
};
@@ -543,11 +543,11 @@ hsi2c3: i2c@...c0000 {
reg = <0x138c0000 0x1000>;
interrupts = <GIC_SPI 451 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&hsi2c3_bus>;
+ pinctrl-names = "default";
- clock-names = "hsi2c";
clocks = <&cmu_peri CLK_GOUT_PERI_HSI2C3_IPCLK>;
+ clock-names = "hsi2c";
status = "disabled";
};
@@ -558,11 +558,11 @@ i2c7: i2c@...d0000 {
reg = <0x138d0000 0x100>;
interrupts = <GIC_SPI 453 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&i2c7_bus>;
+ pinctrl-names = "default";
- clock-names = "i2c";
clocks = <&cmu_peri CLK_GOUT_PERI_I2C7_PCLK>;
+ clock-names = "i2c";
status = "disabled";
};
@@ -573,11 +573,11 @@ i2c8: i2c@...e0000 {
reg = <0x138e0000 0x100>;
interrupts = <GIC_SPI 454 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&i2c8_bus>;
+ pinctrl-names = "default";
- clock-names = "i2c";
clocks = <&cmu_peri CLK_GOUT_PERI_I2C8_PCLK>;
+ clock-names = "i2c";
status = "disabled";
};
@@ -588,11 +588,11 @@ hsi2c4: i2c@...f0000 {
reg = <0x138f0000 0x1000>;
interrupts = <GIC_SPI 456 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&hsi2c4_bus>;
+ pinctrl-names = "default";
- clock-names = "hsi2c";
clocks = <&cmu_peri CLK_GOUT_PERI_HSI2C4_IPCLK>;
+ clock-names = "hsi2c";
status = "disabled";
};
@@ -603,11 +603,11 @@ hsi2c5: i2c@...50000 {
reg = <0x13950000 0x1000>;
interrupts = <GIC_SPI 457 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&hsi2c5_bus>;
+ pinctrl-names = "default";
- clock-names = "hsi2c";
clocks = <&cmu_peri CLK_GOUT_PERI_HSI2C5_IPCLK>;
+ clock-names = "hsi2c";
status = "disabled";
};
@@ -618,11 +618,11 @@ hsi2c6: i2c@...60000 {
reg = <0x13960000 0x1000>;
interrupts = <GIC_SPI 458 IRQ_TYPE_LEVEL_HIGH>;
- pinctrl-names = "default";
pinctrl-0 = <&hsi2c6_bus>;
+ pinctrl-names = "default";
- clock-names = "hsi2c";
clocks = <&cmu_peri CLK_GOUT_PERI_HSI2C6_IPCLK>;
+ clock-names = "hsi2c";
status = "disabled";
};
@@ -668,11 +668,11 @@ cmu_isp: clock-controller@...d0000 {
reg = <0x144d0000 0x1000>;
#clock-cells = <1>;
- clock-names = "oscclk", "cam", "isp", "vra";
clocks = <&oscclk>,
<&cmu_mif CLK_GOUT_MIF_CMU_ISP_CAM>,
<&cmu_mif CLK_GOUT_MIF_CMU_ISP_ISP>,
<&cmu_mif CLK_GOUT_MIF_CMU_ISP_VRA>;
+ clock-names = "oscclk", "cam", "isp", "vra";
};
pinctrl_dispaud: pinctrl@...c0000 {
@@ -686,11 +686,11 @@ cmu_dispaud: clock-controller@...d0000 {
reg = <0x148d0000 0x1000>;
#clock-cells = <1>;
- clock-names = "oscclk", "bus", "decon_eclk", "decon_vclk";
clocks = <&oscclk>,
<&cmu_mif CLK_GOUT_MIF_CMU_DISPAUD_BUS>,
<&cmu_mif CLK_GOUT_MIF_CMU_DISPAUD_DECON_ECLK>,
<&cmu_mif CLK_GOUT_MIF_CMU_DISPAUD_DECON_VCLK>;
+ clock-names = "oscclk", "bus", "decon_eclk", "decon_vclk";
};
};
--
2.51.0
Powered by blists - more mailing lists