[coreboot-gerrit] Change in coreboot[master]: soc/intel: Add Intel SkyLake Scalable Processor support

[Nico Huber (Code Review)]

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

Change subject: soc/intel: Add Intel SkyLake Scalable Processor support
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38548/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/38548/5//COMMIT_MSG@13 
PS5, Line 13:  that will not be shared in public, at least for the time being.
> This is first time for coreboot to support Xeon-SP. […]

Getting support into upstream coreboot implies that the coreboot
community will have to maintain it. The last time we had a serverish
Intel platform upstream (Broadwell-DE) we ended up dropping the
support because of its binary situation, even though people still
use it. So please take this request to discuss it on the mailing list
seriously. If this ends up similarly to BDW-DE, you'll have wasted
your and our time.

Of course, you could avoid any trouble by getting the binary
released. At best, with a plan how long it will be maintained and
what SKUs it will support.