[coreboot-gerrit] Change in coreboot[master]: mb/amg: Get rid of whitespace before tab
build bot (Jenkins) (Code Review)
gerrit at coreboot.org
Mon May 28 13:34:39 CEST 2018
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/26607 )
Change subject: mb/amg: Get rid of whitespace before tab
......................................................................
Patch Set 1:
(7 comments)
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/bimini_fam10/romstage.c
File src/mainboard/amd/bimini_fam10/romstage.c:
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/bimini_fam10/romstage.c@83
PS1, Line 83: inb(0x80); /* Wait sometime before post to port80, otherwise reset was needed. */
line over 80 characters
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/inagua/mainboard.c
File src/mainboard/amd/inagua/mainboard.c:
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/inagua/mainboard.c@20
PS1, Line 20: #include "SBPLATFORM.h" /* Platfrom Specific Definitions */
'Platfrom' may be misspelled - perhaps 'Platform'?
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/parmer/buildOpts.c
File src/mainboard/amd/parmer/buildOpts.c:
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/parmer/buildOpts.c@158
PS1, Line 158: //#define BLDCFG_UMA_ALLOCATION_SIZE 0x1000//0x1800//0x1000 /* (1000 << 16) = 256M*/
line over 80 characters
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/south_station/mainboard.c
File src/mainboard/amd/south_station/mainboard.c:
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/south_station/mainboard.c@21
PS1, Line 21: #include "SBPLATFORM.h" /* Platfrom Specific Definitions */
'Platfrom' may be misspelled - perhaps 'Platform'?
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/thatcher/buildOpts.c
File src/mainboard/amd/thatcher/buildOpts.c:
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/thatcher/buildOpts.c@158
PS1, Line 158: //#define BLDCFG_UMA_ALLOCATION_SIZE 0x1000//0x1800//0x1000 /* (1000 << 16) = 256M*/
line over 80 characters
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/tilapia_fam10/mainboard.c
File src/mainboard/amd/tilapia_fam10/mainboard.c:
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/tilapia_fam10/mainboard.c@168
PS1, Line 168: /* When the gpio40 is configured as GPIO, this will represent the output value*/
line over 80 characters
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/union_station/mainboard.c
File src/mainboard/amd/union_station/mainboard.c:
https://review.coreboot.org/#/c/26607/1/src/mainboard/amd/union_station/mainboard.c@20
PS1, Line 20: #include "SBPLATFORM.h" /* Platfrom Specific Definitions */
'Platfrom' may be misspelled - perhaps 'Platform'?
--
To view, visit https://review.coreboot.org/26607
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3acc5b0c3895459a7aba53a422a978e31f652aa9
Gerrit-Change-Number: 26607
Gerrit-PatchSet: 1
Gerrit-Owner: Elyes HAOUAS <ehaouas at noos.fr>
Gerrit-CC: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-Comment-Date: Mon, 28 May 2018 11:34:39 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180528/81b22dee/attachment.html>
More information about the coreboot-gerrit
mailing list