Change in coreboot[master]: soc/intel/elkhartlake: Introduce Intel PSE

[Nico Huber (Code Review)]

Attention is currently required from: Arthur Heymans, Maulik V Vaghela, Mario Scheithauer, Subrata Banik, Lean Sheng Tan, Michael Niewöhner, Werner Zeh, Patrick Rudolph, Felix Held.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55367 )

Change subject: soc/intel/elkhartlake: Introduce Intel PSE
......................................................................

Patch Set 46: Code-Review-1

(1 comment)

Patchset:

PS46: 
As I understand the situation now, using the PSE is optional and
the program running on it can be open-source. Wouldn't it be much
better to treat it as open-source from the beginning? i.e. inte-
grate the mininum code into coreboot, so every board owner can
add the code they need.

If we treat it like a blob as this change currently does, we would
not only make it harder for board developers to add their code,
we'd also encourage proprietary implementations. IMHO, that's not
a direction a free-software project should move into. On top it
seems more expensive for everyone, especially for the coreboot
community. We'd not only have to maintain the option to load a
blob that shouldn't be one, we'd also have to handle support re-
quests if something goes wrong with it. Helping somebody with
their proprietary blob is not fun, helping somebody to integrate
their open-source code is!