Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37337 )
Change subject: util/romcc: Drop romcc support
......................................................................
util/romcc: Drop romcc support
Finally all boards use a GCC compiled bootblock!
Change-Id: I0c9a1b19dbdc32b43875da7d685718bae9d7f5f4
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
D util/romcc/COPYING
D util/romcc/Makefile
D util/romcc/description.md
D util/romcc/results/linux_test1.out
D util/romcc/results/linux_test10.out
D util/romcc/results/linux_test11.out
D util/romcc/results/linux_test12.out
D util/romcc/results/linux_test13.out
D util/romcc/results/linux_test2.out
D util/romcc/results/linux_test3.out
D util/romcc/results/linux_test4.out
D util/romcc/results/linux_test5.out
D util/romcc/results/linux_test6.out
D util/romcc/results/linux_test7.out
D util/romcc/results/linux_test8.out
D util/romcc/results/linux_test9.out
D util/romcc/romcc.1
D util/romcc/romcc.c
D util/romcc/test.sh
D util/romcc/tests/fail_test1.c
D util/romcc/tests/fail_test10.c
D util/romcc/tests/fail_test11.c
D util/romcc/tests/fail_test2.c
D util/romcc/tests/fail_test3.c
D util/romcc/tests/fail_test4.c
D util/romcc/tests/fail_test5.c
D util/romcc/tests/fail_test6.c
D util/romcc/tests/fail_test7.c
D util/romcc/tests/fail_test8.c
D util/romcc/tests/fail_test9.c
D util/romcc/tests/hello_world.c
D util/romcc/tests/hello_world1.c
D util/romcc/tests/hello_world2.c
D util/romcc/tests/include/linux_console.h
D util/romcc/tests/include/linux_syscall.h
D util/romcc/tests/include/linuxi386_syscall.h
D util/romcc/tests/ldscript.ld
D util/romcc/tests/linux_console.h
D util/romcc/tests/linux_syscall.h
D util/romcc/tests/linux_test1.c
D util/romcc/tests/linux_test10.c
D util/romcc/tests/linux_test11.c
D util/romcc/tests/linux_test12.c
D util/romcc/tests/linux_test13.c
D util/romcc/tests/linux_test2.c
D util/romcc/tests/linux_test3.c
D util/romcc/tests/linux_test4.c
D util/romcc/tests/linux_test5.c
D util/romcc/tests/linux_test6.c
D util/romcc/tests/linux_test7.c
D util/romcc/tests/linux_test8.c
D util/romcc/tests/linux_test9.c
D util/romcc/tests/linuxi386_syscall.h
D util/romcc/tests/raminit_test.c
D util/romcc/tests/raminit_test1.c
D util/romcc/tests/raminit_test2.c
D util/romcc/tests/raminit_test3.c
D util/romcc/tests/raminit_test4.c
D util/romcc/tests/raminit_test5.c
D util/romcc/tests/raminit_test6.c
D util/romcc/tests/raminit_test7.c
D util/romcc/tests/simple_test.c
D util/romcc/tests/simple_test1.c
D util/romcc/tests/simple_test10.c
D util/romcc/tests/simple_test11.c
D util/romcc/tests/simple_test12.c
D util/romcc/tests/simple_test13.c
D util/romcc/tests/simple_test14.c
D util/romcc/tests/simple_test15.c
D util/romcc/tests/simple_test16.c
D util/romcc/tests/simple_test17.c
D util/romcc/tests/simple_test18.c
D util/romcc/tests/simple_test19.c
D util/romcc/tests/simple_test2.c
D util/romcc/tests/simple_test20.c
D util/romcc/tests/simple_test21.c
D util/romcc/tests/simple_test22.c
D util/romcc/tests/simple_test23.c
D util/romcc/tests/simple_test24.c
D util/romcc/tests/simple_test25.c
D util/romcc/tests/simple_test26.c
D util/romcc/tests/simple_test27.c
D util/romcc/tests/simple_test28.c
D util/romcc/tests/simple_test29.c
D util/romcc/tests/simple_test3.c
D util/romcc/tests/simple_test30.c
D util/romcc/tests/simple_test31.c
D util/romcc/tests/simple_test32.c
D util/romcc/tests/simple_test33.c
D util/romcc/tests/simple_test34.c
D util/romcc/tests/simple_test35.c
D util/romcc/tests/simple_test36.c
D util/romcc/tests/simple_test37.c
D util/romcc/tests/simple_test38.c
D util/romcc/tests/simple_test39.c
D util/romcc/tests/simple_test4.c
D util/romcc/tests/simple_test40.c
D util/romcc/tests/simple_test41.c
D util/romcc/tests/simple_test43.c
D util/romcc/tests/simple_test45.c
D util/romcc/tests/simple_test46.c
D util/romcc/tests/simple_test47.c
D util/romcc/tests/simple_test48.c
D util/romcc/tests/simple_test49.c
D util/romcc/tests/simple_test5.c
D util/romcc/tests/simple_test50.c
D util/romcc/tests/simple_test51.c
D util/romcc/tests/simple_test52.c
D util/romcc/tests/simple_test53.c
D util/romcc/tests/simple_test54.c
D util/romcc/tests/simple_test55.c
D util/romcc/tests/simple_test56.c
D util/romcc/tests/simple_test57.c
D util/romcc/tests/simple_test58.c
D util/romcc/tests/simple_test59.c
D util/romcc/tests/simple_test6.c
D util/romcc/tests/simple_test60.c
D util/romcc/tests/simple_test61.c
D util/romcc/tests/simple_test62.c
D util/romcc/tests/simple_test63.c
D util/romcc/tests/simple_test64.c
D util/romcc/tests/simple_test65.c
D util/romcc/tests/simple_test66.c
D util/romcc/tests/simple_test67.c
D util/romcc/tests/simple_test68.c
D util/romcc/tests/simple_test69.c
D util/romcc/tests/simple_test7.c
D util/romcc/tests/simple_test70.c
D util/romcc/tests/simple_test71.c
D util/romcc/tests/simple_test72.c
D util/romcc/tests/simple_test73.c
D util/romcc/tests/simple_test74.c
D util/romcc/tests/simple_test75.c
D util/romcc/tests/simple_test76.c
D util/romcc/tests/simple_test77.c
D util/romcc/tests/simple_test78.c
D util/romcc/tests/simple_test79.c
D util/romcc/tests/simple_test8.c
D util/romcc/tests/simple_test80.c
D util/romcc/tests/simple_test81.c
D util/romcc/tests/simple_test82.c
D util/romcc/tests/simple_test83.c
D util/romcc/tests/simple_test84.c
D util/romcc/tests/simple_test85.c
D util/romcc/tests/simple_test86.c
D util/romcc/tests/simple_test87.c
D util/romcc/tests/simple_test9.c
147 files changed, 0 insertions(+), 50,038 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/37/37337/1
--
To view, visit https://review.coreboot.org/c/coreboot/+/37337
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0c9a1b19dbdc32b43875da7d685718bae9d7f5f4
Gerrit-Change-Number: 37337
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-MessageType: newchange
Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37336 )
Change subject: arch/x86: Drop ROMCC_BOOTBLOCK symbol
......................................................................
arch/x86: Drop ROMCC_BOOTBLOCK symbol
Change-Id: I968c4392849045cd50bfe2c83de44daba38ee245
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/Kconfig
M src/arch/x86/Kconfig
M src/console/Kconfig
M src/cpu/intel/microcode/Kconfig
M src/cpu/x86/Kconfig
M src/mainboard/Kconfig
M src/security/vboot/Kconfig
7 files changed, 2 insertions(+), 33 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/36/37336/1
diff --git a/src/Kconfig b/src/Kconfig
index 2e06299..75e9449 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -1177,9 +1177,6 @@
# src/lib/bootblock.c#main() C entry point.
bool
-config ROMCC_BOOTBLOCK
- bool
-
###############################################################################
# Set default values for symbols created before mainboards. This allows the
# option to be displayed in the general menu, but the default to be loaded in
diff --git a/src/arch/x86/Kconfig b/src/arch/x86/Kconfig
index a788bc0..baea9fb 100644
--- a/src/arch/x86/Kconfig
+++ b/src/arch/x86/Kconfig
@@ -23,7 +23,6 @@
bool
default n
select ARCH_X86
- select BOOTBLOCK_CUSTOM if ROMCC_BOOTBLOCK
config ARCH_VERSTAGE_X86_32
bool
@@ -47,7 +46,6 @@
bool
default n
select ARCH_X86
- select BOOTBLOCK_CUSTOM if ROMCC_BOOTBLOCK
config ARCH_VERSTAGE_X86_64
bool
@@ -235,26 +233,6 @@
Add a spin (JMP .) in assembly_entry.S during early romstage to wait
for a JTAG debugger to break into the execution sequence.
-# Selecting a cbfs prefix from the bootblock is only implemented with romcc
-choice
- prompt "Bootblock behaviour"
- default BOOTBLOCK_SIMPLE
- depends on ROMCC_BOOTBLOCK
-
-config BOOTBLOCK_SIMPLE
- bool "Always load fallback"
-
-config BOOTBLOCK_NORMAL
- select CONFIGURABLE_CBFS_PREFIX
- bool "Switch to normal if CMOS says so"
-
-endchoice
-
-config BOOTBLOCK_SOURCE
- string
- default "bootblock_simple.c" if BOOTBLOCK_SIMPLE
- default "bootblock_normal.c" if BOOTBLOCK_NORMAL
-
config SKIP_MAX_REBOOT_CNT_CLEAR
bool "Do not clear reboot count after successful boot"
depends on BOOTBLOCK_NORMAL
diff --git a/src/console/Kconfig b/src/console/Kconfig
index 9151a32..c0f0f2b 100644
--- a/src/console/Kconfig
+++ b/src/console/Kconfig
@@ -5,7 +5,7 @@
config BOOTBLOCK_CONSOLE
bool "Enable early (bootblock) console output."
- depends on !ROMCC_BOOTBLOCK && !NO_BOOTBLOCK_CONSOLE
+ depends on !NO_BOOTBLOCK_CONSOLE
default y
help
Use console during the bootblock if supported
diff --git a/src/cpu/intel/microcode/Kconfig b/src/cpu/intel/microcode/Kconfig
index 73afe0b..238aad7 100644
--- a/src/cpu/intel/microcode/Kconfig
+++ b/src/cpu/intel/microcode/Kconfig
@@ -1,7 +1,7 @@
config MICROCODE_UPDATE_PRE_RAM
bool
depends on SUPPORT_CPU_UCODE_IN_CBFS
- default y if !ROMCC_BOOTBLOCK
+ default y
help
Select this option if you want to update the microcode
during the cache as ram setup.
diff --git a/src/cpu/x86/Kconfig b/src/cpu/x86/Kconfig
index efb5fa9..76446a0 100644
--- a/src/cpu/x86/Kconfig
+++ b/src/cpu/x86/Kconfig
@@ -77,7 +77,6 @@
config SETUP_XIP_CACHE
bool
- depends on !ROMCC_BOOTBLOCK
depends on !NO_XIP_EARLY_STAGES
help
Select this option to set up an MTRR to cache XIP stages loaded
diff --git a/src/mainboard/Kconfig b/src/mainboard/Kconfig
index 97086b7..c88d317 100644
--- a/src/mainboard/Kconfig
+++ b/src/mainboard/Kconfig
@@ -1,9 +1,5 @@
comment "Important: Run 'make distclean' before switching boards"
-if ROMCC_BOOTBLOCK
-comment "Systems with ROMCC bootblocks will be deprecated soon!"
-endif
-
choice
prompt "Mainboard vendor"
default VENDOR_EMULATION
diff --git a/src/security/vboot/Kconfig b/src/security/vboot/Kconfig
index a829443..e03b51d 100644
--- a/src/security/vboot/Kconfig
+++ b/src/security/vboot/Kconfig
@@ -95,7 +95,6 @@
config VBOOT_STARTS_IN_BOOTBLOCK
bool
default n
- depends on !ROMCC_BOOTBLOCK
help
Firmware verification happens during the end of or right after the
bootblock. This implies that a static VBOOT2_WORK() buffer must be
--
To view, visit https://review.coreboot.org/c/coreboot/+/37336
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I968c4392849045cd50bfe2c83de44daba38ee245
Gerrit-Change-Number: 37336
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-MessageType: newchange
Peichao Li has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37322 )
Change subject: mb/google/hatch/akemi: Set touchpad hold time to 400ns
......................................................................
mb/google/hatch/akemi: Set touchpad hold time to 400ns
According to SI team request, need to tune I2C bus 0 data
hold time more than 300ns
BUG=None
TEST=build firmware and measure I2C bus 0 data hold time
Signed-off-by: Peichao Wang <peichao.wang(a)bitland.corp-partner.google.com>
Change-Id: I75e33419cbaef746487de6ee8628d07cf08adaa9
---
M src/mainboard/google/hatch/variants/akemi/overridetree.cb
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/22/37322/1
diff --git a/src/mainboard/google/hatch/variants/akemi/overridetree.cb b/src/mainboard/google/hatch/variants/akemi/overridetree.cb
index da669e4..937810b 100644
--- a/src/mainboard/google/hatch/variants/akemi/overridetree.cb
+++ b/src/mainboard/google/hatch/variants/akemi/overridetree.cb
@@ -57,6 +57,7 @@
.speed = I2C_SPEED_FAST,
.rise_time_ns = 50,
.fall_time_ns = 15,
+ .data_hold_time_ns = 400,
},
.i2c[1] = {
.speed = I2C_SPEED_FAST,
--
To view, visit https://review.coreboot.org/c/coreboot/+/37322
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I75e33419cbaef746487de6ee8628d07cf08adaa9
Gerrit-Change-Number: 37322
Gerrit-PatchSet: 1
Gerrit-Owner: Peichao Li <peichao.wang(a)bitland.corp-partner.google.com>
Gerrit-MessageType: newchange
Usha P has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37308 )
Change subject: soc/intel/cannonlake: Refactor pch_early_init() code
......................................................................
soc/intel/cannonlake: Refactor pch_early_init() code
This patch keeps required pch_early_init() function like ABASE programming,
GPE and RTC init into bootblock and moves remaining functions like
TCO configuration and SMBUS init into romstage/pch.c in order to maintain
only required chipset programming for bootblock and verstage.
Renamed the pch_init function to bootblock_pch_init and romstage_pch_init
according to the stage it is defined in.
TEST=Able to build and boot hatch successfully.
Change-Id: Idf7b04edc3fce147f7857561ce7d5b0cd05f43fe
Signed-off-by: Usha P <usha.p(a)intel.com>
---
M src/soc/intel/cannonlake/bootblock/bootblock.c
M src/soc/intel/cannonlake/bootblock/pch.c
M src/soc/intel/cannonlake/include/soc/bootblock.h
M src/soc/intel/cannonlake/include/soc/romstage.h
M src/soc/intel/cannonlake/romstage/Makefile.inc
A src/soc/intel/cannonlake/romstage/pch.c
M src/soc/intel/cannonlake/romstage/romstage.c
7 files changed, 36 insertions(+), 13 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/08/37308/1
diff --git a/src/soc/intel/cannonlake/bootblock/bootblock.c b/src/soc/intel/cannonlake/bootblock/bootblock.c
index 9f85397..08c1242 100644
--- a/src/soc/intel/cannonlake/bootblock/bootblock.c
+++ b/src/soc/intel/cannonlake/bootblock/bootblock.c
@@ -74,5 +74,5 @@
*/
gpi_clear_int_cfg();
report_platform_info();
- pch_early_init();
+ bootblock_pch_init();
}
diff --git a/src/soc/intel/cannonlake/bootblock/pch.c b/src/soc/intel/cannonlake/bootblock/pch.c
index 39433a2..a27b507 100644
--- a/src/soc/intel/cannonlake/bootblock/pch.c
+++ b/src/soc/intel/cannonlake/bootblock/pch.c
@@ -2,7 +2,7 @@
* This file is part of the coreboot project.
*
* Copyright (C) 2014 Google Inc.
- * Copyright (C) 2017-2018 Intel Corporation.
+ * Copyright (C) 2017-2019 Intel Corporation.
*
* 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
@@ -25,8 +25,6 @@
#include <intelblocks/pcr.h>
#include <intelblocks/pmclib.h>
#include <intelblocks/rtc.h>
-#include <intelblocks/smbus.h>
-#include <intelblocks/tco.h>
#include <soc/bootblock.h>
#include <soc/iomap.h>
#include <soc/lpc.h>
@@ -35,7 +33,6 @@
#include <soc/pci_devs.h>
#include <soc/pcr_ids.h>
#include <soc/pm.h>
-#include <soc/smbus.h>
#define PCR_PSF3_TO_SHDW_PMC_REG_BASE_CNP_LP 0x1400
#define PCR_PSF3_TO_SHDW_PMC_REG_BASE_CNP_H 0x0980
@@ -180,7 +177,7 @@
pch_enable_lpc();
}
-void pch_early_init(void)
+void bootblock_pch_init(void)
{
/*
* Enabling ABASE for accessing PM1_STS, PM1_EN, PM1_CNT,
@@ -188,12 +185,6 @@
*/
soc_config_acpibase();
- /* Programming TCO_BASE_ADDRESS and TCO Timer Halt */
- tco_configure();
-
- /* Program SMBUS_BASE_ADDRESS and Enable it */
- smbus_common_init();
-
/* Set up GPE configuration */
pmc_gpe_init();
diff --git a/src/soc/intel/cannonlake/include/soc/bootblock.h b/src/soc/intel/cannonlake/include/soc/bootblock.h
index a5c3c32..efc837e 100644
--- a/src/soc/intel/cannonlake/include/soc/bootblock.h
+++ b/src/soc/intel/cannonlake/include/soc/bootblock.h
@@ -23,7 +23,7 @@
void bootblock_pch_early_init(void);
/* Bootblock post console init programming */
-void pch_early_init(void);
+void bootblock_pch_init(void);
void pch_early_iorange_init(void);
void report_platform_info(void);
diff --git a/src/soc/intel/cannonlake/include/soc/romstage.h b/src/soc/intel/cannonlake/include/soc/romstage.h
index 643105a..ab20ee7 100644
--- a/src/soc/intel/cannonlake/include/soc/romstage.h
+++ b/src/soc/intel/cannonlake/include/soc/romstage.h
@@ -24,6 +24,7 @@
/* Provide a callback to allow mainboard to override the DRAM part number. */
void mainboard_get_dram_part_num(const char **part_num, size_t *len);
void systemagent_early_init(void);
+void romstage_pch_init(void);
/* Board type */
enum board_type {
diff --git a/src/soc/intel/cannonlake/romstage/Makefile.inc b/src/soc/intel/cannonlake/romstage/Makefile.inc
index 33d9629..ff3d73d 100644
--- a/src/soc/intel/cannonlake/romstage/Makefile.inc
+++ b/src/soc/intel/cannonlake/romstage/Makefile.inc
@@ -17,3 +17,4 @@
romstage-y += romstage.c
romstage-y += fsp_params.c
romstage-y += systemagent.c
+romstage-y += pch.c
diff --git a/src/soc/intel/cannonlake/romstage/pch.c b/src/soc/intel/cannonlake/romstage/pch.c
new file mode 100644
index 0000000..388ad77
--- /dev/null
+++ b/src/soc/intel/cannonlake/romstage/pch.c
@@ -0,0 +1,28 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2019 Intel Corp.
+ *
+ * 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.
+ */
+
+#include <intelblocks/smbus.h>
+#include <intelblocks/tco.h>
+#include <soc/romstage.h>
+
+void romstage_pch_init(void)
+{
+ /* Programming TCO_BASE_ADDRESS and TCO Timer Halt */
+ tco_configure();
+
+ /* Program SMBUS_BASE_ADDRESS and Enable it */
+ smbus_common_init();
+}
+
diff --git a/src/soc/intel/cannonlake/romstage/romstage.c b/src/soc/intel/cannonlake/romstage/romstage.c
index f782f63..2505683 100644
--- a/src/soc/intel/cannonlake/romstage/romstage.c
+++ b/src/soc/intel/cannonlake/romstage/romstage.c
@@ -132,6 +132,8 @@
/* Program MCHBAR, DMIBAR, GDXBAR and EDRAMBAR */
systemagent_early_init();
+ /* Program PCH init */
+ romstage_pch_init();
/* initialize Heci interface */
heci_init(HECI1_BASE_ADDRESS);
--
To view, visit https://review.coreboot.org/c/coreboot/+/37308
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Idf7b04edc3fce147f7857561ce7d5b0cd05f43fe
Gerrit-Change-Number: 37308
Gerrit-PatchSet: 1
Gerrit-Owner: Usha P <usha.p(a)intel.com>
Gerrit-MessageType: newchange
Martin Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/35429 )
Change subject: PDCurses: Remove unused code/directories
......................................................................
PDCurses: Remove unused code/directories
There are a number of directories that came in with pdcurses that we
don't need as a part of libpayload. Get rid of them.
Signed-off-by: Martin Roth <martin(a)coreboot.org>
Change-Id: I3b09104f64254d768d27a3d2e3fba8418e7ba2ca
---
D payloads/libpayload/curses/PDCurses/demos/README
D payloads/libpayload/curses/PDCurses/demos/firework.c
D payloads/libpayload/curses/PDCurses/demos/newdemo.c
D payloads/libpayload/curses/PDCurses/demos/ptest.c
D payloads/libpayload/curses/PDCurses/demos/rain.c
D payloads/libpayload/curses/PDCurses/demos/testcurs.c
D payloads/libpayload/curses/PDCurses/demos/tui.c
D payloads/libpayload/curses/PDCurses/demos/tui.h
D payloads/libpayload/curses/PDCurses/demos/tuidemo.c
D payloads/libpayload/curses/PDCurses/demos/worm.c
D payloads/libpayload/curses/PDCurses/demos/xmas.c
D payloads/libpayload/curses/PDCurses/dos/README
D payloads/libpayload/curses/PDCurses/dos/bccdos.lrf
D payloads/libpayload/curses/PDCurses/dos/bccdos.mak
D payloads/libpayload/curses/PDCurses/dos/gccdos.mak
D payloads/libpayload/curses/PDCurses/dos/mscdos.lrf
D payloads/libpayload/curses/PDCurses/dos/mscdos.mak
D payloads/libpayload/curses/PDCurses/dos/pdcclip.c
D payloads/libpayload/curses/PDCurses/dos/pdcdisp.c
D payloads/libpayload/curses/PDCurses/dos/pdcdos.h
D payloads/libpayload/curses/PDCurses/dos/pdcgetsc.c
D payloads/libpayload/curses/PDCurses/dos/pdckbd.c
D payloads/libpayload/curses/PDCurses/dos/pdcscrn.c
D payloads/libpayload/curses/PDCurses/dos/pdcsetsc.c
D payloads/libpayload/curses/PDCurses/dos/pdcutil.c
D payloads/libpayload/curses/PDCurses/dos/wccdos16.mak
D payloads/libpayload/curses/PDCurses/dos/wccdos4g.mak
D payloads/libpayload/curses/PDCurses/os2/README
D payloads/libpayload/curses/PDCurses/os2/bccos2.mak
D payloads/libpayload/curses/PDCurses/os2/gccos2.mak
D payloads/libpayload/curses/PDCurses/os2/iccos2.lrf
D payloads/libpayload/curses/PDCurses/os2/iccos2.mak
D payloads/libpayload/curses/PDCurses/os2/pdcclip.c
D payloads/libpayload/curses/PDCurses/os2/pdcdisp.c
D payloads/libpayload/curses/PDCurses/os2/pdcgetsc.c
D payloads/libpayload/curses/PDCurses/os2/pdckbd.c
D payloads/libpayload/curses/PDCurses/os2/pdcos2.h
D payloads/libpayload/curses/PDCurses/os2/pdcscrn.c
D payloads/libpayload/curses/PDCurses/os2/pdcsetsc.c
D payloads/libpayload/curses/PDCurses/os2/pdcutil.c
D payloads/libpayload/curses/PDCurses/os2/wccos2.mak
D payloads/libpayload/curses/PDCurses/sdl1/Makefile
D payloads/libpayload/curses/PDCurses/sdl1/Makefile.mng
D payloads/libpayload/curses/PDCurses/sdl1/README
D payloads/libpayload/curses/PDCurses/sdl1/deffont.h
D payloads/libpayload/curses/PDCurses/sdl1/deficon.h
D payloads/libpayload/curses/PDCurses/sdl1/pdcclip.c
D payloads/libpayload/curses/PDCurses/sdl1/pdcdisp.c
D payloads/libpayload/curses/PDCurses/sdl1/pdcgetsc.c
D payloads/libpayload/curses/PDCurses/sdl1/pdckbd.c
D payloads/libpayload/curses/PDCurses/sdl1/pdcscrn.c
D payloads/libpayload/curses/PDCurses/sdl1/pdcsdl.h
D payloads/libpayload/curses/PDCurses/sdl1/pdcsetsc.c
D payloads/libpayload/curses/PDCurses/sdl1/pdcutil.c
D payloads/libpayload/curses/PDCurses/sdl1/sdltest.c
D payloads/libpayload/curses/PDCurses/win32/README
D payloads/libpayload/curses/PDCurses/win32/bccwin32.mak
D payloads/libpayload/curses/PDCurses/win32/dmcwin32.mak
D payloads/libpayload/curses/PDCurses/win32/gccwin32.mak
D payloads/libpayload/curses/PDCurses/win32/lccwin32.mak
D payloads/libpayload/curses/PDCurses/win32/mingwin32.mak
D payloads/libpayload/curses/PDCurses/win32/pdcclip.c
D payloads/libpayload/curses/PDCurses/win32/pdcdisp.c
D payloads/libpayload/curses/PDCurses/win32/pdcgetsc.c
D payloads/libpayload/curses/PDCurses/win32/pdckbd.c
D payloads/libpayload/curses/PDCurses/win32/pdcscrn.c
D payloads/libpayload/curses/PDCurses/win32/pdcsetsc.c
D payloads/libpayload/curses/PDCurses/win32/pdcurses.ico
D payloads/libpayload/curses/PDCurses/win32/pdcurses.rc
D payloads/libpayload/curses/PDCurses/win32/pdcutil.c
D payloads/libpayload/curses/PDCurses/win32/pdcwin.h
D payloads/libpayload/curses/PDCurses/win32/vcwin32.mak
D payloads/libpayload/curses/PDCurses/win32/wccwin32.mak
D payloads/libpayload/curses/PDCurses/x11/Makefile.aix.in
D payloads/libpayload/curses/PDCurses/x11/Makefile.in
D payloads/libpayload/curses/PDCurses/x11/README
D payloads/libpayload/curses/PDCurses/x11/ScrollBox.c
D payloads/libpayload/curses/PDCurses/x11/ScrollBox.h
D payloads/libpayload/curses/PDCurses/x11/ScrollBoxP.h
D payloads/libpayload/curses/PDCurses/x11/big_icon.xbm
D payloads/libpayload/curses/PDCurses/x11/compose.h
D payloads/libpayload/curses/PDCurses/x11/little_icon.xbm
D payloads/libpayload/curses/PDCurses/x11/ncurses_cfg.h
D payloads/libpayload/curses/PDCurses/x11/pdcclip.c
D payloads/libpayload/curses/PDCurses/x11/pdcdisp.c
D payloads/libpayload/curses/PDCurses/x11/pdcgetsc.c
D payloads/libpayload/curses/PDCurses/x11/pdckbd.c
D payloads/libpayload/curses/PDCurses/x11/pdcscrn.c
D payloads/libpayload/curses/PDCurses/x11/pdcsetsc.c
D payloads/libpayload/curses/PDCurses/x11/pdcutil.c
D payloads/libpayload/curses/PDCurses/x11/pdcx11.c
D payloads/libpayload/curses/PDCurses/x11/pdcx11.h
D payloads/libpayload/curses/PDCurses/x11/sb.c
D payloads/libpayload/curses/PDCurses/x11/x11.c
D payloads/libpayload/curses/PDCurses/x11/xcurses-config.in
95 files changed, 0 insertions(+), 20,512 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/35429/1
--
To view, visit https://review.coreboot.org/c/coreboot/+/35429
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3b09104f64254d768d27a3d2e3fba8418e7ba2ca
Gerrit-Change-Number: 35429
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth(a)google.com>
Gerrit-MessageType: newchange
Kyösti Mälkki has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/30856
Change subject: arch/x86: Drop Kconfig SIPI_VECTOR_IN_ROM
......................................................................
arch/x86: Drop Kconfig SIPI_VECTOR_IN_ROM
This was used to enforce 4kiB alignment of _start16bit in
romcc bootblock. Platforms requiring this moved away to
C_ENVIRONMENT_BOOTBLOCK that globally forces the alignment.
Change-Id: I8ca453bbc56ab2aeb127f3e081c69e1b38bb8396
Signed-off-by: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
---
M src/arch/x86/Kconfig
M src/arch/x86/failover.ld
M src/cpu/intel/car/p4-netburst/cache_as_ram.S
M src/cpu/intel/model_106cx/Kconfig
M src/cpu/intel/socket_LGA775/Kconfig
M src/cpu/intel/socket_mPGA604/Kconfig
M src/cpu/x86/16bit/entry16.inc
7 files changed, 5 insertions(+), 26 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/30856/1
diff --git a/src/arch/x86/Kconfig b/src/arch/x86/Kconfig
index 242a7cf..c2fc914 100644
--- a/src/arch/x86/Kconfig
+++ b/src/arch/x86/Kconfig
@@ -81,13 +81,6 @@
default n
depends on ARCH_X86 && SMP
-# Aligns 16bit entry code in bootblock so that hyper-threading CPUs
-# can boot AP CPUs to enable their shared caches.
-config SIPI_VECTOR_IN_ROM
- bool
- default n
- depends on ARCH_X86
-
# Set the rambase for systems that still need it, only 5 chipsets as of
# Sep 2018. This value was 0x100000, chosen to match the entry point
# of Linux 2.2 in 1999. The new value, 14 MiB, makes a lot more sense
diff --git a/src/arch/x86/failover.ld b/src/arch/x86/failover.ld
index b32aa29..eabc9f7 100644
--- a/src/arch/x86/failover.ld
+++ b/src/arch/x86/failover.ld
@@ -23,12 +23,11 @@
TARGET(binary)
SECTIONS
{
- /* Symbol ap_sipi_vector must be aligned to 4kB to start AP CPUs
- * with Startup IPI message without RAM. Align .rom to next 4 byte
- * boundary anyway, so no pad byte appears between _rom and _start.
+ /* Align .rom to 4 byte boundary so no pad byte appears
+ * between _rom and _start.
*/
.bogus ROMLOC_MIN : {
- . = CONFIG_SIPI_VECTOR_IN_ROM ? ALIGN(4096) : ALIGN(4);
+ . = ALIGN(4);
ROMLOC = .;
} >rom = 0xff
@@ -49,12 +48,7 @@
* may cause the total size of a section to change when the start
* address gets applied.
*/
- ROMLOC_MIN = 0xffffff00 - (_erom - _rom + 16) -
- (CONFIG_SIPI_VECTOR_IN_ROM ? 4096 : 0);
-
- /* Post-check proper SIPI vector. */
- _bogus = ASSERT(!CONFIG_SIPI_VECTOR_IN_ROM || (ap_sipi_vector_in_rom == 0xff),
- "Address mismatch on AP_SIPI_VECTOR");
+ ROMLOC_MIN = 0xffffff00 - (_erom - _rom + 16);
/DISCARD/ : {
*(.comment)
diff --git a/src/cpu/intel/car/p4-netburst/cache_as_ram.S b/src/cpu/intel/car/p4-netburst/cache_as_ram.S
index fda572d..5c579a1 100644
--- a/src/cpu/intel/car/p4-netburst/cache_as_ram.S
+++ b/src/cpu/intel/car/p4-netburst/cache_as_ram.S
@@ -23,10 +23,6 @@
/* Macro to access Local APIC registers at default base. */
#define LAPIC(x) $(LAPIC_DEFAULT_BASE | LAPIC_ ## x)
-#if !IS_ENABLED(CONFIG_C_ENVIRONMENT_BOOTBLOCK)
-/* Fixed location, ASSERTED in failover.ld if it changes. */
-.set ap_sipi_vector_in_rom, 0xff
-#endif
#define CACHE_AS_RAM_SIZE CONFIG_DCACHE_RAM_SIZE
#define CACHE_AS_RAM_BASE CONFIG_DCACHE_RAM_BASE
diff --git a/src/cpu/intel/model_106cx/Kconfig b/src/cpu/intel/model_106cx/Kconfig
index f365cf1..2a324fb 100644
--- a/src/cpu/intel/model_106cx/Kconfig
+++ b/src/cpu/intel/model_106cx/Kconfig
@@ -7,7 +7,6 @@
select SMP
select SSE2
select UDELAY_LAPIC
- select SIPI_VECTOR_IN_ROM
select AP_IN_SIPI_WAIT
select TSC_SYNC_MFENCE
select SUPPORT_CPU_UCODE_IN_CBFS
diff --git a/src/cpu/intel/socket_LGA775/Kconfig b/src/cpu/intel/socket_LGA775/Kconfig
index 8b227bd..6c3d837 100644
--- a/src/cpu/intel/socket_LGA775/Kconfig
+++ b/src/cpu/intel/socket_LGA775/Kconfig
@@ -13,7 +13,6 @@
select CPU_INTEL_MODEL_1067X
select MMX
select SSE
- select SIPI_VECTOR_IN_ROM
config DCACHE_RAM_SIZE
hex
diff --git a/src/cpu/intel/socket_mPGA604/Kconfig b/src/cpu/intel/socket_mPGA604/Kconfig
index ca2f7b3..e860ded 100644
--- a/src/cpu/intel/socket_mPGA604/Kconfig
+++ b/src/cpu/intel/socket_mPGA604/Kconfig
@@ -9,7 +9,6 @@
select MMX
select SSE
select UDELAY_TSC
- select SIPI_VECTOR_IN_ROM
select C_ENVIRONMENT_BOOTBLOCK
# mPGA604 are usually Intel Netburst CPUs which should have SSE2
diff --git a/src/cpu/x86/16bit/entry16.inc b/src/cpu/x86/16bit/entry16.inc
index 2a9f8c5..f110980 100644
--- a/src/cpu/x86/16bit/entry16.inc
+++ b/src/cpu/x86/16bit/entry16.inc
@@ -29,8 +29,7 @@
#include <arch/rom_segs.h>
-#if IS_ENABLED(CONFIG_C_ENVIRONMENT_BOOTBLOCK) || \
- IS_ENABLED(CONFIG_SIPI_VECTOR_IN_ROM)
+#if IS_ENABLED(CONFIG_C_ENVIRONMENT_BOOTBLOCK)
/* Symbol _start16bit must be aligned to 4kB to start AP CPUs with
* Startup IPI message without RAM.
*/
--
To view, visit https://review.coreboot.org/c/coreboot/+/30856
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8ca453bbc56ab2aeb127f3e081c69e1b38bb8396
Gerrit-Change-Number: 30856
Gerrit-PatchSet: 1
Gerrit-Owner: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-MessageType: newchange
Richard Spiegel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/34662 )
Change subject: Documentation/binaries: Add AMD FSP documentation
......................................................................
Documentation/binaries: Add AMD FSP documentation
Create a document explaining, at a high level, the differences between
Intel's FSP and the one developed by AMD.
BUG=none.
TEST=none.
Change-Id: I59a5d34df93cd0ff647e2ccfdbf8700b4df00a59
Signed-off-by: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
---
A Documentation/binaries/AMD_FSP_family_17h.md
A Documentation/binaries/index.md
2 files changed, 49 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/62/34662/1
diff --git a/Documentation/binaries/AMD_FSP_family_17h.md b/Documentation/binaries/AMD_FSP_family_17h.md
new file mode 100644
index 0000000..b46beb3
--- /dev/null
+++ b/Documentation/binaries/AMD_FSP_family_17h.md
@@ -0,0 +1,41 @@
+# FSP implementation differences between Intel and AMD
+
+## Introduction
+Starting with family 17h, AMD is developing an "_As Close As Possible_" FSP
+binary. However, some premisses are different for family 17h and beyond,
+making it necessary to have some FSP implementation differences. Some other
+implementation differences were more of an engineering decision.
+
+The family 17h deviation from older AMD and Intel CPU/SOC are:
+* The memory is initialized by the PSP (similar to Intel's ME) ARM.
+* There's _**no support**_ for cache as RAM.
+* Reset vector is not the old 0xFFFFFFF0.
+
+This document is a "work in progress", documenting the differences at a
+high level. This document will be updated as more information becomes
+available.
+
+## Differences caused by differences in premisses
+1. **No FSP-T**
+Because family 17h does not support CAR, there's no FSP-T.
+2. **FSP-M only reports memory**
+Because memory is inittialized by the PSP, FSP-M only reports the final
+memory configuration.
+3. **FSP-M is loaded to DRAM**
+PSP can be made to load a section of the flash into RAM before releasing
+the reset, thus FSP-M can be made to run directly from memory.
+4. **FSP-M can be made position independent**
+Because it's loaded to memory and does not uses CAR, FSP-M can be made PIC
+(Position Independent Code).
+
+## Differences by engineering decision
+1. **Memory fragmentation**
+Though FSP still fragments memory, it has added control for flexibility
+of where the chunks will reside.
+2. **UPD interface**
+UPD interface uses native intergers and don't need to be packed by compiler.
+3. **UPD with no UEFI dependencies**
+UPD interface can be made C99 or C11 compatible with no hard dependencies
+to UEFI.
+4. **Platform specific code**
+Similar to AGESA, FSP will make call back to platform specific code.
diff --git a/Documentation/binaries/index.md b/Documentation/binaries/index.md
new file mode 100644
index 0000000..9093bf7
--- /dev/null
+++ b/Documentation/binaries/index.md
@@ -0,0 +1,8 @@
+# binaries-specific documentation
+
+This section contains documentation about any binary used by coreboot
+
+## Video
+
+## Platform initialization
+- [AMD FSP](AMD_FSP_family_17h.md)
--
To view, visit https://review.coreboot.org/c/coreboot/+/34662
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I59a5d34df93cd0ff647e2ccfdbf8700b4df00a59
Gerrit-Change-Number: 34662
Gerrit-PatchSet: 1
Gerrit-Owner: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-MessageType: newchange