linux-kernel - Re: [PATCH 1/2] Input: add support for Semtech SX8654 I2C touchscreen controller

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sat, 7 Mar 2015 13:25:43 -0800
From:	Dmitry Torokhov <dmitry.torokhov@...il.com>
To:	Paul Bolle <pebolle@...cali.nl>
Cc:	Sébastien Szymanski 
	<sebastien.szymanski@...adeus.com>, linux-input@...r.kernel.org,
	linux-kernel@...r.kernel.org, Rob Herring <robh+dt@...nel.org>,
	Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	devicetree@...r.kernel.org, Kumar Gala <galak@...eaurora.org>
Subject: Re: [PATCH 1/2] Input: add support for Semtech SX8654 I2C
 touchscreen controller

On Sat, Mar 07, 2015 at 01:32:25PM +0100, Paul Bolle wrote:
> Dmitry Torokhov schreef op vr 06-03-2015 om 16:57 [-0800]:
> > Applied with some cosmetic edits and DT bindings folded into this patch.
> 
> Was one of these cosmetic edits perhaps an edit to use
>     MODULE_LICENSE("GPL v2");
> 
> in sx8654.c?

No it wasn't.

> That would make that macro match the license stated in the
> comment at the top of that file.

I am not sure if anyone cares about exact version of GPL in module
information (2 only vs 2+) since it only used to figure out if the
module taints kernel or not. In fact there are more modules that are v2
only that claim GPL than the ones claiming GPL v2.

dtor@...r-ws:~/kernel/master$ for file in `grep -r -l 'MODULE_LICENSE("GPL")'`; do grep -H '2 as published' $file; done | wc -l
259
dtor@...r-ws:~/kernel/master$ for file in `grep -r -l 'MODULE_LICENSE("GPL v2")'`; do grep -H '2 as published' $file; do ne | wc -l
150

Also:

dtor@...r-ws:~/kernel/master$ for file in `grep -r -l 'MODULE_LICENSE("GPL v2")'`; do grep -H '2 or ' $file; done | wc -l
68
dtor@...r-ws:~/kernel/master$ for file in `grep -r -l 'MODULE_LICENSE("GPL")'`; do grep -H '2 or ' $file; done | wc -l
237

Thanks.

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/