On Sat, Mar 07, 2015 at 01:39:43PM +0100, Paul Bolle wrote: > Only a license nit. > > Brian Norris schreef op vr 06-03-2015 om 17:18 [-0800]: > > > --- /dev/null > > +++ b/drivers/mtd/nand/brcmstb_nand.c > > @@ -0,0 +1,2182 @@ > > +/* > > + * Copyright © 2010-2015 Broadcom Corporation > > + * > > + * 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. > > + * > > + * 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. > > + */ > > This states the license is GPL v2. > > > +MODULE_LICENSE("GPL"); > > So you probably want > MODULE_LICENSE("GPL v2"); > > here. Thanks. Fixed. Brian -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html