Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37394 )
Change subject: arch/x86/Kconfig: Move pagetables down by 4K
......................................................................
arch/x86/Kconfig: Move pagetables down by 4K
In case of 64K bootblock the pagetables doesn't fit, as the CBFS header
also needs a few bytes.
Fixes build error on platforms that use 64KiB bootblock.
Tested on Lenovo T410 with additional x86_64 patches.
Change-Id: I854c5f575e2376827a366cca8d25682c4d90bc8f
Signed-off-by: Patrick Rudolph <patrick.rudolph(a)9elements.com>
---
M src/arch/x86/Kconfig
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/94/37394/1
diff --git a/src/arch/x86/Kconfig b/src/arch/x86/Kconfig
index 0e6f486..1e27970 100644
--- a/src/arch/x86/Kconfig
+++ b/src/arch/x86/Kconfig
@@ -68,7 +68,7 @@
config ARCH_X86_64_PGTBL_LOC
hex "x86_64 page table location in CBFS"
depends on ARCH_BOOTBLOCK_X86_64
- default 0xfffea000
+ default 0xfffe9000
help
The position where to place pagetables. Needs to be known at
compile time. Must not overlap other files in CBFS.
--
To view, visit https://review.coreboot.org/c/coreboot/+/37394
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I854c5f575e2376827a366cca8d25682c4d90bc8f
Gerrit-Change-Number: 37394
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-MessageType: newchange
Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45696 )
Change subject: mb/purism/librem_whl: Drop 3Gbps SATA limit; enable and set SATA tuning params
......................................................................
mb/purism/librem_whl: Drop 3Gbps SATA limit; enable and set SATA tuning params
Some Librem Minis exhibit issues with 6Gbps SATA operation on certain
SSDs, setting the Receiver Equalization Boost Magnitude adjustment
resolves this, so limiting SATA speeds to 3Gbps is no longer needed.
Test: build/boot Librem Mini with Crucial SATA SSD, observe no issues
booting, no ATA-related errors in dmesg on PureOS 10 / kernel 5.8.7
Change-Id: I8b3cbcff7f181bcab35d71e859033578c822bb20
Signed-off-by: Matt DeVillier <matt.devillier(a)puri.sm>
---
M src/mainboard/purism/librem_whl/ramstage.c
M src/mainboard/purism/librem_whl/romstage.c
2 files changed, 8 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/96/45696/1
diff --git a/src/mainboard/purism/librem_whl/ramstage.c b/src/mainboard/purism/librem_whl/ramstage.c
index 07ede66..56ed1b7 100644
--- a/src/mainboard/purism/librem_whl/ramstage.c
+++ b/src/mainboard/purism/librem_whl/ramstage.c
@@ -10,7 +10,4 @@
size_t num_gpios;
const struct pad_config *gpio_table = variant_gpio_table(&num_gpios);
cnl_configure_pads(gpio_table, num_gpios);
-
- /* Limit SATA speed to 3Gbps until correct HSIO PHY settings determined */
- params->SataSpeedLimit = 2;
}
diff --git a/src/mainboard/purism/librem_whl/romstage.c b/src/mainboard/purism/librem_whl/romstage.c
index 9f8d600..c45f138 100644
--- a/src/mainboard/purism/librem_whl/romstage.c
+++ b/src/mainboard/purism/librem_whl/romstage.c
@@ -48,5 +48,12 @@
void mainboard_memory_init_params(FSPM_UPD *memupd)
{
- cannonlake_memcfg_init(&memupd->FspmConfig, &memcfg);
+ FSP_M_CONFIG *mem_cfg = &memupd->FspmConfig;
+ cannonlake_memcfg_init(mem_cfg, &memcfg);
+
+ /* Enable and set SATA HSIO adjustments for ports 0 and 2 */
+ mem_cfg->PchSataHsioRxGen3EqBoostMagEnable[0] = 1;
+ mem_cfg->PchSataHsioRxGen3EqBoostMagEnable[2] = 1;
+ mem_cfg->PchSataHsioRxGen3EqBoostMag[0] = 1;
+ mem_cfg->PchSataHsioRxGen3EqBoostMag[2] = 1;
}
--
To view, visit https://review.coreboot.org/c/coreboot/+/45696
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I8b3cbcff7f181bcab35d71e859033578c822bb20
Gerrit-Change-Number: 45696
Gerrit-PatchSet: 1
Gerrit-Owner: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-MessageType: newchange
Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45695 )
Change subject: mb/purism/librem_skl: Enable and set SATA tuning params
......................................................................
mb/purism/librem_skl: Enable and set SATA tuning params
Some Librems have issues with 6Gbps SATA operation on certain
SSDs, setting the Receiver Equalization Boost Magnitude adjustment
resolves this.
Test: build/boot Librem 15v3 with Crucial SATA SSD, observe no issues
booting, no ATA-related errors in dmesg on PureOS 10 / kernel 5.8.7
Change-Id: I078deeff7fc54694393b5b16c41c5d622b332781
Signed-off-by: Matt DeVillier <matt.devillier(a)puri.sm>
---
M src/mainboard/purism/librem_skl/romstage.c
1 file changed, 6 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/95/45695/1
diff --git a/src/mainboard/purism/librem_skl/romstage.c b/src/mainboard/purism/librem_skl/romstage.c
index de493b0..7be8325 100644
--- a/src/mainboard/purism/librem_skl/romstage.c
+++ b/src/mainboard/purism/librem_skl/romstage.c
@@ -64,4 +64,10 @@
mem_cfg->DqPinsInterleaved = TRUE;
mem_cfg->MemorySpdDataLen = blk.len;
mem_cfg->MemorySpdPtr00 = (uintptr_t) blk.spd_array[0];
+
+ /* Enable and set SATA HSIO adjustments for ports 0 and 2 */
+ mem_cfg->PchSataHsioRxGen3EqBoostMagEnable[0] = 1;
+ mem_cfg->PchSataHsioRxGen3EqBoostMagEnable[2] = 1;
+ mem_cfg->PchSataHsioRxGen3EqBoostMag[0] = 1;
+ mem_cfg->PchSataHsioRxGen3EqBoostMag[2] = 1;
}
--
To view, visit https://review.coreboot.org/c/coreboot/+/45695
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I078deeff7fc54694393b5b16c41c5d622b332781
Gerrit-Change-Number: 45695
Gerrit-PatchSet: 1
Gerrit-Owner: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-MessageType: newchange
Christian Walter has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45659 )
Change subject: Documentation/mainboard: Fix OCP Delta Lake Link
......................................................................
Documentation/mainboard: Fix OCP Delta Lake Link
Change-Id: I379d6a7b72a0398c34ea8eeda09ccd663fc372ce
Signed-off-by: Christian Walter <christian.walter(a)9elements.com>
---
M Documentation/mainboard/index.md
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/59/45659/1
diff --git a/Documentation/mainboard/index.md b/Documentation/mainboard/index.md
index 0eefee8..998de61 100644
--- a/Documentation/mainboard/index.md
+++ b/Documentation/mainboard/index.md
@@ -118,6 +118,7 @@
## OCP
+- [Delta Lake](ocp/deltalake.md)
- [Tioga Pass](ocp/tiogapass.md)
## Open Cellular
--
To view, visit https://review.coreboot.org/c/coreboot/+/45659
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I379d6a7b72a0398c34ea8eeda09ccd663fc372ce
Gerrit-Change-Number: 45659
Gerrit-PatchSet: 1
Gerrit-Owner: Christian Walter <christian.walter(a)9elements.com>
Gerrit-MessageType: newchange