[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