soc/intel/{icl,cnl,tgl}: Always add PM1_TMR block to FADT
Provide the PM1_TMR information in the FADT even if PmTimerDisabled is
set because PM timer emulation is enabled via MSR 121h so the timer will
still work and can be used by things like Tianocore and Windows.
Porting from 662b6cb3ed
(soc/intel/skylake: Always add PM1_TMR block to FADT).
Change-Id: Ie3d592623f3a84051477ffe83a0cf0daf30dd36f
Signed-off-by: Meera Ravindranath <meera.ravindranath@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/37662
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
This commit is contained in:
parent
408d1dac9e
commit
48c7870e52
|
@ -179,7 +179,6 @@ void soc_fill_fadt(acpi_fadt_t *fadt)
|
||||||
const struct soc_intel_cannonlake_config *config;
|
const struct soc_intel_cannonlake_config *config;
|
||||||
config = config_of_soc();
|
config = config_of_soc();
|
||||||
|
|
||||||
if (!config->PmTimerDisabled) {
|
|
||||||
fadt->pm_tmr_blk = pmbase + PM1_TMR;
|
fadt->pm_tmr_blk = pmbase + PM1_TMR;
|
||||||
fadt->pm_tmr_len = 4;
|
fadt->pm_tmr_len = 4;
|
||||||
fadt->x_pm_tmr_blk.space_id = 1;
|
fadt->x_pm_tmr_blk.space_id = 1;
|
||||||
|
@ -188,7 +187,6 @@ void soc_fill_fadt(acpi_fadt_t *fadt)
|
||||||
fadt->x_pm_tmr_blk.access_size = 0;
|
fadt->x_pm_tmr_blk.access_size = 0;
|
||||||
fadt->x_pm_tmr_blk.addrl = pmbase + PM1_TMR;
|
fadt->x_pm_tmr_blk.addrl = pmbase + PM1_TMR;
|
||||||
fadt->x_pm_tmr_blk.addrh = 0x0;
|
fadt->x_pm_tmr_blk.addrh = 0x0;
|
||||||
}
|
|
||||||
|
|
||||||
if (config->s0ix_enable)
|
if (config->s0ix_enable)
|
||||||
fadt->flags |= ACPI_FADT_LOW_PWR_IDLE_S0;
|
fadt->flags |= ACPI_FADT_LOW_PWR_IDLE_S0;
|
||||||
|
|
|
@ -172,7 +172,6 @@ void soc_fill_fadt(acpi_fadt_t *fadt)
|
||||||
|
|
||||||
config_t *config = config_of_soc();
|
config_t *config = config_of_soc();
|
||||||
|
|
||||||
if (!config->PmTimerDisabled) {
|
|
||||||
fadt->pm_tmr_blk = pmbase + PM1_TMR;
|
fadt->pm_tmr_blk = pmbase + PM1_TMR;
|
||||||
fadt->pm_tmr_len = 4;
|
fadt->pm_tmr_len = 4;
|
||||||
fadt->x_pm_tmr_blk.space_id = 1;
|
fadt->x_pm_tmr_blk.space_id = 1;
|
||||||
|
@ -181,7 +180,6 @@ void soc_fill_fadt(acpi_fadt_t *fadt)
|
||||||
fadt->x_pm_tmr_blk.access_size = 0;
|
fadt->x_pm_tmr_blk.access_size = 0;
|
||||||
fadt->x_pm_tmr_blk.addrl = pmbase + PM1_TMR;
|
fadt->x_pm_tmr_blk.addrl = pmbase + PM1_TMR;
|
||||||
fadt->x_pm_tmr_blk.addrh = 0x0;
|
fadt->x_pm_tmr_blk.addrh = 0x0;
|
||||||
}
|
|
||||||
|
|
||||||
if (config->s0ix_enable)
|
if (config->s0ix_enable)
|
||||||
fadt->flags |= ACPI_FADT_LOW_PWR_IDLE_S0;
|
fadt->flags |= ACPI_FADT_LOW_PWR_IDLE_S0;
|
||||||
|
|
|
@ -171,7 +171,6 @@ void soc_fill_fadt(acpi_fadt_t *fadt)
|
||||||
|
|
||||||
config_t *config = config_of_soc();
|
config_t *config = config_of_soc();
|
||||||
|
|
||||||
if (!config->PmTimerDisabled) {
|
|
||||||
fadt->pm_tmr_blk = pmbase + PM1_TMR;
|
fadt->pm_tmr_blk = pmbase + PM1_TMR;
|
||||||
fadt->pm_tmr_len = 4;
|
fadt->pm_tmr_len = 4;
|
||||||
fadt->x_pm_tmr_blk.space_id = 1;
|
fadt->x_pm_tmr_blk.space_id = 1;
|
||||||
|
@ -180,7 +179,6 @@ void soc_fill_fadt(acpi_fadt_t *fadt)
|
||||||
fadt->x_pm_tmr_blk.access_size = 0;
|
fadt->x_pm_tmr_blk.access_size = 0;
|
||||||
fadt->x_pm_tmr_blk.addrl = pmbase + PM1_TMR;
|
fadt->x_pm_tmr_blk.addrl = pmbase + PM1_TMR;
|
||||||
fadt->x_pm_tmr_blk.addrh = 0x0;
|
fadt->x_pm_tmr_blk.addrh = 0x0;
|
||||||
}
|
|
||||||
|
|
||||||
if (config->s0ix_enable)
|
if (config->s0ix_enable)
|
||||||
fadt->flags |= ACPI_FADT_LOW_PWR_IDLE_S0;
|
fadt->flags |= ACPI_FADT_LOW_PWR_IDLE_S0;
|
||||||
|
|
Loading…
Reference in New Issue