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

[Yidi Lin (Code Review)]

Yidi Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46393 )

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

Patch Set 28:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/46393/28/src/soc/mediatek/mt8192/dp... 
PS28, Line 25: 
             : 	dm_file_bytes = cbfs_boot_load_file(dm_file_name, dpm_dm_bin,
             : 					sizeof(dpm_dm_bin), CBFS_TYPE_RAW);
             : 	if (dm_file_bytes == 0) {
             : 		printk(BIOS_ERR, "binary %s not found\n", dm_file_name);
             : 		return -1;
             : 	}
             : 
             : 	pm_file_bytes = cbfs_boot_load_file(pm_file_name, dpm_pm_bin,
             : 					sizeof(dpm_pm_bin), CBFS_TYPE_RAW);
             : 	if (pm_file_bytes == 0) {
             : 		printk(BIOS_ERR, "binary %s not found\n", pm_file_name);
             : 		return -2;
             : 	}
> why can't we use the ocmmon API?

I will squash CB:46932 into this patch in next patchset.