build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44710 )
Change subject: soc/mediatek/mt8192: Do dramc duty calibration ......................................................................
Patch Set 1:
(4 comments)
https://review.coreboot.org/c/coreboot/+/44710/1/src/soc/mediatek/mt8192/dra... File src/soc/mediatek/mt8192/dramc_pi_calibration_api.c:
https://review.coreboot.org/c/coreboot/+/44710/1/src/soc/mediatek/mt8192/dra... PS1, Line 597: static void dramc_duty_set_clk_delay_cell(u8 chn, const s8* duty_delay) "foo* bar" should be "foo *bar"
https://review.coreboot.org/c/coreboot/+/44710/1/src/soc/mediatek/mt8192/dra... PS1, Line 606: static void dramc_duty_set_dqs_delay_cell(u8 chn, const s8* duty_delay) "foo* bar" should be "foo *bar"
https://review.coreboot.org/c/coreboot/+/44710/1/src/soc/mediatek/mt8192/dra... PS1, Line 619: static void dramc_duty_set_wck_delay_cell(u8 chn, const s8* duty_delay) "foo* bar" should be "foo *bar"
https://review.coreboot.org/c/coreboot/+/44710/1/src/soc/mediatek/mt8192/dra... PS1, Line 632: const s8* duty_delay, u8 k_type) "foo* bar" should be "foo *bar"