Hi Daniel,
On Sat, 6 Jun 2020 at 20:02, Simon Glass <s...@chromium.org> wrote:
>
> Hi Daniel,
>
> On Sat, 6 Jun 2020 at 15:31, Daniel Schwierzeck
> <daniel.schwierz...@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.schwierz...@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
