On Thu, 2015-03-05 at 17:27 +0000, Zubair Lutfullah Kakakhel wrote: > --- /dev/null > +++ b/drivers/mtd/nand/jz4780_bch.c > @@ -0,0 +1,359 @@ > +/* > + * JZ4780 BCH controller > + * > + * Copyright (c) 2015 Imagination Technologies > + * Author: Alex Smith <alex@xxxxxxxxxxxxxxxx> > + * > + * 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 states the license is GPL v2 or later. > +MODULE_LICENSE("GPL v2"); So you probably want MODULE_LICENSE("GPL"); here. > --- /dev/null > +++ b/drivers/mtd/nand/jz4780_nand.c > @@ -0,0 +1,373 @@ > +/* > + * JZ4780 NAND driver > + * > + * Copyright (c) 2015 Imagination Technologies > + * Author: Alex Smith <alex@xxxxxxxxxxxxxxxx> > + * > + * 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 states the license is GPL v2 or later. > +MODULE_LICENSE("GPL V2"); So, again, you probably want MODULE_LICENSE("GPL"); here. Actually "GPL V2" would taint the kernel on module load. (This all assumes we're talking about two separate modules here. I don't speak "Makefile" enough to be sure.) Paul Bolle -- 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