UPSTREAM: AGESA: Move EmptyHeap() call

Specification says to do CAR teardown as part of AmdInitPost().
Move initializing the final AGESA heap storage to AmdInitEnv()
so the buffer is not invalidated without writeback.

BUG=none
BRANCH=none
TEST=none

Change-Id: Ib17675c8cddb8b1266f389b6a2c505713897da64
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Original-Commit-Id: 4a6e00fd36
Original-Change-Id: I3a5d497d0e25ec291f722e9f089bc8928238c3f9
Original-Signed-off-by: Kysti Mlkki <kyosti.malkki@gmail.com>
Original-Reviewed-on: https://review.coreboot.org/19024
Original-Tested-by: build bot (Jenkins)
Original-Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://chromium-review.googlesource.com/466059

src/northbridge/amd/agesa/agesawrapper.c

@@ -111,9 +111,6 @@ AGESA_STATUS agesawrapper_amdinitpost(void)
 
 	AmdReleaseStruct(&AmdParamStruct);
 
-	/* Initialize heap space */
-	EmptyHeap();
-
 	return status;
 }
 
@@ -153,6 +150,9 @@ AGESA_STATUS agesawrapper_amdinitenv(void)
 	AMD_INTERFACE_PARAMS AmdParamStruct;
 	AMD_ENV_PARAMS *EnvParam;
 
+	/* Initialize heap space */
+	EmptyHeap();
+
 	memset(&AmdParamStruct, 0, sizeof(AMD_INTERFACE_PARAMS));
 
 	AmdParamStruct.AgesaFunctionName = AMD_INIT_ENV;