[coreboot-gerrit] [M] Change in coreboot[main]: region: Turn region_end() into an inclusive region_last()

[Nico Huber (Code Review)]

Attention is currently required from: Jakub Czapiga, Julius Werner, Jérémy Compostella, Werner Zeh.

Hello Jakub Czapiga, Jérémy Compostella, build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/79946?usp=email

to look at the new patch set (#7).

The following approvals got outdated and were removed:
Verified+1 by build bot (Jenkins)

Change subject: region: Turn region_end() into an inclusive region_last()
......................................................................

region: Turn region_end() into an inclusive region_last()

The current region_end() implementation is susceptible to overflow
if the region is at the end of the addressable space. A common case
with the memory-mapped flash of x86 directly below the 32-bit limit.

Note: This patch also changes console output to inclusive limits.
IMO, to the better.

Change-Id: Ic4bd6eced638745b7e845504da74542e4220554a
Signed-off-by: Nico Huber 
---
M src/commonlib/include/commonlib/region.h
M src/commonlib/region.c
M src/cpu/x86/smm/smm_module_loader.c
M src/drivers/spi/winbond.c
M tests/commonlib/region-test.c
M tests/lib/region_file-test.c
M util/cbfstool/cbfstool.c
M util/cbfstool/cse_serger.c
8 files changed, 23 insertions(+), 27 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/46/79946/7