]> git.proxmox.com Git - mirror_edk2.git/commit - PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.c
PcAtChipsetPkg: Change the flow of PcRtcInit()
authorChao, Zhuoran <zhuoran.chao@intel.com>
Sat, 7 May 2022 09:39:37 +0000 (17:39 +0800)
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Mon, 9 May 2022 02:45:01 +0000 (02:45 +0000)
commit29ae55a0b8194fef2e769f2882297aa601b4777a
tree9321710320c38ac84a31c44b28de26913e9292a5
parent1a6c837638e4081b5154fd224666a80e1eda85fe
PcAtChipsetPkg: Change the flow of PcRtcInit()

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3913

The original flow of PcRtcInit() is like:
1. Guarantee atomic accesses to the RTC time registers and
   read out the value.
2. Program RTC register B. (adopt 12h mode or 24h mode. Current
   bios code sets RTC to 24h mode by default).
3. Then function ConvertRtcTimeToEfiTime converts the RTC time
   value to their 24h mode by checking the hour format bit
   (1:24h mode,0:12h mode).
And here lies the problem: Step3 will fail to adjust the value
if Step2 already sets RTC to 24h mode. The hour value in 12h mode
will not be converted to its 24h mode.
The solution is to program RTC register B a little later when all
the original RTC registers' value is retrieved, adjusted
and validated.

ConvertRtcTimeToEfiTime is modified to be more robust.

Reviewed-by: Ray Ni <ray.ni@intel.com>
Signed-off-by: Zhuoran Chao <zhuoran.chao@intel.com>
PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.c