<p>Nico Huber has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20814">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">intel/sandybridge: Clean VGA BIOS ids up a little<br><br>Sync map_oprom_vendev() with the list of PCI ids in the `gma.c` driver,<br>remove one obsolete Kconfig default override.<br><br>Change-Id: I12f24f415b695c516fbb947114e09c873af2e439<br>Signed-off-by: Nico Huber <nico.h@gmx.de><br>---<br>M src/mainboard/lenovo/t530/Kconfig<br>M src/northbridge/intel/sandybridge/gma.c<br>2 files changed, 6 insertions(+), 8 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/14/20814/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/mainboard/lenovo/t530/Kconfig b/src/mainboard/lenovo/t530/Kconfig<br>index 84ebe86..065fd3c 100644<br>--- a/src/mainboard/lenovo/t530/Kconfig<br>+++ b/src/mainboard/lenovo/t530/Kconfig<br>@@ -52,10 +52,6 @@<br>  int<br>   default 10<br> <br>-config VGA_BIOS_ID<br>-   string<br>-       default "8086,0106"<br>-<br> config VGA_BIOS_FILE<br>       string<br>        default "pci8086,0106.rom"<br>diff --git a/src/northbridge/intel/sandybridge/gma.c b/src/northbridge/intel/sandybridge/gma.c<br>index 6d1044c..097df14 100644<br>--- a/src/northbridge/intel/sandybridge/gma.c<br>+++ b/src/northbridge/intel/sandybridge/gma.c<br>@@ -258,7 +258,7 @@<br> <br> u32 map_oprom_vendev(u32 vendev)<br> {<br>-     u32 new_vendev=vendev;<br>+       u32 new_vendev = vendev;<br> <br>   switch (vendev) {<br>     case 0x80860102:                /* GT1 Desktop */<br>@@ -267,9 +267,11 @@<br>       case 0x80860116:                /* GT2 Mobile */<br>      case 0x80860122:                /* GT2 Desktop >=1.3GHz */<br>         case 0x80860126:                /* GT2 Mobile >=1.3GHz */<br>- case 0x80860156:                /* IVB */<br>-    case 0x80860166:                /* IVB */<br>-            new_vendev=0x80860106;  /* GT1 Mobile */<br>+     case 0x80860152:                /* IVB Desktop */<br>+    case 0x80860156:                /* IVB Mobile */<br>+     case 0x80860162:                /* IVB Desktop */<br>+    case 0x80860166:                /* IVB Mobile */<br>+             new_vendev = 0x80860106;/* GT1 Mobile */<br>              break;<br>        }<br> <br></pre><p>To view, visit <a href="https://review.coreboot.org/20814">change 20814</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/20814"/><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: I12f24f415b695c516fbb947114e09c873af2e439 </div>
<div style="display:none"> Gerrit-Change-Number: 20814 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Nico Huber <nico.h@gmx.de> </div>