On 10/04/2019 17:01:42-0400, Brandon Martin wrote: > On 4/10/19 4:56 PM, Alexandre Belloni wrote: > > Use SPDX-License-Identifier instead of a verbose license text. > > > > The original text refers to version 2 so also update MODULE_LICENSE() to > > "GPL v2" instead of "GPL". > > > > Signed-off-by: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxx> > > --- > > drivers/rtc/rtc-pcf85363.c | 7 +------ > > 1 file changed, 1 insertion(+), 6 deletions(-) > > > > diff --git a/drivers/rtc/rtc-pcf85363.c b/drivers/rtc/rtc-pcf85363.c > > index afb1baacdaa7..a075e77617dc 100644 > > --- a/drivers/rtc/rtc-pcf85363.c > > +++ b/drivers/rtc/rtc-pcf85363.c > > @@ -1,15 +1,10 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > /* > > * drivers/rtc/rtc-pcf85363.c > > * > > * Driver for NXP PCF85363 real-time clock. > > * > > * Copyright (C) 2017 Eric Nelson > > - * > > - * This program is free software; you can redistribute it and/or modify > > - * it under the terms of the GNU General Public License version 2 as > > - * published by the Free Software Foundation. > > - * > > - * Based loosely on rtc-8583 by Russell King, Wolfram Sang and Juergen Beisert > > */ > > #include <linux/module.h> > > #include <linux/i2c.h> > > > > Looks like you did not update MODULE_LICENSE as stated in the comment? Right, I'll remove that from the commit message. Thanks! -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com