<p>Subrata Banik has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21268">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/common: Add functions into common SMM library<br><br>This patch to add helper function to get SMM region start<br>and size based on systemagent common library.<br><br>BRANCH=none<br>BUG=b:63974384<br>TEST=Build and boot eve successfully.<br><br>Change-Id: If10af4a3f6a5bd22db5a03bcd3033a01b1cce0b4<br>Signed-off-by: Subrata Banik <subrata.banik@intel.com><br>---<br>M src/soc/intel/common/block/include/intelblocks/smm.h<br>M src/soc/intel/common/block/smm/Makefile.inc<br>M src/soc/intel/common/block/smm/smm.c<br>3 files changed, 12 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/68/21268/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/common/block/include/intelblocks/smm.h b/src/soc/intel/common/block/include/intelblocks/smm.h<br>index 09378b9..423c8a8 100644<br>--- a/src/soc/intel/common/block/include/intelblocks/smm.h<br>+++ b/src/soc/intel/common/block/include/intelblocks/smm.h<br>@@ -30,5 +30,7 @@<br>  */<br> void smm_southbridge_clear_state(void);<br> void smm_southbridge_enable(void);<br>+/* API to get SMM region start and size based on Host Bridge register */<br>+void smm_region(void **start, size_t *size);<br> <br> #endif<br>diff --git a/src/soc/intel/common/block/smm/Makefile.inc b/src/soc/intel/common/block/smm/Makefile.inc<br>index e505625..82a3191 100644<br>--- a/src/soc/intel/common/block/smm/Makefile.inc<br>+++ b/src/soc/intel/common/block/smm/Makefile.inc<br>@@ -1,3 +1,6 @@<br>+bootblock-$(CONFIG_SOC_INTEL_COMMON_BLOCK_SMM) += smm.c<br>+romstage-$(CONFIG_SOC_INTEL_COMMON_BLOCK_SMM) += smm.c<br>+postcar-$(CONFIG_SOC_INTEL_COMMON_BLOCK_SMM) += smm.c<br> ramstage-$(CONFIG_SOC_INTEL_COMMON_BLOCK_SMM) += smm.c<br> smm-$(CONFIG_SOC_INTEL_COMMON_BLOCK_SMM) += smihandler.c<br> smm-$(CONFIG_SOC_INTEL_COMMON_BLOCK_SMM_IO_TRAP) += smitraphandler.c<br>diff --git a/src/soc/intel/common/block/smm/smm.c b/src/soc/intel/common/block/smm/smm.c<br>index 41f3426..5a7f216 100644<br>--- a/src/soc/intel/common/block/smm/smm.c<br>+++ b/src/soc/intel/common/block/smm/smm.c<br>@@ -19,6 +19,7 @@<br> #include <cpu/x86/smm.h><br> #include <intelblocks/pmclib.h><br> #include <intelblocks/smm.h><br>+#include <intelblocks/systemagent.h><br> #include <soc/pm.h><br> <br> void smm_southbridge_clear_state(void)<br>@@ -77,3 +78,9 @@<br>                  "d" (APM_CNT)<br>     );<br> }<br>+<br>+void smm_region(void **start, size_t *size)<br>+{<br>+  *start = (void *)sa_get_tseg_base();<br>+ *size = sa_get_tseg_size();<br>+}<br></pre><p>To view, visit <a href="https://review.coreboot.org/21268">change 21268</a>. To unsubscribe, 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/21268"/><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-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: If10af4a3f6a5bd22db5a03bcd3033a01b1cce0b4 </div>
<div style="display:none"> Gerrit-Change-Number: 21268 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Subrata Banik <subrata.banik@intel.com> </div>