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 <matthias.bgg@xxxxxxxxx> > 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 <kaichieh.chuang@xxxxxxxxxxxx> >>> --- >>> 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 <ryder.lee@xxxxxxxxxxxx> >>> - * >>> - * 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 <kaichieh.chuang@xxxxxxxxxxxx> >> >> As we said in the last version, you should not delete Ryder from the author list. >> >>> >>> #include <linux/module.h> >>> #include <sound/soc.h> >>> 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 <garlic.tseng@xxxxxxxxxxxx> >>> - * >>> - * 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 <kaichieh.chuang@xxxxxxxxxxxx> >> >> Same here, Garlic is an author as well, don't delete him from the comment. >> >> Regards, >> Matthias >> >> _______________________________________________ >> Linux-mediatek mailing list >> Linux-mediatek@xxxxxxxxxxxxxxxxxxx >> http://lists.infradead.org/mailman/listinfo/linux-mediatek > > _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
