<p>Subrata Banik has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20829">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: Skip disabled IGD device<br><br>If IGD PCI device is disabled:<br>1. BAR for the device will be 0.<br>2. There is no need to allocate framebuffer for this device.<br><br>Some early SOCs don't have GFX model fuse by default hence<br>we need to add a check to ensure PCI device is enable. This<br>code to avoid die inside coreboot for missing resources.<br><br>Change-Id: Ic31d3e57ba730f6b569bf2cc3bdc54cb369b8caf<br>Signed-off-by: Subrata Banik <subrata.banik@intel.com><br>---<br>M src/soc/intel/skylake/igd.c<br>1 file changed, 7 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/20829/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/skylake/igd.c b/src/soc/intel/skylake/igd.c<br>index dbcbd4d..62bcf42 100644<br>--- a/src/soc/intel/skylake/igd.c<br>+++ b/src/soc/intel/skylake/igd.c<br>@@ -36,7 +36,13 @@<br> <br> uintptr_t fsp_soc_get_igd_bar(void)<br> {<br>-    return find_resource(SA_DEV_IGD, PCI_BASE_ADDRESS_2)->base;<br>+       device_t dev = SA_DEV_IGD;<br>+<br>+        /* Check if IGD PCI device is disabled */<br>+    if (!dev->enabled)<br>+                return 0;<br>+<br>+ return find_resource(dev, PCI_BASE_ADDRESS_2)->base;<br> }<br> <br> u32 map_oprom_vendev(u32 vendev)<br></pre><p>To view, visit <a href="https://review.coreboot.org/20829">change 20829</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/20829"/><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: Ic31d3e57ba730f6b569bf2cc3bdc54cb369b8caf </div>
<div style="display:none"> Gerrit-Change-Number: 20829 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Subrata Banik <subrata.banik@intel.com> </div>