<p>Elyes HAOUAS has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/28784">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">cpu/intel/microcode: Replace numbers with macros<br><br>Change-Id: I849dd406f5ccc733d4957eaf1c774745782f531a<br>Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr><br>---<br>M src/cpu/intel/microcode/microcode.c<br>1 file changed, 4 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/84/28784/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/cpu/intel/microcode/microcode.c b/src/cpu/intel/microcode/microcode.c</span><br><span>index 8cfcb95..0dd9a5b 100644</span><br><span>--- a/src/cpu/intel/microcode/microcode.c</span><br><span>+++ b/src/cpu/intel/microcode/microcode.c</span><br><span>@@ -103,7 +103,7 @@</span><br><span> </span><br><span>   msr.lo = (unsigned long)m + sizeof(struct microcode);</span><br><span>        msr.hi = 0;</span><br><span style="color: hsl(0, 100%, 40%);">-     wrmsr(0x79, msr);</span><br><span style="color: hsl(120, 100%, 40%);">+     wrmsr(IA32_BIOS_UPDT_TRIG, msr);</span><br><span> </span><br><span> #if !defined(__ROMCC__)</span><br><span>      printk(BIOS_DEBUG, "microcode: updated to revision "</span><br><span>@@ -162,9 +162,9 @@</span><br><span>         /* CPUID sets MSR 0x8B iff a microcode update has been loaded. */</span><br><span>    msr.lo = 0;</span><br><span>  msr.hi = 0;</span><br><span style="color: hsl(0, 100%, 40%);">-     wrmsr(0x8B, msr);</span><br><span style="color: hsl(120, 100%, 40%);">+     wrmsr(IA32_BIOS_SIGN_ID, msr);</span><br><span>       eax = cpuid_eax(1);</span><br><span style="color: hsl(0, 100%, 40%);">-     msr = rdmsr(0x8B);</span><br><span style="color: hsl(120, 100%, 40%);">+    msr = rdmsr(IA32_BIOS_SIGN_ID);</span><br><span>      rev = msr.hi;</span><br><span>        x86_model = (eax >> 4) & 0x0f;</span><br><span>     x86_family = (eax >> 8) & 0x0f;</span><br><span>@@ -172,7 +172,7 @@</span><br><span> </span><br><span>  pf = 0;</span><br><span>      if ((x86_model >= 5) || (x86_family > 6)) {</span><br><span style="color: hsl(0, 100%, 40%);">-               msr = rdmsr(0x17);</span><br><span style="color: hsl(120, 100%, 40%);">+            msr = rdmsr(IA32_PLATFORM_ID);</span><br><span>               pf = 1 << ((msr.hi >> 18) & 7);</span><br><span>      }</span><br><span> #if !defined(__ROMCC__)</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/28784">change 28784</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/28784"/><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: I849dd406f5ccc733d4957eaf1c774745782f531a </div>
<div style="display:none"> Gerrit-Change-Number: 28784 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Elyes HAOUAS <ehaouas@noos.fr> </div>