diff mbox series

[1/9] block: Cleanup license notice

Message ID 20190117231500.874215239@linutronix.de (mailing list archive)
State New, archived
Headers show
Series [1/9] block: Cleanup license notice | expand

Commit Message

Thomas Gleixner Jan. 17, 2019, 11:14 p.m. UTC
Remove the imprecise and sloppy:

  "This files is licensed under the GPL."

license notice in the top level comment.

1) The file already contains a SPDX license identifier which clearly
   states that the license of the file is GPL V2 only

2) The notice resolves to GPL v1 or later for scanners which is just
   contrary to the intent of SPDX identifiers to provide clear and non
   ambiguous license information. Aside of that the value add of this
   notice is below zero,

Fixes: 6a5ac9846508 ("block: Make struct request_queue smaller for CONFIG_BLK_DEV_ZONED=n")
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: Bart Van Assche <bart.vanassche@wdc.com>
Cc: Damien Le Moal <damien.lemoal@wdc.com>
Cc: Matias Bjorling <mb@lightnvm.io>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Jens Axboe <axboe@kernel.dk>
Cc: linux-block@vger.kernel.org
---

P.S.: This patch is part of a larger cleanup, but independent of other
      patches and is intended to be picked up by the maintainer directly.

 block/blk-mq-debugfs-zoned.c |    2 --
 1 file changed, 2 deletions(-)

Comments

Bart Van Assche Jan. 18, 2019, 2:53 a.m. UTC | #1
On 1/17/19 3:14 PM, Thomas Gleixner wrote:
> Remove the imprecise and sloppy:
> 
>    "This files is licensed under the GPL."
> 
> license notice in the top level comment.
> 
> 1) The file already contains a SPDX license identifier which clearly
>     states that the license of the file is GPL V2 only
> 
> 2) The notice resolves to GPL v1 or later for scanners which is just
>     contrary to the intent of SPDX identifiers to provide clear and non
>     ambiguous license information. Aside of that the value add of this
>     notice is below zero,

My intent was to release this under GPL v2, hence:

Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Jens Axboe Jan. 18, 2019, 4:21 a.m. UTC | #2
On 1/17/19 4:14 PM, Thomas Gleixner wrote:
> Remove the imprecise and sloppy:
> 
>   "This files is licensed under the GPL."
> 
> license notice in the top level comment.
> 
> 1) The file already contains a SPDX license identifier which clearly
>    states that the license of the file is GPL V2 only
> 
> 2) The notice resolves to GPL v1 or later for scanners which is just
>    contrary to the intent of SPDX identifiers to provide clear and non
>    ambiguous license information. Aside of that the value add of this
>    notice is below zero,

Applied, thanks Thomas.
diff mbox series

Patch

--- a/block/blk-mq-debugfs-zoned.c
+++ b/block/blk-mq-debugfs-zoned.c
@@ -1,8 +1,6 @@ 
 // SPDX-License-Identifier: GPL-2.0
 /*
  * Copyright (C) 2017 Western Digital Corporation or its affiliates.
- *
- * This file is released under the GPL.
  */
 
 #include <linux/blkdev.h>