Frank Chu has removed Philipp Deppenwiese from this change. ( https://review.coreboot.org/c/coreboot/+/34743 )
Change subject: mb/google/hatch/variants/helios: Update DPTF parameters and TDP PL1/PL2
......................................................................
Removed reviewer Philipp Deppenwiese.
--
To view, visit https://review.coreboot.org/c/coreboot/+/34743
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic7a96c33ce710c32b57e2ad8066830ff83398c57
Gerrit-Change-Number: 34743
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Chu <frank_chu(a)pegatron.corp-partner.google.com>
Gerrit-Reviewer: Frank Chu <frank_chu(a)pegatron.corp-partner.google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Philip Chen <philipchen(a)google.com>
Gerrit-CC: Kane Chen <kane_chen(a)pegatron.corp-partner.google.com>
Gerrit-CC: Ken Lu <ken_lu(a)pegatron.corp-partner.google.com>
Gerrit-MessageType: deleteReviewer
David Guckian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34698 )
Change subject: intel/fsp_rangeley: Rename raminit.c to memmap.c
......................................................................
Patch Set 2: Code-Review+1
--
To view, visit https://review.coreboot.org/c/coreboot/+/34698
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I704d7cb637e4e12039ade99f57e10af794c8be97
Gerrit-Change-Number: 34698
Gerrit-PatchSet: 2
Gerrit-Owner: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: David Guckian
Gerrit-Reviewer: David Guckian <david.guckian(a)intel.com>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Tue, 06 Aug 2019 08:18:41 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/34736 )
Change subject: soc/intel/common: Select USE_LEGACY_8254_TIMER
......................................................................
soc/intel/common: Select USE_LEGACY_8254_TIMER
Always select USE_LEGACY_8254_TIMER if we know we have to.
Default to y as there might be legacy software not under our control
that needs the 8254 timer.
Add a comment to evaluate if this should be changed to 'default n' in
10 years from now.
The following software needs 8254 timer:
* SeaBios
* Option ROMs
* GRUB <2.03
* Linux <5.3
References:
https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git/commit/?id=c8c4…http://git.savannah.gnu.org/cgit/grub.git/commit/?id=446794de8da4329ea532cb…
Fixes boot failure on SeaBios and GRUB2 bootloader on X11SSH-TF using
SeaBIOS as payload.
Change-Id: I3eae8249e87a54c877e320ebce6595030c48fe2f
Signed-off-by: Patrick Rudolph <patrick.rudolph(a)9elements.com>
---
M payloads/external/SeaBIOS/Kconfig.name
M src/device/Kconfig
M src/soc/intel/common/block/timer/Kconfig
3 files changed, 12 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/36/34736/1
diff --git a/payloads/external/SeaBIOS/Kconfig.name b/payloads/external/SeaBIOS/Kconfig.name
index bb1f30c..8f92997 100644
--- a/payloads/external/SeaBIOS/Kconfig.name
+++ b/payloads/external/SeaBIOS/Kconfig.name
@@ -1,6 +1,7 @@
config PAYLOAD_SEABIOS
bool "SeaBIOS"
depends on ARCH_X86
+ select USE_LEGACY_8254_TIMER if SOC_INTEL_COMMON_BLOCK
help
Select this option if you want to build a coreboot image
with a SeaBIOS payload. If you don't know what this is
diff --git a/src/device/Kconfig b/src/device/Kconfig
index e605bc2..ebf6bfb 100644
--- a/src/device/Kconfig
+++ b/src/device/Kconfig
@@ -86,6 +86,7 @@
bool "Run VGA Option ROMs"
depends on PCI && (ARCH_X86 || ARCH_PPC64) && !MAINBOARD_FORCE_NATIVE_VGA_INIT
select HAVE_VGA_TEXT_FRAMEBUFFER
+ select USE_LEGACY_8254_TIMER if SOC_INTEL_COMMON_BLOCK
help
Execute VGA Option ROMs in coreboot if found. This can be used
to enable PCI/AGP/PCI-E video cards when not using a SeaBIOS
diff --git a/src/soc/intel/common/block/timer/Kconfig b/src/soc/intel/common/block/timer/Kconfig
index a214ef0..50c7677 100644
--- a/src/soc/intel/common/block/timer/Kconfig
+++ b/src/soc/intel/common/block/timer/Kconfig
@@ -5,10 +5,16 @@
config USE_LEGACY_8254_TIMER
bool "Use Legacy 8254 Timer"
- default y if PAYLOAD_SEABIOS || VGA_ROM_RUN
- default n
+ default y
help
This sets the FSP UPD to enable Legacy 8254 clock gating. As per
the FSP Integration guide Legacy 8254 timer clock gating UPD needs
- to be disabled in order to boot SeaBIOS or run OpRom,
- but should otherwise be enabled.
+ to be disabled in order to boot legacy software.
+ TODO after 2029: Evaluate if this should be changed to 'default n'.
+
+if !USE_LEGACY_8254_TIMER
+
+comment "Warning: The following software won't run due to disabled 8254 timer"
+comment " GRUB <2.03, Linux <5.3, SeaBios, Option ROMs"
+
+endif
--
To view, visit https://review.coreboot.org/c/coreboot/+/34736
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3eae8249e87a54c877e320ebce6595030c48fe2f
Gerrit-Change-Number: 34736
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-MessageType: newchange
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34699 )
Change subject: northbridge/intel: Rename ram_calc.c to memmap.c
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/34699
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie69583f28f384eb49517203e1c3867f27e6272de
Gerrit-Change-Number: 34699
Gerrit-PatchSet: 2
Gerrit-Owner: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Damien Zammit
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Tue, 06 Aug 2019 06:06:38 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34698 )
Change subject: intel/fsp_rangeley: Rename raminit.c to memmap.c
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/34698
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I704d7cb637e4e12039ade99f57e10af794c8be97
Gerrit-Change-Number: 34698
Gerrit-PatchSet: 2
Gerrit-Owner: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: David Guckian <david.guckian(a)intel.com>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Tue, 06 Aug 2019 06:06:24 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Martin Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/11109 )
Change subject: Hack up per-board user selectable 64bit scheme
......................................................................
Abandoned
This patch was not touched in over 18 months. Anyone who wants to take over work on this patch, please feel free to restore it and do what you'd like. If you create a new patch based on this work, please credit the original author.
--
To view, visit https://review.coreboot.org/c/coreboot/+/11109
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I91b1316896e69b081c861d99ff0c04be3c1d90c1
Gerrit-Change-Number: 11109
Gerrit-PatchSet: 2
Gerrit-Owner: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: abandon
Martin Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/13554 )
Change subject: soc/intel/broadwell: Fix UART compilation
......................................................................
Abandoned
This patch was not touched in over 18 months. Anyone who wants to take over work on this patch, please feel free to restore it and do what you'd like. If you create a new patch based on this work, please credit the original author.
--
To view, visit https://review.coreboot.org/c/coreboot/+/13554
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I7fe356041c6ac69a30e0e87d9a8649abef6996b7
Gerrit-Change-Number: 13554
Gerrit-PatchSet: 1
Gerrit-Owner: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Duncan Laurie <dlaurie(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: abandon
Martin Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/13553 )
Change subject: drivers/uart: Export uart8250_mem_init from uart8250mem.c
......................................................................
Abandoned
This patch was not touched in over 18 months. Anyone who wants to take over work on this patch, please feel free to restore it and do what you'd like. If you create a new patch based on this work, please credit the original author.
--
To view, visit https://review.coreboot.org/c/coreboot/+/13553
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ibc5b5c43475bf59d0d54cba2afd9362e75fa360c
Gerrit-Change-Number: 13553
Gerrit-PatchSet: 1
Gerrit-Owner: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Duncan Laurie <dlaurie(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: abandon