On 26/01/2016 11:41, Paul Bolle wrote:

John,

On ma, 2016-01-25 at 12:56 +0100, John Crispin wrote:

quoted

--- /dev/null
+++ b/drivers/regulator/mt6323-regulator.c

quoted

+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.

This states this file is licensed GPL v2 only.

quoted

+MODULE_LICENSE("GPL");

And, according to include/linux/module.h, this means "GNU Public License
v2 or later".

So I think there's a (subtle) mismatch between the license ident used
for this driver and the comment above.

Thanks,


Paul Bolle

Hi Paul,

thanks, good catch. luckily i had not sent the next version of the
series yet ;)

	John

	John