Re: [PATCH v2 3/3] arm64: dts: amlogic: add T7 peripheral nodes
From: Ronald Claveau
Date: Thu Sep 24 2026 - 07:36:04 EST
On 9/24/26 7:51 AM, Xianwei Zhao via B4 Relay wrote:
> From: Xianwei Zhao <xianwei.zhao@xxxxxxxxxxx>
>
> Add device tree descriptions for the peripherals on the Amlogic T7
> SoC and the T7 A311D2 AN400 board.
>
> Add support for the T7 Ethernet controller, SPI controllers, I2C
> controllers, UARTs, pinctrl groups, regulators, and reserved memory.
> Also enable the Ethernet controller on the AN400 board.
>
> Signed-off-by: Xianwei Zhao <xianwei.zhao@xxxxxxxxxxx>
> ---
> .../boot/dts/amlogic/amlogic-t7-a311d2-an400.dts | 153 +++++++
> arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi | 462 +++++++++++++++++++++
> 2 files changed, 615 insertions(+)
>
--snip--
> +
> pwm_a_pins: pwm-a {
> mux {
> groups = "pwm_a";
> @@ -622,6 +848,51 @@ uart_a: serial@78000 {
> status = "disabled";
> };
>
> + uart_b: serial@7a000 {
> + compatible = "amlogic,t7-uart", "amlogic,meson-s4-uart";
> + reg = <0x0 0x7a000 0x0 0x18>;
> + interrupts = <GIC_SPI 169 IRQ_TYPE_EDGE_RISING>;
> + clocks = <&xtal>, <&clkc_periphs CLKID_SYS_UART_B>, <&xtal>;
> + clock-names = "xtal", "pclk", "baud";
> + status = "disabled";
> + };
> +
> + uart_c: serial@7c000 {
> + compatible = "amlogic,t7-uart", "amlogic,meson-s4-uart";
> + reg = <0x0 0x7c000 0x0 0x18>;
> + interrupts = <GIC_SPI 170 IRQ_TYPE_EDGE_RISING>;
> + clocks = <&xtal>, <&clkc_periphs CLKID_SYS_UART_C>, <&xtal>;
> + clock-names = "xtal", "pclk", "baud";
> + status = "disabled";
> + };
> +
> + uart_d: serial@7e000 {
> + compatible = "amlogic,t7-uart", "amlogic,meson-s4-uart";
> + reg = <0x0 0x7e000 0x0 0x18>;
> + interrupts = <GIC_SPI 171 IRQ_TYPE_EDGE_RISING>;
> + clocks = <&xtal>, <&clkc_periphs CLKID_SYS_UART_D>, <&xtal>;
> + clock-names = "xtal", "pclk", "baud";
> + status = "disabled";
> + };
> +
> + uart_e: serial@80000 {
> + compatible = "amlogic,t7-uart", "amlogic,meson-s4-uart";
> + reg = <0x0 0x80000 0x0 0x18>;
> + interrupts = <GIC_SPI 172 IRQ_TYPE_EDGE_RISING>;
> + clocks = <&xtal>, <&clkc_periphs CLKID_SYS_UART_E>, <&xtal>;
> + clock-names = "xtal", "pclk", "baud";
> + status = "disabled";
> + };
> +
> + uart_f: serial@82000 {
> + compatible = "amlogic,t7-uart", "amlogic,meson-s4-uart";
> + reg = <0x0 0x82000 0x0 0x18>;
> + interrupts = <GIC_SPI 173 IRQ_TYPE_EDGE_RISING>;
> + clocks = <&xtal>, <&clkc_periphs CLKID_SYS_UART_F>, <&xtal>;
> + clock-names = "xtal", "pclk", "baud";
> + status = "disabled";
> + };
> +
Thank you for this change Xianwei.
Can you please move the UARTs in the right place between i2c_ao_a and
i2c_ao_b ?
> gp0: clock-controller@8080 {
> compatible = "amlogic,t7-gp0-pll";
> reg = <0x0 0x8080 0x0 0x20>;
> @@ -688,6 +959,39 @@ sec_ao: ao-secure@10220 {
> amlogic,has-chip-id;
> };
>
--snip--
> pwm_ab: pwm@58000 {
> compatible = "amlogic,t7-pwm", "amlogic,meson-s4-pwm";
> reg = <0x0 0x58000 0x0 0x24>;
> @@ -756,6 +1138,86 @@ pwm_ao_cd: pwm@60000 {
> status = "disabled";
> };
>
> + i2c0: i2c@66000 {
> + compatible = "amlogic,t7-i2c", "amlogic,meson-axg-i2c";
> + reg = <0x0 0x66000 0x0 0x48>;
> + interrupts = <GIC_SPI 160 IRQ_TYPE_EDGE_RISING>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + clocks = <&clkc_periphs CLKID_SYS_I2C_M_A>;
> + status = "disabled";
> + };
> +
> + i2c1: i2c@68000 {
> + compatible = "amlogic,t7-i2c", "amlogic,meson-axg-i2c";
> + reg = <0x0 0x68000 0x0 0x48>;
> + interrupts = <GIC_SPI 161 IRQ_TYPE_EDGE_RISING>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + clocks = <&clkc_periphs CLKID_SYS_I2C_M_B>;
> + status = "disabled";
> + };
> +
> + i2c2: i2c@6a000 {
> + compatible = "amlogic,t7-i2c", "amlogic,meson-axg-i2c";
> + reg = <0x0 0x6a000 0x0 0x48>;
> + interrupts = <GIC_SPI 162 IRQ_TYPE_EDGE_RISING>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + clocks = <&clkc_periphs CLKID_SYS_I2C_M_C>;
> + status = "disabled";
> + };
> +
> + i2c3: i2c@6c000 {
> + compatible = "amlogic,t7-i2c", "amlogic,meson-axg-i2c";
> + reg = <0x0 0x6c000 0x0 0x48>;
> + interrupts = <GIC_SPI 163 IRQ_TYPE_EDGE_RISING>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + clocks = <&clkc_periphs CLKID_SYS_I2C_M_D>;
> + status = "disabled";
> + };
> +
> + i2c4: i2c@6e000 {
> + compatible = "amlogic,t7-i2c", "amlogic,meson-axg-i2c";
> + reg = <0x0 0x6e000 0x0 0x48>;
> + interrupts = <GIC_SPI 164 IRQ_TYPE_EDGE_RISING>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + clocks = <&clkc_periphs CLKID_SYS_I2C_M_E>;
> + status = "disabled";
> + };
> +
> + i2c5: i2c@70000 {
> + compatible = "amlogic,t7-i2c", "amlogic,meson-axg-i2c";
> + reg = <0x0 0x070000 0x0 0x48>;
> + interrupts = <GIC_SPI 165 IRQ_TYPE_EDGE_RISING>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + clocks = <&clkc_periphs CLKID_SYS_I2C_M_F>;
> + status = "disabled";
> + };
> +
> + i2c_ao_a: i2c@76000 {
> + compatible = "amlogic,t7-i2c", "amlogic,meson-axg-i2c";
> + reg = <0x0 0x076000 0x0 0x48>;
> + interrupts = <GIC_SPI 330 IRQ_TYPE_EDGE_RISING>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + clocks = <&clkc_periphs CLKID_SYS_I2C_AO_A>;
> + status = "disabled";
> + };
> +
> + i2c_ao_b: i2c@86000 {
> + compatible = "amlogic,t7-i2c", "amlogic,meson-axg-i2c";
> + reg = <0x0 0x086000 0x0 0x48>;
> + interrupts = <GIC_SPI 331 IRQ_TYPE_EDGE_RISING>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + clocks = <&clkc_periphs CLKID_SYS_I2C_AO_B>;
> + status = "disabled";
> + };
> +
> sd_emmc_a: mmc@88000 {
> compatible = "amlogic,t7-mmc", "amlogic,meson-axg-mmc";
> reg = <0x0 0x88000 0x0 0x800>;
>
--
Best regards,
Ronald