[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 50:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/35481/50//COMMIT_MSG@16 
PS50, Line 16: draining out of battery
> Doesn’t the hang also drain the battery? […]

the reboot process would consume more power consumption, and may cause I/O on more peripherals (for example SPI), especially causing TPM issues.

there's currently no standard way of handling DRAM initialization error on chromebooks, but I think we should just hang(halt) or poweroff. Let's try hang for now.