[coreboot-gerrit] Change in coreboot[master]: amd/pi: Relocate amd_initcpuio

Marshall Dawson (Code Review) gerrit at coreboot.org
Thu May 4 22:13:58 CEST 2017


Marshall Dawson has uploaded a new change for review. ( https://review.coreboot.org/19574 )

Change subject: amd/pi: Relocate amd_initcpuio
......................................................................

amd/pi: Relocate amd_initcpuio

Move amd_initcpuio() to the end of amdinitpost().  A subsequent
patch will use this to communicate to the PSP.

Change-Id: I3ec97d9b4ce4f58c3931068213c5dc7109d39397
Signed-off-by: Marshall Dawson <marshalldawson3rd at gmail.com>
---
M src/northbridge/amd/pi/agesawrapper.c
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/74/19574/1

diff --git a/src/northbridge/amd/pi/agesawrapper.c b/src/northbridge/amd/pi/agesawrapper.c
index 0afd0ca..f7515c0 100644
--- a/src/northbridge/amd/pi/agesawrapper.c
+++ b/src/northbridge/amd/pi/agesawrapper.c
@@ -172,6 +172,9 @@
 	/* Initialize heap space */
 	EmptyHeap();
 
+	/* Enable MMIO on AMD CPU Address Map Controller */
+	amd_initcpuio();
+
 	return status;
 }
 
@@ -245,9 +248,6 @@
 	AGESA_STATUS status;
 	AMD_INTERFACE_PARAMS AmdParamStruct;
 	AMD_MID_PARAMS *MidParam;
-
-	/* Enable MMIO on AMD CPU Address Map Controller */
-	amd_initcpuio ();
 
 	LibAmdMemFill (&AmdParamStruct,
 		       0,

-- 
To view, visit https://review.coreboot.org/19574
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3ec97d9b4ce4f58c3931068213c5dc7109d39397
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Marshall Dawson <marshalldawson3rd at gmail.com>



More information about the coreboot-gerrit mailing list