build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46877 )
Change subject: soc/mediatek/mt8192: add rtc MT6359P driver ......................................................................
Patch Set 3:
(9 comments)
https://review.coreboot.org/c/coreboot/+/46877/3/src/soc/mediatek/mt8192/inc... File src/soc/mediatek/mt8192/include/soc/rtc.h:
https://review.coreboot.org/c/coreboot/+/46877/3/src/soc/mediatek/mt8192/inc... PS3, Line 113: RTC_GP_OSC32_CON = 2U << 13, /* Keep RG_EOSC_RSV[0] to low for lower leakage current hw design change */ line over 96 characters
https://review.coreboot.org/c/coreboot/+/46877/3/src/soc/mediatek/mt8192/inc... PS3, Line 118: OSC32CON_ANALOG_SETTING = RTC_GP_OSC32_CON | RTC_EOSC32_CHOP_EN | RTC_EOSC32_VCT_EN |\ Avoid unnecessary line continuations
https://review.coreboot.org/c/coreboot/+/46877/3/src/soc/mediatek/mt8192/inc... PS3, Line 119: RTC_GPS_CKOUT_EN | RTC_EMBCK_SEL_OPTION | RTC_EMB_K_EOSC32_MODE line over 96 characters
https://review.coreboot.org/c/coreboot/+/46877/3/src/soc/mediatek/mt8192/rtc... File src/soc/mediatek/mt8192/rtc.c:
https://review.coreboot.org/c/coreboot/+/46877/3/src/soc/mediatek/mt8192/rtc... PS3, Line 190: if (middle == left) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/46877/3/src/soc/mediatek/mt8192/rtc... PS3, Line 195: if (val >= RTC_FQMTR_LOW_BASE && val <= RTC_FQMTR_HIGH_BASE) { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/46877/3/src/soc/mediatek/mt8192/rtc... PS3, Line 198: if (val > RTC_FQMTR_HIGH_BASE) { braces {} are not necessary for any arm of this statement
https://review.coreboot.org/c/coreboot/+/46877/3/src/soc/mediatek/mt8192/rtc... PS3, Line 201: else { else should follow close brace '}'
https://review.coreboot.org/c/coreboot/+/46877/3/src/soc/mediatek/mt8192/rtc... PS3, Line 201: else { braces {} are not necessary for single statement blocks
https://review.coreboot.org/c/coreboot/+/46877/3/src/soc/mediatek/mt8192/rtc... PS3, Line 382: rtc_write(RTC_AL_YEA, (year | RTC_K_EOSC_RSV_0) & (~RTC_K_EOSC_RSV_1) & (~RTC_K_EOSC_RSV_2)); line over 96 characters