<p>Furquan Shaikh <strong>uploaded patch set #2</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. It helps<br>clean up the devicetree by avoiding unnecessary pointers to chip<br>structure as children or next_sibling and then skipping those elements<br>in post processing.<br><br>With this change, the devicetree will only contain struct device in<br>the tree. Every device can then hold a pointer to its chip. When<br>generating static.c, chip structure is emitted before device structure<br>to ensure that the device structure has chip within its scope.<br><br>Externally, the only visible change in static.c should be the order in<br>which chip elements are emitted i.e. previously all chips under a<br>particular device were emitted to static.c and then the devices using<br>those chips. Now, a chip is emitted just before device that is using<br>it.<br><br>This clean up is part of a bigger change required to support overriding<br>of devicetree by variants.<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 is unchanged from before.<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, 189 insertions(+), 117 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/2</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: 2 </div>
<div style="display:none"> Gerrit-Owner: Furquan Shaikh <furquan@google.com> </div>