Re: [PATCH 1/2] Revert "patman: invoke the checkpatch.pl script with '--u-boot' and '--strict'"

[Sjoerd Simons <sjoerd@collabora.com>]

On Fri, 2023-01-13 at 08:50 -0500, Maxim Cournoyer wrote:
> This reverts commit 648d8186dd7f9c444fb07f355090d275dcdd4de4, because
> it broke usage of patman on Linux, whose check script doesn't know
> about '--strict' or '--u-boot'.

I can confirm that this reverts fixes the reported issue:

Tested-by: Sjoerd Simons <sjoerd@collabora.com>

> 
> Reported-by: Sjoerd Simons <sjoerd@collabora.com>
> Signed-off-by: Maxim Cournoyer <maxim.cournoyer@savoirfairelinux.com>
> ---
> 
>  tools/patman/checkpatch.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/patman/checkpatch.py b/tools/patman/checkpatch.py
> index 012c0d895c..d1b902dd96 100644
> --- a/tools/patman/checkpatch.py
> +++ b/tools/patman/checkpatch.py
> @@ -211,7 +211,7 @@ def check_patch(fname, verbose=False,
> show_types=False, use_tree=False):
>              stdout: Full output of checkpatch
>      """
>      chk = find_check_patch()
> -    args = [chk, '--u-boot', '--strict']
> +    args = [chk]
>      if not use_tree:
>          args.append('--no-tree')
>      if show_types:
> 
> base-commit: 87c9e117bf57d6bb42c5521a3f6ec9ca7d97e5fa

-- 
Sjoerd Simons
Collabora Ltd.