lkml.org 
[lkml]   [2018]   [Feb]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] auxdisplay: img-ascii-lcd: add MODULE_LICENSE("GPL")
Date
Miguel Ojeda <miguel.ojeda.sandonis@gmail.com> writes:

> Hi Daniel,
>
> On Sun, Nov 26, 2017 at 3:53 AM, Daniel Axtens <dja@axtens.net> wrote:
>> This matches the header at the top of the file and squashes:
>>
>> WARNING: modpost: missing MODULE_LICENSE() in drivers/auxdisplay/img-ascii-lcd.o
>> see include/linux/module.h for more information
>>
>> Signed-off-by: Daniel Axtens <dja@axtens.net>
>> ---
>> drivers/auxdisplay/img-ascii-lcd.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/auxdisplay/img-ascii-lcd.c b/drivers/auxdisplay/img-ascii-lcd.c
>> index db040b378224..7315ffa79537 100644
>> --- a/drivers/auxdisplay/img-ascii-lcd.c
>> +++ b/drivers/auxdisplay/img-ascii-lcd.c
>> @@ -441,3 +441,5 @@ static struct platform_driver img_ascii_lcd_driver = {
>> .remove = img_ascii_lcd_remove,
>> };
>> module_platform_driver(img_ascii_lcd_driver);
>> +
>> +MODULE_LICENSE("GPL");
>
> This was submitted just a few days before yours, and now it is in at
> 09c479f7f1fbfaf848e5813996793966cd50be81
> ("auxdisplay: img-ascii-lcd: add missing
> MODULE_DESCRIPTION/AUTHOR/LICENSE").
>
> In any case, thanks for the patch! :-)

Thanks!

I was submitting the patches in part because I was considering
submitting a change that made the license warning a fatal error and I
wanted to make sure I had covered the obvious missing ones first.

Would making the warning fatal impact on you?

Regards,
Daniel
>
> Cheers,
> Miguel
>
>> --
>> 2.11.0
>>

\
 
 \ /
  Last update: 2018-02-24 16:09    [W:0.060 / U:2.888 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site