2014-07-15 23:25:00

by Bjorn Helgaas

[permalink] [raw]
Subject: [PATCH] power/reset: Fix GPL v2 license string typo

Per license_is_gpl_compatible(), the MODULE_LICENSE() string for GPL v2 is
"GPL v2", not "GPLv2". Use "GPL v2" so this module doesn't taint the
kernel.

Signed-off-by: Bjorn Helgaas <[email protected]>
---
drivers/power/reset/restart-poweroff.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/power/reset/restart-poweroff.c b/drivers/power/reset/restart-poweroff.c
index 5758033e0c16..3e51f8d29bfe 100644
--- a/drivers/power/reset/restart-poweroff.c
+++ b/drivers/power/reset/restart-poweroff.c
@@ -62,5 +62,5 @@ module_platform_driver(restart_poweroff_driver);

MODULE_AUTHOR("Andrew Lunn <[email protected]");
MODULE_DESCRIPTION("restart poweroff driver");
-MODULE_LICENSE("GPLv2");
+MODULE_LICENSE("GPL v2");
MODULE_ALIAS("platform:poweroff-restart");


2014-07-16 13:51:31

by Andrew Lunn

[permalink] [raw]
Subject: Re: [PATCH] power/reset: Fix GPL v2 license string typo

On Tue, Jul 15, 2014 at 05:24:50PM -0600, Bjorn Helgaas wrote:
> Per license_is_gpl_compatible(), the MODULE_LICENSE() string for GPL v2 is
> "GPL v2", not "GPLv2". Use "GPL v2" so this module doesn't taint the
> kernel.
>
> Signed-off-by: Bjorn Helgaas <[email protected]>

Acked-by: Andrew Lunn <[email protected]>

Andrew

> ---
> drivers/power/reset/restart-poweroff.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/power/reset/restart-poweroff.c b/drivers/power/reset/restart-poweroff.c
> index 5758033e0c16..3e51f8d29bfe 100644
> --- a/drivers/power/reset/restart-poweroff.c
> +++ b/drivers/power/reset/restart-poweroff.c
> @@ -62,5 +62,5 @@ module_platform_driver(restart_poweroff_driver);
>
> MODULE_AUTHOR("Andrew Lunn <[email protected]");
> MODULE_DESCRIPTION("restart poweroff driver");
> -MODULE_LICENSE("GPLv2");
> +MODULE_LICENSE("GPL v2");
> MODULE_ALIAS("platform:poweroff-restart");
>

2014-07-16 18:42:05

by Sebastian Reichel

[permalink] [raw]
Subject: Re: [PATCH] power/reset: Fix GPL v2 license string typo

On Tue, Jul 15, 2014 at 05:24:50PM -0600, Bjorn Helgaas wrote:
> Per license_is_gpl_compatible(), the MODULE_LICENSE() string for GPL v2 is
> "GPL v2", not "GPLv2". Use "GPL v2" so this module doesn't taint the
> kernel.
>
> Signed-off-by: Bjorn Helgaas <[email protected]>

I put this into my dev branch on kernel.org, which I use to queue
patches until I get access to the battery.git repository:

https://git.kernel.org/cgit/linux/kernel/git/sre/linux-power-supply.git/commit/?h=dev&id=162c1a01825553e055ef3867f7598ca8dddb6c6e

-- Sebastian

Attachments:
(No filename) (548.00 B)
 signature.asc (819.00 B)
Digital signature Download all attachments