[coreboot-gerrit] Change in coreboot[master]: device: Introduce RESOURCE_ALLOCATION_TOP_DOWN

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41957 )

Change subject: device: Introduce RESOURCE_ALLOCATION_TOP_DOWN
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41957/2/src/lib/memrange.c 
File src/lib/memrange.c:

https://review.coreboot.org/c/coreboot/+/41957/2/src/lib/memrange.c@430 
PS2, Line 430: 		*stolen_base = ALIGN_DOWN(range_entry_end(r) - size, POWER_OF_2(align));
Don't we need to handle the from_top logic in memranges_find_entry() for region size checking? We're doing the align up check there, but I'm not convinced that's enough.