pussuw commented on code in PR #12353:
URL: https://github.com/apache/nuttx/pull/12353#discussion_r1601878364
##########
arch/arm64/src/imx9/imx9_boot.c:
##########
@@ -44,19 +44,39 @@
#include "imx9_gpio.h"
#include "imx9_lowputc.h"
+/****************************************************************************
+ * Pre-processor Definitions
+ ****************************************************************************/
+
+#define CONFIG_SYS_CLOCK_HW_CYCLES_PER_SEC 1700000000
+
/****************************************************************************
* Private Data
****************************************************************************/
+#ifdef CONFIG_IMX9_BOOTLOADER
+extern uint8_t _szbootloader[];
+#endif
+
static const struct arm_mmu_region g_mmu_regions[] =
{
- MMU_REGION_FLAT_ENTRY("DEVICE_REGION",
- CONFIG_DEVICEIO_BASEADDR, CONFIG_DEVICEIO_SIZE,
- MT_DEVICE_NGNRNE | MT_RW | MT_SECURE),
+ MMU_REGION_FLAT_ENTRY("DEVICE_REGION",
+ CONFIG_DEVICEIO_BASEADDR, CONFIG_DEVICEIO_SIZE,
+ MT_DEVICE_NGNRNE | MT_RW | MT_SECURE),
+
+ MMU_REGION_FLAT_ENTRY("DRAM0_S0",
+ CONFIG_RAMBANK1_ADDR, CONFIG_RAMBANK1_SIZE,
+ MT_NORMAL | MT_RW | MT_SECURE),
+#ifdef CONFIG_IMX9_BOOTLOADER
+ /* OCRAM secure region protection keeps the binary at RO mode,
+ * overriding any MMU settings. However, data and heap accesses
+ * beyond the binary require RW access.
+ */
- MMU_REGION_FLAT_ENTRY("DRAM0_S0",
- CONFIG_RAMBANK1_ADDR, CONFIG_RAMBANK1_SIZE,
- MT_NORMAL | MT_RW | MT_SECURE),
+ MMU_REGION_FLAT_ENTRY("BOOTLOADER",
+ (uint64_t)_stext, (uint64_t)_szbootloader,
+ MT_NORMAL | MT_RW | MT_SECURE),
Review Comment:
Should this then be _etext ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
