Hello build bot (Jenkins), Patrick Georgi, Duncan Laurie, Aaron Durbin,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39810
to look at the new patch set (#4).
Change subject: memranges: Change align attribute to be log2 of required alignment ......................................................................
memranges: Change align attribute to be log2 of required alignment
This change updates the align attribute of memranges to be represented as log2 of the required alignment. This makes it consistent with how alignment is stored in struct resource as well.
Additionally, since memranges only allow power of 2 alignments, this change allows getting rid of checks at runtime and hence failure cases for non-power of 2 alignments.
This change also updates the type of align to be unsigned char.
BUG=b:149186922
Signed-off-by: Furquan Shaikh furquan@google.com Change-Id: Ie4d3868cdff55b2c7908b9b3ccd5f30a5288e62f --- M src/include/memrange.h M src/lib/memrange.c 2 files changed, 22 insertions(+), 31 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/10/39810/4