[coreboot-gerrit] Change in coreboot[master]: arch/x86/postcar_loader: Fix printed MTRR range computation

[Werner Zeh (Code Review)]

Werner Zeh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/63541 )

Change subject: arch/x86/postcar_loader: Fix printed MTRR range computation
......................................................................

arch/x86/postcar_loader: Fix printed MTRR range computation

There is a log message in 'postcar_var_mtrr_set()' showing the used
variable MTRRs which prints the start and the end address as well as the
size of the MTRR region. Instead of the end address the next address
after the end is computed which leads to a wrong log output.

This patch uses 'addr + size - 1' instead of 'addr + size' to compute
the end address correctly.

Change-Id: I0ca292f9cf272564cb5ef1c4ea38f5c483605c94
Signed-off-by: Werner Zeh 
---
M src/arch/x86/postcar_loader.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/41/63541/1

diff --git a/src/arch/x86/postcar_loader.c b/src/arch/x86/postcar_loader.c
index 2609fd6..0fe174b 100644
--- a/src/arch/x86/postcar_loader.c
+++ b/src/arch/x86/postcar_loader.c
@@ -60,7 +60,7 @@
    struct postcar_frame *pcf = ctx->arg;

printk(BIOS_DEBUG, "MTRR Range: Start=%lx End=%lx (Size %zx)\n",
-			addr, addr + size, size);
+			addr, addr + size - 1, size);

stack_push(pcf, mask.hi);
    stack_push(pcf, mask.lo);