[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 2:

(1 comment)

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@29 
PS2, Line 29: 	size_t align;
> Is size_t still justified here, struct resource uses unsigned char.

Valid point. I think it makes sense to switch to unsigned char.