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.

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

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