lkml.org 
[lkml]   [2017]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 01/10] ASoC: tlv320aic32x4: Remove filename from header and switch to SPDX
From
Date
On Wed, 2017-12-13 at 09:44 -0600, Andrew F. Davis wrote:
> On 12/12/2017 08:19 PM, Joe Perches wrote:
> > On Tue, 2017-12-12 at 16:43 -0600, Andrew F. Davis wrote:
> > > The filename is often wrong after moves and adds nothing of value,
> > > lets drop it. While already here, switch to SPDX.
> >
> > []
> > > diff --git a/sound/soc/codecs/tlv320aic32x4-i2c.c b/sound/soc/codecs/tlv320aic32x4-i2c.c
> >
> > []
> > > @@ -1,21 +1,10 @@
> > > +// SPDX-License-Identifier: GPL-2.0
> > > /*
> > > - * linux/sound/soc/codecs/tlv320aic32x4-i2c.c
> > > - *
> > > * Copyright 2011 NW Digital Radio
> > > *
> > > * Author: Jeremy McDermond <nh6z@nh6z.net>
> > > *
> > > * Based on sound/soc/codecs/wm8974 and TI driver for kernel 2.6.27.
> > > - *
> > > - * 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 Free Software Foundation; either version 2 of the License, or
> > > - * (at your option) any later version.
> > > - *
> > > - * 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.
> >
> > nack. Note the "(at your option) any later version"
> >
> > If you are going to update to spdx and remove the old license
> > verbiage, make sure you are adding the correct spdx tag.
> >
>
> My bad, will fix.
>
> By the way, do you like these spdx update patch? Or would you rather I
> drop this one and wait to let some script do it for the whole subsystem?

I would not touch any license text until an automated tool
can properly do a pass over the entire kernel source tree.

All the bsd variants probably require that the original text
be left in place too.

\
 
 \ /
  Last update: 2017-12-13 16:51    [W:0.062 / U:1.764 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site