diff options
| author | Richard Spiegel <richard.spiegel@silverbackltd.com> | 2017-12-11 16:23:58 -0700 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-01-02 20:43:07 +0000 |
| commit | fc511277a57857fe5fd41bf0d877d1dd17f3b92f (patch) | |
| tree | 2891ae152f7bd1be34c0e12e4bd80d005167d020 /src/soc/amd/common/block/pi/agesawrapper.c | |
| parent | 5abe2d14721f1a2292b330fb9c6d52d405e092bf (diff) | |
| download | coreboot-fc511277a57857fe5fd41bf0d877d1dd17f3b92f.tar.xz | |
soc/amd/common/block/pi: Format files to standards
Files agesawrapper.c and heapmanager.c have several non-conformity with
coreboot standards, including lines longer than 80 characters, use of
"} else {" after a return and wrong comment block formatting. Fix all
such issues, so that it passes commit tests.
BUG=b:69262110
TEST=Build with no error gardenia and kahlee (no code change, just file
formatting).
Change-Id: Iefe741cd62bc41a7975c3dd10ac9355352de3abb
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/22823
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/soc/amd/common/block/pi/agesawrapper.c')
| -rw-r--r-- | src/soc/amd/common/block/pi/agesawrapper.c | 21 |
1 files changed, 13 insertions, 8 deletions
diff --git a/src/soc/amd/common/block/pi/agesawrapper.c b/src/soc/amd/common/block/pi/agesawrapper.c index 7fb5485fe5..f2327f1679 100644 --- a/src/soc/amd/common/block/pi/agesawrapper.c +++ b/src/soc/amd/common/block/pi/agesawrapper.c @@ -244,10 +244,11 @@ AGESA_STATUS agesawrapper_amdinitenv(void) if (status != AGESA_SUCCESS) agesawrapper_readeventlog(EnvParam->StdHeader.HeapStatus); - /* Initialize Subordinate Bus Number and Secondary Bus Number + /* + * Initialize Subordinate Bus Number and Secondary Bus Number * In platform BIOS this address is allocated by PCI enumeration code - Modify D1F0x18 - */ + * Modify D1F0x18 + */ return status; } @@ -329,7 +330,10 @@ AGESA_STATUS agesawrapper_amdinitlate(void) AmdParamStruct.StdHeader.Func = 0; AmdParamStruct.StdHeader.ImageBasePtr = 0; - /* NOTE: if not call amdcreatestruct, the initializer(AmdInitLateInitializer) would not be called */ + /* + * NOTE: if not call amdcreatestruct, the initializer + * (AmdInitLateInitializer) would not be called. + */ AmdCreateStruct(&AmdParamStruct); AmdLateParams = (AMD_LATE_PARAMS *)AmdParamStruct.NewStructPtr; @@ -354,8 +358,9 @@ AGESA_STATUS agesawrapper_amdinitlate(void) printk(BIOS_DEBUG, "DmiTable:%x, AcpiPstatein: %x, AcpiSrat:%x," "AcpiSlit:%x, Mce:%x, Cmc:%x," "Alib:%x, AcpiIvrs:%x in %s\n", - (unsigned int)DmiTable, (unsigned int)AcpiPstate, (unsigned int)AcpiSrat, - (unsigned int)AcpiSlit, (unsigned int)AcpiWheaMce, (unsigned int)AcpiWheaCmc, + (unsigned int)DmiTable, (unsigned int)AcpiPstate, + (unsigned int)AcpiSrat, (unsigned int)AcpiSlit, + (unsigned int)AcpiWheaMce, (unsigned int)AcpiWheaCmc, (unsigned int)AcpiAlib, (unsigned int)AcpiIvrs, __func__); /* AmdReleaseStruct (&AmdParamStruct); */ @@ -420,8 +425,8 @@ static int agesa_locate_stage_file(const char *name, struct region_device *rdev) const void *agesawrapper_locate_module (const CHAR8 name[8]) { - const void* agesa; - const AMD_IMAGE_HEADER* image; + const void *agesa; + const AMD_IMAGE_HEADER *image; struct region_device rdev; size_t file_size; const char *fname = CONFIG_AGESA_CBFS_NAME; |