<p>Furquan Shaikh <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://review.coreboot.org/26720">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">util/sconfig: Re-factor sconfig to not assume chip as device<br><br>This change adds a new structure "struct chip" to identify elements of<br>type chip rather than re-using the structure for device. Until now<br>chip was treated as a device while generating the parse tree and then<br>device tree postprocessing skipped over all the chip entries in<br>children and sibling pointers of device nodes.<br><br>With this change, the device tree will only contain struct device in<br>the parsed tree. It helps by avoiding unnecessary pointers to chip<br>structure as children or next_sibling and then skipping those elements<br>in post processing. Every device can then hold a pointer to its chip.<br><br>When generating static.c, chip structure is emitted before device<br>structure to ensure that the device structure has chip within its<br>scope. Externally, the only visible change in static.c should be the<br>order in which chip/device elements are emitted i.e. previously all<br>chips under a particular device were emitted to static.c and then the<br>devices using those chips. Now, a chip is emitted just before device<br>that is using it.<br><br>BUG=b:80081934<br>TEST=Verified that abuild is successful for all boards. Also, verified<br>that static.c generated for eve, kahlee, scarlet, asrock imb_a180 is<br>unchanged from before in node definitions.<br><br>Change-Id: I255092f527c8eecb144385eb681df20e54caf8f5<br>Signed-off-by: Furquan Shaikh <furquan@google.com><br>---<br>M util/sconfig/main.c<br>M util/sconfig/sconfig.h<br>M util/sconfig/sconfig.tab.c_shipped<br>M util/sconfig/sconfig.tab.h_shipped<br>M util/sconfig/sconfig.y<br>5 files changed, 183 insertions(+), 111 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/20/26720/3</pre><p>To view, visit <a href="https://review.coreboot.org/26720">change 26720</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/26720"/><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: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I255092f527c8eecb144385eb681df20e54caf8f5 </div>
<div style="display:none"> Gerrit-Change-Number: 26720 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Furquan Shaikh <furquan@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Furquan Shaikh <furquan@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>