linux-kernel - Re: [PATCH V2] mfd: da9063: Update author information to remove incorrect e-mail addresses

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:	Tue, 9 Aug 2016 13:49:04 +0100
From:	Lee Jones <lee.jones@...aro.org>
To:	Steve Twiss <stwiss.opensource@...semi.com>
Cc:	LINUXKERNEL <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH V2] mfd: da9063: Update author information to remove
 incorrect e-mail addresses

On Mon, 08 Aug 2016, Steve Twiss wrote:

> From: Steve Twiss <stwiss.opensource@...semi.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@...semi.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@...semi.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@...semi.com>,
> - *         Michal Hajduk <michal.hajduk@...semi.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@...semi.com>, Michal Hajduk <michal.hajduk@...semi.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@...semi.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@...semi.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@...semi.com>
> - *	   Krystian Garbaciak <krystian.garbaciak@...semi.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@...semi.com>
> - * Author: Krystian Garbaciak <krystian.garbaciak@...semi.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@...semi.com>
> - *	   Krystian Garbaciak <krystian.garbaciak@...semi.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