Documentation/dev-tools/checkpatch.rst

1 .. SPDX-License-Identifier: GPL-2.0-only
27  - -q,  --quiet
31  - -v,  --verbose
35  - --no-tree
39  - --no-signoff
41    Disable the 'Signed-off-by' line check.  The sign-off is a simple line at
43    or otherwise have the right to pass it on as an open-source patch.
47 	 Signed-off-by: Random J Developer <random@developer.example.org>
49    Setting this flag effectively stops a message for a missing signed-off-by
52  - --patch
57  - --emacs
63  - --terse
67  - --showfile
71  - -g,  --git
77    - <rev>
78    - <rev>^
79    - <rev>~n
83    - <rev1>..<rev2>
84    - <rev1>...<rev2>
85    - <rev>-<count>
87  - -f,  --file
92  - --subjective,  --strict
97  - --list-types
105  - --types TYPE(,TYPE2...)
111      ./scripts/checkpatch.pl mypatch.patch --types EMAIL_SUBJECT,BRACES
113  - --ignore TYPE(,TYPE2...)
119      ./scripts/checkpatch.pl mypatch.patch --ignore EMAIL_SUBJECT,BRACES
121  - --show-types
126  - --max-line-length=n
134    file contexts, the --strict flag must also be enabled.
136  - --min-conf-desc-length=n
140  - --tab-size=n
144  - --root=PATH
151  - --no-summary
155  - --mailback
160  - --summary-file
164  - --debug KEY=[0|1]
169  - --fix
172    <inputfile>.EXPERIMENTAL-checkpatch-fixes is created which has the
175  - --fix-inplace
177    EXPERIMENTAL - Similar to --fix but input file is overwritten with fixes.
182  - --ignore-perl-version
187  - --codespell
191  - --codespellfile
196  - --typedefsfile
200  - --color[=WHEN]
205  - --kconfig-prefix=WORD
209  - -h, --help, --version
219  - ERROR
224  - WARNING
229  - CHECK
243 ----------------
250     See: https://www.kernel.org/doc/html/latest/core-api/memory-allocation.html
263     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#allocating-memory
269     See: https://www.kernel.org/doc/html/latest/core-api/memory-allocation.html
273 ---------
290     See: https://www.kernel.org/doc/html/latest/process/deprecated.html#bug-and-bug-on
299 …nel.org/doc/html/latest/process/deprecated.html#simple-strtol-simple-strtoll-simple-strtoul-simple…
335     See: https://lore.kernel.org/lkml/1400106425.12666.6.camel@joe-AO725/
339     old flavourful RCU APIs by their new vanilla-RCU counterparts.
343     See: https://www.kernel.org/doc/html/latest/RCU/whatisRCU.html#full-list-of-rcu-apis
357     See: https://www.kernel.org/doc/html/latest/driver-api/driver-model/device.html#attributes
366     See: https://www.kernel.org/doc/html/latest/driver-api/driver-model/device.html#attributes
375     See: https://www.kernel.org/doc/html/latest/driver-api/driver-model/device.html#attributes
384     See: https://www.kernel.org/doc/html/latest/driver-api/driver-model/device.html#attributes
395       &zero     ->  SYSCTL_ZERO
396       &one      ->  SYSCTL_ONE
397       &int_max  ->  SYSCTL_INT_MAX
401       1. https://lore.kernel.org/lkml/20190430180111.10688-1-mcroce@redhat.com/
402       2. https://lore.kernel.org/lkml/20190531131422.14970-1-mcroce@redhat.com/
428     See: https://lore.kernel.org/lkml/20080320201723.b87b3732.akpm@linux-foundation.org/
432     prevent warnings on conversion of device->sem to device->mutex.
446     See: https://www.kernel.org/doc/html/latest/locking/lockdep-design.html#annotations
457 --------
460     The comment style is incorrect.  The preferred style for multi-
475     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#commenting
481     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#commenting
487     See: https://lore.kernel.org/lkml/20200401101714.44781-1-elver@google.com/
500 --------------
503     The signed-off-by line does not fall in line with the standards
506 …//www.kernel.org/doc/html/latest/process/submitting-patches.html#developer-s-certificate-of-origin…
528 …  See: https://www.kernel.org/doc/html/latest/process/submitting-patches.html#describe-your-changes
535 …  See: https://www.kernel.org/doc/html/latest/process/submitting-patches.html#describe-your-changes
538     The author's email does not match with that in the Signed-off-by:
544       - The email names do not match.
545       - The email addresses do not match.
546       - The email subaddresses do not match.
547       - The email comments do not match.
550     The patch is missing a Signed-off-by line.  A signed-off-by
554 ….kernel.org/doc/html/latest/process/submitting-patches.html#sign-your-work-the-developer-s-certifi…
563 ….kernel.org/doc/html/latest/process/submitting-patches.html#sign-your-work-the-developer-s-certifi…
571     See: https://lore.kernel.org/lkml/20150611134006.9df79a893e3636019ad2759e@linux-foundation.org/
575     have a Change-Id like::
577       Change-Id: Ic8aaa0728a43936cd4c6e1ed590e01ba8f0fbf5b
578       Signed-off-by: A. U. Thor <author@example.com>
580     The Change-Id line must be removed before submitting.
593 …  See: https://www.kernel.org/doc/html/latest/process/submitting-patches.html#describe-your-changes
600 …  See: https://www.kernel.org/doc/html/latest/process/submitting-patches.html#describe-your-changes
604 ----------------
621     See: https://lore.kernel.org/lkml/1365563834.27174.12.camel@joe-AO722/
633 ---------------------------
640     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#indentation
672     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#indentation
676     To use a different maximum line length, the --max-line-length=n option
680     ("checkpatch/coding-style: deprecate 80-column warning") increased the
684 …See: https://www.kernel.org/doc/html/latest/process/coding-style.html#breaking-long-lines-and-stri…
688     To use a different maximum line length, the --max-line-length=n option
691 …See: https://www.kernel.org/doc/html/latest/process/coding-style.html#breaking-long-lines-and-stri…
695     To use a different maximum line length, the --max-line-length=n option
698 …See: https://www.kernel.org/doc/html/latest/process/coding-style.html#breaking-long-lines-and-stri…
709       struct_identifier->member[index].
722       member1->member2->member3.foo1;
723       member1->member2->member3.foo2;
725     then store the member1->member2->member3 part in a temporary variable.
747 ------------------------------
786     read-only.  The __read_mostly annotation thus should be removed.
795     non-deterministic builds.
797     See: https://www.kernel.org/doc/html/latest/kbuild/reproducible-builds.html#timestamps
849     See: https://lore.kernel.org/lkml/1377655732.3619.19.camel@joe-AO722/
853     do - while block.  Same should also be the case for macros
862     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#macros-enums-and-rtl
887     See: https://lore.kernel.org/lkml/1399671106.2912.21.camel@joe-AO725/
890     If function-like macros do not utilize a parameter, it might result
899       WARNING: Argument 'a' is not used in function-like macro.
901     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#macros-enums-and-rtl
904     For the multi-statement macros, it is necessary to use the do-while
905     loop to avoid unpredictable code paths. The do-while loop helps to
907     function-like macro can be used as a function only.
910     do-while loop. Although the code is syntactically correct but using
911     the do-while loop is redundant. So remove the do-while loop for single
920 -----------------------
925     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#naming
946     Note that this does not work with -f (--file) checkpatch option
1005 -----------
1009     Typically only three permissions are used - 0644 (RW), 0444 (RO)
1022     any local user to write arbitrary values into device registers - a
1025     See: https://lore.kernel.org/linux-arm-kernel/cover.1296818921.git.segoon@openwall.com/
1033     understand than their symbolic counterparts because many command-line
1040 …See: https://lore.kernel.org/lkml/CA+55aFw5v23T-zvDZp-MmD_EYxF8WbafwwB59934FV7g21uMGQ@mail.gmail.c…
1044 --------------------
1059     This applies for all non-functional blocks.
1068     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#placing-braces-and-spaces
1099     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#placing-braces-and-spaces
1105     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#spaces
1109     next line.  For any non-functional block it should be on the same line
1112     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#placing-braces-and-spaces
1124     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#spaces
1129     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#spaces
1136     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#spaces
1143           (foo->bar)();
1147           foo->bar();
1151           if ((foo->bar) && (foo->baz))
1156           if (foo->bar && foo->baz)
1161           &(foo->bar)
1162           *(foo->bar)
1166           &foo->bar
1167           *foo->bar
1176     See: https://www.kernel.org/doc/html/latest/process/coding-style.html#placing-braces-and-spaces
1180 ------
1203     For DOS-formatted patches, there are extra ^M symbols at the end of
1207     DT bindings moved to a json-schema based format instead of
1210     See: https://www.kernel.org/doc/html/latest/devicetree/bindings/writing-schema.html
1217     DT only tree created with git-filter-branch.
1219 …//www.kernel.org/doc/html/latest/devicetree/bindings/submitting-patches.html#i-for-patch-submitters
1236     The patch file does not appear to be in unified-diff format.  Please
1247     See: https://www.kernel.org/doc/html/latest/process/license-rules.html