<p>Matt DeVillier has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22118">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">drv/intel/gma/opregion: fix opregion version<br><br>Per Intel spec, the OpRegion struture version field is 32 bits,<br>broken down into two 16-bit components: version and revision.<br>The version component occupies the upper half of the field,<br>so left-shift the value appropriately; the revision value is<br>zero and so does not need to be set.<br><br>Although the Linux i915 driver doesn't care if the version is<br>shifted correctly or not, under Windows the Intel drivers won't<br>load unless both the offset and value are correct (at least<br>when booted in UEFI mode without the use of a VBIOS).<br><br>Additionaly, correct the raw / unshifted version value to match<br>Intel's Tianocore reference code.<br><br>TEST: Boot Windows [8.1,10] in UEFI mode w/GOP graphics init,<br>observe Intel GPU drivers load correctly / have full capability.<br><br>Change-Id: Ic2903ee4829689ec4117aec93dce0b87cec6f313<br>Signed-off-by: Matt DeVillier <matt.devillier@gmail.com><br>---<br>M src/drivers/intel/gma/opregion.c<br>M src/drivers/intel/gma/opregion.h<br>2 files changed, 2 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/18/22118/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/drivers/intel/gma/opregion.c b/src/drivers/intel/gma/opregion.c<br>index 84bca9b..46031e3 100644<br>--- a/src/drivers/intel/gma/opregion.c<br>+++ b/src/drivers/intel/gma/opregion.c<br>@@ -260,7 +260,7 @@<br> <br>        /* 8kb */<br>     opregion->header.size = sizeof(igd_opregion_t) / 1024;<br>-    opregion->header.version = IGD_OPREGION_VERSION;<br>+  opregion->header.version = (IGD_OPREGION_VERSION << 16);<br> <br>  // FIXME We just assume we're mobile for now<br>      opregion->header.mailboxes = MAILBOXES_MOBILE;<br>diff --git a/src/drivers/intel/gma/opregion.h b/src/drivers/intel/gma/opregion.h<br>index 3ae68e5..5d299ef 100644<br>--- a/src/drivers/intel/gma/opregion.h<br>+++ b/src/drivers/intel/gma/opregion.h<br>@@ -44,7 +44,7 @@<br> } __packed opregion_header_t;<br> <br> #define IGD_OPREGION_SIGNATURE "IntelGraphicsMem"<br>-#define IGD_OPREGION_VERSION  2<br>+#define IGD_OPREGION_VERSION  0x0200<br> <br> #define IGD_MBOX1  (1 << 0)<br> #define IGD_MBOX2      (1 << 1)<br></pre><p>To view, visit <a href="https://review.coreboot.org/22118">change 22118</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/22118"/><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: Ic2903ee4829689ec4117aec93dce0b87cec6f313 </div>
<div style="display:none"> Gerrit-Change-Number: 22118 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Matt DeVillier <matt.devillier@gmail.com> </div>