<p>Subrata Banik has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/c/coreboot/+/30155">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/common: Add option to publish MP service PPI<br><br>This patch ensures to have required configuration option in<br>intel common cpu code to publish MP service PPI based on<br>devicetree.cb selection on supported platform.<br><br>TEST=Support platform should select publish_mp_service_ppi=1<br>to enable MP service PPI publish option.<br><br>Change-Id: Ie076c93429f3e647cfc9feb415b33fb27b2f1254<br>Signed-off-by: Subrata Banik <subrata.banik@intel.com><br>---<br>M src/soc/intel/common/block/chip/chip.c<br>M src/soc/intel/common/block/include/intelblocks/chip.h<br>2 files changed, 30 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/55/30155/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/soc/intel/common/block/chip/chip.c b/src/soc/intel/common/block/chip/chip.c</span><br><span>index aecf060..1514120 100644</span><br><span>--- a/src/soc/intel/common/block/chip/chip.c</span><br><span>+++ b/src/soc/intel/common/block/chip/chip.c</span><br><span>@@ -47,3 +47,18 @@</span><br><span> </span><br><span> return common_config->use_fsp_mp_init;</span><br><span> }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+/*</span><br><span style="color: hsl(120, 100%, 40%);">+ * This function will get MP service PPI config</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * Return values:</span><br><span style="color: hsl(120, 100%, 40%);">+ * 0 = Don't create PPI structure</span><br><span style="color: hsl(120, 100%, 40%);">+ * 1 = Create PPI structure</span><br><span style="color: hsl(120, 100%, 40%);">+ */</span><br><span style="color: hsl(120, 100%, 40%);">+int chip_get_mp_service_ppi_config(void)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+ const struct soc_intel_common_config *common_config;</span><br><span style="color: hsl(120, 100%, 40%);">+ common_config = chip_get_common_soc_structure();</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ return common_config->publish_mp_service_ppi;</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span>diff --git a/src/soc/intel/common/block/include/intelblocks/chip.h b/src/soc/intel/common/block/include/intelblocks/chip.h</span><br><span>index d761f6b..fe6d8d7 100644</span><br><span>--- a/src/soc/intel/common/block/include/intelblocks/chip.h</span><br><span>+++ b/src/soc/intel/common/block/include/intelblocks/chip.h</span><br><span>@@ -39,6 +39,12 @@</span><br><span> * 1 = Make use of FSP MP Init</span><br><span> */</span><br><span> uint8_t use_fsp_mp_init;</span><br><span style="color: hsl(120, 100%, 40%);">+ /*</span><br><span style="color: hsl(120, 100%, 40%);">+ * Create MP service PPI</span><br><span style="color: hsl(120, 100%, 40%);">+ * 0 = Don't create PPI structure</span><br><span style="color: hsl(120, 100%, 40%);">+ * 1 = Create PPI structure</span><br><span style="color: hsl(120, 100%, 40%);">+ */</span><br><span style="color: hsl(120, 100%, 40%);">+ uint8_t publish_mp_service_ppi;</span><br><span> };</span><br><span> </span><br><span> /* This function to retrieve soc config structure required by common code */</span><br><span>@@ -53,4 +59,13 @@</span><br><span> */</span><br><span> int chip_get_fsp_mp_init(void);</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/*</span><br><span style="color: hsl(120, 100%, 40%);">+ * This function will get MP service PPI config</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * Return values:</span><br><span style="color: hsl(120, 100%, 40%);">+ * 0 = Don't create PPI structure</span><br><span style="color: hsl(120, 100%, 40%);">+ * 1 = Create PPI structure</span><br><span style="color: hsl(120, 100%, 40%);">+ */</span><br><span style="color: hsl(120, 100%, 40%);">+int chip_get_mp_service_ppi_config(void);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> #endif /* SOC_INTEL_COMMON_BLOCK_CHIP_H */</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/30155">change 30155</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/c/coreboot/+/30155"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ie076c93429f3e647cfc9feb415b33fb27b2f1254 </div>
<div style="display:none"> Gerrit-Change-Number: 30155 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Subrata Banik <subrata.banik@intel.com> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>