[coreboot-gerrit] Change in coreboot[master]: util/sconfig: Re-factor emitting of headers to static.c

Furquan Shaikh (Code Review) gerrit at coreboot.org
Thu May 31 08:51:17 CEST 2018


Furquan Shaikh has uploaded a new patch set (#2). ( https://review.coreboot.org/26726 )

Change subject: util/sconfig: Re-factor emitting of headers to static.c
......................................................................

util/sconfig: Re-factor emitting of headers to static.c

This change removes call to add_header from parsing functions and
moves it to a local function within main.c. It also adds a new
function emit_headers that is responsible for creating the linked list
for chip headers and emitting those to static.c

BUG=b:80081934
TEST=Verified that static.c for all files compiled using abuild is the
same with and without this change.

Change-Id: I24d526e81323115d3cc927242a4b9e49414afbe0
Signed-off-by: Furquan Shaikh <furquan at google.com>
---
M util/sconfig/main.c
M util/sconfig/sconfig.h
M util/sconfig/sconfig.tab.c_shipped
M util/sconfig/sconfig.y
4 files changed, 61 insertions(+), 56 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/26/26726/2
-- 
To view, visit https://review.coreboot.org/26726
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I24d526e81323115d3cc927242a4b9e49414afbe0
Gerrit-Change-Number: 26726
Gerrit-PatchSet: 2
Gerrit-Owner: Furquan Shaikh <furquan at google.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180531/66a9e70f/attachment.html>


More information about the coreboot-gerrit mailing list