build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22980 )
Change subject: sb/intel/i82801ix: Use the common ACPI pirq generator
......................................................................
Patch Set 12:
Build Successful
https://qa.coreboot.org/job/coreboot-gerrit/67220/ : SUCCESS
https://qa.coreboot.org/job/coreboot-checkpatch/21774/ : SUCCESS
--
To view, visit https://review.coreboot.org/22980
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: I62e520f53fa3f928a8e6f3b3cf33af2acdd53ed9
Gerrit-Change-Number: 22980
Gerrit-PatchSet: 12
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: 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: Thu, 08 Feb 2018 17:07:47 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23287 )
Change subject: sb/intel/{bd82x6,ibexpeak}: Move RCBA macros to a common location
......................................................................
Patch Set 6:
Build Successful
https://qa.coreboot.org/job/coreboot-gerrit/67218/ : SUCCESS
https://qa.coreboot.org/job/coreboot-checkpatch/21773/ : SUCCESS
--
To view, visit https://review.coreboot.org/23287
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: I33ec6801b115c0d64de1d2a0dc5d439186f3580a
Gerrit-Change-Number: 23287
Gerrit-PatchSet: 6
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
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: Thu, 08 Feb 2018 16:44:06 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
Robert Reeves has posted comments on this change. ( https://review.coreboot.org/23651 )
Change subject: mainboard/hp: Add HP Elitebook 8770w
......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/23651/1/src/mainboard/hp/8770w/cmos.layout
File src/mainboard/hp/8770w/cmos.layout:
https://review.coreboot.org/#/c/23651/1/src/mainboard/hp/8770w/cmos.layout@…
PS1, Line 91: 6 0 Emergency
: 6 1 Alert
: 6 2 Critical
: 6 3 Error
: 6 4 Warning
: 6 5 Notice
: 6 6 Info
: 6 7 Debug
: 6 8 Spew
Applied the change recommended by Arthur Heymans on the previous version of this submission. Funnily enough, I had never changed this to begin with, that was just how it was on the original file when I started this port in December.
https://review.coreboot.org/#/c/23651/1/src/mainboard/hp/8770w/romstage.c
File src/mainboard/hp/8770w/romstage.c:
https://review.coreboot.org/#/c/23651/1/src/mainboard/hp/8770w/romstage.c@42
PS1, Line 42: {
: }
Got rid of the whitespace here.
--
To view, visit https://review.coreboot.org/23651
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: Ic4ff64e9cf0c7a51ac48ca2fe6fe8beab02e9f9a
Gerrit-Change-Number: 23651
Gerrit-PatchSet: 1
Gerrit-Owner: Robert Reeves <xiinc37(a)gmail.com>
Gerrit-Reviewer: Robert Reeves <xiinc37(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Thu, 08 Feb 2018 14:33:47 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
Elyes HAOUAS has uploaded this change for review. ( https://review.coreboot.org/23656
Change subject: cpu/x86: Get rid off device_t
......................................................................
cpu/x86: Get rid off device_t
Use of `device_t` has been
abandoned in ramstage
Change-Id: I4c8acebb4a957a9600de15ea844f620a8909977b
Signed-off-by: Elyes HAOUAS <ehaouas(a)noos.fr>
---
M src/cpu/x86/lapic/lapic_cpu_init.c
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/23656/1
diff --git a/src/cpu/x86/lapic/lapic_cpu_init.c b/src/cpu/x86/lapic/lapic_cpu_init.c
index 825739c..137b93c 100644
--- a/src/cpu/x86/lapic/lapic_cpu_init.c
+++ b/src/cpu/x86/lapic/lapic_cpu_init.c
@@ -464,9 +464,9 @@
}
-static void smm_other_cpus(struct bus *cpu_bus, device_t bsp_cpu)
+static void smm_other_cpus(struct bus *cpu_bus, struct device *bsp_cpu)
{
- device_t cpu;
+ struct device *cpu;
int pre_count = atomic_read(&active_cpus);
/* Loop through the cpus once to let them run through SMM relocator */
--
To view, visit https://review.coreboot.org/23656
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: I4c8acebb4a957a9600de15ea844f620a8909977b
Gerrit-Change-Number: 23656
Gerrit-PatchSet: 1
Gerrit-Owner: Elyes HAOUAS <ehaouas(a)noos.fr>