[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1453804890.17181.93.camel@tiscali.nl>
Date: Tue, 26 Jan 2016 11:41:30 +0100
From: Paul Bolle <pebolle@...cali.nl>
To: John Crispin <blogic@...nwrt.org>
Cc: Liam Girdwood <lgirdwood@...il.com>,
Mark Brown <broonie@...nel.org>,
Chen Zhong <chen.zhong@...iatek.com>,
Matthias Brugger <matthias.bgg@...il.com>,
Henry Chen <henryc.chen@...iatek.com>,
Steven Liu <steven.liu@...iatek.com>,
linux-mediatek@...ts.infradead.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH V3 2/2] regulator: mt6323: Add support for MT6323
regulator
John,
On ma, 2016-01-25 at 12:56 +0100, John Crispin wrote:
> --- /dev/null
> +++ b/drivers/regulator/mt6323-regulator.c
> + * 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.
> +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
Powered by blists - more mailing lists