[PATCH v5 1/2] arm: mediatek: add mt8195 SOC support

Simon Glass sjg at chromium.org
Sat Nov 18 18:10:16 CET 2023


Hi,

On Fri, 4 Aug 2023 at 05:05, Macpaul Lin <macpaul.lin at mediatek.com> wrote:
>
> From: Fabien Parent <fparent at baylibre.com>
>
> The MediaTek MT8195 is a ARM64-based SoC with a quad-core Cortex-A73 and
> a quad-core Cortex-A53. It is including UART, SPI, USB3.0 device and hosts,
> SD and MMC cards, UFS, PWM, I2C, I2S, S/PDIF, and several LPDDR3
> and LPDDR4 options.
>
> Signed-off-by: Fabien Parent <fparent at baylibre.com>
> Signed-off-by: Macpaul Lin <macpaul.lin at mediatek.com>
> ---
>  MAINTAINERS                            |   2 +
>  arch/arm/dts/mt8195.dtsi               | 370 +++++++++++++++++++++++++
>  arch/arm/mach-mediatek/Kconfig         |  13 +-
>  arch/arm/mach-mediatek/Makefile        |   1 +
>  arch/arm/mach-mediatek/mt8195/Kconfig  |  13 +
>  arch/arm/mach-mediatek/mt8195/Makefile |   3 +
>  arch/arm/mach-mediatek/mt8195/init.c   |  97 +++++++
>  7 files changed, 498 insertions(+), 1 deletion(-)
>  create mode 100644 arch/arm/dts/mt8195.dtsi
>  create mode 100644 arch/arm/mach-mediatek/mt8195/Kconfig
>  create mode 100644 arch/arm/mach-mediatek/mt8195/Makefile
>  create mode 100644 arch/arm/mach-mediatek/mt8195/init.c

Reviewed-by: Simon Glass <sjg at chromium.org>

nits below

>
> Changes for v2:
>  - Correct node name to t-phy for u3phy0.
>  - Add platform compatible string "mediatek,mt8195-tphy" to all usb phy nodes.
>  - remove clock nodes that software cannot controlled in phy nodes.
>  - Test and add back "mac" for HOST only xhci nodes.
>
> Changes for v3:
>  - Revise device node name from "xhciX: xhciX@" to "xhciX: xhci@".
>
> Changes for v4:
>  - No change.
>
> Changes for v5:
>  - Fix Copyright year to 2023.
>  - Fix memory map in dram_init() to support 8GB onboard memory.
>  - Add '#if !IS_ENABLED(CONFIG_SYSRESET)' with reset_cpu().
>  - Correct reset_cpu() function prototype.
>  - rebase patchset to v2023-10.rc1
>  - Add missing arch/arm/mach-mediatek/mt8195/Kconfig.
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 47581cf6fb..4d0f017e7e 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -369,8 +369,10 @@ ARM MEDIATEK
>  M:     Ryder Lee <ryder.lee at mediatek.com>
>  M:     Weijie Gao <weijie.gao at mediatek.com>
>  M:     Chunfeng Yun <chunfeng.yun at mediatek.com>
> +M:     Macpaul Lin <macpaul.lin at mediatek.com>
>  R:     GSS_MTK_Uboot_upstream <GSS_MTK_Uboot_upstream at mediatek.com>
>  S:     Maintained
> +F:     arch/arm/dts/mt8195.dtsi
>  F:     arch/arm/mach-mediatek/
>  F:     arch/arm/include/asm/arch-mediatek/
>  F:     board/mediatek/


> diff --git a/arch/arm/mach-mediatek/Kconfig b/arch/arm/mach-mediatek/Kconfig
> index 04aa2fd97f..3a2af1cdee 100644
> --- a/arch/arm/mach-mediatek/Kconfig
> +++ b/arch/arm/mach-mediatek/Kconfig
> @@ -67,6 +67,15 @@ config TARGET_MT8183
>           SD and MMC cards, UFS, PWM, I2C, I2S, S/PDIF, and several LPDDR3
>           and LPDDR4 options.
>
> +config TARGET_MT8195
> +       bool "MediaTek MT8195 SoC"
> +       select ARM64
> +       help
> +         The MediaTek MT8195 is a ARM64-based SoC with a quad-core Cortex-A73 and
> +         a quad-core Cortex-A53. It is including UART, SPI, USB3.0 device and hosts,
> +         SD and MMC cards, UFS, PWM, I2C, I2S, S/PDIF, and several LPDDR3
> +         and LPDDR4 options.
> +
>  config TARGET_MT8512
>          bool "MediaTek MT8512 M1 Board"
>          select ARM64
> @@ -105,6 +114,7 @@ config SYS_BOARD
>         default "mt7981" if TARGET_MT7981
>         default "mt7986" if TARGET_MT7986
>         default "mt8183" if TARGET_MT8183
> +       default "mt8195" if TARGET_MT8195
>         default "mt8512" if TARGET_MT8512
>         default "mt8516" if TARGET_MT8516
>         default "mt8518" if TARGET_MT8518
> @@ -122,6 +132,7 @@ config SYS_CONFIG_NAME
>         default "mt7981" if TARGET_MT7981
>         default "mt7986" if TARGET_MT7986
>         default "mt8183" if TARGET_MT8183
> +       default "mt8195" if TARGET_MT8195
>         default "mt8512" if TARGET_MT8512
>         default "mt8516" if TARGET_MT8516
>         default "mt8518" if TARGET_MT8518
> @@ -134,7 +145,7 @@ config SYS_CONFIG_NAME
>  config MTK_BROM_HEADER_INFO
>         string
>         default "media=nor" if TARGET_MT8518 || TARGET_MT8512 || TARGET_MT7629 || TARGET_MT7622
> -       default "media=emmc" if TARGET_MT8516 || TARGET_MT8365 || TARGET_MT8183
> +       default "media=emmc" if TARGET_MT8516 || TARGET_MT8365 || TARGET_MT8183 || TARGET_MT8195
>         default "media=snand;nandinfo=2k+64" if TARGET_MT7981 || TARGET_MT7986
>         default "lk=1" if TARGET_MT7623
>
> diff --git a/arch/arm/mach-mediatek/Makefile b/arch/arm/mach-mediatek/Makefile
> index fc85293f71..fbbb5431d1 100644
> --- a/arch/arm/mach-mediatek/Makefile
> +++ b/arch/arm/mach-mediatek/Makefile
> @@ -10,5 +10,6 @@ obj-$(CONFIG_TARGET_MT7629) += mt7629/
>  obj-$(CONFIG_TARGET_MT7981) += mt7981/
>  obj-$(CONFIG_TARGET_MT7986) += mt7986/
>  obj-$(CONFIG_TARGET_MT8183) += mt8183/
> +obj-$(CONFIG_TARGET_MT8195) += mt8195/
>  obj-$(CONFIG_TARGET_MT8516) += mt8516/
>  obj-$(CONFIG_TARGET_MT8518) += mt8518/
> diff --git a/arch/arm/mach-mediatek/mt8195/Kconfig b/arch/arm/mach-mediatek/mt8195/Kconfig
> new file mode 100644
> index 0000000000..a34fa2cf2e
> --- /dev/null
> +++ b/arch/arm/mach-mediatek/mt8195/Kconfig
> @@ -0,0 +1,13 @@
> +if TARGET_MT8195
> +
> +config SYS_BOARD
> +       default "mt8195"
> +
> +config SYS_CONFIG_NAME
> +       default "mt8195"
> +
> +config MTK_BROM_HEADER_INFO
> +       string
> +       default "media=emmc"
> +
> +endif
> diff --git a/arch/arm/mach-mediatek/mt8195/Makefile b/arch/arm/mach-mediatek/mt8195/Makefile
> new file mode 100644
> index 0000000000..886ab7e4eb
> --- /dev/null
> +++ b/arch/arm/mach-mediatek/mt8195/Makefile
> @@ -0,0 +1,3 @@
> +# SPDX-License-Identifier:     GPL-2.0
> +
> +obj-y += init.o
> diff --git a/arch/arm/mach-mediatek/mt8195/init.c b/arch/arm/mach-mediatek/mt8195/init.c
> new file mode 100644
> index 0000000000..8d6b700e7e
> --- /dev/null
> +++ b/arch/arm/mach-mediatek/mt8195/init.c
> @@ -0,0 +1,97 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Copyright (C) 2023 MediaTek Inc.
> + * Copyright (C) 2023 BayLibre, SAS
> + * Author: Macpaul Lin <macpaul.lin at mediatek.com>
> + * Author: Fabien Parent <fparent at baylibre.com>
> + */
> +
> +#include <clk.h>
> +#include <common.h>
> +#include <cpu_func.h>
> +#include <dm.h>
> +#include <fdtdec.h>
> +#include <ram.h>
> +#include <asm/arch/misc.h>
> +#include <asm/armv8/mmu.h>
> +#include <asm/sections.h>
> +#include <asm/system.h>
> +#include <dm/uclass.h>
> +
> +DECLARE_GLOBAL_DATA_PTR;
> +
> +int dram_init(void)
> +{
> +       int ret;
> +
> +       ret = fdtdec_setup_memory_banksize();
> +       if (ret)
> +               return ret;
> +
> +       fdtdec_setup_mem_size_base();
> +
> +       /*
> +        * Limit gd->ram_top not exceeding SZ_4G.
> +        * Because some periphals like mmc requires DMA buffer

spelling

> +        * allocaed below SZ_4G.

spelling

Also please don't put punctuation after SZ_4G as it makes it harder
for people to select it and search


> +        *
> +        * Note: SZ_2M is for adjusting gd->relocaddr,
> +        *       the reserved memory for u-boot itself.
> +        */
> +       if (gd->ram_base + gd->ram_size >= SZ_4G)
> +               gd->mon_len = SZ_4G + gd->ram_base + SZ_2M;
> +
> +       return 0;
> +}
> +
> +int dram_init_banksize(void)
> +{
> +       gd->bd->bi_dram[0].start = gd->ram_base;
> +       gd->bd->bi_dram[0].size = gd->ram_size;
> +
> +       return 0;
> +}
> +
> +int mtk_pll_early_init(void)
> +{
> +       return 0;
> +}
> +
> +int mtk_soc_early_init(void)
> +{
> +       return 0;
> +}
> +
> +#if !IS_ENABLED(CONFIG_SYSRESET)
> +void reset_cpu(void)
> +{
> +       psci_system_reset();
> +}
> +#endif
> +
> +int print_cpuinfo(void)
> +{
> +       printf("CPU:   MediaTek MT8195\n");
> +       return 0;

You should use a sysinfo driver to provide this information

> +}
> +
> +static struct mm_region mt8195_mem_map[] = {
> +       {
> +               /* DDR */
> +               .virt = 0x40000000UL,
> +               .phys = 0x40000000UL,
> +               .size = 0x200000000UL,
> +               .attrs = PTE_BLOCK_MEMTYPE(MT_NORMAL) | PTE_BLOCK_OUTER_SHARE,
> +       }, {
> +               .virt = 0x00000000UL,
> +               .phys = 0x00000000UL,
> +               .size = 0x20000000UL,
> +               .attrs = PTE_BLOCK_MEMTYPE(MT_DEVICE_NGNRNE) |
> +                        PTE_BLOCK_NON_SHARE |
> +                        PTE_BLOCK_PXN | PTE_BLOCK_UXN
> +       }, {
> +               0,
> +       }
> +};
> +
> +struct mm_region *mem_map = mt8195_mem_map;
> --
> 2.18.0
>

Regards,
Simon


More information about the U-Boot mailing list