[coreboot-gerrit] Change in ...coreboot[master]: qclib: Add qclib support

Patrick Georgi (Code Review) gerrit at coreboot.org
Fri Dec 7 19:16:38 CET 2018


Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29967 )

Change subject: qclib: Add qclib support
......................................................................


Patch Set 2:

(1 comment)

lots of parts are commented out. please sort that out before the next iteration.
Also, we indent code by tabs (as the build bot helpfully pointed out), so please account for that.

https://review.coreboot.org/#/c/29967/2/src/mainboard/google/mistral/mainboard.c 
File src/mainboard/google/mistral/mainboard.c:

https://review.coreboot.org/#/c/29967/2/src/mainboard/google/mistral/mainboard.c@21 
PS2, Line 21: #if 0
what's this for? if it's not needed, kick it out entirely.



-- 
To view, visit https://review.coreboot.org/c/coreboot/+/29967
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I534af71163d034ea04420dda6a94ce31b08c8a07
Gerrit-Change-Number: 29967
Gerrit-PatchSet: 2
Gerrit-Owner: nsekar at codeaurora.org
Gerrit-Reviewer: Julius Werner <jwerner at chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth at google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi at google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-Reviewer: nsekar at codeaurora.org
Gerrit-Comment-Date: Fri, 07 Dec 2018 18:16:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181207/2fa1d73c/attachment.html>


More information about the coreboot-gerrit mailing list