Xi Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44700 )
Change subject: soc/mediatek/mt8192: Do dramc init settings ......................................................................
Patch Set 28:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44700/28/src/soc/mediatek/mt8192/dr... File src/soc/mediatek/mt8192/dramc_pi_main.c:
https://review.coreboot.org/c/coreboot/+/44700/28/src/soc/mediatek/mt8192/dr... PS28, Line 7: static void set_vcore_voltage_for_each_freq(const struct ddr_cali *cali)
that open brace { should be on the previous line
bot makes a mistake?