<p>Subrata Banik has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21290">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: Use SA library to get smm region information<br><br>This patch uses system agent common library to<br>know tseg region start and size. Unable to remove smm_region()<br>function from soc code as SMM common library<br>is not yet available for skylake use.<br><br>BRANCH=none<br>BUG=b:63974384<br>TEST=Build and boot eve successfully.<br><br>Change-Id: If98b65805753db2c30d6fea29e401a17cef39799<br>Signed-off-by: Subrata Banik <subrata.banik@intel.com><br>---<br>M src/soc/intel/skylake/memmap.c<br>1 file changed, 7 insertions(+), 23 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/90/21290/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/skylake/memmap.c b/src/soc/intel/skylake/memmap.c<br>index 1ccaf1d..229afa7 100644<br>--- a/src/soc/intel/skylake/memmap.c<br>+++ b/src/soc/intel/skylake/memmap.c<br>@@ -38,27 +38,10 @@<br>  return 8*MiB;<br> }<br> <br>-/* Returns base of requested region encoded in the system agent. */<br>-static inline uintptr_t system_agent_region_base(size_t reg)<br>-{<br>-        /* All regions concerned for have 1 MiB alignment. */<br>-        return ALIGN_DOWN(pci_read_config32(SA_DEV_ROOT, reg), 1*MiB);<br>-}<br>-<br>-static inline uintptr_t smm_region_start(void)<br>-{<br>-   return system_agent_region_base(TSEG);<br>-}<br>-<br>-static inline size_t smm_region_size(void)<br>-{<br>-       return system_agent_region_base(BGSM) - smm_region_start();<br>-}<br>-<br> void smm_region(void **start, size_t *size)<br> {<br>- *start = (void *)smm_region_start();<br>- *size = smm_region_size();<br>+   *start = (void *)sa_get_tseg_base();<br>+ *size = sa_get_tseg_size();<br> }<br> <br> /*<br>@@ -76,11 +59,12 @@<br> {<br>      uintptr_t sub_base;<br>   size_t sub_size;<br>+     void *smm_base;<br>       const size_t ied_size = CONFIG_IED_REGION_SIZE;<br>       const size_t cache_size = CONFIG_SMM_RESERVED_SIZE;<br> <br>-       sub_base = smm_region_start();<br>-       sub_size = smm_region_size();<br>+        smm_region(&smm_base, &sub_size);<br>+    sub_base = (uintptr_t)smm_base;<br> <br>    switch (sub) {<br>        case SMM_SUBREGION_HANDLER:<br>@@ -170,7 +154,7 @@<br>              dram_base -= sa_get_gsm_size();<br>       }<br>     /* Get TSEG size */<br>-  dram_base -= smm_region_size();<br>+      dram_base -= sa_get_tseg_size();<br> <br>   /* Get DPR size */<br>    if (IS_ENABLED(CONFIG_SA_ENABLE_DPR))<br>@@ -224,7 +208,7 @@<br>     * PRMMR_BASE MSR. The system hangs if PRMRR_BASE MSR is read before<br>   * PRMRR_MASK MSR lock bit is set.<br>     */<br>-  if (smm_region_start() == 0)<br>+ if (sa_get_tseg_base() == 0)<br>          return 0;<br> <br>  return calculate_dram_base();<br></pre><p>To view, visit <a href="https://review.coreboot.org/21290">change 21290</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/21290"/><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: If98b65805753db2c30d6fea29e401a17cef39799 </div>
<div style="display:none"> Gerrit-Change-Number: 21290 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Subrata Banik <subrata.banik@intel.com> </div>