Dear Stanislav, On Mon, Feb 12, 2018 at 2:06 AM, Jaegeuk Kim <jaegeuk@xxxxxxxxxx> wrote: > On 02/06, Stanislav Nijnikov wrote: >> This patch introduces attribute group to show existing sysfs entries. >> >> Signed-off-by: Stanislav Nijnikov <stanislav.nijnikov@xxxxxxx> <snip> >> --- /dev/null >> +++ b/drivers/scsi/ufs/ufs-sysfs.c >> @@ -0,0 +1,156 @@ >> +// SPDX-License-Identifier: GPL-2.0-only I commend you for using license ids here. But there is an issue with your attempt to use the latest and greatest license ids from SPDX: GPL-2.0-only is not a documented license id in our doc [1] Until this is updated (help welcomed including patching a few 10K+ files) , this should be GPL-2.0 instead to avoid confusion and keep things homogeneous and tidy. FYI, using only the doc as the reference was brought forward by Russell King and Christoph Hellwig and in particular with this: On Thu, Nov 9, 2017 at 5:44 PM, Russell King <rmk@xxxxxxxxxxxxxxx> wrote: > I'd be more comfortable if we could have something in the kernel tree > that identifies the SPDX tags and their meaning, maybe with the > _standard_ file header for that license included, so there is no > argument about what any particular SPDX tag means. Based on this, Thomas Gleixner updated the doc alright following this important point. So, in case of doubt the doc should be the reference for this and nothing else. And if there are issue with the doc, then we can fix it with a patch ;) CC: Russell King <rmk@xxxxxxxxxxxxxxx> CC: Christoph Hellwig <hch@xxxxxxxxxxxxx> CC: Thomas Gleixner <tglx@xxxxxxxxxxxxx> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/license-rules.rst -- Cordially Philippe Ombredanne
