mirror of
https://github.com/CTCaer/switch-l4t-atf.git
synced 2024-11-24 02:09:46 +00:00
cf57ff8b99
As it turns out, Gerrit's merge commits don't always respect that format so these mistakes have to be ignored as false positives. Change-Id: I4e38d9c34c95588e7916fba4c154f017d8c92dec Signed-off-by: John Tsichritzis <john.tsichritzis@arm.com>
92 lines
2.8 KiB
Plaintext
92 lines
2.8 KiB
Plaintext
#
|
|
# Copyright (c) 2016-2019, ARM Limited and Contributors. All rights reserved.
|
|
#
|
|
# SPDX-License-Identifier: BSD-3-Clause
|
|
#
|
|
|
|
#
|
|
# Configure how the Linux checkpatch script should be invoked in the context of
|
|
# the Trusted Firmware source tree.
|
|
#
|
|
|
|
# This is not Linux so don't expect a Linux tree!
|
|
--no-tree
|
|
|
|
# The Linux kernel expects the SPDX license tag in the first line of each file.
|
|
# We don't follow this in the Trusted Firmware.
|
|
--ignore SPDX_LICENSE_TAG
|
|
|
|
# This clarifes the lines indications in the report.
|
|
#
|
|
# E.g.:
|
|
# Without this option, we have the following output:
|
|
# #333: FILE: drivers/arm/gic/arm_gic.c:160:
|
|
# So we have 2 lines indications (333 and 160), which is confusing.
|
|
# We only care about the position in the source file.
|
|
#
|
|
# With this option, it becomes:
|
|
# drivers/arm/gic/arm_gic.c:160:
|
|
--showfile
|
|
|
|
# Don't show some messages like the list of ignored types or the suggestion to
|
|
# use "--fix" or report changes to the maintainers.
|
|
--quiet
|
|
|
|
#
|
|
# Ignore the following message types, as they don't necessarily make sense in
|
|
# the context of the Trusted Firmware.
|
|
#
|
|
|
|
# COMPLEX_MACRO generates false positives.
|
|
--ignore COMPLEX_MACRO
|
|
|
|
# Commit messages might contain a Gerrit Change-Id.
|
|
--ignore GERRIT_CHANGE_ID
|
|
|
|
# Do not check the format of commit messages, as Gerrit's merge commits do not
|
|
# preserve it.
|
|
--ignore GIT_COMMIT_ID
|
|
|
|
# FILE_PATH_CHANGES reports this kind of message:
|
|
# "added, moved or deleted file(s), does MAINTAINERS need updating?"
|
|
# We do not use this MAINTAINERS file process in TF.
|
|
--ignore FILE_PATH_CHANGES
|
|
|
|
# AVOID_EXTERNS reports this kind of messages:
|
|
# "externs should be avoided in .c files"
|
|
# We don't follow this convention in TF.
|
|
--ignore AVOID_EXTERNS
|
|
|
|
# NEW_TYPEDEFS reports this kind of messages:
|
|
# "do not add new typedefs"
|
|
# We allow adding new typedefs in TF.
|
|
--ignore NEW_TYPEDEFS
|
|
|
|
# Avoid "Does not appear to be a unified-diff format patch" message
|
|
--ignore NOT_UNIFIED_DIFF
|
|
|
|
# VOLATILE reports this kind of messages:
|
|
# "Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt"
|
|
# We allow the usage of the volatile keyword in TF.
|
|
--ignore VOLATILE
|
|
|
|
# BRACES reports this kind of messages:
|
|
# braces {} are not necessary for any arm of this statement
|
|
--ignore BRACES
|
|
|
|
# PREFER_KERNEL_TYPES reports this kind of messages (when using --strict):
|
|
# "Prefer kernel type 'u32' over 'uint32_t'"
|
|
--ignore PREFER_KERNEL_TYPES
|
|
|
|
# USLEEP_RANGE reports this kind of messages (when using --strict):
|
|
# "usleep_range is preferred over udelay; see Documentation/timers/timers-howto.txt"
|
|
--ignore USLEEP_RANGE
|
|
|
|
# COMPARISON_TO_NULL reports this kind of messages (when using --strict):
|
|
# Comparison to NULL could be written ""
|
|
--ignore COMPARISON_TO_NULL
|
|
|
|
# UNNECESSARY_PARENTHESES reports this kind of messages (when using --strict):
|
|
# Unnecessary parentheses around ""
|
|
--ignore UNNECESSARY_PARENTHESES
|