Re: [PATCH] ASoC: mt6797: switch to SPDX license tag

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Apr 26, 2018 at 12:46:54PM +0200, Matthias Brugger wrote:
> On 04/26/2018 04:56 AM, KaiChieh Chuang wrote:
> > diff --git a/sound/soc/mediatek/mt6797/mt6797-mt6351.c b/sound/soc/mediatek/mt6797/mt6797-mt6351.c
> > index d42a9d9fbf3e..c3f58bac0960 100644
> > --- a/sound/soc/mediatek/mt6797/mt6797-mt6351.c
> > +++ b/sound/soc/mediatek/mt6797/mt6797-mt6351.c
> > @@ -1,17 +1,9 @@
> > +// SPDX-License-Identifier: GPL-2.0
> >  /*
> >   * mt6797-mt6351.c  --  MT6797 MT6351 ALSA SoC machine driver
> >   *

Please convert the entire header block to a C++ comment so it looks more
intentional that it's this way.

> > - * Author: Ryder Lee <ryder.lee@xxxxxxxxxxxx>
> > + * Author: KaiChieh Chuang <kaichieh.chuang@xxxxxxxxxxxx>

> I think you don't want to delete the first author but add you as a second one.
> I've seen this in other parts of the patch as well. Please check.

Yeah, looks like an over eager removal of the license text boiler
plate.

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux