public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH V2] mfd: da9063: Update author information to remove incorrect e-mail addresses
@ 2016-08-08 13:16 Steve Twiss
  2016-08-09 12:49 ` Lee Jones
  0 siblings, 1 reply; 2+ messages in thread
From: Steve Twiss @ 2016-08-08 13:16 UTC (permalink / raw)
  To: LINUXKERNEL, Lee Jones

From: Steve Twiss <stwiss.opensource@diasemi.com>

Remove incorrect e-mail addresses from the copyright header and
MODULE_AUTHOR() macro. These e-mail addresses are no longer in use.

The author names have not been changed, only the e-mail addresses have
been deleted from the source files.

Signed-off-by: Steve Twiss <stwiss.opensource@diasemi.com>

---
Patch History
-------------
PATCH V2
- Also include company name. Change now of the form: <NAME>, <COMPANY>
  e.g. Krystian Garbaciak, Dialog Semiconductor
- I have *not* changed the MODULE_AUTHOR() information to include the
  company name, this still removes the old, invalid, e-mail address.

Hi Lee, 

I have completed the changes to add the company name to the copyright
headers, however, I have *not* changed any MODULE_AUTHOR() macros
to include the company name.

The alterations to the drivers/mfd/da9063-core.c file include the removal
of the e-mail addresses only. This is because I have added the company name
at the top of the file in the copyright header and because the
recommendation from the include/linux/module.h file allows just the NAME
to exist (without E-MAIL).

https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/include/linux/module.h

If that is still not acceptible, please let me know.

Just to re-iterate our point of view. Our intention with this change is
to encourage the group contact e-mail address for any support questions.
Not to mess with copyright or author information.

We have an e-mail address in the MAINTAINERS file covering all of the
developers under a single contact group, support.opensource@diasemi.com,
but that is not acceptable for individual submissions where a single person
is responsible. 

Regards,
Steve


 drivers/mfd/da9063-core.c            | 7 ++++---
 drivers/mfd/da9063-i2c.c             | 2 +-
 drivers/mfd/da9063-irq.c             | 2 +-
 include/linux/mfd/da9063/core.h      | 4 ++--
 include/linux/mfd/da9063/pdata.h     | 4 ++--
 include/linux/mfd/da9063/registers.h | 4 ++--
 6 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/drivers/mfd/da9063-core.c b/drivers/mfd/da9063-core.c
index af841c1..6c2870d 100644
--- a/drivers/mfd/da9063-core.c
+++ b/drivers/mfd/da9063-core.c
@@ -4,8 +4,8 @@
  * Copyright 2012 Dialog Semiconductors Ltd.
  * Copyright 2013 Philipp Zabel, Pengutronix
  *
- * Author: Krystian Garbaciak <krystian.garbaciak@diasemi.com>,
- *         Michal Hajduk <michal.hajduk@diasemi.com>
+ * Author: Krystian Garbaciak, Dialog Semiconductor
+ * Author: Michal Hajduk, Dialog Semiconductor
  *
  *  This program is free software; you can redistribute  it and/or modify it
  *  under  the terms of  the GNU General  Public License as published by the
@@ -242,5 +242,6 @@ void da9063_device_exit(struct da9063 *da9063)
 }
 
 MODULE_DESCRIPTION("PMIC driver for Dialog DA9063");
-MODULE_AUTHOR("Krystian Garbaciak <krystian.garbaciak@diasemi.com>, Michal Hajduk <michal.hajduk@diasemi.com>");
+MODULE_AUTHOR("Krystian Garbaciak");
+MODULE_AUTHOR("Michal Hajduk");
 MODULE_LICENSE("GPL");
diff --git a/drivers/mfd/da9063-i2c.c b/drivers/mfd/da9063-i2c.c
index 7390108..981805a 100644
--- a/drivers/mfd/da9063-i2c.c
+++ b/drivers/mfd/da9063-i2c.c
@@ -3,7 +3,7 @@
  * Copyright 2012 Dialog Semiconductor Ltd.
  * Copyright 2013 Philipp Zabel, Pengutronix
  *
- * Author: Krystian Garbaciak <krystian.garbaciak@diasemi.com>
+ * Author: Krystian Garbaciak, Dialog Semiconductor
  *
  *  This program is free software; you can redistribute  it and/or modify it
  *  under  the terms of  the GNU General  Public License as published by the
diff --git a/drivers/mfd/da9063-irq.c b/drivers/mfd/da9063-irq.c
index 7e903fc..207bbfe 100644
--- a/drivers/mfd/da9063-irq.c
+++ b/drivers/mfd/da9063-irq.c
@@ -3,7 +3,7 @@
  * Copyright 2012 Dialog Semiconductor Ltd.
  * Copyright 2013 Philipp Zabel, Pengutronix
  *
- * Author: Michal Hajduk <michal.hajduk@diasemi.com>
+ * Author: Michal Hajduk, Dialog Semiconductor
  *
  *  This program is free software; you can redistribute  it and/or modify it
  *  under  the terms of  the GNU General  Public License as published by the
diff --git a/include/linux/mfd/da9063/core.h b/include/linux/mfd/da9063/core.h
index 621af82..f3ae65d 100644
--- a/include/linux/mfd/da9063/core.h
+++ b/include/linux/mfd/da9063/core.h
@@ -3,8 +3,8 @@
  *
  * Copyright 2012 Dialog Semiconductor Ltd.
  *
- * Author: Michal Hajduk <michal.hajduk@diasemi.com>
- *	   Krystian Garbaciak <krystian.garbaciak@diasemi.com>
+ * Author: Michal Hajduk, Dialog Semiconductor
+ * Author: Krystian Garbaciak, Dialog Semiconductor
  *
  *  This program is free software; you can redistribute  it and/or modify it
  *  under  the terms of  the GNU General  Public License as published by the
diff --git a/include/linux/mfd/da9063/pdata.h b/include/linux/mfd/da9063/pdata.h
index 612383b..8a12570 100644
--- a/include/linux/mfd/da9063/pdata.h
+++ b/include/linux/mfd/da9063/pdata.h
@@ -3,8 +3,8 @@
  *
  * Copyright 2012 Dialog Semiconductor Ltd.
  *
- * Author: Michal Hajduk <michal.hajduk@diasemi.com>
- * Author: Krystian Garbaciak <krystian.garbaciak@diasemi.com>
+ * Author: Michal Hajduk, Dialog Semiconductor
+ * Author: Krystian Garbaciak, Dialog Semiconductor
  *
  *  This program is free software; you can redistribute  it and/or modify it
  *  under  the terms of  the GNU General  Public License as published by the
diff --git a/include/linux/mfd/da9063/registers.h b/include/linux/mfd/da9063/registers.h
index 2e0ba6d..5d42859 100644
--- a/include/linux/mfd/da9063/registers.h
+++ b/include/linux/mfd/da9063/registers.h
@@ -3,8 +3,8 @@
  *
  * Copyright 2012 Dialog Semiconductor Ltd.
  *
- * Author: Michal Hajduk <michal.hajduk@diasemi.com>
- *	   Krystian Garbaciak <krystian.garbaciak@diasemi.com>
+ * Author: Michal Hajduk, Dialog Semiconductor
+ * Author: Krystian Garbaciak, Dialog Semiconductor
  *
  *  This program is free software; you can redistribute  it and/or modify it
  *  under  the terms of  the GNU General  Public License as published by the
-- 
end-of-patch for PATCH V2

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH V2] mfd: da9063: Update author information to remove incorrect e-mail addresses
  2016-08-08 13:16 [PATCH V2] mfd: da9063: Update author information to remove incorrect e-mail addresses Steve Twiss
@ 2016-08-09 12:49 ` Lee Jones
  0 siblings, 0 replies; 2+ messages in thread
From: Lee Jones @ 2016-08-09 12:49 UTC (permalink / raw)
  To: Steve Twiss; +Cc: LINUXKERNEL

On Mon, 08 Aug 2016, Steve Twiss wrote:

> From: Steve Twiss <stwiss.opensource@diasemi.com>
> 
> Remove incorrect e-mail addresses from the copyright header and
> MODULE_AUTHOR() macro. These e-mail addresses are no longer in use.
> 
> The author names have not been changed, only the e-mail addresses have
> been deleted from the source files.
> 
> Signed-off-by: Steve Twiss <stwiss.opensource@diasemi.com>
> 
> ---
> Patch History
> -------------
> PATCH V2
> - Also include company name. Change now of the form: <NAME>, <COMPANY>
>   e.g. Krystian Garbaciak, Dialog Semiconductor
> - I have *not* changed the MODULE_AUTHOR() information to include the
>   company name, this still removes the old, invalid, e-mail address.
> 
> Hi Lee, 
> 
> I have completed the changes to add the company name to the copyright
> headers, however, I have *not* changed any MODULE_AUTHOR() macros
> to include the company name.
> 
> The alterations to the drivers/mfd/da9063-core.c file include the removal
> of the e-mail addresses only. This is because I have added the company name
> at the top of the file in the copyright header and because the
> recommendation from the include/linux/module.h file allows just the NAME
> to exist (without E-MAIL).
> 
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/include/linux/module.h
> 
> If that is still not acceptible, please let me know.
> 
> Just to re-iterate our point of view. Our intention with this change is
> to encourage the group contact e-mail address for any support questions.
> Not to mess with copyright or author information.
> 
> We have an e-mail address in the MAINTAINERS file covering all of the
> developers under a single contact group, support.opensource@diasemi.com,
> but that is not acceptable for individual submissions where a single person
> is responsible. 
> 
>  drivers/mfd/da9063-core.c            | 7 ++++---
>  drivers/mfd/da9063-i2c.c             | 2 +-
>  drivers/mfd/da9063-irq.c             | 2 +-
>  include/linux/mfd/da9063/core.h      | 4 ++--
>  include/linux/mfd/da9063/pdata.h     | 4 ++--
>  include/linux/mfd/da9063/registers.h | 4 ++--
>  6 files changed, 12 insertions(+), 11 deletions(-)

Applied, thanks.

> diff --git a/drivers/mfd/da9063-core.c b/drivers/mfd/da9063-core.c
> index af841c1..6c2870d 100644
> --- a/drivers/mfd/da9063-core.c
> +++ b/drivers/mfd/da9063-core.c
> @@ -4,8 +4,8 @@
>   * Copyright 2012 Dialog Semiconductors Ltd.
>   * Copyright 2013 Philipp Zabel, Pengutronix
>   *
> - * Author: Krystian Garbaciak <krystian.garbaciak@diasemi.com>,
> - *         Michal Hajduk <michal.hajduk@diasemi.com>
> + * Author: Krystian Garbaciak, Dialog Semiconductor
> + * Author: Michal Hajduk, Dialog Semiconductor
>   *
>   *  This program is free software; you can redistribute  it and/or modify it
>   *  under  the terms of  the GNU General  Public License as published by the
> @@ -242,5 +242,6 @@ void da9063_device_exit(struct da9063 *da9063)
>  }
>  
>  MODULE_DESCRIPTION("PMIC driver for Dialog DA9063");
> -MODULE_AUTHOR("Krystian Garbaciak <krystian.garbaciak@diasemi.com>, Michal Hajduk <michal.hajduk@diasemi.com>");
> +MODULE_AUTHOR("Krystian Garbaciak");
> +MODULE_AUTHOR("Michal Hajduk");
>  MODULE_LICENSE("GPL");
> diff --git a/drivers/mfd/da9063-i2c.c b/drivers/mfd/da9063-i2c.c
> index 7390108..981805a 100644
> --- a/drivers/mfd/da9063-i2c.c
> +++ b/drivers/mfd/da9063-i2c.c
> @@ -3,7 +3,7 @@
>   * Copyright 2012 Dialog Semiconductor Ltd.
>   * Copyright 2013 Philipp Zabel, Pengutronix
>   *
> - * Author: Krystian Garbaciak <krystian.garbaciak@diasemi.com>
> + * Author: Krystian Garbaciak, Dialog Semiconductor
>   *
>   *  This program is free software; you can redistribute  it and/or modify it
>   *  under  the terms of  the GNU General  Public License as published by the
> diff --git a/drivers/mfd/da9063-irq.c b/drivers/mfd/da9063-irq.c
> index 7e903fc..207bbfe 100644
> --- a/drivers/mfd/da9063-irq.c
> +++ b/drivers/mfd/da9063-irq.c
> @@ -3,7 +3,7 @@
>   * Copyright 2012 Dialog Semiconductor Ltd.
>   * Copyright 2013 Philipp Zabel, Pengutronix
>   *
> - * Author: Michal Hajduk <michal.hajduk@diasemi.com>
> + * Author: Michal Hajduk, Dialog Semiconductor
>   *
>   *  This program is free software; you can redistribute  it and/or modify it
>   *  under  the terms of  the GNU General  Public License as published by the
> diff --git a/include/linux/mfd/da9063/core.h b/include/linux/mfd/da9063/core.h
> index 621af82..f3ae65d 100644
> --- a/include/linux/mfd/da9063/core.h
> +++ b/include/linux/mfd/da9063/core.h
> @@ -3,8 +3,8 @@
>   *
>   * Copyright 2012 Dialog Semiconductor Ltd.
>   *
> - * Author: Michal Hajduk <michal.hajduk@diasemi.com>
> - *	   Krystian Garbaciak <krystian.garbaciak@diasemi.com>
> + * Author: Michal Hajduk, Dialog Semiconductor
> + * Author: Krystian Garbaciak, Dialog Semiconductor
>   *
>   *  This program is free software; you can redistribute  it and/or modify it
>   *  under  the terms of  the GNU General  Public License as published by the
> diff --git a/include/linux/mfd/da9063/pdata.h b/include/linux/mfd/da9063/pdata.h
> index 612383b..8a12570 100644
> --- a/include/linux/mfd/da9063/pdata.h
> +++ b/include/linux/mfd/da9063/pdata.h
> @@ -3,8 +3,8 @@
>   *
>   * Copyright 2012 Dialog Semiconductor Ltd.
>   *
> - * Author: Michal Hajduk <michal.hajduk@diasemi.com>
> - * Author: Krystian Garbaciak <krystian.garbaciak@diasemi.com>
> + * Author: Michal Hajduk, Dialog Semiconductor
> + * Author: Krystian Garbaciak, Dialog Semiconductor
>   *
>   *  This program is free software; you can redistribute  it and/or modify it
>   *  under  the terms of  the GNU General  Public License as published by the
> diff --git a/include/linux/mfd/da9063/registers.h b/include/linux/mfd/da9063/registers.h
> index 2e0ba6d..5d42859 100644
> --- a/include/linux/mfd/da9063/registers.h
> +++ b/include/linux/mfd/da9063/registers.h
> @@ -3,8 +3,8 @@
>   *
>   * Copyright 2012 Dialog Semiconductor Ltd.
>   *
> - * Author: Michal Hajduk <michal.hajduk@diasemi.com>
> - *	   Krystian Garbaciak <krystian.garbaciak@diasemi.com>
> + * Author: Michal Hajduk, Dialog Semiconductor
> + * Author: Krystian Garbaciak, Dialog Semiconductor
>   *
>   *  This program is free software; you can redistribute  it and/or modify it
>   *  under  the terms of  the GNU General  Public License as published by the

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-08-09 12:47 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-08 13:16 [PATCH V2] mfd: da9063: Update author information to remove incorrect e-mail addresses Steve Twiss
2016-08-09 12:49 ` Lee Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox