August 2017 - coreboot-gerrit - mail.coreboot.org

Change in coreboot[master]: intelmetool: Add support for Sunrise Point-H
by build bot (Jenkins) (Code Review) Aug. 31, 2017

Aug. 31, 2017

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 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14915/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59635/ : 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: 1 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: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 31 Aug 2017 07:34:53 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: intelmetool: Add support for Sunrise Point-H
by Shawn C (Code Review) Aug. 31, 2017

Aug. 31, 2017

Shawn C has uploaded this change for review. ( https://review.coreboot.org/21301 Change subject: intelmetool: Add support for Sunrise Point-H ...................................................................... intelmetool: Add support for Sunrise Point-H Tested on P10S-M WS. Change-Id: I62f78fe5ca03bf70497939a12f0036bf247b2aa7 Signed-off-by: citypw <citypw(a)gmail.com> --- M util/intelmetool/intelmetool.h 1 file changed, 5 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/01/21301/1 diff --git a/util/intelmetool/intelmetool.h b/util/intelmetool/intelmetool.h index 491ab5f..d29289d 100644 --- a/util/intelmetool/intelmetool.h +++ b/util/intelmetool/intelmetool.h @@ -237,6 +237,8 @@ #define PCI_DEVICE_ID_INTEL_LYNXPOINT_4 0x9C3A /* Lynx Point LP */ #define PCI_DEVICE_ID_INTEL_WILDCAT_1 0x9CBA /* Wildcat Point LP */ #define PCI_DEVICE_ID_INTEL_WILDCAT_2 0x9CBB /* Wildcat Point LP 2 */ +#define PCI_DEVICE_ID_INTEL_SUNRISE_H1 0xa13a /* SUNRISE Point-H 1 */ +#define PCI_DEVICE_ID_INTEL_SUNRISE_H2 0xa13b /* SUNRISE Point-H 2 */ #define PCI_DEV_HAS_SUPPORTED_ME(x) ( \ ( (x) == PCI_DEVICE_ID_INTEL_COUGARPOINT_1 ) || \ @@ -277,4 +279,6 @@ ( (x) == PCI_DEVICE_ID_INTEL_LYNXPOINT_3 ) || \ ( (x) == PCI_DEVICE_ID_INTEL_LYNXPOINT_4 ) || \ ( (x) == PCI_DEVICE_ID_INTEL_WILDCAT_1 ) || \ - ( (x) == PCI_DEVICE_ID_INTEL_WILDCAT_2)) + ( (x) == PCI_DEVICE_ID_INTEL_WILDCAT_2 ) || \ + ( (x) == PCI_DEVICE_ID_INTEL_SUNRISE_H1 ) || \ + ( (x) == PCI_DEVICE_ID_INTEL_SUNRISE_H2)) -- To view, visit https://review.coreboot.org/21301 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I62f78fe5ca03bf70497939a12f0036bf247b2aa7 Gerrit-Change-Number: 21301 Gerrit-PatchSet: 1 Gerrit-Owner: Shawn C <citypw(a)gmail.com>

1 0

Change in coreboot[master]: ifdtool: optimize some code and data type
by build bot (Jenkins) (Code Review) Aug. 31, 2017

Aug. 31, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21288 ) Change subject: ifdtool: optimize some code and data type ...................................................................... Patch Set 7: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14914/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59634/ : SUCCESS -- To view, visit https://review.coreboot.org/21288 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6d05418c8b32fa31dcd038a3e56f9aefe13fa9c4 Gerrit-Change-Number: 21288 Gerrit-PatchSet: 7 Gerrit-Owner: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Ronald G. Minnich <rminnich(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 31 Aug 2017 07:15:53 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: clang: Allow ANY_TOOLCHAIN for non-coreboot clang
by build bot (Jenkins) (Code Review) Aug. 30, 2017

Aug. 30, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21300 ) Change subject: clang: Allow ANY_TOOLCHAIN for non-coreboot clang ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14912/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59633/ : SUCCESS -- To view, visit https://review.coreboot.org/21300 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2d9d02f360d54ed92d6b6f55e6fcd530aae79adb Gerrit-Change-Number: 21300 Gerrit-PatchSet: 1 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 31 Aug 2017 06:30:31 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: clang: Enable integrated assembler on clang builds
by build bot (Jenkins) (Code Review) Aug. 30, 2017

Aug. 30, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21221 ) Change subject: clang: Enable integrated assembler on clang builds ...................................................................... Patch Set 4: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14913/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59632/ : SUCCESS -- To view, visit https://review.coreboot.org/21221 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I883bf7eb2ab52ba3d7a284c96d4aade8bc1ee4ae Gerrit-Change-Number: 21221 Gerrit-PatchSet: 4 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philippe Mathieu-Daudé <f4bug(a)amsat.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 31 Aug 2017 06:27:36 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: clang: Allow ANY_TOOLCHAIN for non-coreboot clang
by Damien Zammit (Code Review) Aug. 30, 2017

Aug. 30, 2017

Damien Zammit has uploaded this change for review. ( https://review.coreboot.org/21300 Change subject: clang: Allow ANY_TOOLCHAIN for non-coreboot clang ...................................................................... clang: Allow ANY_TOOLCHAIN for non-coreboot clang Previously, only when selecting GCC could any toolchain be selected, this allows compiling with distro clang/llvm. Change-Id: I2d9d02f360d54ed92d6b6f55e6fcd530aae79adb Signed-off-by: Damien Zammit <damien(a)zamaudio.com> --- M src/Kconfig 1 file changed, 0 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/21300/1 diff --git a/src/Kconfig b/src/Kconfig index 637f575..015c38f 100644 --- a/src/Kconfig +++ b/src/Kconfig @@ -78,7 +78,6 @@ config ANY_TOOLCHAIN bool "Allow building with any toolchain" default n - depends on COMPILER_GCC help Many toolchains break when building coreboot since it uses quite unusual linker features. Unless developers explicitely request it, -- To view, visit https://review.coreboot.org/21300 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I2d9d02f360d54ed92d6b6f55e6fcd530aae79adb Gerrit-Change-Number: 21300 Gerrit-PatchSet: 1 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com>

1 0

Change in coreboot[master]: entry32.inc: Fix inconsistent instructions between gcc/clang
by build bot (Jenkins) (Code Review) Aug. 30, 2017

Aug. 30, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21220 ) Change subject: entry32.inc: Fix inconsistent instructions between gcc/clang ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14906/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59624/ : SUCCESS -- To view, visit https://review.coreboot.org/21220 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I67121754ace0dff58035dd5670219616576789f2 Gerrit-Change-Number: 21220 Gerrit-PatchSet: 3 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 31 Aug 2017 03:08:33 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: cpu/x86 asm: Just use the correct op suffix
by build bot (Jenkins) (Code Review) Aug. 30, 2017

Aug. 30, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21218 ) Change subject: cpu/x86 asm: Just use the correct op suffix ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14904/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59623/ : SUCCESS -- To view, visit https://review.coreboot.org/21218 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6095d03d0289b418a49a10f135de5eb0e117cae0 Gerrit-Change-Number: 21218 Gerrit-PatchSet: 2 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philippe Mathieu-Daudé <f4bug(a)amsat.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 31 Aug 2017 03:02:45 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: clang: Enable integrated assembler on clang builds
by build bot (Jenkins) (Code Review) Aug. 30, 2017

Aug. 30, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21221 ) Change subject: clang: Enable integrated assembler on clang builds ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14907/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59625/ : SUCCESS -- To view, visit https://review.coreboot.org/21221 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I883bf7eb2ab52ba3d7a284c96d4aade8bc1ee4ae Gerrit-Change-Number: 21221 Gerrit-PatchSet: 3 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philippe Mathieu-Daudé <f4bug(a)amsat.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 31 Aug 2017 03:00:49 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: cpu/x86/smm: Fix explicit 'addr32' usage in clang builds
by build bot (Jenkins) (Code Review) Aug. 30, 2017

Aug. 30, 2017

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/21219 ) Change subject: cpu/x86/smm: Fix explicit 'addr32' usage in clang builds ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/14905/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/59622/ : SUCCESS -- To view, visit https://review.coreboot.org/21219 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia160d3f7da6653ea24c8229dc26f265e5f15aabb Gerrit-Change-Number: 21219 Gerrit-PatchSet: 3 Gerrit-Owner: Damien Zammit <damien(a)zamaudio.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philippe Mathieu-Daudé <f4bug(a)amsat.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Thu, 31 Aug 2017 03:00:46 +0000 Gerrit-HasComments: No

1 0