[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/mt8183: Handle memory test failure

[Hung-Te Lin (Code Review)]

Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35481 )

Change subject: soc/mediatek/mt8183: Handle memory test failure
......................................................................

Patch Set 49:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35481/49//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/35481/49//COMMIT_MSG@11 
PS49, Line 11: If partial calibration fails or the mem test following it
             : fails, hang forever.
> Why is that a good idea?

That means we have no any other ways to initialize DRAM (usually implies hardware failure). Instead of falling into reboot loop and drains out of battery, it seems better to just hang so user may notice that and send to RMA.