<p><a href="https://review.coreboot.org/29345">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/29345/1//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/29345/1//COMMIT_MSG@24">Patch Set #1, Line 24:</a> <code style="font-family:monospace,monospace">the empty resource</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">The particular empty resource was filled later:</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">It's NOT filled in later.  Look at a current boot log. You'll see these that you're talking about:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> Show resources in subtree (Root Device)...After assigning values.<br>...<br>  DOMAIN: 0000 resource base 1000 size 100 align 8 gran 0 limit ffff flags 40040100 index 10000000<br>  DOMAIN: 0000 resource base f0000000 size 4d8a100 align 26 gran 0 limit f7ffffff flags 40040200 index 0<br>...<br>  PCI: 00:18.0 resource base 0 size 0 align 0 gran 0 limit 0 flags 1 index 1080</pre><p style="white-space: pre-wrap; word-wrap: break-word;">That last entry is what I'm talking about.  Note that the index between these is different, so they're not the same resource node.</p><p style="white-space: pre-wrap; word-wrap: break-word;">>Have you check if it's still being created? Or if not being created, does it affects the boot process anyway (it might be created and never used)?</p><p style="white-space: pre-wrap; word-wrap: break-word;"><br>No, it's not being created any more, that's the point of this patch.  Since it wasn't being filled in, it wasn't being used, and it should be deleted.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, I've tested booting.  That's how I verified that only the empty resource changed.</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/29345">change 29345</a>. To unsubscribe, or for help writing mail filters, 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/29345"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: I83bd3ea8db141416632c12fc883386070363f2f1 </div>
<div style="display:none"> Gerrit-Change-Number: 29345 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Daniel Kurtz <djkurtz@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Marc Jones <marc@marcjonesconsulting.com> </div>
<div style="display:none"> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Raul Rangel <rrangel@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Spiegel <richard.spiegel@silverbackltd.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 30 Oct 2018 15:29:31 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>