July 2017 - coreboot-gerrit - mail.coreboot.org

Change in coreboot[master]: src/include: Add guards on all header files
by build bot (Jenkins) (Code Review) July 28, 2017

July 28, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20734 ) Change subject: src/include: Add guards on all header files ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/57989/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13427/ : SUCCESS -- To view, visit https://review.coreboot.org/20734 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2d7d4e0b25f2cf3eef2040f89d5ebc711909cdd7 Gerrit-Change-Number: 20734 Gerrit-PatchSet: 2 Gerrit-Owner: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> 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-Comment-Date: Fri, 28 Jul 2017 21:05:15 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: Documentation: Create Lesson2.md
by build bot (Jenkins) (Code Review) July 28, 2017

July 28, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20212 ) Change subject: Documentation: Create Lesson2.md ...................................................................... Patch Set 8: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/57988/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13426/ : SUCCESS -- To view, visit https://review.coreboot.org/20212 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I756b273cf832fc014ba2077a5a4fe4d8009aae6d Gerrit-Change-Number: 20212 Gerrit-PatchSet: 8 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: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Simon Glass <sjg(a)chromium.org> Gerrit-Reviewer: Yves Roth <yves.r.roth(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 28 Jul 2017 20:45:58 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: lib/program.ld: Guard .id section placement
by build bot (Jenkins) (Code Review) July 28, 2017

July 28, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20810 ) Change subject: lib/program.ld: Guard .id section placement ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/57987/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13425/ : SUCCESS -- To view, visit https://review.coreboot.org/20810 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idc7cbd670ce4f75b7790ff8d95578683e355ba7e Gerrit-Change-Number: 20810 Gerrit-PatchSet: 1 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 28 Jul 2017 17:48:41 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: soc/intel/skylake: Use common/block/gpio
by build bot (Jenkins) (Code Review) July 28, 2017

July 28, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/19201 ) Change subject: soc/intel/skylake: Use common/block/gpio ...................................................................... Patch Set 38: Build Unstable https://qa.coreboot.org/job/coreboot-gerrit/57986/ : UNSTABLE https://qa.coreboot.org/job/coreboot-checkpatch/13424/ : SUCCESS -- To view, visit https://review.coreboot.org/19201 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I06e06dbcb6d0d6fe277dfad57b82aca51f94b099 Gerrit-Change-Number: 19201 Gerrit-PatchSet: 38 Gerrit-Owner: Hannah Williams <hannah.williams(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov(a)gmail.com> Gerrit-Reviewer: Barnali Sarkar <barnali.sarkar(a)intel.com> Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Shaunak Saha <shaunak.saha(a)intel.com> Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com> Gerrit-Reviewer: V Sowmya <v.sowmya(a)intel.com> Gerrit-Reviewer: Youness Alaoui <snifikino(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Reviewer: sowmya v <sowmyav235(a)gmail.com> Gerrit-Comment-Date: Fri, 28 Jul 2017 17:46:51 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: lib/program.ld: Guard .id section placement
by Nico Huber (Code Review) July 28, 2017

July 28, 2017

Nico Huber has uploaded this change for review. ( https://review.coreboot.org/20810 Change subject: lib/program.ld: Guard .id section placement ...................................................................... lib/program.ld: Guard .id section placement For x86, we place the .id section at 4GiB - CONFIG_ID_SECTION_OFFSET. To take effect, we have to guard the conflicting default placement in `program.ld`. Also, as we only include the .id section into the boot- block, guard it by ENV_BOOTBLOCK too. Change-Id: Idc7cbd670ce4f75b7790ff8d95578683e355ba7e Signed-off-by: Nico Huber <nico.h(a)gmx.de> --- M src/lib/program.ld 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/10/20810/1 diff --git a/src/lib/program.ld b/src/lib/program.ld index 1526aad..668b29b 100644 --- a/src/lib/program.ld +++ b/src/lib/program.ld @@ -34,7 +34,10 @@ *(.rom.data); *(.text._start); *(.text.stage_entry); +#if ENV_BOOTBLOCK && !(IS_ENABLED(CONFIG_ARCH_BOOTBLOCK_X86_32) || \ + IS_ENABLED(CONFIG_ARCH_BOOTBLOCK_X86_64)) KEEP(*(.id)); +#endif *(.text); *(.text.*); -- To view, visit https://review.coreboot.org/20810 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Idc7cbd670ce4f75b7790ff8d95578683e355ba7e Gerrit-Change-Number: 20810 Gerrit-PatchSet: 1 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>

1 0

Change in coreboot[master]: buildgcc: Update GCC to 7.1.0
by build bot (Jenkins) (Code Review) July 28, 2017

July 28, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20809 ) Change subject: buildgcc: Update GCC to 7.1.0 ...................................................................... Patch Set 1: Verified-1 Build Unstable https://qa.coreboot.org/job/coreboot-toolchain/344/ : UNSTABLE https://qa.coreboot.org/job/coreboot-gerrit/57977/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13416/ : SUCCESS -- To view, visit https://review.coreboot.org/20809 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I85ee0efc6497ba8a38f6295e30524c90c689d909 Gerrit-Change-Number: 20809 Gerrit-PatchSet: 1 Gerrit-Owner: Iru Cai <mytbk920423(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 28 Jul 2017 17:15:07 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: buildgcc: Update binutils to 2.29
by build bot (Jenkins) (Code Review) July 28, 2017

July 28, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20806 ) Change subject: buildgcc: Update binutils to 2.29 ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/57978/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13417/ : SUCCESS https://qa.coreboot.org/job/coreboot-toolchain/343/ : SUCCESS -- To view, visit https://review.coreboot.org/20806 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I25134dbadf07a2f0cb356c8ac8f2c612a957d176 Gerrit-Change-Number: 20806 Gerrit-PatchSet: 3 Gerrit-Owner: Iru Cai <mytbk920423(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 28 Jul 2017 17:05:32 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: Documentation: Create Lesson2.md
by build bot (Jenkins) (Code Review) July 28, 2017

July 28, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20212 ) Change subject: Documentation: Create Lesson2.md ...................................................................... Patch Set 7: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/57984/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13422/ : SUCCESS -- To view, visit https://review.coreboot.org/20212 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I756b273cf832fc014ba2077a5a4fe4d8009aae6d Gerrit-Change-Number: 20212 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: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Simon Glass <sjg(a)chromium.org> Gerrit-Reviewer: Yves Roth <yves.r.roth(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 28 Jul 2017 17:04:40 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: payloads/external/tianocore: build UEFI payload
by build bot (Jenkins) (Code Review) July 28, 2017

July 28, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/15057 ) Change subject: payloads/external/tianocore: build UEFI payload ...................................................................... Patch Set 36: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/57985/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13423/ : 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: I0bf4cedec2d6821ae2a04184ebb5cf88979ccee3 Gerrit-Change-Number: 15057 Gerrit-PatchSet: 36 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: Simon Glass <sjg(a)chromium.org> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 28 Jul 2017 16:48:39 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: mb/lenovo/t*00/romstage: Switch to new hybrid driver
by build bot (Jenkins) (Code Review) July 28, 2017

July 28, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20794 ) Change subject: mb/lenovo/t*00/romstage: Switch to new hybrid driver ...................................................................... Patch Set 5: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/57983/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13421/ : SUCCESS -- To view, visit https://review.coreboot.org/20794 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ibf18b75e8afe2568de8498b39a608dac8db3ba73 Gerrit-Change-Number: 20794 Gerrit-PatchSet: 5 Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 28 Jul 2017 16:14:11 +0000 Gerrit-HasComments: No

1 0