[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp2_0: Allow to add FSP binaries from repo for IceLake

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37561 )

Change subject: drivers/intel/fsp2_0: Allow to add FSP binaries from repo for IceLake
......................................................................

Patch Set 7:
> > Should we drop `vendorcode/intel/fsp/fsp2_0/icelake/`?
>
> Maybe I misunderstand things, but adding the option to read it for the 3rdparty repo is there to be an addition. It still should work without the in-tree repo checked out, right?

Not really. Well, depends on what should work. It wouldn't boot anyway
without the binaries. And for build tests, we assume the 3rdparty/ repos
to be checked out.

It's not a requirement to drop the stale copies now. Still, I'd like to
take the opportunity to discuss it. Otherwise we might drag them along
for ever without anybody making use of them.