From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthias Brugger Subject: Re: [PATCH v2 1/2] ASoC: mt6797: switch to SPDX license tag Date: Fri, 27 Apr 2018 11:27:38 +0200 Message-ID: <15b3eeb7-8f78-686e-77fc-f27d65fd959b@gmail.com> References: <20180427015445.4334-1-kaichieh.chuang@mediatek.com> <20180427015445.4334-2-kaichieh.chuang@mediatek.com> <1524821008.12322.208.camel@mtkswgap22> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1524821008.12322.208.camel@mtkswgap22> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Sean Wang Cc: alsa-devel@alsa-project.org, ryder.lee@mediatek.com, chipeng.chang@mediatek.com, wsd_upstream@mediatek.com, garlic.tseng@mediatek.com, broonie@kernel.org, linux-mediatek@lists.infradead.org, KaiChieh Chuang List-Id: linux-mediatek@lists.infradead.org On 04/27/2018 11:23 AM, Sean Wang wrote: > Ryder and Garlic are not actual authors. > > There are totally copy-n-paste errors on Author fields when KaiChieh is > making the 1st version. > > also can see > > http://lists.infradead.org/pipermail/linux-mediatek/2018-April/013036.html > > http://lists.infradead.org/pipermail/linux-mediatek/2018-April/013036.html > > Sorry I didn't read these two emails. Reviewed-by: Matthias Brugger > On Fri, 2018-04-27 at 11:15 +0200, Matthias Brugger wrote: >> >> On 04/27/2018 03:54 AM, KaiChieh Chuang wrote: >>> Signed-off-by: KaiChieh Chuang >>> --- >>> sound/soc/mediatek/mt6797/Makefile | 13 +------------ >>> sound/soc/mediatek/mt6797/mt6797-afe-clk.c | 21 ++++++--------------- >>> sound/soc/mediatek/mt6797/mt6797-afe-clk.h | 10 +--------- >>> sound/soc/mediatek/mt6797/mt6797-afe-common.h | 10 +--------- >>> sound/soc/mediatek/mt6797/mt6797-afe-pcm.c | 21 ++++++--------------- >>> sound/soc/mediatek/mt6797/mt6797-interconnection.h | 10 +--------- >>> sound/soc/mediatek/mt6797/mt6797-mt6351.c | 21 ++++++--------------- >>> sound/soc/mediatek/mt6797/mt6797-reg.h | 12 ++---------- >>> 8 files changed, 24 insertions(+), 94 deletions(-) >>> >> >> [...] >> >>> diff --git a/sound/soc/mediatek/mt6797/mt6797-mt6351.c b/sound/soc/mediatek/mt6797/mt6797-mt6351.c >>> index d42a9d9fbf3e..9e5866d6819a 100644 >>> --- a/sound/soc/mediatek/mt6797/mt6797-mt6351.c >>> +++ b/sound/soc/mediatek/mt6797/mt6797-mt6351.c >>> @@ -1,18 +1,9 @@ >>> -/* >>> - * mt6797-mt6351.c -- MT6797 MT6351 ALSA SoC machine driver >>> - * >>> - * Copyright (c) 2018 MediaTek Inc. >>> - * Author: Ryder Lee >>> - * >>> - * This program is free software; you can redistribute it and/or modify >>> - * it under the terms of the GNU General Public License version 2 and >>> - * only version 2 as published by the Free Software Foundation. >>> - * >>> - * This program is distributed in the hope that it will be useful, >>> - * but WITHOUT ANY WARRANTY; without even the implied warranty of >>> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >>> - * GNU General Public License for more details. >>> - */ >>> +// SPDX-License-Identifier: GPL-2.0 >>> +// >>> +// mt6797-mt6351.c -- MT6797 MT6351 ALSA SoC machine driver >>> +// >>> +// Copyright (c) 2018 MediaTek Inc. >>> +// Author: KaiChieh Chuang >> >> As we said in the last version, you should not delete Ryder from the author list. >> >>> >>> #include >>> #include >>> diff --git a/sound/soc/mediatek/mt6797/mt6797-reg.h b/sound/soc/mediatek/mt6797/mt6797-reg.h >>> index 3330f73fc8bb..ffb55367f59f 100644 >>> --- a/sound/soc/mediatek/mt6797/mt6797-reg.h >>> +++ b/sound/soc/mediatek/mt6797/mt6797-reg.h >>> @@ -1,17 +1,9 @@ >>> +/* SPDX-License-Identifier: GPL-2.0 */ >>> /* >>> * mt6797-reg.h -- Mediatek 6797 audio driver reg definition >>> * >>> * Copyright (c) 2018 MediaTek Inc. >>> - * Author: Garlic Tseng >>> - * >>> - * This program is free software; you can redistribute it and/or modify >>> - * it under the terms of the GNU General Public License version 2 and >>> - * only version 2 as published by the Free Software Foundation. >>> - * >>> - * This program is distributed in the hope that it will be useful, >>> - * but WITHOUT ANY WARRANTY; without even the implied warranty of >>> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >>> - * GNU General Public License for more details. >>> + * Author: KaiChieh Chuang >> >> Same here, Garlic is an author as well, don't delete him from the comment. >> >> Regards, >> Matthias >> >> _______________________________________________ >> Linux-mediatek mailing list >> Linux-mediatek@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-mediatek > >