cpu/intel/: Fix regression with smmrelocate
Fix regression with commit d53fd70
intel/smm/gen1: Use smm_subregion().
The bitmask on SMRR register parameter was inverted for
selected models.
Change-Id: Ia572ca3bdd4da371985691b5d249f998382fbe48
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/35780
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-by: Bill XIE <persmule@hardenedlinux.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
This commit is contained in:
parent
30d3c9ed48
commit
5a2bbcaa93
|
@ -121,7 +121,7 @@ static void fill_in_relocation_params(struct smm_relocation_params *params)
|
||||||
|
|
||||||
/* On model_6fx and model_1067x bits [0:11] on smrr_base are reserved */
|
/* On model_6fx and model_1067x bits [0:11] on smrr_base are reserved */
|
||||||
if (cpu_has_alternative_smrr())
|
if (cpu_has_alternative_smrr())
|
||||||
params->smrr_base.lo &= ~rmask;
|
params->smrr_base.lo &= rmask;
|
||||||
|
|
||||||
smm_subregion(SMM_SUBREGION_CHIPSET, ¶ms->ied_base, ¶ms->ied_size);
|
smm_subregion(SMM_SUBREGION_CHIPSET, ¶ms->ied_base, ¶ms->ied_size);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue