diff options
author | Oliver Smith <osmith@sysmocom.de> | 2021-12-22 15:10:03 +0100 |
---|---|---|
committer | Oliver Smith <osmith@sysmocom.de> | 2021-12-22 15:10:03 +0100 |
commit | d7fe8fd047e459858c99ce37af8858c71d7245ff (patch) | |
tree | 9750eb52f35f93e4baa7a6253a9c449669063638 | |
parent | 82f21fa6fc477ae4ece7bf1f557630d30dff8d9d (diff) |
lint: checkpatch_osmo: ignore TRACING_LOGGING
Don't recommend to use ftrace instead of using __func__ inside printf.
This only applies to the linux kernel, not to the Osmocom code base.
Related: OS#5087
Related: https://gerrit.osmocom.org/c/libosmocore/+/6357
Change-Id: Ied23be32d3371a8b9ac9938a336474f71e7a3ccd
-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 d1ce63a..8514f9c 100755 --- a/lint/checkpatch/checkpatch_osmo.sh +++ b/lint/checkpatch/checkpatch_osmo.sh @@ -70,6 +70,7 @@ exclude_paths_project() { # * SPDX_LICENSE_TAG: we don't place it on line 1 # * SPLIT_STRING: we do split long messages over multiple lines # * STRING_FRAGMENTS: sometimes used intentionally to improve readability +# * TRACING_LOGGING: recommends to use kernel's internal ftrace instead of printf("%s()\n", __func__) # * TRAILING_STATEMENTS: not followed, e.g. 'while (osmo_select_main_ctx(1) > 0);' is put in one line # * 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) @@ -108,6 +109,7 @@ $SCRIPT_DIR/checkpatch.pl \ --ignore SPDX_LICENSE_TAG \ --ignore SPLIT_STRING \ --ignore STRING_FRAGMENTS \ + --ignore TRACING_LOGGING \ --ignore TRAILING_STATEMENTS \ --ignore UNNECESSARY_BREAK \ --ignore UNNECESSARY_INT \ |