Hannah Williams has posted comments on this change. ( https://review.coreboot.org/18558 )
Change subject: soc/intel/common/block: Add Intel common RTC code support
......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/#/c/18558/3/src/soc/intel/common/block/include/i...
File src/soc/intel/common/block/include/intelblocks/rtc.h:
Line 19: void soc_rtc_config(void);
Why is this called soc_rtc_config when its really only about enabling the u
The register name is RTC Configuration
--
To view, visit
https://review.coreboot.org/18558
To unsubscribe, visit
https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id9dfcdbc300c25f43936d1efb5d6f9d81d3c8453
Gerrit-PatchSet: 3
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Barnali Sarkar
barnali.sarkar@intel.com
Gerrit-Reviewer: Aaron Durbin
adurbin@chromium.org
Gerrit-Reviewer: Balaji Manigandan
balaji.manigandan@intel.com
Gerrit-Reviewer: Bora Guvendik
bora.guvendik@intel.com
Gerrit-Reviewer: Hannah Williams
hannah.williams@intel.com
Gerrit-Reviewer: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Reviewer: Subrata Banik
subrata.banik@intel.com
Gerrit-Reviewer: Sumeet R Pawnikar
sumeet.r.pawnikar@intel.com
Gerrit-Reviewer: build bot (Jenkins)
Gerrit-HasComments: Yes