sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/amd/common: Improve misc. formatting in AGESA wrapper

Browse Source 
Improve the file with:
 * C99 inializations for structures
 * reorder include files for aesthetics
 * remove extraneous whitespace
 * remove a stale comment
 * make variable naming consistent
 * make function arguments consistent

This change clears up all remaining checkpatch issues with the wrapper
with the exception of errors created with AMD definitions, e.g.
  ERROR: need consistent spacing around '*' (ctx:WxV)
  #32: FILE: src/soc/amd/common/block/pi/agesawrapper.c:32:
  void __attribute__((weak)) SetFchMidParams(FCH_INTERFACE *params) {}

BUG=b:62240746
TEST=Build and boot Kahlee

Change-Id: I40985e0cf50df6aa4d830937e7f6b6e7908f72fe
Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Reviewed-on: https://review.coreboot.org/22889
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
This commit is contained in:
Marshall Dawson
2017-12-13 20:01:59 -07:00
committed by Martin Roth
parent f3c57a7c28
commit 857a387520
1 changed files with 89 additions and 128 deletions
Download Patch File Download Diff File Expand all files Collapse all files

189
src/soc/amd/common/block/pi/agesawrapper.c
View File

@@ -13,17 +13,18 @@
* GNU General Public License for more details. * GNU General Public License for more details.
*/ */
#include <amdblocks/agesawrapper.h>
#include <arch/early_variables.h> #include <arch/early_variables.h>
#include <arch/acpi.h>
#include <cpu/x86/mtrr.h>
#include <cbfs.h> #include <cbfs.h>
#include <cbmem.h> #include <cbmem.h>
#include <delay.h> #include <delay.h>
#include <cpu/x86/mtrr.h>
#include <amdblocks/BiosCallOuts.h>
#include <rules.h> #include <rules.h>
#include <rmodule.h> #include <rmodule.h>
#include <string.h> #include <string.h>
#include <timestamp.h> #include <timestamp.h>
#include <amdblocks/agesawrapper.h>
#include <amdblocks/BiosCallOuts.h>
void __attribute__((weak)) SetMemParams(AMD_POST_PARAMS *PostParams) {} void __attribute__((weak)) SetMemParams(AMD_POST_PARAMS *PostParams) {}
void __attribute__((weak)) OemPostParams(AMD_POST_PARAMS *PostParams) {} void __attribute__((weak)) OemPostParams(AMD_POST_PARAMS *PostParams) {}
@@ -71,26 +72,19 @@ static AGESA_STATUS agesawrapper_readeventlog(UINT8 HeapStatus)
AGESA_STATUS agesawrapper_amdinitreset(void) AGESA_STATUS agesawrapper_amdinitreset(void)
{ {
AGESA_STATUS status; AGESA_STATUS status;
AMD_INTERFACE_PARAMS AmdParamStruct; AMD_RESET_PARAMS ResetParams;
AMD_RESET_PARAMS AmdResetParams; AMD_INTERFACE_PARAMS AmdParamStruct = {
.AgesaFunctionName = AMD_INIT_RESET,
memset(&AmdParamStruct, 0, sizeof(AmdParamStruct)); .AllocationMethod = ByHost,
memset(&AmdResetParams, 0, sizeof(AmdResetParams)); .NewStructSize = sizeof(AMD_RESET_PARAMS),
.NewStructPtr = &ResetParams,
AmdParamStruct.AgesaFunctionName = AMD_INIT_RESET; .StdHeader.CalloutPtr = &GetBiosCallout
AmdParamStruct.AllocationMethod = ByHost; };
AmdParamStruct.NewStructSize = sizeof(AMD_RESET_PARAMS);
AmdParamStruct.NewStructPtr = &AmdResetParams;
AmdParamStruct.StdHeader.AltImageBasePtr = 0;
AmdParamStruct.StdHeader.CalloutPtr = &GetBiosCallout;
AmdParamStruct.StdHeader.Func = 0;
AmdParamStruct.StdHeader.ImageBasePtr = 0;
AmdCreateStruct(&AmdParamStruct); AmdCreateStruct(&AmdParamStruct);
SetFchResetParams(&ResetParams.FchInterface);
SetFchResetParams(&AmdResetParams.FchInterface);
timestamp_add_now(TS_AGESA_INIT_RESET_START); timestamp_add_now(TS_AGESA_INIT_RESET_START);
status = AmdInitReset(&AmdResetParams); status = AmdInitReset(&ResetParams);
timestamp_add_now(TS_AGESA_INIT_RESET_DONE); timestamp_add_now(TS_AGESA_INIT_RESET_DONE);
if (status != AGESA_SUCCESS) if (status != AGESA_SUCCESS)
@@ -102,26 +96,22 @@ AGESA_STATUS agesawrapper_amdinitreset(void)
AGESA_STATUS agesawrapper_amdinitearly(void) AGESA_STATUS agesawrapper_amdinitearly(void)
{ {
AGESA_STATUS status; AGESA_STATUS status;
AMD_INTERFACE_PARAMS AmdParamStruct; AMD_EARLY_PARAMS *EarlyParams;
AMD_EARLY_PARAMS *AmdEarlyParamsPtr; AMD_INTERFACE_PARAMS AmdParamStruct = {
.AgesaFunctionName = AMD_INIT_EARLY,
.AllocationMethod = PreMemHeap,
.StdHeader.CalloutPtr = &GetBiosCallout,
};
memset(&AmdParamStruct, 0, sizeof(AmdParamStruct));
AmdParamStruct.AgesaFunctionName = AMD_INIT_EARLY;
AmdParamStruct.AllocationMethod = PreMemHeap;
AmdParamStruct.StdHeader.AltImageBasePtr = 0;
AmdParamStruct.StdHeader.CalloutPtr = &GetBiosCallout;
AmdParamStruct.StdHeader.Func = 0;
AmdParamStruct.StdHeader.ImageBasePtr = 0;
AmdCreateStruct(&AmdParamStruct); AmdCreateStruct(&AmdParamStruct);
AmdEarlyParamsPtr = (AMD_EARLY_PARAMS *)AmdParamStruct.NewStructPtr; EarlyParams = (AMD_EARLY_PARAMS *)AmdParamStruct.NewStructPtr;
OemCustomizeInitEarly (AmdEarlyParamsPtr); OemCustomizeInitEarly(EarlyParams);
AmdEarlyParamsPtr->GnbConfig.PsppPolicy = PsppDisabled; EarlyParams->GnbConfig.PsppPolicy = PsppDisabled;
timestamp_add_now(TS_AGESA_INIT_EARLY_START); timestamp_add_now(TS_AGESA_INIT_EARLY_START);
status = AmdInitEarly ((AMD_EARLY_PARAMS *)AmdParamStruct.NewStructPtr); status = AmdInitEarly(EarlyParams);
timestamp_add_now(TS_AGESA_INIT_EARLY_DONE); timestamp_add_now(TS_AGESA_INIT_EARLY_DONE);
if (status != AGESA_SUCCESS) if (status != AGESA_SUCCESS)
@@ -167,21 +157,16 @@ static void print_init_post_settings(AMD_POST_PARAMS *parms)
AGESA_STATUS agesawrapper_amdinitpost(void) AGESA_STATUS agesawrapper_amdinitpost(void)
{ {
AGESA_STATUS status; AGESA_STATUS status;
AMD_INTERFACE_PARAMS AmdParamStruct; AMD_INTERFACE_PARAMS AmdParamStruct = {
.AgesaFunctionName = AMD_INIT_POST,
.AllocationMethod = PreMemHeap,
.StdHeader.CalloutPtr = &GetBiosCallout,
};
AMD_POST_PARAMS *PostParams; AMD_POST_PARAMS *PostParams;
memset(&AmdParamStruct, 0, sizeof(AmdParamStruct));
AmdParamStruct.AgesaFunctionName = AMD_INIT_POST;
AmdParamStruct.AllocationMethod = PreMemHeap;
AmdParamStruct.StdHeader.AltImageBasePtr = 0;
AmdParamStruct.StdHeader.CalloutPtr = &GetBiosCallout;
AmdParamStruct.StdHeader.Func = 0;
AmdParamStruct.StdHeader.ImageBasePtr = 0;
AmdCreateStruct(&AmdParamStruct); AmdCreateStruct(&AmdParamStruct);
PostParams = (AMD_POST_PARAMS *)AmdParamStruct.NewStructPtr;
PostParams = (AMD_POST_PARAMS *)AmdParamStruct.NewStructPtr;
PostParams->MemConfig.UmaMode = CONFIG_GFXUMA ? UMA_AUTO : UMA_NONE; PostParams->MemConfig.UmaMode = CONFIG_GFXUMA ? UMA_AUTO : UMA_NONE;
PostParams->MemConfig.UmaSize = 0; PostParams->MemConfig.UmaSize = 0;
PostParams->MemConfig.BottomIo = (UINT16) PostParams->MemConfig.BottomIo = (UINT16)
@@ -223,34 +208,25 @@ AGESA_STATUS agesawrapper_amdinitpost(void)
AGESA_STATUS agesawrapper_amdinitenv(void) AGESA_STATUS agesawrapper_amdinitenv(void)
{ {
AGESA_STATUS status; AGESA_STATUS status;
AMD_INTERFACE_PARAMS AmdParamStruct; AMD_INTERFACE_PARAMS AmdParamStruct = {
AMD_ENV_PARAMS *EnvParam; .AgesaFunctionName = AMD_INIT_ENV,
.AllocationMethod = PostMemDram,
.StdHeader.CalloutPtr = &GetBiosCallout,
};
AMD_ENV_PARAMS *EnvParams;
memset(&AmdParamStruct, 0, sizeof(AmdParamStruct));
AmdParamStruct.AgesaFunctionName = AMD_INIT_ENV;
AmdParamStruct.AllocationMethod = PostMemDram;
AmdParamStruct.StdHeader.AltImageBasePtr = 0;
AmdParamStruct.StdHeader.CalloutPtr = &GetBiosCallout;
AmdParamStruct.StdHeader.Func = 0;
AmdParamStruct.StdHeader.ImageBasePtr = 0;
status = AmdCreateStruct(&AmdParamStruct); status = AmdCreateStruct(&AmdParamStruct);
EnvParam = (AMD_ENV_PARAMS *)AmdParamStruct.NewStructPtr; EnvParams = (AMD_ENV_PARAMS *)AmdParamStruct.NewStructPtr;
SetFchEnvParams(&EnvParam->FchInterface); SetFchEnvParams(&EnvParams->FchInterface);
SetNbEnvParams(&EnvParam->GnbEnvConfiguration); SetNbEnvParams(&EnvParams->GnbEnvConfiguration);
timestamp_add_now(TS_AGESA_INIT_ENV_START); timestamp_add_now(TS_AGESA_INIT_ENV_START);
status = AmdInitEnv (EnvParam); status = AmdInitEnv(EnvParams);
timestamp_add_now(TS_AGESA_INIT_ENV_DONE); timestamp_add_now(TS_AGESA_INIT_ENV_DONE);
if (status != AGESA_SUCCESS) if (status != AGESA_SUCCESS)
agesawrapper_readeventlog(EnvParam->StdHeader.HeapStatus); agesawrapper_readeventlog(EnvParams->StdHeader.HeapStatus);
/*
* Initialize Subordinate Bus Number and Secondary Bus Number
* In platform BIOS this address is allocated by PCI enumeration code
* Modify D1F0x18
*/
return status; return status;
} }
@@ -284,29 +260,24 @@ VOID* agesawrapper_getlateinitptr (int pick)
AGESA_STATUS agesawrapper_amdinitmid(void) AGESA_STATUS agesawrapper_amdinitmid(void)
{ {
AGESA_STATUS status; AGESA_STATUS status;
AMD_INTERFACE_PARAMS AmdParamStruct; AMD_INTERFACE_PARAMS AmdParamStruct = {
AMD_MID_PARAMS *MidParam; .AgesaFunctionName = AMD_INIT_MID,
.AllocationMethod = PostMemDram,
.StdHeader.CalloutPtr = &GetBiosCallout,
};
AMD_MID_PARAMS *MidParams;
/* Enable MMIO on AMD CPU Address Map Controller */ /* Enable MMIO on AMD CPU Address Map Controller */
amd_initcpuio(); amd_initcpuio();
memset(&AmdParamStruct, 0, sizeof(AmdParamStruct));
AmdParamStruct.AgesaFunctionName = AMD_INIT_MID;
AmdParamStruct.AllocationMethod = PostMemDram;
AmdParamStruct.StdHeader.AltImageBasePtr = 0;
AmdParamStruct.StdHeader.CalloutPtr = &GetBiosCallout;
AmdParamStruct.StdHeader.Func = 0;
AmdParamStruct.StdHeader.ImageBasePtr = 0;
AmdCreateStruct(&AmdParamStruct); AmdCreateStruct(&AmdParamStruct);
MidParam = (AMD_MID_PARAMS *)AmdParamStruct.NewStructPtr; MidParams = (AMD_MID_PARAMS *)AmdParamStruct.NewStructPtr;
SetFchMidParams(&MidParam->FchInterface); SetFchMidParams(&MidParams->FchInterface);
SetNbMidParams(&MidParam->GnbMidConfiguration); SetNbMidParams(&MidParams->GnbMidConfiguration);
timestamp_add_now(TS_AGESA_INIT_MID_START); timestamp_add_now(TS_AGESA_INIT_MID_START);
status = AmdInitMid ((AMD_MID_PARAMS *)AmdParamStruct.NewStructPtr); status = AmdInitMid(MidParams);
timestamp_add_now(TS_AGESA_INIT_MID_DONE); timestamp_add_now(TS_AGESA_INIT_MID_DONE);
if (status != AGESA_SUCCESS) if (status != AGESA_SUCCESS)
@@ -319,61 +290,51 @@ AGESA_STATUS agesawrapper_amdinitmid(void)
AGESA_STATUS agesawrapper_amdinitlate(void) AGESA_STATUS agesawrapper_amdinitlate(void)
{ {
AGESA_STATUS Status; AGESA_STATUS Status;
AMD_INTERFACE_PARAMS AmdParamStruct; AMD_INTERFACE_PARAMS AmdParamStruct = {
AMD_LATE_PARAMS *AmdLateParams; .AgesaFunctionName = AMD_INIT_LATE,
.AllocationMethod = PostMemDram,
memset(&AmdParamStruct, 0, sizeof(AmdParamStruct)); .StdHeader.CalloutPtr = &GetBiosCallout,
};
AmdParamStruct.AgesaFunctionName = AMD_INIT_LATE; AMD_LATE_PARAMS *LateParams;
AmdParamStruct.AllocationMethod = PostMemDram;
AmdParamStruct.StdHeader.AltImageBasePtr = 0;
AmdParamStruct.StdHeader.CalloutPtr = &GetBiosCallout;
AmdParamStruct.StdHeader.HeapStatus = HEAP_SYSTEM_MEM;
AmdParamStruct.StdHeader.Func = 0;
AmdParamStruct.StdHeader.ImageBasePtr = 0;
/* /*
* NOTE: if not call amdcreatestruct, the initializer * NOTE: if not call amdcreatestruct, the initializer
* (AmdInitLateInitializer) would not be called. * (AmdInitLateInitializer) would not be called.
*/ */
AmdCreateStruct(&AmdParamStruct); AmdCreateStruct(&AmdParamStruct);
AmdLateParams = (AMD_LATE_PARAMS *)AmdParamStruct.NewStructPtr; LateParams = (AMD_LATE_PARAMS *)AmdParamStruct.NewStructPtr;
timestamp_add_now(TS_AGESA_INIT_LATE_START); timestamp_add_now(TS_AGESA_INIT_LATE_START);
Status = AmdInitLate(AmdLateParams); Status = AmdInitLate(LateParams);
timestamp_add_now(TS_AGESA_INIT_LATE_DONE); timestamp_add_now(TS_AGESA_INIT_LATE_DONE);
if (Status != AGESA_SUCCESS) { if (Status != AGESA_SUCCESS) {
agesawrapper_readeventlog(AmdLateParams->StdHeader.HeapStatus); agesawrapper_readeventlog(LateParams->StdHeader.HeapStatus);
ASSERT(Status == AGESA_SUCCESS); ASSERT(Status == AGESA_SUCCESS);
} }
DmiTable = AmdLateParams->DmiTable; DmiTable = LateParams->DmiTable;
AcpiPstate = AmdLateParams->AcpiPState; AcpiPstate = LateParams->AcpiPState;
AcpiWheaMce = AmdLateParams->AcpiWheaMce; AcpiWheaMce = LateParams->AcpiWheaMce;
AcpiWheaCmc = AmdLateParams->AcpiWheaCmc; AcpiWheaCmc = LateParams->AcpiWheaCmc;
AcpiAlib = AmdLateParams->AcpiAlib; AcpiAlib = LateParams->AcpiAlib;
AcpiIvrs = AmdLateParams->AcpiIvrs; AcpiIvrs = LateParams->AcpiIvrs;
AcpiCrat = AmdLateParams->AcpiCrat; AcpiCrat = LateParams->AcpiCrat;
printk(BIOS_DEBUG, "DmiTable:%x, AcpiPstatein: %x, AcpiSrat:%x," printk(BIOS_DEBUG, "DmiTable:%p, AcpiPstatein: %p, AcpiSrat:%p,"
"AcpiSlit:%x, Mce:%x, Cmc:%x," "AcpiSlit:%p, Mce:%p, Cmc:%p,"
"Alib:%x, AcpiIvrs:%x in %s\n", "Alib:%p, AcpiIvrs:%p in %s\n",
(unsigned int)DmiTable, (unsigned int)AcpiPstate, DmiTable, AcpiPstate, AcpiSrat,
(unsigned int)AcpiSrat, (unsigned int)AcpiSlit, AcpiSlit, AcpiWheaMce, AcpiWheaCmc,
(unsigned int)AcpiWheaMce, (unsigned int)AcpiWheaCmc, AcpiAlib, AcpiIvrs, __func__);
(unsigned int)AcpiAlib, (unsigned int)AcpiIvrs, __func__);
/* AmdReleaseStruct (&AmdParamStruct); */ /* AmdReleaseStruct (&AmdParamStruct); */
return Status; return Status;
} }
AGESA_STATUS agesawrapper_amdlaterunaptask ( AGESA_STATUS agesawrapper_amdlaterunaptask(UINT32 Func, UINTN Data,
UINT32 Func, VOID *ConfigPtr)
UINTN Data,
VOID *ConfigPtr
)
{ {
AGESA_STATUS Status; AGESA_STATUS Status;
AP_EXE_PARAMS ApExeParams; AP_EXE_PARAMS ApExeParams;