mturney mturney has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35508 )
Change subject: HACK trogdor: SoC makefile BLOB support HACK ......................................................................
Patch Set 17:
(1 comment)
https://review.coreboot.org/c/coreboot/+/35508/15/src/soc/qualcomm/sc7180/Ma... File src/soc/qualcomm/sc7180/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/35508/15/src/soc/qualcomm/sc7180/Ma... PS15, Line 102: DCB_FILE := $(SC7180_BLOB)/600E_0100_1_1_dcb.bin
Julius, […]
Julius, After discussing with DRAM team there is a hard dependency (buried within QCLib) on the actual filename. If we make it a generic name in the blobs directory we shift the dependency to qclib.c code in coreboot. Internally we feel this is a bad trade-off, especially since the main issue you describe (silently build without the file) disappears when the licensing is resolved and the blobs are actually pre-populated or available for builds.
We can discuss further but you need to be aware that the filename dependency exists and the only discussion is where to bury the code/mechanism that addresses the dependency.