Re: [PATCH] auxdisplay: img-ascii-lcd: add MODULE_LICENSE("GPL")

From: Miguel Ojeda
Date: Sun Feb 25 2018 - 07:25:50 EST


On Sat, Feb 24, 2018 at 4:08 PM, Daniel Axtens <dja@xxxxxxxxxx> wrote:
> Miguel Ojeda <miguel.ojeda.sandonis@xxxxxxxxx> writes:
>
>> Hi Daniel,
>>
>> On Sun, Nov 26, 2017 at 3:53 AM, Daniel Axtens <dja@xxxxxxxxxx> 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@xxxxxxxxxx>
>>> ---
>>> 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.

Apart from that one, the rest of drivers/auxdisplay is fine.

Cheers,
Miguel

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