diff options
author | Patrick Rudolph <patrick.rudolph@9elements.com> | 2025-04-22 10:48:00 +0200 |
---|---|---|
committer | Matt DeVillier <matt.devillier@gmail.com> | 2025-04-27 21:32:00 +0000 |
commit | 1044f03878c30a12ebc6ce20641b4358204d2327 (patch) | |
tree | e80daf67d472ed154a76d0cb5c46dc9b3b5672f6 /src/cpu/intel/model_f3x | |
parent | 2170ad0c60d1fc2b95f6a174e65fd9b3110444c0 (diff) | |
download | coreboot-main.tar.gz coreboot-main.tar.bz2 coreboot-main.zip |
On recent AMD platforms the VRT bit in the StatusD register is
read-writeable and set every 1024msec when RTC power is good.
This leads to a timeout in RtcWaitToUpdate() waiting for the bit
to be set and the gEfiRealTimeClockArchProtocolGuid won't be installed.
The protocol is critical to boot.
Adjust the code to not clear the VRT bit, as RtcWaitToUpdate() will
return an error, as it assumes the VRT bit is read-only and hardwired
to one as on Intel ICHs. While the timeout could be increased it
would also increase boot time by up to a second.
On platforms where the VRT bit is read-only the introduced code
does the same as before.
Change-Id: I8bc432114c83fa5f5fb35a144e3a35c38ee8a3de
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/87415
Reviewed-by: Matt DeVillier <matt.devillier@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@mailbox.org>
Diffstat (limited to 'src/cpu/intel/model_f3x')
0 files changed, 0 insertions, 0 deletions