[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.
>
> Shall we merge it like this and I open an additional CR to cleanup vendorcode/intel/fsp2_0 ?

If you are in a hurry, we can do that. I would just have waited
for people to comment here.