diff options
author | Marshall Dawson <marshalldawson3rd@gmail.com> | 2017-12-13 20:01:59 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2018-02-07 16:37:39 +0000 |
commit | 857a38752091ef8adc9926a58beba1f52baa181f (patch) | |
tree | d8de0d4bcf84629010f61b44d9269f82d80ad5c4 /3rdparty | |
parent | f3c57a7c28a3c66d2ade3e2b3108fd58b60c2a7c (diff) | |
download | coreboot-857a38752091ef8adc9926a58beba1f52baa181f.tar.xz |
soc/amd/common: Improve misc. formatting in AGESA wrapper
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>
Diffstat (limited to '3rdparty')
0 files changed, 0 insertions, 0 deletions