Change in coreboot[master]: Documentation/soc/intel/fsp: Add link for external FSP2.1 spec
Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/35238 )
Change subject: Documentation/soc/intel/fsp: Add link for external FSP2.1 spec ......................................................................
Documentation/soc/intel/fsp: Add link for external FSP2.1 spec
Change-Id: I5d1abd6252bda2be09285cd878a483f055abcd7b Signed-off-by: Subrata Banik subrata.banik@intel.com --- M Documentation/soc/intel/fsp/index.md 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/35238/1
diff --git a/Documentation/soc/intel/fsp/index.md b/Documentation/soc/intel/fsp/index.md index 6269445..9f307e6 100644 --- a/Documentation/soc/intel/fsp/index.md +++ b/Documentation/soc/intel/fsp/index.md @@ -45,6 +45,8 @@
* [FSP Specification 2.0](https://www.intel.com/content/dam/www/public/us/en/documents/technical-speci...)
+* [FSP Specification 2.1](https://cdrdv2.intel.com/v1/dl/getContent/609253) + ## Additional Features in FSP 2.1 specification
- [PPI](ppi/ppi.md)
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35238 )
Change subject: Documentation/soc/intel/fsp: Add link for external FSP2.1 spec ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/35238/1/Documentation/soc/intel/fsp... File Documentation/soc/intel/fsp/index.md:
https://review.coreboot.org/c/coreboot/+/35238/1/Documentation/soc/intel/fsp... PS1, Line 48: https://cdrdv2.intel.com/v1/dl/getContent/609253 Just curious: This link looks very different than the links that were added previously for specification 1.0 , 1.1 and 2.0. Is that intentional?
Hello Nathaniel L Desimone, Aamir Bohra, Philipp Deppenwiese, build bot (Jenkins), Furquan Shaikh, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/35238
to look at the new patch set (#2).
Change subject: Documentation/soc/intel/fsp: Add link for external FSP2.1 spec ......................................................................
Documentation/soc/intel/fsp: Add link for external FSP2.1 spec
Change-Id: I5d1abd6252bda2be09285cd878a483f055abcd7b Signed-off-by: Subrata Banik subrata.banik@intel.com --- M Documentation/soc/intel/fsp/index.md 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/35238/2
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35238 )
Change subject: Documentation/soc/intel/fsp: Add link for external FSP2.1 spec ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/35238/1/Documentation/soc/intel/fsp... File Documentation/soc/intel/fsp/index.md:
https://review.coreboot.org/c/coreboot/+/35238/1/Documentation/soc/intel/fsp... PS1, Line 48: https://cdrdv2.intel.com/v1/dl/getContent/609253
Just curious: This link looks very different than the links that were added previously for specifica […]
this is downloadable link unlike previous links where we can browse
Aamir Bohra has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35238 )
Change subject: Documentation/soc/intel/fsp: Add link for external FSP2.1 spec ......................................................................
Patch Set 3: Code-Review+2
Hello Nathaniel L Desimone, Aamir Bohra, Philipp Deppenwiese, build bot (Jenkins), Furquan Shaikh, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/35238
to look at the new patch set (#4).
Change subject: Documentation/soc/intel/fsp: Add link for external FSP2.1 spec ......................................................................
Documentation/soc/intel/fsp: Add link for external FSP2.1 spec
Change-Id: I5d1abd6252bda2be09285cd878a483f055abcd7b Signed-off-by: Subrata Banik subrata.banik@intel.com --- M Documentation/soc/intel/fsp/index.md 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/35238/4
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/35238 )
Change subject: Documentation/soc/intel/fsp: Add link for external FSP2.1 spec ......................................................................
Documentation/soc/intel/fsp: Add link for external FSP2.1 spec
Change-Id: I5d1abd6252bda2be09285cd878a483f055abcd7b Signed-off-by: Subrata Banik subrata.banik@intel.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/35238 Reviewed-by: Aamir Bohra aamir.bohra@intel.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M Documentation/soc/intel/fsp/index.md 1 file changed, 2 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Aamir Bohra: Looks good to me, approved
diff --git a/Documentation/soc/intel/fsp/index.md b/Documentation/soc/intel/fsp/index.md index 6269445..cd7fe0b 100644 --- a/Documentation/soc/intel/fsp/index.md +++ b/Documentation/soc/intel/fsp/index.md @@ -45,6 +45,8 @@
* [FSP Specification 2.0](https://www.intel.com/content/dam/www/public/us/en/documents/technical-speci...)
+* [FSP Specification 2.1](https://cdrdv2.intel.com/v1/dl/getContent/611786) + ## Additional Features in FSP 2.1 specification
- [PPI](ppi/ppi.md)
-
Aamir Bohra (Code Review) -
Furquan Shaikh (Code Review)
-
Patrick Georgi (Code Review)
-
Subrata Banik (Code Review)