[U-Boot] [PATCH v2 1/7] patman: Fix up checkpatch parsing to deal with 'CHECK' lines
Doug Anderson
dianders at chromium.org
Tue Apr 2 01:16:23 CEST 2013
Simon,
On Tue, Mar 26, 2013 at 4:09 PM, Simon Glass <sjg at chromium.org> wrote:
> checkpatch has a new type of warning, a 'CHECK'. At present patman fails
> with these, which makes it less than useful.
>
> Add support for checks, making it backwards compatible with the old
> checkpatch.
>
> At the same time, clean up formatting of the CheckPatches() output,
> fix erroneous "internal error" if multiple patches have warnings and
> be more robust to new types of problems.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> Changes in v2:
> - Update code to remove match_count
> - Update commit message to add detail on what is changing
> - Update tests to check the 'checks' value, and add a new test
> - Use namedtuple to hold the return value from CheckPatch() function
>
> tools/patman/checkpatch.py | 110 ++++++++++++++++++++++++++++-----------------
> tools/patman/test.py | 64 +++++++++++++++++---------
> 2 files changed, 112 insertions(+), 62 deletions(-)
Thanks for fixing and for fixing up the tests as well.
Reviewed-by: Doug Anderson <dianders at chromium.org>
More information about the U-Boot
mailing list