[PATCH v3 3/8] patman: invoke the checkpatch.pl script with '--u-boot' and '--strict'
Sjoerd Simons
sjoerd at collabora.com
Fri Jan 13 11:19:39 CET 2023
Hey,
This breaks using patman for linux as it's check-patch has no clue
about `--u-boot` :). I'm not sure if people using patman for linux
patches is meant to be a supported use-case though (the documentation
seems to indicate it is)
On Wed, 2022-12-21 at 00:28 +0000, Simon Glass wrote:
> 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 <sjg at chromium.org>
> Signed-off-by: Maxim Cournoyer <maxim.cournoyer at savoirfairelinux.com>
> ---
>
> Changes in v3:
> - Fix typo in commit message
>
> Changes in v2:
> - Reword the commit message with a better explanation of the problem
> - Add the '--strict' argument to script invocation
>
> tools/patman/checkpatch.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Applied to u-boot-dm/next, thanks!
--
Sjoerd Simons
Collabora Ltd.
More information about the U-Boot
mailing list