The newest checkpatch version now flags code such as
int i;
for (i = 0; ...
because there is no blank line between a declaration and the next
statement. We have always allowed variable declarations inline with the
code in coreboot and libpayload, so this doesn't make much sense for
us. Add it to the ignore list.
BRANCH=None
BUG=None
TEST=None
Change-Id: I865d69ccdd30e89083cfee5ca9c2332bfc6c1970
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/341513
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/346297
Reviewed-by: Aaron Durbin <adurbin@google.com>
Commit-Queue: Martin Roth <martinroth@chromium.org>
Trybot-Ready: Martin Roth <martinroth@chromium.org>
Tested-by: Martin Roth <martinroth@chromium.org>
- Remove submodules, since CrOS glues repos together using portage
- Add COMMIT-QUEUE.ini, PRESUBMIT.cfg, .checkpatch.conf from 2016.02
Change-Id: I06b428191699034bf20b086ce88b7b27c3deb41b
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/327003
Reviewed-by: Stefan Reinauer <reinauer@google.com>