<p>Martin Roth has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/27535">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">arch/riscv: Fix makefile to only set flags for riscv<br><br>This was updating flags for ALL architectures, not just riscv.<br>That was bad.<br><br>Change-Id: Ifa12ad98b04a5ac36148609ccdf46ca427fc5a27<br>Signed-off-by: Martin Roth <martin@coreboot.org><br>---<br>M src/arch/riscv/Makefile.inc<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/35/27535/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/arch/riscv/Makefile.inc b/src/arch/riscv/Makefile.inc</span><br><span>index 90e4240..a797076 100644</span><br><span>--- a/src/arch/riscv/Makefile.inc</span><br><span>+++ b/src/arch/riscv/Makefile.inc</span><br><span>@@ -21,7 +21,6 @@</span><br><span> </span><br><span> ifeq ($(CONFIG_ARCH_RAMSTAGE_RISCV),y)</span><br><span> check-ramstage-overlap-regions += stack</span><br><span style="color: hsl(0, 100%, 40%);">-endif</span><br><span> </span><br><span> riscv_flags = -I$(src)/arch/riscv/ -mcmodel=$(CONFIG_RISCV_CODEMODEL) -march=$(CONFIG_RISCV_ARCH) -mabi=$(CONFIG_RISCV_ABI)</span><br><span> </span><br><span>@@ -32,6 +31,7 @@</span><br><span> COMPILER_RT_romstage  = $(shell  $(GCC_romstage) $(riscv_flags) -print-libgcc-file-name)</span><br><span> </span><br><span> COMPILER_RT_ramstage  = $(shell  $(GCC_ramstage) $(riscv_flags) -print-libgcc-file-name)</span><br><span style="color: hsl(120, 100%, 40%);">+endif</span><br><span> </span><br><span> ################################################################################</span><br><span> ## bootblock</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/27535">change 27535</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/27535"/><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: Ifa12ad98b04a5ac36148609ccdf46ca427fc5a27 </div>
<div style="display:none"> Gerrit-Change-Number: 27535 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>