Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79026?usp=email )
Change subject: soc/qualcomm/{sc7180,sc7280}: Allow building without QC blobs repo ......................................................................
Patch Set 4:
(1 comment)
Patchset:
PS4: (Strange... I don't understand why I didn't get an email for this even though I have notifications for soc/qualcomm enabled.)
I don't think this was a good idea and I would prefer to revert this patch. The blobs are essential to make these SoCs run and with this patch people just silently get non-working images back without anything telling them that they need to enable some Kconfig that isn't selected automatically.
Can you explain what other cases you have that need this and why they can't just build with the blobs instead? My understanding is that our build tests already build with `abuild -B` (CI testing on these boards has been working just fine for years, after all). I'm not quite sure how Coverity works (does it even build every board individually?) but if it currently is not using these blobs, I don't see why we couldn't just turn them on there. That seems better than tricking people into building broken images by accident.