[coreboot-gerrit] Change in coreboot[master]: SPM: mt8183: support spm suspend in coreboot

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34545 )

Change subject: SPM: mt8183: support spm suspend in coreboot
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34545/1/src/soc/mediatek/mt8183/spm... 
File src/soc/mediatek/mt8183/spm.c:

https://review.coreboot.org/c/coreboot/+/34545/1/src/soc/mediatek/mt8183/spm... 
PS1, Line 77: 	unsigned int con1, retry = 0, timeout = 5000;
You might want to add the unit to the variable name: timeout_us. coreboot’s stopwatch framework should help you with this.