[coreboot-gerrit] Change in ...coreboot[master]: mediatek/mt8183: fix mode register setting fail issue

[huayang duan (Code Review)]

huayang duan has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32284 )

Change subject: mediatek/mt8183: fix mode register setting fail issue
......................................................................

Patch Set 5:

(3 comments)

https://review.coreboot.org/#/c/32284/4//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/32284/4//COMMIT_MSG@9 
PS4, Line 9: the
> The

Done

https://review.coreboot.org/#/c/32284/4/src/soc/mediatek/mt8183/dramc_init_s... 
File src/soc/mediatek/mt8183/dramc_init_setting.c:

https://review.coreboot.org/#/c/32284/4/src/soc/mediatek/mt8183/dramc_init_s... 
PS4, Line 21: #include <delay.h>
> sort the include lines

Done

https://review.coreboot.org/#/c/32284/4/src/soc/mediatek/mt8183/dramc_init_s... 
PS4, Line 668: dramc_mode_reg_init_sequence
> This array should be mark as 'static' if it won't be access from any other file. […]

Done