<p>Patrick Georgi has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21723">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">drivers/intel/fsp2_0: use common code to fetch vbt.bin<br><br>No need for having two of everything in the coreboot codebase.<br><br>Change-Id: Ie1cdd1783dd5dababd1e97436a4ce1a4f068d5b3<br>Signed-off-by: Patrick Georgi <pgeorgi@google.com><br>---<br>M src/drivers/intel/fsp2_0/graphics.c<br>1 file changed, 8 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/23/21723/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/drivers/intel/fsp2_0/graphics.c b/src/drivers/intel/fsp2_0/graphics.c<br>index 3207e1d..84f2d6d 100644<br>--- a/src/drivers/intel/fsp2_0/graphics.c<br>+++ b/src/drivers/intel/fsp2_0/graphics.c<br>@@ -14,6 +14,7 @@<br> #include <cbfs.h><br> #include <console/console.h><br> #include <fsp/util.h><br>+#include <soc/intel/common/vbt.h><br> <br> enum pixel_format {<br>   pixel_rgbx_8bpc = 0,<br>@@ -95,13 +96,16 @@<br> <br> uintptr_t fsp_load_vbt(void)<br> {<br>-      void *vbt;<br>+   struct region_device rdev;<br>+   void *vbt_data = NULL;<br> <br>-    vbt = cbfs_boot_map_with_leak("vbt.bin", CBFS_TYPE_RAW, NULL);<br>-     if (!vbt)<br>+    if (locate_vbt(&rdev) != CB_ERR)<br>+         vbt_data = rdev_mmap_full(&rdev);<br>+<br>+     if (vbt_data == NULL)<br>                 printk(BIOS_NOTICE, "Could not locate a VBT file in CBFS\n");<br> <br>-   return (uintptr_t)vbt;<br>+       return (uintptr_t)vbt_data;<br> }<br> <br> int fill_lb_framebuffer(struct lb_framebuffer *framebuffer)<br></pre><p>To view, visit <a href="https://review.coreboot.org/21723">change 21723</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/21723"/><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: Ie1cdd1783dd5dababd1e97436a4ce1a4f068d5b3 </div>
<div style="display:none"> Gerrit-Change-Number: 21723 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Patrick Georgi <pgeorgi@google.com> </div>