Hello Angel Pons, Arthur Heymans, Matt DeVillier,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/libgfxinit/+/36443
to review the following change.
Change subject: gma config: Add Comet Lake PCI IDs
......................................................................
gma config: Add Comet Lake PCI IDs
Comet Lake is still treated like Kaby Lake :)
Change-Id: I543f1a50c204d963de80111aa0afd724e4093c4a
Signed-off-by: Nico Huber <nico.h(a)gmx.de>
---
M common/hw-gfx-gma-config.ads.template
1 file changed, 15 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/libgfxinit refs/changes/43/36443/1
diff --git a/common/hw-gfx-gma-config.ads.template b/common/hw-gfx-gma-config.ads.template
index 0e49943..7347341 100644
--- a/common/hw-gfx-gma-config.ads.template
+++ b/common/hw-gfx-gma-config.ads.template
@@ -428,6 +428,17 @@
function Is_Coffee_Lake (Device_Id : Word16) return Boolean is
((Device_Id and 16#fff0#) = 16#3e90#);
+ function Is_Comet_Lake_U (Device_Id : Word16) return Boolean is
+ ((Device_Id and 16#ff9f#) = 16#9b01# or
+ (Device_Id and 16#ff9f#) = 16#9b8a# or
+ (Device_Id and 16#ff9f#) = 16#9b8c#);
+ function Is_Comet_Lake (Device_Id : Word16) return Boolean is
+ ((Device_Id and 16#ff8f#) = 16#9b82# or
+ (Device_Id and 16#ff8f#) = 16#9b84# or
+ (Device_Id and 16#ff8f#) = 16#9b85# or
+ (Device_Id and 16#ff8f#) = 16#9b86# or
+ (Device_Id and 16#ff8f#) = 16#9b88#);
+
function Is_GPU (Device_Id : Word16; CPU : CPU_Type; CPU_Var : CPU_Variant)
return Boolean is
(case CPU is
@@ -452,10 +463,12 @@
when Kabylake => (case CPU_Var is
when Normal =>
Is_Kaby_Lake (Device_Id) or
- Is_Coffee_Lake (Device_Id),
+ Is_Coffee_Lake (Device_Id) or
+ Is_Comet_Lake (Device_Id),
when ULT =>
Is_Kaby_Lake_U (Device_Id) or
- Is_Coffee_Lake_U (Device_Id),
+ Is_Coffee_Lake_U (Device_Id) or
+ Is_Comet_Lake_U (Device_Id),
when ULX =>
Is_Kaby_Lake_Y (Device_Id) or
Is_Kaby_Lake_Y_AML (Device_Id) or
--
To view, visit https://review.coreboot.org/c/libgfxinit/+/36443
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: libgfxinit
Gerrit-Branch: master
Gerrit-Change-Id: I543f1a50c204d963de80111aa0afd724e4093c4a
Gerrit-Change-Number: 36443
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-MessageType: newchange
Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37288 )
Change subject: cpu/x86/smm: Add smm_size to relocatable smmstub
......................................................................
cpu/x86/smm: Add smm_size to relocatable smmstub
To mitigate against sinkhole in software which is required on
pre-sandybridge hardware, the smm entry point needs to check if the
LAPIC base is between smbase and smbase + smmsize. The size needs to
be available early so add them to the relocatable module parameters.
When the smmstub is used to relocate SMM the default SMM size 0x10000
is provided. On the permanent handler the size provided by
get_smm_info() is used.
Change-Id: I0df6e51bcba284350f1c849ef3d012860757544b
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/cpu/x86/smm/smm_module_loader.c
M src/cpu/x86/smm/smm_stub.S
M src/include/cpu/x86/smm.h
3 files changed, 10 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/88/37288/1
diff --git a/src/cpu/x86/smm/smm_module_loader.c b/src/cpu/x86/smm/smm_module_loader.c
index c6c6b38..4431a44 100644
--- a/src/cpu/x86/smm/smm_module_loader.c
+++ b/src/cpu/x86/smm/smm_module_loader.c
@@ -173,8 +173,9 @@
* concurrent areas requested. The save state always lives at the top of SMRAM
* space, and the entry point is at offset 0x8000.
*/
-static int smm_module_setup_stub(void *smbase, struct smm_loader_params *params,
- void *fxsave_area)
+static int smm_module_setup_stub(void *smbase, size_t smm_size,
+ struct smm_loader_params *params,
+ void *fxsave_area)
{
size_t total_save_state_size;
size_t smm_stub_size;
@@ -266,6 +267,7 @@
stub_params->fxsave_area = (uintptr_t)fxsave_area;
stub_params->fxsave_area_size = FXSAVE_SIZE;
stub_params->runtime.smbase = (uintptr_t)smbase;
+ stub_params->runtime.smm_size = smm_size;
stub_params->runtime.save_state_size = params->per_cpu_save_state_size;
/* Initialize the APIC id to CPU number table to be 1:1 */
@@ -305,7 +307,8 @@
if (params->num_concurrent_stacks == 0)
params->num_concurrent_stacks = CONFIG_MAX_CPUS;
- return smm_module_setup_stub(smram, params, fxsave_area_relocation);
+ return smm_module_setup_stub(smram, SMM_DEFAULT_SIZE,
+ params, fxsave_area_relocation);
}
/* The SMM module is placed within the provided region in the following
@@ -391,5 +394,5 @@
params->handler = rmodule_entry(&smm_mod);
params->handler_arg = rmodule_parameters(&smm_mod);
- return smm_module_setup_stub(smram, params, fxsave_area);
+ return smm_module_setup_stub(smram, size, params, fxsave_area);
}
diff --git a/src/cpu/x86/smm/smm_stub.S b/src/cpu/x86/smm/smm_stub.S
index f0e55f9..304ea4b 100644
--- a/src/cpu/x86/smm/smm_stub.S
+++ b/src/cpu/x86/smm/smm_stub.S
@@ -42,6 +42,8 @@
smm_runtime:
smbase:
.long 0
+smm_size:
+.long 0
save_state_size:
.long 0
/* apic_to_cpu_num is a table mapping the default APIC id to CPU num. If the
diff --git a/src/include/cpu/x86/smm.h b/src/include/cpu/x86/smm.h
index cf107b1..2e3c639 100644
--- a/src/include/cpu/x86/smm.h
+++ b/src/include/cpu/x86/smm.h
@@ -63,6 +63,7 @@
struct smm_runtime {
u32 smbase;
+ u32 smm_size;
u32 save_state_size;
/* The apic_id_to_cpu provides a mapping from APIC id to CPU number.
* The CPU number is indicated by the index into the array by matching
--
To view, visit https://review.coreboot.org/c/coreboot/+/37288
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0df6e51bcba284350f1c849ef3d012860757544b
Gerrit-Change-Number: 37288
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-MessageType: newchange
Peter Lemenkov has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/38482 )
Change subject: mb/lenovo/x230: Add Lenovo X230 Tablet to the list of X230 variants
......................................................................
mb/lenovo/x230: Add Lenovo X230 Tablet to the list of X230 variants
Lenovo ThinkPad X230t Convertible Laptop works well with X230 default
image (see CB:34361).
Change-Id: Ib0a73fd551f0d26c789d3fd13541b2d1571742cb
Signed-off-by: Peter Lemenkov <lemenkov(a)gmail.com>
---
M src/mainboard/lenovo/x230/Kconfig.name
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/82/38482/1
diff --git a/src/mainboard/lenovo/x230/Kconfig.name b/src/mainboard/lenovo/x230/Kconfig.name
index d207653..4f02d55 100644
--- a/src/mainboard/lenovo/x230/Kconfig.name
+++ b/src/mainboard/lenovo/x230/Kconfig.name
@@ -1,2 +1,2 @@
config BOARD_LENOVO_X230
- bool "ThinkPad X230"
+ bool "ThinkPad X230 / X230t"
--
To view, visit https://review.coreboot.org/c/coreboot/+/38482
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib0a73fd551f0d26c789d3fd13541b2d1571742cb
Gerrit-Change-Number: 38482
Gerrit-PatchSet: 1
Gerrit-Owner: Peter Lemenkov <lemenkov(a)gmail.com>
Gerrit-MessageType: newchange