[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/mt8192: Add dpm loader

[Yu-Ping Wu (Code Review)]

Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46393 )

Change subject: soc/mediatek/mt8192: Add dpm loader
......................................................................

Patch Set 35:

(2 comments)

https://review.coreboot.org/c/coreboot/+/46393/35/src/soc/mediatek/mt8192/dp... 
File src/soc/mediatek/mt8192/dpm.c:

https://review.coreboot.org/c/coreboot/+/46393/35/src/soc/mediatek/mt8192/dp... 
PS35, Line 29: ret = -1;
Simply write

return -1;

here, and remove the 'ret' variable.

https://review.coreboot.org/c/coreboot/+/46393/35/src/soc/mediatek/mt8192/dp... 
PS35, Line 31: 		ret = -2;
return -2 (Actually I wonder if we can also return -1 here since mtlib_load_and_run() should have printed the file name in the error message)