gpio: ath79: add missing MODULE_DESCRIPTION/LICENSE

PATCH In review Jesse Chan 3 participants 3 messages Nov 20, 2017 – Nov 30, 2017

Subsystems (2)

GPIO Atheros 71XX/9XXX GPIO driver

lore

Share & tools

lore

Jesse Chan

author

2017-11-20 20:54 UTC #0 lore raw

PATCH 1/1 gpio: ath79: add missing MODULE_DESCRIPTION/LICENSE

This change resolves a new compile-time warning
when built as a loadable module:

WARNING: modpost: missing MODULE_LICENSE() in drivers/gpio/gpio-ath79.o see include/linux/module.h for more information

This adds the license as "GPL v2", which matches the header of the file.

MODULE_DESCRIPTION is also added.

Signed-off-by Jesse Chan <[email protected]>

---
 drivers/gpio/gpio-ath79.c | 3 +++
 1 file changed, 3 insertions(+)

drivers/gpio/gpio-ath79.c +3 -0

index 5fad89dfab7e..3ae7c1876bf4 100644
@@ -324,3 +324,6 @@ static struct platform_driver ath79_gpio_driver = {
 };
 
 module_platform_driver(ath79_gpio_driver);
+
+MODULE_DESCRIPTION("Atheros AR71XX/AR724X/AR913X GPIO API support");
+MODULE_LICENSE("GPL v2");

Signature · 1 line

2.14.1

Alban

maintainer Acked-by on v1 1/1

2017-11-20 21:55 UTC #1 lore raw

Quoted 11 lines · On Mon, 20 Nov 2017 12:54:26 -0800 Jesse Chan <[email protected]> wrote:

This change resolves a new compile-time warning
when built as a loadable module:
WARNING: modpost: missing MODULE_LICENSE() in drivers/gpio/gpio-ath79.o see include/linux/module.h for more information
This adds the license as "GPL v2", which matches the header of the file.
MODULE_DESCRIPTION is also added.
Signed-off-by Jesse Chan <[email protected]>

Acked-by Alban Bedel <[email protected]>

Alban

Linus Walleij

2017-11-30 13:45 UTC #2 lore raw

In reply to Jesse Chan

Quoted 11 lines · Jesse Chan

This change resolves a new compile-time warning
when built as a loadable module:
WARNING: modpost: missing MODULE_LICENSE() in drivers/gpio/gpio-ath79.o see include/linux/module.h for more information
This adds the license as "GPL v2", which matches the header of the file.
MODULE_DESCRIPTION is also added.
Signed-off-by Jesse Chan <[email protected]>

Patch applied with ACK.

Yours,
Linus Walleij