From 7b7a75c3ca5acf9e769415d98c4e4609ee9b080a Mon Sep 17 00:00:00 2001
From: Gilles DOFFE <gilles.doffe@savoirfairelinux.com>
Date: Sun, 2 Aug 2020 23:48:59 +0200
Subject: [PATCH] cpu/stm32: setup memory length for stm32mp157x

The stm32mp157c has the particularity to not having flash memory but
only SRAM.
Thus a part of SRAM must be considered as a ROM to flash the firmware
into.
In case of the stm32mp157c, the RETRAM (64kB) is used as ROM and the
4 banks of SRAM (384kB) are used as RAM.
However, as ROM_LEN, RAM_LEN, ROM_START_ADDRESS, RAM_START_ADDRESS and
ROM_OFFSET could be overloaded by user, set them with "?=" operator.
If the ROM_START_ADDRESS and the RAM_START_ADDRESS are not set at the
end of that file, it is a classic stm32 MCU with flash, thus set this
variables with common memory addresses for stm32 MCU family:
ROM_START_ADDR ?= 0x08000000
RAM_START_ADDR ?= 0x20000000

Signed-off-by: Gilles DOFFE <gilles.doffe@savoirfairelinux.com>
---
 cpu/stm32/stm32_mem_lengths.mk | 37 ++++++++++++++++++++++++++++++----
 1 file changed, 33 insertions(+), 4 deletions(-)

diff --git a/cpu/stm32/stm32_mem_lengths.mk b/cpu/stm32/stm32_mem_lengths.mk
index 9fcba4afd1..f87649d245 100644
--- a/cpu/stm32/stm32_mem_lengths.mk
+++ b/cpu/stm32/stm32_mem_lengths.mk
@@ -1,7 +1,3 @@
-# Set the common memory addresses for stm32 MCU family
-ROM_START_ADDR ?= 0x08000000
-RAM_START_ADDR ?= 0x20000000
-
 # The next block takes care of setting the rigth lengths of RAM and ROM
 # for the stm32 family. Most of the CPUs should have been taken into
 # account here, so no need to assign the lengths per model.
@@ -278,6 +274,13 @@ else ifeq ($(STM32_TYPE), W)
       endif
     endif
   endif
+else ifeq ($(STM32_TYPE), MP)
+  ifeq ($(STM32_FAMILY), 1)
+    ifeq ($(STM32_MODEL), 157)
+      RAM_START_ADDR = 0x10000000
+      RAM_LEN = 384K
+    endif
+  endif
 endif
 
 ifeq ($(RAM_LEN), )
@@ -308,6 +311,26 @@ else ifeq ($(STM32_ROMSIZE), H)
   ROM_LEN = 1536K
 else ifeq ($(STM32_ROMSIZE), I)
   ROM_LEN = 2048K
+else ifeq ($(STM32_TYPE), MP)
+  ifeq ($(STM32_FAMILY), 1)
+    # STM32MP1 family has no flash ROM memory.
+    # Thus a part of SRAM must be considered as ROM.
+    # RETRAM (0x0 address) is setup as ROM by default.
+    # However in RIOT, vector table is relocated using VTOR register.
+    # Considering the minimum alignment is 128 words and knowing the number of
+    # interrupt vectors for a given MCU, if a device has for example 150 interrupt
+    # channels:
+    # Vector table has a size of 150 * 4 = 600 = 0x258
+    # As the table should be 128 word aligned, vector table size reserved is 0x400.
+    ifeq ($(STM32_MODEL), 157)
+      ROM_START_ADDR ?= 0x0
+      ifneq (,$(filter stm32mp1_eng_mode,$(USEMODULE)))
+        ROM_OFFSET ?= 0x400
+      endif
+      ROM_LEN ?= 64K
+    endif
+  endif
+else
 endif
 
 ifeq ($(STM32_PINCOUNT), A)
@@ -348,4 +371,10 @@ else ifeq ($(STM32_PINCOUNT), V)
   STM32_PIN = 100
 else ifeq ($(STM32_PINCOUNT), Z)
   STM32_PIN = 144
+else ifeq ($(STM32_PINCOUNT), AC)
+  STM32_PIN = 361
 endif
+
+# Set the common memory addresses for stm32 MCU family
+ROM_START_ADDR ?= 0x08000000
+RAM_START_ADDR ?= 0x20000000