From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965054AbcAZLAK (ORCPT ); Tue, 26 Jan 2016 06:00:10 -0500 Received: from arrakis.dune.hu ([78.24.191.176]:55493 "EHLO arrakis.dune.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932536AbcAZLAG (ORCPT ); Tue, 26 Jan 2016 06:00:06 -0500 Subject: Re: [PATCH V3 2/2] regulator: mt6323: Add support for MT6323 regulator To: Paul Bolle References: <1453722989-41622-1-git-send-email-blogic@openwrt.org> <1453722989-41622-2-git-send-email-blogic@openwrt.org> <1453804890.17181.93.camel@tiscali.nl> Cc: Liam Girdwood , Mark Brown , Chen Zhong , Matthias Brugger , Henry Chen , Steven Liu , linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org From: John Crispin Message-ID: <56A751AF.3030601@openwrt.org> Date: Tue, 26 Jan 2016 11:59:59 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <1453804890.17181.93.camel@tiscali.nl> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 26/01/2016 11:41, Paul Bolle wrote: > 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 > Hi Paul, thanks, good catch. luckily i had not sent the next version of the series yet ;) John John