Yidi Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46389 )
Change subject: mediatek/mt8192: add spmfw loader ......................................................................
Patch Set 4:
Patch Set 3:
No rule to make target '3rdparty/blobs/soc/mediatek/mt8192/spm_firmware.bin
@yidi, you have to also upload the changes for 3rdparty/blobs, and then a 'update submodule' commit before landing this.
The blob tree@coreboot.org[1] is quite different from the blob@googlesource[2]. And I heard from Roger this patch is still using hex array for some reason. So I think we can either move this patch to the end of the chain or revise the Makefile.
1:https://review.coreboot.org/plugins/gitiles/blobs/+log/refs/heads/master 2:https://chromium.googlesource.com/chromiumos/third_party/coreboot/blobs/+log...