Messages in this thread | | | From | "Winkler, Tomas" <> | Subject | RE: [char-misc] mei: convert to SPDX license tags | Date | Sun, 10 Mar 2019 10:14:03 +0000 |
| |
> On Sun, Mar 10, 2019 at 12:03:23PM +0200, Tomas Winkler wrote: > > Signed-off-by: Tomas Winkler <tomas.winkler@intel.com> > > I can not take patches without any changelog text.
Ok, my mistake. > > Especially a patch that does not actually do what your subject: line says it does.
Yep, I should be more verbose. > > > --- a/drivers/misc/mei/Makefile > > +++ b/drivers/misc/mei/Makefile > > @@ -1,7 +1,7 @@ > > # SPDX-License-Identifier: GPL-2.0 > > # > > # Makefile - Intel Management Engine Interface (Intel MEI) Linux > > driver -# Copyright (c) 2010-2014, Intel Corporation. > > +# Copyright (c) 2010-2019, Intel Corporation. > > # > > obj-$(CONFIG_INTEL_MEI) += mei.o > > mei-objs := init.o > > As an example, why did you update this copyright line? That has nothing to do > with the SPDX identifier :(
Right, will update the commit message. > > > > --- a/drivers/misc/mei/client.c > > +++ b/drivers/misc/mei/client.c > > @@ -1,17 +1,8 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > /* > > + * Copyright(c) 2003-2019, Intel Corporation. All rights reserved. > > * > > * Intel Management Engine Interface (Intel MEI) Linux driver > > - * Copyright (c) 2003-2012, Intel Corporation. > > Are you _SURE_ you mean to change the dates of the copyright in this manner > for this file (and elsewhere?) Was the copyright really updated every year for > the past 7 years?
Yes, this particular file was updated, will go over other files.
> > Please break this up into two (at the least) patches and resend as a series.
Can you be please, more specific what split are you expecting. > > This patch makes me feel like it is a kernel intern application process patch, > and that's not good, you all know better than this. No, sorry, I take that back, > the intern application patches are of much higher quality... > Have hard time to put the soul into it, where it comes to legal :) Will fix. Tomas
| |