Attention is currently required from: Hung-Te Lin, Yidi Lin, Yu-Ping Wu.
Gavin Liu has posted comments on this change by Gavin Liu. ( https://review.coreboot.org/c/coreboot/+/86252?usp=email )
Change subject: soc/mediatek/mt8196: Add TF-A static library path ......................................................................
Patch Set 2:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/86252/comment/e92cabb0_8a832b87?usp... : PS1, Line 7: add
Add
Done
https://review.coreboot.org/c/coreboot/+/86252/comment/af251f05_763eddff?usp... : PS1, Line 9: Add TF-A static library path to build argument.
BL31 build argument.
Done
File src/soc/mediatek/mt8196/Makefile.mk:
https://review.coreboot.org/c/coreboot/+/86252/comment/35d71ff9_942de681?usp... : PS1, Line 96: MTKLIB_PATH
`BL31_LIB`
Done
https://review.coreboot.org/c/coreboot/+/86252/comment/1fbd63ca_66b08608?usp... : PS1, Line 96: libmtk.a
We should use more specific file name in the blobs repo. How about `bl31_lib. […]
We have already pushed the corresponding tf-a patch. If we change the naming, it maybe affect tf-a. The naming is reference other platform in coreboot.