[v2,06/14] thermal: mvebu: Fix license declaration
Eduardo Valentin
eduardo.valentin at ti.com
Thu Apr 11 14:55:57 EDT 2013
Ezequiel,
On 11-04-2013 12:40, Eduardo Valentin wrote:
> Hi Rui,
>
> On 22-03-2013 18:25, Ezequiel Garcia wrote:
>> The license text declared in the header explicit says GPL v2 only,
>> but the MODULE_LICENSE macro says GPL, which means GPL v2 or later.
>> Fix it.
>>
>> Signed-off-by: Ezequiel Garcia <ezequiel.garcia at free-electrons.com>
>>
>
> This specific patch is fine to me.
>
> Acked-by: Eduardo Valentin <eduardo.valentin at ti.com>
>
I know you have discarded the idea of merging the driver into a single
driver, but can you please resend this patch on top of
drivers/thermal/kirkwood_thermal.c file. This change still applies.
>> ---
>> drivers/thermal/mvebu_thermal.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/thermal/mvebu_thermal.c
>> b/drivers/thermal/mvebu_thermal.c
>> index 2aeaf75..0c9344d 100644
>> --- a/drivers/thermal/mvebu_thermal.c
>> +++ b/drivers/thermal/mvebu_thermal.c
>> @@ -128,4 +128,4 @@ module_platform_driver(mvebu_thermal_driver);
>>
>> MODULE_AUTHOR("Nobuhiro Iwamatsu <iwamatsu at nigauri.org>");
>> MODULE_DESCRIPTION("mvebu thermal driver");
>> -MODULE_LICENSE("GPL");
>> +MODULE_LICENSE("GPL v2");
>>
>
> Side question, while here:
> Ezequiel, does this driver need a MODULE_ALIAS
>
>
>
More information about the linux-arm-kernel
mailing list