On 5/12/25 19:21, Marek Vasut wrote: > Add hardware initialization for the STM32MP13xx in SPL. This is > similar to STM32MP15xx except the code has to enable MCE to bring > DRAM controller up later. > > Signed-off-by: Marek Vasut <[email protected]> Reviewed-by: Patrice Chotard <[email protected]> Thanks Patrice > --- > Cc: Cheick Traore <[email protected]> > Cc: Fabrice Gasnier <[email protected]> > Cc: Gatien Chevallier <[email protected]> > Cc: Lionel Debieve <[email protected]> > Cc: Pascal Zimmermann <[email protected]> > Cc: Patrice Chotard <[email protected]> > Cc: Patrick Delaunay <[email protected]> > Cc: Simon Glass <[email protected]> > Cc: Sughosh Ganu <[email protected]> > Cc: Tom Rini <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > --- > arch/arm/mach-stm32mp/include/mach/stm32.h | 2 + > arch/arm/mach-stm32mp/stm32mp1/stm32mp13x.c | 204 ++++++++++++++++++++ > 2 files changed, 206 insertions(+) > > diff --git a/arch/arm/mach-stm32mp/include/mach/stm32.h > b/arch/arm/mach-stm32mp/include/mach/stm32.h > index dfba57e7dc4..37f3e8595b2 100644 > --- a/arch/arm/mach-stm32mp/include/mach/stm32.h > +++ b/arch/arm/mach-stm32mp/include/mach/stm32.h > @@ -156,6 +156,8 @@ enum forced_boot_mode { > #endif > > #ifdef CONFIG_STM32MP13X > +#define TAMP_BACKUP_MAGIC_NUMBER TAMP_BACKUP_REGISTER(4) > +#define TAMP_BACKUP_BRANCH_ADDRESS TAMP_BACKUP_REGISTER(5) > #define TAMP_BOOTCOUNT TAMP_BACKUP_REGISTER(31) > #define TAMP_BOOT_CONTEXT TAMP_BACKUP_REGISTER(30) > #endif > diff --git a/arch/arm/mach-stm32mp/stm32mp1/stm32mp13x.c > b/arch/arm/mach-stm32mp/stm32mp1/stm32mp13x.c > index 4a811065fc3..79b2f2d0bba 100644 > --- a/arch/arm/mach-stm32mp/stm32mp1/stm32mp13x.c > +++ b/arch/arm/mach-stm32mp/stm32mp1/stm32mp13x.c > @@ -6,11 +6,59 @@ > #define LOG_CATEGORY LOGC_ARCH > > #include <config.h> > +#include <cpu_func.h> > #include <log.h> > #include <syscon.h> > #include <asm/io.h> > +#include <asm/system.h> > +#include <asm/arch/bsec.h> > #include <asm/arch/stm32.h> > #include <asm/arch/sys_proto.h> > +#include <dm/device.h> > +#include <dm/uclass.h> > +#include <linux/bitfield.h> > +#include <malloc.h> > + > +/* RCC register */ > +#define RCC_TZCR (STM32_RCC_BASE + 0x00) > +#define RCC_BDCR (STM32_RCC_BASE + 0x400) > +#define RCC_DBGCFGR (STM32_RCC_BASE + 0x468) > +#define RCC_MP_APB5ENSETR (STM32_RCC_BASE + 0x740) > +#define RCC_MP_AHB6ENSETR (STM32_RCC_BASE + 0x780) > + > +#define RCC_BDCR_VSWRST BIT(31) > +#define RCC_BDCR_RTCSRC GENMASK(17, 16) > + > +#define RCC_DBGCFGR_DBGCKEN BIT(8) > + > +/* DBGMCU register */ > +#define DBGMCU_APB4FZ1 (STM32_DBGMCU_BASE + 0x2c) > +#define DBGMCU_APB4FZ1_IWDG2 BIT(2) > + > +/* Security register */ > +#define ETZPC_TZMA1_SIZE (STM32_ETZPC_BASE + 0x04) > +#define ETZPC_DECPROT0 (STM32_ETZPC_BASE + 0x10) > + > +#define TZC_ACTION (STM32_TZC_BASE + 0x004) > +#define TZC_GATE_KEEPER (STM32_TZC_BASE + 0x008) > +#define TZC_REGION_BASE(n) (STM32_TZC_BASE + 0x100 + (0x20 * (n))) > +#define TZC_REGION_TOP(n) (STM32_TZC_BASE + 0x108 + (0x20 * (n))) > +#define TZC_REGION_ATTRIBUTE(n) (STM32_TZC_BASE + 0x110 + (0x20 * (n))) > +#define TZC_REGION_ID_ACCESS(n) (STM32_TZC_BASE + 0x114 + (0x20 * (n))) > + > +#define TAMP_CR1 (STM32_TAMP_BASE + 0x00) > + > +#define PWR_CR1 (STM32_PWR_BASE + 0x00) > +#define PWR_CR1_DBP BIT(8) > + > +/* boot interface from Bootrom > + * - boot instance = bit 31:16 > + * - boot device = bit 15:0 > + */ > +#define BOOTROM_MODE_MASK GENMASK(15, 0) > +#define BOOTROM_MODE_SHIFT 0 > +#define BOOTROM_INSTANCE_MASK GENMASK(31, 16) > +#define BOOTROM_INSTANCE_SHIFT 16 > > /* SYSCFG register */ > #define SYSCFG_IDC_OFFSET 0x380 > @@ -23,6 +71,162 @@ > #define RPN_SHIFT 0 > #define RPN_MASK GENMASK(11, 0) > > +static void security_init(void) > +{ > + /* Disable the backup domain write protection */ > + /* the protection is enable at each reset by hardware */ > + /* And must be disable by software */ > + setbits_le32(PWR_CR1, PWR_CR1_DBP); > + > + while (!(readl(PWR_CR1) & PWR_CR1_DBP)) > + ; > + > + /* If RTC clock isn't enable so this is a cold boot then we need > + * to reset the backup domain > + */ > + if (!(readl(RCC_BDCR) & RCC_BDCR_RTCSRC)) { > + setbits_le32(RCC_BDCR, RCC_BDCR_VSWRST); > + while (!(readl(RCC_BDCR) & RCC_BDCR_VSWRST)) > + ; > + clrbits_le32(RCC_BDCR, RCC_BDCR_VSWRST); > + } > + > + /* allow non secure access in Write/Read for all peripheral */ > + writel(0, ETZPC_DECPROT0); > + > + /* Open SYSRAM for no secure access */ > + writel(0x0, ETZPC_TZMA1_SIZE); > + > + /* enable MCE clock */ > + writel(BIT(1), RCC_MP_AHB6ENSETR); > + > + /* enable TZC clock */ > + writel(BIT(11), RCC_MP_APB5ENSETR); > + > + /* Disable Filter 0 */ > + writel(0, TZC_GATE_KEEPER); > + > + /* Region 0 set to no access by default */ > + /* bit 0 / 16 => nsaid0 read/write Enable > + * bit 1 / 17 => nsaid1 read/write Enable > + * ... > + * bit 15 / 31 => nsaid15 read/write Enable > + */ > + writel(0xFFFFFFFF, TZC_REGION_ID_ACCESS(0)); > + > + /* bit 30 / 31 => Secure Global Enable : write/read */ > + writel(BIT(0) | BIT(30) | BIT(31), TZC_REGION_ATTRIBUTE(0)); > + > + writel(0xFFFFFFFF, TZC_REGION_ID_ACCESS(1)); > + writel(0xC0000000, TZC_REGION_BASE(1)); > + writel(0xDDFFFFFF, TZC_REGION_TOP(1)); > + writel(BIT(0) | BIT(30) | BIT(31), TZC_REGION_ATTRIBUTE(1)); > + > + writel(0x00000000, TZC_REGION_ID_ACCESS(2)); > + writel(0xDE000000, TZC_REGION_BASE(2)); > + writel(0xDFFFFFFF, TZC_REGION_TOP(2)); > + writel(BIT(0) | BIT(30) | BIT(31), TZC_REGION_ATTRIBUTE(2)); > + > + writel(0xFFFFFFFF, TZC_REGION_ID_ACCESS(3)); > + writel(0x00000000, TZC_REGION_BASE(3)); > + writel(0xBFFFFFFF, TZC_REGION_TOP(3)); > + writel(BIT(0) | BIT(30) | BIT(31), TZC_REGION_ATTRIBUTE(3)); > + > + /* Set Action */ > + writel(BIT(0), TZC_ACTION); > + > + /* Enable Filter 0 */ > + writel(BIT(0), TZC_GATE_KEEPER); > + > + /* RCC trust zone deactivated */ > + writel(0x0, RCC_TZCR); > + > + /* TAMP: deactivate the internal tamper > + * Bit 23 ITAMP8E: monotonic counter overflow > + * Bit 20 ITAMP5E: RTC calendar overflow > + * Bit 19 ITAMP4E: HSE monitoring > + * Bit 18 ITAMP3E: LSE monitoring > + * Bit 16 ITAMP1E: RTC power domain supply monitoring > + */ > + writel(0x0, TAMP_CR1); > +} > + > +/* > + * Debug init > + */ > +void dbgmcu_init(void) > +{ > + /* > + * Freeze IWDG2 if Cortex-A7 is in debug mode > + * done in TF-A for TRUSTED boot and > + * DBGMCU access is controlled by BSEC_DENABLE.DBGSWENABLE > + */ > + if (bsec_dbgswenable()) { > + setbits_le32(RCC_DBGCFGR, RCC_DBGCFGR_DBGCKEN); > + setbits_le32(DBGMCU_APB4FZ1, DBGMCU_APB4FZ1_IWDG2); > + } > +} > + > +void spl_board_init(void) > +{ > + struct udevice *dev; > + u8 *tlb; > + int ret; > + > + dbgmcu_init(); > + > + /* force probe of BSEC driver to shadow the upper OTP */ > + ret = uclass_get_device_by_driver(UCLASS_MISC, > DM_DRIVER_GET(stm32mp_bsec), &dev); > + if (ret) > + log_warning("BSEC probe failed: %d\n", ret); > + > + /* Enable Dcache here, now that DRAM is available */ > + if (IS_ENABLED(CONFIG_XPL_BUILD) && IS_ENABLED(CONFIG_STM32MP13X)) { > + tlb = memalign(0x4000, PGTABLE_SIZE); > + if (!tlb) > + return; > + > + gd->arch.tlb_size = PGTABLE_SIZE; > + gd->arch.tlb_addr = (unsigned long)tlb; > + dcache_enable(); > + } > +} > + > +/* get bootmode from ROM code boot context: saved in TAMP register */ > +static void update_bootmode(void) > +{ > + u32 boot_mode; > + u32 bootrom_itf = readl(get_stm32mp_rom_api_table()); > + u32 bootrom_device, bootrom_instance; > + > + /* enable TAMP clock = RTCAPBEN */ > + writel(BIT(8), RCC_MP_APB5ENSETR); > + > + /* read bootrom context */ > + bootrom_device = > + (bootrom_itf & BOOTROM_MODE_MASK) >> BOOTROM_MODE_SHIFT; > + bootrom_instance = > + (bootrom_itf & BOOTROM_INSTANCE_MASK) >> BOOTROM_INSTANCE_SHIFT; > + boot_mode = > + ((bootrom_device << BOOT_TYPE_SHIFT) & BOOT_TYPE_MASK) | > + ((bootrom_instance << BOOT_INSTANCE_SHIFT) & > + BOOT_INSTANCE_MASK); > + > + /* save the boot mode in TAMP backup register */ > + clrsetbits_le32(TAMP_BOOT_CONTEXT, > + TAMP_BOOT_MODE_MASK, > + boot_mode << TAMP_BOOT_MODE_SHIFT); > +} > + > +/* weak function: STM32MP15x mach init for boot without TFA */ > +void stm32mp_cpu_init(void) > +{ > + if (IS_ENABLED(CONFIG_XPL_BUILD)) { > + security_init(); > + update_bootmode(); > + } > +} > + > static u32 read_idc(void) > { > void *syscfg = syscon_get_first_range(STM32MP_SYSCON_SYSCFG);
