u-boot.git/tools/patman/checkpatch.py, branch v2023.04

Revert "patman: invoke the checkpatch.pl script with '--u-boot' and '--strict'"

2023-01-18T21:55:41Z

This reverts commit 648d8186dd7f9c444fb07f355090d275dcdd4de4, because it broke usage of patman on Linux, whose check script doesn't know about '--strict' or '--u-boot'. Reported-by: Sjoerd Simons Signed-off-by: Maxim Cournoyer Reviewed-by: Simon Glass Tested-by: Sjoerd Simons

patman: invoke the checkpatch.pl script with '--u-boot' and '--strict'

2023-01-06T02:21:57Z

This resolves 10 out of 11 test failures seen when running './patman test' from the 'tools/patman' subdirectory. This was caused by the .checkpatch.conf configuration file at the root of the project not being picked up. Make the test suite of patman independent from it by always invoking the checkpatch.pl script with the minimally required arguments for the test suite to pass. Reviewed-by: Simon Glass Signed-off-by: Maxim Cournoyer

patman: By default don't pass "--no-tree" to checkpatch for linux

2022-07-26T08:30:56Z

When you pass "--no-tree" to checkpatch it disables some extra checks that are important for Linux. Specifically I want checks like: warning: DT compatible string "boogie,woogie" appears un-documented check ./Documentation/devicetree/bindings/ Let's make the default for Linux to _not_ pass --no-tree. We'll have a config option and command line flag to override. Signed-off-by: Douglas Anderson Reviewed-by: Simon Glass

patman: Correct pylint errors

2022-03-02T15:28:12Z

Fix pylint errors that can be fixed and mask those that seem to be incorrect. Signed-off-by: Simon Glass

patman: Rename Color() method to build()

2022-02-09T19:30:13Z

This method has the same name as its class which is confusing. It is also annoying when searching the code. It builds a string with a colour, so rename it to build(). Signed-off-by: Simon Glass

patman: Convert camel case in gitutil.py

2022-02-09T19:26:12Z

Convert this file to snake case and update all files which use it. Signed-off-by: Simon Glass

patman: Convert camel case in checkpatch.py

2022-02-09T19:26:12Z

Convert this file to snake case and update all files which use it. Signed-off-by: Simon Glass

patman: Convert camel case in command.py

2022-02-09T19:26:12Z

Convert this file to snake case and update all files which use it. Signed-off-by: Simon Glass

patman: Parse checkpatch by message instead of by line

2021-04-29T10:23:39Z

Parse each empty-line-delimited message separately. This saves having to deal with all the different line content styles, we only care about the header ERROR | WARNING | NOTE... Also make checkpatch print line information for a uboot specific warning. Signed-off-by: Evan Benn Reviewed-by: Simon Glass

patman: Improve handling of files

2020-11-05T16:11:31Z

Sometimes warnings are associated with a file and sometimes with the patch as a whole. Update the regular expression to handle both cases, even in emacs mode. Also add support for detecting new files. Signed-off-by: Simon Glass