<div dir="ltr">I will change the ident in the macro.<div><br></div><div>Thanks,</div><div><br></div><div>Benjamin</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-07-07 9:59 GMT+02:00 Paul Bolle <span dir="ltr"><<a href="mailto:pebolle@tiscali.nl" target="_blank">pebolle@tiscali.nl</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">A nit only, I'm afraid: a license mismatch.<br>
<br>
On ma, 2015-07-06 at 13:40 +0200, Benjamin Gaignard wrote:<br>
> --- /dev/null<br>
> +++ b/drivers/smaf/smaf-core.c<br>
<span class=""><br>
> + * License terms:  GNU General Public License (GPL), version 2<br>
<br>
</span>> +MODULE_LICENSE("GPL");<br>
<br>
The comment at the top of this file states, succinctly, that the license<br>
is GPL v2. And, according to include/linux/module.h, the<br>
MODULE_LICENSE() macro here states that the license is GPL v2 or later.<br>
So I think that either that comment or the ident used in that macro<br>
needs to change.<br>
<br>
Ditto for 2/2.<br>
<br>
Thanks,<br>
<br>
<br>
Paul Bolle<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><span style="border-collapse:collapse;font-family:arial,sans-serif;font-size:13px"><p style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px">Benjamin Gaignard </p><p style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px">Graphic Working Group</p><p style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><span lang="EN-US" style="font-size:10pt;color:rgb(0,176,80)"><span style="color:rgb(0,68,252)"><a href="http://www.linaro.org/" style="color:rgb(0,0,204)" target="_blank">Linaro.org</a></span><b> </b></span><b><span lang="EN-US" style="font-size:10pt">│ </span></b><span lang="EN-US" style="font-size:10pt">Open source software for ARM SoCs</span></p><p style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><u></u></p><p style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><span lang="EN-US" style="font-size:10pt">Follow <b>Linaro: </b></span><span style="font-size:10pt;color:rgb(0,68,252)"><a href="http://www.facebook.com/pages/Linaro" style="color:rgb(0,0,204)" target="_blank"><span style="color:blue">Facebook</span></a></span><span style="font-size:10pt"> | <span style="color:rgb(0,68,252)"><a href="http://twitter.com/#!/linaroorg" style="color:rgb(0,0,204)" target="_blank"><span style="color:blue">Twitter</span></a></span> | <span style="color:rgb(0,68,252)"><a href="http://www.linaro.org/linaro-blog/" style="color:rgb(0,0,204)" target="_blank"><span style="color:blue">Blog</span></a></span></span></p></span></div>
</div>