<p>Keith Hui has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21347">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">cpu/intel/car/romstage.c: Drop superfluous code<br><br>Both functions are never called under their intended configurations.<br><br>The setup_stack_and_mtrrs() here guarded by LATE_CBMEM_INIT may even<br>get in the way of versions implemented elsewhere as boards move to<br>EARLY_CBMEM_INIT.<br><br>Boot tested on asus/p2b-ls with [1] applied. It is therefore<br>strongly recommended that it be merged before this.<br><br>[1] https://review.coreboot.org/c/21313/<br><br>Change-Id: I8ac0f4c2a4642712df062f8b9001d912c7612d9a<br>Signed-off-by: Keith Hui <buurin@gmail.com><br>---<br>M src/cpu/intel/car/romstage.c<br>1 file changed, 0 insertions(+), 29 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/21347/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/cpu/intel/car/romstage.c b/src/cpu/intel/car/romstage.c<br>index 1477612..828b171 100644<br>--- a/src/cpu/intel/car/romstage.c<br>+++ b/src/cpu/intel/car/romstage.c<br>@@ -46,32 +46,3 @@<br> <br>      return romstage_stack_after_car;<br> }<br>-<br>-asmlinkage void romstage_after_car(void)<br>-{<br>-       /* Load the ramstage. */<br>-     run_ramstage();<br>-}<br>-<br>-#if IS_ENABLED(CONFIG_LATE_CBMEM_INIT)<br>-/* setup_stack_and_mtrrs() determines the stack to use after<br>- * cache-as-ram is torn down as well as the MTRR settings to use. */<br>-void *setup_stack_and_mtrrs(void)<br>-{<br>-        struct postcar_frame pcf;<br>-<br>- postcar_frame_init_lowmem(&pcf);<br>-<br>-      /* Cache the ROM as WP just below 4GiB. */<br>-   postcar_frame_add_mtrr(&pcf, -CACHE_ROM_SIZE, CACHE_ROM_SIZE,<br>-            MTRR_TYPE_WRPROT);<br>-<br>-        /* Cache RAM as WB from 0 -> CACHE_TMP_RAMTOP. */<br>- postcar_frame_add_mtrr(&pcf, 0, CACHE_TMP_RAMTOP, MTRR_TYPE_WRBACK);<br>-<br>-  /* Save the number of MTRRs to setup. Return the stack location<br>-       * pointing to the number of MTRRs.<br>-   */<br>-  return postcar_commit_mtrrs(&pcf);<br>-}<br>-#endif /* CONFIG_LATE_CBMEM_INIT */<br></pre><p>To view, visit <a href="https://review.coreboot.org/21347">change 21347</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/21347"/><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: I8ac0f4c2a4642712df062f8b9001d912c7612d9a </div>
<div style="display:none"> Gerrit-Change-Number: 21347 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Keith Hui <buurin@gmail.com> </div>