aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOliver Smith <osmith@sysmocom.de>2021-07-12 16:58:51 +0200
committerOliver Smith <osmith@sysmocom.de>2021-07-15 10:21:47 +0200
commit7f0fdda23c7d065e134785f29cb61266f31eaf3a (patch)
tree4f6e39f96f1f2eaa08855ecb5d3912807d0ae32e
parent7c3c0bdaa55fac0b1f7813471841509b0e48d5d5 (diff)
lint: ignore PREFER_DEFINED_ATTRIBUTE_MACRO
Don't complain that macros such as __packed should be used, which are defined in the Linux kernel but not in libosmocore. For example: src/gsm/gsm0808.c:85: WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __packed over __attribute__((packed)) Related: OS#5087 Change-Id: I2bf3b7d60e99cf91f7b619af54167a11cdfae8c6
-rwxr-xr-xlint/checkpatch/checkpatch_osmo.sh2
1 files changed, 2 insertions, 0 deletions
diff --git a/lint/checkpatch/checkpatch_osmo.sh b/lint/checkpatch/checkpatch_osmo.sh
index 4be1348..88f0b2d 100755
--- a/lint/checkpatch/checkpatch_osmo.sh
+++ b/lint/checkpatch/checkpatch_osmo.sh
@@ -21,6 +21,7 @@ SCRIPT_DIR="$(dirname "$(realpath "$0")")"
# * INITIALISED_STATIC: we use this, see also http://lkml.iu.edu/hypermail/linux/kernel/0808.1/2235.html
# * LINE_CONTINUATIONS: false positives
# * LINE_SPACING: we don't always put a blank line after declarations
+# * PREFER_DEFINED_ATTRIBUTE_MACRO: macros like __packed not defined in libosmocore
# * PREFER_FALLTHROUGH: pseudo keyword macro "fallthrough" is not defined in libosmocore
# * REPEATED_WORD: false positives in doxygen descriptions (e.g. '\param[in] data Data passed through...')
# * SPDX_LICENSE_TAG: we don't place it on line 1
@@ -45,6 +46,7 @@ $SCRIPT_DIR/checkpatch.pl \
--ignore INITIALISED_STATIC \
--ignore LINE_CONTINUATIONS \
--ignore LINE_SPACING \
+ --ignore PREFER_DEFINED_ATTRIBUTE_MACRO \
--ignore PREFER_FALLTHROUGH \
--ignore REPEATED_WORD \
--ignore SPDX_LICENSE_TAG \