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

patman: Deal with git safe-directory warning

2025-05-27T09:07:42+00:00

When running tests where the .git directory is not owned by the current
user, various warnings are produced and the tests fail. This happens in
CI.

For patman itself, modify the gitutil.get_top_level() function to return
None in this case. Ensure that the warning is not shown, since it creates
about 1000 lines of output.

For checkpatch, the same warning is produced even though --no-tree is
given. Suppress that as well.

Signed-off-by: Simon Glass

patman: Allow setting the current directory when sending

2025-05-27T09:07:42+00:00

Plumb a current-working-directory (cwd) through from send all the way to
the command gitutil libraries. This will allow better testing of this
functionality, since we can use a test directory.

Signed-off-by: Simon Glass

u_boot_pylib: Move gitutil into the library

2025-02-17T17:17:55+00:00

Move this file into U-Boot's Python library, so that it is no-longer
part of patman.

This makes a start on:

https://source.denx.de/u-boot/custodians/u-boot-dm/-/issues/35

Signed-off-by: Simon Glass

patman: Check patches in parallel

2023-03-08T21:15:14+00:00

For large series this can take a while. Run checkpatch in parallel to
try to reduce the time. The checkpatch information is still reported in
sequential order, so a very slow patch at the start can still slow
things down. But overall this gives good results.

Signed-off-by: Simon Glass 
Reviewed-by: Douglas Anderson

patman: Move library functions into a library directory

2023-03-08T19:40:49+00:00

The patman directory has a number of modules which are used by other tools
in U-Boot. This makes it hard to package the tools using pypi since the
common files must be copied along with the tool that uses them.

To address this, move these files into a new u_boot_pylib library. This
can be packaged separately and listed as a dependency of each tool.

Signed-off-by: Simon Glass

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

2023-01-18T21:55:41+00:00

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:57+00:00

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:56+00:00

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:12+00:00

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:13+00:00

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