Okash Khawaja has posted comments on this change. ( https://review.coreboot.org/28527 )
Change subject: wedge100s: enable mrc cache in fmap
......................................................................
Patch Set 1:
hey, could you please review this. thanks
--
To view, visit https://review.coreboot.org/28527
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: I27cd236f67a6500b40fc3eb731397d408402f041
Gerrit-Change-Number: 28527
Gerrit-PatchSet: 1
Gerrit-Owner: Okash Khawaja <okash.khawaja(a)gmail.com>
Gerrit-Reviewer: Okash Khawaja <okash.khawaja(a)gmail.com>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Comment-Date: Thu, 06 Sep 2018 11:56:07 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/28467 )
Change subject: src/include: Introduce guid_t type
......................................................................
src/include: Introduce guid_t type
Duplicate the guid_t and GUID_INIT framework from the Linux
driver. Adapt it for coreboot, and create supporting copy
and compare functions.
Change-Id: Ia1cd7a1f0e0f900858830e1a6a7e2bbbe272fa30
Signed-off-by: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Reviewed-on: https://review.coreboot.org/28467
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Martin Roth <martinroth(a)google.com>
---
A src/include/uuid.h
1 file changed, 42 insertions(+), 0 deletions(-)
Approvals:
build bot (Jenkins): Verified
Martin Roth: Looks good to me, approved
diff --git a/src/include/uuid.h b/src/include/uuid.h
new file mode 100644
index 0000000..0209055
--- /dev/null
+++ b/src/include/uuid.h
@@ -0,0 +1,42 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2010, Intel Corp. Huang Ying <ying.huang(a)intel.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _UUID_H_
+#define _UUID_H_
+
+#include <string.h>
+
+typedef struct {
+ uint8_t b[16];
+} __packed guid_t;
+
+#define GUID_INIT(a, b, c, d0, d1, d2, d3, d4, d5, d6, d7) \
+((guid_t) \
+{{ (a) & 0xff, ((a) >> 8) & 0xff, ((a) >> 16) & 0xff, ((a) >> 24) & 0xff, \
+ (b) & 0xff, ((b) >> 8) & 0xff, \
+ (c) & 0xff, ((c) >> 8) & 0xff, \
+ (d0), (d1), (d2), (d3), (d4), (d5), (d6), (d7) } })
+
+static inline int guidcmp(const guid_t *guid1, const guid_t *guid2)
+{
+ return memcmp(guid1, guid2, sizeof(guid_t));
+}
+
+static inline guid_t *guidcpy(guid_t *dest, const guid_t *src)
+{
+ return (guid_t *)memcpy(dest, src, sizeof(guid_t));
+}
+
+#endif /* _UUID_H_ */
--
To view, visit https://review.coreboot.org/28467
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia1cd7a1f0e0f900858830e1a6a7e2bbbe272fa30
Gerrit-Change-Number: 28467
Gerrit-PatchSet: 2
Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Michael van der Westhuizen has uploaded this change for review. ( https://review.coreboot.org/28526
Change subject: qemu-q35: Use the TSC for udelay
......................................................................
qemu-q35: Use the TSC for udelay
Use the TSC for delays on q35, ensuring that the TSC delay code is
included in the correct stages when selected.
Tested on qemu-35 and wedge-100s (for no regressions).
Change-Id: I3f8368509807974bfcf2a0fcff7a4aa21adf47ed
Signed-off-by: Michael van der Westhuizen <rmikey(a)fb.com>
---
M src/cpu/qemu-x86/Kconfig
M src/cpu/qemu-x86/Makefile.inc
2 files changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/26/28526/1
diff --git a/src/cpu/qemu-x86/Kconfig b/src/cpu/qemu-x86/Kconfig
index ea2bc46..ab8e1a9 100644
--- a/src/cpu/qemu-x86/Kconfig
+++ b/src/cpu/qemu-x86/Kconfig
@@ -20,3 +20,4 @@
select ARCH_ROMSTAGE_X86_32
select ARCH_RAMSTAGE_X86_32
select SMP
+ select UDELAY_TSC
diff --git a/src/cpu/qemu-x86/Makefile.inc b/src/cpu/qemu-x86/Makefile.inc
index b5f8369..288eea6 100644
--- a/src/cpu/qemu-x86/Makefile.inc
+++ b/src/cpu/qemu-x86/Makefile.inc
@@ -16,3 +16,4 @@
subdirs-y += ../x86/mtrr
subdirs-y += ../x86/lapic
subdirs-y += ../x86/smm
+subdirs-y += ../x86/tsc
--
To view, visit https://review.coreboot.org/28526
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: I3f8368509807974bfcf2a0fcff7a4aa21adf47ed
Gerrit-Change-Number: 28526
Gerrit-PatchSet: 1
Gerrit-Owner: Michael van der Westhuizen (1002100)
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/28470 )
Change subject: arch/x86: Add BERT region support functions
......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/28470/2//COMMIT_MSG
Commit Message:
https://review.coreboot.org/#/c/28470/2//COMMIT_MSG@12
PS2, Line 12: Reserved
where does that come from ? I don't find it in the spec. Why doesn't it work with type 16 ?
--
To view, visit https://review.coreboot.org/28470
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: I54826981639b5647a8ca33b8b55ff097681402b9
Gerrit-Change-Number: 28470
Gerrit-PatchSet: 2
Gerrit-Owner: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Martin Roth <martinroth(a)google.com>
Gerrit-CC: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Thu, 06 Sep 2018 08:39:46 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
Mike Banon has restored this change. ( https://review.coreboot.org/28425 )
Change subject: AMD microcodes: scripts for applying the unofficial (not-merged-yet) updates
......................................................................
Restored
--
To view, visit https://review.coreboot.org/28425
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: restore
Gerrit-Change-Id: Ic16c1c4b86576ee7505cceed871b47a2b82f3c56
Gerrit-Change-Number: 28425
Gerrit-PatchSet: 2
Gerrit-Owner: Mike Banon <mikebdp2(a)gmail.com>
Gerrit-Reviewer: Mike Banon <mikebdp2(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
YanRu Chen has posted comments on this change. ( https://review.coreboot.org/28484 )
Change subject: rammus: add SPD mapping for rammus and shyvana support.
......................................................................
Patch Set 1:
(1 comment)
The memory strapping table of rammus project start from 0b0001 to 0b0100. We don't have table for 0b000, so we copy 0b0001 setting to 0b0000 to follow your non empty requirement.
https://review.coreboot.org/#/c/28484/1/src/mainboard/google/poppy/variants…
File src/mainboard/google/poppy/variants/rammus/Makefile.inc:
https://review.coreboot.org/#/c/28484/1/src/mainboard/google/poppy/variants…
PS1, Line 1: samsung_dimm_K4E8E324EB-EGCF
> Memory ID 0 and 1 are using the same memory?
The memory strapping table of rammus project start from 0b0001 to 0b0100. We don't have table for 0b000, so we copy 0b0001 setting to 0b0000 to follow your non empty requirement.
--
To view, visit https://review.coreboot.org/28484
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: I7cb5b7f2bcdc6fbe0cbc640cad4af014f1a0edd6
Gerrit-Change-Number: 28484
Gerrit-PatchSet: 1
Gerrit-Owner: YanRu Chen <kane_chen(a)pegatron.corp-partner.google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: YanRu Chen <kane_chen(a)pegatron.corp-partner.google.com>
Gerrit-Reviewer: Zhuohao Lee <zhuohao(a)chromium.org>
Gerrit-CC: Jerry Chou <jerry_chou(a)pegatron.corp-partner.google.com>
Gerrit-CC: Ken Lu <ken_lu(a)pegatron.corp-partner.google.com>
Gerrit-Comment-Date: Thu, 06 Sep 2018 06:58:34 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No