checkpatch: add of_device_id to structs that should be const

2015-02-14T05:21:40Z

Uses of struct of_device_id are most commonly const. Suggest using it as such. Signed-off-by: Joe Perches Acked-by: Rob Herring Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds

checkpatch: try to avoid poor patch subject lines

2015-02-14T05:21:40Z

Naming the tool that found an issue in the subject line isn't very useful. Emit a warning when a common tool (currently checkpatch, sparse or smatch) is in the subject line. Signed-off-by: Joe Perches Suggested-by: Al Viro Acked-by: Dan Carpenter Acked-by: Greg Kroah-Hartman Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds

checkpatch: make sure a commit reference description uses parentheses

2015-02-14T05:21:40Z

The preferred style for a commit reference in a commit log is: commit ("") A recent commit removed this check for parentheses. Add it back. Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> </article> <article> <h1>checkpatch: add --strict test for spaces around arithmetic</h1> <p>2015-02-14T05:21:40Z</p> Some prefer code to have spaces around arithmetic so instead of: a = b*c+d; suggest a = b * c + d; Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> </article> <article> <h1>checkpatch: neaten printk_ratelimited message position</h1> <p>2015-02-14T05:21:40Z</p> Just neatening... Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> </article> <article> <h1>checkpatch: improve "no space necessary after cast" test</h1> <p>2015-02-14T05:21:40Z</p> Code like: if (a < sizeof(<type>) && and { .len = sizeof(<type>) }, incorrectly emits that warning, so add more exceptions to avoid the warning. Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> </article> <article> <h1>checkpatch: improve seq_print->seq_puts suggestion</h1> <p>2015-02-14T05:21:40Z</p> Improve the format specifier test by removing any %% before looking for any remaining % format specifier. Signed-off-by: Heba Aamer <heba93aamer@gmail.com> Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> </article> <article> <h1>checkpatch: add ability to --fix unnecessary blank lines around braces</h1> <p>2015-02-14T05:21:39Z</p> There's a --strict test for these blank lines. Add the ability to automatically remove them with --fix. Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> </article> <article> <h1>checkpatch: add types for other OS typedefs</h1> <p>2015-02-14T05:21:39Z</p> bsd and sysv use different typedefs for unsigned types. These are in types.h but not in checkpatch, so add them to checkpatch's ability to know types. This can avoid false positives for code like: void foo(void) { int x; uint y; [...]; } where checkpatch incorrectly emits a warning for "missing a blank line after declarations". Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> </article> <article> <h1>checkpatch: add ability to coalesce commit descriptions on multiple lines</h1> <p>2015-02-14T05:21:39Z</p> If a git commit description is split on consecutive lines, coalesce it before testing. This allows: commit <foo> ("some long description") Signed-off-by: Joe Perches <joe@perches.com> Reported-by: Paul Bolle <pebolle@tiscali.nl> Tested-by: Paul Bolle <pebolle@tiscali.nl> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> </article> </main></body></html>

user/sven/linux.git/scripts/checkpatch.pl, branch v4.0

checkpatch: add of_device_id to structs that should be const

checkpatch: try to avoid poor patch subject lines

checkpatch: make sure a commit reference description uses parentheses