Attention is currently required from: Hung-Te Lin, Yidi Lin, Yu-Ping Wu.
Jarried Lin has posted comments on this change by Jarried Lin. ( https://review.coreboot.org/c/coreboot/+/86353?usp=email )
Change subject: soc/mediatek/mt8196: Fix RTC recovery by disabling external XTAL ......................................................................
Patch Set 2:
(8 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/86353/comment/a10199cb_e177d923?usp... : PS1, Line 7: issue of RTC recovery failed
RTC recovery by disabling external XTAL
Done
https://review.coreboot.org/c/coreboot/+/86353/comment/255f0256_b384072a?usp... : PS1, Line 9: decide whether
decides whether to
Done
https://review.coreboot.org/c/coreboot/+/86353/comment/3516b8f8_e0475c07?usp... : PS1, Line 9: Config
Configure
Done
https://review.coreboot.org/c/coreboot/+/86353/comment/7c2aa11e_4b6abcef?usp... : PS1, Line 9: pmic
PMIC
Done
https://review.coreboot.org/c/coreboot/+/86353/comment/c5380f91_193b142f?usp... : PS1, Line 9: , : t
. T
Done
https://review.coreboot.org/c/coreboot/+/86353/comment/7aa3e538_d1b45f0f?usp... : PS1, Line 10: , which means without external : xtal.
to disable external xtal.
Done
https://review.coreboot.org/c/coreboot/+/86353/comment/a8214d86_11c9d48b?usp... : PS1, Line 16: rtc will boot normally
RTC boots normally.
Done
https://review.coreboot.org/c/coreboot/+/86353/comment/aabf0a8d_3061e688?usp... : PS1, Line 16: charge
charger ?
Done