Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42193 )
Change subject: soc/mediatek/mt8183: Fix some programming errors of DRAMC setting ......................................................................
Patch Set 6:
(3 comments)
https://review.coreboot.org/c/coreboot/+/42193/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/42193/2//COMMIT_MSG@7 PS2, Line 7: soc/mediatek/mt8183: Fix some programming errors of DRAMC setting
Maybe something like […]
Huayang, could you change the commit message title to what I suggested?
https://review.coreboot.org/c/coreboot/+/42193/6/src/soc/mediatek/mt8183/dra... File src/soc/mediatek/mt8183/dramc_init_setting.c:
https://review.coreboot.org/c/coreboot/+/42193/6/src/soc/mediatek/mt8183/dra... PS6, Line 894: Remove this tab.
https://review.coreboot.org/c/coreboot/+/42193/6/src/soc/mediatek/mt8183/dra... PS6, Line 895: Remove this tab.