Huayang Duan has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35555 )
Change subject: soc/mediatek/mt8183: Run calibration with multiple frequencies for DVFS switch ......................................................................
Patch Set 16:
(5 comments)
https://review.coreboot.org/c/coreboot/+/35555/12/src/soc/mediatek/mt8183/dr... File src/soc/mediatek/mt8183/dramc_init_setting.c:
https://review.coreboot.org/c/coreboot/+/35555/12/src/soc/mediatek/mt8183/dr... PS12, Line 1086: dramc_set_broadcast
is this a general bug fix?
is a improve, not a bug. no need set those register to 2 channel.
https://review.coreboot.org/c/coreboot/+/35555/12/src/soc/mediatek/mt8183/dr... PS12, Line 417: setbits_le32(&ch[0].phy.b[0].dq[8], (0x1 << 0) | (0x1 << 1) | (0x1 << 2)); : setbits_le32(&ch[0].phy.b[1].dq[8], (0x1 << 0) | (0x1 << 1) | (0x1 << 2)); : setbits_le32(&ch[0].phy.ca_cmd[9], (0x1 << 0) | (0x1 << 1) | (0x1 << 2));
is this really related to this patch (enable multi-frequency)? […]
is a improve, not a bug. no need set those register to 2 channel.
https://review.coreboot.org/c/coreboot/+/35555/12/src/soc/mediatek/mt8183/dr... PS12, Line 921: clrbits_le32(&ch[0].phy.b[1].dq[7], (0x1 << 7) | (0x1 << 13));
is this a general bug fix?
fix typo error, this error will make DVFS switch fail.
https://review.coreboot.org/c/coreboot/+/35555/12/src/soc/mediatek/mt8183/dr... PS12, Line 983: [1]
is this a general bug fix?
fix typo error, this error will make DVFS switch fail.
https://review.coreboot.org/c/coreboot/+/35555/12/src/soc/mediatek/mt8183/dr... PS12, Line 1215: 0
is this a general bug fix?
is a improve, not a bug. no need set those register to 2 channel.