[U-Boot] [RFC PATCH] checkpatch.pl: Add ability to ignore various messages

Joe Perches joe at perches.com
Thu Jul 28 00:10:39 CEST 2011


On Wed, 2011-07-27 at 23:17 +0200, Wolfgang Denk wrote:
> Dear Joe,
> In message <b41636a5e5756917b07e50a4b8309dbb092b0a17.1303778411.git.joe at perches.com> you wrote:
> > Some users would like the ability to not emit some of the
> > messages that checkpatch produces.
> > Add command line option to "--ignore" various message types.
> > Add .checkpatch.conf file to store a default set of "ignore" flags
> > Categorize all ERROR, WARN and CHK types with flags.
> > Signed-off-by: Joe Perches <joe at perches.com>
> > ---
> >  scripts/checkpatch.pl |  456 +++++++++++++++++++++++++++++++++++--------------
> >  1 files changed, 327 insertions(+), 129 deletions(-)
> Did you receive any feedback for your proposal, especially from Andrew
> Morton?
> This feature would be really useful for us, but I'd rather see this in
> mainline than starting a fork.

A better version is in Linus' current 3.0+ as of a couple of days ago.
It's not in 3.0.

commit 000d1cc1829f938c87402fc2fd4bb5e8daed6b52
Author: Joe Perches <joe at perches.com>
Date:   Mon Jul 25 17:13:25 2011 -0700

    checkpatch.pl: add ability to ignore various messages
    
    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 exis
        ./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 <joe at perches.com>
    Acked-by: Mike Frysinger <vapier at gentoo.org>
    Cc: Andy Whitcroft <apw at canonical.com>
    Signed-off-by: Andrew Morton <akpm at linux-foundation.org>
    Signed-off-by: Linus Torvalds <torvalds at linux-foundation.org>





More information about the U-Boot mailing list