mirror of
https://github.com/fail0verflow/switch-coreboot.git
synced 2025-05-04 01:39:18 -04:00
e2a5efc68a
Linux' newest checkpatch.pl flags comments like these: /* This is a concise 2-line comment that explains what the code does in * sufficient detail without wasting too much vertical space. */ do_stuff_that_needs_explaining(); Comments like these have been used in our code base for a long time and I don't think we should disallow them now. Ending the comment on the same line doesn't really hurt readability and wastes less screen real estate (which in turn usually helps overall code comprehension). BUG=None BRANCH=None TEST=None Change-Id: I4d0d7c166c3ecd81a8a0d8a4c5133414c9cb5841 Signed-off-by: Julius Werner <jwerner@chromium.org> Reviewed-on: https://review.coreboot.org/16060 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <pgeorgi@google.com> Reviewed-on: https://chromium-review.googlesource.com/368401 Commit-Ready: Furquan Shaikh <furquan@chromium.org> Tested-by: Furquan Shaikh <furquan@chromium.org> Reviewed-by: Aaron Durbin <adurbin@chromium.org>
17 lines
443 B
Text
17 lines
443 B
Text
# Not Linux, so don't expect a Linux tree.
|
|
--no-tree
|
|
|
|
# Require every Kconfig symbol to have at least two lines in their description.
|
|
--min-conf-desc-length=2
|
|
|
|
# Ignore aspects we don't follow here.
|
|
--ignore C99_COMMENTS
|
|
--ignore GLOBAL_INITIALISERS
|
|
--ignore INITIALISED_STATIC
|
|
--ignore LINE_SPACING
|
|
--ignore NEW_TYPEDEFS
|
|
--ignore PREFER_ALIGNED
|
|
--ignore PREFER_PACKED
|
|
--ignore PREFER_PRINTF
|
|
--ignore SPLIT_STRING
|
|
--ignore BLOCK_COMMENT_STYLE
|