[coreboot-gerrit] Change in coreboot[master]: mainboard/amd: Add required callouts to Stoney BiosCallouts.c

Martin Roth (Code Review) gerrit at coreboot.org
Tue Sep 26 22:50:10 CEST 2017


Martin Roth has uploaded this change for review. ( https://review.coreboot.org/21708


Change subject: mainboard/amd: Add required callouts to Stoney BiosCallouts.c
......................................................................

mainboard/amd: Add required callouts to Stoney BiosCallouts.c

Hook the new required AGESA callout functions into the callouts tables.

BUG=b:66690176
TEST=Build and boot Kahlee - see the functions get called.

Change-Id: Ife9c2b20e59ede404edb1f700238e425fea35914
Signed-off-by: Martin Roth <martinroth at google.com>
---
M src/mainboard/amd/gardenia/BiosCallOuts.c
M src/mainboard/amd/gardenia/bootblock/BiosCallOuts.c
M src/mainboard/google/kahlee/BiosCallOuts.c
M src/mainboard/google/kahlee/bootblock/BiosCallOuts.c
4 files changed, 50 insertions(+), 14 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/08/21708/1

diff --git a/src/mainboard/amd/gardenia/BiosCallOuts.c b/src/mainboard/amd/gardenia/BiosCallOuts.c
index 1d06411..c947b0c 100644
--- a/src/mainboard/amd/gardenia/BiosCallOuts.c
+++ b/src/mainboard/amd/gardenia/BiosCallOuts.c
@@ -84,18 +84,35 @@
 }
 
 const BIOS_CALLOUT_STRUCT BiosCallouts[] = {
+	/* Required callouts */
 	{AGESA_ALLOCATE_BUFFER,          agesa_AllocateBuffer },
 	{AGESA_DEALLOCATE_BUFFER,        agesa_DeallocateBuffer },
+	{AGESA_DO_RESET,                 agesa_Reset },
 	{AGESA_LOCATE_BUFFER,            agesa_LocateBuffer },
 	{AGESA_READ_SPD,                 agesa_ReadSpd },
-	{AGESA_DO_RESET,                 agesa_Reset },
-	{AGESA_READ_SPD_RECOVERY,        agesa_NoopUnsupported },
 	{AGESA_RUNFUNC_ONAP,             agesa_RunFuncOnAp },
-	{AGESA_GET_IDS_INIT_DATA,        agesa_EmptyIdsInitData },
-	{AGESA_HOOKBEFORE_DQS_TRAINING,  agesa_NoopSuccess },
-	{AGESA_HOOKBEFORE_EXIT_SELF_REF, agesa_NoopSuccess },
-	{AGESA_FCH_OEM_CALLOUT,          fch_initenv },
-	{AGESA_GNB_GFX_GET_VBIOS_IMAGE,  agesa_GfxGetVbiosImage }
+	{AGESA_RUNFUNC_ON_ALL_APS,       agesa_RunFcnOnAllAps },
+	{AMD_LATE_RUN_AP_TASK,           agesa_LateRunApTask },
+	{AGESA_GNB_PCIE_SLOT_RESET,      agesa_PcieSlotResetControl },
+	{AGESA_WAIT_FOR_ALL_APS,         agesa_WaitForAllApsFinished },
+	{AGESA_IDLE_AN_AP,               agesa_IdleAnAp },
+
+	/* Optional callouts */
+	{AGESA_GET_IDS_INIT_DATA,             agesa_EmptyIdsInitData },
+	//AgesaHeapRebase - Hook ID?
+	{AGESA_HOOKBEFORE_DRAM_INIT,          agesa_NoopUnsupported },
+	{AGESA_HOOKBEFORE_DQS_TRAINING,       agesa_NoopUnsupported },
+	{AGESA_EXTERNAL_2D_TRAIN_VREF_CHANGE, agesa_NoopUnsupported },
+	{AGESA_HOOKBEFORE_EXIT_SELF_REF,      agesa_NoopUnsupported },
+	{AGESA_GNB_GFX_GET_VBIOS_IMAGE,       agesa_GfxGetVbiosImage },
+	{AGESA_FCH_OEM_CALLOUT,               fch_initenv },
+	{AGESA_EXTERNAL_VOLTAGE_ADJUST,       agesa_NoopUnsupported },
+	{AGESA_GNB_PCIE_CLK_REQ,              agesa_NoopUnsupported },
+
+	/* Deprecated */
+	{AGESA_HOOKBEFORE_DRAM_INIT_RECOVERY, agesa_NoopUnsupported},
+	{AGESA_READ_SPD_RECOVERY,             agesa_NoopUnsupported },
+
 };
 
 const int BiosCalloutsLen = ARRAY_SIZE(BiosCallouts);
diff --git a/src/mainboard/amd/gardenia/bootblock/BiosCallOuts.c b/src/mainboard/amd/gardenia/bootblock/BiosCallOuts.c
index b0a3e23..1b7a341 100644
--- a/src/mainboard/amd/gardenia/bootblock/BiosCallOuts.c
+++ b/src/mainboard/amd/gardenia/bootblock/BiosCallOuts.c
@@ -64,6 +64,7 @@
 
 const BIOS_CALLOUT_STRUCT BiosCallouts[] = {
 	{AGESA_FCH_OEM_CALLOUT,          fch_initreset },
+	{AGESA_GNB_PCIE_SLOT_RESET,      agesa_PcieSlotResetControl }
 };
 
 const int BiosCalloutsLen = ARRAY_SIZE(BiosCallouts);
diff --git a/src/mainboard/google/kahlee/BiosCallOuts.c b/src/mainboard/google/kahlee/BiosCallOuts.c
index f651876..2e57166 100644
--- a/src/mainboard/google/kahlee/BiosCallOuts.c
+++ b/src/mainboard/google/kahlee/BiosCallOuts.c
@@ -65,18 +65,35 @@
 }
 
 const BIOS_CALLOUT_STRUCT BiosCallouts[] = {
+	/* Required callouts */
 	{AGESA_ALLOCATE_BUFFER,          agesa_AllocateBuffer },
 	{AGESA_DEALLOCATE_BUFFER,        agesa_DeallocateBuffer },
+	{AGESA_DO_RESET,                 agesa_Reset },
 	{AGESA_LOCATE_BUFFER,            agesa_LocateBuffer },
 	{AGESA_READ_SPD,                 agesa_ReadSpd },
-	{AGESA_DO_RESET,                 agesa_Reset },
-	{AGESA_READ_SPD_RECOVERY,        agesa_NoopUnsupported },
 	{AGESA_RUNFUNC_ONAP,             agesa_RunFuncOnAp },
-	{AGESA_GET_IDS_INIT_DATA,        agesa_EmptyIdsInitData },
-	{AGESA_HOOKBEFORE_DQS_TRAINING,  agesa_NoopSuccess },
-	{AGESA_HOOKBEFORE_EXIT_SELF_REF, agesa_NoopSuccess },
-	{AGESA_FCH_OEM_CALLOUT,          fch_initenv },
-	{AGESA_GNB_GFX_GET_VBIOS_IMAGE,  agesa_GfxGetVbiosImage }
+	{AGESA_RUNFUNC_ON_ALL_APS,       agesa_RunFcnOnAllAps },
+	{AMD_LATE_RUN_AP_TASK,           agesa_LateRunApTask },
+	{AGESA_GNB_PCIE_SLOT_RESET,      agesa_PcieSlotResetControl },
+	{AGESA_WAIT_FOR_ALL_APS,         agesa_WaitForAllApsFinished },
+	{AGESA_IDLE_AN_AP,               agesa_IdleAnAp },
+
+	/* Optional callouts */
+	{AGESA_GET_IDS_INIT_DATA,             agesa_EmptyIdsInitData },
+	//AgesaHeapRebase - Hook ID?
+	{AGESA_HOOKBEFORE_DRAM_INIT,          agesa_NoopUnsupported },
+	{AGESA_HOOKBEFORE_DQS_TRAINING,       agesa_NoopUnsupported },
+	{AGESA_EXTERNAL_2D_TRAIN_VREF_CHANGE, agesa_NoopUnsupported },
+	{AGESA_HOOKBEFORE_EXIT_SELF_REF,      agesa_NoopUnsupported },
+	{AGESA_GNB_GFX_GET_VBIOS_IMAGE,       agesa_GfxGetVbiosImage },
+	{AGESA_FCH_OEM_CALLOUT,               fch_initenv },
+	{AGESA_EXTERNAL_VOLTAGE_ADJUST,       agesa_NoopUnsupported },
+	{AGESA_GNB_PCIE_CLK_REQ,              agesa_NoopUnsupported },
+
+	/* Deprecated */
+	{AGESA_HOOKBEFORE_DRAM_INIT_RECOVERY, agesa_NoopUnsupported},
+	{AGESA_READ_SPD_RECOVERY,             agesa_NoopUnsupported },
+
 };
 
 const int BiosCalloutsLen = ARRAY_SIZE(BiosCallouts);
diff --git a/src/mainboard/google/kahlee/bootblock/BiosCallOuts.c b/src/mainboard/google/kahlee/bootblock/BiosCallOuts.c
index 162fc50..4dce042 100644
--- a/src/mainboard/google/kahlee/bootblock/BiosCallOuts.c
+++ b/src/mainboard/google/kahlee/bootblock/BiosCallOuts.c
@@ -40,6 +40,7 @@
 
 const BIOS_CALLOUT_STRUCT BiosCallouts[] = {
 	{AGESA_FCH_OEM_CALLOUT,          fch_initreset },
+	{AGESA_GNB_PCIE_SLOT_RESET,      agesa_PcieSlotResetControl }
 };
 
 const int BiosCalloutsLen = ARRAY_SIZE(BiosCallouts);

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ife9c2b20e59ede404edb1f700238e425fea35914
Gerrit-Change-Number: 21708
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth at google.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170926/92ebf74c/attachment-0001.html>


More information about the coreboot-gerrit mailing list