[U-Boot] [RESEND PATCH v4 0/12] x86: Better support of coreboot

[U-Boot] [RESEND PATCH v4 0/12] x86: Better support of coreboot

[Bin Meng]

In theory U-Boot built for coreboot is supposed to run as a payload
to be loaded by coreboot on every board that coreboot supports.
The U-Boot build process uses SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE
which are hardcoded in board defconfig and Kconfig files. For better
support of coreboot, we want to make these two options configurable
so that we can easily change them during 'make menuconfig' so that
the generated U-Boot image for coreboot is board configuration aware.

Note v2/v3/v4 patch series aims to better support coreboot, while v1
patch series just tried to resolve the issues seen on qemu. Several
issues are fixed to make coreboot support in U-Boot more robust.

See v1 patch discussion
@ http://lists.denx.de/pipermail/u-boot/2015-January/200140.html

The official qemu U-Boot support will come in the future. This patch
series have been tested with coreboot running on qemu and Intel Crown
Bay (my own unofficiall simple port, not in coreboot mainline) then
loading the U-Boot built with the new mechanism.

Changes in v4:
- Hide XIP_ROM_SIZE too after rebase
- Rebase to u-boot-x86/next

Changes in v3:
- Make coreboot options wrapped by TARGET_COREBOOT
- Remove these options from include/configs/coreboot.h for bisectability
- Move CONFIG_VIDEO_X86 to drivers/video/Kconfig
- Fix several typos in README.x86

Changes in v2:
- Fix the CONFIG_COLLECT_TIMESTAMPS typo in the comment block
  and commit message
- Spell out TSC, MSR and PIT in the Kconfig help
- New patch to move CONFIG_X86_RESET_VECTOR and CONFIG_SYS_X86_START16
  to Kconfig
- New patch to hide ROM chip size when CONFIG_X86_RESET_VECTOR is
  not selected
- New patch to make SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE configurable
- New patch to move coreboot specific defines from coreboot.h to Kconfig
- New patch to move CONFIG_SYS_CAR_xxx to Kconfig
- New patch to remove include/configs/coreboot.h
- New patch to make chromebook_link the default board for coreboot
- Leave CROS_EC defines unchanged in coreboot.h
- New patch to configure pci memory regions
- New patch to update REAME.x86 for coreboot support

Bin Meng (12):
  x86: coreboot: Set up timer base correctly
  x86: Allow a hardcoded TSC frequency provided by Kconfig
  x86: Move CONFIG_X86_RESET_VECTOR and CONFIG_SYS_X86_START16 to
    Kconfig
  x86: Hide ROM chip size when CONFIG_X86_RESET_VECTOR is not selected
  x86: coreboot: Make SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE
    configurable
  x86: coreboot: Move coreboot-specific defines from coreboot.h to
    Kconfig
  x86: Move CONFIG_SYS_CAR_xxx to Kconfig
  x86: Remove include/configs/coreboot.h
  x86: Make chromebook_link the default board for coreboot
  x86: coreboot: Wrap cros_ec initialization
  x86: coreboot: Configure pci memory regions
  x86: Update README.x86 for coreboot support

 arch/x86/Kconfig                     |  33 ++++++
 arch/x86/cpu/coreboot/Kconfig        |  15 +++
 arch/x86/cpu/coreboot/pci.c          |  30 ++++-
 arch/x86/cpu/coreboot/timestamp.c    |  33 +++---
 arch/x86/dts/Makefile                |   3 +-
 arch/x86/dts/chromebook_link.dts     | 217 ++++++++++++++++++++++++++++++++++-
 arch/x86/dts/link.dts                | 216 ----------------------------------
 arch/x86/lib/tsc_timer.c             |   8 +-
 board/coreboot/coreboot/Kconfig      |  27 ++++-
 board/coreboot/coreboot/MAINTAINERS  |   2 +-
 board/coreboot/coreboot/coreboot.c   |   2 +
 board/google/chromebook_link/Kconfig |   9 ++
 board/intel/crownbay/Kconfig         |   1 +
 configs/chromebook_link_defconfig    |   1 +
 configs/coreboot-x86_defconfig       |   1 -
 doc/README.x86                       |  39 ++++++-
 drivers/video/Kconfig                |   8 ++
 include/configs/chromebook_link.h    |   7 +-
 include/configs/coreboot.h           |  75 ------------
 include/configs/crownbay.h           |   2 -
 20 files changed, 406 insertions(+), 323 deletions(-)
 create mode 100644 arch/x86/cpu/coreboot/Kconfig
 mode change 120000 => 100644 arch/x86/dts/chromebook_link.dts
 delete mode 100644 arch/x86/dts/link.dts
 delete mode 100644 include/configs/coreboot.h

-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 01/12] x86: coreboot: Set up timer base correctly

[Bin Meng]

If coreboot is built with CONFIG_COLLECT_TIMESTAMPS, use the value
of base_time in coreboot's timestamp table as our timer base,
otherwise TSC counter value will be used.

Sometimes even coreboot is built with CONFIG_COLLECT_TIMESTAMPS,
the value of base_time in the timestamp table is still zero, so
we must exclude this case too (this is currently seen on booting
coreboot in qemu).

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>

---

Changes in v4: None
Changes in v3: None
Changes in v2:
- Fix the CONFIG_COLLECT_TIMESTAMPS typo in the comment block
  and commit message

 arch/x86/cpu/coreboot/timestamp.c | 33 ++++++++++++++++++++-------------
 1 file changed, 20 insertions(+), 13 deletions(-)

diff --git a/arch/x86/cpu/coreboot/timestamp.c b/arch/x86/cpu/coreboot/timestamp.c
index bd3558a..0edee6b 100644
--- a/arch/x86/cpu/coreboot/timestamp.c
+++ b/arch/x86/cpu/coreboot/timestamp.c
@@ -3,18 +3,7 @@
  *
  * Copyright (C) 2011 The ChromiumOS Authors.  All rights reserved.
  *
- * 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.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA, 02110-1301 USA
+ * SPDX-License-Identifier:	GPL-2.0+
  */
 
 #include <common.h>
@@ -38,9 +27,27 @@ static struct timestamp_table *ts_table  __attribute__((section(".data")));
 
 void timestamp_init(void)
 {
+#ifdef CONFIG_SYS_X86_TSC_TIMER
+	uint64_t base_time;
+#endif
+
 	ts_table = lib_sysinfo.tstamp_table;
 #ifdef CONFIG_SYS_X86_TSC_TIMER
-	timer_set_base(ts_table->base_time);
+	/*
+	 * If coreboot is built with CONFIG_COLLECT_TIMESTAMPS, use the value
+	 * of base_time in coreboot's timestamp table as our timer base,
+	 * otherwise TSC counter value will be used.
+	 *
+	 * Sometimes even coreboot is built with CONFIG_COLLECT_TIMESTAMPS,
+	 * the value of base_time in the timestamp table is still zero, so
+	 * we must exclude this case too (this is currently seen on booting
+	 * coreboot in qemu)
+	 */
+	if (ts_table && ts_table->base_time)
+		base_time = ts_table->base_time;
+	else
+		base_time = rdtsc();
+	timer_set_base(base_time);
 #endif
 	timestamp_add_now(TS_U_BOOT_INITTED);
 }
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 02/12] x86: Allow a hardcoded TSC frequency provided by Kconfig

[Bin Meng]

By default U-Boot automatically calibrates TSC running frequency via
MSR and PIT. The calibration may not work on every x86 processor, so
a new Kconfig option CONFIG_TSC_CALIBRATION_BYPASS is introduced to
allow bypassing the calibration and assign a hardcoded TSC frequency
CONFIG_TSC_FREQ_IN_MHZ.

Normally the bypass should be turned on in a simulation environment
like qemu.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>

---

Changes in v4: None
Changes in v3: None
Changes in v2:
- Spell out TSC, MSR and PIT in the Kconfig help

 arch/x86/Kconfig         | 20 ++++++++++++++++++++
 arch/x86/lib/tsc_timer.c |  8 ++++++--
 2 files changed, 26 insertions(+), 2 deletions(-)

diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index e992e64..ffcb4cb 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -313,6 +313,26 @@ config FRAMEBUFFER_VESA_MODE
 
 endmenu
 
+config TSC_CALIBRATION_BYPASS
+	bool "Bypass Time-Stamp Counter (TSC) calibration"
+	default n
+	help
+	  By default U-Boot automatically calibrates Time-Stamp Counter (TSC)
+	  running frequency via Model-Specific Register (MSR) and Programmable
+	  Interval Timer (PIT). If the calibration does not work on your board,
+	  select this option and provide a hardcoded TSC running frequency with
+	  CONFIG_TSC_FREQ_IN_MHZ below.
+
+	  Normally this option should be turned on in a simulation environment
+	  like qemu.
+
+config TSC_FREQ_IN_MHZ
+	int "Time-Stamp Counter (TSC) running frequency in MHz"
+	depends on TSC_CALIBRATION_BYPASS
+	default 1000
+	help
+	  The running frequency in MHz of Time-Stamp Counter (TSC).
+
 source "arch/x86/cpu/ivybridge/Kconfig"
 
 source "arch/x86/cpu/queensbay/Kconfig"
diff --git a/arch/x86/lib/tsc_timer.c b/arch/x86/lib/tsc_timer.c
index fb9afed..7f5ba2c 100644
--- a/arch/x86/lib/tsc_timer.c
+++ b/arch/x86/lib/tsc_timer.c
@@ -78,7 +78,7 @@ static int match_cpu(u8 family, u8 model)
  *
  * Returns the calibration value or 0 if MSR calibration failed.
  */
-static unsigned long try_msr_calibrate_tsc(void)
+static unsigned long __maybe_unused try_msr_calibrate_tsc(void)
 {
 	u32 lo, hi, ratio, freq_id, freq;
 	unsigned long res;
@@ -199,7 +199,7 @@ static inline int pit_expect_msb(unsigned char val, u64 *tscp,
 #define MAX_QUICK_PIT_MS 50
 #define MAX_QUICK_PIT_ITERATIONS (MAX_QUICK_PIT_MS * PIT_TICK_RATE / 1000 / 256)
 
-static unsigned long quick_pit_calibrate(void)
+static unsigned long __maybe_unused quick_pit_calibrate(void)
 {
 	int i;
 	u64 tsc, delta;
@@ -306,6 +306,9 @@ unsigned __attribute__((no_instrument_function)) long get_tbclk_mhz(void)
 	if (gd->arch.tsc_mhz)
 		return gd->arch.tsc_mhz;
 
+#ifdef CONFIG_TSC_CALIBRATION_BYPASS
+	fast_calibrate = CONFIG_TSC_FREQ_IN_MHZ;
+#else
 	fast_calibrate = try_msr_calibrate_tsc();
 	if (!fast_calibrate) {
 
@@ -313,6 +316,7 @@ unsigned __attribute__((no_instrument_function)) long get_tbclk_mhz(void)
 		if (!fast_calibrate)
 			panic("TSC frequency is ZERO");
 	}
+#endif
 
 	gd->arch.tsc_mhz = fast_calibrate;
 	return fast_calibrate;
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 03/12] x86: Move CONFIG_X86_RESET_VECTOR and CONFIG_SYS_X86_START16 to Kconfig

[Bin Meng]

Convert CONFIG_X86_RESET_VECTOR and CONFIG_SYS_X86_START16 to Kconfig
options so that we can remove them from board configuration file.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>

---

Changes in v4: None
Changes in v3: None
Changes in v2:
- New patch to move CONFIG_X86_RESET_VECTOR and CONFIG_SYS_X86_START16
  to Kconfig

 arch/x86/Kconfig                     | 9 +++++++++
 board/google/chromebook_link/Kconfig | 1 +
 board/intel/crownbay/Kconfig         | 1 +
 include/configs/chromebook_link.h    | 2 --
 include/configs/crownbay.h           | 2 --
 5 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index ffcb4cb..ba0c1aa 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -66,6 +66,15 @@ config SMM_TSEG
 config SMM_TSEG_SIZE
 	hex
 
+config X86_RESET_VECTOR
+	bool
+	default n
+
+config SYS_X86_START16
+	hex
+	depends on X86_RESET_VECTOR
+	default 0xfffff800
+
 config BOARD_ROMSIZE_KB_512
 	bool
 config BOARD_ROMSIZE_KB_1024
diff --git a/board/google/chromebook_link/Kconfig b/board/google/chromebook_link/Kconfig
index 7f79fd2..a9a55e8 100644
--- a/board/google/chromebook_link/Kconfig
+++ b/board/google/chromebook_link/Kconfig
@@ -14,6 +14,7 @@ config SYS_CONFIG_NAME
 
 config BOARD_SPECIFIC_OPTIONS # dummy
 	def_bool y
+	select X86_RESET_VECTOR
 	select CPU_INTEL_SOCKET_RPGA989
 	select NORTHBRIDGE_INTEL_IVYBRIDGE
 	select SOUTHBRIDGE_INTEL_C216
diff --git a/board/intel/crownbay/Kconfig b/board/intel/crownbay/Kconfig
index 4709f9b..762663a 100644
--- a/board/intel/crownbay/Kconfig
+++ b/board/intel/crownbay/Kconfig
@@ -14,6 +14,7 @@ config SYS_CONFIG_NAME
 
 config BOARD_SPECIFIC_OPTIONS # dummy
 	def_bool y
+	select X86_RESET_VECTOR
 	select INTEL_QUEENSBAY
 	select BOARD_ROMSIZE_KB_1024
 
diff --git a/include/configs/chromebook_link.h b/include/configs/chromebook_link.h
index 8930210..449f0c2 100644
--- a/include/configs/chromebook_link.h
+++ b/include/configs/chromebook_link.h
@@ -19,11 +19,9 @@
 #define CONFIG_SYS_CAR_SIZE			(128 * 1024)
 #define CONFIG_SYS_MONITOR_LEN			(1 << 20)
 #define CONFIG_DCACHE_RAM_MRC_VAR_SIZE		0x4000
-#define CONFIG_SYS_X86_START16			0xfffff800
 #define CONFIG_BOARD_EARLY_INIT_F
 #define CONFIG_DISPLAY_CPUINFO
 
-#define CONFIG_X86_RESET_VECTOR
 #define CONFIG_NR_DRAM_BANKS			8
 #define CONFIG_X86_MRC_ADDR			0xfffa0000
 #define CONFIG_CACHE_MRC_SIZE_KB		512
diff --git a/include/configs/crownbay.h b/include/configs/crownbay.h
index eadb339..b927b1c 100644
--- a/include/configs/crownbay.h
+++ b/include/configs/crownbay.h
@@ -14,10 +14,8 @@
 #include <configs/x86-common.h>
 
 #define CONFIG_SYS_MONITOR_LEN		(1 << 20)
-#define CONFIG_SYS_X86_START16		0xfffff800
 #define CONFIG_BOARD_EARLY_INIT_F
 
-#define CONFIG_X86_RESET_VECTOR
 #define CONFIG_NR_DRAM_BANKS		1
 
 #define CONFIG_X86_SERIAL
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 04/12] x86: Hide ROM chip size when CONFIG_X86_RESET_VECTOR is not selected

[Bin Meng]

When CONFIG_X86_RESET_VECTOR is not selected, specifying the ROM chip
size is meaningless, hence hide it.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>

---

Changes in v4:
- Hide XIP_ROM_SIZE too after rebase

Changes in v3: None
Changes in v2:
- New patch to hide ROM chip size when CONFIG_X86_RESET_VECTOR is
  not selected

 arch/x86/Kconfig | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index ba0c1aa..512b6f6 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -49,6 +49,7 @@ config RAMBASE
 
 config XIP_ROM_SIZE
 	hex
+	depends on X86_RESET_VECTOR
 	default ROM_SIZE
 
 config CPU_ADDR_BITS
@@ -90,6 +91,7 @@ config BOARD_ROMSIZE_KB_16384
 
 choice
 	prompt "ROM chip size"
+	depends on X86_RESET_VECTOR
 	default UBOOT_ROMSIZE_KB_512 if BOARD_ROMSIZE_KB_512
 	default UBOOT_ROMSIZE_KB_1024 if BOARD_ROMSIZE_KB_1024
 	default UBOOT_ROMSIZE_KB_2048 if BOARD_ROMSIZE_KB_2048
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 05/12] x86: coreboot: Make SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE configurable

[Bin Meng]

In theory U-Boot built for coreboot is supposed to run as a payload
to be loaded by coreboot on every board that coreboot supports.
The U-Boot build process uses SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE
which are hardcoded in board defconfig and Kconfig files. For better
support of coreboot, we want to make these two options configurable
so that we can easily change them during 'make menuconfig' so that
the generated U-Boot image for coreboot is board configuration aware.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>

---

Changes in v4: None
Changes in v3: None
Changes in v2:
- New patch to make SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE configurable

 board/coreboot/coreboot/Kconfig | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/board/coreboot/coreboot/Kconfig b/board/coreboot/coreboot/Kconfig
index 6ca6ced..e5ccf58 100644
--- a/board/coreboot/coreboot/Kconfig
+++ b/board/coreboot/coreboot/Kconfig
@@ -9,7 +9,20 @@ config SYS_VENDOR
 config SYS_SOC
 	default "coreboot"
 
+comment "coreboot-specific options"
+
 config SYS_CONFIG_NAME
+	string "Board configuration file"
 	default "coreboot"
+	help
+	  This option selects the board configuration file in include/configs/
+	  directory to be used to build U-Boot for coreboot.
+
+config DEFAULT_DEVICE_TREE
+	string "Board Device Tree Source (dts) file"
+	default "link"
+	help
+	  This option selects the board Device Tree Source (dts) file in
+	  arch/x86/dts/ directory to be used to build U-Boot for coreboot.
 
 endif
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 06/12] x86: coreboot: Move coreboot-specific defines from coreboot.h to Kconfig

[Bin Meng]

There are many places in the U-Boot source tree which refer to
CONFIG_SYS_COREBOOT, CONFIG_CBMEM_CONSOLE and CONFIG_VIDEO_COREBOOT
that is currently defined in coreboot.h.

Move them to arch/x86/cpu/coreboot/Kconfig so that we can switch
to board configuration file to build U-Boot later.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>

---

Changes in v4: None
Changes in v3:
- Make coreboot options wrapped by TARGET_COREBOOT
- Remove these options from include/configs/coreboot.h for bisectability

Changes in v2:
- New patch to move coreboot specific defines from coreboot.h to Kconfig

 arch/x86/Kconfig              |  2 ++
 arch/x86/cpu/coreboot/Kconfig | 15 +++++++++++++++
 include/configs/coreboot.h    |  5 -----
 3 files changed, 17 insertions(+), 5 deletions(-)
 create mode 100644 arch/x86/cpu/coreboot/Kconfig

diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index 512b6f6..90e828a 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -344,6 +344,8 @@ config TSC_FREQ_IN_MHZ
 	help
 	  The running frequency in MHz of Time-Stamp Counter (TSC).
 
+source "arch/x86/cpu/coreboot/Kconfig"
+
 source "arch/x86/cpu/ivybridge/Kconfig"
 
 source "arch/x86/cpu/queensbay/Kconfig"
diff --git a/arch/x86/cpu/coreboot/Kconfig b/arch/x86/cpu/coreboot/Kconfig
new file mode 100644
index 0000000..e0e3c64
--- /dev/null
+++ b/arch/x86/cpu/coreboot/Kconfig
@@ -0,0 +1,15 @@
+if TARGET_COREBOOT
+
+config SYS_COREBOOT
+	bool
+	default y
+
+config CBMEM_CONSOLE
+	bool
+	default y
+
+config VIDEO_COREBOOT
+	bool
+	default y
+
+endif
diff --git a/include/configs/coreboot.h b/include/configs/coreboot.h
index 990a2d1..a0d9952 100644
--- a/include/configs/coreboot.h
+++ b/include/configs/coreboot.h
@@ -19,7 +19,6 @@
  * High Level Configuration Options
  * (easy to change)
  */
-#define CONFIG_SYS_COREBOOT
 #define CONFIG_LAST_STAGE_INIT
 #define CONFIG_SYS_EARLY_PCI_INIT
 
@@ -55,10 +54,6 @@
 					"stdout=vga,serial,cbmem\0" \
 					"stderr=vga,serial,cbmem\0"
 
-#define CONFIG_CBMEM_CONSOLE
-
-#define CONFIG_VIDEO_COREBOOT
-
 #define CONFIG_NR_DRAM_BANKS			4
 
 #define CONFIG_TRACE
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 07/12] x86: Move CONFIG_SYS_CAR_xxx to Kconfig

[Bin Meng]

Move CONFIG_SYS_CAR_ADDR and CONFIG_SYS_CAR_SIZE to Kconfig so that
we don't need them in the board configuration file thus the same
board configuratoin file can be used to build both coreboot version
and bare version.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>

---

Changes in v4: None
Changes in v3: None
Changes in v2:
- New patch to move CONFIG_SYS_CAR_xxx to Kconfig

 board/coreboot/coreboot/Kconfig      | 12 ++++++++++++
 board/google/chromebook_link/Kconfig |  8 ++++++++
 include/configs/chromebook_link.h    |  4 ++--
 3 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/board/coreboot/coreboot/Kconfig b/board/coreboot/coreboot/Kconfig
index e5ccf58..f2cd754 100644
--- a/board/coreboot/coreboot/Kconfig
+++ b/board/coreboot/coreboot/Kconfig
@@ -25,4 +25,16 @@ config DEFAULT_DEVICE_TREE
 	  This option selects the board Device Tree Source (dts) file in
 	  arch/x86/dts/ directory to be used to build U-Boot for coreboot.
 
+config SYS_CAR_ADDR
+	hex "Board specific Cache-As-RAM (CAR) address"
+	default 0x19200000
+	help
+	  This option specifies the board specific Cache-As-RAM (CAR) address.
+
+config SYS_CAR_SIZE
+	hex "Board specific Cache-As-RAM (CAR) size"
+	default 0x4000
+	help
+	  This option specifies the board specific Cache-As-RAM (CAR) size.
+
 endif
diff --git a/board/google/chromebook_link/Kconfig b/board/google/chromebook_link/Kconfig
index a9a55e8..33a31f3 100644
--- a/board/google/chromebook_link/Kconfig
+++ b/board/google/chromebook_link/Kconfig
@@ -30,4 +30,12 @@ config EARLY_POST_CROS_EC
 	bool "Enable early post to Chrome OS EC"
 	default y
 
+config SYS_CAR_ADDR
+	hex
+	default 0xff7e0000
+
+config SYS_CAR_SIZE
+	hex
+	default 0x20000
+
 endif
diff --git a/include/configs/chromebook_link.h b/include/configs/chromebook_link.h
index 449f0c2..318f1a8 100644
--- a/include/configs/chromebook_link.h
+++ b/include/configs/chromebook_link.h
@@ -15,9 +15,9 @@
 
 #include <configs/x86-common.h>
 
-#define CONFIG_SYS_CAR_ADDR			0xff7e0000
-#define CONFIG_SYS_CAR_SIZE			(128 * 1024)
+
 #define CONFIG_SYS_MONITOR_LEN			(1 << 20)
+
 #define CONFIG_DCACHE_RAM_MRC_VAR_SIZE		0x4000
 #define CONFIG_BOARD_EARLY_INIT_F
 #define CONFIG_DISPLAY_CPUINFO
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 08/12] x86: Remove include/configs/coreboot.h

[Bin Meng]

Since we already swtiched to use the new mechanism for building
U-Boot for coreboot, coreboot.h is no longer needed so remove it.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>

---

Changes in v4: None
Changes in v3: None
Changes in v2:
- New patch to remove include/configs/coreboot.h

 board/coreboot/coreboot/MAINTAINERS |  2 +-
 include/configs/coreboot.h          | 70 -------------------------------------
 2 files changed, 1 insertion(+), 71 deletions(-)
 delete mode 100644 include/configs/coreboot.h

diff --git a/board/coreboot/coreboot/MAINTAINERS b/board/coreboot/coreboot/MAINTAINERS
index 6ce66f5..2736aa0 100644
--- a/board/coreboot/coreboot/MAINTAINERS
+++ b/board/coreboot/coreboot/MAINTAINERS
@@ -2,5 +2,5 @@ COREBOOT BOARD
 M:	Simon Glass <sjg@chromium.org>
 S:	Maintained
 F:	board/coreboot/coreboot/
-F:	include/configs/coreboot.h
+F:	include/configs/chromebook_link.h
 F:	configs/coreboot-x86_defconfig
diff --git a/include/configs/coreboot.h b/include/configs/coreboot.h
deleted file mode 100644
index a0d9952..0000000
--- a/include/configs/coreboot.h
+++ /dev/null
@@ -1,70 +0,0 @@
-/*
- * Copyright (c) 2011 The Chromium OS Authors.
- * (C) Copyright 2008
- * Graeme Russ, graeme.russ at gmail.com.
- *
- * SPDX-License-Identifier:	GPL-2.0+
- */
-
-/*
- * board/config.h - configuration options, board specific
- */
-
-#ifndef __CONFIG_H
-#define __CONFIG_H
-
-#include <configs/x86-common.h>
-
-/*
- * High Level Configuration Options
- * (easy to change)
- */
-#define CONFIG_LAST_STAGE_INIT
-#define CONFIG_SYS_EARLY_PCI_INIT
-
-#define CONFIG_SYS_CAR_ADDR			0x19200000
-#define CONFIG_SYS_CAR_SIZE			(16 * 1024)
-#define CONFIG_SYS_MONITOR_LEN			(256 * 1024)
-
-#define CONFIG_TRACE_EARLY_SIZE		(8 << 20)
-#define CONFIG_TRACE_EARLY
-#define CONFIG_TRACE_EARLY_ADDR		0x01400000
-
-#define CONFIG_BOOTSTAGE
-#define CONFIG_BOOTSTAGE_REPORT
-#define CONFIG_BOOTSTAGE_FDT
-#define CONFIG_CMD_BOOTSTAGE
-/* Place to stash bootstage data from first-stage U-Boot */
-#define CONFIG_BOOTSTAGE_STASH		0x0110f000
-#define CONFIG_BOOTSTAGE_STASH_SIZE	0x7fc
-#define CONFIG_BOOTSTAGE_USER_COUNT	60
-
-#define CONFIG_SCSI_DEV_LIST		{PCI_VENDOR_ID_INTEL, \
-			PCI_DEVICE_ID_INTEL_NM10_AHCI},	      \
-	{PCI_VENDOR_ID_INTEL,		\
-			PCI_DEVICE_ID_INTEL_COUGARPOINT_AHCI_MOBILE}, \
-	{PCI_VENDOR_ID_INTEL, \
-			PCI_DEVICE_ID_INTEL_COUGARPOINT_AHCI_SERIES6}, \
-	{PCI_VENDOR_ID_INTEL,		\
-			PCI_DEVICE_ID_INTEL_PANTHERPOINT_AHCI_MOBILE}
-
-#define CONFIG_X86_SERIAL
-
-#define CONFIG_STD_DEVICES_SETTINGS     "stdin=usbkbd,vga,serial\0" \
-					"stdout=vga,serial,cbmem\0" \
-					"stderr=vga,serial,cbmem\0"
-
-#define CONFIG_NR_DRAM_BANKS			4
-
-#define CONFIG_TRACE
-#define CONFIG_CMD_TRACE
-#define CONFIG_TRACE_BUFFER_SIZE	(16 << 20)
-
-#define CONFIG_BOOTDELAY	2
-
-#define CONFIG_CROS_EC
-#define CONFIG_CROS_EC_LPC
-#define CONFIG_CMD_CROS_EC
-#define CONFIG_ARCH_EARLY_INIT_R
-
-#endif	/* __CONFIG_H */
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 09/12] x86: Make chromebook_link the default board for coreboot

[Bin Meng]

Change SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE to chromebook_link
which is currently the only real board officially supported to run
U-Boot loaded by coreboot.

Note the symbolic link file chromebook_link.dts is deleted and
link.dts is renamed to chromebook_link.dts.

To avoid multiple definition of video_hw_init, the CONFIG_VIDEO_X86
define needs to be moved to arch/x86/cpu/ivybridge/Kconfig.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>

---

Changes in v4:
- Rebase to u-boot-x86/next

Changes in v3:
- Move CONFIG_VIDEO_X86 to drivers/video/Kconfig

Changes in v2:
- New patch to make chromebook_link the default board for coreboot

 arch/x86/dts/Makefile             |   3 +-
 arch/x86/dts/chromebook_link.dts  | 217 +++++++++++++++++++++++++++++++++++++-
 arch/x86/dts/link.dts             | 216 -------------------------------------
 board/coreboot/coreboot/Kconfig   |   4 +-
 configs/chromebook_link_defconfig |   1 +
 configs/coreboot-x86_defconfig    |   1 -
 drivers/video/Kconfig             |   8 ++
 include/configs/chromebook_link.h |   1 -
 8 files changed, 228 insertions(+), 223 deletions(-)
 mode change 120000 => 100644 arch/x86/dts/chromebook_link.dts
 delete mode 100644 arch/x86/dts/link.dts

diff --git a/arch/x86/dts/Makefile b/arch/x86/dts/Makefile
index 5525094..97ed884 100644
--- a/arch/x86/dts/Makefile
+++ b/arch/x86/dts/Makefile
@@ -1,5 +1,4 @@
-dtb-y += link.dtb \
-	chromebook_link.dtb \
+dtb-y += chromebook_link.dtb \
 	crownbay.dtb
 
 targets += $(dtb-y)
diff --git a/arch/x86/dts/chromebook_link.dts b/arch/x86/dts/chromebook_link.dts
deleted file mode 120000
index 6f8c5cd..0000000
--- a/arch/x86/dts/chromebook_link.dts
+++ /dev/null
@@ -1 +0,0 @@
-link.dts
\ No newline at end of file
diff --git a/arch/x86/dts/chromebook_link.dts b/arch/x86/dts/chromebook_link.dts
new file mode 100644
index 0000000..9490b16
--- /dev/null
+++ b/arch/x86/dts/chromebook_link.dts
@@ -0,0 +1,216 @@
+/dts-v1/;
+
+/include/ "skeleton.dtsi"
+/include/ "serial.dtsi"
+
+/ {
+	model = "Google Link";
+	compatible = "google,link", "intel,celeron-ivybridge";
+
+	config {
+	       silent_console = <0>;
+	};
+
+	gpioa {
+		compatible = "intel,ich6-gpio";
+		u-boot,dm-pre-reloc;
+		reg = <0 0x10>;
+		bank-name = "A";
+	};
+
+	gpiob {
+		compatible = "intel,ich6-gpio";
+		u-boot,dm-pre-reloc;
+		reg = <0x30 0x10>;
+		bank-name = "B";
+	};
+
+	gpioc {
+		compatible = "intel,ich6-gpio";
+		u-boot,dm-pre-reloc;
+		reg = <0x40 0x10>;
+		bank-name = "C";
+	};
+
+	chosen {
+		stdout-path = "/serial";
+	};
+
+	spd {
+		compatible = "memory-spd";
+		#address-cells = <1>;
+		#size-cells = <0>;
+		elpida_4Gb_1600_x16 {
+			reg = <0>;
+			data = [92 10 0b 03 04 19 02 02
+				03 52 01 08 0a 00 fe 00
+				69 78 69 3c 69 11 18 81
+				20 08 3c 3c 01 40 83 81
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 0f 11 42 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 02 fe 00
+				11 52 00 00 00 07 7f 37
+				45 42 4a 32 30 55 47 36
+				45 42 55 30 2d 47 4e 2d
+				46 20 30 20 02 fe 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00];
+		};
+		samsung_4Gb_1600_1.35v_x16 {
+			reg = <1>;
+			data = [92 11 0b 03 04 19 02 02
+				03 11 01 08 0a 00 fe 00
+				69 78 69 3c 69 11 18 81
+				f0 0a 3c 3c 01 40 83 01
+				00 80 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 0f 11 02 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 80 ce 01
+				00 00 00 00 00 00 6a 04
+				4d 34 37 31 42 35 36 37
+				34 42 48 30 2d 59 4b 30
+				20 20 00 00 80 ce 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00];
+			};
+		micron_4Gb_1600_1.35v_x16 {
+			reg = <2>;
+			data = [92 11 0b 03 04 19 02 02
+				03 11 01 08 0a 00 fe 00
+				69 78 69 3c 69 11 18 81
+				20 08 3c 3c 01 40 83 05
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 0f 01 02 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 80 2c 00
+				00 00 00 00 00 00 ad 75
+				34 4b 54 46 32 35 36 36
+				34 48 5a 2d 31 47 36 45
+				31 20 45 31 80 2c 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				00 00 00 00 00 00 00 00
+				ff ff ff ff ff ff ff ff
+				ff ff ff ff ff ff ff ff
+				ff ff ff ff ff ff ff ff
+				ff ff ff ff ff ff ff ff
+				ff ff ff ff ff ff ff ff
+				ff ff ff ff ff ff ff ff
+				ff ff ff ff ff ff ff ff
+				ff ff ff ff ff ff ff ff
+				ff ff ff ff ff ff ff ff
+				ff ff ff ff ff ff ff ff];
+		};
+	};
+
+	spi {
+		#address-cells = <1>;
+		#size-cells = <0>;
+		compatible = "intel,ich9";
+		spi-flash at 0 {
+			reg = <0>;
+			compatible = "winbond,w25q64", "spi-flash";
+			memory-map = <0xff800000 0x00800000>;
+		};
+	};
+
+	pci {
+		sata {
+			compatible = "intel,pantherpoint-ahci";
+			intel,sata-mode = "ahci";
+			intel,sata-port-map = <1>;
+			intel,sata-port0-gen3-tx = <0x00880a7f>;
+		};
+
+		gma {
+			compatible = "intel,gma";
+			intel,dp_hotplug = <0 0 0x06>;
+			intel,panel-port-select = <1>;
+			intel,panel-power-cycle-delay = <6>;
+			intel,panel-power-up-delay = <2000>;
+			intel,panel-power-down-delay = <500>;
+			intel,panel-power-backlight-on-delay = <2000>;
+			intel,panel-power-backlight-off-delay = <2000>;
+			intel,cpu-backlight = <0x00000200>;
+			intel,pch-backlight = <0x04000000>;
+		};
+
+		lpc {
+			compatible = "intel,lpc";
+			#address-cells = <1>;
+			#size-cells = <1>;
+			gen-dec = <0x800 0xfc 0x900 0xfc>;
+			intel,gen-dec = <0x800 0xfc 0x900 0xfc>;
+			intel,pirq-routing = <0x8b 0x8a 0x8b 0x8b
+						0x80 0x80 0x80 0x80>;
+			intel,gpi-routing = <0 0 0 0 0 0 0 2
+						1 0 0 0 0 0 0 0>;
+			/* Enable EC SMI source */
+			intel,alt-gp-smi-enable = <0x0100>;
+
+			cros-ec at 200 {
+				compatible = "google,cros-ec";
+				reg = <0x204 1 0x200 1 0x880 0x80>;
+
+				/* Describes the flash memory within the EC */
+				#address-cells = <1>;
+				#size-cells = <1>;
+				flash at 8000000 {
+					reg = <0x08000000 0x20000>;
+					erase-value = <0xff>;
+				};
+			};
+		};
+	};
+
+	microcode {
+		update at 0 {
+#include "microcode/m12306a9_0000001b.dtsi"
+		};
+	};
+
+};
diff --git a/arch/x86/dts/link.dts b/arch/x86/dts/link.dts
deleted file mode 100644
index 9490b16..0000000
--- a/arch/x86/dts/link.dts
+++ /dev/null
@@ -1,216 +0,0 @@
-/dts-v1/;
-
-/include/ "skeleton.dtsi"
-/include/ "serial.dtsi"
-
-/ {
-	model = "Google Link";
-	compatible = "google,link", "intel,celeron-ivybridge";
-
-	config {
-	       silent_console = <0>;
-	};
-
-	gpioa {
-		compatible = "intel,ich6-gpio";
-		u-boot,dm-pre-reloc;
-		reg = <0 0x10>;
-		bank-name = "A";
-	};
-
-	gpiob {
-		compatible = "intel,ich6-gpio";
-		u-boot,dm-pre-reloc;
-		reg = <0x30 0x10>;
-		bank-name = "B";
-	};
-
-	gpioc {
-		compatible = "intel,ich6-gpio";
-		u-boot,dm-pre-reloc;
-		reg = <0x40 0x10>;
-		bank-name = "C";
-	};
-
-	chosen {
-		stdout-path = "/serial";
-	};
-
-	spd {
-		compatible = "memory-spd";
-		#address-cells = <1>;
-		#size-cells = <0>;
-		elpida_4Gb_1600_x16 {
-			reg = <0>;
-			data = [92 10 0b 03 04 19 02 02
-				03 52 01 08 0a 00 fe 00
-				69 78 69 3c 69 11 18 81
-				20 08 3c 3c 01 40 83 81
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 0f 11 42 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 02 fe 00
-				11 52 00 00 00 07 7f 37
-				45 42 4a 32 30 55 47 36
-				45 42 55 30 2d 47 4e 2d
-				46 20 30 20 02 fe 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00];
-		};
-		samsung_4Gb_1600_1.35v_x16 {
-			reg = <1>;
-			data = [92 11 0b 03 04 19 02 02
-				03 11 01 08 0a 00 fe 00
-				69 78 69 3c 69 11 18 81
-				f0 0a 3c 3c 01 40 83 01
-				00 80 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 0f 11 02 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 80 ce 01
-				00 00 00 00 00 00 6a 04
-				4d 34 37 31 42 35 36 37
-				34 42 48 30 2d 59 4b 30
-				20 20 00 00 80 ce 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00];
-			};
-		micron_4Gb_1600_1.35v_x16 {
-			reg = <2>;
-			data = [92 11 0b 03 04 19 02 02
-				03 11 01 08 0a 00 fe 00
-				69 78 69 3c 69 11 18 81
-				20 08 3c 3c 01 40 83 05
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 0f 01 02 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 80 2c 00
-				00 00 00 00 00 00 ad 75
-				34 4b 54 46 32 35 36 36
-				34 48 5a 2d 31 47 36 45
-				31 20 45 31 80 2c 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				00 00 00 00 00 00 00 00
-				ff ff ff ff ff ff ff ff
-				ff ff ff ff ff ff ff ff
-				ff ff ff ff ff ff ff ff
-				ff ff ff ff ff ff ff ff
-				ff ff ff ff ff ff ff ff
-				ff ff ff ff ff ff ff ff
-				ff ff ff ff ff ff ff ff
-				ff ff ff ff ff ff ff ff
-				ff ff ff ff ff ff ff ff
-				ff ff ff ff ff ff ff ff];
-		};
-	};
-
-	spi {
-		#address-cells = <1>;
-		#size-cells = <0>;
-		compatible = "intel,ich9";
-		spi-flash at 0 {
-			reg = <0>;
-			compatible = "winbond,w25q64", "spi-flash";
-			memory-map = <0xff800000 0x00800000>;
-		};
-	};
-
-	pci {
-		sata {
-			compatible = "intel,pantherpoint-ahci";
-			intel,sata-mode = "ahci";
-			intel,sata-port-map = <1>;
-			intel,sata-port0-gen3-tx = <0x00880a7f>;
-		};
-
-		gma {
-			compatible = "intel,gma";
-			intel,dp_hotplug = <0 0 0x06>;
-			intel,panel-port-select = <1>;
-			intel,panel-power-cycle-delay = <6>;
-			intel,panel-power-up-delay = <2000>;
-			intel,panel-power-down-delay = <500>;
-			intel,panel-power-backlight-on-delay = <2000>;
-			intel,panel-power-backlight-off-delay = <2000>;
-			intel,cpu-backlight = <0x00000200>;
-			intel,pch-backlight = <0x04000000>;
-		};
-
-		lpc {
-			compatible = "intel,lpc";
-			#address-cells = <1>;
-			#size-cells = <1>;
-			gen-dec = <0x800 0xfc 0x900 0xfc>;
-			intel,gen-dec = <0x800 0xfc 0x900 0xfc>;
-			intel,pirq-routing = <0x8b 0x8a 0x8b 0x8b
-						0x80 0x80 0x80 0x80>;
-			intel,gpi-routing = <0 0 0 0 0 0 0 2
-						1 0 0 0 0 0 0 0>;
-			/* Enable EC SMI source */
-			intel,alt-gp-smi-enable = <0x0100>;
-
-			cros-ec at 200 {
-				compatible = "google,cros-ec";
-				reg = <0x204 1 0x200 1 0x880 0x80>;
-
-				/* Describes the flash memory within the EC */
-				#address-cells = <1>;
-				#size-cells = <1>;
-				flash at 8000000 {
-					reg = <0x08000000 0x20000>;
-					erase-value = <0xff>;
-				};
-			};
-		};
-	};
-
-	microcode {
-		update at 0 {
-#include "microcode/m12306a9_0000001b.dtsi"
-		};
-	};
-
-};
diff --git a/board/coreboot/coreboot/Kconfig b/board/coreboot/coreboot/Kconfig
index f2cd754..981de1f 100644
--- a/board/coreboot/coreboot/Kconfig
+++ b/board/coreboot/coreboot/Kconfig
@@ -13,14 +13,14 @@ comment "coreboot-specific options"
 
 config SYS_CONFIG_NAME
 	string "Board configuration file"
-	default "coreboot"
+	default "chromebook_link"
 	help
 	  This option selects the board configuration file in include/configs/
 	  directory to be used to build U-Boot for coreboot.
 
 config DEFAULT_DEVICE_TREE
 	string "Board Device Tree Source (dts) file"
-	default "link"
+	default "chromebook_link"
 	help
 	  This option selects the board Device Tree Source (dts) file in
 	  arch/x86/dts/ directory to be used to build U-Boot for coreboot.
diff --git a/configs/chromebook_link_defconfig b/configs/chromebook_link_defconfig
index b83803e..e956835 100644
--- a/configs/chromebook_link_defconfig
+++ b/configs/chromebook_link_defconfig
@@ -6,5 +6,6 @@ CONFIG_OF_SEPARATE=y
 CONFIG_DEFAULT_DEVICE_TREE="chromebook_link"
 CONFIG_HAVE_MRC=y
 CONFIG_SMM_TSEG_SIZE=0x800000
+CONFIG_VIDEO_X86=y
 CONFIG_FRAMEBUFFER_SET_VESA_MODE=y
 CONFIG_FRAMEBUFFER_VESA_MODE_11A=y
diff --git a/configs/coreboot-x86_defconfig b/configs/coreboot-x86_defconfig
index 6249db7..3cc034a 100644
--- a/configs/coreboot-x86_defconfig
+++ b/configs/coreboot-x86_defconfig
@@ -2,4 +2,3 @@ CONFIG_SYS_EXTRA_OPTIONS="SYS_TEXT_BASE=0x01110000"
 CONFIG_X86=y
 CONFIG_TARGET_COREBOOT=y
 CONFIG_OF_CONTROL=y
-CONFIG_DEFAULT_DEVICE_TREE="link"
diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
index e69de29..fdbf3f6 100644
--- a/drivers/video/Kconfig
+++ b/drivers/video/Kconfig
@@ -0,0 +1,8 @@
+config VIDEO_X86
+	bool "Enable x86 video driver support"
+	depends on X86
+	default n
+	help
+	  Turn on this option to enable a very simple driver which uses vesa
+	  to discover the video mode and then provides a frame buffer for use
+	  by U-Boot.
diff --git a/include/configs/chromebook_link.h b/include/configs/chromebook_link.h
index 318f1a8..e0bf309 100644
--- a/include/configs/chromebook_link.h
+++ b/include/configs/chromebook_link.h
@@ -39,7 +39,6 @@
 
 #define CONFIG_X86_OPTION_ROM_FILE		pci8086,0166.bin
 #define CONFIG_X86_OPTION_ROM_ADDR		0xfff90000
-#define CONFIG_VIDEO_X86
 
 #define CONFIG_PCI_MEM_BUS	0xe0000000
 #define CONFIG_PCI_MEM_PHYS	CONFIG_PCI_MEM_BUS
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 10/12] x86: coreboot: Wrap cros_ec initialization

[Bin Meng]

cros_ec_board_init() should be called only when CONFIG_CROS_EC is
enabled.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>

---

Changes in v4: None
Changes in v3: None
Changes in v2:
- Leave CROS_EC defines unchanged in coreboot.h

 board/coreboot/coreboot/coreboot.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/board/coreboot/coreboot/coreboot.c b/board/coreboot/coreboot/coreboot.c
index 154faf6..e076ea6 100644
--- a/board/coreboot/coreboot/coreboot.c
+++ b/board/coreboot/coreboot/coreboot.c
@@ -10,8 +10,10 @@
 
 int arch_early_init_r(void)
 {
+#ifdef CONFIG_CROS_EC
 	if (cros_ec_board_init())
 		return -1;
+#endif
 
 	return 0;
 }
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 11/12] x86: coreboot: Configure pci memory regions

[Bin Meng]

Configure coreboot pci memory regions so that pci device drivers
could work correctly.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>

---

Changes in v4: None
Changes in v3: None
Changes in v2:
- New patch to configure pci memory regions

 arch/x86/cpu/coreboot/pci.c | 30 ++++++++++++++++++++++++++++--
 1 file changed, 28 insertions(+), 2 deletions(-)

diff --git a/arch/x86/cpu/coreboot/pci.c b/arch/x86/cpu/coreboot/pci.c
index 6a3dd93..c9983f1 100644
--- a/arch/x86/cpu/coreboot/pci.c
+++ b/arch/x86/cpu/coreboot/pci.c
@@ -13,6 +13,8 @@
 #include <pci.h>
 #include <asm/pci.h>
 
+DECLARE_GLOBAL_DATA_PTR;
+
 static void config_pci_bridge(struct pci_controller *hose, pci_dev_t dev,
 			      struct pci_config_table *table)
 {
@@ -35,7 +37,31 @@ void board_pci_setup_hose(struct pci_controller *hose)
 	hose->first_busno = 0;
 	hose->last_busno = 0;
 
-	pci_set_region(hose->regions + 0, 0x0, 0x0, 0xffffffff,
+	/* PCI memory space */
+	pci_set_region(hose->regions + 0,
+		       CONFIG_PCI_MEM_BUS,
+		       CONFIG_PCI_MEM_PHYS,
+		       CONFIG_PCI_MEM_SIZE,
 		       PCI_REGION_MEM);
-	hose->region_count = 1;
+
+	/* PCI IO space */
+	pci_set_region(hose->regions + 1,
+		       CONFIG_PCI_IO_BUS,
+		       CONFIG_PCI_IO_PHYS,
+		       CONFIG_PCI_IO_SIZE,
+		       PCI_REGION_IO);
+
+	pci_set_region(hose->regions + 2,
+		       CONFIG_PCI_PREF_BUS,
+		       CONFIG_PCI_PREF_PHYS,
+		       CONFIG_PCI_PREF_SIZE,
+		       PCI_REGION_PREFETCH);
+
+	pci_set_region(hose->regions + 3,
+		       0,
+		       0,
+		       gd->ram_size,
+		       PCI_REGION_MEM | PCI_REGION_SYS_MEMORY);
+
+	hose->region_count = 4;
 }
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 12/12] x86: Update README.x86 for coreboot support

[Bin Meng]

Update README.x86 to include new build instructions for U-Boot as
the coreboot payload and testing considerations with coreboot.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>

---

Changes in v4: None
Changes in v3:
- Fix several typos in README.x86

Changes in v2:
- New patch to update REAME.x86 for coreboot support

 doc/README.x86 | 39 +++++++++++++++++++++++++++++++++++++--
 1 file changed, 37 insertions(+), 2 deletions(-)

diff --git a/doc/README.x86 b/doc/README.x86
index b474161..7df8cc5 100644
--- a/doc/README.x86
+++ b/doc/README.x86
@@ -32,6 +32,21 @@ on other architectures, like below:
 $ make coreboot-x86_defconfig
 $ make all
 
+Note this default configuration will build a U-Boot payload for the Link board.
+To build a coreboot payload against another board, you can change the build
+configuration during the 'make menuconfig' process.
+
+x86 architecture  --->
+	...
+	(chromebook_link) Board configuration file
+	(chromebook_link) Board Device Tree Source (dts) file
+	(0x19200000) Board specific Cache-As-RAM (CAR) address
+	(0x4000) Board specific Cache-As-RAM (CAR) size
+
+Change the 'Board configuration file' and 'Board Device Tree Source (dts) file'
+to point to a new board. You can also change the Cache-As-RAM (CAR) related
+settings here if the default values do not fit your new board.
+
 Building ROM version of U-Boot (hereafter referred to as u-boot.rom) is a
 little bit tricky, as generally it requires several binary blobs which are not
 shipped in the U-Boot source tree. Due to this reason, the u-boot.rom build is
@@ -88,11 +103,31 @@ in this FSP package too.
 Rename the first one to fsp.bin and second one to cmc.bin and put them in the
 board directory.
 
-Now you can build U-Boot and obtaim u-boot.rom
+Now you can build U-Boot and obtain u-boot.rom
 
 $ make crownbay_defconfig
 $ make all
 
+Test with coreboot
+------------------
+For testing U-Boot as the coreboot payload, there are things that need be paid
+attention to. coreboot supports loading an ELF executable and a 32-bit plain
+binary, as well as other supported payloads. With the default configuration,
+U-Boot is set up to use a separate Device Tree Blob (dtb). As of today, the
+generated u-boot-dtb.bin needs to be packaged by the cbfstool utility (a tool
+provided by coreboot) manually as coreboot's 'make menuconfig' does not provide
+this capability yet. The command is as follows:
+
+# in the coreboot root directory
+$ ./build/util/cbfstool/cbfstool build/coreboot.rom add-flat-binary \
+  -f u-boot-dtb.bin -n fallback/payload -c lzma -l 0x1110000 -e 0x1110015
+
+Make sure 0x1110000 matches CONFIG_SYS_TEXT_BASE and 0x1110015 matches the
+symbol address of _start (in arch/x86/cpu/start.S).
+
+If you want to use ELF as the coreboot payload, change U-Boot configuration to
+use CONFIG_OF_EMBED.
+
 CPU Microcode
 -------------
 Modern CPU usually requires a special bit stream called microcode [5] to be
@@ -106,7 +141,7 @@ x86 has been converted to use driver model for serial and GPIO.
 Device Tree
 -----------
 x86 uses device tree to configure the board thus requires CONFIG_OF_CONTROL to
-be turned on. Not every device on the board is configured via devie tree, but
+be turned on. Not every device on the board is configured via device tree, but
 more and more devices will be added as time goes by. Check out the directory
 arch/x86/dts/ for these device tree source files.
 
-- 
1.8.2.1

[U-Boot] [RESEND PATCH v4 0/12] x86: Better support of coreboot

[Simon Glass]

Hi Bin,

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> In theory U-Boot built for coreboot is supposed to run as a payload
> to be loaded by coreboot on every board that coreboot supports.
> The U-Boot build process uses SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE
> which are hardcoded in board defconfig and Kconfig files. For better
> support of coreboot, we want to make these two options configurable
> so that we can easily change them during 'make menuconfig' so that
> the generated U-Boot image for coreboot is board configuration aware.
>
> Note v2/v3/v4 patch series aims to better support coreboot, while v1
> patch series just tried to resolve the issues seen on qemu. Several
> issues are fixed to make coreboot support in U-Boot more robust.
>
> See v1 patch discussion
> @ http://lists.denx.de/pipermail/u-boot/2015-January/200140.html
>
> The official qemu U-Boot support will come in the future. This patch
> series have been tested with coreboot running on qemu and Intel Crown
> Bay (my own unofficiall simple port, not in coreboot mainline) then
> loading the U-Boot built with the new mechanism.
>
> Changes in v4:
> - Hide XIP_ROM_SIZE too after rebase
> - Rebase to u-boot-x86/next

I need to change XIP_ROM_SIZE to depend on X86_RESET_VECTOR also to
make this build for coreboot. Is that expected?

Regards,
Simon

[U-Boot] [RESEND PATCH v4 06/12] x86: coreboot: Move coreboot-specific defines from coreboot.h to Kconfig

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> There are many places in the U-Boot source tree which refer to
> CONFIG_SYS_COREBOOT, CONFIG_CBMEM_CONSOLE and CONFIG_VIDEO_COREBOOT
> that is currently defined in coreboot.h.
>
> Move them to arch/x86/cpu/coreboot/Kconfig so that we can switch
> to board configuration file to build U-Boot later.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>
> ---
>
> Changes in v4: None
> Changes in v3:
> - Make coreboot options wrapped by TARGET_COREBOOT
> - Remove these options from include/configs/coreboot.h for bisectability
>
> Changes in v2:
> - New patch to move coreboot specific defines from coreboot.h to Kconfig
>
>  arch/x86/Kconfig              |  2 ++
>  arch/x86/cpu/coreboot/Kconfig | 15 +++++++++++++++
>  include/configs/coreboot.h    |  5 -----
>  3 files changed, 17 insertions(+), 5 deletions(-)
>  create mode 100644 arch/x86/cpu/coreboot/Kconfig

Acked-by: Simon Glass <sjg@chromium.org>

[U-Boot] [RESEND PATCH v4 09/12] x86: Make chromebook_link the default board for coreboot

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> Change SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE to chromebook_link
> which is currently the only real board officially supported to run
> U-Boot loaded by coreboot.
>
> Note the symbolic link file chromebook_link.dts is deleted and
> link.dts is renamed to chromebook_link.dts.
>
> To avoid multiple definition of video_hw_init, the CONFIG_VIDEO_X86
> define needs to be moved to arch/x86/cpu/ivybridge/Kconfig.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>
> ---
>
> Changes in v4:
> - Rebase to u-boot-x86/next
>
> Changes in v3:
> - Move CONFIG_VIDEO_X86 to drivers/video/Kconfig
>
> Changes in v2:
> - New patch to make chromebook_link the default board for coreboot
>
>  arch/x86/dts/Makefile             |   3 +-
>  arch/x86/dts/chromebook_link.dts  | 217 +++++++++++++++++++++++++++++++++++++-
>  arch/x86/dts/link.dts             | 216 -------------------------------------
>  board/coreboot/coreboot/Kconfig   |   4 +-
>  configs/chromebook_link_defconfig |   1 +
>  configs/coreboot-x86_defconfig    |   1 -
>  drivers/video/Kconfig             |   8 ++
>  include/configs/chromebook_link.h |   1 -
>  8 files changed, 228 insertions(+), 223 deletions(-)
>  mode change 120000 => 100644 arch/x86/dts/chromebook_link.dts
>  delete mode 100644 arch/x86/dts/link.dts

Acked-by: Simon Glass <sjg@chromium.org>

[U-Boot] [RESEND PATCH v4 12/12] x86: Update README.x86 for coreboot support

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> Update README.x86 to include new build instructions for U-Boot as
> the coreboot payload and testing considerations with coreboot.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> Acked-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Changes in v4: None
> Changes in v3:
> - Fix several typos in README.x86
>
> Changes in v2:
> - New patch to update REAME.x86 for coreboot support
>
>  doc/README.x86 | 39 +++++++++++++++++++++++++++++++++++++--
>  1 file changed, 37 insertions(+), 2 deletions(-)

Acked-by: Simon Glass <sjg@chromium.org>

[U-Boot] [RESEND PATCH v4 0/12] x86: Better support of coreboot

[Bin Meng]

Hi Simon,

On Tue, Jan 6, 2015 at 10:21 PM, Simon Glass <sjg@chromium.org> wrote:
> Hi Bin,
>
> On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
>> In theory U-Boot built for coreboot is supposed to run as a payload
>> to be loaded by coreboot on every board that coreboot supports.
>> The U-Boot build process uses SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE
>> which are hardcoded in board defconfig and Kconfig files. For better
>> support of coreboot, we want to make these two options configurable
>> so that we can easily change them during 'make menuconfig' so that
>> the generated U-Boot image for coreboot is board configuration aware.
>>
>> Note v2/v3/v4 patch series aims to better support coreboot, while v1
>> patch series just tried to resolve the issues seen on qemu. Several
>> issues are fixed to make coreboot support in U-Boot more robust.
>>
>> See v1 patch discussion
>> @ http://lists.denx.de/pipermail/u-boot/2015-January/200140.html
>>
>> The official qemu U-Boot support will come in the future. This patch
>> series have been tested with coreboot running on qemu and Intel Crown
>> Bay (my own unofficiall simple port, not in coreboot mainline) then
>> loading the U-Boot built with the new mechanism.
>>
>> Changes in v4:
>> - Hide XIP_ROM_SIZE too after rebase
>> - Rebase to u-boot-x86/next
>
> I need to change XIP_ROM_SIZE to depend on X86_RESET_VECTOR also to
> make this build for coreboot. Is that expected?
>

Yes, really sorry about that. I've resent a v4 patch just now. See
http://patchwork.ozlabs.org/patch/425685/

Regards,
Bin

[U-Boot] [RESEND PATCH v4 0/12] x86: Better support of coreboot

[Simon Glass]

Hi Bin,

On 6 January 2015 at 07:35, Bin Meng <bmeng.cn@gmail.com> wrote:
> Hi Simon,
>
> On Tue, Jan 6, 2015 at 10:21 PM, Simon Glass <sjg@chromium.org> wrote:
>> Hi Bin,
>>
>> On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
>>> In theory U-Boot built for coreboot is supposed to run as a payload
>>> to be loaded by coreboot on every board that coreboot supports.
>>> The U-Boot build process uses SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE
>>> which are hardcoded in board defconfig and Kconfig files. For better
>>> support of coreboot, we want to make these two options configurable
>>> so that we can easily change them during 'make menuconfig' so that
>>> the generated U-Boot image for coreboot is board configuration aware.
>>>
>>> Note v2/v3/v4 patch series aims to better support coreboot, while v1
>>> patch series just tried to resolve the issues seen on qemu. Several
>>> issues are fixed to make coreboot support in U-Boot more robust.
>>>
>>> See v1 patch discussion
>>> @ http://lists.denx.de/pipermail/u-boot/2015-January/200140.html
>>>
>>> The official qemu U-Boot support will come in the future. This patch
>>> series have been tested with coreboot running on qemu and Intel Crown
>>> Bay (my own unofficiall simple port, not in coreboot mainline) then
>>> loading the U-Boot built with the new mechanism.
>>>
>>> Changes in v4:
>>> - Hide XIP_ROM_SIZE too after rebase
>>> - Rebase to u-boot-x86/next
>>
>> I need to change XIP_ROM_SIZE to depend on X86_RESET_VECTOR also to
>> make this build for coreboot. Is that expected?
>>
>
> Yes, really sorry about that. I've resent a v4 patch just now. See
> http://patchwork.ozlabs.org/patch/425685/

OK thanks - it seems to work. I've pushed the series to x86/testing
and will pull it in later on after I've done a full build.

Regards,
Simon

[U-Boot] [RESEND PATCH v4 01/12] x86: coreboot: Set up timer base correctly

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
>
> If coreboot is built with CONFIG_COLLECT_TIMESTAMPS, use the value
> of base_time in coreboot's timestamp table as our timer base,
> otherwise TSC counter value will be used.
>
> Sometimes even coreboot is built with CONFIG_COLLECT_TIMESTAMPS,
> the value of base_time in the timestamp table is still zero, so
> we must exclude this case too (this is currently seen on booting
> coreboot in qemu).
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> Acked-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2:
> - Fix the CONFIG_COLLECT_TIMESTAMPS typo in the comment block
>   and commit message

Applied to u-boot-x86/next, thanks!

[U-Boot] [RESEND PATCH v4 02/12] x86: Allow a hardcoded TSC frequency provided by Kconfig

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> By default U-Boot automatically calibrates TSC running frequency via
> MSR and PIT. The calibration may not work on every x86 processor, so
> a new Kconfig option CONFIG_TSC_CALIBRATION_BYPASS is introduced to
> allow bypassing the calibration and assign a hardcoded TSC frequency
> CONFIG_TSC_FREQ_IN_MHZ.
>
> Normally the bypass should be turned on in a simulation environment
> like qemu.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> Acked-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2:
> - Spell out TSC, MSR and PIT in the Kconfig help
>
>  arch/x86/Kconfig         | 20 ++++++++++++++++++++
>  arch/x86/lib/tsc_timer.c |  8 ++++++--
>  2 files changed, 26 insertions(+), 2 deletions(-)

Applied to u-boot-x86/next, thanks!

[U-Boot] [RESEND PATCH v4 03/12] x86: Move CONFIG_X86_RESET_VECTOR and CONFIG_SYS_X86_START16 to Kconfig

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> Convert CONFIG_X86_RESET_VECTOR and CONFIG_SYS_X86_START16 to Kconfig
> options so that we can remove them from board configuration file.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> Acked-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2:
> - New patch to move CONFIG_X86_RESET_VECTOR and CONFIG_SYS_X86_START16
>   to Kconfig
>
>  arch/x86/Kconfig                     | 9 +++++++++
>  board/google/chromebook_link/Kconfig | 1 +
>  board/intel/crownbay/Kconfig         | 1 +
>  include/configs/chromebook_link.h    | 2 --
>  include/configs/crownbay.h           | 2 --
>  5 files changed, 11 insertions(+), 4 deletions(-)

Applied to u-boot-x86/next, thanks!

[U-Boot] [RESEND PATCH v4 04/12] x86: Hide ROM chip size when CONFIG_X86_RESET_VECTOR is not selected

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> When CONFIG_X86_RESET_VECTOR is not selected, specifying the ROM chip
> size is meaningless, hence hide it.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> Acked-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Changes in v4:
> - Hide XIP_ROM_SIZE too after rebase
>
> Changes in v3: None
> Changes in v2:
> - New patch to hide ROM chip size when CONFIG_X86_RESET_VECTOR is
>   not selected
>
>  arch/x86/Kconfig | 2 ++
>  1 file changed, 2 insertions(+)

Applied to u-boot-x86/next, thanks!

[U-Boot] [RESEND PATCH v4 05/12] x86: coreboot: Make SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE configurable

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> In theory U-Boot built for coreboot is supposed to run as a payload
> to be loaded by coreboot on every board that coreboot supports.
> The U-Boot build process uses SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE
> which are hardcoded in board defconfig and Kconfig files. For better
> support of coreboot, we want to make these two options configurable
> so that we can easily change them during 'make menuconfig' so that
> the generated U-Boot image for coreboot is board configuration aware.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> Acked-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2:
> - New patch to make SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE configurable
>
>  board/coreboot/coreboot/Kconfig | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
>

Applied to u-boot-x86/next, thanks!

[U-Boot] [RESEND PATCH v4 06/12] x86: coreboot: Move coreboot-specific defines from coreboot.h to Kconfig

[Simon Glass]

On 6 January 2015 at 07:28, Simon Glass <sjg@chromium.org> wrote:
> On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
>> There are many places in the U-Boot source tree which refer to
>> CONFIG_SYS_COREBOOT, CONFIG_CBMEM_CONSOLE and CONFIG_VIDEO_COREBOOT
>> that is currently defined in coreboot.h.
>>
>> Move them to arch/x86/cpu/coreboot/Kconfig so that we can switch
>> to board configuration file to build U-Boot later.
>>
>> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>>
>> ---
>>
>> Changes in v4: None
>> Changes in v3:
>> - Make coreboot options wrapped by TARGET_COREBOOT
>> - Remove these options from include/configs/coreboot.h for bisectability
>>
>> Changes in v2:
>> - New patch to move coreboot specific defines from coreboot.h to Kconfig
>>
>>  arch/x86/Kconfig              |  2 ++
>>  arch/x86/cpu/coreboot/Kconfig | 15 +++++++++++++++
>>  include/configs/coreboot.h    |  5 -----
>>  3 files changed, 17 insertions(+), 5 deletions(-)
>>  create mode 100644 arch/x86/cpu/coreboot/Kconfig
>
> Acked-by: Simon Glass <sjg@chromium.org>

Applied to u-boot-x86/next, thanks!

[U-Boot] [RESEND PATCH v4 07/12] x86: Move CONFIG_SYS_CAR_xxx to Kconfig

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> Move CONFIG_SYS_CAR_ADDR and CONFIG_SYS_CAR_SIZE to Kconfig so that
> we don't need them in the board configuration file thus the same
> board configuratoin file can be used to build both coreboot version
> and bare version.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> Acked-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2:
> - New patch to move CONFIG_SYS_CAR_xxx to Kconfig
>
>  board/coreboot/coreboot/Kconfig      | 12 ++++++++++++
>  board/google/chromebook_link/Kconfig |  8 ++++++++
>  include/configs/chromebook_link.h    |  4 ++--
>  3 files changed, 22 insertions(+), 2 deletions(-)
>

Applied to u-boot-x86/next, thanks!

[U-Boot] [RESEND PATCH v4 08/12] x86: Remove include/configs/coreboot.h

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> Since we already swtiched to use the new mechanism for building
> U-Boot for coreboot, coreboot.h is no longer needed so remove it.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> Acked-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2:
> - New patch to remove include/configs/coreboot.h
>
>  board/coreboot/coreboot/MAINTAINERS |  2 +-
>  include/configs/coreboot.h          | 70 -------------------------------------
>  2 files changed, 1 insertion(+), 71 deletions(-)
>  delete mode 100644 include/configs/coreboot.h

Applied to u-boot-x86/next, thanks!

[U-Boot] [RESEND PATCH v4 09/12] x86: Make chromebook_link the default board for coreboot

[Simon Glass]

On 6 January 2015 at 07:30, Simon Glass <sjg@chromium.org> wrote:
> On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
>> Change SYS_CONFIG_NAME and DEFAULT_DEVICE_TREE to chromebook_link
>> which is currently the only real board officially supported to run
>> U-Boot loaded by coreboot.
>>
>> Note the symbolic link file chromebook_link.dts is deleted and
>> link.dts is renamed to chromebook_link.dts.
>>
>> To avoid multiple definition of video_hw_init, the CONFIG_VIDEO_X86
>> define needs to be moved to arch/x86/cpu/ivybridge/Kconfig.
>>
>> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>>
>> ---
>>
>> Changes in v4:
>> - Rebase to u-boot-x86/next
>>
>> Changes in v3:
>> - Move CONFIG_VIDEO_X86 to drivers/video/Kconfig
>>
>> Changes in v2:
>> - New patch to make chromebook_link the default board for coreboot
>>
>>  arch/x86/dts/Makefile             |   3 +-
>>  arch/x86/dts/chromebook_link.dts  | 217 +++++++++++++++++++++++++++++++++++++-
>>  arch/x86/dts/link.dts             | 216 -------------------------------------
>>  board/coreboot/coreboot/Kconfig   |   4 +-
>>  configs/chromebook_link_defconfig |   1 +
>>  configs/coreboot-x86_defconfig    |   1 -
>>  drivers/video/Kconfig             |   8 ++
>>  include/configs/chromebook_link.h |   1 -
>>  8 files changed, 228 insertions(+), 223 deletions(-)
>>  mode change 120000 => 100644 arch/x86/dts/chromebook_link.dts
>>  delete mode 100644 arch/x86/dts/link.dts
>
> Acked-by: Simon Glass <sjg@chromium.org>

Applied to u-boot-x86/next, thanks!

[U-Boot] [RESEND PATCH v4 10/12] x86: coreboot: Wrap cros_ec initialization

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> cros_ec_board_init() should be called only when CONFIG_CROS_EC is
> enabled.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> Acked-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2:
> - Leave CROS_EC defines unchanged in coreboot.h
>
>  board/coreboot/coreboot/coreboot.c | 2 ++
>  1 file changed, 2 insertions(+)

Applied to u-boot-x86/next, thanks!

[U-Boot] [RESEND PATCH v4 11/12] x86: coreboot: Configure pci memory regions

[Simon Glass]

On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
> Configure coreboot pci memory regions so that pci device drivers
> could work correctly.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> Acked-by: Simon Glass <sjg@chromium.org>
>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2:
> - New patch to configure pci memory regions
>
>  arch/x86/cpu/coreboot/pci.c | 30 ++++++++++++++++++++++++++++--
>  1 file changed, 28 insertions(+), 2 deletions(-)

Applied to u-boot-x86/next, thanks!

[U-Boot] [RESEND PATCH v4 12/12] x86: Update README.x86 for coreboot support

[Simon Glass]

On 6 January 2015 at 07:30, Simon Glass <sjg@chromium.org> wrote:
> On 6 January 2015 at 07:14, Bin Meng <bmeng.cn@gmail.com> wrote:
>> Update README.x86 to include new build instructions for U-Boot as
>> the coreboot payload and testing considerations with coreboot.
>>
>> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>> Acked-by: Simon Glass <sjg@chromium.org>
>>
>> ---
>>
>> Changes in v4: None
>> Changes in v3:
>> - Fix several typos in README.x86
>>
>> Changes in v2:
>> - New patch to update REAME.x86 for coreboot support
>>
>>  doc/README.x86 | 39 +++++++++++++++++++++++++++++++++++++--
>>  1 file changed, 37 insertions(+), 2 deletions(-)
>
> Acked-by: Simon Glass <sjg@chromium.org>

Applied to u-boot-x86/next, thanks!