soc/intel/xeon_sp: Add function to clear PMCON status bits

This patch adds an SoC function to clear GEN_PMCON_A status bits to
align with other IA coreboot implementations.

BUG=b:211954778
TEST=None.

Signed-off-by: Subrata Banik <subratabanik@google.com>
Change-Id: I22650f539a1646f93f2c6494cbf54b8ca785d6ad
Reviewed-on: https://review.coreboot.org/c/coreboot/+/61652
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
This commit is contained in:
Subrata Banik 2022-02-06 19:35:56 +05:30 committed by Felix Held
parent 42914feb1f
commit 9e00a817f3
3 changed files with 19 additions and 0 deletions

View File

@ -121,4 +121,7 @@ uint16_t get_pmbase(void);
void pmc_lock_smi(void);
/* Clear PMCON status bits */
void pmc_clear_pmcon_sts(void);
#endif

View File

@ -22,6 +22,7 @@
#define PWRMBASE 0x48
#define GEN_PMCON_A 0xa0
#define DISB (1 << 23)
#define MS4V (1 << 18)
#define GBL_RST_STS (1 << 16)
#define SMI_LOCK (1 << 4)
#define GEN_PMCON_B 0xa4

View File

@ -179,3 +179,18 @@ void pmc_soc_set_afterg3_en(const bool on)
reg8 |= SLEEP_AFTER_POWER_FAIL;
pci_write_config8(PCH_DEV_PMC, GEN_PMCON_B, reg8);
}
void pmc_clear_pmcon_sts(void)
{
uint32_t reg_val;
const pci_devfn_t dev = PCH_DEV_PMC;
reg_val = pci_read_config32(dev, GEN_PMCON_A);
/*
* Clear SUS_PWR_FLR, GBL_RST_STS, HOST_RST_STS, PWR_FLR bits
* while retaining MS4V write-1-to-clear bit
*/
reg_val &= ~(MS4V);
pci_write_config32(dev, GEN_PMCON_A, reg_val);
}