linux-toradex.git/scripts/checkpatch.pl, branch Colibri_T30_LinuxImageV2.3Beta3_20141031

checkpatch: add missing WARN argument for min_t and max_t tests

2011-08-25T23:25:33+00:00

The test for bad usage of min_t() and max_t() is missing the --ignore
type.  Add it.

Signed-off-by: Hui Zhu 
Signed-off-by: Joe Perches 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

checkpatch.pl: update $logFunctions

2011-07-26T03:57:16+00:00

Previous behavior allowed only alphabetic prefixes like pr_info to exceed
the 80 column line length limit.

ath6kl wants to add a digit into the prefix, so allow numbers as well as
digits in the _ printks.

__ratelimited and __once and WARN_RATELIMIT
and WARN_ONCE may now exceed 80 cols.

Add missing _printk type for completeness.

Signed-off-by: Joe Perches 
Cc: Andy Whitcroft 
Cc: Kalle Valo 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

checkpatch: make utf-8 test --strict

2011-07-26T03:57:16+00:00

Some patches are sent in using ISO-8859 or even Windows codepage 1252.

Make checkpatch accept these by default and only emit the "Invalid UTF-8"
message when using --strict.

Signed-off-by: Joe Perches 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

checkpatch.pl: add ability to ignore various messages

2011-07-26T03:57:16+00:00

Some users would like the ability to not emit some of the messages that
checkpatch produces.  This can make it easier to use checkpatch in other
projects and integrate into scm hook scripts.

Add command line option to "--ignore" various message types.  Add option
--show-types to emit the "type" of each message.  Categorize all ERROR,
WARN and CHK messages with types.

Add optional .checkpatch.conf file to store default options.
3 paths are searched for .checkpatch.conf
    .             customized per-tree configurations
    $HOME         user global configuration when per-tree configs don't exist
    ./scripts     lk defaults to override script
The .conf file can contain any valid command-line argument and
the contents are prepended to any additional command line arguments.
Multiple lines may be used, blank lines are ignored, # is a comment.

Update "false positive" output for readability.

Update version to 0.32

Signed-off-by: Joe Perches 
Acked-by: Mike Frysinger 
Cc: Andy Whitcroft 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

checkpatch: add a "prefer __aligned" check

2011-07-26T03:57:16+00:00

Prefer the use of __aligned(size) over __attribute__((__aligned___(size)))

Link: http://lkml.kernel.org/r/20110609094526.1571774c.akpm@linux-foundation.org

Suggested-by: Andrew Morton 
Signed-off-by: Joe Perches 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

checkpatch: validate signature styles and To: and Cc: lines

2011-07-26T03:57:16+00:00

Signatures have many forms and can sometimes cause problems if not in the
correct format when using git send-email or quilt.

Try to verify the signature tags and email addresses to use the generally
accepted "Signed-off-by: Full Name " form.

Original idea by Anish Kumar 

Signed-off-by: Joe Perches 
Cc: Anish Kumar 
Cc: Nick Bowler 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

checkpatch: add __rcu as a sparse modifier

2011-07-26T03:57:16+00:00

Fix "need consistent spacing around '*'" error after a __rcu sparse
annotation which was caused by the missing __rcu entry in the
checkpatch.pl internal list of sparse keywords.

Signed-off-by: Sven Eckelmann 
Cc: Andy Whitcroft 
Cc: Joe Perches 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

checkpatch: suggest using min_t or max_t

2011-07-26T03:57:16+00:00

A common issue with min() or max() is using a cast on one or both of the
arguments when using min_t/max_t could be better.

Add cast detection to uses of min/max and suggest an appropriate use of
min_t or max_t instead.

Caveat:  This only works for min() or max() on a single line.
         It does not find min() or max() split across multiple lines.

This does find:
	min((u32)foo, bar);
But it does not find:
	max((unsigned long)foo,
	    bar);

Suggested-by: Andrew Morton 
Signed-off-by: Joe Perches 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

checkpatch: add warning for uses of printk_ratelimit

2011-06-16T03:03:59+00:00

Warn about uses of printk_ratelimit() because it uses a global state and
can hide subsequent useful messages.

Signed-off-by: Joe Perches 
Cc: Andy Whitcroft 
Cc: Richard Weinberger 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

checkpatch: fix defect in printk(KERN_ 80 column exceptions

2011-05-25T15:39:56+00:00

Currently, printk lines with a only KERN_PREFIX and a quoted string
without a comma or close paren that exceed 80 columns are flagged with a
warning.

ie:
	printk(KERN_WARNING "some long string that extends beond 80 cols..."
	       "and is continued on another line\n");

Allow this form instead of emitting a warning.

Signed-off-by: Joe Perches 
Cc: Andy Whitcroft 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds