2018-01-10 16:28:37

by Arnd Bergmann

[permalink] [raw]
Subject: [PATCH] ARM: pxa/tosa-bt: add MODULE_LICENSE tag

Without this tag, we get a build warning:

WARNING: modpost: missing MODULE_LICENSE() in arch/arm/mach-pxa/tosa-bt.o

For completeness, I'm also adding author and description fields.

Signed-off-by: Arnd Bergmann <[email protected]>
---
arch/arm/mach-pxa/tosa-bt.c | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/arch/arm/mach-pxa/tosa-bt.c b/arch/arm/mach-pxa/tosa-bt.c
index 107f37210fb9..83606087edc7 100644
--- a/arch/arm/mach-pxa/tosa-bt.c
+++ b/arch/arm/mach-pxa/tosa-bt.c
@@ -132,3 +132,7 @@ static struct platform_driver tosa_bt_driver = {
},
};
module_platform_driver(tosa_bt_driver);
+
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Dmitry Baryshkov");
+MODULE_DESCRIPTION("Bluetooth built-in chip control");
--
2.9.0


2018-01-10 20:22:24

by Robert Jarzmik

[permalink] [raw]
Subject: Re: [PATCH] ARM: pxa/tosa-bt: add MODULE_LICENSE tag

Arnd Bergmann <[email protected]> writes:

> Without this tag, we get a build warning:
>
> WARNING: modpost: missing MODULE_LICENSE() in arch/arm/mach-pxa/tosa-bt.o
>
> For completeness, I'm also adding author and description fields.
>
> Signed-off-by: Arnd Bergmann <[email protected]>
Queued into pxa/for-next, thanks.

Cheers.

--
Robert

2018-01-19 16:03:34

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH] ARM: pxa/tosa-bt: add MODULE_LICENSE tag

On Wed, Jan 10, 2018 at 9:22 PM, Robert Jarzmik <[email protected]> wrote:
> Arnd Bergmann <[email protected]> writes:
>
>> Without this tag, we get a build warning:
>>
>> WARNING: modpost: missing MODULE_LICENSE() in arch/arm/mach-pxa/tosa-bt.o
>>
>> For completeness, I'm also adding author and description fields.
>>
>> Signed-off-by: Arnd Bergmann <[email protected]>
>
> Queued into pxa/for-next, thanks.

I see now that you didn't send this back to [email protected]. I assume you planned
to send it for 4.17, but kernelci now still reports the warning for
both 4.15 and
linux-next (4.16). Should we just include this patch in the 4.16 fixes instead?

Arnd

2018-01-19 17:20:48

by Robert Jarzmik

[permalink] [raw]
Subject: Re: [PATCH] ARM: pxa/tosa-bt: add MODULE_LICENSE tag

Arnd Bergmann <[email protected]> writes:

> On Wed, Jan 10, 2018 at 9:22 PM, Robert Jarzmik <[email protected]> wrote:
>> Arnd Bergmann <[email protected]> writes:
>>
>>> Without this tag, we get a build warning:
>>>
>>> WARNING: modpost: missing MODULE_LICENSE() in arch/arm/mach-pxa/tosa-bt.o
>>>
>>> For completeness, I'm also adding author and description fields.
>>>
>>> Signed-off-by: Arnd Bergmann <[email protected]>
>>
>> Queued into pxa/for-next, thanks.
>
> I see now that you didn't send this back to [email protected]. I assume you planned
> to send it for 4.17, but kernelci now still reports the warning for
> both 4.15 and
> linux-next (4.16). Should we just include this patch in the 4.16 fixes
> instead?
Yeah sure, go ahead and merge it into 4.16 staging branch, with my :
Acked-by: Robert Jarzmik <[email protected]>

Cheers.

--
Robert

2018-01-22 11:37:14

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH] ARM: pxa/tosa-bt: add MODULE_LICENSE tag

On Fri, Jan 19, 2018 at 6:18 PM, Robert Jarzmik <[email protected]> wrote:
> Arnd Bergmann <[email protected]> writes:
>
>> On Wed, Jan 10, 2018 at 9:22 PM, Robert Jarzmik <[email protected]> wrote:
>>> Arnd Bergmann <[email protected]> writes:
>>>
>>>> Without this tag, we get a build warning:
>>>>
>>>> WARNING: modpost: missing MODULE_LICENSE() in arch/arm/mach-pxa/tosa-bt.o
>>>>
>>>> For completeness, I'm also adding author and description fields.
>>>>
>>>> Signed-off-by: Arnd Bergmann <[email protected]>
>>>
>>> Queued into pxa/for-next, thanks.
>>
>> I see now that you didn't send this back to [email protected]. I assume you planned
>> to send it for 4.17, but kernelci now still reports the warning for
>> both 4.15 and
>> linux-next (4.16). Should we just include this patch in the 4.16 fixes
>> instead?
> Yeah sure, go ahead and merge it into 4.16 staging branch, with my :
> Acked-by: Robert Jarzmik <[email protected]>

Applied to next/soc now, thanks!

Arnd