Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34332 )
Change subject: mediatek/mt8183: support DDR frequency 3600Mbps ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/34332/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/34332/1//COMMIT_MSG@9 PS1, Line 9: DDR frequency fix at 3600Mbps Please elaborate. For a diffstat of over 10 lines (here over a thousand), an elaborate commit message is required.
https://review.coreboot.org/c/coreboot/+/34332/1//COMMIT_MSG@16 PS1, Line 16: Change-Id: Ic1378ca43fb333c445ca77e7dc0844cdf65f2207 Please move this directly above the Signed-off-by line.