<p>Subrata Banik has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20830">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/apollolake: 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: Ied677e8c77fa7b166b016da458caad0e4702b5d8<br>Signed-off-by: Subrata Banik <subrata.banik@intel.com><br>---<br>M src/soc/intel/apollolake/graphics.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/30/20830/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/apollolake/graphics.c b/src/soc/intel/apollolake/graphics.c<br>index c873825..90859d5 100644<br>--- a/src/soc/intel/apollolake/graphics.c<br>+++ b/src/soc/intel/apollolake/graphics.c<br>@@ -27,7 +27,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> static void igd_set_resources(struct device *dev)<br></pre><p>To view, visit <a href="https://review.coreboot.org/20830">change 20830</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/20830"/><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: Ied677e8c77fa7b166b016da458caad0e4702b5d8 </div>
<div style="display:none"> Gerrit-Change-Number: 20830 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Subrata Banik <subrata.banik@intel.com> </div>