[v1] backlight: pwm_bl: convert to use SPDX identifier
diff mbox series

Message ID 20190619135927.29745-1-andriy.shevchenko@linux.intel.com
State New
Headers show
Series
  • [v1] backlight: pwm_bl: convert to use SPDX identifier
Related show

Commit Message

Andy Shevchenko June 19, 2019, 1:59 p.m. UTC
Reduce size of duplicated comments by switching to use SPDX identifier.

No functional change.

While here, correct MODULE_LICENSE() string to be aligned with license text.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/video/backlight/pwm_bl.c | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

Comments

Daniel Thompson June 20, 2019, 2:14 p.m. UTC | #1
On 19/06/2019 14:59, Andy Shevchenko wrote:
> Reduce size of duplicated comments by switching to use SPDX identifier.
> 
> No functional change.
> 
> While here, correct MODULE_LICENSE() string to be aligned with license text.
> 
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Acked-by: Daniel Thompson <daniel.thompson@linaro.org>


> ---
>   drivers/video/backlight/pwm_bl.c | 11 +++--------
>   1 file changed, 3 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c
> index fb45f866b923..1f7f8d5c0bf1 100644
> --- a/drivers/video/backlight/pwm_bl.c
> +++ b/drivers/video/backlight/pwm_bl.c
> @@ -1,13 +1,8 @@
> +// SPDX-License-Identifier: GPL-2.0
>   /*
> - * linux/drivers/video/backlight/pwm_bl.c
> - *
> - * simple PWM based backlight control, board code has to setup
> + * Simple PWM based backlight control, board code has to setup
>    * 1) pin configuration so PWM waveforms can output
>    * 2) platform_data being correctly configured
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License version 2 as
> - * published by the Free Software Foundation.
>    */
>   
>   #include <linux/delay.h>
> @@ -708,5 +703,5 @@ static struct platform_driver pwm_backlight_driver = {
>   module_platform_driver(pwm_backlight_driver);
>   
>   MODULE_DESCRIPTION("PWM based Backlight Driver");
> -MODULE_LICENSE("GPL");
> +MODULE_LICENSE("GPL v2");
>   MODULE_ALIAS("platform:pwm-backlight");
>
Lee Jones June 26, 2019, 10:56 a.m. UTC | #2
On Wed, 19 Jun 2019, Andy Shevchenko wrote:

> Reduce size of duplicated comments by switching to use SPDX identifier.
> 
> No functional change.
> 
> While here, correct MODULE_LICENSE() string to be aligned with license text.
> 
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
>  drivers/video/backlight/pwm_bl.c | 11 +++--------
>  1 file changed, 3 insertions(+), 8 deletions(-)

Applied, thanks.

Patch
diff mbox series

diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c
index fb45f866b923..1f7f8d5c0bf1 100644
--- a/drivers/video/backlight/pwm_bl.c
+++ b/drivers/video/backlight/pwm_bl.c
@@ -1,13 +1,8 @@ 
+// SPDX-License-Identifier: GPL-2.0
 /*
- * linux/drivers/video/backlight/pwm_bl.c
- *
- * simple PWM based backlight control, board code has to setup
+ * Simple PWM based backlight control, board code has to setup
  * 1) pin configuration so PWM waveforms can output
  * 2) platform_data being correctly configured
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
  */
 
 #include <linux/delay.h>
@@ -708,5 +703,5 @@  static struct platform_driver pwm_backlight_driver = {
 module_platform_driver(pwm_backlight_driver);
 
 MODULE_DESCRIPTION("PWM based Backlight Driver");
-MODULE_LICENSE("GPL");
+MODULE_LICENSE("GPL v2");
 MODULE_ALIAS("platform:pwm-backlight");