Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31249
Change subject: soc/intel/denverton_ns: Don't use CONFIG_CBFS_SIZE
......................................................................
soc/intel/denverton_ns: Don't use CONFIG_CBFS_SIZE
CONFIG_CBFS_SIZE is only meaningful to generate the default fmap
layout and ought not to be used in the code directly.
Change-Id: Iae72a9fb02d62d7548d34689f5eb371f34cd3d81
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/soc/intel/denverton_ns/bootblock/bootblock.c
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/49/31249/1
diff --git a/src/soc/intel/denverton_ns/bootblock/bootblock.c b/src/soc/intel/denverton_ns/bootblock/bootblock.c
index 110d67d..58144bd 100644
--- a/src/soc/intel/denverton_ns/bootblock/bootblock.c
+++ b/src/soc/intel/denverton_ns/bootblock/bootblock.c
@@ -37,8 +37,8 @@
.MicrocodeRegionLength =
(UINT32)CONFIG_CPU_MICROCODE_CBFS_LEN,
.CodeRegionBase =
- (UINT32)(0x100000000ULL - CONFIG_CBFS_SIZE),
- .CodeRegionLength = (UINT32)CONFIG_CBFS_SIZE,
+ (UINT32)(0x100000000ULL - CONFIG_ROM_SIZE),
+ .CodeRegionLength = (UINT32)CONFIG_ROM_SIZE,
.Reserved1 = {0},
},
.FsptConfig = {
--
To view, visit https://review.coreboot.org/c/coreboot/+/31249
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iae72a9fb02d62d7548d34689f5eb371f34cd3d81
Gerrit-Change-Number: 31249
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-MessageType: newchange
Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/30791
Change subject: src/drivers/intel/fsp1_0: Move PLATFORM_USES_FSP1_0
......................................................................
src/drivers/intel/fsp1_0: Move PLATFORM_USES_FSP1_0
This is a better location.
Change-Id: Ic1c86c26a66c33760484bb6a86e9763c148a7c96
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/cpu/x86/Kconfig
M src/drivers/intel/fsp1_0/Kconfig
2 files changed, 9 insertions(+), 7 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/91/30791/1
diff --git a/src/cpu/x86/Kconfig b/src/cpu/x86/Kconfig
index 232fe63..608afd7 100644
--- a/src/cpu/x86/Kconfig
+++ b/src/cpu/x86/Kconfig
@@ -151,13 +151,6 @@
This option informs the MTRR code to use the RdMem and WrMem fields
in the fixed MTRR MSRs.
-config PLATFORM_USES_FSP1_0
- bool
- default n
- help
- Selected for Intel processors/platform combinations that use the
- Intel Firmware Support Package (FSP) 1.0 for initialization.
-
config MIRROR_PAYLOAD_TO_RAM_BEFORE_LOADING
def_bool n
help
diff --git a/src/drivers/intel/fsp1_0/Kconfig b/src/drivers/intel/fsp1_0/Kconfig
index c7f6c18..4d1c4df 100644
--- a/src/drivers/intel/fsp1_0/Kconfig
+++ b/src/drivers/intel/fsp1_0/Kconfig
@@ -13,6 +13,15 @@
## GNU General Public License for more details.
##
+config PLATFORM_USES_FSP1_0
+ bool
+ default n
+ select CACHE_MRC_SETTINGS
+ help
+ Selected for Intel processors/platform combinations that use the
+ Intel Firmware Support Package (FSP) 1.0 for initialization.
+
+
if PLATFORM_USES_FSP1_0
comment "Intel FSP"
--
To view, visit https://review.coreboot.org/c/coreboot/+/30791
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic1c86c26a66c33760484bb6a86e9763c148a7c96
Gerrit-Change-Number: 30791
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-MessageType: newchange
Hello Patrick Rudolph,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/31114
to review the following change.
Change subject: mb/lenovo/z61t/acpi_tables: Add critical and passive threshold
......................................................................
mb/lenovo/z61t/acpi_tables: Add critical and passive threshold
Add critical and passive threshold to be advertised in thermal zone 0.
This commit follows up on commit afa627db with Change-Id
Ic75a80994b27ac19651ed52b7fc3c00c65cd9c01.
Change-Id: I7d645818d2b2f6b7c16d2d813bf982a55774561d
Signed-off-by: Patrick Rudolph <siro(a)das-labor.org>
Signed-off-by: Peter Lemenkov <lemenkov(a)gmail.com>
---
M src/mainboard/lenovo/z61t/acpi_tables.c
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/14/31114/1
diff --git a/src/mainboard/lenovo/z61t/acpi_tables.c b/src/mainboard/lenovo/z61t/acpi_tables.c
index 3e9a810..183b7fe 100644
--- a/src/mainboard/lenovo/z61t/acpi_tables.c
+++ b/src/mainboard/lenovo/z61t/acpi_tables.c
@@ -23,4 +23,7 @@
gnvs->cmap = 0x01;
gnvs->cmbp = 0x01;
+ /* Set thermal levels */
+ gnvs->tcrt = 100;
+ gnvs->tpsv = 90;
}
--
To view, visit https://review.coreboot.org/c/coreboot/+/31114
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I7d645818d2b2f6b7c16d2d813bf982a55774561d
Gerrit-Change-Number: 31114
Gerrit-PatchSet: 1
Gerrit-Owner: Peter Lemenkov <lemenkov(a)gmail.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-MessageType: newchange
Hello Iru Cai,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/30950
to review the following change.
Change subject: Documentation: Add HP EliteBook 8760w
......................................................................
Documentation: Add HP EliteBook 8760w
Also add the HP EliteBook document from wiki.
Change-Id: I189db9c279705af53d82af66d0c2e8afb6f84d73
Signed-off-by: Iru Cai <mytbk920423(a)gmail.com>
---
A Documentation/mainboard/hp/8760w.md
A Documentation/mainboard/hp/elitebook_series.md
M Documentation/mainboard/index.md
3 files changed, 149 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/30950/1
diff --git a/Documentation/mainboard/hp/8760w.md b/Documentation/mainboard/hp/8760w.md
new file mode 100644
index 0000000..ba4da7d
--- /dev/null
+++ b/Documentation/mainboard/hp/8760w.md
@@ -0,0 +1,33 @@
+# HP EliteBook 8760w
+
+This page describes how to run coreboot on the [HP EliteBook 8760w].
+
+## Required proprietary blobs
+
+- Intel Firmware Descriptor, ME and GbE firmware
+- EC: please read [EliteBook Series](elitebook_series)
+
+## Flashing instructions
+
+HP EliteBook 8760w has an 8MB SOIC-8 flash chip on the bottom of the
+mainboard. You just need to remove the service cover, and use an SOIC-8
+clip to read and flash the chip.
+
+## Untested
+
+- dock: serial port, parallel port, ...
+- TPM
+- S3 suspend/resume
+- Gigabit Ethernet
+- Using `me_cleaner`
+
+## Working
+
+- SeaBIOS payload
+- EHCI debug: the port is at the right next to the charging port
+- USB
+- WLAN
+- WWAN
+- SATA
+
+[HP EliteBook 8760w]: https://support.hp.com/us-en/product/hp-elitebook-8760w-mobile-workstation/…
diff --git a/Documentation/mainboard/hp/elitebook_series.md b/Documentation/mainboard/hp/elitebook_series.md
new file mode 100644
index 0000000..6668928
--- /dev/null
+++ b/Documentation/mainboard/hp/elitebook_series.md
@@ -0,0 +1,111 @@
+# HP EliteBook series
+
+This document is about HP EliteBook series laptops up to Ivy Bridge era
+which use SMSC KBC1126 as embedded controller.
+
+## EC
+
+SMSC KBC1098/KBC1126 has been used in HP EliteBooks for many generations.
+They use similar EC firmware that will load other code and data from the
+SPI flash chip, so we need to put some firmware blobs to the coreboot image.
+
+The following document takes EliteBook 2760p as an example.
+
+First, you need to extract the blobs needed by EC firmware using util/kbc1126.
+You can extract them from your backup firmware image, or firmware update
+provided by HP with [unar] as follows:
+
+```bash
+wget https://ftp.hp.com/pub/softpaq/sp79501-80000/sp79710.exe
+unar sp79710.exe
+${COREBOOT_DIR}/util/kbc1126/kbc1126_ec_dump sp79710/Rompaq/68SOU.BIN
+mv 68SOU.BIN.fw1 ${COREBOOT_DIR}/2760p-fw1.bin
+mv 68SOU.BIN.fw2 ${COREBOOT_DIR}/2760p-fw2.bin
+```
+
+When you config coreboot, select:
+
+```text
+Chipset --->
+ [*] Add firmware images for KBC1126 EC
+ (2760p-fw1.bin) KBC1126 firmware #1 path and filename
+ (2760p-fw2.bin) KBC1126 filename #2 path and filename
+```
+
+## Super I/O
+
+EliteBook 8000 series laptops have SMSC LPC47n217 Super I/O to provide
+a serial port and a parallel port, you can debug the laptop via this
+serial port.
+
+## porting
+
+To port coreboot to an HP EliteBook laptop, you need to do the following:
+
+- select Kconfig option `EC_HP_KBC1126`
+- select Kconfig option `SUPERIO_SMSC_LPC47N217` if there is LPC47n217 Super I/O
+- initialize EC and Super I/O in romstage
+- add EC and Super I/O support to devicetree.cb
+
+To get the related values for EC in devicetree.cb, you need to extract the EFI
+module EcThermalInit from the vendor UEFI firmware with [UEFITool]. Usually,
+`ec_data_port`, `ec_cmd_port` and `ec_ctrl_reg` has the following values:
+
+- For xx60 series: 0x60, 0x64, 0xca
+- For xx70 series: 0x62, 0x66, 0x81
+
+You can use [radare2] and the following [r2pipe] Python script to find
+these values from the EcThermalInit EFI module:
+
+```python
+#!/usr/bin/env python
+
+# install radare2 and use `pip3 install --user r2pipe` to install r2pipe
+
+import r2pipe
+import sys
+
+if len(sys.argv) < 2:
+ fn = "ecthermalinit.efi"
+else:
+ fn = sys.argv[1]
+
+r2 = r2pipe.open(fn)
+r2.cmd("aa")
+entryf = r2.cmdj("pdfj")
+
+for insn in entryf["ops"]:
+ if "lea r8" in insn["opcode"]:
+ _callback = insn["ptr"]
+ break
+
+r2.cmd("af @ {}".format(_callback))
+callbackf_insns = r2.cmdj("pdfj @ {}".format(_callback))["ops"]
+
+def find_port(addr):
+ ops = r2.cmdj("pdfj @ {}".format(addr))["ops"]
+ for insn in ops:
+ if "lea r8d" in insn["opcode"]:
+ return insn["ptr"]
+
+ctrl_reg_found = False
+
+for i in range(0, len(callbackf_insns)):
+ if not ctrl_reg_found and "mov cl" in callbackf_insns[i]["opcode"]:
+ ctrl_reg_found = True
+ ctrl_reg = callbackf_insns[i]["ptr"]
+ print("ec_ctrl_reg = 0x%02x" % ctrl_reg)
+ cmd_port = find_port(callbackf_insns[i+1]["jump"])
+ data_port = find_port(callbackf_insns[i+3]["jump"])
+ print("ec_cmd_port = 0x%02x\nec_data_port = 0x%02x" % (cmd_port, data_port))
+
+ if "mov bl" in callbackf_insns[i]["opcode"]:
+ ctrl_value = callbackf_insns[i]["ptr"]
+ print("ec_fan_ctrl_value = 0x%02x" % ctrl_value)
+```
+
+
+[unar]: https://theunarchiver.com/command-line
+[UEFITool]: https://github.com/LongSoft/UEFITool
+[radare2]: https://radare.org/
+[r2pipe]: https://github.com/radare/radare2-r2pipe
diff --git a/Documentation/mainboard/index.md b/Documentation/mainboard/index.md
index 4e2b742..606901f 100644
--- a/Documentation/mainboard/index.md
+++ b/Documentation/mainboard/index.md
@@ -46,6 +46,11 @@
- [Compaq 8200 Elite SFF](hp/compaq_8200_sff.md)
+### EliteBook series
+
+- [EliteBook common](hp/elitebook_series.md)
+- [EliteBook 8760w](hp/8760w.md)
+
## Lenovo
- [T4xx common](lenovo/t4xx_series.md)
--
To view, visit https://review.coreboot.org/c/coreboot/+/30950
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I189db9c279705af53d82af66d0c2e8afb6f84d73
Gerrit-Change-Number: 30950
Gerrit-PatchSet: 1
Gerrit-Owner: Iru Cai (vimacs) <mytbk920423(a)gmail.com>
Gerrit-Reviewer: Iru Cai <mytbk920423(a)gmail.com>
Gerrit-MessageType: newchange