The ath12k next generation refactoring resulted in some long function
names, which it turn resulted in some long lines in ath12k/wifi7
reported by checkpatch. Suppress those new warnings.
Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
Sparse does not fully understand the guard cleanup feature, and can
produce warnings of the form:
warning: context imbalance in '<function>' - wrong count at exit
These are false positives when guard() is being used, so ignore
them.
Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
Sparse does not fully understand the guard cleanup feature, and can
produce warnings of the form:
warning: context imbalance in '<function>' - wrong count at exit
These are false positives when guard() is being used, so ignore
them.
Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
Update to the latest checkpatch.
With this version there is a new warning that is tricky to fix
cleanly, so add that warning to the checkpatch_filter list.
Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
Update to the latest checkpatch.
With this version there is a new warning that is tricky to fix
cleanly, so add that warning to the checkpatch_filter list.
Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
The ath12k-ng refactoring of ath12k introduces subdirectories.
Update ath12k-check to properly handle the subdirectories.
Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
A new ath12k patch [1] triggers a PREFER_PR_LEVEL checkpatch warning:
drivers/net/wireless/ath/ath12k/debug.c:69: Prefer [subsystem eg: netdev]_dbg([subsystem]dev, ... then dev_dbg(dev, ... then pr_debug(... to printk(KERN_DEBUG ...
Add a rule to ignore it.
Link: https://lore.kernel.org/all/20250204-unlink_link_arvif_from_chanctx-v2-2-764fb5973c1a@oss.qualcomm.com/
Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
Filter the warnings about copyright statements exceeding the max line
length. Since the global tool won't associate these with a symbol
name, filter on the literal "Copyright (c)" itself.
Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
Filter the warnings about the upcoming for_each_ar() macro's variable reuse. As
the global tool apparently doesn't create symbol names for macro. Add a hack
for those cases where we check if the actual line in the file contains the tag
name.
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
As people have had problems setting ath12k-check improve the instructions how
to set up everything. Hopefully we get more consistent results that way. The
instructions are visible --help switch.
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
As the compiler used to compile the kernel can be different from the default
host compiler, print both the host GCC and the compiler specified in .config
file.
Screenshot:
ath12k-check (md5sum a143ac447ffce2545530c2b4ad96a2e8)
python: 3.11.2 (main, Mar 13 2023, 12:18:29) [GCC 12.2.0]
host gcc: gcc (Debian 12.2.0-14) 12.2.0
config cc: x86_64-linux-gcc (GCC) 13.2.0
sparse: v0.6.4-39-gce1a6720f69e
checkpatch.pl: Version: 0.32 (md5sum 47ef327d772c156e53a36597723fc781)
gtags: gtags (Global) 6.6.9
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Print a warning if checkpatch.pl md5sum doesn't match. This way it's easier to
make sure that correct version of checkpatch is used.
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Print a warning if checkpatch.pl md5sum doesn't match. This way it's easier to
make sure that correct version of checkpatch is used.
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Document the checkpatch.pl version I'm using at the moment:
362173572a40 checkpatch: improve EMBEDDED_FILENAME test
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
