Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29779 )
Change subject: mb/google/kahlee: Enable 2T mode for liara in DVT phase
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/29779
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ibb4cc1fd2bb54984cb7a8856ed7b9f49b78eddce
Gerrit-Change-Number: 29779
Gerrit-PatchSet: 1
Gerrit-Owner: chris wang <Chris.Wang(a)amd.com>
Gerrit-Reviewer: Amanda Hwang <amanda_hwang(a)compal.corp-partner.google.com>
Gerrit-Reviewer: Chris Wang <chris.wang(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: chris wang <Chris.Wang(a)amd.com>
Gerrit-Comment-Date: Fri, 23 Nov 2018 21:52:21 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29816 )
Change subject: mainboard/lenovo/t530: Improve configuration
......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/#/c/29816/2/src/mainboard/lenovo/t530/Kconfig
File src/mainboard/lenovo/t530/Kconfig:
https://review.coreboot.org/#/c/29816/2/src/mainboard/lenovo/t530/Kconfig@41
PS2, Line 41: 0x2a
> Would be great to make cmos. […]
so basically move it to mainboard/lenovo?
https://review.coreboot.org/#/c/29816/2/src/mainboard/lenovo/t530/Kconfig@95
PS2, Line 95: config CBFS_SIZE
: hex
: default 0x001d9000 if VBOOT
: default 0x0069b800
> Is this even used when using an FMDFILE? I thought it was only used to generate a default FMD?
AFAIK it should be used
--
To view, visit https://review.coreboot.org/c/coreboot/+/29816
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I28c2fb92d574c8070dbf767adc5839faf4fb47a0
Gerrit-Change-Number: 29816
Gerrit-PatchSet: 3
Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Comment-Date: Fri, 23 Nov 2018 19:23:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-MessageType: comment
HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/29821
Change subject: {mb/cubieboard,soc/intel/quark}: Remove define __SIMPLE_DEVICE__
......................................................................
{mb/cubieboard,soc/intel/quark}: Remove define __SIMPLE_DEVICE__
As it is already in Makefile, "define __SIMPLE_DEVICE__" is
not needed.
Change-Id: I8db1b15c9186536c9b8a6b5d667fa5a11af1bad2
Signed-off-by: Elyes HAOUAS <ehaouas(a)noos.fr>
---
M src/mainboard/cubietech/cubieboard/romstage.c
M src/soc/intel/quark/romstage/car.c
M src/soc/intel/quark/romstage/fsp1_1.c
M src/soc/intel/quark/romstage/romstage.c
4 files changed, 0 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/21/29821/1
diff --git a/src/mainboard/cubietech/cubieboard/romstage.c b/src/mainboard/cubietech/cubieboard/romstage.c
index 665eb9d..f13dbf4 100644
--- a/src/mainboard/cubietech/cubieboard/romstage.c
+++ b/src/mainboard/cubietech/cubieboard/romstage.c
@@ -29,7 +29,6 @@
#include <cpu/allwinner/a10/gpio.h>
#include <cpu/allwinner/a10/twi.h>
#include <program_loading.h>
-#define __SIMPLE_DEVICE__
#include <device/device.h>
#include <drivers/xpowers/axp209/axp209.h>
#include <drivers/xpowers/axp209/chip.h>
diff --git a/src/soc/intel/quark/romstage/car.c b/src/soc/intel/quark/romstage/car.c
index 781349c..9f052a3 100644
--- a/src/soc/intel/quark/romstage/car.c
+++ b/src/soc/intel/quark/romstage/car.c
@@ -13,7 +13,6 @@
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*/
-#define __SIMPLE_DEVICE__
#include <console/console.h>
#include <fsp/util.h>
diff --git a/src/soc/intel/quark/romstage/fsp1_1.c b/src/soc/intel/quark/romstage/fsp1_1.c
index 6ce2079..d5c5f19 100644
--- a/src/soc/intel/quark/romstage/fsp1_1.c
+++ b/src/soc/intel/quark/romstage/fsp1_1.c
@@ -13,7 +13,6 @@
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*/
-#define __SIMPLE_DEVICE__
#include <arch/early_variables.h>
#include <console/console.h>
diff --git a/src/soc/intel/quark/romstage/romstage.c b/src/soc/intel/quark/romstage/romstage.c
index c5f59c6..f2a280e 100644
--- a/src/soc/intel/quark/romstage/romstage.c
+++ b/src/soc/intel/quark/romstage/romstage.c
@@ -13,7 +13,6 @@
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*/
-#define __SIMPLE_DEVICE__
#include <arch/early_variables.h>
#include <cbfs.h>
--
To view, visit https://review.coreboot.org/c/coreboot/+/29821
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8db1b15c9186536c9b8a6b5d667fa5a11af1bad2
Gerrit-Change-Number: 29821
Gerrit-PatchSet: 1
Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr>
Gerrit-MessageType: newchange
Felix Held has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/27686 )
Change subject: northbridge/nehalem: clean up header file
......................................................................
Abandoned
seems to be a duplicate of an already merged patch
--
To view, visit https://review.coreboot.org/c/coreboot/+/27686
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1f758203afdcb1b18f3c0d786698f9fbf2246e0e
Gerrit-Change-Number: 27686
Gerrit-PatchSet: 1
Gerrit-Owner: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-MessageType: abandon
Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29801 )
Change subject: arch/x86: drop special case cbfs locator
......................................................................
Patch Set 1: Code-Review+2
Tested on the OCP Wedge100s
--
To view, visit https://review.coreboot.org/c/coreboot/+/29801
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I6fa78073ee4015d7769ed588dc67f9b019d42d07
Gerrit-Change-Number: 29801
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Fri, 23 Nov 2018 18:29:58 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29805 )
Change subject: Documentation/../../icelake: Add Ice Lake coreboot development documentation
......................................................................
Patch Set 3:
Please give more a detailed overview of the supported icelake features and their requirements. Also please recommend to use 3rdparty/fsp via kconfig instead of copying binaries into the tree.
--
To view, visit https://review.coreboot.org/c/coreboot/+/29805
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ief4df6ca11f95b75ecddeb560f7887bfadced086
Gerrit-Change-Number: 29805
Gerrit-PatchSet: 3
Gerrit-Owner: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Aamir Bohra <aamir.bohra(a)intel.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan(a)intel.com>
Gerrit-Reviewer: Duncan Laurie <dlaurie(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: Shelley Chen <shchen(a)google.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Comment-Date: Fri, 23 Nov 2018 15:43:52 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment