diff mbox series

arm64: dts: Add support for Unisoc's UMS512

Message ID 20230118084025.2898404-1-chunyan.zhang@unisoc.com (mailing list archive)
State Changes Requested
Headers show
Series arm64: dts: Add support for Unisoc's UMS512 | expand

Commit Message

Chunyan Zhang Jan. 18, 2023, 8:40 a.m. UTC
Add basic support for Unisoc's UMS512, with this patch,
the board ums512-1h10 can run into console.

Signed-off-by: Chunyan Zhang <chunyan.zhang@unisoc.com>
---
 arch/arm64/boot/dts/sprd/Makefile        |   3 +-
 arch/arm64/boot/dts/sprd/ums512-1h10.dts |  64 ++
 arch/arm64/boot/dts/sprd/ums512.dtsi     | 919 +++++++++++++++++++++++
 3 files changed, 985 insertions(+), 1 deletion(-)
 create mode 100644 arch/arm64/boot/dts/sprd/ums512-1h10.dts
 create mode 100644 arch/arm64/boot/dts/sprd/ums512.dtsi

Comments

Krzysztof Kozlowski Jan. 19, 2023, 11:40 a.m. UTC | #1
On 18/01/2023 09:40, Chunyan Zhang wrote:
> Add basic support for Unisoc's UMS512, with this patch,
> the board ums512-1h10 can run into console.
> 

Use subject prefixes matching the subsystem (which you can get for
example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory
your patch is touching), e.g. "arm64: dts: sprd:"

> Signed-off-by: Chunyan Zhang <chunyan.zhang@unisoc.com>
> ---
>  arch/arm64/boot/dts/sprd/Makefile        |   3 +-
>  arch/arm64/boot/dts/sprd/ums512-1h10.dts |  64 ++
>  arch/arm64/boot/dts/sprd/ums512.dtsi     | 919 +++++++++++++++++++++++
>  3 files changed, 985 insertions(+), 1 deletion(-)
>  create mode 100644 arch/arm64/boot/dts/sprd/ums512-1h10.dts
>  create mode 100644 arch/arm64/boot/dts/sprd/ums512.dtsi
> 
> diff --git a/arch/arm64/boot/dts/sprd/Makefile b/arch/arm64/boot/dts/sprd/Makefile
> index f4f1f5148cc2..97522fb0bf66 100644
> --- a/arch/arm64/boot/dts/sprd/Makefile
> +++ b/arch/arm64/boot/dts/sprd/Makefile
> @@ -1,4 +1,5 @@
>  # SPDX-License-Identifier: GPL-2.0
>  dtb-$(CONFIG_ARCH_SPRD) += sc9836-openphone.dtb \
>  			sp9860g-1h10.dtb	\
> -			sp9863a-1h10.dtb
> +			sp9863a-1h10.dtb	\
> +			ums512-1h10.dtb
> diff --git a/arch/arm64/boot/dts/sprd/ums512-1h10.dts b/arch/arm64/boot/dts/sprd/ums512-1h10.dts
> new file mode 100644
> index 000000000000..d549a8c361d4
> --- /dev/null
> +++ b/arch/arm64/boot/dts/sprd/ums512-1h10.dts
> @@ -0,0 +1,64 @@
> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> +/*
> + * Unisoc UMS512-1h10 boards DTS file
> + *
> + * Copyright (C) 2021, Unisoc Inc.
> + */
> +
> +/dts-v1/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include "ums512.dtsi"
> +
> +/ {
> +	model = "Unisoc UMS512-1H10 Board";
> +
> +	compatible = "sprd,ums512-1h10", "sprd,ums512";
> +
> +	aliases {
> +		serial0 = &uart0;
> +		serial1 = &uart1;
> +	};
> +
> +	memory@80000000 {
> +		device_type = "memory";
> +		reg = <0x0 0x80000000 0x0 0x80000000>;
> +	};
> +
> +	chosen {
> +		stdout-path = "serial1:115200n8";
> +		bootargs = "earlycon";

earlycon is for debugging, not wide, mainline usage. Drop.

> +	};
> +};
> +
> +&uart0 {
> +	status = "okay";
> +};
> +
> +&uart1 {
> +	status = "okay";
> +};
> +
> +/* SD card */
> +&sdio0 {
> +	status = "okay";
> +	bus-width = <4>;
> +	no-sdio;
> +	no-mmc;
> +	sprd,phy-delay-sd-uhs-sdr104 = <0x7f 0x73 0x72 0x72>;
> +	sprd,phy-delay-sd-uhs-sdr50 = <0x6e 0x7f 0x01 0x01>;
> +	sprd,phy-delay-sd-highspeed = <0x7f 0x1a 0x9a 0x9a>;
> +	sprd,phy-delay-legacy = <0x7f 0x1a 0x9a 0x9a>;
> +	sd-uhs-sdr104;
> +	sd-uhs-sdr50;
> +};
> +
> +/* EMMC storage */
> +&sdio3 {
> +	status = "okay";
> +	bus-width = <8>;
> +	no-sdio;
> +	no-sd;
> +	non-removable;
> +	cap-mmc-hw-reset;
> +};
> diff --git a/arch/arm64/boot/dts/sprd/ums512.dtsi b/arch/arm64/boot/dts/sprd/ums512.dtsi
> new file mode 100644
> index 000000000000..028e17f8198e
> --- /dev/null
> +++ b/arch/arm64/boot/dts/sprd/ums512.dtsi
> @@ -0,0 +1,919 @@
> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> +/*
> + * Unisoc UMS512 SoC DTS file
> + *
> + * Copyright (C) 2021, Unisoc Inc.
> + */
> +
> +#include <dt-bindings/clock/sprd,ums512-clk.h>
> +#include <dt-bindings/interrupt-controller/arm-gic.h>
> +
> +/ {
> +	interrupt-parent = <&gic>;
> +	#address-cells = <2>;
> +	#size-cells = <2>;
> +
> +	cpus {
> +		#address-cells = <2>;
> +		#size-cells = <0>;
> +
> +		cpu-map {
> +			cluster0 {
> +				core0 {
> +					cpu = <&CPU0>;
> +				};
> +				core1 {
> +					cpu = <&CPU1>;
> +				};
> +				core2 {
> +					cpu = <&CPU2>;
> +				};
> +				core3 {
> +					cpu = <&CPU3>;
> +				};
> +				core4 {
> +					cpu = <&CPU4>;
> +				};
> +				core5 {
> +					cpu = <&CPU5>;
> +				};
> +				core6 {
> +					cpu = <&CPU6>;
> +				};
> +				core7 {
> +					cpu = <&CPU7>;
> +				};
> +			};
> +		};
> +
> +		CPU0: cpu@0 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a55";
> +			reg = <0x0 0x0>;
> +			enable-method = "psci";
> +			cpu-idle-states = <&CORE_PD>;
> +		};
> +
> +		CPU1: cpu@100 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a55";
> +			reg = <0x0 0x100>;
> +			enable-method = "psci";
> +			cpu-idle-states = <&CORE_PD>;
> +		};
> +
> +		CPU2: cpu@200 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a55";
> +			reg = <0x0 0x200>;
> +			enable-method = "psci";
> +			cpu-idle-states = <&CORE_PD>;
> +		};
> +
> +		CPU3: cpu@300 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a55";
> +			reg = <0x0 0x300>;
> +			enable-method = "psci";
> +			cpu-idle-states = <&CORE_PD>;
> +		};
> +
> +		CPU4: cpu@400 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a55";
> +			reg = <0x0 0x400>;
> +			enable-method = "psci";
> +			cpu-idle-states = <&CORE_PD>;
> +		};
> +
> +		CPU5: cpu@500 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a55";
> +			reg = <0x0 0x500>;
> +			enable-method = "psci";
> +			cpu-idle-states = <&CORE_PD>;
> +		};
> +
> +		CPU6: cpu@600 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a55";
> +			reg = <0x0 0x600>;
> +			enable-method = "psci";
> +			cpu-idle-states = <&CORE_PD>;
> +		};
> +
> +		CPU7: cpu@700 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a55";
> +			reg = <0x0 0x700>;
> +			enable-method = "psci";
> +			cpu-idle-states = <&CORE_PD>;
> +		};
> +	};
> +
> +	idle-states {
> +		entry-method = "psci";
> +		CORE_PD: core-pd {
> +			compatible = "arm,idle-state";
> +			entry-latency-us = <4000>;
> +			exit-latency-us = <4000>;
> +			min-residency-us = <10000>;
> +			local-timer-stop;
> +			arm,psci-suspend-param = <0x00010000>;
> +		};
> +	};
> +
> +	psci {
> +		compatible = "arm,psci-0.2";
> +		method = "smc";
> +	};
> +
> +	timer {
> +		compatible = "arm,armv8-timer";
> +		interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_HIGH>, /* Physical Secure PPI */
> +			     <GIC_PPI 14 IRQ_TYPE_LEVEL_HIGH>, /* Physical Non-Secure PPI */
> +			     <GIC_PPI 11 IRQ_TYPE_LEVEL_HIGH>, /* Virtual PPI */
> +			     <GIC_PPI 10 IRQ_TYPE_LEVEL_HIGH>; /* Hipervisor PPI */
> +	};
> +
> +	pmu {
> +		compatible = "arm,armv8-pmuv3";
> +		interrupts = <GIC_SPI 112 IRQ_TYPE_LEVEL_HIGH>,
> +			     <GIC_SPI 113 IRQ_TYPE_LEVEL_HIGH>,
> +			     <GIC_SPI 114 IRQ_TYPE_LEVEL_HIGH>,
> +			     <GIC_SPI 115 IRQ_TYPE_LEVEL_HIGH>,
> +			     <GIC_SPI 116 IRQ_TYPE_LEVEL_HIGH>,
> +			     <GIC_SPI 117 IRQ_TYPE_LEVEL_HIGH>,
> +			     <GIC_SPI 118 IRQ_TYPE_LEVEL_HIGH>,
> +			     <GIC_SPI 119 IRQ_TYPE_LEVEL_HIGH>;
> +	};
> +
> +	soc: soc {
> +		compatible = "simple-bus";
> +		#address-cells = <2>;
> +		#size-cells = <2>;
> +		ranges;
> +
> +		gic: interrupt-controller@12000000 {
> +			compatible = "arm,gic-v3";
> +			#interrupt-cells = <3>;
> +			#address-cells = <2>;
> +			#size-cells = <2>;
> +			ranges;
> +			redistributor-stride = <0x0 0x20000>;	/* 128KB stride */
> +			#redistributor-regions = <1>;
> +			interrupt-controller;
> +			reg = <0x0 0x12000000 0 0x20000>,	/* GICD */
> +			      <0x0 0x12040000 0 0x100000>;	/* GICR */

Keep reg as second property, after compatible. This applies everywhere.

> +			interrupts = <GIC_PPI 9 IRQ_TYPE_LEVEL_HIGH>;
> +		};
> +
> +		ap_ahb_regs: syscon@20100000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x20100000 0 0x4000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x20100000 0x4000>;
> +
> +			apahb_gate: clock-controller@0 {
> +				compatible = "sprd,ums512-apahb-gate";
> +				reg = <0x0 0x3000>;
> +				clocks = <&ext_26m>;
> +				clock-names = "ext-26m";
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		pub_apb_regs: syscon@31050000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x31050000 0 0x9000>;
> +		};
> +
> +		top_dvfs_apb_regs: syscon@322a0000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x322a0000 0 0x8000>;
> +		};
> +
> +		ap_intc0_regs: syscon@32310000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x32310000 0 0x1000>;
> +		};
> +
> +		ap_intc1_regs: syscon@32320000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x32320000 0 0x1000>;
> +		};
> +
> +		ap_intc2_regs: syscon@32330000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x32330000 0 0x1000>;
> +		};
> +
> +		ap_intc3_regs: syscon@32340000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x32340000 0 0x1000>;
> +		};
> +
> +		ap_intc4_regs: syscon@32350000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x32350000 0 0x1000>;
> +		};
> +
> +		ap_intc5_regs: syscon@32360000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x32360000 0 0x1000>;
> +		};
> +
> +		anlg_phy_g0_regs: syscon@32390000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x32390000 0 0x3000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x32390000 0x3000>;
> +
> +			dpll0: clock-controller@0 {
> +				compatible = "sprd,ums512-g0-pll";
> +				reg = <0x0 0x100>;
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		anlg_phy_g2_regs: syscon@323b0000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x323b0000 0 0x3000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x323b0000 0x3000>;
> +
> +			mpll1: clock-controller@0 {
> +				compatible = "sprd,ums512-g2-pll";
> +				reg = <0x0 0x100>;
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		anlg_phy_g3_regs: syscon@323c0000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x323c0000 0 0x3000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x323c0000 0x3000>;
> +
> +			pll1: clock-controller@0 {
> +				compatible = "sprd,ums512-g3-pll";
> +				reg = <0x0 0x3000>;
> +				clocks = <&ext_26m>;
> +				clock-names = "ext-26m";
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		anlg_phy_gc_regs: syscon@323e0000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x323e0000 0 0x3000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x323e0000 0x3000>;
> +
> +			pll2: clock-controller@0 {
> +				compatible = "sprd,ums512-gc-pll";
> +				reg = <0x0 0x100>;
> +				clock-names = "ext-26m";
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		anlg_phy_g10_regs: syscon@323f0000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x323f0000 0 0x3000>;
> +		};
> +
> +		aon_apb_regs: syscon@327d0000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x327d0000 0 0x3000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x327d0000 0x3000>;
> +
> +			aonapb_gate: clock-controller@0 {
> +				compatible = "sprd,ums512-aon-gate";
> +				reg = <0x0 0x3000>;
> +				clocks = <&ext_26m>;
> +				clock-names = "ext-26m";
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		pmu_apb_regs: syscon@327e0000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x327e0000 0 0x3000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x327e0000 0x3000>;
> +
> +			pmu_gate: clock-controller@0 {
> +				compatible = "sprd,ums512-pmu-gate";
> +				reg = <0x0 0x3000>;
> +				clocks = <&ext_26m>;
> +				clock-names = "ext-26m";
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		audcp_apb_regs: syscon@3350d000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x3350d000 0 0x1000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x3350d000 0x1000>;
> +
> +			audcpapb_gate: clock-controller@0 {
> +				compatible = "sprd,ums512-audcpapb-gate";
> +				reg = <0x0 0x300>;
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		audcp_ahb_regs: syscon@335e0000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x335e0000 0 0x1000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x335e0000 0x1000>;
> +
> +			audcpahb_gate: clock-controller@0 {
> +				compatible = "sprd,ums512-audcpahb-gate";
> +				reg = <0x0 0x300>;
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		gpu_apb_regs: syscon@60100000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x60100000 0 0x3000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x60100000 0x3000>;
> +
> +			gpu_clk: clock-controller@0 {
> +				compatible = "sprd,ums512-gpu-clk";
> +				clocks = <&ext_26m>;
> +				clock-names = "ext-26m";
> +				reg = <0x0 0x100>;
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		gpu_dvfs_apb_regs: syscon@60110000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x60110000 0 0x3000>;
> +		};
> +
> +		mm_ahb_regs: syscon@62200000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x62200000 0 0x3000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x62200000 0x3000>;
> +
> +			mm_gate: clock-controller@0 {
> +				compatible = "sprd,ums512-mm-gate-clk";
> +				reg = <0x0 0x3000>;
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		ap_apb_regs: syscon@71000000 {
> +			compatible = "sprd,ums512-glbregs", "syscon",
> +				     "simple-mfd";
> +			reg = <0 0x71000000 0 0x3000>;
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0 0x71000000 0x3000>;
> +
> +			apapb_gate: clock-controller@0 {
> +				compatible = "sprd,ums512-apapb-gate";
> +				reg = <0x0 0x3000>;
> +				#clock-cells = <1>;
> +			};
> +		};
> +
> +		ap_clk: clock-controller@20200000 {
> +			compatible = "sprd,ums512-ap-clk";
> +			reg = <0 0x20200000 0 0x1000>;
> +			clocks = <&ext_26m>;
> +			clock-names = "ext-26m";
> +			#clock-cells = <1>;
> +		};
> +
> +		aon_clk: clock-controller@32080000 {
> +			compatible = "sprd,ums512-aonapb-clk";
> +			reg = <0 0x32080000 0 0x1000>;
> +			clocks = <&ext_26m>, <&ext_32k>,
> +				 <&ext_4m>, <&rco_100m>;
> +			clock-names = "ext-26m", "ext-32k",
> +				      "ext-4m", "rco-100m";
> +			#clock-cells = <1>;
> +		};
> +
> +		mm_clk: clock-controller@62100000 {
> +			compatible = "sprd,ums512-mm-clk";
> +			reg = <0 0x62100000 0 0x1000>;
> +			clocks = <&ext_26m>;
> +			clock-names = "ext-26m";
> +			#clock-cells = <1>;
> +		};
> +
> +		/* SoC Funnel */
> +		funnel@3c002000 {
> +			compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
> +			reg = <0 0x3c002000 0 0x1000>;
> +			clocks = <&ext_26m>;
> +			clock-names = "apb_pclk";
> +
> +			out-ports {
> +				port {
> +					funnel_soc_out_port: endpoint {
> +						remote-endpoint = <&etb_in>;
> +					};
> +				};
> +			};
> +
> +			in-ports {
> +				#address-cells = <1>;
> +				#size-cells = <0>;
> +
> +				port@1 {
> +					reg = <1>;
> +					funnel_soc_in_port: endpoint {
> +						remote-endpoint =
> +						<&funnel_corinth_out_port>;
> +					};
> +				};
> +			};
> +		};
> +
> +		/* SoC ETF */
> +		soc_etb: etb@3c003000 {
> +			compatible = "arm,coresight-tmc", "arm,primecell";
> +			reg = <0 0x3c003000 0 0x1000>;
> +			clocks = <&ext_26m>;
> +			clock-names = "apb_pclk";
> +
> +			in-ports {
> +				port {
> +					etb_in: endpoint {
> +						remote-endpoint =
> +						<&funnel_soc_out_port>;
> +					};
> +				};
> +			};
> +		};
> +
> +		/* AP-CPU Funnel for core3/4/5/7 */
> +		funnel@3e001000 {
> +			compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
> +			reg = <0 0x3e001000 0 0x1000>;
> +			clocks = <&ext_26m>;
> +			clock-names = "apb_pclk";
> +
> +			out-ports {
> +				port {
> +					funnel_corinth_lit_out_port: endpoint {
> +						remote-endpoint =
> +						<&corinth_etf_lit_in>;
> +					};
> +				};
> +			};
> +
> +			in-ports {
> +				#address-cells = <1>;
> +				#size-cells = <0>;
> +
> +				port@0 {
> +					reg = <0>;
> +					funnel_core_in_port3: endpoint {
> +						remote-endpoint = <&etm3_out>;
> +					};
> +				};
> +
> +				port@1 {
> +					reg = <1>;
> +					funnel_core_in_port4: endpoint {
> +						remote-endpoint = <&etm4_out>;
> +					};
> +				};
> +
> +				port@2 {
> +					reg = <2>;
> +					funnel_core_in_port5: endpoint {
> +						remote-endpoint = <&etm5_out>;
> +					};
> +				};
> +
> +				port@3 {
> +					reg = <3>;
> +					funnel_core_in_port7: endpoint {
> +						remote-endpoint = <&etm7_out>;
> +					};
> +				};
> +			};
> +		};
> +
> +		/* AP-CPU ETF for little cores */
> +		etf@3e002000 {
> +			compatible = "arm,coresight-tmc", "arm,primecell";
> +			reg = <0 0x3e002000 0 0x1000>;
> +			clocks = <&ext_26m>;
> +			clock-names = "apb_pclk";
> +
> +			out-ports {
> +				port {
> +					corinth_etf_lit_out: endpoint {
> +						remote-endpoint =
> +						<&funnel_corinth_from_lit_in_port>;
> +					};
> +				};
> +			};
> +
> +			in-ports {
> +				port {
> +					corinth_etf_lit_in: endpoint {
> +						remote-endpoint =
> +						<&funnel_corinth_lit_out_port>;
> +					};
> +				};
> +			};
> +		};
> +
> +		/* AP-CPU ETF for big cores */
> +		etf@3e003000 {
> +			compatible = "arm,coresight-tmc", "arm,primecell";
> +			reg = <0 0x3e003000 0 0x1000>;
> +			clocks = <&ext_26m>;
> +			clock-names = "apb_pclk";
> +
> +			out-ports {
> +				port {
> +					corinth_etf_big_out: endpoint {
> +						remote-endpoint =
> +						<&funnel_corinth_from_big_in_port>;
> +					};
> +				};
> +			};
> +
> +			in-ports {
> +				port {
> +					corinth_etf_big_in: endpoint {
> +						remote-endpoint =
> +						<&funnel_corinth_big_out_port>;
> +					};
> +				};
> +			};
> +		};
> +
> +		/* Funnel to SoC */
> +		funnel@3e004000 {
> +			compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
> +			reg = <0 0x3e004000 0 0x1000>;
> +			clocks = <&ext_26m>;
> +			clock-names = "apb_pclk";
> +
> +			out-ports {
> +				port {
> +					funnel_corinth_out_port: endpoint {
> +						remote-endpoint =
> +						<&funnel_soc_in_port>;
> +					};
> +				};
> +			};
> +
> +			in-ports {
> +				#address-cells = <1>;
> +				#size-cells = <0>;
> +
> +				port@0 {
> +					reg = <0>;
> +					funnel_corinth_from_lit_in_port: endpoint {
> +						remote-endpoint = <&corinth_etf_lit_out>;
> +					};
> +				};
> +
> +				port@1 {
> +					reg = <1>;
> +					funnel_corinth_from_big_in_port: endpoint {
> +						remote-endpoint = <&corinth_etf_big_out>;
> +					};
> +				};
> +			};
> +		};
> +
> +		/* AP-CPU Funnel for core0/1/2/6 */
> +		funnel@3e005000 {
> +			compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
> +			reg = <0 0x3e005000 0 0x1000>;
> +			clocks = <&ext_26m>;
> +			clock-names = "apb_pclk";
> +
> +			out-ports {
> +				port {
> +					funnel_corinth_big_out_port: endpoint {
> +						remote-endpoint = <&corinth_etf_big_in>;
> +					};
> +				};
> +			};
> +
> +			in-ports {
> +				#address-cells = <1>;
> +				#size-cells = <0>;
> +
> +				port@0 {
> +					reg = <0>;
> +					funnel_core_in_port0: endpoint {
> +						remote-endpoint = <&etm0_out>;
> +					};
> +				};
> +
> +				port@1 {
> +					reg = <1>;
> +					funnel_core_in_port1: endpoint {
> +						remote-endpoint = <&etm1_out>;
> +					};
> +				};
> +
> +				port@2 {
> +					reg = <2>;
> +					funnel_core_in_port2: endpoint {
> +						remote-endpoint = <&etm2_out>;
> +					};
> +				};
> +
> +				port@3 {
> +					reg = <3>;
> +					funnel_core_in_port6: endpoint {
> +						remote-endpoint = <&etm6_out>;
> +					};
> +				};
> +			};
> +		};
> +
> +		etm0: etm@3f040000 {
> +			compatible = "arm,coresight-etm4x", "arm,primecell";
> +			reg = <0 0x3f040000 0 0x1000>;
> +			cpu = <&CPU0>;
> +			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> +			clock-names = "apb_pclk", "clk_cs", "cs_src";
> +
> +			out-ports {
> +				port {
> +					etm0_out: endpoint {
> +						remote-endpoint =
> +						<&funnel_core_in_port0>;
> +					};
> +				};
> +			};
> +		};
> +
> +		etm1: etm@3f140000 {
> +			compatible = "arm,coresight-etm4x", "arm,primecell";
> +			reg = <0 0x3f140000 0 0x1000>;
> +			cpu = <&CPU1>;
> +			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> +			clock-names = "apb_pclk", "clk_cs", "cs_src";
> +
> +			out-ports {
> +				port {
> +					etm1_out: endpoint {
> +						remote-endpoint =
> +						<&funnel_core_in_port1>;
> +					};
> +				};
> +			};
> +		};
> +
> +		etm2: etm@3f240000 {
> +			compatible = "arm,coresight-etm4x", "arm,primecell";
> +			reg = <0 0x3f240000 0 0x1000>;
> +			cpu = <&CPU2>;
> +			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> +			clock-names = "apb_pclk", "clk_cs", "cs_src";
> +
> +			out-ports {
> +				port {
> +					etm2_out: endpoint {
> +						remote-endpoint =
> +						<&funnel_core_in_port2>;
> +					};
> +				};
> +			};
> +		};
> +
> +		etm3: etm@3f340000 {
> +			compatible = "arm,coresight-etm4x", "arm,primecell";
> +			reg = <0 0x3f340000 0 0x1000>;
> +			cpu = <&CPU3>;
> +			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> +			clock-names = "apb_pclk", "clk_cs", "cs_src";
> +
> +			out-ports {
> +				port {
> +					etm3_out: endpoint {
> +						remote-endpoint =
> +						<&funnel_core_in_port3>;
> +					};
> +				};
> +			};
> +		};
> +
> +		etm4: etm@3f440000 {
> +			compatible = "arm,coresight-etm4x", "arm,primecell";
> +			reg = <0 0x3f440000 0 0x1000>;
> +			cpu = <&CPU4>;
> +			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> +			clock-names = "apb_pclk", "clk_cs", "cs_src";
> +
> +			out-ports {
> +				port {
> +					etm4_out: endpoint {
> +						remote-endpoint =
> +						<&funnel_core_in_port4>;
> +					};
> +				};
> +			};
> +		};
> +
> +		etm5: etm@3f540000 {
> +			compatible = "arm,coresight-etm4x", "arm,primecell";
> +			reg = <0 0x3f540000 0 0x1000>;
> +			cpu = <&CPU5>;
> +			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> +			clock-names = "apb_pclk", "clk_cs", "cs_src";
> +
> +			out-ports {
> +				port {
> +					etm5_out: endpoint {
> +						remote-endpoint =
> +						<&funnel_core_in_port5>;
> +					};
> +				};
> +			};
> +		};
> +
> +		etm6: etm@3f640000 {
> +			compatible = "arm,coresight-etm4x", "arm,primecell";
> +			reg = <0 0x3f640000 0 0x1000>;
> +			cpu = <&CPU6>;
> +			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> +			clock-names = "apb_pclk", "clk_cs", "cs_src";
> +
> +			out-ports {
> +				port {
> +					etm6_out: endpoint {
> +						remote-endpoint =
> +						<&funnel_core_in_port6>;
> +					};
> +				};
> +			};
> +		};
> +
> +		etm7: etm@3f740000 {
> +			compatible = "arm,coresight-etm4x", "arm,primecell";
> +			reg = <0 0x3f740000 0 0x1000>;
> +			cpu = <&CPU7>;
> +			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> +			clock-names = "apb_pclk", "clk_cs", "cs_src";
> +
> +			out-ports {
> +				port {
> +					etm7_out: endpoint {
> +						remote-endpoint =
> +						<&funnel_core_in_port7>;
> +					};
> +				};
> +			};
> +		};
> +
> +
> +		apb@70000000 {
> +			compatible = "simple-bus";
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +			ranges = <0 0x0 0x70000000 0x10000000>;
> +
> +			uart0: serial@0 {
> +				compatible = "sprd,ums512-uart",
> +					     "sprd,sc9836-uart";
> +				reg = <0x0 0x100>;
> +				interrupts = <GIC_SPI 2 IRQ_TYPE_LEVEL_HIGH>;
> +				clocks = <&ext_26m>;
> +				status = "disabled";
> +			};
> +
> +			uart1: serial@100000 {
> +				compatible = "sprd,ums512-uart",
> +					     "sprd,sc9836-uart";
> +				reg = <0x100000 0x100>;
> +				interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>;
> +				clocks = <&ext_26m>;
> +				status = "disabled";
> +			};
> +		};
> +
> +		ap-apb {

Non-unit-address nodes cannot be mixed with unit address ones. Something
is wrong here.

> +			compatible = "simple-bus";
> +			#address-cells = <2>;
> +			#size-cells = <2>;
> +			ranges;
> +
> +			sdio0: sdio@71100000 {
> +				compatible = "sprd,sdhci-r11";
> +				reg = <0 0x71100000 0 0x1000>;
> +				interrupts = <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>;
> +				clock-names = "sdio", "enable";
> +				clocks = <&ap_clk CLK_SDIO0_2X>,
> +					 <&apapb_gate CLK_SDIO0_EB>;
> +				assigned-clocks = <&ap_clk CLK_SDIO0_2X>;
> +				assigned-clock-parents = <&pll1 CLK_RPLL>;
> +				status = "disabled";
> +			};
> +
> +			sdio3: sdio@71400000 {
> +				compatible = "sprd,sdhci-r11";
> +				reg = <0 0x71400000 0 0x1000>;
> +				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> +				clock-names = "sdio", "enable";
> +				clocks = <&ap_clk CLK_EMMC_2X>,
> +					 <&apapb_gate CLK_EMMC_EB>;
> +				assigned-clocks = <&ap_clk CLK_EMMC_2X>;
> +				assigned-clock-parents = <&pll1 CLK_RPLL>;
> +				status = "disabled";
> +			};
> +		};
> +
> +		aon {

Same problem.

> +			compatible = "simple-bus";
> +			#address-cells = <2>;
> +			#size-cells = <2>;
> +			ranges;
> +
> +			adi_bus: spi@32100000 {
> +				compatible = "sprd,ums512-adi";
> +				reg = <0 0x32100000 0 0x100000>;
> +				#address-cells = <1>;
> +				#size-cells = <0>;
> +				sprd,hw-channels = <2 0x18cc>, <3 0x18cc>, <13 0x1854>, <15 0x1874>,
> +					<17 0x1844>,<19 0x1844>, <21 0x1864>, <30 0x1820>,
> +					<35 0x19b8>, <39 0x19ac>;
> +			};
> +		};
> +	};
> +
> +	ext_26m: ext-26m {

Node names should be generic, so at least clock prefix or suffix. This
applies to all further places as well.

https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation

> +		compatible = "fixed-clock";
> +		#clock-cells = <0>;
> +		clock-frequency = <26000000>;
> +		clock-output-names = "ext-26m";
> +	};
> +
> +	ext_32k: ext-32k {
> +		compatible = "fixed-clock";
> +		#clock-cells = <0>;
> +		clock-frequency = <32768>;
> +		clock-output-names = "ext-32k";
> +	};
> +
> +	ext_4m: ext-4m {
> +		compatible = "fixed-clock";
> +		#clock-cells = <0>;
> +		clock-frequency = <4000000>;
> +		clock-output-names = "ext-4m";
> +	};
> +
> +	rco_100m: rco-100m {
> +		compatible = "fixed-clock";
> +		#clock-cells = <0>;
> +		clock-frequency = <100000000>;
> +		clock-output-names = "rco-100m";
> +	};
> +};

Best regards,
Krzysztof
Chunyan Zhang Feb. 6, 2023, 9:28 a.m. UTC | #2
Hi Krzysztof,

Thanks for the review, and sorry for the late reply, I just came back
from a long holiday.

On Thu, 19 Jan 2023 at 19:40, Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 18/01/2023 09:40, Chunyan Zhang wrote:
> > Add basic support for Unisoc's UMS512, with this patch,
> > the board ums512-1h10 can run into console.
> >
>
> Use subject prefixes matching the subsystem (which you can get for
> example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory
> your patch is touching), e.g. "arm64: dts: sprd:"
>
> > Signed-off-by: Chunyan Zhang <chunyan.zhang@unisoc.com>
> > ---
> >  arch/arm64/boot/dts/sprd/Makefile        |   3 +-
> >  arch/arm64/boot/dts/sprd/ums512-1h10.dts |  64 ++
> >  arch/arm64/boot/dts/sprd/ums512.dtsi     | 919 +++++++++++++++++++++++
> >  3 files changed, 985 insertions(+), 1 deletion(-)
> >  create mode 100644 arch/arm64/boot/dts/sprd/ums512-1h10.dts
> >  create mode 100644 arch/arm64/boot/dts/sprd/ums512.dtsi
> >
> > diff --git a/arch/arm64/boot/dts/sprd/Makefile b/arch/arm64/boot/dts/sprd/Makefile
> > index f4f1f5148cc2..97522fb0bf66 100644
> > --- a/arch/arm64/boot/dts/sprd/Makefile
> > +++ b/arch/arm64/boot/dts/sprd/Makefile
> > @@ -1,4 +1,5 @@
> >  # SPDX-License-Identifier: GPL-2.0
> >  dtb-$(CONFIG_ARCH_SPRD) += sc9836-openphone.dtb \
> >                       sp9860g-1h10.dtb        \
> > -                     sp9863a-1h10.dtb
> > +                     sp9863a-1h10.dtb        \
> > +                     ums512-1h10.dtb
> > diff --git a/arch/arm64/boot/dts/sprd/ums512-1h10.dts b/arch/arm64/boot/dts/sprd/ums512-1h10.dts
> > new file mode 100644
> > index 000000000000..d549a8c361d4
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/sprd/ums512-1h10.dts
> > @@ -0,0 +1,64 @@
> > +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> > +/*
> > + * Unisoc UMS512-1h10 boards DTS file
> > + *
> > + * Copyright (C) 2021, Unisoc Inc.
> > + */
> > +
> > +/dts-v1/;
> > +
> > +#include <dt-bindings/gpio/gpio.h>
> > +#include "ums512.dtsi"
> > +
> > +/ {
> > +     model = "Unisoc UMS512-1H10 Board";
> > +
> > +     compatible = "sprd,ums512-1h10", "sprd,ums512";
> > +
> > +     aliases {
> > +             serial0 = &uart0;
> > +             serial1 = &uart1;
> > +     };
> > +
> > +     memory@80000000 {
> > +             device_type = "memory";
> > +             reg = <0x0 0x80000000 0x0 0x80000000>;
> > +     };
> > +
> > +     chosen {
> > +             stdout-path = "serial1:115200n8";
> > +             bootargs = "earlycon";
>
> earlycon is for debugging, not wide, mainline usage. Drop.
>
> > +     };
> > +};
> > +
> > +&uart0 {
> > +     status = "okay";
> > +};
> > +
> > +&uart1 {
> > +     status = "okay";
> > +};
> > +
> > +/* SD card */
> > +&sdio0 {
> > +     status = "okay";
> > +     bus-width = <4>;
> > +     no-sdio;
> > +     no-mmc;
> > +     sprd,phy-delay-sd-uhs-sdr104 = <0x7f 0x73 0x72 0x72>;
> > +     sprd,phy-delay-sd-uhs-sdr50 = <0x6e 0x7f 0x01 0x01>;
> > +     sprd,phy-delay-sd-highspeed = <0x7f 0x1a 0x9a 0x9a>;
> > +     sprd,phy-delay-legacy = <0x7f 0x1a 0x9a 0x9a>;
> > +     sd-uhs-sdr104;
> > +     sd-uhs-sdr50;
> > +};
> > +
> > +/* EMMC storage */
> > +&sdio3 {
> > +     status = "okay";
> > +     bus-width = <8>;
> > +     no-sdio;
> > +     no-sd;
> > +     non-removable;
> > +     cap-mmc-hw-reset;
> > +};
> > diff --git a/arch/arm64/boot/dts/sprd/ums512.dtsi b/arch/arm64/boot/dts/sprd/ums512.dtsi
> > new file mode 100644
> > index 000000000000..028e17f8198e
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/sprd/ums512.dtsi
> > @@ -0,0 +1,919 @@
> > +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> > +/*
> > + * Unisoc UMS512 SoC DTS file
> > + *
> > + * Copyright (C) 2021, Unisoc Inc.
> > + */
> > +
> > +#include <dt-bindings/clock/sprd,ums512-clk.h>
> > +#include <dt-bindings/interrupt-controller/arm-gic.h>
> > +
> > +/ {
> > +     interrupt-parent = <&gic>;
> > +     #address-cells = <2>;
> > +     #size-cells = <2>;
> > +
> > +     cpus {
> > +             #address-cells = <2>;
> > +             #size-cells = <0>;
> > +
> > +             cpu-map {
> > +                     cluster0 {
> > +                             core0 {
> > +                                     cpu = <&CPU0>;
> > +                             };
> > +                             core1 {
> > +                                     cpu = <&CPU1>;
> > +                             };
> > +                             core2 {
> > +                                     cpu = <&CPU2>;
> > +                             };
> > +                             core3 {
> > +                                     cpu = <&CPU3>;
> > +                             };
> > +                             core4 {
> > +                                     cpu = <&CPU4>;
> > +                             };
> > +                             core5 {
> > +                                     cpu = <&CPU5>;
> > +                             };
> > +                             core6 {
> > +                                     cpu = <&CPU6>;
> > +                             };
> > +                             core7 {
> > +                                     cpu = <&CPU7>;
> > +                             };
> > +                     };
> > +             };
> > +
> > +             CPU0: cpu@0 {
> > +                     device_type = "cpu";
> > +                     compatible = "arm,cortex-a55";
> > +                     reg = <0x0 0x0>;
> > +                     enable-method = "psci";
> > +                     cpu-idle-states = <&CORE_PD>;
> > +             };
> > +
> > +             CPU1: cpu@100 {
> > +                     device_type = "cpu";
> > +                     compatible = "arm,cortex-a55";
> > +                     reg = <0x0 0x100>;
> > +                     enable-method = "psci";
> > +                     cpu-idle-states = <&CORE_PD>;
> > +             };
> > +
> > +             CPU2: cpu@200 {
> > +                     device_type = "cpu";
> > +                     compatible = "arm,cortex-a55";
> > +                     reg = <0x0 0x200>;
> > +                     enable-method = "psci";
> > +                     cpu-idle-states = <&CORE_PD>;
> > +             };
> > +
> > +             CPU3: cpu@300 {
> > +                     device_type = "cpu";
> > +                     compatible = "arm,cortex-a55";
> > +                     reg = <0x0 0x300>;
> > +                     enable-method = "psci";
> > +                     cpu-idle-states = <&CORE_PD>;
> > +             };
> > +
> > +             CPU4: cpu@400 {
> > +                     device_type = "cpu";
> > +                     compatible = "arm,cortex-a55";
> > +                     reg = <0x0 0x400>;
> > +                     enable-method = "psci";
> > +                     cpu-idle-states = <&CORE_PD>;
> > +             };
> > +
> > +             CPU5: cpu@500 {
> > +                     device_type = "cpu";
> > +                     compatible = "arm,cortex-a55";
> > +                     reg = <0x0 0x500>;
> > +                     enable-method = "psci";
> > +                     cpu-idle-states = <&CORE_PD>;
> > +             };
> > +
> > +             CPU6: cpu@600 {
> > +                     device_type = "cpu";
> > +                     compatible = "arm,cortex-a55";
> > +                     reg = <0x0 0x600>;
> > +                     enable-method = "psci";
> > +                     cpu-idle-states = <&CORE_PD>;
> > +             };
> > +
> > +             CPU7: cpu@700 {
> > +                     device_type = "cpu";
> > +                     compatible = "arm,cortex-a55";
> > +                     reg = <0x0 0x700>;
> > +                     enable-method = "psci";
> > +                     cpu-idle-states = <&CORE_PD>;
> > +             };
> > +     };
> > +
> > +     idle-states {
> > +             entry-method = "psci";
> > +             CORE_PD: core-pd {
> > +                     compatible = "arm,idle-state";
> > +                     entry-latency-us = <4000>;
> > +                     exit-latency-us = <4000>;
> > +                     min-residency-us = <10000>;
> > +                     local-timer-stop;
> > +                     arm,psci-suspend-param = <0x00010000>;
> > +             };
> > +     };
> > +
> > +     psci {
> > +             compatible = "arm,psci-0.2";
> > +             method = "smc";
> > +     };
> > +
> > +     timer {
> > +             compatible = "arm,armv8-timer";
> > +             interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_HIGH>, /* Physical Secure PPI */
> > +                          <GIC_PPI 14 IRQ_TYPE_LEVEL_HIGH>, /* Physical Non-Secure PPI */
> > +                          <GIC_PPI 11 IRQ_TYPE_LEVEL_HIGH>, /* Virtual PPI */
> > +                          <GIC_PPI 10 IRQ_TYPE_LEVEL_HIGH>; /* Hipervisor PPI */
> > +     };
> > +
> > +     pmu {
> > +             compatible = "arm,armv8-pmuv3";
> > +             interrupts = <GIC_SPI 112 IRQ_TYPE_LEVEL_HIGH>,
> > +                          <GIC_SPI 113 IRQ_TYPE_LEVEL_HIGH>,
> > +                          <GIC_SPI 114 IRQ_TYPE_LEVEL_HIGH>,
> > +                          <GIC_SPI 115 IRQ_TYPE_LEVEL_HIGH>,
> > +                          <GIC_SPI 116 IRQ_TYPE_LEVEL_HIGH>,
> > +                          <GIC_SPI 117 IRQ_TYPE_LEVEL_HIGH>,
> > +                          <GIC_SPI 118 IRQ_TYPE_LEVEL_HIGH>,
> > +                          <GIC_SPI 119 IRQ_TYPE_LEVEL_HIGH>;
> > +     };
> > +
> > +     soc: soc {
> > +             compatible = "simple-bus";
> > +             #address-cells = <2>;
> > +             #size-cells = <2>;
> > +             ranges;
> > +
> > +             gic: interrupt-controller@12000000 {
> > +                     compatible = "arm,gic-v3";
> > +                     #interrupt-cells = <3>;
> > +                     #address-cells = <2>;
> > +                     #size-cells = <2>;
> > +                     ranges;
> > +                     redistributor-stride = <0x0 0x20000>;   /* 128KB stride */
> > +                     #redistributor-regions = <1>;
> > +                     interrupt-controller;
> > +                     reg = <0x0 0x12000000 0 0x20000>,       /* GICD */
> > +                           <0x0 0x12040000 0 0x100000>;      /* GICR */
>
> Keep reg as second property, after compatible. This applies everywhere.
>
> > +                     interrupts = <GIC_PPI 9 IRQ_TYPE_LEVEL_HIGH>;
> > +             };
> > +
> > +             ap_ahb_regs: syscon@20100000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x20100000 0 0x4000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x20100000 0x4000>;
> > +
> > +                     apahb_gate: clock-controller@0 {
> > +                             compatible = "sprd,ums512-apahb-gate";
> > +                             reg = <0x0 0x3000>;
> > +                             clocks = <&ext_26m>;
> > +                             clock-names = "ext-26m";
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             pub_apb_regs: syscon@31050000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x31050000 0 0x9000>;
> > +             };
> > +
> > +             top_dvfs_apb_regs: syscon@322a0000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x322a0000 0 0x8000>;
> > +             };
> > +
> > +             ap_intc0_regs: syscon@32310000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x32310000 0 0x1000>;
> > +             };
> > +
> > +             ap_intc1_regs: syscon@32320000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x32320000 0 0x1000>;
> > +             };
> > +
> > +             ap_intc2_regs: syscon@32330000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x32330000 0 0x1000>;
> > +             };
> > +
> > +             ap_intc3_regs: syscon@32340000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x32340000 0 0x1000>;
> > +             };
> > +
> > +             ap_intc4_regs: syscon@32350000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x32350000 0 0x1000>;
> > +             };
> > +
> > +             ap_intc5_regs: syscon@32360000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x32360000 0 0x1000>;
> > +             };
> > +
> > +             anlg_phy_g0_regs: syscon@32390000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x32390000 0 0x3000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x32390000 0x3000>;
> > +
> > +                     dpll0: clock-controller@0 {
> > +                             compatible = "sprd,ums512-g0-pll";
> > +                             reg = <0x0 0x100>;
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             anlg_phy_g2_regs: syscon@323b0000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x323b0000 0 0x3000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x323b0000 0x3000>;
> > +
> > +                     mpll1: clock-controller@0 {
> > +                             compatible = "sprd,ums512-g2-pll";
> > +                             reg = <0x0 0x100>;
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             anlg_phy_g3_regs: syscon@323c0000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x323c0000 0 0x3000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x323c0000 0x3000>;
> > +
> > +                     pll1: clock-controller@0 {
> > +                             compatible = "sprd,ums512-g3-pll";
> > +                             reg = <0x0 0x3000>;
> > +                             clocks = <&ext_26m>;
> > +                             clock-names = "ext-26m";
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             anlg_phy_gc_regs: syscon@323e0000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x323e0000 0 0x3000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x323e0000 0x3000>;
> > +
> > +                     pll2: clock-controller@0 {
> > +                             compatible = "sprd,ums512-gc-pll";
> > +                             reg = <0x0 0x100>;
> > +                             clock-names = "ext-26m";
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             anlg_phy_g10_regs: syscon@323f0000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x323f0000 0 0x3000>;
> > +             };
> > +
> > +             aon_apb_regs: syscon@327d0000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x327d0000 0 0x3000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x327d0000 0x3000>;
> > +
> > +                     aonapb_gate: clock-controller@0 {
> > +                             compatible = "sprd,ums512-aon-gate";
> > +                             reg = <0x0 0x3000>;
> > +                             clocks = <&ext_26m>;
> > +                             clock-names = "ext-26m";
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             pmu_apb_regs: syscon@327e0000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x327e0000 0 0x3000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x327e0000 0x3000>;
> > +
> > +                     pmu_gate: clock-controller@0 {
> > +                             compatible = "sprd,ums512-pmu-gate";
> > +                             reg = <0x0 0x3000>;
> > +                             clocks = <&ext_26m>;
> > +                             clock-names = "ext-26m";
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             audcp_apb_regs: syscon@3350d000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x3350d000 0 0x1000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x3350d000 0x1000>;
> > +
> > +                     audcpapb_gate: clock-controller@0 {
> > +                             compatible = "sprd,ums512-audcpapb-gate";
> > +                             reg = <0x0 0x300>;
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             audcp_ahb_regs: syscon@335e0000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x335e0000 0 0x1000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x335e0000 0x1000>;
> > +
> > +                     audcpahb_gate: clock-controller@0 {
> > +                             compatible = "sprd,ums512-audcpahb-gate";
> > +                             reg = <0x0 0x300>;
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             gpu_apb_regs: syscon@60100000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x60100000 0 0x3000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x60100000 0x3000>;
> > +
> > +                     gpu_clk: clock-controller@0 {
> > +                             compatible = "sprd,ums512-gpu-clk";
> > +                             clocks = <&ext_26m>;
> > +                             clock-names = "ext-26m";
> > +                             reg = <0x0 0x100>;
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             gpu_dvfs_apb_regs: syscon@60110000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x60110000 0 0x3000>;
> > +             };
> > +
> > +             mm_ahb_regs: syscon@62200000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x62200000 0 0x3000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x62200000 0x3000>;
> > +
> > +                     mm_gate: clock-controller@0 {
> > +                             compatible = "sprd,ums512-mm-gate-clk";
> > +                             reg = <0x0 0x3000>;
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             ap_apb_regs: syscon@71000000 {
> > +                     compatible = "sprd,ums512-glbregs", "syscon",
> > +                                  "simple-mfd";
> > +                     reg = <0 0x71000000 0 0x3000>;
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0 0x71000000 0x3000>;
> > +
> > +                     apapb_gate: clock-controller@0 {
> > +                             compatible = "sprd,ums512-apapb-gate";
> > +                             reg = <0x0 0x3000>;
> > +                             #clock-cells = <1>;
> > +                     };
> > +             };
> > +
> > +             ap_clk: clock-controller@20200000 {
> > +                     compatible = "sprd,ums512-ap-clk";
> > +                     reg = <0 0x20200000 0 0x1000>;
> > +                     clocks = <&ext_26m>;
> > +                     clock-names = "ext-26m";
> > +                     #clock-cells = <1>;
> > +             };
> > +
> > +             aon_clk: clock-controller@32080000 {
> > +                     compatible = "sprd,ums512-aonapb-clk";
> > +                     reg = <0 0x32080000 0 0x1000>;
> > +                     clocks = <&ext_26m>, <&ext_32k>,
> > +                              <&ext_4m>, <&rco_100m>;
> > +                     clock-names = "ext-26m", "ext-32k",
> > +                                   "ext-4m", "rco-100m";
> > +                     #clock-cells = <1>;
> > +             };
> > +
> > +             mm_clk: clock-controller@62100000 {
> > +                     compatible = "sprd,ums512-mm-clk";
> > +                     reg = <0 0x62100000 0 0x1000>;
> > +                     clocks = <&ext_26m>;
> > +                     clock-names = "ext-26m";
> > +                     #clock-cells = <1>;
> > +             };
> > +
> > +             /* SoC Funnel */
> > +             funnel@3c002000 {
> > +                     compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
> > +                     reg = <0 0x3c002000 0 0x1000>;
> > +                     clocks = <&ext_26m>;
> > +                     clock-names = "apb_pclk";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     funnel_soc_out_port: endpoint {
> > +                                             remote-endpoint = <&etb_in>;
> > +                                     };
> > +                             };
> > +                     };
> > +
> > +                     in-ports {
> > +                             #address-cells = <1>;
> > +                             #size-cells = <0>;
> > +
> > +                             port@1 {
> > +                                     reg = <1>;
> > +                                     funnel_soc_in_port: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_corinth_out_port>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             /* SoC ETF */
> > +             soc_etb: etb@3c003000 {
> > +                     compatible = "arm,coresight-tmc", "arm,primecell";
> > +                     reg = <0 0x3c003000 0 0x1000>;
> > +                     clocks = <&ext_26m>;
> > +                     clock-names = "apb_pclk";
> > +
> > +                     in-ports {
> > +                             port {
> > +                                     etb_in: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_soc_out_port>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             /* AP-CPU Funnel for core3/4/5/7 */
> > +             funnel@3e001000 {
> > +                     compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
> > +                     reg = <0 0x3e001000 0 0x1000>;
> > +                     clocks = <&ext_26m>;
> > +                     clock-names = "apb_pclk";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     funnel_corinth_lit_out_port: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&corinth_etf_lit_in>;
> > +                                     };
> > +                             };
> > +                     };
> > +
> > +                     in-ports {
> > +                             #address-cells = <1>;
> > +                             #size-cells = <0>;
> > +
> > +                             port@0 {
> > +                                     reg = <0>;
> > +                                     funnel_core_in_port3: endpoint {
> > +                                             remote-endpoint = <&etm3_out>;
> > +                                     };
> > +                             };
> > +
> > +                             port@1 {
> > +                                     reg = <1>;
> > +                                     funnel_core_in_port4: endpoint {
> > +                                             remote-endpoint = <&etm4_out>;
> > +                                     };
> > +                             };
> > +
> > +                             port@2 {
> > +                                     reg = <2>;
> > +                                     funnel_core_in_port5: endpoint {
> > +                                             remote-endpoint = <&etm5_out>;
> > +                                     };
> > +                             };
> > +
> > +                             port@3 {
> > +                                     reg = <3>;
> > +                                     funnel_core_in_port7: endpoint {
> > +                                             remote-endpoint = <&etm7_out>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             /* AP-CPU ETF for little cores */
> > +             etf@3e002000 {
> > +                     compatible = "arm,coresight-tmc", "arm,primecell";
> > +                     reg = <0 0x3e002000 0 0x1000>;
> > +                     clocks = <&ext_26m>;
> > +                     clock-names = "apb_pclk";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     corinth_etf_lit_out: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_corinth_from_lit_in_port>;
> > +                                     };
> > +                             };
> > +                     };
> > +
> > +                     in-ports {
> > +                             port {
> > +                                     corinth_etf_lit_in: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_corinth_lit_out_port>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             /* AP-CPU ETF for big cores */
> > +             etf@3e003000 {
> > +                     compatible = "arm,coresight-tmc", "arm,primecell";
> > +                     reg = <0 0x3e003000 0 0x1000>;
> > +                     clocks = <&ext_26m>;
> > +                     clock-names = "apb_pclk";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     corinth_etf_big_out: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_corinth_from_big_in_port>;
> > +                                     };
> > +                             };
> > +                     };
> > +
> > +                     in-ports {
> > +                             port {
> > +                                     corinth_etf_big_in: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_corinth_big_out_port>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             /* Funnel to SoC */
> > +             funnel@3e004000 {
> > +                     compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
> > +                     reg = <0 0x3e004000 0 0x1000>;
> > +                     clocks = <&ext_26m>;
> > +                     clock-names = "apb_pclk";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     funnel_corinth_out_port: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_soc_in_port>;
> > +                                     };
> > +                             };
> > +                     };
> > +
> > +                     in-ports {
> > +                             #address-cells = <1>;
> > +                             #size-cells = <0>;
> > +
> > +                             port@0 {
> > +                                     reg = <0>;
> > +                                     funnel_corinth_from_lit_in_port: endpoint {
> > +                                             remote-endpoint = <&corinth_etf_lit_out>;
> > +                                     };
> > +                             };
> > +
> > +                             port@1 {
> > +                                     reg = <1>;
> > +                                     funnel_corinth_from_big_in_port: endpoint {
> > +                                             remote-endpoint = <&corinth_etf_big_out>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             /* AP-CPU Funnel for core0/1/2/6 */
> > +             funnel@3e005000 {
> > +                     compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
> > +                     reg = <0 0x3e005000 0 0x1000>;
> > +                     clocks = <&ext_26m>;
> > +                     clock-names = "apb_pclk";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     funnel_corinth_big_out_port: endpoint {
> > +                                             remote-endpoint = <&corinth_etf_big_in>;
> > +                                     };
> > +                             };
> > +                     };
> > +
> > +                     in-ports {
> > +                             #address-cells = <1>;
> > +                             #size-cells = <0>;
> > +
> > +                             port@0 {
> > +                                     reg = <0>;
> > +                                     funnel_core_in_port0: endpoint {
> > +                                             remote-endpoint = <&etm0_out>;
> > +                                     };
> > +                             };
> > +
> > +                             port@1 {
> > +                                     reg = <1>;
> > +                                     funnel_core_in_port1: endpoint {
> > +                                             remote-endpoint = <&etm1_out>;
> > +                                     };
> > +                             };
> > +
> > +                             port@2 {
> > +                                     reg = <2>;
> > +                                     funnel_core_in_port2: endpoint {
> > +                                             remote-endpoint = <&etm2_out>;
> > +                                     };
> > +                             };
> > +
> > +                             port@3 {
> > +                                     reg = <3>;
> > +                                     funnel_core_in_port6: endpoint {
> > +                                             remote-endpoint = <&etm6_out>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             etm0: etm@3f040000 {
> > +                     compatible = "arm,coresight-etm4x", "arm,primecell";
> > +                     reg = <0 0x3f040000 0 0x1000>;
> > +                     cpu = <&CPU0>;
> > +                     clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> > +                     clock-names = "apb_pclk", "clk_cs", "cs_src";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     etm0_out: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_core_in_port0>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             etm1: etm@3f140000 {
> > +                     compatible = "arm,coresight-etm4x", "arm,primecell";
> > +                     reg = <0 0x3f140000 0 0x1000>;
> > +                     cpu = <&CPU1>;
> > +                     clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> > +                     clock-names = "apb_pclk", "clk_cs", "cs_src";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     etm1_out: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_core_in_port1>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             etm2: etm@3f240000 {
> > +                     compatible = "arm,coresight-etm4x", "arm,primecell";
> > +                     reg = <0 0x3f240000 0 0x1000>;
> > +                     cpu = <&CPU2>;
> > +                     clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> > +                     clock-names = "apb_pclk", "clk_cs", "cs_src";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     etm2_out: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_core_in_port2>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             etm3: etm@3f340000 {
> > +                     compatible = "arm,coresight-etm4x", "arm,primecell";
> > +                     reg = <0 0x3f340000 0 0x1000>;
> > +                     cpu = <&CPU3>;
> > +                     clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> > +                     clock-names = "apb_pclk", "clk_cs", "cs_src";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     etm3_out: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_core_in_port3>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             etm4: etm@3f440000 {
> > +                     compatible = "arm,coresight-etm4x", "arm,primecell";
> > +                     reg = <0 0x3f440000 0 0x1000>;
> > +                     cpu = <&CPU4>;
> > +                     clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> > +                     clock-names = "apb_pclk", "clk_cs", "cs_src";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     etm4_out: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_core_in_port4>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             etm5: etm@3f540000 {
> > +                     compatible = "arm,coresight-etm4x", "arm,primecell";
> > +                     reg = <0 0x3f540000 0 0x1000>;
> > +                     cpu = <&CPU5>;
> > +                     clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> > +                     clock-names = "apb_pclk", "clk_cs", "cs_src";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     etm5_out: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_core_in_port5>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             etm6: etm@3f640000 {
> > +                     compatible = "arm,coresight-etm4x", "arm,primecell";
> > +                     reg = <0 0x3f640000 0 0x1000>;
> > +                     cpu = <&CPU6>;
> > +                     clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> > +                     clock-names = "apb_pclk", "clk_cs", "cs_src";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     etm6_out: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_core_in_port6>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +             etm7: etm@3f740000 {
> > +                     compatible = "arm,coresight-etm4x", "arm,primecell";
> > +                     reg = <0 0x3f740000 0 0x1000>;
> > +                     cpu = <&CPU7>;
> > +                     clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
> > +                     clock-names = "apb_pclk", "clk_cs", "cs_src";
> > +
> > +                     out-ports {
> > +                             port {
> > +                                     etm7_out: endpoint {
> > +                                             remote-endpoint =
> > +                                             <&funnel_core_in_port7>;
> > +                                     };
> > +                             };
> > +                     };
> > +             };
> > +
> > +
> > +             apb@70000000 {
> > +                     compatible = "simple-bus";
> > +                     #address-cells = <1>;
> > +                     #size-cells = <1>;
> > +                     ranges = <0 0x0 0x70000000 0x10000000>;
> > +
> > +                     uart0: serial@0 {
> > +                             compatible = "sprd,ums512-uart",
> > +                                          "sprd,sc9836-uart";
> > +                             reg = <0x0 0x100>;
> > +                             interrupts = <GIC_SPI 2 IRQ_TYPE_LEVEL_HIGH>;
> > +                             clocks = <&ext_26m>;
> > +                             status = "disabled";
> > +                     };
> > +
> > +                     uart1: serial@100000 {
> > +                             compatible = "sprd,ums512-uart",
> > +                                          "sprd,sc9836-uart";
> > +                             reg = <0x100000 0x100>;
> > +                             interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>;
> > +                             clocks = <&ext_26m>;
> > +                             status = "disabled";
> > +                     };
> > +             };
> > +
> > +             ap-apb {
>
> Non-unit-address nodes cannot be mixed with unit address ones. Something
> is wrong here.

To make sure I understand correctly, did you mean non-unit-address
nodes shouldn't be parent of unit-address nodes?

Thanks,
Chunyan

>
> > +                     compatible = "simple-bus";
> > +                     #address-cells = <2>;
> > +                     #size-cells = <2>;
> > +                     ranges;
> > +
> > +                     sdio0: sdio@71100000 {
> > +                             compatible = "sprd,sdhci-r11";
> > +                             reg = <0 0x71100000 0 0x1000>;
> > +                             interrupts = <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>;
> > +                             clock-names = "sdio", "enable";
> > +                             clocks = <&ap_clk CLK_SDIO0_2X>,
> > +                                      <&apapb_gate CLK_SDIO0_EB>;
> > +                             assigned-clocks = <&ap_clk CLK_SDIO0_2X>;
> > +                             assigned-clock-parents = <&pll1 CLK_RPLL>;
> > +                             status = "disabled";
> > +                     };
> > +
> > +                     sdio3: sdio@71400000 {
> > +                             compatible = "sprd,sdhci-r11";
> > +                             reg = <0 0x71400000 0 0x1000>;
> > +                             interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> > +                             clock-names = "sdio", "enable";
> > +                             clocks = <&ap_clk CLK_EMMC_2X>,
> > +                                      <&apapb_gate CLK_EMMC_EB>;
> > +                             assigned-clocks = <&ap_clk CLK_EMMC_2X>;
> > +                             assigned-clock-parents = <&pll1 CLK_RPLL>;
> > +                             status = "disabled";
> > +                     };
> > +             };
> > +
> > +             aon {
>
> Same problem.
>
> > +                     compatible = "simple-bus";
> > +                     #address-cells = <2>;
> > +                     #size-cells = <2>;
> > +                     ranges;
> > +
> > +                     adi_bus: spi@32100000 {
> > +                             compatible = "sprd,ums512-adi";
> > +                             reg = <0 0x32100000 0 0x100000>;
> > +                             #address-cells = <1>;
> > +                             #size-cells = <0>;
> > +                             sprd,hw-channels = <2 0x18cc>, <3 0x18cc>, <13 0x1854>, <15 0x1874>,
> > +                                     <17 0x1844>,<19 0x1844>, <21 0x1864>, <30 0x1820>,
> > +                                     <35 0x19b8>, <39 0x19ac>;
> > +                     };
> > +             };
> > +     };
> > +
> > +     ext_26m: ext-26m {
>
> Node names should be generic, so at least clock prefix or suffix. This
> applies to all further places as well.
>
> https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation
>
> > +             compatible = "fixed-clock";
> > +             #clock-cells = <0>;
> > +             clock-frequency = <26000000>;
> > +             clock-output-names = "ext-26m";
> > +     };
> > +
> > +     ext_32k: ext-32k {
> > +             compatible = "fixed-clock";
> > +             #clock-cells = <0>;
> > +             clock-frequency = <32768>;
> > +             clock-output-names = "ext-32k";
> > +     };
> > +
> > +     ext_4m: ext-4m {
> > +             compatible = "fixed-clock";
> > +             #clock-cells = <0>;
> > +             clock-frequency = <4000000>;
> > +             clock-output-names = "ext-4m";
> > +     };
> > +
> > +     rco_100m: rco-100m {
> > +             compatible = "fixed-clock";
> > +             #clock-cells = <0>;
> > +             clock-frequency = <100000000>;
> > +             clock-output-names = "rco-100m";
> > +     };
> > +};
>
> Best regards,
> Krzysztof
>
Krzysztof Kozlowski Feb. 6, 2023, 10:34 a.m. UTC | #3
On 06/02/2023 10:28, Chunyan Zhang wrote:
> Hi Krzysztof,
> 
> Thanks for the review, and sorry for the late reply, I just came back
> from a long holiday.

You quoted below entire email, so I have no clue whether there is
relevant part or not. Trim your responses and do not top-post.

Best regards,
Krzysztof
Chunyan Zhang Feb. 7, 2023, 2:19 a.m. UTC | #4
On Thu, 19 Jan 2023 at 19:40, Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 18/01/2023 09:40, Chunyan Zhang wrote:
> > Add basic support for Unisoc's UMS512, with this patch,
> > the board ums512-1h10 can run into console.
> >

[snip]

> > +             ap-apb {
>
> Non-unit-address nodes cannot be mixed with unit address ones. Something
> is wrong here.

To make sure I understand correctly, did you mean non-unit-address
nodes shouldn't be the parent of unit-address nodes?

Does that mean the bus node should have a unique base address like:
        ap-apb@70000000 {
        ...

Thanks,
Chunyan

>
> > +                     compatible = "simple-bus";
> > +                     #address-cells = <2>;
> > +                     #size-cells = <2>;
> > +                     ranges;
> > +
> > +                     sdio0: sdio@71100000 {
> > +                             compatible = "sprd,sdhci-r11";
> > +                             reg = <0 0x71100000 0 0x1000>;
> > +                             interrupts = <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>;
> > +                             clock-names = "sdio", "enable";
> > +                             clocks = <&ap_clk CLK_SDIO0_2X>,
> > +                                      <&apapb_gate CLK_SDIO0_EB>;
> > +                             assigned-clocks = <&ap_clk CLK_SDIO0_2X>;
> > +                             assigned-clock-parents = <&pll1 CLK_RPLL>;
> > +                             status = "disabled";
> > +                     };
Krzysztof Kozlowski Feb. 7, 2023, 7:59 a.m. UTC | #5
On 07/02/2023 03:19, Chunyan Zhang wrote:
> On Thu, 19 Jan 2023 at 19:40, Krzysztof Kozlowski
> <krzysztof.kozlowski@linaro.org> wrote:
>>
>> On 18/01/2023 09:40, Chunyan Zhang wrote:
>>> Add basic support for Unisoc's UMS512, with this patch,
>>> the board ums512-1h10 can run into console.
>>>
> 
> [snip]
> 
>>> +             ap-apb {
>>
>> Non-unit-address nodes cannot be mixed with unit address ones. Something
>> is wrong here.
> 
> To make sure I understand correctly, did you mean non-unit-address
> nodes shouldn't be the parent of unit-address nodes?
> 
> Does that mean the bus node should have a unique base address like:
>         ap-apb@70000000 {

No. I mean, run dtbs W=1 or dtbs_check DT_SCHEMA_FILES=simple-bus


Best regards,
Krzysztof
Chunyan Zhang Feb. 7, 2023, 8:21 a.m. UTC | #6
On Tue, 7 Feb 2023 at 15:59, Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 07/02/2023 03:19, Chunyan Zhang wrote:
> > On Thu, 19 Jan 2023 at 19:40, Krzysztof Kozlowski
> > <krzysztof.kozlowski@linaro.org> wrote:
> >>
> >> On 18/01/2023 09:40, Chunyan Zhang wrote:
> >>> Add basic support for Unisoc's UMS512, with this patch,
> >>> the board ums512-1h10 can run into console.
> >>>
> >
> > [snip]
> >
> >>> +             ap-apb {
> >>
> >> Non-unit-address nodes cannot be mixed with unit address ones. Something
> >> is wrong here.
> >
> > To make sure I understand correctly, did you mean non-unit-address
> > nodes shouldn't be the parent of unit-address nodes?
> >
> > Does that mean the bus node should have a unique base address like:
> >         ap-apb@70000000 {
>
> No. I mean, run dtbs W=1 or dtbs_check DT_SCHEMA_FILES=simple-bus

Ok, I will fix that according to the report.

BTW, where can I see what W=1 means?

Thanks,
Chunyan

>
>
> Best regards,
> Krzysztof
>
Krzysztof Kozlowski Feb. 7, 2023, 8:40 a.m. UTC | #7
On 07/02/2023 09:21, Chunyan Zhang wrote:
>>>> Non-unit-address nodes cannot be mixed with unit address ones. Something
>>>> is wrong here.
>>>
>>> To make sure I understand correctly, did you mean non-unit-address
>>> nodes shouldn't be the parent of unit-address nodes?
>>>
>>> Does that mean the bus node should have a unique base address like:
>>>         ap-apb@70000000 {
>>
>> No. I mean, run dtbs W=1 or dtbs_check DT_SCHEMA_FILES=simple-bus
> 
> Ok, I will fix that according to the report.
> 
> BTW, where can I see what W=1 means?

In standard kernel documentation or make help.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/sprd/Makefile b/arch/arm64/boot/dts/sprd/Makefile
index f4f1f5148cc2..97522fb0bf66 100644
--- a/arch/arm64/boot/dts/sprd/Makefile
+++ b/arch/arm64/boot/dts/sprd/Makefile
@@ -1,4 +1,5 @@ 
 # SPDX-License-Identifier: GPL-2.0
 dtb-$(CONFIG_ARCH_SPRD) += sc9836-openphone.dtb \
 			sp9860g-1h10.dtb	\
-			sp9863a-1h10.dtb
+			sp9863a-1h10.dtb	\
+			ums512-1h10.dtb
diff --git a/arch/arm64/boot/dts/sprd/ums512-1h10.dts b/arch/arm64/boot/dts/sprd/ums512-1h10.dts
new file mode 100644
index 000000000000..d549a8c361d4
--- /dev/null
+++ b/arch/arm64/boot/dts/sprd/ums512-1h10.dts
@@ -0,0 +1,64 @@ 
+// SPDX-License-Identifier: (GPL-2.0 OR MIT)
+/*
+ * Unisoc UMS512-1h10 boards DTS file
+ *
+ * Copyright (C) 2021, Unisoc Inc.
+ */
+
+/dts-v1/;
+
+#include <dt-bindings/gpio/gpio.h>
+#include "ums512.dtsi"
+
+/ {
+	model = "Unisoc UMS512-1H10 Board";
+
+	compatible = "sprd,ums512-1h10", "sprd,ums512";
+
+	aliases {
+		serial0 = &uart0;
+		serial1 = &uart1;
+	};
+
+	memory@80000000 {
+		device_type = "memory";
+		reg = <0x0 0x80000000 0x0 0x80000000>;
+	};
+
+	chosen {
+		stdout-path = "serial1:115200n8";
+		bootargs = "earlycon";
+	};
+};
+
+&uart0 {
+	status = "okay";
+};
+
+&uart1 {
+	status = "okay";
+};
+
+/* SD card */
+&sdio0 {
+	status = "okay";
+	bus-width = <4>;
+	no-sdio;
+	no-mmc;
+	sprd,phy-delay-sd-uhs-sdr104 = <0x7f 0x73 0x72 0x72>;
+	sprd,phy-delay-sd-uhs-sdr50 = <0x6e 0x7f 0x01 0x01>;
+	sprd,phy-delay-sd-highspeed = <0x7f 0x1a 0x9a 0x9a>;
+	sprd,phy-delay-legacy = <0x7f 0x1a 0x9a 0x9a>;
+	sd-uhs-sdr104;
+	sd-uhs-sdr50;
+};
+
+/* EMMC storage */
+&sdio3 {
+	status = "okay";
+	bus-width = <8>;
+	no-sdio;
+	no-sd;
+	non-removable;
+	cap-mmc-hw-reset;
+};
diff --git a/arch/arm64/boot/dts/sprd/ums512.dtsi b/arch/arm64/boot/dts/sprd/ums512.dtsi
new file mode 100644
index 000000000000..028e17f8198e
--- /dev/null
+++ b/arch/arm64/boot/dts/sprd/ums512.dtsi
@@ -0,0 +1,919 @@ 
+// SPDX-License-Identifier: (GPL-2.0 OR MIT)
+/*
+ * Unisoc UMS512 SoC DTS file
+ *
+ * Copyright (C) 2021, Unisoc Inc.
+ */
+
+#include <dt-bindings/clock/sprd,ums512-clk.h>
+#include <dt-bindings/interrupt-controller/arm-gic.h>
+
+/ {
+	interrupt-parent = <&gic>;
+	#address-cells = <2>;
+	#size-cells = <2>;
+
+	cpus {
+		#address-cells = <2>;
+		#size-cells = <0>;
+
+		cpu-map {
+			cluster0 {
+				core0 {
+					cpu = <&CPU0>;
+				};
+				core1 {
+					cpu = <&CPU1>;
+				};
+				core2 {
+					cpu = <&CPU2>;
+				};
+				core3 {
+					cpu = <&CPU3>;
+				};
+				core4 {
+					cpu = <&CPU4>;
+				};
+				core5 {
+					cpu = <&CPU5>;
+				};
+				core6 {
+					cpu = <&CPU6>;
+				};
+				core7 {
+					cpu = <&CPU7>;
+				};
+			};
+		};
+
+		CPU0: cpu@0 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a55";
+			reg = <0x0 0x0>;
+			enable-method = "psci";
+			cpu-idle-states = <&CORE_PD>;
+		};
+
+		CPU1: cpu@100 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a55";
+			reg = <0x0 0x100>;
+			enable-method = "psci";
+			cpu-idle-states = <&CORE_PD>;
+		};
+
+		CPU2: cpu@200 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a55";
+			reg = <0x0 0x200>;
+			enable-method = "psci";
+			cpu-idle-states = <&CORE_PD>;
+		};
+
+		CPU3: cpu@300 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a55";
+			reg = <0x0 0x300>;
+			enable-method = "psci";
+			cpu-idle-states = <&CORE_PD>;
+		};
+
+		CPU4: cpu@400 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a55";
+			reg = <0x0 0x400>;
+			enable-method = "psci";
+			cpu-idle-states = <&CORE_PD>;
+		};
+
+		CPU5: cpu@500 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a55";
+			reg = <0x0 0x500>;
+			enable-method = "psci";
+			cpu-idle-states = <&CORE_PD>;
+		};
+
+		CPU6: cpu@600 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a55";
+			reg = <0x0 0x600>;
+			enable-method = "psci";
+			cpu-idle-states = <&CORE_PD>;
+		};
+
+		CPU7: cpu@700 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a55";
+			reg = <0x0 0x700>;
+			enable-method = "psci";
+			cpu-idle-states = <&CORE_PD>;
+		};
+	};
+
+	idle-states {
+		entry-method = "psci";
+		CORE_PD: core-pd {
+			compatible = "arm,idle-state";
+			entry-latency-us = <4000>;
+			exit-latency-us = <4000>;
+			min-residency-us = <10000>;
+			local-timer-stop;
+			arm,psci-suspend-param = <0x00010000>;
+		};
+	};
+
+	psci {
+		compatible = "arm,psci-0.2";
+		method = "smc";
+	};
+
+	timer {
+		compatible = "arm,armv8-timer";
+		interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_HIGH>, /* Physical Secure PPI */
+			     <GIC_PPI 14 IRQ_TYPE_LEVEL_HIGH>, /* Physical Non-Secure PPI */
+			     <GIC_PPI 11 IRQ_TYPE_LEVEL_HIGH>, /* Virtual PPI */
+			     <GIC_PPI 10 IRQ_TYPE_LEVEL_HIGH>; /* Hipervisor PPI */
+	};
+
+	pmu {
+		compatible = "arm,armv8-pmuv3";
+		interrupts = <GIC_SPI 112 IRQ_TYPE_LEVEL_HIGH>,
+			     <GIC_SPI 113 IRQ_TYPE_LEVEL_HIGH>,
+			     <GIC_SPI 114 IRQ_TYPE_LEVEL_HIGH>,
+			     <GIC_SPI 115 IRQ_TYPE_LEVEL_HIGH>,
+			     <GIC_SPI 116 IRQ_TYPE_LEVEL_HIGH>,
+			     <GIC_SPI 117 IRQ_TYPE_LEVEL_HIGH>,
+			     <GIC_SPI 118 IRQ_TYPE_LEVEL_HIGH>,
+			     <GIC_SPI 119 IRQ_TYPE_LEVEL_HIGH>;
+	};
+
+	soc: soc {
+		compatible = "simple-bus";
+		#address-cells = <2>;
+		#size-cells = <2>;
+		ranges;
+
+		gic: interrupt-controller@12000000 {
+			compatible = "arm,gic-v3";
+			#interrupt-cells = <3>;
+			#address-cells = <2>;
+			#size-cells = <2>;
+			ranges;
+			redistributor-stride = <0x0 0x20000>;	/* 128KB stride */
+			#redistributor-regions = <1>;
+			interrupt-controller;
+			reg = <0x0 0x12000000 0 0x20000>,	/* GICD */
+			      <0x0 0x12040000 0 0x100000>;	/* GICR */
+			interrupts = <GIC_PPI 9 IRQ_TYPE_LEVEL_HIGH>;
+		};
+
+		ap_ahb_regs: syscon@20100000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x20100000 0 0x4000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x20100000 0x4000>;
+
+			apahb_gate: clock-controller@0 {
+				compatible = "sprd,ums512-apahb-gate";
+				reg = <0x0 0x3000>;
+				clocks = <&ext_26m>;
+				clock-names = "ext-26m";
+				#clock-cells = <1>;
+			};
+		};
+
+		pub_apb_regs: syscon@31050000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x31050000 0 0x9000>;
+		};
+
+		top_dvfs_apb_regs: syscon@322a0000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x322a0000 0 0x8000>;
+		};
+
+		ap_intc0_regs: syscon@32310000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x32310000 0 0x1000>;
+		};
+
+		ap_intc1_regs: syscon@32320000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x32320000 0 0x1000>;
+		};
+
+		ap_intc2_regs: syscon@32330000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x32330000 0 0x1000>;
+		};
+
+		ap_intc3_regs: syscon@32340000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x32340000 0 0x1000>;
+		};
+
+		ap_intc4_regs: syscon@32350000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x32350000 0 0x1000>;
+		};
+
+		ap_intc5_regs: syscon@32360000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x32360000 0 0x1000>;
+		};
+
+		anlg_phy_g0_regs: syscon@32390000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x32390000 0 0x3000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x32390000 0x3000>;
+
+			dpll0: clock-controller@0 {
+				compatible = "sprd,ums512-g0-pll";
+				reg = <0x0 0x100>;
+				#clock-cells = <1>;
+			};
+		};
+
+		anlg_phy_g2_regs: syscon@323b0000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x323b0000 0 0x3000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x323b0000 0x3000>;
+
+			mpll1: clock-controller@0 {
+				compatible = "sprd,ums512-g2-pll";
+				reg = <0x0 0x100>;
+				#clock-cells = <1>;
+			};
+		};
+
+		anlg_phy_g3_regs: syscon@323c0000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x323c0000 0 0x3000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x323c0000 0x3000>;
+
+			pll1: clock-controller@0 {
+				compatible = "sprd,ums512-g3-pll";
+				reg = <0x0 0x3000>;
+				clocks = <&ext_26m>;
+				clock-names = "ext-26m";
+				#clock-cells = <1>;
+			};
+		};
+
+		anlg_phy_gc_regs: syscon@323e0000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x323e0000 0 0x3000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x323e0000 0x3000>;
+
+			pll2: clock-controller@0 {
+				compatible = "sprd,ums512-gc-pll";
+				reg = <0x0 0x100>;
+				clock-names = "ext-26m";
+				#clock-cells = <1>;
+			};
+		};
+
+		anlg_phy_g10_regs: syscon@323f0000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x323f0000 0 0x3000>;
+		};
+
+		aon_apb_regs: syscon@327d0000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x327d0000 0 0x3000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x327d0000 0x3000>;
+
+			aonapb_gate: clock-controller@0 {
+				compatible = "sprd,ums512-aon-gate";
+				reg = <0x0 0x3000>;
+				clocks = <&ext_26m>;
+				clock-names = "ext-26m";
+				#clock-cells = <1>;
+			};
+		};
+
+		pmu_apb_regs: syscon@327e0000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x327e0000 0 0x3000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x327e0000 0x3000>;
+
+			pmu_gate: clock-controller@0 {
+				compatible = "sprd,ums512-pmu-gate";
+				reg = <0x0 0x3000>;
+				clocks = <&ext_26m>;
+				clock-names = "ext-26m";
+				#clock-cells = <1>;
+			};
+		};
+
+		audcp_apb_regs: syscon@3350d000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x3350d000 0 0x1000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x3350d000 0x1000>;
+
+			audcpapb_gate: clock-controller@0 {
+				compatible = "sprd,ums512-audcpapb-gate";
+				reg = <0x0 0x300>;
+				#clock-cells = <1>;
+			};
+		};
+
+		audcp_ahb_regs: syscon@335e0000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x335e0000 0 0x1000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x335e0000 0x1000>;
+
+			audcpahb_gate: clock-controller@0 {
+				compatible = "sprd,ums512-audcpahb-gate";
+				reg = <0x0 0x300>;
+				#clock-cells = <1>;
+			};
+		};
+
+		gpu_apb_regs: syscon@60100000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x60100000 0 0x3000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x60100000 0x3000>;
+
+			gpu_clk: clock-controller@0 {
+				compatible = "sprd,ums512-gpu-clk";
+				clocks = <&ext_26m>;
+				clock-names = "ext-26m";
+				reg = <0x0 0x100>;
+				#clock-cells = <1>;
+			};
+		};
+
+		gpu_dvfs_apb_regs: syscon@60110000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x60110000 0 0x3000>;
+		};
+
+		mm_ahb_regs: syscon@62200000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x62200000 0 0x3000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x62200000 0x3000>;
+
+			mm_gate: clock-controller@0 {
+				compatible = "sprd,ums512-mm-gate-clk";
+				reg = <0x0 0x3000>;
+				#clock-cells = <1>;
+			};
+		};
+
+		ap_apb_regs: syscon@71000000 {
+			compatible = "sprd,ums512-glbregs", "syscon",
+				     "simple-mfd";
+			reg = <0 0x71000000 0 0x3000>;
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0 0x71000000 0x3000>;
+
+			apapb_gate: clock-controller@0 {
+				compatible = "sprd,ums512-apapb-gate";
+				reg = <0x0 0x3000>;
+				#clock-cells = <1>;
+			};
+		};
+
+		ap_clk: clock-controller@20200000 {
+			compatible = "sprd,ums512-ap-clk";
+			reg = <0 0x20200000 0 0x1000>;
+			clocks = <&ext_26m>;
+			clock-names = "ext-26m";
+			#clock-cells = <1>;
+		};
+
+		aon_clk: clock-controller@32080000 {
+			compatible = "sprd,ums512-aonapb-clk";
+			reg = <0 0x32080000 0 0x1000>;
+			clocks = <&ext_26m>, <&ext_32k>,
+				 <&ext_4m>, <&rco_100m>;
+			clock-names = "ext-26m", "ext-32k",
+				      "ext-4m", "rco-100m";
+			#clock-cells = <1>;
+		};
+
+		mm_clk: clock-controller@62100000 {
+			compatible = "sprd,ums512-mm-clk";
+			reg = <0 0x62100000 0 0x1000>;
+			clocks = <&ext_26m>;
+			clock-names = "ext-26m";
+			#clock-cells = <1>;
+		};
+
+		/* SoC Funnel */
+		funnel@3c002000 {
+			compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
+			reg = <0 0x3c002000 0 0x1000>;
+			clocks = <&ext_26m>;
+			clock-names = "apb_pclk";
+
+			out-ports {
+				port {
+					funnel_soc_out_port: endpoint {
+						remote-endpoint = <&etb_in>;
+					};
+				};
+			};
+
+			in-ports {
+				#address-cells = <1>;
+				#size-cells = <0>;
+
+				port@1 {
+					reg = <1>;
+					funnel_soc_in_port: endpoint {
+						remote-endpoint =
+						<&funnel_corinth_out_port>;
+					};
+				};
+			};
+		};
+
+		/* SoC ETF */
+		soc_etb: etb@3c003000 {
+			compatible = "arm,coresight-tmc", "arm,primecell";
+			reg = <0 0x3c003000 0 0x1000>;
+			clocks = <&ext_26m>;
+			clock-names = "apb_pclk";
+
+			in-ports {
+				port {
+					etb_in: endpoint {
+						remote-endpoint =
+						<&funnel_soc_out_port>;
+					};
+				};
+			};
+		};
+
+		/* AP-CPU Funnel for core3/4/5/7 */
+		funnel@3e001000 {
+			compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
+			reg = <0 0x3e001000 0 0x1000>;
+			clocks = <&ext_26m>;
+			clock-names = "apb_pclk";
+
+			out-ports {
+				port {
+					funnel_corinth_lit_out_port: endpoint {
+						remote-endpoint =
+						<&corinth_etf_lit_in>;
+					};
+				};
+			};
+
+			in-ports {
+				#address-cells = <1>;
+				#size-cells = <0>;
+
+				port@0 {
+					reg = <0>;
+					funnel_core_in_port3: endpoint {
+						remote-endpoint = <&etm3_out>;
+					};
+				};
+
+				port@1 {
+					reg = <1>;
+					funnel_core_in_port4: endpoint {
+						remote-endpoint = <&etm4_out>;
+					};
+				};
+
+				port@2 {
+					reg = <2>;
+					funnel_core_in_port5: endpoint {
+						remote-endpoint = <&etm5_out>;
+					};
+				};
+
+				port@3 {
+					reg = <3>;
+					funnel_core_in_port7: endpoint {
+						remote-endpoint = <&etm7_out>;
+					};
+				};
+			};
+		};
+
+		/* AP-CPU ETF for little cores */
+		etf@3e002000 {
+			compatible = "arm,coresight-tmc", "arm,primecell";
+			reg = <0 0x3e002000 0 0x1000>;
+			clocks = <&ext_26m>;
+			clock-names = "apb_pclk";
+
+			out-ports {
+				port {
+					corinth_etf_lit_out: endpoint {
+						remote-endpoint =
+						<&funnel_corinth_from_lit_in_port>;
+					};
+				};
+			};
+
+			in-ports {
+				port {
+					corinth_etf_lit_in: endpoint {
+						remote-endpoint =
+						<&funnel_corinth_lit_out_port>;
+					};
+				};
+			};
+		};
+
+		/* AP-CPU ETF for big cores */
+		etf@3e003000 {
+			compatible = "arm,coresight-tmc", "arm,primecell";
+			reg = <0 0x3e003000 0 0x1000>;
+			clocks = <&ext_26m>;
+			clock-names = "apb_pclk";
+
+			out-ports {
+				port {
+					corinth_etf_big_out: endpoint {
+						remote-endpoint =
+						<&funnel_corinth_from_big_in_port>;
+					};
+				};
+			};
+
+			in-ports {
+				port {
+					corinth_etf_big_in: endpoint {
+						remote-endpoint =
+						<&funnel_corinth_big_out_port>;
+					};
+				};
+			};
+		};
+
+		/* Funnel to SoC */
+		funnel@3e004000 {
+			compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
+			reg = <0 0x3e004000 0 0x1000>;
+			clocks = <&ext_26m>;
+			clock-names = "apb_pclk";
+
+			out-ports {
+				port {
+					funnel_corinth_out_port: endpoint {
+						remote-endpoint =
+						<&funnel_soc_in_port>;
+					};
+				};
+			};
+
+			in-ports {
+				#address-cells = <1>;
+				#size-cells = <0>;
+
+				port@0 {
+					reg = <0>;
+					funnel_corinth_from_lit_in_port: endpoint {
+						remote-endpoint = <&corinth_etf_lit_out>;
+					};
+				};
+
+				port@1 {
+					reg = <1>;
+					funnel_corinth_from_big_in_port: endpoint {
+						remote-endpoint = <&corinth_etf_big_out>;
+					};
+				};
+			};
+		};
+
+		/* AP-CPU Funnel for core0/1/2/6 */
+		funnel@3e005000 {
+			compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
+			reg = <0 0x3e005000 0 0x1000>;
+			clocks = <&ext_26m>;
+			clock-names = "apb_pclk";
+
+			out-ports {
+				port {
+					funnel_corinth_big_out_port: endpoint {
+						remote-endpoint = <&corinth_etf_big_in>;
+					};
+				};
+			};
+
+			in-ports {
+				#address-cells = <1>;
+				#size-cells = <0>;
+
+				port@0 {
+					reg = <0>;
+					funnel_core_in_port0: endpoint {
+						remote-endpoint = <&etm0_out>;
+					};
+				};
+
+				port@1 {
+					reg = <1>;
+					funnel_core_in_port1: endpoint {
+						remote-endpoint = <&etm1_out>;
+					};
+				};
+
+				port@2 {
+					reg = <2>;
+					funnel_core_in_port2: endpoint {
+						remote-endpoint = <&etm2_out>;
+					};
+				};
+
+				port@3 {
+					reg = <3>;
+					funnel_core_in_port6: endpoint {
+						remote-endpoint = <&etm6_out>;
+					};
+				};
+			};
+		};
+
+		etm0: etm@3f040000 {
+			compatible = "arm,coresight-etm4x", "arm,primecell";
+			reg = <0 0x3f040000 0 0x1000>;
+			cpu = <&CPU0>;
+			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
+			clock-names = "apb_pclk", "clk_cs", "cs_src";
+
+			out-ports {
+				port {
+					etm0_out: endpoint {
+						remote-endpoint =
+						<&funnel_core_in_port0>;
+					};
+				};
+			};
+		};
+
+		etm1: etm@3f140000 {
+			compatible = "arm,coresight-etm4x", "arm,primecell";
+			reg = <0 0x3f140000 0 0x1000>;
+			cpu = <&CPU1>;
+			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
+			clock-names = "apb_pclk", "clk_cs", "cs_src";
+
+			out-ports {
+				port {
+					etm1_out: endpoint {
+						remote-endpoint =
+						<&funnel_core_in_port1>;
+					};
+				};
+			};
+		};
+
+		etm2: etm@3f240000 {
+			compatible = "arm,coresight-etm4x", "arm,primecell";
+			reg = <0 0x3f240000 0 0x1000>;
+			cpu = <&CPU2>;
+			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
+			clock-names = "apb_pclk", "clk_cs", "cs_src";
+
+			out-ports {
+				port {
+					etm2_out: endpoint {
+						remote-endpoint =
+						<&funnel_core_in_port2>;
+					};
+				};
+			};
+		};
+
+		etm3: etm@3f340000 {
+			compatible = "arm,coresight-etm4x", "arm,primecell";
+			reg = <0 0x3f340000 0 0x1000>;
+			cpu = <&CPU3>;
+			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
+			clock-names = "apb_pclk", "clk_cs", "cs_src";
+
+			out-ports {
+				port {
+					etm3_out: endpoint {
+						remote-endpoint =
+						<&funnel_core_in_port3>;
+					};
+				};
+			};
+		};
+
+		etm4: etm@3f440000 {
+			compatible = "arm,coresight-etm4x", "arm,primecell";
+			reg = <0 0x3f440000 0 0x1000>;
+			cpu = <&CPU4>;
+			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
+			clock-names = "apb_pclk", "clk_cs", "cs_src";
+
+			out-ports {
+				port {
+					etm4_out: endpoint {
+						remote-endpoint =
+						<&funnel_core_in_port4>;
+					};
+				};
+			};
+		};
+
+		etm5: etm@3f540000 {
+			compatible = "arm,coresight-etm4x", "arm,primecell";
+			reg = <0 0x3f540000 0 0x1000>;
+			cpu = <&CPU5>;
+			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
+			clock-names = "apb_pclk", "clk_cs", "cs_src";
+
+			out-ports {
+				port {
+					etm5_out: endpoint {
+						remote-endpoint =
+						<&funnel_core_in_port5>;
+					};
+				};
+			};
+		};
+
+		etm6: etm@3f640000 {
+			compatible = "arm,coresight-etm4x", "arm,primecell";
+			reg = <0 0x3f640000 0 0x1000>;
+			cpu = <&CPU6>;
+			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
+			clock-names = "apb_pclk", "clk_cs", "cs_src";
+
+			out-ports {
+				port {
+					etm6_out: endpoint {
+						remote-endpoint =
+						<&funnel_core_in_port6>;
+					};
+				};
+			};
+		};
+
+		etm7: etm@3f740000 {
+			compatible = "arm,coresight-etm4x", "arm,primecell";
+			reg = <0 0x3f740000 0 0x1000>;
+			cpu = <&CPU7>;
+			clocks = <&ext_26m>, <&aon_clk CLK_CSSYS>, <&pll2 CLK_TWPLL_512M>;
+			clock-names = "apb_pclk", "clk_cs", "cs_src";
+
+			out-ports {
+				port {
+					etm7_out: endpoint {
+						remote-endpoint =
+						<&funnel_core_in_port7>;
+					};
+				};
+			};
+		};
+
+
+		apb@70000000 {
+			compatible = "simple-bus";
+			#address-cells = <1>;
+			#size-cells = <1>;
+			ranges = <0 0x0 0x70000000 0x10000000>;
+
+			uart0: serial@0 {
+				compatible = "sprd,ums512-uart",
+					     "sprd,sc9836-uart";
+				reg = <0x0 0x100>;
+				interrupts = <GIC_SPI 2 IRQ_TYPE_LEVEL_HIGH>;
+				clocks = <&ext_26m>;
+				status = "disabled";
+			};
+
+			uart1: serial@100000 {
+				compatible = "sprd,ums512-uart",
+					     "sprd,sc9836-uart";
+				reg = <0x100000 0x100>;
+				interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>;
+				clocks = <&ext_26m>;
+				status = "disabled";
+			};
+		};
+
+		ap-apb {
+			compatible = "simple-bus";
+			#address-cells = <2>;
+			#size-cells = <2>;
+			ranges;
+
+			sdio0: sdio@71100000 {
+				compatible = "sprd,sdhci-r11";
+				reg = <0 0x71100000 0 0x1000>;
+				interrupts = <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>;
+				clock-names = "sdio", "enable";
+				clocks = <&ap_clk CLK_SDIO0_2X>,
+					 <&apapb_gate CLK_SDIO0_EB>;
+				assigned-clocks = <&ap_clk CLK_SDIO0_2X>;
+				assigned-clock-parents = <&pll1 CLK_RPLL>;
+				status = "disabled";
+			};
+
+			sdio3: sdio@71400000 {
+				compatible = "sprd,sdhci-r11";
+				reg = <0 0x71400000 0 0x1000>;
+				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
+				clock-names = "sdio", "enable";
+				clocks = <&ap_clk CLK_EMMC_2X>,
+					 <&apapb_gate CLK_EMMC_EB>;
+				assigned-clocks = <&ap_clk CLK_EMMC_2X>;
+				assigned-clock-parents = <&pll1 CLK_RPLL>;
+				status = "disabled";
+			};
+		};
+
+		aon {
+			compatible = "simple-bus";
+			#address-cells = <2>;
+			#size-cells = <2>;
+			ranges;
+
+			adi_bus: spi@32100000 {
+				compatible = "sprd,ums512-adi";
+				reg = <0 0x32100000 0 0x100000>;
+				#address-cells = <1>;
+				#size-cells = <0>;
+				sprd,hw-channels = <2 0x18cc>, <3 0x18cc>, <13 0x1854>, <15 0x1874>,
+					<17 0x1844>,<19 0x1844>, <21 0x1864>, <30 0x1820>,
+					<35 0x19b8>, <39 0x19ac>;
+			};
+		};
+	};
+
+	ext_26m: ext-26m {
+		compatible = "fixed-clock";
+		#clock-cells = <0>;
+		clock-frequency = <26000000>;
+		clock-output-names = "ext-26m";
+	};
+
+	ext_32k: ext-32k {
+		compatible = "fixed-clock";
+		#clock-cells = <0>;
+		clock-frequency = <32768>;
+		clock-output-names = "ext-32k";
+	};
+
+	ext_4m: ext-4m {
+		compatible = "fixed-clock";
+		#clock-cells = <0>;
+		clock-frequency = <4000000>;
+		clock-output-names = "ext-4m";
+	};
+
+	rco_100m: rco-100m {
+		compatible = "fixed-clock";
+		#clock-cells = <0>;
+		clock-frequency = <100000000>;
+		clock-output-names = "rco-100m";
+	};
+};