<p>Arthur Heymans has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21448">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">drivers/intel/gma/vbt.c: Add GENERAL DEFINITIONS block to fake vbt<br><br>This is needed for Linux (at least 4.12) to know on which DDC gmbus to<br>probe for VGA edid.<br><br>TESTED on dg43gt: Use NGI with a payload like grub that doesn't<br>override VBT (seavgabios) with a DVI-I to VGA adapter and observe that<br>Linux can properly modeset the display.<br><br>Change-Id: I5bce00d2e37d77c27cf0022c7dab3444c3699dcf<br>Signed-off-by: Arthur Heymans <arthur@aheymans.xyz><br>---<br>M src/drivers/intel/gma/vbt.c<br>1 file changed, 15 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/48/21448/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/drivers/intel/gma/vbt.c b/src/drivers/intel/gma/vbt.c<br>index c908eb4..ffb56e9 100644<br>--- a/src/drivers/intel/gma/vbt.c<br>+++ b/src/drivers/intel/gma/vbt.c<br>@@ -69,6 +69,21 @@<br> <br>    ptr += sizeof (*genfeat);<br> <br>+ *ptr++ = BDB_GENERAL_DEFINITIONS;<br>+    *ptr++ = sizeof(struct bdb_general_definitions);<br>+     *ptr++ = sizeof(struct bdb_general_definitions) >> 8;<br>+  struct bdb_general_definitions *const gendef =<br>+               (struct bdb_general_definitions *)ptr;<br>+       if (IS_ENABLED(CONFIG_GFX_GMA_ANALOG_I2C_HDMI_B))<br>+            gendef->crt_ddc_gmbus_pin = GMBUS_PORT_DPB;<br>+       else if (IS_ENABLED(CONFIG_GFX_GMA_ANALOG_I2C_HDMI_C))<br>+               gendef->crt_ddc_gmbus_pin = GMBUS_PORT_DPC;<br>+       else if (IS_ENABLED(CONFIG_GFX_GMA_ANALOG_I2C_HDMI_D))<br>+               gendef->crt_ddc_gmbus_pin = GMBUS_PORT_DPD;<br>+       else<br>+         gendef->crt_ddc_gmbus_pin = GMBUS_PORT_VGADDC;<br>+    ptr += sizeof(*gendef);<br>+<br>    bdb_head->bdb_size = ptr - (u8 *)bdb_head;<br>         head->vbt_size = ptr - (u8 *)head;<br>         head->vbt_checksum = 0;<br></pre><p>To view, visit <a href="https://review.coreboot.org/21448">change 21448</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/21448"/><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: I5bce00d2e37d77c27cf0022c7dab3444c3699dcf </div>
<div style="display:none"> Gerrit-Change-Number: 21448 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Arthur Heymans <arthur@aheymans.xyz> </div>