[coreboot-gerrit] Change in ...coreboot[master]: Documentation/../../icelake: Add Ice Lake coreboot development docume...

[Philipp Deppenwiese (Code Review)]

Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29805 )

Change subject: Documentation/../../icelake: Add Ice Lake coreboot development documentation
......................................................................


Patch Set 5:

> Patch Set 3:
> 
> > Please give more a detailed overview of the supported icelake
>  > features and their requirements. Also please recommend to use
>  > 3rdparty/fsp via kconfig instead of copying binaries into the tree.
> 
> Philipp,
> 
> Kindly consider the fact that ICL SOC development is still ongoing on and many such feature will be not in public as soc is not out in market for everyone's consumption. i have also mentioned a Intel public link for ICL, you may find more feature details if you have partner access. Kindly consider this case as we are up streaming ICL code almost same time SOC development is going on.
> 
> Apart from that once product is out, i hope your FSP binary problem will also resolve as it will available in separate git like other program. And i know already a process been set to make use of that. Due to the fact that ICL FSP binaries are still not available externally for all so, we have to make use of copy stuff.

Got it. Then do a follow-up CL later on


-- 
To view, visit https://review.coreboot.org/c/coreboot/+/29805
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ief4df6ca11f95b75ecddeb560f7887bfadced086
Gerrit-Change-Number: 29805
Gerrit-PatchSet: 5
Gerrit-Owner: Subrata Banik <subrata.banik at intel.com>
Gerrit-Reviewer: Aamir Bohra <aamir.bohra at intel.com>
Gerrit-Reviewer: Aaron Durbin <adurbin at chromium.org>
Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan at intel.com>
Gerrit-Reviewer: Duncan Laurie <dlaurie at chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan at google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi at google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph at 9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki at gmail.com>
Gerrit-Reviewer: Shelley Chen <shchen at google.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer at coreboot.org>
Gerrit-Reviewer: Subrata Banik <subrata.banik at intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-Comment-Date: Mon, 26 Nov 2018 16:16:56 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181126/496d6cc9/attachment.html>