coreboot-gerrit August 2017

coreboot-gerrit@coreboot.org

1 participants
1999 discussions

Change in coreboot[master]: util/lint: Change final-newlines to lint-test from lint-extended-test
by build bot (Jenkins) (Code Review) 11 Aug '17

11 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20872 ) Change subject: util/lint: Change final-newlines to lint-test from lint-extended-test ...................................................................... Patch Set 4: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58435/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13842/ : SUCCESS -- To view, visit https://review.coreboot.org/20872 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie945dd6caba84e0d23af499d43367d70b1696089 Gerrit-Change-Number: 20872 Gerrit-PatchSet: 4 Gerrit-Owner: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Patrick Georgi <pgeorgi(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, 11 Aug 2017 15:36:23 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: util/lint: Change final-newlines to lint-test from lint-extended-test
by build bot (Jenkins) (Code Review) 11 Aug '17

11 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20872 ) Change subject: util/lint: Change final-newlines to lint-test from lint-extended-test ...................................................................... Patch Set 3: Verified-1 Build Failed https://qa.coreboot.org/job/coreboot-gerrit/58434/ : ABORTED https://qa.coreboot.org/job/coreboot-checkpatch/13841/ : SUCCESS -- To view, visit https://review.coreboot.org/20872 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie945dd6caba84e0d23af499d43367d70b1696089 Gerrit-Change-Number: 20872 Gerrit-PatchSet: 3 Gerrit-Owner: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Patrick Georgi <pgeorgi(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, 11 Aug 2017 15:30:54 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: util/cbmem, util/cbfstool: resolve macro errors on FreeBSD
by build bot (Jenkins) (Code Review) 11 Aug '17

11 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20924 ) Change subject: util/cbmem, util/cbfstool: resolve macro errors on FreeBSD ...................................................................... Patch Set 5: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58433/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13840/ : SUCCESS -- To view, visit https://review.coreboot.org/20924 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6ec9bc7fea72aa69a41439e002f381bd5e5b6bc6 Gerrit-Change-Number: 20924 Gerrit-PatchSet: 5 Gerrit-Owner: Idwer Vollering <vidwer(a)gmail.com> Gerrit-Reviewer: Caveh Jalali <caveh(a)google.com> Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Idwer Vollering <vidwer(a)gmail.com> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Peter Stuge <peter(a)stuge.se> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 11 Aug 2017 14:45:23 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: soc/intel/skylake: Configure FSP to skip ME MBP step
by Duncan Laurie (Code Review) 11 Aug '17

11 Aug '17

Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/20951 ) Change subject: soc/intel/skylake: Configure FSP to skip ME MBP step ...................................................................... Patch Set 1: > > (1 comment) > > I'm not sure if its right way to skip MBP HOB creation and continue > rest of ME further communication. i remember in SKL time, i had > some doubt and somebody from ME team had given me some explanation > why we still need MBP hob although we are not using this in chrome > case. I need to look into this. In skylake there was PcdSkipMbpHob which defaulted to TRUE. -- To view, visit https://review.coreboot.org/20951 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5ea22ec4b8b47fa17b1cf2bf562337bfaad5ec0d Gerrit-Change-Number: 20951 Gerrit-PatchSet: 1 Gerrit-Owner: Duncan Laurie <dlaurie(a)chromium.org> Gerrit-Reviewer: Duncan Laurie <dlaurie(a)chromium.org> Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com> Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 11 Aug 2017 13:45:40 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: soc/intel/cannonlake: Add proper support to enable UART2 in 16550 mode
by build bot (Jenkins) (Code Review) 11 Aug '17

11 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20940 ) Change subject: soc/intel/cannonlake: Add proper support to enable UART2 in 16550 mode ...................................................................... Patch Set 3: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58432/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13839/ : SUCCESS -- To view, visit https://review.coreboot.org/20940 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ief58fdcb8a91f9951a48c3bd7490b1c7fee17e48 Gerrit-Change-Number: 20940 Gerrit-PatchSet: 3 Gerrit-Owner: Subrata Banik <subrata.banik(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan(a)intel.com> Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Reviewer: dhaval v sharma <dhaval.v.sharma(a)intel.com> Gerrit-Comment-Date: Fri, 11 Aug 2017 13:44:48 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: soc/intel/cannonlake: Remove unused systemagent registers
by build bot (Jenkins) (Code Review) 11 Aug '17

11 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20942 ) Change subject: soc/intel/cannonlake: Remove unused systemagent registers ...................................................................... Patch Set 4: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58431/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13838/ : SUCCESS -- To view, visit https://review.coreboot.org/20942 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I13b9c83097fc98183ea138c9087b5fc7834efd58 Gerrit-Change-Number: 20942 Gerrit-PatchSet: 4 Gerrit-Owner: Subrata Banik <subrata.banik(a)intel.com> Gerrit-Reviewer: Aamir Bohra <aamir.bohra(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan(a)intel.com> Gerrit-Reviewer: Barnali Sarkar <barnali.sarkar(a)intel.com> Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Reviewer: dhaval v sharma <dhaval.v.sharma(a)intel.com> Gerrit-Comment-Date: Fri, 11 Aug 2017 13:42:46 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: soc/intel/cannonlake: Initialize struct member to 0
by build bot (Jenkins) (Code Review) 11 Aug '17

11 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20941 ) Change subject: soc/intel/cannonlake: Initialize struct member to 0 ...................................................................... Patch Set 4: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58430/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13836/ : SUCCESS -- To view, visit https://review.coreboot.org/20941 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I01b95d31bfb1d2e9af1704a28dacb9cfd1cdcb50 Gerrit-Change-Number: 20941 Gerrit-PatchSet: 4 Gerrit-Owner: Subrata Banik <subrata.banik(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan(a)intel.com> Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Reviewer: dhaval v sharma <dhaval.v.sharma(a)intel.com> Gerrit-Comment-Date: Fri, 11 Aug 2017 13:40:25 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: soc/intel/skylake: Add proper support to enable UART2 in 16550 mode
by build bot (Jenkins) (Code Review) 11 Aug '17

11 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20939 ) Change subject: soc/intel/skylake: Add proper support to enable UART2 in 16550 mode ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58429/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13837/ : SUCCESS -- To view, visit https://review.coreboot.org/20939 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5f23fef4522743efd49167afb04d56032e16e417 Gerrit-Change-Number: 20939 Gerrit-PatchSet: 3 Gerrit-Owner: Subrata Banik <subrata.banik(a)intel.com> Gerrit-Reviewer: Aamir Bohra <aamir.bohra(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan(a)intel.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Reviewer: dhaval v sharma <dhaval.v.sharma(a)intel.com> Gerrit-Comment-Date: Fri, 11 Aug 2017 13:39:23 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: util/cbmem, util/cbfstool: resolve macro errors on FreeBSD
by build bot (Jenkins) (Code Review) 11 Aug '17

11 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20924 ) Change subject: util/cbmem, util/cbfstool: resolve macro errors on FreeBSD ...................................................................... Patch Set 4: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58428/ : SUCCESS -- To view, visit https://review.coreboot.org/20924 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6ec9bc7fea72aa69a41439e002f381bd5e5b6bc6 Gerrit-Change-Number: 20924 Gerrit-PatchSet: 4 Gerrit-Owner: Idwer Vollering <vidwer(a)gmail.com> Gerrit-Reviewer: Caveh Jalali <caveh(a)google.com> Gerrit-Reviewer: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Idwer Vollering <vidwer(a)gmail.com> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Peter Stuge <peter(a)stuge.se> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 11 Aug 2017 13:37:38 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: libpayload: Add pci_free_dev() and some boilerplate
by build bot (Jenkins) (Code Review) 11 Aug '17

11 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20955 ) Change subject: libpayload: Add pci_free_dev() and some boilerplate ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58427/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/13835/ : SUCCESS -- To view, visit https://review.coreboot.org/20955 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If0d46fab141da525f8f115d3f6045a8c417569eb Gerrit-Change-Number: 20955 Gerrit-PatchSet: 1 Gerrit-Owner: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 11 Aug 2017 11:37:40 +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