[coreboot-gerrit] Change in coreboot[master]: Signed-off-by: Elyes Haouas <[e-mail]> Change-Id: I80f1b65071c...

[build bot (Jenkins) (Code Review)]

Attention is currently required from: Raul Rangel, Patrick Rudolph, Mariusz Szafrański, Jonathan Zhang, Angel Pons, Arthur Heymans, Andrey Petrov, Jason Glenesk, Anjaneya "Reddy" Chagam, Damien Zammit, Marshall Dawson, Johnny Lin, Christian Walter, Tim Wawrzynczak, Suresh Bellampalli, Vanessa Eusebio, Michal Motyl, Fred Reitberger, Alexander Couzens, Werner Zeh, Tim Chu, Felix Held.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62619 )

Change subject: Signed-off-by: Elyes Haouas  Change-Id: I80f1b65071c2c7dec2c73a208bf4a9f5d11a6ccb
......................................................................

Patch Set 1:

(6 comments)

File src/include/device/pci_ids.h:

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-143234):
https://review.coreboot.org/c/coreboot/+/62619/comment/8eab3486_199487bd 
PS1, Line 2117: #define PCI_VID_AKS			0x416c
'AKS' may be misspelled - perhaps 'ASK'?

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-143234):
https://review.coreboot.org/c/coreboot/+/62619/comment/88f643ab_fb0c280c 
PS1, Line 2118: #define PCI_DID_AKS_ALADDINCARD		0x0100
'AKS' may be misspelled - perhaps 'ASK'?

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-143234):
https://review.coreboot.org/c/coreboot/+/62619/comment/b447518c_d2098687 
PS1, Line 2119: #define PCI_DID_AKS_CPC			0x0200
'AKS' may be misspelled - perhaps 'ASK'?

File src/soc/intel/skylake/vr_config.c:

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-143234):
https://review.coreboot.org/c/coreboot/+/62619/comment/aa51e8e8_cb30a78f 
PS1, Line 215: 		         (igd_id == PCI_DID_INTEL_KBL_GT3E_SULTM_2)) {
code indent should use tabs where possible

File src/southbridge/amd/agesa/hudson/hudson.c:

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-143234):
https://review.coreboot.org/c/coreboot/+/62619/comment/4d7dbe3d_67a78216 
PS1, Line 66: 			else if (sd_device_id == PCI_DID_AMD_YANGTZE_SD) {
else should follow close brace '}'

File src/southbridge/amd/pi/hudson/hudson.c:

Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-143234):
https://review.coreboot.org/c/coreboot/+/62619/comment/0d0697c9_de31d9d7 
PS1, Line 41: 			else if (sd_device_id == PCI_DID_AMD_YANGTZE_SD) {
else should follow close brace '}'