Messages in this thread | | | Date | Sun, 20 Jan 2013 12:47:29 -0800 | From | Anton Vorontsov <> | Subject | Re: [PATCH] power: reset: qnap-poweroff: Fix License String |
| |
On Sun, Jan 20, 2013 at 09:13:36PM +0100, Uwe Kleine-König wrote: > On Tue, Jan 08, 2013 at 07:15:26PM +0100, Andrew Lunn wrote: > > GPLv2+ is not a valid license string. Replace it with one that is. > > > > Signed-off-by: Andrew Lunn <andrew@lunn.ch> > > --- > > drivers/power/reset/qnap-poweroff.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/power/reset/qnap-poweroff.c b/drivers/power/reset/qnap-poweroff.c > > index ca0b476..8af772b 100644 > > --- a/drivers/power/reset/qnap-poweroff.c > > +++ b/drivers/power/reset/qnap-poweroff.c > > @@ -121,4 +121,4 @@ module_platform_driver(qnap_power_off_driver); > > > > MODULE_AUTHOR("Andrew Lunn <andrew@lunn.ch>"); > > MODULE_DESCRIPTION("QNAP Power off driver"); > > -MODULE_LICENSE("GPLv2+"); > > +MODULE_LICENSE("GPL v2"); > This change is wrong. > > According to include/linux/module.h "GPL v2" means exactly that: version > 2. As the file specifies v2 or later in the header you have to use "GPL" > which means v2 or later.
Does it even make sense to have the two separate things ("GPL v2" and "GPL")?
Suppose there is a global change that modifies a bunch of drivers, some of them are GPLv2+. Now, the author of the global change is submitting it under "GPL v2 only" license, which, by definition, turns any GPLv2+ code into "GPL v2 only", right?
So, changing from GPLv2+ to "GPL v2 only" is OK, but not the other way around.
IANAL, tho.
Anton
p.s. Yes, in this particular driver it also makes sense to remove "or later" words from the header, just to be consistent. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |