diff options
author | Konstantin Porotchkin <kostap@marvell.com> | 2018-07-23 15:10:56 +0300 |
---|---|---|
committer | Konstantin Porotchkin <kostap@marvell.com> | 2018-09-03 14:45:20 +0300 |
commit | 71b6429c5769b0c5b7165217dd441ea2a0318bfa (patch) | |
tree | 257af02f9592823a4c48a2b52207f86dd33e5207 | |
parent | 497a8a0f7d3eb04109ec0cd1dbf040fc33d1c95c (diff) |
scripts: add more ignores to patch checking configuration
- move back to the default line length of 80 characters for
making future mainline code submissions easier
- ignore warning about non-constant structures- seems to be
too strict and usually ignored anyway
- ignore warning abour SPDX header format since it's differs
from the kernel headers.
Change-Id: I3b81391470d27e6bed7bc6ff6be9fba5fc5d37e8
Signed-off-by: Konstantin Porotchkin <kostap@marvell.com>
Reviewed-on: http://vgitil04.il.marvell.com:8080/58281
Tested-by: iSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: Igal Liberman <igall@marvell.com>
-rw-r--r-- | .checkpatch.conf | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/.checkpatch.conf b/.checkpatch.conf index b1081d50..033794a5 100644 --- a/.checkpatch.conf +++ b/.checkpatch.conf @@ -90,8 +90,11 @@ # We allow the usage of the volatile keyword in TF. --ignore VOLATILE -# Marvell coding style allows usage of 120-symbol strings -# Regular mainline code limitation is 80 characters -# For reducing the amount of warnings on Marvell CI, the maximum -# line length is set to 120 characters ---max-line-length 120 +# Requesting all structures to be "const" seems to be a very strict requirement. +# We usually ignore this warning, so it's better not to show it at all +--ignore CONST_STRUCT + +# The checkpatch is too strict about SPDX license tag location +# It's expected to be in the first file line as in kernel. +# However TF-A uses different header format with copyright at the header start. +--ignore SPDX_LICENSE_TAG |