coreboot-gerrit June 2018

coreboot-gerrit@coreboot.org

1 participants
2751 discussions

Change in coreboot[master]: arch/x86: include verstage.c only when verstage is a separate stage
by build bot (Jenkins) (Code Review) 13 Jun '18

13 Jun '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/26924 ) Change subject: arch/x86: include verstage.c only when verstage is a separate stage ...................................................................... Patch Set 3: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/28963/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/74861/ : SUCCESS -- To view, visit https://review.coreboot.org/26924 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia75205001f2443cb8221a0762f182aae01ee615e Gerrit-Change-Number: 26924 Gerrit-PatchSet: 3 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 13 Jun 2018 14:50:32 +0000 Gerrit-HasComments: No Gerrit-HasLabels: No

1 0

Change in coreboot[master]: cpu/intel/car: Align the stack to 16 bytes before romstage_main
by build bot (Jenkins) (Code Review) 13 Jun '18

13 Jun '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/26797 ) Change subject: cpu/intel/car: Align the stack to 16 bytes before romstage_main ...................................................................... Patch Set 6: Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/28961/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/74859/ : SUCCESS -- To view, visit https://review.coreboot.org/26797 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1415c18779bc481fdec5f72f83c06a58ce6d5c39 Gerrit-Change-Number: 26797 Gerrit-PatchSet: 6 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 13 Jun 2018 14:46:54 +0000 Gerrit-HasComments: No Gerrit-HasLabels: No

1 0

Change in coreboot[master]: cbfstool/fit.c: Fix for older CPUs without total_size in mcu_header
by build bot (Jenkins) (Code Review) 13 Jun '18

13 Jun '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/27090 ) Change subject: cbfstool/fit.c: Fix for older CPUs without total_size in mcu_header ...................................................................... Patch Set 2: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/28962/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/74860/ : SUCCESS -- To view, visit https://review.coreboot.org/27090 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia50c087af41b0df14b607ce3c3b4eabc602e8738 Gerrit-Change-Number: 27090 Gerrit-PatchSet: 2 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 13 Jun 2018 14:46:36 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in coreboot[master]: cbfstool/fit.c: Fix for older CPUs without total_size in mcu_header
by Patrick Georgi (Code Review) 13 Jun '18

13 Jun '18

Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/27090 ) Change subject: cbfstool/fit.c: Fix for older CPUs without total_size in mcu_header ...................................................................... Patch Set 2: Code-Review+2 -- To view, visit https://review.coreboot.org/27090 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia50c087af41b0df14b607ce3c3b4eabc602e8738 Gerrit-Change-Number: 27090 Gerrit-PatchSet: 2 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 13 Jun 2018 14:45:59 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in coreboot[master]: src/mainboard/*: Remove empty vendor folders
by Angel Pons (Code Review) 13 Jun '18

13 Jun '18

Angel Pons has uploaded this change for review. ( https://review.coreboot.org/27092 Change subject: src/mainboard/*: Remove empty vendor folders ...................................................................... src/mainboard/*: Remove empty vendor folders After removing most geode_lx boards, some mainboard directories are left empty. This patch cleans them up. Change-Id: I2e99eba3d49dec90ceb2ce0c7f61612a9840ce59 Signed-off-by: Angel Pons <th3fanbus(a)gmail.com> --- D src/mainboard/aaeon/Kconfig.name D src/mainboard/artecgroup/Kconfig.name D src/mainboard/bachmann/Kconfig.name D src/mainboard/digitallogic/Kconfig.name D src/mainboard/traverse/Kconfig.name D src/mainboard/winent/Kconfig.name 6 files changed, 0 insertions(+), 12 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/92/27092/1 diff --git a/src/mainboard/aaeon/Kconfig.name b/src/mainboard/aaeon/Kconfig.name deleted file mode 100644 index 5a4e51e..0000000 --- a/src/mainboard/aaeon/Kconfig.name +++ /dev/null @@ -1,2 +0,0 @@ -config VENDOR_AAEON - bool "Aaeon" diff --git a/src/mainboard/artecgroup/Kconfig.name b/src/mainboard/artecgroup/Kconfig.name deleted file mode 100644 index 7319f9d..0000000 --- a/src/mainboard/artecgroup/Kconfig.name +++ /dev/null @@ -1,2 +0,0 @@ -config VENDOR_ARTECGROUP - bool "Artec Group" diff --git a/src/mainboard/bachmann/Kconfig.name b/src/mainboard/bachmann/Kconfig.name deleted file mode 100644 index ab6bc1e..0000000 --- a/src/mainboard/bachmann/Kconfig.name +++ /dev/null @@ -1,2 +0,0 @@ -config VENDOR_BACHMANN - bool "Bachmann electronic" diff --git a/src/mainboard/digitallogic/Kconfig.name b/src/mainboard/digitallogic/Kconfig.name deleted file mode 100644 index e47eadf..0000000 --- a/src/mainboard/digitallogic/Kconfig.name +++ /dev/null @@ -1,2 +0,0 @@ -config VENDOR_DIGITALLOGIC - bool "DIGITAL-LOGIC" diff --git a/src/mainboard/traverse/Kconfig.name b/src/mainboard/traverse/Kconfig.name deleted file mode 100644 index e4763a6..0000000 --- a/src/mainboard/traverse/Kconfig.name +++ /dev/null @@ -1,2 +0,0 @@ -config VENDOR_TRAVERSE - bool "Traverse Technologies" diff --git a/src/mainboard/winent/Kconfig.name b/src/mainboard/winent/Kconfig.name deleted file mode 100644 index 6405d0b..0000000 --- a/src/mainboard/winent/Kconfig.name +++ /dev/null @@ -1,2 +0,0 @@ -config VENDOR_WINENT - bool "Win Enterprises" -- To view, visit https://review.coreboot.org/27092 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I2e99eba3d49dec90ceb2ce0c7f61612a9840ce59 Gerrit-Change-Number: 27092 Gerrit-PatchSet: 1 Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>

1 0

Change in coreboot[master]: cpu/intel/haswell: Use the common intel romstage_main function
by build bot (Jenkins) (Code Review) 13 Jun '18

13 Jun '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/26835 ) Change subject: cpu/intel/haswell: Use the common intel romstage_main function ...................................................................... Patch Set 3: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/28960/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/74858/ : SUCCESS -- To view, visit https://review.coreboot.org/26835 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6453c40bf4ebe4695684c1bd3a403d6def82814f Gerrit-Change-Number: 26835 Gerrit-PatchSet: 3 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 13 Jun 2018 14:42:38 +0000 Gerrit-HasComments: No Gerrit-HasLabels: Yes

1 0

Change in coreboot[master]: [WIP]src/cpu/intel/car: Update microcode before setting up CARe
by Arthur Heymans (Code Review) 13 Jun '18

13 Jun '18

Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/27091 to look at the new patch set (#2). Change subject: [WIP]src/cpu/intel/car: Update microcode before setting up CARe ...................................................................... [WIP]src/cpu/intel/car: Update microcode before setting up CARe This reuses cbfstool adding a FIT table with pointers to microcode headers meant for later Intel CPUs. Change-Id: I8bd192f3f345651db0010239f99293ae63b00652 Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz> --- M src/cpu/intel/car/p4-netburst/cache_as_ram_bootblock.S A src/cpu/intel/microcode/microcode.S M src/cpu/intel/socket_LGA775/Kconfig M src/cpu/intel/socket_LGA775/Makefile.inc 4 files changed, 157 insertions(+), 6 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/91/27091/2 -- To view, visit https://review.coreboot.org/27091 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I8bd192f3f345651db0010239f99293ae63b00652 Gerrit-Change-Number: 27091 Gerrit-PatchSet: 2 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in coreboot[master]: cbfstool/fit.c: Fix for older CPUs without total_size in mcu_header
by Arthur Heymans (Code Review) 13 Jun '18

13 Jun '18

Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/27090 to look at the new patch set (#2). Change subject: cbfstool/fit.c: Fix for older CPUs without total_size in mcu_header ...................................................................... cbfstool/fit.c: Fix for older CPUs without total_size in mcu_header Some older CPUs have a fixed size of 2048 bytes for microcode total size. Change-Id: Ia50c087af41b0df14b607ce3c3b4eabc602e8738 Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz> --- M util/cbfstool/fit.c 1 file changed, 7 insertions(+), 2 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/90/27090/2 -- To view, visit https://review.coreboot.org/27090 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia50c087af41b0df14b607ce3c3b4eabc602e8738 Gerrit-Change-Number: 27090 Gerrit-PatchSet: 2 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in coreboot[master]: [WIP]src/cpu/intel/car: Update microcode before setting up CARe
by build bot (Jenkins) (Code Review) 13 Jun '18

13 Jun '18

1 0

Change in coreboot[master]: [WIP]src/cpu/intel/car: Update microcode before setting up CARe
by build bot (Jenkins) (Code Review) 13 Jun '18

13 Jun '18

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/27091 ) Change subject: [WIP]src/cpu/intel/car: Update microcode before setting up CARe ...................................................................... Patch Set 1: (12 comments) https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S File src/cpu/intel/microcode/microcode.S: https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 40: trailing whitespace https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 45: trailing whitespace https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 63: trailing whitespace https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 71: * Date : 32bit trailing whitespace https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 83: trailing whitespace https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 97: trailing whitespace https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 100: trailing whitespace https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 106: trailing whitespace https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 120: trailing whitespace https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 127: trailing whitespace https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 136: trailing whitespace https://review.coreboot.org/#/c/27091/1/src/cpu/intel/microcode/microcode.S… PS1, Line 141: trailing whitespace -- To view, visit https://review.coreboot.org/27091 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8bd192f3f345651db0010239f99293ae63b00652 Gerrit-Change-Number: 27091 Gerrit-PatchSet: 1 Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 13 Jun 2018 13:56:55 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: No

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit June 2018