jasonbu commented on code in PR #16344:
URL: https://github.com/apache/nuttx/pull/16344#discussion_r2081502559
##########
arch/arm/src/armv7-a/arm_addrenv_utils.c:
##########
@@ -97,7 +96,9 @@ int arm_addrenv_create_region(uintptr_t **list, unsigned int
listlen,
nmapped = 0;
for (i = 0; i < nlist; i++)
{
- /* Allocate one physical page for the L2 page table */
+ /* Allocate one physical page for the L2 page table
+ * Makesure physical page is sequential, alloc all l2 table first.
Review Comment:
added a REVISIT: here to notice driver should not depend on physic address
contiguous.
--
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
