Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/41103 )
Change subject: memrange: Update comment to indicate limit is inclusive for memranges_next_entry ......................................................................
memrange: Update comment to indicate limit is inclusive for memranges_next_entry
This change updates the comment for memranges_next_entry() to indicate that the limit provided by the caller is inclusive.
Change-Id: Id40263efcb9417ed31c130996e56c30dbbc82e02 Signed-off-by: Furquan Shaikh furquan@google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/41103 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Aaron Durbin adurbin@chromium.org --- M src/include/memrange.h 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Aaron Durbin: Looks good to me, approved
diff --git a/src/include/memrange.h b/src/include/memrange.h index deb8c1e..83e3826 100644 --- a/src/include/memrange.h +++ b/src/include/memrange.h @@ -162,7 +162,7 @@ const struct range_entry *r);
/* Steals memory from the available list in given ranges as per the constraints: - * limit = Upper bound for the memory range to steal. + * limit = Upper bound for the memory range to steal (Inclusive). * size = Requested size for the stolen memory. * align = Required alignment(log 2) for the starting address of the stolen memory. * tag = Use a range that matches the given tag.