coreboot-gerrit August 2017

coreboot-gerrit@coreboot.org

1 participants
1999 discussions

Change in coreboot[master]: nb/amd/amdfam10/amdfam10.h: Clean up header
by build bot (Jenkins) (Code Review) 12 Aug '17

12 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20974 ) Change subject: nb/amd/amdfam10/amdfam10.h: Clean up header ...................................................................... Patch Set 3: Verified-1 Build Unstable https://qa.coreboot.org/job/coreboot-gerrit/58484/ : UNSTABLE https://qa.coreboot.org/job/coreboot-checkpatch/13886/ : SUCCESS -- To view, visit https://review.coreboot.org/20974 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3df97a038cf265423cb7019d1639a3d47d4871f7 Gerrit-Change-Number: 20974 Gerrit-PatchSet: 3 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 12 Aug 2017 20:06:55 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: nb/amd/amdfam10/amdfam10.h: Clean up header
by Arthur Heymans (Code Review) 12 Aug '17

12 Aug '17

Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/20974 to look at the new patch set (#3). Change subject: nb/amd/amdfam10/amdfam10.h: Clean up header ...................................................................... nb/amd/amdfam10/amdfam10.h: Clean up header Remove superfluous includes in amdfam10.h header. Change-Id: I3df97a038cf265423cb7019d1639a3d47d4871f7 Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz> --- M src/mainboard/advansus/a785e-i/romstage.c M src/mainboard/amd/bimini_fam10/romstage.c M src/mainboard/amd/mahogany_fam10/romstage.c M src/mainboard/amd/serengeti_cheetah_fam10/romstage.c M src/mainboard/amd/tilapia_fam10/romstage.c M src/mainboard/asus/kcma-d8/romstage.c M src/mainboard/asus/kfsn4-dre/romstage.c M src/mainboard/asus/kgpe-d16/romstage.c M src/mainboard/asus/m4a78-em/romstage.c M src/mainboard/asus/m4a785-m/romstage.c M src/mainboard/asus/m5a88-v/romstage.c M src/mainboard/avalue/eax-785e/romstage.c M src/mainboard/gigabyte/ma785gm/romstage.c M src/mainboard/gigabyte/ma785gmt/romstage.c M src/mainboard/gigabyte/ma78gm/romstage.c M src/mainboard/hp/dl165_g6_fam10/romstage.c M src/mainboard/iei/kino-780am2-fam10/romstage.c M src/mainboard/jetway/pa78vm5/romstage.c M src/mainboard/msi/ms9652_fam10/romstage.c M src/mainboard/supermicro/h8dmr_fam10/romstage.c M src/mainboard/supermicro/h8qme_fam10/romstage.c M src/mainboard/supermicro/h8scm_fam10/romstage.c M src/mainboard/tyan/s2912_fam10/romstage.c M src/northbridge/amd/amdfam10/Makefile.inc M src/northbridge/amd/amdfam10/amdfam10.h M src/northbridge/amd/amdfam10/inline_helper.c 26 files changed, 29 insertions(+), 9 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/74/20974/3 -- To view, visit https://review.coreboot.org/20974 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I3df97a038cf265423cb7019d1639a3d47d4871f7 Gerrit-Change-Number: 20974 Gerrit-PatchSet: 3 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in coreboot[master]: nb/amd/mct_ddr3/mct_d.c: Break out of allow_config_restore loop
by build bot (Jenkins) (Code Review) 12 Aug '17

12 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20971 ) Change subject: nb/amd/mct_ddr3/mct_d.c: Break out of allow_config_restore loop ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58479/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13881/ : SUCCESS -- To view, visit https://review.coreboot.org/20971 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie669d1bc38411e40ab95c28b17ee8818874dc741 Gerrit-Change-Number: 20971 Gerrit-PatchSet: 3 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Raptor Engineering Automated Test Stand <noreply(a)raptorengineeringinc.com> Gerrit-Reviewer: Timothy Pearson <tpearson(a)raptorengineering.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 12 Aug 2017 19:59:52 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: [WIP]nb/amd/amdfam10/amdfam10.h: Clean up header
by build bot (Jenkins) (Code Review) 12 Aug '17

12 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20974 ) Change subject: [WIP]nb/amd/amdfam10/amdfam10.h: Clean up header ...................................................................... Patch Set 2: Verified-1 Build Unstable https://qa.coreboot.org/job/coreboot-gerrit/58483/ : UNSTABLE https://qa.coreboot.org/job/coreboot-checkpatch/13885/ : SUCCESS -- To view, visit https://review.coreboot.org/20974 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3df97a038cf265423cb7019d1639a3d47d4871f7 Gerrit-Change-Number: 20974 Gerrit-PatchSet: 2 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 12 Aug 2017 19:56:40 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: cpu/amd/fam10: Link fidvid.c
by build bot (Jenkins) (Code Review) 12 Aug '17

12 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20973 ) Change subject: cpu/amd/fam10: Link fidvid.c ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58482/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13883/ : SUCCESS -- To view, visit https://review.coreboot.org/20973 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I724fa6c3ba0cc4fdfb11cc28fd54611c5959f2fb Gerrit-Change-Number: 20973 Gerrit-PatchSet: 2 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 12 Aug 2017 19:54:02 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: [WIP]nb/amd/amdfam10/amdfam10.h: Clean up header
by build bot (Jenkins) (Code Review) 12 Aug '17

12 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20974 ) Change subject: [WIP]nb/amd/amdfam10/amdfam10.h: Clean up header ...................................................................... Patch Set 1: Verified-1 Build Failed https://qa.coreboot.org/job/coreboot-gerrit/58480/ : ABORTED https://qa.coreboot.org/job/coreboot-checkpatch/13882/ : SUCCESS -- To view, visit https://review.coreboot.org/20974 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3df97a038cf265423cb7019d1639a3d47d4871f7 Gerrit-Change-Number: 20974 Gerrit-PatchSet: 1 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 12 Aug 2017 19:48:51 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: nb/amd/mct: Don't include all needed headers in mct_d.h
by Arthur Heymans (Code Review) 12 Aug '17

12 Aug '17

Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/20972 to look at the new patch set (#3). Change subject: nb/amd/mct: Don't include all needed headers in mct_d.h ...................................................................... nb/amd/mct: Don't include all needed headers in mct_d.h Include header that the *.c files need in the *.c file itself. Change-Id: I454eb9f00c8ffebd7a48bf9cb65d012b3fc335f2 Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz> --- M src/northbridge/amd/amdmct/mct/mct_d.c M src/northbridge/amd/amdmct/mct/mct_d.h M src/northbridge/amd/amdmct/mct/mctardk3.c M src/northbridge/amd/amdmct/mct/mctardk4.c M src/northbridge/amd/amdmct/mct/mctcsi_d.c M src/northbridge/amd/amdmct/mct/mctdqs_d.c M src/northbridge/amd/amdmct/mct/mctmtr_d.c M src/northbridge/amd/amdmct/mct/mctndi_d.c M src/northbridge/amd/amdmct/mct/mctpro_d.c M src/northbridge/amd/amdmct/mct/mctsrc.c M src/northbridge/amd/amdmct/mct/mcttmrl.c 11 files changed, 29 insertions(+), 11 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/72/20972/3 -- To view, visit https://review.coreboot.org/20972 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I454eb9f00c8ffebd7a48bf9cb65d012b3fc335f2 Gerrit-Change-Number: 20972 Gerrit-PatchSet: 3 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in coreboot[master]: nb/amd/mct: Don't include all needed headers in mct_d.h
by build bot (Jenkins) (Code Review) 12 Aug '17

12 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20972 ) Change subject: nb/amd/mct: Don't include all needed headers in mct_d.h ...................................................................... Patch Set 2: Build Unstable https://qa.coreboot.org/job/coreboot-gerrit/58478/ : UNSTABLE https://qa.coreboot.org/job/coreboot-checkpatch/13880/ : SUCCESS -- To view, visit https://review.coreboot.org/20972 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I454eb9f00c8ffebd7a48bf9cb65d012b3fc335f2 Gerrit-Change-Number: 20972 Gerrit-PatchSet: 2 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 12 Aug 2017 19:46:31 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: cpu/amd/fam10: Link fidvid.c
by build bot (Jenkins) (Code Review) 12 Aug '17

12 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20973 ) Change subject: cpu/amd/fam10: Link fidvid.c ...................................................................... Patch Set 1: Verified-1 Build Unstable https://qa.coreboot.org/job/coreboot-gerrit/58477/ : UNSTABLE https://qa.coreboot.org/job/coreboot-checkpatch/13879/ : SUCCESS -- To view, visit https://review.coreboot.org/20973 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I724fa6c3ba0cc4fdfb11cc28fd54611c5959f2fb Gerrit-Change-Number: 20973 Gerrit-PatchSet: 1 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 12 Aug 2017 19:44:50 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: nb/amd/mct_ddr3/mct_d.c: Break out of allow_config_restore loop
by Raptor Engineering Automated Test Stand (Code Review) 12 Aug '17

12 Aug '17

Raptor Engineering Automated Test Stand has posted comments on this change. ( https://review.coreboot.org/20971 ) Change subject: nb/amd/mct_ddr3/mct_d.c: Break out of allow_config_restore loop ...................................................................... Patch Set 1: Verified-1 ASUS KFSN4-DRE: Build FAILED. Test ABORTED. Build log: https://testlab.coreboot.org/raptor/1502563986-0-20971,1.log.bz2 ASUS KFSN4-DRE: BUILD_FAILURE QEMU x86_64 Q35: Boot successful. Boot log: https://testlab.coreboot.org/raptor/1502564376-1-20971,1.log.bz2 ASUS KGPE-D16: Test FAILED. Boot log: https://testlab.coreboot.org/raptor/1502566851-3-20971,1.log.bz2 ASUS KGPE-D16: BOOT_FAILURE -- To view, visit https://review.coreboot.org/20971 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie669d1bc38411e40ab95c28b17ee8818874dc741 Gerrit-Change-Number: 20971 Gerrit-PatchSet: 1 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Raptor Engineering Automated Test Stand <noreply(a)raptorengineeringinc.com> Gerrit-Reviewer: Timothy Pearson <tpearson(a)raptorengineering.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sat, 12 Aug 2017 19:40:53 +0000 Gerrit-HasComments: No

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit August 2017