linux-kernel - Re: [PATCH v2] drm: dw-hdmi-i2s: add missing company name on Copyright

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: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Mon, 07 Aug 2017 11:55:36 +0300
From:   Laurent Pinchart <laurent.pinchart@...asonboard.com>
To:     Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
Cc:     David Airlie <airlied@...ux.ie>,
        Archit Taneja <architt@...eaurora.org>,
        Andrzej Hajda <a.hajda@...sung.com>,
        Russell King <rmk+kernel@...linux.org.uk>,
        Mark Brown <broonie@...nel.org>,
        Linux-Renesas <linux-renesas-soc@...r.kernel.org>,
        Linux-Kernel <linux-kernel@...r.kernel.org>,
        Jose Abreu <joabreu@...opsys.com>,
        dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH v2] drm: dw-hdmi-i2s: add missing company name on Copyright

Hi Morimoto-san,

Thank you for the patch.

On Monday 07 Aug 2017 04:09:41 Kuninori Morimoto wrote:
> From: Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
> 
> This driver's Copyright is under Renesas Solutions Corp.
> This patch updates the year, because this driver was moved
> into synopsys folder in 2017.
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
> ---
> v1 -> v2
> 
>  - update year 2016 -> 2017
> 
>  drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c index
> b2cf59f..3b7e5c5 100644
> --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c
> @@ -1,7 +1,8 @@
>  /*
>   * dw-hdmi-i2s-audio.c
>   *
> - * Copyright (c) 2016 Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
> + * Copyright (c) 2017 Renesas Solutions Corp.
> + * Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>

What does this mean ? The first line makes it clear that you want to assign 
copyright ownership to Renesas for the work you've done on this driver. The 
second line, however, isn't very clear. If you want to list your e-mail 
address as an author or contact person, you should make that explicit:

 * Author: Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>

or

 * Contact: Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>

Or it could be that you have joint copyright ownership with Renesas, or own 
parts of the copyright yourself, in which case it should be

 * Copyright (c) 2017 Renesas Solutions Corp.
 * Copyright (c) 2017 Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>

>   *
>   * 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

-- 
Regards,

Laurent Pinchart