From 2fc9b7acfca7aacf53539978db8a7493f9985755 Mon Sep 17 00:00:00 2001 From: Gregory Nutt Date: Tue, 29 Jan 2019 17:13:54 -0600 Subject: [PATCH] Update some comments. --- arch/arm/src/lpc17xx/lpc17_allocateheap.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/arch/arm/src/lpc17xx/lpc17_allocateheap.c b/arch/arm/src/lpc17xx/lpc17_allocateheap.c index 4f78ebbd69f..e2ca6933ca4 100644 --- a/arch/arm/src/lpc17xx/lpc17_allocateheap.c +++ b/arch/arm/src/lpc17xx/lpc17_allocateheap.c @@ -406,7 +406,12 @@ void up_addregion(void) heap_size = dram_end - dram_start; #if defined(CONFIG_BUILD_PROTECTED) && defined(CONFIG_MM_KERNEL_HEAP) - /* Allow user-mode access to the external DRAM heap memory */ + /* Allow user-mode access to the external DRAM heap memory. + * + * REVISIT: In PROTECTED mode, it would be necessary to align + * dram_start to an address that is easily spanned by an MPU + * region if dram_start is not at the beginning of CS0. + */ lpc17_mpu_uheap(dram_start, heap_size); #endif