lkml.org 
[lkml]   [2018]   [Feb]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] auxdisplay: img-ascii-lcd: add MODULE_LICENSE("GPL")
On Sun, Feb 25, 2018 at 1:25 PM, Miguel Ojeda
<miguel.ojeda.sandonis@gmail.com> wrote:
> On Sat, Feb 24, 2018 at 4:08 PM, Daniel Axtens <dja@axtens.net> wrote:
>> 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?
>
> drivers/auxdisplay/arm-charlcd.c is missing MODULE_LICENSE. I will
> send the patch for that which should be in for 4.17.

Never mind that, it is builtin.

So all is good for drivers/auxdisplay already.

Cheers,
Miguel

>
> Apart from that one, the rest of drivers/auxdisplay is fine.
>
> Cheers,
> Miguel
>
>>
>> Regards,
>> Daniel
>>>
>>> Cheers,
>>> Miguel
>>>
>>>> --
>>>> 2.11.0
>>>>

\
 
 \ /
  Last update: 2018-02-25 13:41    [W:0.072 / U:0.108 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site