<p>Subrata Banik has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21267">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/common: Add functions into common system agent library<br><br>This patch to add helper function to get tseg memory base and<br>size for HW based memory layout design.<br><br>BRANCH=none<br>BUG=b:63974384<br>TEST=Build and boot eve successfully.<br><br>Change-Id: I4c8b79f047e3dc6b2deb17fdb745f004004526b6<br>Signed-off-by: Subrata Banik <subrata.banik@intel.com><br>---<br>M src/soc/intel/common/block/include/intelblocks/systemagent.h<br>M src/soc/intel/common/block/systemagent/systemagent_early.c<br>2 files changed, 23 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/67/21267/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/common/block/include/intelblocks/systemagent.h b/src/soc/intel/common/block/include/intelblocks/systemagent.h<br>index 5f04e36..4ca19e6 100644<br>--- a/src/soc/intel/common/block/include/intelblocks/systemagent.h<br>+++ b/src/soc/intel/common/block/include/intelblocks/systemagent.h<br>@@ -74,11 +74,15 @@<br> /* API to enable Power Aware Interrupt Routing through MCHBAR */<br> void enable_power_aware_intr(void);<br> /* API to get TOLUD base address */<br>-uint32_t sa_get_tolud_base(void);<br>+uintptr_t sa_get_tolud_base(void);<br> /* API to get DSM size */<br> size_t sa_get_dsm_size(void);<br> /* API to get GSM size */<br> size_t sa_get_gsm_size(void);<br>+/* API to get TSEG base address */<br>+uintptr_t sa_get_tseg_base(void);<br>+/* API to get TSEG size */<br>+size_t sa_get_tseg_size(void);<br> /* API to get DPR size */<br> size_t sa_get_dpr_size(void);<br> /*<br>diff --git a/src/soc/intel/common/block/systemagent/systemagent_early.c b/src/soc/intel/common/block/systemagent/systemagent_early.c<br>index 855d953..4abc15f 100644<br>--- a/src/soc/intel/common/block/systemagent/systemagent_early.c<br>+++ b/src/soc/intel/common/block/systemagent/systemagent_early.c<br>@@ -133,7 +133,7 @@<br>   MCHBAR8(BIOS_RESET_CPL) = bios_reset_cpl;<br> }<br> <br>-uint32_t sa_get_tolud_base(void)<br>+uintptr_t sa_get_tolud_base(void)<br> {<br>   /* All regions concerned for have 1 MiB alignment. */<br>         return ALIGN_DOWN(pci_read_config32(SA_DEV_ROOT, TOLUD), 1*MiB);<br>@@ -147,6 +147,12 @@<br> size_t sa_get_dsm_size(void)<br> {<br>     return (((sa_get_ggc_reg() & G_GMS_MASK) >> G_GMS_OFFSET) * 32*MiB);<br>+}<br>+<br>+static uintptr_t sa_get_gsm_base(void)<br>+{<br>+   /* All regions concerned for have 1 MiB alignment. */<br>+        return ALIGN_DOWN(pci_read_config32(SA_DEV_ROOT, BGSM), 1*MiB);<br> }<br> <br> size_t sa_get_gsm_size(void)<br>@@ -165,6 +171,17 @@<br>           return 0;<br> }<br> <br>+uintptr_t sa_get_tseg_base(void)<br>+{<br>+      /* All regions concerned for have 1 MiB alignment. */<br>+        return ALIGN_DOWN(pci_read_config32(SA_DEV_ROOT, TSEG), 1*MiB);<br>+}<br>+<br>+size_t sa_get_tseg_size(void)<br>+{<br>+   return sa_get_gsm_base() - sa_get_tseg_base();<br>+}<br>+<br> /*<br>  * Get DPR size in case CONFIG_SA_ENABLE_DPR is selected by SoC.<br>  */<br></pre><p>To view, visit <a href="https://review.coreboot.org/21267">change 21267</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/21267"/><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: I4c8b79f047e3dc6b2deb17fdb745f004004526b6 </div>
<div style="display:none"> Gerrit-Change-Number: 21267 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Subrata Banik <subrata.banik@intel.com> </div>