build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44700 )
Change subject: soc/mediatek/mt8192: Do dramc init settings ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/44700/1/src/soc/mediatek/mt8192/dra... File src/soc/mediatek/mt8192/dramc_pi_main.c:
https://review.coreboot.org/c/coreboot/+/44700/1/src/soc/mediatek/mt8192/dra... PS1, Line 7: static void set_vcore_voltage_for_each_freq(const struct ddr_cali *cali) that open brace { should be on the previous line
https://review.coreboot.org/c/coreboot/+/44700/1/src/soc/mediatek/mt8192/dra... PS1, Line 56: first_freq_k= false; spaces required around that '=' (ctx:VxW)