diff options
author | Subrata Banik <subratabanik@google.com> | 2024-11-07 17:54:10 +0000 |
---|---|---|
committer | Subrata Banik <subratabanik@google.com> | 2024-11-12 03:16:31 +0000 |
commit | 28f71b5c3adf2bf82f55b359f26a58a78b0f172e (patch) | |
tree | ecd1271280d215f6476a2c70ac906f627749dc53 /Makefile.mk | |
parent | 2dd8f2e13b9e8bf10cd98e707534975f9ebb0ac4 (diff) |
soc/intel/cmn/pmc: Perform PM register init for CSE
Before entering FSP-M, AP firmware must ensure the PM1_CNT register
reflects the correct sleep state if a global reset occurred.
This is crucial when Intel CSE has reset the system, as indicated by
the global reset bit and wake status register.
If PM1_CNT doesn't contain a valid sleep state after a CSE reset, AP
firmware must enforce an S5 exit path before handing control to FSP-M
for CSE initialization. This ensures proper system initialization and
avoids potential issues caused by an inconsistent sleep state.
Additionally, clears the PM1 status register (PM1_STS) after retrieving
the power state. This prevents stale status information from persisting
across power cycles, which could lead to confusion during subsequent
boots.
BUG=b:265939425
TEST=Verified that `prev_sleep_state` holds the correct value
(5 for S5) after CSE performs a global reset.
Fixes: Inconsistent sleep state after CSE reset.
Change-Id: Iae9c026da86fef4a3571e06b1bb20504c3d8c9be
Signed-off-by: Subrata Banik <subratabanik@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/85025
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
Reviewed-by: Kapil Porwal <kapilporwal@google.com>
Diffstat (limited to 'Makefile.mk')
0 files changed, 0 insertions, 0 deletions