Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2018-02-08 Thread Philippe Ombredanne
On Thu, Feb 1, 2018 at 10:14 PM, Rob Herring wrote: > Add SPDX license tag check based on the rules defined in > Documentation/process/license-rules.rst. To summarize, SPDX license tags > should be on the 1st line (or 2nd line in scripts) using the appropriate > comment style for

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2018-02-08 Thread Philippe Ombredanne
On Thu, Feb 1, 2018 at 10:14 PM, Rob Herring wrote: > Add SPDX license tag check based on the rules defined in > Documentation/process/license-rules.rst. To summarize, SPDX license tags > should be on the 1st line (or 2nd line in scripts) using the appropriate > comment style for the file type. >

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2018-02-01 Thread Greg Kroah-Hartman
On Thu, Feb 01, 2018 at 03:14:29PM -0600, Rob Herring wrote: > Add SPDX license tag check based on the rules defined in > Documentation/process/license-rules.rst. To summarize, SPDX license tags > should be on the 1st line (or 2nd line in scripts) using the appropriate > comment style for the file

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2018-02-01 Thread Greg Kroah-Hartman
On Thu, Feb 01, 2018 at 03:14:29PM -0600, Rob Herring wrote: > Add SPDX license tag check based on the rules defined in > Documentation/process/license-rules.rst. To summarize, SPDX license tags > should be on the 1st line (or 2nd line in scripts) using the appropriate > comment style for the file

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2018-02-01 Thread Joe Perches
On Thu, 2018-02-01 at 15:14 -0600, Rob Herring wrote: > Add SPDX license tag check based on the rules defined in > Documentation/process/license-rules.rst. To summarize, SPDX license tags > should be on the 1st line (or 2nd line in scripts) using the appropriate > comment style for the file type.

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2018-02-01 Thread Joe Perches
On Thu, 2018-02-01 at 15:14 -0600, Rob Herring wrote: > Add SPDX license tag check based on the rules defined in > Documentation/process/license-rules.rst. To summarize, SPDX license tags > should be on the 1st line (or 2nd line in scripts) using the appropriate > comment style for the file type.

[PATCH] checkpatch.pl: Add SPDX license tag check

2018-02-01 Thread Rob Herring
Add SPDX license tag check based on the rules defined in Documentation/process/license-rules.rst. To summarize, SPDX license tags should be on the 1st line (or 2nd line in scripts) using the appropriate comment style for the file type. Cc: Andy Whitcroft Cc: Joe Perches

[PATCH] checkpatch.pl: Add SPDX license tag check

2018-02-01 Thread Rob Herring
Add SPDX license tag check based on the rules defined in Documentation/process/license-rules.rst. To summarize, SPDX license tags should be on the 1st line (or 2nd line in scripts) using the appropriate comment style for the file type. Cc: Andy Whitcroft Cc: Joe Perches Cc: Greg Kroah-Hartman

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Joe Perches
On Thu, 2017-11-09 at 12:12 -0600, Rob Herring wrote: > On Thu, Nov 9, 2017 at 9:39 AM, Joe Perches wrote: > > On Thu, 2017-11-09 at 07:47 -0600, Rob Herring wrote: > > > On Wed, Nov 8, 2017 at 8:10 PM, Joe Perches wrote: > > > > On Wed, 2017-11-08 at 19:10

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Joe Perches
On Thu, 2017-11-09 at 12:12 -0600, Rob Herring wrote: > On Thu, Nov 9, 2017 at 9:39 AM, Joe Perches wrote: > > On Thu, 2017-11-09 at 07:47 -0600, Rob Herring wrote: > > > On Wed, Nov 8, 2017 at 8:10 PM, Joe Perches wrote: > > > > On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: > > > > >

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Rob Herring
On Thu, Nov 9, 2017 at 9:39 AM, Joe Perches wrote: > On Thu, 2017-11-09 at 07:47 -0600, Rob Herring wrote: >> On Wed, Nov 8, 2017 at 8:10 PM, Joe Perches wrote: >> > On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: >> > > Add a check warning if

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Rob Herring
On Thu, Nov 9, 2017 at 9:39 AM, Joe Perches wrote: > On Thu, 2017-11-09 at 07:47 -0600, Rob Herring wrote: >> On Wed, Nov 8, 2017 at 8:10 PM, Joe Perches wrote: >> > On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: >> > > Add a check warning if SPDX-License-Identifier tags are not used in

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Joe Perches
On Thu, 2017-11-09 at 07:47 -0600, Rob Herring wrote: > On Wed, Nov 8, 2017 at 8:10 PM, Joe Perches wrote: > > On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: > > > Add a check warning if SPDX-License-Identifier tags are not used in > > > newly added files. > > > > If

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Joe Perches
On Thu, 2017-11-09 at 07:47 -0600, Rob Herring wrote: > On Wed, Nov 8, 2017 at 8:10 PM, Joe Perches wrote: > > On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: > > > Add a check warning if SPDX-License-Identifier tags are not used in > > > newly added files. > > > > If this is to be done,

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Joe Perches
On Thu, 2017-11-09 at 10:12 +0100, Greg Kroah-Hartman wrote: > On Wed, Nov 08, 2017 at 06:10:19PM -0800, Joe Perches wrote: > > On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: > > > Add a check warning if SPDX-License-Identifier tags are not used in > > > newly added files. > > > > If this

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Joe Perches
On Thu, 2017-11-09 at 10:12 +0100, Greg Kroah-Hartman wrote: > On Wed, Nov 08, 2017 at 06:10:19PM -0800, Joe Perches wrote: > > On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: > > > Add a check warning if SPDX-License-Identifier tags are not used in > > > newly added files. > > > > If this

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Rob Herring
On Wed, Nov 8, 2017 at 8:10 PM, Joe Perches wrote: > On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: >> Add a check warning if SPDX-License-Identifier tags are not used in >> newly added files. > > If this is to be done, and I think it's not a great idea, Which part? SPDX

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Rob Herring
On Wed, Nov 8, 2017 at 8:10 PM, Joe Perches wrote: > On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: >> Add a check warning if SPDX-License-Identifier tags are not used in >> newly added files. > > If this is to be done, and I think it's not a great idea, Which part? SPDX tags or checking

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Greg Kroah-Hartman
On Wed, Nov 08, 2017 at 06:10:19PM -0800, Joe Perches wrote: > On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: > > Add a check warning if SPDX-License-Identifier tags are not used in > > newly added files. > > If this is to be done, and I think it's not a great idea, > there are better ways

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-09 Thread Greg Kroah-Hartman
On Wed, Nov 08, 2017 at 06:10:19PM -0800, Joe Perches wrote: > On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: > > Add a check warning if SPDX-License-Identifier tags are not used in > > newly added files. > > If this is to be done, and I think it's not a great idea, > there are better ways

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-08 Thread Joe Perches
On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: > Add a check warning if SPDX-License-Identifier tags are not used in > newly added files. If this is to be done, and I think it's not a great idea, there are better ways of doing this that emit this warning on a per-file basis instead of a

Re: [PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-08 Thread Joe Perches
On Wed, 2017-11-08 at 19:10 -0600, Rob Herring wrote: > Add a check warning if SPDX-License-Identifier tags are not used in > newly added files. If this is to be done, and I think it's not a great idea, there are better ways of doing this that emit this warning on a per-file basis instead of a

[PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-08 Thread Rob Herring
Add a check warning if SPDX-License-Identifier tags are not used in newly added files. Cc: Andy Whitcroft Cc: Joe Perches Cc: Greg Kroah-Hartman Signed-off-by: Rob Herring --- I rewrote my previous version to

[PATCH] checkpatch.pl: Add SPDX license tag check

2017-11-08 Thread Rob Herring
Add a check warning if SPDX-License-Identifier tags are not used in newly added files. Cc: Andy Whitcroft Cc: Joe Perches Cc: Greg Kroah-Hartman Signed-off-by: Rob Herring --- I rewrote my previous version to check more than just dts files. It also now looks for a tag in added files rather

Re: [PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-06-09 Thread Rob Herring
On Wed, Jun 07, 2017 at 05:45:03PM +0900, Masahiro Yamada wrote: > Hi Rob, > > 2017-02-24 0:56 GMT+09:00 Rob Herring : > > Add a check for using SPDX-License-Identifier tags to define the license of > > .dts{i} and DT header files rather than using free form license text. This >

Re: [PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-06-09 Thread Rob Herring
On Wed, Jun 07, 2017 at 05:45:03PM +0900, Masahiro Yamada wrote: > Hi Rob, > > 2017-02-24 0:56 GMT+09:00 Rob Herring : > > Add a check for using SPDX-License-Identifier tags to define the license of > > .dts{i} and DT header files rather than using free form license text. This > > check looks for

Re: [PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-06-07 Thread Masahiro Yamada
Hi Rob, 2017-02-24 0:56 GMT+09:00 Rob Herring : > Add a check for using SPDX-License-Identifier tags to define the license of > .dts{i} and DT header files rather than using free form license text. This > check looks for GPL, BSD, or X11(really incorrectly labeled MIT license) >

Re: [PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-06-07 Thread Masahiro Yamada
Hi Rob, 2017-02-24 0:56 GMT+09:00 Rob Herring : > Add a check for using SPDX-License-Identifier tags to define the license of > .dts{i} and DT header files rather than using free form license text. This > check looks for GPL, BSD, or X11(really incorrectly labeled MIT license) > license text

Re: [PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-02-23 Thread Joe Perches
On Thu, 2017-02-23 at 12:45 -0600, Rob Herring wrote: > On Thu, Feb 23, 2017 at 11:51 AM, Joe Perches wrote: > > On Thu, 2017-02-23 at 09:56 -0600, Rob Herring wrote: > > > Add a check for using SPDX-License-Identifier tags to define the license > > > of > > > .dts{i} and DT

Re: [PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-02-23 Thread Joe Perches
On Thu, 2017-02-23 at 12:45 -0600, Rob Herring wrote: > On Thu, Feb 23, 2017 at 11:51 AM, Joe Perches wrote: > > On Thu, 2017-02-23 at 09:56 -0600, Rob Herring wrote: > > > Add a check for using SPDX-License-Identifier tags to define the license > > > of > > > .dts{i} and DT header files rather

Re: [PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-02-23 Thread Rob Herring
On Thu, Feb 23, 2017 at 11:51 AM, Joe Perches wrote: > On Thu, 2017-02-23 at 09:56 -0600, Rob Herring wrote: >> Add a check for using SPDX-License-Identifier tags to define the license of >> .dts{i} and DT header files rather than using free form license text. This >> check

Re: [PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-02-23 Thread Rob Herring
On Thu, Feb 23, 2017 at 11:51 AM, Joe Perches wrote: > On Thu, 2017-02-23 at 09:56 -0600, Rob Herring wrote: >> Add a check for using SPDX-License-Identifier tags to define the license of >> .dts{i} and DT header files rather than using free form license text. This >> check looks for GPL, BSD, or

Re: [PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-02-23 Thread Joe Perches
On Thu, 2017-02-23 at 09:56 -0600, Rob Herring wrote: > Add a check for using SPDX-License-Identifier tags to define the license of > .dts{i} and DT header files rather than using free form license text. This > check looks for GPL, BSD, or X11(really incorrectly labeled MIT license) > license text

Re: [PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-02-23 Thread Joe Perches
On Thu, 2017-02-23 at 09:56 -0600, Rob Herring wrote: > Add a check for using SPDX-License-Identifier tags to define the license of > .dts{i} and DT header files rather than using free form license text. This > check looks for GPL, BSD, or X11(really incorrectly labeled MIT license) > license text

[PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-02-23 Thread Rob Herring
Add a check for using SPDX-License-Identifier tags to define the license of .dts{i} and DT header files rather than using free form license text. This check looks for GPL, BSD, or X11(really incorrectly labeled MIT license) license text which are the commonly used DT licenses. Signed-off-by: Rob

[PATCH] checkpatch.pl: Add SPDX license tag check for dts files and headers

2017-02-23 Thread Rob Herring
Add a check for using SPDX-License-Identifier tags to define the license of .dts{i} and DT header files rather than using free form license text. This check looks for GPL, BSD, or X11(really incorrectly labeled MIT license) license text which are the commonly used DT licenses. Signed-off-by: Rob