coreboot-gerrit August 2017

coreboot-gerrit@coreboot.org

1 participants
1999 discussions

Change in coreboot[master]: intel/common/cpu: Add function to get microcode patch pointer
by build bot (Jenkins) (Code Review) 17 Aug '17

17 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21061 ) Change subject: intel/common/cpu: Add function to get microcode patch pointer ...................................................................... Patch Set 1: Verified-1 Build Unstable https://qa.coreboot.org/job/coreboot-gerrit/58715/ : UNSTABLE https://qa.coreboot.org/job/coreboot-checkpatch/14092/ : SUCCESS -- To view, visit https://review.coreboot.org/21061 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia71395f4e5b2b4fcd4e4660b66e8beb99eda65b8 Gerrit-Change-Number: 21061 Gerrit-PatchSet: 1 Gerrit-Owner: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Duncan Laurie <dlaurie(a)chromium.org> Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com> Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 17 Aug 2017 03:18:28 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: intel/common/mp_init: Refactor MP Init code to get rid of microcode p...
by build bot (Jenkins) (Code Review) 17 Aug '17

17 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21010 ) Change subject: intel/common/mp_init: Refactor MP Init code to get rid of microcode param ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58716/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/14093/ : SUCCESS -- To view, visit https://review.coreboot.org/21010 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib03bb4a3063d243d97b132e0dc288ef3868a5a7b Gerrit-Change-Number: 21010 Gerrit-PatchSet: 3 Gerrit-Owner: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Duncan Laurie <dlaurie(a)chromium.org> Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com> Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 17 Aug 2017 02:57:57 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: intel/common/block/sgx: Refactor SGX common code
by build bot (Jenkins) (Code Review) 17 Aug '17

17 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21006 ) Change subject: intel/common/block/sgx: Refactor SGX common code ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58714/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/14091/ : SUCCESS -- To view, visit https://review.coreboot.org/21006 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic2fb00edbf6e98de17c12145c6f38eacd99399ad Gerrit-Change-Number: 21006 Gerrit-PatchSet: 3 Gerrit-Owner: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Duncan Laurie <dlaurie(a)chromium.org> Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com> Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 17 Aug 2017 02:56:25 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: intel/common/sgx: Use intel_mp_current_microcode() to get microcode p...
by build bot (Jenkins) (Code Review) 17 Aug '17

17 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21009 ) Change subject: intel/common/sgx: Use intel_mp_current_microcode() to get microcode pointer ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58712/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/14089/ : SUCCESS -- To view, visit https://review.coreboot.org/21009 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I9196c30ec7ea52d7184a96b33835def197e2c799 Gerrit-Change-Number: 21009 Gerrit-PatchSet: 3 Gerrit-Owner: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Duncan Laurie <dlaurie(a)chromium.org> Gerrit-Reviewer: Hannah Williams <hannah.williams(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-Comment-Date: Thu, 17 Aug 2017 02:54:42 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: soc/intel/skylake: Fix SGX init sequence
by build bot (Jenkins) (Code Review) 17 Aug '17

17 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21007 ) Change subject: soc/intel/skylake: Fix SGX init sequence ...................................................................... Patch Set 3: Build Unstable https://qa.coreboot.org/job/coreboot-gerrit/58713/ : UNSTABLE https://qa.coreboot.org/job/coreboot-checkpatch/14090/ : SUCCESS -- To view, visit https://review.coreboot.org/21007 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I88f330eb9757cdc3dbfc7609729c6ceb7d58a0e1 Gerrit-Change-Number: 21007 Gerrit-PatchSet: 3 Gerrit-Owner: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Duncan Laurie <dlaurie(a)chromium.org> Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com> Gerrit-Reviewer: Lijian Zhao <lijian.zhao(a)intel.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 17 Aug 2017 02:53:08 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: intel/common/cpu: Add function to get microcode patch pointer
by Pratikkumar V Prajapati (Code Review) 17 Aug '17

17 Aug '17

Pratikkumar V Prajapati has uploaded this change for review. ( https://review.coreboot.org/21061 Change subject: intel/common/cpu: Add function to get microcode patch pointer ...................................................................... intel/common/cpu: Add function to get microcode patch pointer Add mp_current_microcode() function to get the microcode patch pointer. Use this function to avoid reading the microcode patch from the boot media. init_cpus() would initialize microcode_patch global variable to point to microcode patch in boot media and this function can be used to access the pointer. Change-Id: Ia71395f4e5b2b4fcd4e4660b66e8beb99eda65b8 Signed-off-by: Pratik Prajapati <pratikkumar.v.prajapati(a)intel.com> --- M src/soc/intel/common/block/cpu/mp_init.c M src/soc/intel/common/block/include/intelblocks/mp_init.h 2 files changed, 20 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/61/21061/1 diff --git a/src/soc/intel/common/block/cpu/mp_init.c b/src/soc/intel/common/block/cpu/mp_init.c index fd8b5db..5bbbb2c 100644 --- a/src/soc/intel/common/block/cpu/mp_init.c +++ b/src/soc/intel/common/block/cpu/mp_init.c @@ -93,6 +93,17 @@ } /* + * Function to get the microcode patch pointer. Use this function to avoid + * reading the microcode patch from the boot media. init_cpus() would + * initialize microcode_patch global variable to point to microcode patch + * in boot media and this function can be used to access the pointer. + */ +const void *intel_mp_current_microcode(void) +{ + return microcode_patch; +} + +/* * MP Init callback function(get_microcode_info) to find the Microcode at * Pre MP Init phase. This function is common among all SOCs and thus its in * Common CPU block. @@ -102,7 +113,7 @@ */ void get_microcode_info(const void **microcode, int *parallel) { - *microcode =microcode_patch; + *microcode = intel_mp_current_microcode(); *parallel = 1; } diff --git a/src/soc/intel/common/block/include/intelblocks/mp_init.h b/src/soc/intel/common/block/include/intelblocks/mp_init.h index 1e5531c..d123207 100644 --- a/src/soc/intel/common/block/include/intelblocks/mp_init.h +++ b/src/soc/intel/common/block/include/intelblocks/mp_init.h @@ -43,6 +43,14 @@ int get_cpu_count(void); /* + * Function to get the microcode patch pointer. Use this function to avoid + * reading the microcode patch from the boot media. init_cpus() would + * initialize microcode_patch global variable to point to microcode patch + * in boot media and this function can be used to access the pointer. + */ +const void *intel_mp_current_microcode(void); + +/* * MP Init callback function(get_microcode_info) to find the Microcode at * Pre MP Init phase. This function is common among all SOCs and thus its in * Common CPU block. -- To view, visit https://review.coreboot.org/21061 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ia71395f4e5b2b4fcd4e4660b66e8beb99eda65b8 Gerrit-Change-Number: 21061 Gerrit-PatchSet: 1 Gerrit-Owner: Pratikkumar V Prajapati <pratikkumar.v.prajapati(a)intel.com>

1 0

Change in coreboot[master]: mainboard/google/coral: Add keyboard backlight support
by build bot (Jenkins) (Code Review) 17 Aug '17

17 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21027 ) Change subject: mainboard/google/coral: Add keyboard backlight support ...................................................................... Patch Set 2: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58711/ : SUCCESS -- To view, visit https://review.coreboot.org/21027 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I777247a6b58d3d50b72f12ca2fcab49a06ed5431 Gerrit-Change-Number: 21027 Gerrit-PatchSet: 2 Gerrit-Owner: Pan Sheng-Liang <sl.pan.quantw(a)gmail.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Tim Chen <Tim-Chen(a)quantatw.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 17 Aug 2017 02:38:31 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: binarypi mainboards: Clean up IS_ENABLED fan control
by build bot (Jenkins) (Code Review) 17 Aug '17

17 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21058 ) Change subject: binarypi mainboards: Clean up IS_ENABLED fan control ...................................................................... Patch Set 3: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58709/ : SUCCESS -- To view, visit https://review.coreboot.org/21058 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If88af42d00227285931829441909a982fc292b2b Gerrit-Change-Number: 21058 Gerrit-PatchSet: 3 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 17 Aug 2017 01:17:12 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: AGESA mainboards: Clean up IS_ENABLED fan control
by build bot (Jenkins) (Code Review) 17 Aug '17

17 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21059 ) Change subject: AGESA mainboards: Clean up IS_ENABLED fan control ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58707/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/14088/ : SUCCESS -- To view, visit https://review.coreboot.org/21059 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I04a9fbbf6f64f45e1a0b544267bfe840ce7fa1d9 Gerrit-Change-Number: 21059 Gerrit-PatchSet: 2 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 17 Aug 2017 01:06:58 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: amd/bettong: Fix IS_ENABLED for xHCI
by build bot (Jenkins) (Code Review) 17 Aug '17

17 Aug '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21060 ) Change subject: amd/bettong: Fix IS_ENABLED for xHCI ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-gerrit/58708/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/14087/ : SUCCESS -- To view, visit https://review.coreboot.org/21060 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icf6db485735cb8bbadb3e742a079d0bafaacd79c Gerrit-Change-Number: 21060 Gerrit-PatchSet: 2 Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: John E. Kabat Jr. <john.kabat(a)scarletltd.com> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 17 Aug 2017 01:06:21 +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