Hi Weijie, I manually fixed the last 3 patches in the series adding support for newer SoCs to mkimage/mtk_image.
On my BPi-R3 (mt7986a) I was now trying to use mkimage instead of bromimage in the same way you introduced that option for mt7622[1]: The resulting bl2.img works fine on SPI-NAND and eMMC, however, booting from SPI-NOR or SD card doesn't work. On SPI-NOR: F0: 102B 0000 FA: 0000 0000 V0: 7027 6006 [0001] 00: 1017 0000 FA: 5100 0000 01: 102A 0001 02: 5100 0000 BP: 2000 00C0 [0001] EC: 0000 0000 [0000] T0: 0000 0213 [010F] System halt! On SD card: F0: 102B 0000 FA: 1040 0000 FA: 1040 0000 [0200] F9: 103F 0000 F3: 1001 0000 [0200] F3: 1001 0000 F6: 300C 0028 F5: 0000 0000 L0: 8005 0000 [0001] 00: 1012 0000 BP: 2000 00C0 [0001] EC: 0000 0000 [3000] T0: 0000 00E8 [010F] System halt! All generate image headers also differ from the ones generated by bromimage, however, for SPI-NAND and eMMC they still work. hexdiff of SD card bl2.img @@ -65,7 +65,7 @@ 00004420 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff |................| * 00004600 42 52 4c 59 54 00 00 00 01 00 00 00 00 4a 00 00 |BRLYT........J..| -00004610 88 51 03 00 42 42 42 42 08 00 01 00 00 06 00 00 |.Q..BBBB........| +00004610 88 51 03 00 42 42 42 42 08 00 01 00 00 4a 00 00 |.Q..BBBB.....J..| 00004620 88 51 03 00 ff ff ff ff ff ff ff ff ff ff ff ff |.Q..............| 00004630 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff |................| * @@ -73,15 +73,15 @@ 00004a10 4f 00 00 00 01 00 00 00 01 00 05 01 00 0d 20 00 |O............. .| 00004a20 88 07 03 00 88 0d 03 00 00 03 00 00 20 00 00 00 |............ ...| 00004a30 00 03 00 00 01 00 00 00 4d 4d 4d 01 0c 00 01 00 |........MMM.....| -00004a40 01 00 00 00 4d 4d 4d 01 14 00 02 00 00 00 00 00 |....MMM.........| -00004a50 10 00 00 00 80 00 00 00 4d 4d 4d 01 14 02 03 00 |........MMM.....| -00004a60 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| -* -00004c60 00 00 00 00 00 00 00 00 00 00 00 00 4d 4d 4d 03 |............MMM.| -00004c70 64 00 07 00 90 11 00 00 00 00 00 00 00 00 00 00 |d...............| -00004c80 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| +00004a40 01 00 00 00 4d 4d 4d 03 64 00 07 00 90 11 00 00 |....MMM.d.......| +00004a50 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| * -00004cc0 00 64 00 00 88 13 00 00 00 00 00 00 00 00 00 00 |.d..............| +00004a90 00 00 00 00 00 00 00 00 00 64 00 00 88 13 00 00 |.........d......| +00004aa0 00 00 00 00 00 00 00 00 4d 4d 4d 01 14 02 03 00 |........MMM.....| +00004ab0 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| +* +00004cb0 00 00 00 00 00 00 00 00 00 00 00 00 4d 4d 4d 01 |............MMM.| +00004cc0 14 00 02 00 00 00 00 00 10 00 00 00 80 00 00 00 |................| 00004cd0 4d 4d 4d 01 30 00 08 00 03 00 00 00 00 00 00 00 |MMM.0...........| 00004ce0 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| * hexdiff of SPI-NOR bl2.img @@ -11,15 +11,15 @@ 00000610 4f 00 00 00 01 00 00 00 01 00 05 01 00 0d 20 00 |O............. .| 00000620 70 0a 03 00 70 10 03 00 00 03 00 00 20 00 00 00 |p...p....... ...| 00000630 00 03 00 00 01 00 00 00 4d 4d 4d 01 0c 00 01 00 |........MMM.....| -00000640 01 00 00 00 4d 4d 4d 03 64 00 07 00 90 11 00 00 |....MMM.d.......| -00000650 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| +00000640 01 00 00 00 4d 4d 4d 01 14 00 02 00 00 00 00 00 |....MMM.........| +00000650 10 00 00 00 80 00 00 00 4d 4d 4d 01 14 02 03 00 |........MMM.....| +00000660 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| * -00000690 00 00 00 00 00 00 00 00 00 64 00 00 88 13 00 00 |.........d......| -000006a0 00 00 00 00 00 00 00 00 4d 4d 4d 01 14 02 03 00 |........MMM.....| -000006b0 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| +00000860 00 00 00 00 00 00 00 00 00 00 00 00 4d 4d 4d 03 |............MMM.| +00000870 64 00 07 00 90 11 00 00 00 00 00 00 00 00 00 00 |d...............| +00000880 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| * -000008b0 00 00 00 00 00 00 00 00 00 00 00 00 4d 4d 4d 01 |............MMM.| -000008c0 14 00 02 00 00 00 00 00 10 00 00 00 80 00 00 00 |................| +000008c0 00 64 00 00 88 13 00 00 00 00 00 00 00 00 00 00 |.d..............| 000008d0 4d 4d 4d 01 30 00 08 00 03 00 00 00 00 00 00 00 |MMM.0...........| 000008e0 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| * I'll try to have a look at this more deepply in the next days, but if you already spot the error I'd also be glad ;) Cheers Daniel [1]: https://github.com/dangowrt/arm-trusted-firmware/commit/3b8a17bd847d864379fe9953a82991b522f64add On Fri, Aug 05, 2022 at 04:43:39PM +0800, Weijie Gao wrote: > Hi Daniel, > > It turns out that this was caused by a setting change of company's mail > gateway. > I'll send v2 later. > > Best Regards, > > Weijie > > On Thu, 2022-08-04 at 16:50 +0800, Weijie Gao wrote: > > Hi Daniel, > > > > Thanks for the reminder. > > I found more errornous line-breaks in other patches... > > I'll find a way to fix that. > > > > Best Regards, > > Weijie > > > > On Thu, 2022-08-04 at 10:37 +0200, Daniel Golle wrote: > > > Hi Weijie, > > > > > > happy to see this series posted! > > > Trying to apply it unfortunately fails due to errornous line- > > > breaks, > > > supposedly inserted by your MUA, see below. > > > > > > I didn't go beyond the first patch and it'd be nice if you report > > > the > > > whole series without the wrong line-breaks. > > > > > > Cheers > > > > > > > > > Daniel > > > > > > > > > On Thu, Aug 04, 2022 at 11:34:28AM +0800, Weijie Gao wrote: > > > > This patch adds basic support for MediaTek MT7986 SoC. > > > > This include the file that will initialize the SoC after boot and > > > > its > > > > device tree. > > > > > > > > Signed-off-by: Weijie Gao <weijie....@mediatek.com> > > > > --- > > > > arch/arm/dts/mt7986-u-boot.dtsi | 33 ++ > > > > arch/arm/dts/mt7986.dtsi | 341 > > > > ++++++++++++++++++ > > > > arch/arm/mach-mediatek/Kconfig | 11 + > > > > arch/arm/mach-mediatek/Makefile | 1 + > > > > arch/arm/mach-mediatek/mt7986/Makefile | 4 + > > > > arch/arm/mach-mediatek/mt7986/init.c | 53 +++ > > > > arch/arm/mach-mediatek/mt7986/lowlevel_init.S | 30 ++ > > > > 7 files changed, 473 insertions(+) > > > > create mode 100644 arch/arm/dts/mt7986-u-boot.dtsi > > > > create mode 100644 arch/arm/dts/mt7986.dtsi > > > > create mode 100644 arch/arm/mach-mediatek/mt7986/Makefile > > > > create mode 100644 arch/arm/mach-mediatek/mt7986/init.c > > > > create mode 100644 arch/arm/mach-mediatek/mt7986/lowlevel_init.S > > > > > > > > diff --git a/arch/arm/dts/mt7986-u-boot.dtsi > > > > b/arch/arm/dts/mt7986-u-boot.dtsi > > > > > > The above two lines should be a single line. > > > > > > > new file mode 100644 > > > > index 0000000000..95671f8afa > > > > --- /dev/null > > > > +++ b/arch/arm/dts/mt7986-u-boot.dtsi > > > > @@ -0,0 +1,33 @@ > > > > +// SPDX-License-Identifier: GPL-2.0 > > > > +/* > > > > + * Copyright (c) 2022 MediaTek Inc. > > > > + * Author: Sam Shih <sam.s...@mediatek.com> > > > > + */ > > > > + > > > > +&topckgen { > > > > + u-boot,dm-pre-reloc; > > > > +}; > > > > + > > > > +&pericfg { > > > > + u-boot,dm-pre-reloc; > > > > +}; > > > > + > > > > +&apmixedsys { > > > > + u-boot,dm-pre-reloc; > > > > +}; > > > > + > > > > +&timer0 { > > > > + u-boot,dm-pre-reloc; > > > > +}; > > > > + > > > > +&uart0 { > > > > + u-boot,dm-pre-reloc; > > > > +}; > > > > + > > > > +&snand { > > > > + u-boot,dm-pre-reloc; > > > > +}; > > > > + > > > > +&pinctrl { > > > > + u-boot,dm-pre-reloc; > > > > +}; > > > > diff --git a/arch/arm/dts/mt7986.dtsi b/arch/arm/dts/mt7986.dtsi > > > > new file mode 100644 > > > > index 0000000000..f235bd8b8c > > > > --- /dev/null > > > > +++ b/arch/arm/dts/mt7986.dtsi > > > > @@ -0,0 +1,341 @@ > > > > +// SPDX-License-Identifier: GPL-2.0 > > > > +/* > > > > + * Copyright (c) 2022 MediaTek Inc. > > > > + * Author: Sam Shih <sam.s...@mediatek.com> > > > > + */ > > > > + > > > > +#include <dt-bindings/interrupt-controller/irq.h> > > > > +#include <dt-bindings/interrupt-controller/arm-gic.h> > > > > +#include <dt-bindings/phy/phy.h> > > > > +#include <dt-bindings/clock/mt7986-clk.h> > > > > +#include <dt-bindings/reset/mt7629-reset.h> > > > > +#include <dt-bindings/pinctrl/mt65xx.h> > > > > + > > > > +/ { > > > > + compatible = "mediatek,mt7986"; > > > > + interrupt-parent = <&gic>; > > > > + #address-cells = <1>; > > > > + #size-cells = <1>; > > > > + > > > > + config { > > > > + u-boot,mmc-env-partition = "u-boot-env"; > > > > + }; > > > > + > > > > + cpus { > > > > + #address-cells = <1>; > > > > + #size-cells = <0>; > > > > + cpu0: cpu@0 { > > > > + device_type = "cpu"; > > > > + compatible = "arm,cortex-a53"; > > > > + reg = <0x0>; > > > > + }; > > > > + cpu1: cpu@1 { > > > > + device_type = "cpu"; > > > > + compatible = "arm,cortex-a53"; > > > > + reg = <0x1>; > > > > + }; > > > > + cpu2: cpu@2 { > > > > + device_type = "cpu"; > > > > + compatible = "arm,cortex-a53"; > > > > + reg = <0x1>; > > > > + }; > > > > + cpu3: cpu@3 { > > > > + device_type = "cpu"; > > > > + compatible = "arm,cortex-a53"; > > > > + reg = <0x1>; > > > > + }; > > > > + }; > > > > + > > > > + dummy_clk: dummy12m { > > > > + compatible = "fixed-clock"; > > > > + clock-frequency = <12000000>; > > > > + #clock-cells = <0>; > > > > + /* must need this line, or uart uanable to get > > > > dummy_clk */ > > > > + u-boot,dm-pre-reloc; > > > > + }; > > > > + > > > > + timer { > > > > + compatible = "arm,armv8-timer"; > > > > + interrupt-parent = <&gic>; > > > > + clock-frequency = <13000000>; > > > > + interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_LOW>, > > > > + <GIC_PPI 14 IRQ_TYPE_LEVEL_LOW>, > > > > + <GIC_PPI 11 IRQ_TYPE_LEVEL_LOW>, > > > > + <GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>; > > > > + arm,cpu-registers-not-fw-configured; > > > > + }; > > > > + > > > > + timer0: timer@10008000 { > > > > + compatible = "mediatek,mt7986-timer"; > > > > + reg = <0x10008000 0x1000>; > > > > + interrupts = <GIC_SPI 130 IRQ_TYPE_LEVEL_HIGH>; > > > > + clocks = <&infracfg CK_INFRA_CK_F26M>; > > > > + clock-names = "gpt-clk"; > > > > + u-boot,dm-pre-reloc; > > > > + }; > > > > + > > > > + watchdog: watchdog@1001c000 { > > > > + compatible = "mediatek,mt7986-wdt"; > > > > + reg = <0x1001c000 0x1000>; > > > > + interrupts = <GIC_SPI 110 IRQ_TYPE_LEVEL_HIGH>; > > > > + #reset-cells = <1>; > > > > + status = "disabled"; > > > > + }; > > > > + > > > > + gic: interrupt-controller@c000000 { > > > > + compatible = "arm,gic-v3"; > > > > + #interrupt-cells = <3>; > > > > + interrupt-parent = <&gic>; > > > > + interrupt-controller; > > > > + reg = <0x0c000000 0x40000>, /* GICD */ > > > > + <0x0c080000 0x200000>; /* GICR */ > > > > + > > > > + interrupts = <GIC_PPI 9 IRQ_TYPE_LEVEL_HIGH>; > > > > + }; > > > > + > > > > + fixed_plls: apmixedsys@1001E000 { > > > > + compatible = "mediatek,mt7986-fixed-plls"; > > > > + reg = <0x1001E000 0x1000>; > > > > + #clock-cells = <1>; > > > > + }; > > > > + > > > > + topckgen: topckgen@1001B000 { > > > > + compatible = "mediatek,mt7986-topckgen"; > > > > + reg = <0x1001B000 0x1000>; > > > > + clock-parent = <&fixed_plls>; > > > > + #clock-cells = <1>; > > > > + }; > > > > + > > > > + infracfg_ao: infracfg_ao@10001000 { > > > > + compatible = "mediatek,mt7986-infracfg_ao"; > > > > + reg = <0x10001000 0x68>; > > > > + clock-parent = <&infracfg>; > > > > + #clock-cells = <1>; > > > > + }; > > > > + > > > > + infracfg: infracfg@10001040 { > > > > + compatible = "mediatek,mt7986-infracfg"; > > > > + reg = <0x10001000 0x1000>; > > > > + clock-parent = <&topckgen>; > > > > + #clock-cells = <1>; > > > > + }; > > > > + > > > > + pinctrl: pinctrl@1001f000 { > > > > + compatible = "mediatek,mt7986-pinctrl"; > > > > + reg = <0x1001f000 0x1000>, > > > > + <0x11c30000 0x1000>, > > > > + <0x11c40000 0x1000>, > > > > + <0x11e20000 0x1000>, > > > > + <0x11e30000 0x1000>, > > > > + <0x11f00000 0x1000>, > > > > + <0x11f10000 0x1000>, > > > > + <0x1000b000 0x1000>; > > > > + reg-names = "gpio_base", "iocfg_rt_base", > > > > "iocfg_rb_base", > > > > + "iocfg_lt_base", "iocfg_lb_base", > > > > "iocfg_tr_base", > > > > + "iocfg_tl_base", "eint"; > > > > + gpio: gpio-controller { > > > > + gpio-controller; > > > > + #gpio-cells = <2>; > > > > + }; > > > > + }; > > > > + > > > > + pwm: pwm@10048000 { > > > > + compatible = "mediatek,mt7986-pwm"; > > > > + reg = <0x10048000 0x1000>; > > > > + #clock-cells = <1>; > > > > + #pwm-cells = <2>; > > > > + interrupts = <GIC_SPI 137 IRQ_TYPE_LEVEL_HIGH>; > > > > + clocks = <&infracfg CK_INFRA_PWM>, > > > > + <&infracfg_ao CK_INFRA_PWM_BSEL>, > > > > + <&infracfg_ao CK_INFRA_PWM1_CK>, > > > > + <&infracfg_ao CK_INFRA_PWM2_CK>; > > > > + assigned-clocks = <&topckgen CK_TOP_PWM_SEL>, > > > > + <&infracfg CK_INFRA_PWM_BSEL>, > > > > + <&infracfg CK_INFRA_PWM1_SEL>, > > > > + <&infracfg CK_INFRA_PWM2_SEL>; > > > > + assigned-clock-parents = <&topckgen > > > > CK_TOP_CB_M_D4>, > > > > + <&infracfg > > > > CK_INFRA_PWM>, > > > > + <&infracfg > > > > CK_INFRA_PWM>, > > > > + <&infracfg > > > > CK_INFRA_PWM>; > > > > + clock-names = "top", "main", "pwm1", "pwm2"; > > > > + status = "disabled"; > > > > + u-boot,dm-pre-reloc; > > > > + }; > > > > + > > > > + uart0: serial@11002000 { > > > > + compatible = "mediatek,hsuart"; > > > > + reg = <0x11002000 0x400>; > > > > + interrupts = <GIC_SPI 123 IRQ_TYPE_LEVEL_HIGH>; > > > > + clocks = <&infracfg_ao CK_INFRA_UART0_CK>; > > > > + assigned-clocks = <&topckgen CK_TOP_UART_SEL>, > > > > + <&infracfg_ao > > > > CK_INFRA_UART0_SEL>; > > > > + assigned-clock-parents = <&topckgen > > > > CK_TOP_CB_CKSQ_40M>, > > > > + <&infracfg > > > > CK_INFRA_UART>; > > > > + mediatek,force-highspeed; > > > > + status = "disabled"; > > > > + u-boot,dm-pre-reloc; > > > > + }; > > > > + > > > > + uart1: serial@11003000 { > > > > + compatible = "mediatek,hsuart"; > > > > + reg = <0x11003000 0x400>; > > > > + interrupts = <GIC_SPI 124 IRQ_TYPE_LEVEL_HIGH>; > > > > + clocks = <&infracfg_ao CK_INFRA_UART1_CK>; > > > > + assigned-clocks = <&infracfg > > > > CK_INFRA_UART1_SEL>; > > > > + assigned-clock-parents = <&infracfg > > > > CK_INFRA_CK_F26M>; > > > > + mediatek,force-highspeed; > > > > + status = "disabled"; > > > > + }; > > > > + > > > > + uart2: serial@11004000 { > > > > + compatible = "mediatek,hsuart"; > > > > + reg = <0x11004000 0x400>; > > > > + interrupts = <GIC_SPI 124 IRQ_TYPE_LEVEL_HIGH>; > > > > + clocks = <&infracfg_ao CK_INFRA_UART2_CK>; > > > > + assigned-clocks = <&infracfg > > > > CK_INFRA_UART2_SEL>; > > > > + assigned-clock-parents = <&infracfg > > > > CK_INFRA_CK_F26M>; > > > > + mediatek,force-highspeed; > > > > + status = "disabled"; > > > > + }; > > > > + > > > > + snand: snand@11005000 { > > > > + compatible = "mediatek,mt7986-snand"; > > > > + reg = <0x11005000 0x1000>, > > > > + <0x11006000 0x1000>; > > > > + reg-names = "nfi", "ecc"; > > > > + clocks = <&infracfg_ao CK_INFRA_SPINFI1_CK>, > > > > + <&infracfg_ao CK_INFRA_NFI1_CK>, > > > > + <&infracfg_ao CK_INFRA_NFI_HCK_CK>; > > > > + clock-names = "pad_clk", "nfi_clk", "nfi_hclk"; > > > > + assigned-clocks = <&topckgen CK_TOP_SPINFI_SEL>, > > > > + <&topckgen CK_TOP_NFI1X_SEL>; > > > > + assigned-clock-parents = <&topckgen > > > > CK_TOP_CB_M_D8>, > > > > + <&topckgen > > > > CK_TOP_CB_M_D8>; > > > > + status = "disabled"; > > > > + }; > > > > + > > > > + ethsys: syscon@15000000 { > > > > + compatible = "mediatek,mt7986-ethsys", "syscon"; > > > > + reg = <0x15000000 0x1000>; > > > > + clock-parent = <&topckgen>; > > > > + #clock-cells = <1>; > > > > + #reset-cells = <1>; > > > > + }; > > > > + > > > > + eth: ethernet@15100000 { > > > > + compatible = "mediatek,mt7986-eth", "syscon"; > > > > + reg = <0x15100000 0x20000>; > > > > + resets = <ðsys ETHSYS_FE_RST>; > > > > + reset-names = "fe"; > > > > + mediatek,ethsys = <ðsys>; > > > > + mediatek,sgmiisys = <&sgmiisys0>; > > > > + #address-cells = <1>; > > > > + #size-cells = <0>; > > > > + status = "disabled"; > > > > + }; > > > > + > > > > + sgmiisys0: syscon@10060000 { > > > > + compatible = "mediatek,mt7986-sgmiisys", > > > > "syscon"; > > > > + reg = <0x10060000 0x1000>; > > > > + #clock-cells = <1>; > > > > + }; > > > > + > > > > + sgmiisys1: syscon@10070000 { > > > > + compatible = "mediatek,mt7986-sgmiisys", > > > > "syscon"; > > > > + reg = <0x10070000 0x1000>; > > > > + #clock-cells = <1>; > > > > + }; > > > > + > > > > + spi0: spi@1100a000 { > > > > + compatible = "mediatek,ipm-spi"; > > > > + reg = <0x1100a000 0x100>; > > > > + clocks = <&infracfg_ao CK_INFRA_SPI0_CK>, > > > > + <&topckgen CK_TOP_SPI_SEL>; > > > > + assigned-clocks = <&topckgen CK_TOP_SPI_SEL>, > > > > + <&infracfg CK_INFRA_SPI0_SEL>; > > > > + assigned-clock-parents = <&topckgen > > > > CK_TOP_CB_M_D2>, > > > > + <&topckgen > > > > CK_INFRA_ISPI0>; > > > > + clock-names = "sel-clk", "spi-clk"; > > > > + interrupts = <GIC_SPI 140 IRQ_TYPE_LEVEL_HIGH>; > > > > + status = "disabled"; > > > > + }; > > > > + > > > > + spi1: spi@1100b000 { > > > > + compatible = "mediatek,ipm-spi"; > > > > + reg = <0x1100b000 0x100>; > > > > + interrupts = <GIC_SPI 141 IRQ_TYPE_LEVEL_HIGH>; > > > > + status = "disabled"; > > > > + }; > > > > + > > > > + mmc0: mmc@11230000 { > > > > + compatible = "mediatek,mt7986-mmc"; > > > > + reg = <0x11230000 0x1000>, > > > > + <0x11C20000 0x1000>; > > > > + interrupts = <GIC_SPI 143 IRQ_TYPE_LEVEL_HIGH>; > > > > + clocks = <&topckgen CK_TOP_EMMC_416M>, > > > > + <&topckgen CK_TOP_EMMC_250M>, > > > > + <&infracfg_ao CK_INFRA_MSDC_CK>; > > > > + assigned-clocks = <&topckgen > > > > CK_TOP_EMMC_416M_SEL>, > > > > + <&topckgen > > > > CK_TOP_EMMC_250M_SEL>; > > > > + assigned-clock-parents = <&topckgen > > > > CK_TOP_CB_M_416M>, > > > > + <&topckgen > > > > CK_TOP_NET1_D5_D2>; > > > > + clock-names = "source", "hclk", "source_cg"; > > > > + status = "disabled"; > > > > + }; > > > > + > > > > + xhci: xhci@11200000 { > > > > + compatible = "mediatek,mt7986-xhci", > > > > + "mediatek,mtk-xhci"; > > > > + reg = <0x11200000 0x2e00>, > > > > + <0x11203e00 0x0100>; > > > > + reg-names = "mac", "ippc"; > > > > + interrupts = <GIC_SPI 173 IRQ_TYPE_LEVEL_HIGH>; > > > > + phys = <&u2port0 PHY_TYPE_USB2>, > > > > + <&u3port0 PHY_TYPE_USB3>, > > > > + <&u2port1 PHY_TYPE_USB2>; > > > > + clocks = <&dummy_clk>, > > > > + <&dummy_clk>, > > > > + <&dummy_clk>, > > > > + <&dummy_clk>, > > > > + <&dummy_clk>; > > > > + clock-names = "sys_ck", > > > > + "xhci_ck", > > > > + "ref_ck", > > > > + "mcu_ck", > > > > + "dma_ck"; > > > > + tpl-support; > > > > + status = "okay"; > > > > + }; > > > > + > > > > + usbtphy: usb-phy@11e10000 { > > > > + compatible = "mediatek,mt7986", > > > > + "mediatek,generic-tphy-v2"; > > > > + #address-cells = <1>; > > > > + #size-cells = <1>; > > > > + status = "okay"; > > > > + > > > > + u2port0: usb-phy@11e10000 { > > > > + reg = <0x11e10000 0x700>; > > > > + clocks = <&dummy_clk>; > > > > + clock-names = "ref"; > > > > + #phy-cells = <1>; > > > > + status = "okay"; > > > > + }; > > > > + > > > > + u3port0: usb-phy@11e10700 { > > > > + reg = <0x11e10700 0x900>; > > > > + clocks = <&dummy_clk>; > > > > + clock-names = "ref"; > > > > + #phy-cells = <1>; > > > > + status = "okay"; > > > > + }; > > > > + > > > > + u2port1: usb-phy@11e11000 { > > > > + reg = <0x11e11000 0x700>; > > > > + clocks = <&dummy_clk>; > > > > + clock-names = "ref"; > > > > + #phy-cells = <1>; > > > > + status = "okay"; > > > > + }; > > > > + }; > > > > +}; > > > > diff --git a/arch/arm/mach-mediatek/Kconfig b/arch/arm/mach- > > > > mediatek/Kconfig > > > > index f79a5c62cd..e059a013db 100644 > > > > --- a/arch/arm/mach-mediatek/Kconfig > > > > +++ b/arch/arm/mach-mediatek/Kconfig > > > > @@ -40,6 +40,14 @@ config TARGET_MT7629 > > > > including DDR3, crypto engine, 3x3 11n/ac Wi-Fi, > > > > Gigabit > > > > Ethernet, > > > > switch, USB3.0, PCIe, UART, SPI, I2C and PWM. > > > > > > > > +config TARGET_MT7986 > > > > + bool "MediaTek MT7986 SoC" > > > > + select ARM64 > > > > + help > > > > + The MediaTek MT7986 is a ARM64-based SoC with a quad- > > > > core Cortex-A53. > > > > + including UART, SPI, SPI flash, USB3.0, MMC, NAND, > > > > SNFI, > > > > PWM, PCIe, > > > > + Gigabit Ethernet, I2C, built-in 4x4 Wi-Fi, and PCIe. > > > > + > > > > config TARGET_MT8183 > > > > bool "MediaTek MT8183 SoC" > > > > select ARM64 > > > > @@ -84,6 +92,7 @@ config SYS_BOARD > > > > default "mt7622" if TARGET_MT7622 > > > > default "mt7623" if TARGET_MT7623 > > > > default "mt7629" if TARGET_MT7629 > > > > + default "mt7986" if TARGET_MT7986 > > > > default "mt8183" if TARGET_MT8183 > > > > default "mt8512" if TARGET_MT8512 > > > > default "mt8516" if TARGET_MT8516 > > > > @@ -99,6 +108,7 @@ config SYS_CONFIG_NAME > > > > default "mt7622" if TARGET_MT7622 > > > > default "mt7623" if TARGET_MT7623 > > > > default "mt7629" if TARGET_MT7629 > > > > + default "mt7986" if TARGET_MT7986 > > > > default "mt8183" if TARGET_MT8183 > > > > default "mt8512" if TARGET_MT8512 > > > > default "mt8516" if TARGET_MT8516 > > > > @@ -113,6 +123,7 @@ config MTK_BROM_HEADER_INFO > > > > string > > > > default "media=nor" if TARGET_MT8518 || TARGET_MT8512 || > > > > TARGET_MT7629 || > > > > TARGET_MT7622 > > > > > > Same here. > > > > > > > default "media=emmc" if TARGET_MT8516 || TARGET_MT8365 > > > > || > > > > TARGET_MT8183 > > > > + default "media=snand;nandinfo=2k+64" if TARGET_MT7986 > > > > default "lk=1" if TARGET_MT7623 > > > > > > > > endif > > > > diff --git a/arch/arm/mach-mediatek/Makefile > > > > b/arch/arm/mach-mediatek/Makefile > > > > > > And here. > > > > > > > index 0f5b0c16d2..fe5c3a837c 100644 > > > > --- a/arch/arm/mach-mediatek/Makefile > > > > +++ b/arch/arm/mach-mediatek/Makefile > > > > @@ -7,6 +7,7 @@ obj-$(CONFIG_MT8512) += mt8512/ > > > > obj-$(CONFIG_TARGET_MT7622) += mt7622/ > > > > obj-$(CONFIG_TARGET_MT7623) += mt7623/ > > > > obj-$(CONFIG_TARGET_MT7629) += mt7629/ > > > > +obj-$(CONFIG_TARGET_MT7986) += mt7986/ > > > > obj-$(CONFIG_TARGET_MT8183) += mt8183/ > > > > obj-$(CONFIG_TARGET_MT8516) += mt8516/ > > > > obj-$(CONFIG_TARGET_MT8518) += mt8518/ > > > > diff --git a/arch/arm/mach-mediatek/mt7986/Makefile > > > > b/arch/arm/mach-mediatek/mt7986/Makefile > > > > > > And here. > > > > > > > new file mode 100644 > > > > index 0000000000..007eb4a367 > > > > --- /dev/null > > > > +++ b/arch/arm/mach-mediatek/mt7986/Makefile > > > > @@ -0,0 +1,4 @@ > > > > +# SPDX-License-Identifier: GPL-2.0 > > > > + > > > > +obj-y += init.o > > > > +obj-y += lowlevel_init.o > > > > diff --git a/arch/arm/mach-mediatek/mt7986/init.c > > > > b/arch/arm/mach-mediatek/mt7986/init.c > > > > > > And here again. > > > > > > > new file mode 100644 > > > > index 0000000000..4884cbdc67 > > > > --- /dev/null > > > > +++ b/arch/arm/mach-mediatek/mt7986/init.c > > > > @@ -0,0 +1,53 @@ > > > > +// SPDX-License-Identifier: GPL-2.0 > > > > +/* > > > > + * Copyright (C) 2022 MediaTek Inc. > > > > + * Author: Sam Shih <sam.s...@mediatek.com> > > > > + */ > > > > + > > > > +#include <fdtdec.h> > > > > +#include <asm/armv8/mmu.h> > > > > +#include <init.h> > > > > +#include <asm/system.h> > > > > +#include <asm/global_data.h> > > > > +#include <linux/sizes.h> > > > > + > > > > +DECLARE_GLOBAL_DATA_PTR; > > > > + > > > > +int print_cpuinfo(void) > > > > +{ > > > > + printf("CPU: MediaTek MT7986\n"); > > > > + return 0; > > > > +} > > > > + > > > > +int dram_init(void) > > > > +{ > > > > + gd->ram_size = get_ram_size((void > > > > *)CONFIG_SYS_SDRAM_BASE, > > > > SZ_2G); > > > > + > > > > + return 0; > > > > +} > > > > + > > > > +void reset_cpu(ulong addr) > > > > +{ > > > > + psci_system_reset(); > > > > +} > > > > + > > > > +static struct mm_region mt7986_mem_map[] = { > > > > + { > > > > + /* DDR */ > > > > + .virt = 0x40000000UL, > > > > + .phys = 0x40000000UL, > > > > + .size = 0x80000000UL, > > > > + .attrs = PTE_BLOCK_MEMTYPE(MT_NORMAL) | > > > > PTE_BLOCK_OUTER_SHARE, > > > > + }, { > > > > + .virt = 0x00000000UL, > > > > + .phys = 0x00000000UL, > > > > + .size = 0x40000000UL, > > > > + .attrs = PTE_BLOCK_MEMTYPE(MT_DEVICE_NGNRNE) | > > > > + PTE_BLOCK_NON_SHARE | > > > > + PTE_BLOCK_PXN | PTE_BLOCK_UXN > > > > + }, { > > > > + 0, > > > > + } > > > > +}; > > > > + > > > > +struct mm_region *mem_map = mt7986_mem_map; > > > > diff --git a/arch/arm/mach-mediatek/mt7986/lowlevel_init.S > > > > b/arch/arm/mach-mediatek/mt7986/lowlevel_init.S > > > > > > And here. > > > > > > > new file mode 100644 > > > > index 0000000000..244d2c1385 > > > > --- /dev/null > > > > +++ b/arch/arm/mach-mediatek/mt7986/lowlevel_init.S > > > > @@ -0,0 +1,30 @@ > > > > +/* SPDX-License-Identifier: GPL-2.0 */ > > > > +/* > > > > + * Copyright (C) 2022 MediaTek Inc. > > > > + * Author: Sam Shih <sam.s...@mediatek.com> > > > > + */ > > > > + > > > > +/* > > > > + * Switch from AArch64 EL2 to AArch32 EL2 > > > > + * @param inputs: > > > > + * x0: argument, zero > > > > + * x1: machine nr > > > > + * x2: fdt address > > > > + * x3: input argument > > > > + * x4: kernel entry point > > > > + * @param outputs for secure firmware: > > > > + * x0: function id > > > > + * x1: kernel entry point > > > > + * x2: machine nr > > > > + * x3: fdt address > > > > +*/ > > > > + > > > > +.global armv8_el2_to_aarch32 > > > > +armv8_el2_to_aarch32: > > > > + mov x3, x2 > > > > + mov x2, x1 > > > > + mov x1, x4 > > > > + mov x4, #0 > > > > + ldr x0, =0x82000200 > > > > + SMC #0 > > > > + ret > > > > -- > > > > 2.17.1 > > > >