diff options
author | Oliver Smith <osmith@sysmocom.de> | 2021-12-08 11:18:37 +0100 |
---|---|---|
committer | Oliver Smith <osmith@sysmocom.de> | 2021-12-08 11:24:29 +0100 |
commit | 425cbb5e28b207a27bea2ed476d8b687107254ec (patch) | |
tree | 30241368356e0c8eea39e422f10a2b2d117aa19f | |
parent | 78c08a33bd5b98cfe1271f77a8ef8d6b7db3180f (diff) |
lint: checkpatch_osmo: ignore VOLATILE
The warning about potentially using volatile wrong is not useful for us,
it makes sense to use it in embedded projects.
Related: OS#5087
Related: https://www.kernel.org/doc/html/latest/process/volatile-considered-harmful.html
Related: https://gerrit.osmocom.org/c/simtrace2/+/26463
Change-Id: Ie81db479c66749531ed1c81cf076ce248aa22f69
-rwxr-xr-x | lint/checkpatch/checkpatch_osmo.sh | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/lint/checkpatch/checkpatch_osmo.sh b/lint/checkpatch/checkpatch_osmo.sh index 126a8b4..ae66e71 100755 --- a/lint/checkpatch/checkpatch_osmo.sh +++ b/lint/checkpatch/checkpatch_osmo.sh @@ -73,6 +73,7 @@ exclude_paths_project() { # * UNNECESSARY_BREAK: not followed (see https://gerrit.osmocom.org/c/libosmo-netif/+/26429) # * UNNECESSARY_INT: not followed (see https://gerrit.osmocom.org/c/libosmocore/+/25345) # * UNSPECIFIED_INT: not followed (doesn't seem useful for us) +# * VOLATILE: using volatile makes sense in embedded projects so this warning is not useful for us cd "$PROJECT_DIR" @@ -109,6 +110,7 @@ $SCRIPT_DIR/checkpatch.pl \ --ignore UNNECESSARY_BREAK \ --ignore UNNECESSARY_INT \ --ignore UNSPECIFIED_INT \ + --ignore VOLATILE \ --max-line-length 120 \ --no-signoff \ --no-tree \ |