[PATCH] patman: use new --u-boot option with checkpath.pl
Daniel Schwierzeck
daniel.schwierzeck at gmail.com
Sun Jun 7 16:14:59 CEST 2020
Am 07.06.20 um 05:44 schrieb 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?
>
hm, missed that earlier but you already added that with commit
b77df5980c3e24e34e0db3a52c5bce396a81c430. Then you can ignore this patch ;)
--
- Daniel
More information about the U-Boot
mailing list