Yidi Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46395 )
Change subject: soc/mediatek/mt8192: add rtc MT6359P driver ......................................................................
Patch Set 42:
(3 comments)
https://review.coreboot.org/c/coreboot/+/46395/42/src/soc/mediatek/common/rt... File src/soc/mediatek/common/rtc_cali.c:
https://review.coreboot.org/c/coreboot/+/46395/42/src/soc/mediatek/common/rt... PS42, Line 77: remove blank line
https://review.coreboot.org/c/coreboot/+/46395/42/src/soc/mediatek/mt8192/in... File src/soc/mediatek/mt8192/include/soc/rtc.h:
https://review.coreboot.org/c/coreboot/+/46395/42/src/soc/mediatek/mt8192/in... PS42, Line 235: remove blank line
https://review.coreboot.org/c/coreboot/+/46395/42/src/soc/mediatek/mt8192/rt... File src/soc/mediatek/mt8192/rtc.c:
https://review.coreboot.org/c/coreboot/+/46395/42/src/soc/mediatek/mt8192/rt... PS42, Line 362: remove blank line