From 2db8a96ea7786be2ee026f457690905b7fda6529 Mon Sep 17 00:00:00 2001 From: Julius Werner Date: Fri, 29 Apr 2016 19:00:25 -0700 Subject: [PATCH] checkpatch: Ignore LINE_SPACING 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 Reviewed-on: https://chromium-review.googlesource.com/341513 Reviewed-by: Aaron Durbin Reviewed-on: https://chromium-review.googlesource.com/346297 Reviewed-by: Aaron Durbin Commit-Queue: Martin Roth Trybot-Ready: Martin Roth Tested-by: Martin Roth --- .checkpatch.conf | 1 + 1 file changed, 1 insertion(+) diff --git a/.checkpatch.conf b/.checkpatch.conf index 499d0f1084..4b88745ba2 100644 --- a/.checkpatch.conf +++ b/.checkpatch.conf @@ -8,6 +8,7 @@ --ignore C99_COMMENTS --ignore GLOBAL_INITIALISERS --ignore INITIALISED_STATIC +--ignore LINE_SPACING --ignore NEW_TYPEDEFS --ignore PREFER_ALIGNED --ignore PREFER_PACKED