build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44713 )
Change subject: soc/mediatek/mt8192: Add dramc ac timing setting ......................................................................
Patch Set 2:
(4 comments)
https://review.coreboot.org/c/coreboot/+/44713/2/src/soc/mediatek/mt8192/dra... File src/soc/mediatek/mt8192/dramc_pi_main.c:
https://review.coreboot.org/c/coreboot/+/44713/2/src/soc/mediatek/mt8192/dra... PS2, Line 77: {.trfc = 100, .trfc_05t = 0, .trfc_pb = 44, .trfrc_pb05t = 0, .tx_ref_cnt = 115}, line over 96 characters
https://review.coreboot.org/c/coreboot/+/44713/2/src/soc/mediatek/mt8192/dra... PS2, Line 83: {.trfc = 137, .trfc_05t = 1, .trfc_pb = 63, .trfrc_pb05t = 0, .tx_ref_cnt = 154}, line over 96 characters
https://review.coreboot.org/c/coreboot/+/44713/2/src/soc/mediatek/mt8192/dra... PS2, Line 145: SET32_BITFIELDS(&ch[chn].ao.shu_ac_time_05t, SHU_AC_TIME_05T_TRFC_05T, trfc_05t); line over 96 characters
https://review.coreboot.org/c/coreboot/+/44713/2/src/soc/mediatek/mt8192/dra... PS2, Line 148: SET32_BITFIELDS(&ch[chn].ao.shu_ac_time_05t, SHU_AC_TIME_05T_TRFCPB_05T, trfrc_pb05t); line over 96 characters