[PATCH] patman: use new --u-boot option with checkpath.pl

[Simon Glass]

Hi Daniel,

On Sat, 6 Jun 2020 at 20:02, Simon Glass <sjg at chromium.org> wrote:
>
> Hi Daniel,
>
> On Sat, 6 Jun 2020 at 15:31, Daniel Schwierzeck
> <daniel.schwierzeck at gmail.com> wrote:
> >
> > checkpatch.pl now supports a --u-boot option for U-Boot specific
> > checks. Use that in patman to check the patch series.
> >
> > Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck at gmail.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 795b519314..7f507154b8 100644
> > --- a/tools/patman/checkpatch.py
> > +++ b/tools/patman/checkpatch.py
> > @@ -64,7 +64,7 @@ def CheckPatch(fname, verbose=False):
> >      result.problems = []
> >      chk = FindCheckPatch()
> >      item = {}
> > -    result.stdout = command.Output(chk, '--no-tree', fname,
> > +    result.stdout = command.Output(chk, '--no-tree', '--u-boot', fname,
> >                                     raise_on_error=False)
>
> Can we make this conditional on something? Maybe detecting a U-Boot
> tree? Patman is used for linux and Zephyr, for example.

Actually I just remembered that I was hoping we could just adjust the
.checkpatch.conf file in the U-Boot directory to add the option. Would
that work?

>
> >      #pipe = subprocess.Popen(cmd, stdout=subprocess.PIPE)
> >      #stdout, stderr = pipe.communicate()
> > --
> > 2.27.0
> >

Regards,
Simon