Attention is currently required from: Yu-Ping Wu, hsin-hsiung wang. Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51246 )
Change subject: soc/mediatek/mt8192: mt6315: update initial setting ......................................................................
Patch Set 1:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/51246/comment/4aead0fd_88069ba9 PS1, Line 7: update are we updating this to fix a bug or simply revising?
https://review.coreboot.org/c/coreboot/+/51246/comment/c53ceb48_46c7be0d PS1, Line 9: According to current flow, we don't check the boot reason of mt6315, so : removing the unused setting about boot status. : Reset power-off sequence to zero to make sure it meets hardware : requirement. - Remove unused boot status settings - Reset the power-off sequence to zero to meet hardware requirement
Even better if you can provide more info on what hardware requirement this would help to meet?
-
Hung-Te Lin (Code Review)