Re: [RESEND PATCH 0/2] Remove the mistakes detected by the check tools.

From	Mathieu Poirier <mathieu.poirier@linaro.org>
Newsgroups	linux.kernel
Subject	Re: [RESEND PATCH 0/2] Remove the mistakes detected by the check tools.
Date	2016-01-22 18:30 +0100
Message-ID	<qTNH4-7bM-15@gated-at.bofh.it> (permalink)
References	<qSUKB-2IV-9@gated-at.bofh.it> <qTqKv-7ZR-27@gated-at.bofh.it> <qTG2S-20Q-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 22 January 2016 at 02:15, Eric Long <eric.long@linaro.org> wrote:
> On 22 January 2016 at 00:56, Mathieu Poirier <mathieu.poirier@linaro.org> wrote:
>>
>> Please run checkpath.pl on your patches.  Also if the changes are of a
>> different nature, ex. indentation problem and wrong argument to a
>> function, please provide two separate patches.
>>
>
> Hi Mathieu,
>
> I had already ran checkpath.pl, and there was no error and warning.

Using a 4.4 kernel I got the following chackpatch warnings:

mpoirier@t430:~/work/linaro/coresight/kernel-maint$
./scripts/checkpatch.pl
0001-coresight-Remove-the-mistakes-detected-by-sparse-too.patch
0002-coresight-Remove-the-mistakes-detected-by-smatch-too.patch
---------------------------------------------------------------
0001-coresight-Remove-the-mistakes-detected-by-sparse-too.patch
---------------------------------------------------------------
WARNING: A patch subject line should describe the change not the tool
that found it
#4:
Subject: [PATCH 1/2] coresight: Remove the mistakes detected by sparse tool

total: 0 errors, 1 warnings, 32 lines checked

0001-coresight-Remove-the-mistakes-detected-by-sparse-too.patch has
style problems, please review.
---------------------------------------------------------------
0002-coresight-Remove-the-mistakes-detected-by-smatch-too.patch
---------------------------------------------------------------
WARNING: A patch subject line should describe the change not the tool
that found it
#4:
Subject: [PATCH 2/2] coresight: Remove the mistakes detected by smatch tool

total: 0 errors, 1 warnings, 8 lines checked

0002-coresight-Remove-the-mistakes-detected-by-smatch-too.patch has
style problems, please review.

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.
mpoirier@t430:~/work/linaro/coresight/kernel-maint$ ls


> All the changes are either indentation problems or wrong argument,
> except one static defined problem. So I think there is no need to
> separate the patch.

After fixing the above checkpatch warning bunching changes by tools
won't make sense anymore.

> By the way, the patches I had send were created base on the check
> tools, one for smatch and another for sparse. In case I need to separate
> the patches, do I need to follow the this principle. Thanks.

It's usually one semantic change per patch - the tool that highlighted
the problem is secondary.

>
> Best Regards.
> Eric

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[RESEND PATCH 0/2] Remove the mistakes detected by the check tools. Eric Long <eric.long@linaro.org> - 2016-01-20 07:50 +0100
  [RESEND PATCH 2/2] coresight: Remove the mistakes detected by smatch tool Eric Long <eric.long@linaro.org> - 2016-01-20 07:50 +0100
  Re: [RESEND PATCH 0/2] Remove the mistakes detected by the check tools. Mathieu Poirier <mathieu.poirier@linaro.org> - 2016-01-21 18:00 +0100
    Re: [RESEND PATCH 0/2] Remove the mistakes detected by the check tools. Eric Long <eric.long@linaro.org> - 2016-01-22 10:20 +0100
      Re: [RESEND PATCH 0/2] Remove the mistakes detected by the check tools. Mathieu Poirier <mathieu.poirier@linaro.org> - 2016-01-22 18:30 +0100

csiph-web