[coreboot-gerrit] Change in ...coreboot[master]: cpu/amd/quadcore/quadcore.c: Remove not used variable

Martin Roth (Code Review) gerrit at coreboot.org
Thu Nov 29 20:54:17 CET 2018


Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29941 )

Change subject: cpu/amd/quadcore/quadcore.c: Remove not used variable
......................................................................


Patch Set 1:

Would it be possible to group these into reasonable sized patches - maybe 15 or 20 changes in a patch?  Say do all of the ones in the src/cpu/amd or src/cpu subdirectory instead of doing them one-by-one?  If that's not needed, and there are really only a few issues, I'm fine with them being done one-off, but I'm doubtful that that's the case.


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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I826ba637c77dcdac04d0494c00b1b90e5303c63e
Gerrit-Change-Number: 29941
Gerrit-PatchSet: 1
Gerrit-Owner: HAOUAS Elyes <ehaouas at noos.fr>
Gerrit-CC: Martin Roth <martinroth at google.com>
Gerrit-Comment-Date: Thu, 29 Nov 2018 19:54:17 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181129/c3d667ec/attachment.html>


More information about the coreboot-gerrit mailing list