From 1b457f851795a2bae402a0c1e9e190cb5c6add20 Mon Sep 17 00:00:00 2001
From: Felix Held <felix.held@amd.corp-partner.google.com>
Date: Sun, 19 Apr 2020 00:02:01 +0200
Subject: [PATCH] soc/amd/stoneyridge/memmap: fix bug in bert_reserved_region

Changing the local pointer "start" has no effect.
Changing the value it points to has.

Change-Id: I1b689896fcf255b795b27d7a7163849d6dfdb00e
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/40506
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
---
 src/soc/amd/stoneyridge/memmap.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/soc/amd/stoneyridge/memmap.c b/src/soc/amd/stoneyridge/memmap.c
index 0c8d9c0cdf..34ddcb33c9 100644
--- a/src/soc/amd/stoneyridge/memmap.c
+++ b/src/soc/amd/stoneyridge/memmap.c
@@ -32,7 +32,7 @@ void bert_reserved_region(void **start, size_t *size)
 	if (CONFIG(ACPI_BERT))
 		*start = cbmem_top();
 	else
-		start = NULL;
+		*start = NULL;
 	*size = BERT_REGION_MAX_SIZE;
 }