Hello HAOUAS Elyes, Subrata Banik, Tristan Corrick, build bot (Jenkins), Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/29680
to look at the new patch set (#4).
Change subject: Kconfig: Unify power-after-failure options
......................................................................
Kconfig: Unify power-after-failure options
The newest and most useful incarnation was hiding in soc/intel/common/.
We move it into the Mainboard menu and extend it with various flags to
be selected to control the default and which options are visible. Also
add a new `int` config MAINBOARD_POWER_FAILURE_STATE that moves the
boolean to int conversion into Kconfig:
0 - S5
1 - S0
2 - previous state
This patch focuses on the Kconfig code. The C code could be unified as
well, e.g. starting with a common enum and safe wrapper around the
get_option() call.
TEST=Did what-jenkins-does with and without this commit and compared
binaries. Nothing changed for the default configurations.
Change-Id: I61259f864c8a8cfc7099cc2699059f972fa056c0
Signed-off-by: Nico Huber <nico.h(a)gmx.de>
---
M src/mainboard/Kconfig
M src/mainboard/asrock/h81m-hds/Kconfig
M src/mainboard/asus/kcma-d8/Kconfig
M src/mainboard/asus/kfsn4-dre/Kconfig
M src/mainboard/asus/kgpe-d16/Kconfig
M src/mainboard/msi/ms9652_fam10/Kconfig
M src/mainboard/samsung/lumpy/Kconfig
M src/mainboard/samsung/stumpy/Kconfig
M src/soc/intel/broadwell/Kconfig
M src/soc/intel/broadwell/lpc.c
M src/soc/intel/broadwell/smihandler.c
M src/soc/intel/common/block/include/intelblocks/pmclib.h
M src/soc/intel/common/block/pmc/Kconfig
M src/soc/intel/common/block/pmc/pmclib.c
M src/southbridge/amd/agesa/hudson/sm.c
M src/southbridge/amd/amd8111/Kconfig
M src/southbridge/amd/amd8111/acpi.c
M src/southbridge/amd/pi/hudson/sm.c
M src/southbridge/amd/sb700/Kconfig
M src/southbridge/amd/sb700/sm.c
M src/southbridge/amd/sb800/sm.c
M src/southbridge/intel/bd82x6x/lpc.c
M src/southbridge/intel/common/Kconfig
M src/southbridge/intel/common/pmutil.h
M src/southbridge/intel/common/smihandler.c
M src/southbridge/intel/fsp_rangeley/soc.h
M src/southbridge/intel/i82801dx/Kconfig
M src/southbridge/intel/i82801dx/i82801dx.h
M src/southbridge/intel/i82801dx/lpc.c
M src/southbridge/intel/i82801dx/smihandler.c
M src/southbridge/intel/i82801gx/Kconfig
M src/southbridge/intel/i82801gx/i82801gx.h
M src/southbridge/intel/i82801gx/lpc.c
M src/southbridge/intel/i82801gx/smihandler.c
M src/southbridge/intel/i82801ix/i82801ix.h
M src/southbridge/intel/i82801ix/lpc.c
M src/southbridge/intel/i82801jx/Kconfig
M src/southbridge/intel/i82801jx/i82801jx.h
M src/southbridge/intel/i82801jx/lpc.c
M src/southbridge/intel/ibexpeak/Kconfig
M src/southbridge/intel/ibexpeak/lpc.c
M src/southbridge/intel/ibexpeak/pch.h
M src/southbridge/intel/ibexpeak/smihandler.c
M src/southbridge/intel/lynxpoint/Kconfig
M src/southbridge/intel/lynxpoint/lpc.c
M src/southbridge/intel/lynxpoint/pch.h
M src/southbridge/intel/lynxpoint/smihandler.c
M src/southbridge/nvidia/ck804/Kconfig
M src/southbridge/nvidia/ck804/lpc.c
M src/southbridge/nvidia/mcp55/Kconfig
M src/southbridge/nvidia/mcp55/lpc.c
M src/superio/nuvoton/nct5572d/Kconfig
M src/superio/nuvoton/nct5572d/superio.c
M src/superio/winbond/w83667hg-a/Kconfig
M src/superio/winbond/w83667hg-a/superio.c
55 files changed, 121 insertions(+), 164 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/29680/4
--
To view, visit https://review.coreboot.org/c/coreboot/+/29680
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I61259f864c8a8cfc7099cc2699059f972fa056c0
Gerrit-Change-Number: 29680
Gerrit-PatchSet: 4
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: HAOUAS Elyes <ehaouas(a)noos.fr>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Tristan Corrick <tristan(a)corrick.kiwi>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Hellsenberg <th3fanbus(a)gmail.com>
Gerrit-MessageType: newpatchset
PraveenX Hodagatta Pranesh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/29808
Change subject: soc/intel/skylake: Add device settings for PL4 power limit
......................................................................
soc/intel/skylake: Add device settings for PL4 power limit
PL4 is a preemptive CPU package peak power limit,it will never be exceeded.
Power is preemptively lowered before limit is reached.
This change provides option in devicetree and feeds FSP PowerLimit4 UPD for
power limit purpose.
Signed-off-by: Praveen hodagatta pranesh <praveenx.hodagatta.pranesh(a)intel.com>
Change-Id: I64b5a029104a102e5741e8b37c7992f2693180e8
---
M src/soc/intel/skylake/chip.h
M src/soc/intel/skylake/chip_fsp20.c
2 files changed, 4 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/08/29808/1
diff --git a/src/soc/intel/skylake/chip.h b/src/soc/intel/skylake/chip.h
index b1ffcb2..21dddd4 100644
--- a/src/soc/intel/skylake/chip.h
+++ b/src/soc/intel/skylake/chip.h
@@ -97,6 +97,9 @@
/* TCC activation offset */
int tcc_offset;
+ /* Power Limit Related */
+ u32 PowerLimit4;
+
/* PL2 Override value in Watts */
u32 tdp_pl2_override;
/* PL1 Override value in Watts */
diff --git a/src/soc/intel/skylake/chip_fsp20.c b/src/soc/intel/skylake/chip_fsp20.c
index 8a78348..18c2aef 100644
--- a/src/soc/intel/skylake/chip_fsp20.c
+++ b/src/soc/intel/skylake/chip_fsp20.c
@@ -369,6 +369,7 @@
tconfig->PchLockDownGlobalSmi = config->LockDownConfigGlobalSmi;
tconfig->PchLockDownRtcLock = config->LockDownConfigRtcLock;
+ tconfig->PowerLimit4 = config->PowerLimit4;
/*
* To disable HECI, the Psf needs to be left unlocked
* by FSP till end of post sequence. Based on the devicetree
--
To view, visit https://review.coreboot.org/c/coreboot/+/29808
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I64b5a029104a102e5741e8b37c7992f2693180e8
Gerrit-Change-Number: 29808
Gerrit-PatchSet: 1
Gerrit-Owner: PraveenX Hodagatta Pranesh <praveenx.hodagatta.pranesh(a)intel.com>
Gerrit-MessageType: newchange
Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/29805
Change subject: Documentation/../../icelake: Add Ice Lake coreboot development documentation
......................................................................
Documentation/../../icelake: Add Ice Lake coreboot development documentation
Add documentation for Ice Lake processor family coreboot development.
Documented so far:
* What is Ice Lake
* Development Strategy
* Create coreboot Image
* Flashing coreboot
Change-Id: Ief4df6ca11f95b75ecddeb560f7887bfadced086
Signed-off-by: Subrata Banik <subrata.banik(a)intel.com>
---
A Documentation/soc/intel/icelake/IceLake_Coreboot_Development.md
M Documentation/soc/intel/icelake/index.md
2 files changed, 60 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/05/29805/1
diff --git a/Documentation/soc/intel/icelake/IceLake_Coreboot_Development.md b/Documentation/soc/intel/icelake/IceLake_Coreboot_Development.md
new file mode 100644
index 0000000..d60cf79
--- /dev/null
+++ b/Documentation/soc/intel/icelake/IceLake_Coreboot_Development.md
@@ -0,0 +1,56 @@
+# Intel Ice Lake coreboot development
+
+## Introduction
+
+This document captures the coreboot development strategy for Intel SoC named Ice lake.
+
+The Ice Lake processor family is the next generation Intel® Core processor family.
+These processors utilize Intels industry-leading 10 nm+ process technology.
+
+* Reference: https://www.intel.in/content/www/in/en/design/products-and-solutions/proces…
+
+## Development Strategy
+
+Like any other Intel SoC, Ice Lake coreboot development is also based on "Intel common code development model".
+
+1. Intel develops initial Firmware code for Ice Lake SoC.
+ * CL: https://review.coreboot.org/#/c/coreboot/+/29162/
+
+2. Additionally provides Firmware code support for Intel Reference Platform (RVP), known as Ice lake RVP with same SoC.
+ * CL: https://review.coreboot.org/#/c/coreboot/+/29164/
+
+3. OEMs to design based on reference platform and make use of mainboard sample code. Rigth now Dragonegg is one of Ice Lake based mainboard developed by Google
+ * CL: https://review.coreboot.org/#/c/coreboot/+/29749/
+
+### Summary:
+* SoC as Ice Lake.
+* Reference platform as icelake_rvp.
+* OEM board as Dragonegg.
+
+## Create coreboot Image
+
+1. Clone latest coreboot code as below
+$ git clone http://review.coreboot.org/p/coreboot
+
+2. Place blobs (ucode, me.bin and FSP packages) in appropriate locations
+
+Note:
+Consider the fact that ucode an ME binary release process will remain same for Ice Lake program as well.
+After PRQ,FSP binary will be available externally as any other program.
+
+3. Create coreboot .config
+
+4. Build toolchain
+
+CPUS=$(nproc--ignore=1) make crossgcc-i386 iasl
+
+5. Build image
+$ make # the image is generated as build/coreboot.rom
+
+## Flashing coreboot
+
+Flashing is the same as other Intel boards:
+
+$ dut-control spi2_vref:pp3300 spi2_buf_en:on spi2_buf_on_flex_en:on warm_reset:on
+$ sudo flashrom -n -p ft2232_spi:type=servo-v2 -w <bios_image>
+$ dut-control spi2_vref:off spi2_buf_en:off spi2_buf_on_flex_en:off warm_reset:off
diff --git a/Documentation/soc/intel/icelake/index.md b/Documentation/soc/intel/icelake/index.md
index b4f512c..c295f8a 100644
--- a/Documentation/soc/intel/icelake/index.md
+++ b/Documentation/soc/intel/icelake/index.md
@@ -2,6 +2,10 @@
This section contains documentation about coreboot on specific Intel "Ice Lake" SOCs.
+## Ice Lake coreboot development
+
+- [Ice Lake coreboot development](IceLake_Coreboot_Development.md)
+
## Multiprocessor Init
- [Multiprocessor Init](MultiProcessorInit.md)
--
To view, visit https://review.coreboot.org/c/coreboot/+/29805
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ief4df6ca11f95b75ecddeb560f7887bfadced086
Gerrit-Change-Number: 29805
Gerrit-PatchSet: 1
Gerrit-Owner: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-MessageType: newchange
Sumeet R Pawnikar has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/29804
Change subject: mb/intel/icelake_rvp: Add DPTF information
......................................................................
mb/intel/icelake_rvp: Add DPTF information
Currently, using these placeholder values. These will be
updated after thermal tuning.
Change-Id: Id206417e2e8029abe1de2a35f58dc4e801f3986b
Signed-off-by: Sumeet Pawnikar <sumeet.r.pawnikar(a)intel.com>
---
A src/mainboard/intel/icelake_rvp/acpi/dptf.asl
M src/mainboard/intel/icelake_rvp/dsdt.asl
2 files changed, 47 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/04/29804/1
diff --git a/src/mainboard/intel/icelake_rvp/acpi/dptf.asl b/src/mainboard/intel/icelake_rvp/acpi/dptf.asl
new file mode 100644
index 0000000..cbe0792
--- /dev/null
+++ b/src/mainboard/intel/icelake_rvp/acpi/dptf.asl
@@ -0,0 +1,43 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2018 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
+ * 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.
+ */
+
+#define DPTF_CPU_PASSIVE 80
+#define DPTF_CPU_CRITICAL 99
+
+Name (DTRT, Package () {
+ /* CPU Throttle Effect on CPU */
+ Package () { \_SB.PCI0.B0D4, \_SB.PCI0.B0D4, 100, 50, 0, 0, 0, 0 },
+})
+
+Name (MPPC, Package ()
+{
+ 0x2, /* Revision */
+ Package () { /* Power Limit 1 */
+ 0, /* PowerLimitIndex, 0 for Power Limit 1 */
+ 1600, /* PowerLimitMinimum */
+ 15000, /* PowerLimitMaximum */
+ 1000, /* TimeWindowMinimum */
+ 1000, /* TimeWindowMaximum */
+ 200 /* StepSize */
+ },
+ Package () { /* Power Limit 2 */
+ 1, /* PowerLimitIndex, 1 for Power Limit 2 */
+ 8000, /* PowerLimitMinimum */
+ 8000, /* PowerLimitMaximum */
+ 1000, /* TimeWindowMinimum */
+ 1000, /* TimeWindowMaximum */
+ 1000 /* StepSize */
+ }
+})
diff --git a/src/mainboard/intel/icelake_rvp/dsdt.asl b/src/mainboard/intel/icelake_rvp/dsdt.asl
index 7951501..2cc4079 100644
--- a/src/mainboard/intel/icelake_rvp/dsdt.asl
+++ b/src/mainboard/intel/icelake_rvp/dsdt.asl
@@ -33,7 +33,11 @@
{
#include <soc/intel/icelake/acpi/northbridge.asl>
#include <soc/intel/icelake/acpi/southbridge.asl>
+
}
+
+ // Dynamic Platform Thermal Framework
+ #include "acpi/dptf.asl"
}
#if IS_ENABLED(CONFIG_CHROMEOS)
--
To view, visit https://review.coreboot.org/c/coreboot/+/29804
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id206417e2e8029abe1de2a35f58dc4e801f3986b
Gerrit-Change-Number: 29804
Gerrit-PatchSet: 1
Gerrit-Owner: Sumeet R Pawnikar <sumeet.r.pawnikar(a)intel.com>
Gerrit-MessageType: newchange
Werner Zeh has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/29776 )
Change subject: util/cbfstool: Fix GCC error due to a shadowed declaration
......................................................................
util/cbfstool: Fix GCC error due to a shadowed declaration
There is already a function with the name buffer_size(). Adding a local
variable with the same name will lead to the following error on older
GCC versions (e.g. version 4.4.7):
declaration of 'buffer_size' shadows a global declaration
To fix this rename the local variable to buffer_len.
Change-Id: Ifae3a17152f2f9852d29a4ac038f7e5a75a41614
Signed-off-by: Werner Zeh <werner.zeh(a)siemens.com>
Reviewed-on: https://review.coreboot.org/c/29776
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Joel Kitching <kitching(a)google.com>
Reviewed-by: Mario Scheithauer <mario.scheithauer(a)siemens.com>
---
M util/cbfstool/cbfs_image.c
1 file changed, 5 insertions(+), 5 deletions(-)
Approvals:
build bot (Jenkins): Verified
Mario Scheithauer: Looks good to me, approved
Joel Kitching: Looks good to me, approved
diff --git a/util/cbfstool/cbfs_image.c b/util/cbfstool/cbfs_image.c
index 3c5d29c..6ccc4f9 100644
--- a/util/cbfstool/cbfs_image.c
+++ b/util/cbfstool/cbfs_image.c
@@ -1299,7 +1299,7 @@
unsigned int decompressed_size = 0;
unsigned int compression = cbfs_file_get_compression_info(entry,
&decompressed_size);
- unsigned int buffer_size;
+ unsigned int buffer_len;
decomp_func_ptr decompress;
if (do_processing) {
@@ -1308,11 +1308,11 @@
ERROR("looking up decompression routine failed\n");
return -1;
}
- buffer_size = decompressed_size;
+ buffer_len = decompressed_size;
} else {
/* Force nop decompression */
decompress = decompression_function(CBFS_COMPRESS_NONE);
- buffer_size = compressed_size;
+ buffer_len = compressed_size;
}
LOG("Found file %.30s at 0x%x, type %.12s, compressed %d, size %d\n",
@@ -1321,8 +1321,8 @@
decompressed_size);
buffer_init(&buffer, strdup("(cbfs_export_entry)"), NULL, 0);
- buffer.data = malloc(buffer_size);
- buffer.size = buffer_size;
+ buffer.data = malloc(buffer_len);
+ buffer.size = buffer_len;
if (decompress(CBFS_SUBHEADER(entry), compressed_size,
buffer.data, buffer.size, NULL)) {
--
To view, visit https://review.coreboot.org/c/coreboot/+/29776
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ifae3a17152f2f9852d29a4ac038f7e5a75a41614
Gerrit-Change-Number: 29776
Gerrit-PatchSet: 3
Gerrit-Owner: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Reviewer: Joel Kitching <kitching(a)google.com>
Gerrit-Reviewer: Mario Scheithauer <mario.scheithauer(a)siemens.com>
Gerrit-Reviewer: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: merged