coreboot-gerrit July 2017

coreboot-gerrit@coreboot.org

1 participants
1657 discussions

Change in coreboot[master]: payloads/external/tianocore: Use consistent print statement syntax
by Logan Carlson (Code Review) 26 Jul '17

26 Jul '17

Logan Carlson has abandoned this change. ( https://review.coreboot.org/20783 ) Change subject: payloads/external/tianocore: Use consistent print statement syntax ...................................................................... Abandoned -- To view, visit https://review.coreboot.org/20783 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: abandon Gerrit-Change-Id: I6e2b71a5ddf117398d3f9f30981cc54c926c05a2 Gerrit-Change-Number: 20783 Gerrit-PatchSet: 1 Gerrit-Owner: Logan Carlson <logancarlson(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in coreboot[master]: payloads/external/tianocore: Use consistent print statement syntax
by build bot (Jenkins) (Code Review) 26 Jul '17

26 Jul '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20783 ) Change subject: payloads/external/tianocore: Use consistent print statement syntax ...................................................................... Patch Set 1: Verified-1 Build Failed https://qa.coreboot.org/job/coreboot-gerrit/57868/ : FAILURE https://qa.coreboot.org/job/coreboot-checkpatch/13315/ : SUCCESS -- To view, visit https://review.coreboot.org/20783 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6e2b71a5ddf117398d3f9f30981cc54c926c05a2 Gerrit-Change-Number: 20783 Gerrit-PatchSet: 1 Gerrit-Owner: Logan Carlson <logancarlson(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 26 Jul 2017 20:14:05 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: payloads/external/tianocore: Add and apply patches for tianocore
by build bot (Jenkins) (Code Review) 26 Jul '17

26 Jul '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20639 ) Change subject: payloads/external/tianocore: Add and apply patches for tianocore ...................................................................... Patch Set 7: Verified-1 Build Failed https://qa.coreboot.org/job/coreboot-gerrit/57867/ : FAILURE https://qa.coreboot.org/job/coreboot-checkpatch/13314/ : SUCCESS -- To view, visit https://review.coreboot.org/20639 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4eaa602418995a68753b1ed13af8c6374eaa312f Gerrit-Change-Number: 20639 Gerrit-PatchSet: 7 Gerrit-Owner: Evelyn Huang <evhuang(a)google.com> Gerrit-Reviewer: Evelyn Huang <evhuang(a)google.com> Gerrit-Reviewer: Logan Carlson <logancarlson(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Prabal Saha <coolstarorganization(a)gmail.com> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 26 Jul 2017 20:13:41 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: payloads/external/tianocore: Use consistent print statement syntax
by Logan Carlson (Code Review) 26 Jul '17

26 Jul '17

Logan Carlson has uploaded this change for review. ( https://review.coreboot.org/20783 Change subject: payloads/external/tianocore: Use consistent print statement syntax ...................................................................... payloads/external/tianocore: Use consistent print statement syntax Change occurences of printf statements to echo statements to improve consistency. Change-Id: I6e2b71a5ddf117398d3f9f30981cc54c926c05a2 Signed-off-by: Logan Carlson <logancarlson(a)google.com> --- M payloads/external/tianocore/Makefile 1 file changed, 7 insertions(+), 7 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/83/20783/1 diff --git a/payloads/external/tianocore/Makefile b/payloads/external/tianocore/Makefile index fdca95c..f2a507c 100644 --- a/payloads/external/tianocore/Makefile +++ b/payloads/external/tianocore/Makefile @@ -56,19 +56,19 @@ echo "Applying $$patch"; \ cd $(project_dir); \ git am --keep-cr $$patch || \ - ( printf " Error when applying patches.\n"; git am --abort; exit 1; ); \ + ( echo " Error when applying patches."; git am --abort; exit 1; ); \ done checktools: - printf "Checking uuid-dev..." + echo "Checking uuid-dev..." echo "#include <uuid/uuid.h>" > libtest.c echo "int main(int argc, char **argv) { (void) argc; (void) argv; return 0; }" >> libtest.c - $(HOSTCC) $(HOSTCCFLAGS) libtest.c -o libtest >/dev/null 2>&1 && printf " found uuid-dev.\n" || \ - ( printf " Not found.\n"; echo "ERROR: please_install uuid-dev (uuid-devel)"; exit 1 ) + $(HOSTCC) $(HOSTCCFLAGS) libtest.c -o libtest >/dev/null 2>&1 && echo " found uuid-dev." || \ + ( echo " Not found."; echo "ERROR: please_install uuid-dev (uuid-devel)"; exit 1 ) rm -rf libtest.c libtest - printf "Checking nasm..." - type nasm > /dev/null 2>&1 && printf " found nasm.\n" || \ - ( printf " Not found.\n"; echo "Error: Please install nasm."; exit 1 ) + echo "Checking nasm..." + type nasm > /dev/null 2>&1 && echo " found nasm." || \ + ( echo " Not found."; echo "Error: Please install nasm."; exit 1 ) config: checkout checktools unset CC; $(MAKE) -C $(project_dir)/BaseTools -- To view, visit https://review.coreboot.org/20783 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I6e2b71a5ddf117398d3f9f30981cc54c926c05a2 Gerrit-Change-Number: 20783 Gerrit-PatchSet: 1 Gerrit-Owner: Logan Carlson <logancarlson(a)google.com>

1 0

Change in coreboot[master]: mainboard/intel/glkrvp: Add support for GLKRVP
by build bot (Jenkins) (Code Review) 26 Jul '17

26 Jul '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/19604 ) Change subject: mainboard/intel/glkrvp: Add support for GLKRVP ...................................................................... Patch Set 78: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/57864/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13311/ : SUCCESS -- To view, visit https://review.coreboot.org/19604 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iab688aca6a4f5c5e32801215ba3a1a440e50fbef Gerrit-Change-Number: 19604 Gerrit-PatchSet: 78 Gerrit-Owner: Hannah Williams <hannah.williams(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Abhay Kumar <abhay.kumar(a)intel.com> Gerrit-Reviewer: Andrey Petrov <andrey.petrov(a)gmail.com> Gerrit-Reviewer: Bora Guvendik <bora.guvendik(a)intel.com> Gerrit-Reviewer: Brandon Breitenstein <brandon.breitenstein(a)intel.com> Gerrit-Reviewer: Brenton Dong <brenton.m.dong(a)intel.com> Gerrit-Reviewer: Cole Nelson <colex.nelson(a)intel.com> Gerrit-Reviewer: Han Lim Ng <nhlhanlim93(a)gmail.com> Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Ravishankar Sarawadi <ravishankar.sarawadi(a)intel.com> Gerrit-Reviewer: Sathyanarayana Nujella <sathyanarayana.nujella(a)intel.com> Gerrit-Reviewer: Shaunak Saha <shaunak.saha(a)intel.com> Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com> Gerrit-Reviewer: Sumeet R Pawnikar <sumeet.r.pawnikar(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 26 Jul 2017 20:01:57 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: payloads/external/tianocore: build UEFI payload
by build bot (Jenkins) (Code Review) 26 Jul '17

26 Jul '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/15057 ) Change subject: payloads/external/tianocore: build UEFI payload ...................................................................... Patch Set 28: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/57863/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13310/ : SUCCESS -- To view, visit https://review.coreboot.org/15057 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9719ca5c39fccb856dfe096d449760a937d51fd1 Gerrit-Change-Number: 15057 Gerrit-PatchSet: 28 Gerrit-Owner: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Evelyn Huang <evhuang(a)google.com> Gerrit-Reviewer: Lee Leahy <leroy.p.leahy(a)intel.com> Gerrit-Reviewer: Logan Carlson <logancarlson(a)google.com> Gerrit-Reviewer: Marc Jones <marc(a)marcjonesconsulting.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com> Gerrit-Reviewer: Maurice Ma Gerrit-Reviewer: Maurice Ma <mauricexma(a)gmail.com> Gerrit-Reviewer: Naresh Solanki <naresh.solanki(a)intel.com> Gerrit-Reviewer: Pablo Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: Philippe Mathieu-Daudé <f4bug(a)amsat.org> Gerrit-Reviewer: Prabal Saha <coolstarorganization(a)gmail.com> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 26 Jul 2017 19:23:22 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: util/inteltool: Clean up checkpatch.pl warnings
by build bot (Jenkins) (Code Review) 26 Jul '17

26 Jul '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20771 ) Change subject: util/inteltool: Clean up checkpatch.pl warnings ...................................................................... Patch Set 2: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/57862/ : SUCCESS -- To view, visit https://review.coreboot.org/20771 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id5a7dc3a9672266b66d7f46db2ff087b98fd174d Gerrit-Change-Number: 20771 Gerrit-PatchSet: 2 Gerrit-Owner: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 26 Jul 2017 19:00:20 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: payloads/external/tianocore: Add and apply patches for tianocore
by build bot (Jenkins) (Code Review) 26 Jul '17

26 Jul '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20639 ) Change subject: payloads/external/tianocore: Add and apply patches for tianocore ...................................................................... Patch Set 6: Verified-1 Build Failed https://qa.coreboot.org/job/coreboot-gerrit/57859/ : FAILURE https://qa.coreboot.org/job/coreboot-checkpatch/13307/ : SUCCESS -- To view, visit https://review.coreboot.org/20639 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4eaa602418995a68753b1ed13af8c6374eaa312f Gerrit-Change-Number: 20639 Gerrit-PatchSet: 6 Gerrit-Owner: Evelyn Huang <evhuang(a)google.com> Gerrit-Reviewer: Evelyn Huang <evhuang(a)google.com> Gerrit-Reviewer: Logan Carlson <logancarlson(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Prabal Saha <coolstarorganization(a)gmail.com> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 26 Jul 2017 17:46:45 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: payloads/external/iPXE: Add more Kconfig options
by build bot (Jenkins) (Code Review) 26 Jul '17

26 Jul '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20782 ) Change subject: payloads/external/iPXE: Add more Kconfig options ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/57857/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13305/ : SUCCESS -- To view, visit https://review.coreboot.org/20782 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie1083d8571d9d1f1c7c71659fb6ff0de2eecad0e Gerrit-Change-Number: 20782 Gerrit-PatchSet: 1 Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 26 Jul 2017 17:46:31 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: payloads/external/tianocore: Add and apply patches for tianocore
by build bot (Jenkins) (Code Review) 26 Jul '17

26 Jul '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20639 ) Change subject: payloads/external/tianocore: Add and apply patches for tianocore ...................................................................... Patch Set 5: Verified-1 Build Failed https://qa.coreboot.org/job/coreboot-gerrit/57858/ : FAILURE https://qa.coreboot.org/job/coreboot-checkpatch/13306/ : SUCCESS -- To view, visit https://review.coreboot.org/20639 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4eaa602418995a68753b1ed13af8c6374eaa312f Gerrit-Change-Number: 20639 Gerrit-PatchSet: 5 Gerrit-Owner: Evelyn Huang <evhuang(a)google.com> Gerrit-Reviewer: Evelyn Huang <evhuang(a)google.com> Gerrit-Reviewer: Logan Carlson <logancarlson(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Prabal Saha <coolstarorganization(a)gmail.com> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 26 Jul 2017 17:44:10 +0000 Gerrit-HasComments: No

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit July 2017