build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40525 )
Change subject: soc/mediatek/mt8183: High frequency should use range 1 [DRAFT] [DONOT MERGE] ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/40525/1/src/soc/mediatek/mt8183/dra... File src/soc/mediatek/mt8183/dramc_pi_calibration_api.c:
https://review.coreboot.org/c/coreboot/+/40525/1/src/soc/mediatek/mt8183/dra... PS1, Line 278: dramc_mode_reg_write_by_rank(chn, rank, 12, final_vref | ( 0x1 << 6)); space prohibited after that open parenthesis '('
https://review.coreboot.org/c/coreboot/+/40525/1/src/soc/mediatek/mt8183/dra... PS1, Line 1834: vref_begin = params->tx_vref[chn][rank] | (vref_range << 6) ; space prohibited before semicolon