Re: [PATCH] checkpatch: add filename in the summary

From: Paolo Ciarrocchi
Date: Tue Jan 22 2008 - 08:21:13 EST


On Jan 14, 2008 11:23 PM, Paolo Ciarrocchi <paolo.ciarrocchi@xxxxxxxxx> wrote:
> Add a filename option (default to 0)
> in order to get the following summary output:
> ./scripts/checkpatch.pl --filename --file ./arch/sparc/kernel/apc.c
> ...
> ...
> ...
> ./arch/sparc/kernel/apc.c --> total: 18 errors, 3 warnings, 186 lines checked
> ---
>
> Andy,
> I wonder why checkpatch doesn't show all the command line options.
> At the moment, it only prints:
> options: -q => quiet
> --no-tree => run without a kernel tree
> --terse => one line per report
> --emacs => emacs compile window format
> --file => check a source file
> --strict => enable more subjective tests
> --root => path to the kernel tree root
>
> but I see a lot more options:
> q|quiet
> tree
> signoff
> patch
> test-type
> emacs
> terse
> file
> subjective
> strict
> root=s
> summary
> mailback
>
[patch snipped]

Hi Andy,
do you plan to document all the command line options that checkpatch
is currently accepting?

Thanks.

Ciao,
--
Paolo
http://paolo.ciarrocchi.googlepages.com/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/