[coreboot-gerrit] Change in coreboot[master]: soc/amd/common: Fix most checkpatch errors

Marshall Dawson (Code Review) gerrit at coreboot.org
Sat Jun 17 14:22:03 CEST 2017


Hello build bot (Jenkins), 

I'd like you to reexamine a change. Please visit

    https://review.coreboot.org/20249

to look at the new patch set (#2).

Change subject: soc/amd/common: Fix most checkpatch errors
......................................................................

soc/amd/common: Fix most checkpatch errors

Correct the majority of reported errors and mark most of the
remaining ones as todo.  Some of the lines requiring a >80
break are indented too much currently.

Changes to agesawrapper.c cause the build to change, so this
file is also left as-is.  Make hex values consistenly lower-case.

BUG=chrome-os-partner:622407746

Change-Id: I0464f0cafac4ee67edc95d377dcf7aab9a90c66b
Signed-off-by: Marshall Dawson <marshalldawson3rd at gmail.com>
---
M src/soc/amd/common/BiosCallOuts.h
M src/soc/amd/common/agesawrapper.h
M src/soc/amd/common/agesawrapper_call.h
M src/soc/amd/common/amd_pci_util.c
M src/soc/amd/common/amd_pci_util.h
M src/soc/amd/common/cache_as_ram.inc
M src/soc/amd/common/def_callouts.c
M src/soc/amd/common/dimmSpd.h
M src/soc/amd/common/heapmanager.c
M src/soc/amd/common/spi.c
10 files changed, 249 insertions(+), 177 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/49/20249/2
-- 
To view, visit https://review.coreboot.org/20249
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I0464f0cafac4ee67edc95d377dcf7aab9a90c66b
Gerrit-Change-Number: 20249
Gerrit-PatchSet: 2
Gerrit-Owner: Marshall Dawson <marshalldawson3rd at gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>



More information about the coreboot-gerrit mailing list