Change in ...coreboot[master]: Documentation/soc/intel/fsp: Move mp_service_ppi document in corrct d...

[Subrata Banik (Code Review)]

Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31840

Change subject: Documentation/soc/intel/fsp: Move mp_service_ppi document in corrct directory
......................................................................

Documentation/soc/intel/fsp: Move mp_service_ppi document in corrct directory

This patch moves mp service ppi document from icelake/MultiProcesorInit.md
to ppi/mp_service_ppi.c.

Change-Id: I1bbaeb2644f219b5a1fda0c7c4b594184d53958c
Signed-off-by: Subrata Banik 
---
R Documentation/soc/intel/fsp/ppi/coreboot_publish_mp_service_api.png
R Documentation/soc/intel/fsp/ppi/mp_service_ppi.md
M Documentation/soc/intel/fsp/ppi/ppi.md
M Documentation/soc/intel/icelake/index.md
4 files changed, 5 insertions(+), 4 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/40/31840/1

diff --git a/Documentation/soc/intel/icelake/coreboot_publish_mp_service_api.png b/Documentation/soc/intel/fsp/ppi/coreboot_publish_mp_service_api.png
similarity index 100%
rename from Documentation/soc/intel/icelake/coreboot_publish_mp_service_api.png
rename to Documentation/soc/intel/fsp/ppi/coreboot_publish_mp_service_api.png
Binary files differ
diff --git a/Documentation/soc/intel/icelake/MultiProcessorInit.md b/Documentation/soc/intel/fsp/ppi/mp_service_ppi.md
similarity index 100%
rename from Documentation/soc/intel/icelake/MultiProcessorInit.md
rename to Documentation/soc/intel/fsp/ppi/mp_service_ppi.md
diff --git a/Documentation/soc/intel/fsp/ppi/ppi.md b/Documentation/soc/intel/fsp/ppi/ppi.md
index 66dbf07..6d7afb4 100644
--- a/Documentation/soc/intel/fsp/ppi/ppi.md
+++ b/Documentation/soc/intel/fsp/ppi/ppi.md
@@ -7,3 +7,8 @@
 able to execute the same in FSP's context.

* [What is PPI](https://www.intel.com/content/dam/www/public/us/en/documents/reference-guide...)
+
+## List of PPI service
+
+### Publish MP Service PPI from boot firmware (coreboot) to initialize CPU
+- [MP Service PPI](mp_service_ppi.md)
diff --git a/Documentation/soc/intel/icelake/index.md b/Documentation/soc/intel/icelake/index.md
index 450ab0f..71397d2 100644
--- a/Documentation/soc/intel/icelake/index.md
+++ b/Documentation/soc/intel/icelake/index.md
@@ -5,7 +5,3 @@
 ## Ice Lake coreboot development

- [Ice Lake coreboot development](iceLake_coreboot_development.md)
-
-## Multiprocessor Init
-
-- [Multiprocessor Init](MultiProcessorInit.md)

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31840 )

Change subject: Documentation/soc/intel/fsp: Move mp_service_ppi document in corrct directory
......................................................................

Patch Set 1: Code-Review+2

[Philipp Deppenwiese (Code Review)]

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

Change subject: Documentation/soc/intel/fsp: Move mp_service_ppi document in corrct directory
......................................................................

Patch Set 1: Code-Review+2

[Werner Zeh (Code Review)]

Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31840 )

Change subject: Documentation/soc/intel/fsp: Move mp_service_ppi document in corrct directory
......................................................................

Patch Set 2: Code-Review+1

(2 comments)

https://review.coreboot.org/#/c/31840/2/Documentation/soc/intel/fsp/ppi/mp_s... 
File Documentation/soc/intel/fsp/ppi/mp_service_ppi.md:

https://review.coreboot.org/#/c/31840/2/Documentation/soc/intel/fsp/ppi/mp_s... 
PS2, Line 24: CPU complexity and there might be some cases where certain feature programming mightbe
extra space missing.

https://review.coreboot.org/#/c/31840/2/Documentation/soc/intel/fsp/ppi/mp_s... 
PS2, Line 35: I
Should be lowercase as the sentence is not over yet.

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/31840 )

Change subject: Documentation/soc/intel/fsp: Move mp_service_ppi document in corrct directory
......................................................................

Documentation/soc/intel/fsp: Move mp_service_ppi document in corrct directory

This patch moves mp service ppi document from icelake/MultiProcesorInit.md
to ppi/mp_service_ppi.c.

Change-Id: I1bbaeb2644f219b5a1fda0c7c4b594184d53958c
Signed-off-by: Subrata Banik 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/31840
Reviewed-by: Werner Zeh 
Reviewed-by: Patrick Rudolph 
Reviewed-by: Philipp Deppenwiese 
Tested-by: build bot (Jenkins) 
---
R Documentation/soc/intel/fsp/ppi/coreboot_publish_mp_service_api.png
R Documentation/soc/intel/fsp/ppi/mp_service_ppi.md
M Documentation/soc/intel/fsp/ppi/ppi.md
M Documentation/soc/intel/icelake/index.md
4 files changed, 5 insertions(+), 4 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Philipp Deppenwiese: Looks good to me, approved
  Werner Zeh: Looks good to me, but someone else must approve
  Patrick Rudolph: Looks good to me, approved

diff --git a/Documentation/soc/intel/icelake/coreboot_publish_mp_service_api.png b/Documentation/soc/intel/fsp/ppi/coreboot_publish_mp_service_api.png
similarity index 100%
rename from Documentation/soc/intel/icelake/coreboot_publish_mp_service_api.png
rename to Documentation/soc/intel/fsp/ppi/coreboot_publish_mp_service_api.png
Binary files differ
diff --git a/Documentation/soc/intel/icelake/MultiProcessorInit.md b/Documentation/soc/intel/fsp/ppi/mp_service_ppi.md
similarity index 100%
rename from Documentation/soc/intel/icelake/MultiProcessorInit.md
rename to Documentation/soc/intel/fsp/ppi/mp_service_ppi.md
diff --git a/Documentation/soc/intel/fsp/ppi/ppi.md b/Documentation/soc/intel/fsp/ppi/ppi.md
index 66dbf07..6d7afb4 100644
--- a/Documentation/soc/intel/fsp/ppi/ppi.md
+++ b/Documentation/soc/intel/fsp/ppi/ppi.md
@@ -7,3 +7,8 @@
 able to execute the same in FSP's context.

* [What is PPI](https://www.intel.com/content/dam/www/public/us/en/documents/reference-guide...)
+
+## List of PPI service
+
+### Publish MP Service PPI from boot firmware (coreboot) to initialize CPU
+- [MP Service PPI](mp_service_ppi.md)
diff --git a/Documentation/soc/intel/icelake/index.md b/Documentation/soc/intel/icelake/index.md
index 450ab0f..71397d2 100644
--- a/Documentation/soc/intel/icelake/index.md
+++ b/Documentation/soc/intel/icelake/index.md
@@ -5,7 +5,3 @@
 ## Ice Lake coreboot development

- [Ice Lake coreboot development](iceLake_coreboot_development.md)
-
-## Multiprocessor Init
-
-- [Multiprocessor Init](MultiProcessorInit.md)