coreboot-gerrit September 2017

coreboot-gerrit@coreboot.org

1 participants
1960 discussions

Change in coreboot[master]: ifdtool: refactor region_name(s) and new_layout
by build bot (Jenkins) (Code Review) 03 Sep '17

03 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21315 ) Change subject: ifdtool: refactor region_name(s) and new_layout ...................................................................... Patch Set 5: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15033/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59763/ : SUCCESS -- To view, visit https://review.coreboot.org/21315 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia2b92c3f835f9849a243a48c37588c2dbe7449bf Gerrit-Change-Number: 21315 Gerrit-PatchSet: 5 Gerrit-Owner: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Nicola Corna <nicola(a)corna.info> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 03 Sep 2017 08:53:46 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: [WIP]nb/i945/raminit: Use common ddr2 decode functions
by build bot (Jenkins) (Code Review) 03 Sep '17

03 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/18305 ) Change subject: [WIP]nb/i945/raminit: Use common ddr2 decode functions ...................................................................... Patch Set 50: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15031/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59762/ : SUCCESS -- To view, visit https://review.coreboot.org/18305 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I97c93939d11807752797785dd88c70b43a236ee3 Gerrit-Change-Number: 18305 Gerrit-PatchSet: 50 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Reviewer: coreboot org <coreboot.org(a)gmail.com> Gerrit-Comment-Date: Sun, 03 Sep 2017 05:17:29 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: intelmetool: Add support for Sunrise Point-H
by build bot (Jenkins) (Code Review) 03 Sep '17

03 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21301 ) Change subject: intelmetool: Add support for Sunrise Point-H ...................................................................... Patch Set 4: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15030/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59761/ : SUCCESS -- To view, visit https://review.coreboot.org/21301 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I62f78fe5ca03bf70497939a12f0036bf247b2aa7 Gerrit-Change-Number: 21301 Gerrit-PatchSet: 4 Gerrit-Owner: Shawn C <citypw(a)gmail.com> Gerrit-Reviewer: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: Shawn C <citypw(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 03 Sep 2017 04:13:57 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: intelmetool: Add support for Sunrise Point-H
by build bot (Jenkins) (Code Review) 03 Sep '17

03 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21301 ) Change subject: intelmetool: Add support for Sunrise Point-H ...................................................................... Patch Set 3: Verified-1 Build Failed https://qa.coreboot.org/job/coreboot-gerrit/59760/ : ABORTED https://qa.coreboot.org/job/coreboot-checkpatch/15029/ : SUCCESS -- To view, visit https://review.coreboot.org/21301 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I62f78fe5ca03bf70497939a12f0036bf247b2aa7 Gerrit-Change-Number: 21301 Gerrit-PatchSet: 3 Gerrit-Owner: Shawn C <citypw(a)gmail.com> Gerrit-Reviewer: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 03 Sep 2017 04:07:58 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: intelmetool: Add support for Sunrise Point-H
by build bot (Jenkins) (Code Review) 03 Sep '17

03 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21301 ) Change subject: intelmetool: Add support for Sunrise Point-H ...................................................................... Patch Set 2: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/59759/ : SUCCESS -- To view, visit https://review.coreboot.org/21301 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I62f78fe5ca03bf70497939a12f0036bf247b2aa7 Gerrit-Change-Number: 21301 Gerrit-PatchSet: 2 Gerrit-Owner: Shawn C <citypw(a)gmail.com> Gerrit-Reviewer: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 03 Sep 2017 03:40:50 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: device/Kconfig: Make Ada dependent on GCC toolchain
by Nico Huber (Code Review) 03 Sep '17

03 Sep '17

Nico Huber has posted comments on this change. ( https://review.coreboot.org/21355 ) Change subject: device/Kconfig: Make Ada dependent on GCC toolchain ...................................................................... Patch Set 1: Code-Review-1 It's probably no problem at all to use a combination of Clang/GNAT. Is there any board with libgfxinit yet that compiles with Clang? I'd like to test it and untangle this. Should be only some changes to `toolchain.inc`. -- To view, visit https://review.coreboot.org/21355 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5db0353cdab4d24fd6832393901d8f47cf25849d Gerrit-Change-Number: 21355 Gerrit-PatchSet: 1 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 03 Sep 2017 01:09:27 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: cpu/x86/sipi_vector.S: Use correct op suffix
by build bot (Jenkins) (Code Review) 03 Sep '17

03 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21359 ) Change subject: cpu/x86/sipi_vector.S: Use correct op suffix ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15028/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59758/ : SUCCESS -- To view, visit https://review.coreboot.org/21359 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I82da5a9065b382e182dc7d502c7dca2fc717543b Gerrit-Change-Number: 21359 Gerrit-PatchSet: 1 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 03 Sep 2017 01:01:58 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: fsp1_0/fsp_util: Rename find_fsp to find_fsp_dummy
by Damien Zammit (Code Review) 03 Sep '17

03 Sep '17

Damien Zammit has posted comments on this change. ( https://review.coreboot.org/21353 ) Change subject: fsp1_0/fsp_util: Rename find_fsp to find_fsp_dummy ...................................................................... Patch Set 1: (1 comment) See inline comment https://review.coreboot.org/#/c/21353/1/src/drivers/intel/fsp1_0/fsp_util.c File src/drivers/intel/fsp1_0/fsp_util.c: https://review.coreboot.org/#/c/21353/1/src/drivers/intel/fsp1_0/fsp_util.c… PS1, Line 112: "find_fsp:\n\t" The problem is with this line here. The C function is called find_fsp, so you can't call an inline asm label the same thing. -- To view, visit https://review.coreboot.org/21353 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I84c152f9a580fdfc40e9f6e998d2d6484b7f47df Gerrit-Change-Number: 21353 Gerrit-PatchSet: 1 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 03 Sep 2017 01:00:55 +0000 Gerrit-HasComments: Yes

1 0

Change in coreboot[master]: armv7/Makefile.inc: Make cflags compiler agnostic
by build bot (Jenkins) (Code Review) 03 Sep '17

03 Sep '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21357 ) Change subject: armv7/Makefile.inc: Make cflags compiler agnostic ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/15026/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59756/ : SUCCESS -- To view, visit https://review.coreboot.org/21357 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I00b7ab5ad25349382bef3cc5001f3a3297f11ca0 Gerrit-Change-Number: 21357 Gerrit-PatchSet: 1 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 03 Sep 2017 00:56:43 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: abuild: Allow building with any toolchain
by Martin Roth (Code Review) 03 Sep '17

03 Sep '17

Martin Roth has posted comments on this change. ( https://review.coreboot.org/21352 ) Change subject: abuild: Allow building with any toolchain ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://review.coreboot.org/21352 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idd599b0d2c324ad88ba3c83cdf3b180eb6d1fc80 Gerrit-Change-Number: 21352 Gerrit-PatchSet: 1 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 03 Sep 2017 00:55:42 +0000 Gerrit-HasComments: No

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit September 2017