[coreboot-gerrit] Change in coreboot[master]: memranges: Change align attribute to be log2 of required alignment

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39810 )

Change subject: memranges: Change align attribute to be log2 of required alignment
......................................................................

Patch Set 5:

(2 comments)

https://review.coreboot.org/c/coreboot/+/39810/2/src/include/memrange.h 
File src/include/memrange.h:

https://review.coreboot.org/c/coreboot/+/39810/2/src/include/memrange.h@116 
PS2, Line 116: log2(4 * KiB)
> We could just use '12'. […]

True. I will update this.

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

https://review.coreboot.org/c/coreboot/+/39810/2/src/lib/memrange.c@236 
PS2, Line 236: 1
> 1ULL throughout as resource_t is a 64-bit value.

Done