Message ID | 20171120205713.86952-1-jc@linux.com |
---|---|
State | Accepted |
Commit | d822401d1c6898a4a4ee03977b78b8cec402e88a |
Headers | show |
Series | mtd: nand: denali_pci: add missing MODULE_DESCRIPTION/AUTHOR/LICENSE | expand |
2017-11-21 5:57 GMT+09:00 Jesse Chan <jc@linux.com>: > This change resolves a new compile-time warning > when built as a loadable module: > > WARNING: modpost: missing MODULE_LICENSE() in drivers/mtd/nand/denali_pci.o > see include/linux/module.h for more information > > This adds the license as "GPL v2", which matches the header of the file. > > MODULE_DESCRIPTION and MODULE_AUTHOR are also added. > > Signed-off-by: Jesse Chan <jc@linux.com> > --- > drivers/mtd/nand/denali_pci.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/mtd/nand/denali_pci.c b/drivers/mtd/nand/denali_pci.c > index 81370c79aa48..7ad0db65a6fa 100644 > --- a/drivers/mtd/nand/denali_pci.c > +++ b/drivers/mtd/nand/denali_pci.c > @@ -124,3 +124,7 @@ static struct pci_driver denali_pci_driver = { > }; > > module_pci_driver(denali_pci_driver); > + > +MODULE_DESCRIPTION("PCI driver for Denali NAND controller"); > +MODULE_AUTHOR("Intel Corporation and its suppliers"); > +MODULE_LICENSE("GPL v2"); Acked-by: Masahiro Yamada <yamada.masahiro@socionext.com> -- Best Regards Masahiro Yamada
On Mon, 20 Nov 2017 12:57:13 -0800 Jesse Chan <jc@linux.com> wrote: > This change resolves a new compile-time warning > when built as a loadable module: > > WARNING: modpost: missing MODULE_LICENSE() in drivers/mtd/nand/denali_pci.o > see include/linux/module.h for more information > > This adds the license as "GPL v2", which matches the header of the file. > > MODULE_DESCRIPTION and MODULE_AUTHOR are also added. > Applied. Thanks, Boris > Signed-off-by: Jesse Chan <jc@linux.com> > --- > drivers/mtd/nand/denali_pci.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/mtd/nand/denali_pci.c b/drivers/mtd/nand/denali_pci.c > index 81370c79aa48..7ad0db65a6fa 100644 > --- a/drivers/mtd/nand/denali_pci.c > +++ b/drivers/mtd/nand/denali_pci.c > @@ -124,3 +124,7 @@ static struct pci_driver denali_pci_driver = { > }; > > module_pci_driver(denali_pci_driver); > + > +MODULE_DESCRIPTION("PCI driver for Denali NAND controller"); > +MODULE_AUTHOR("Intel Corporation and its suppliers"); > +MODULE_LICENSE("GPL v2");
diff --git a/drivers/mtd/nand/denali_pci.c b/drivers/mtd/nand/denali_pci.c index 81370c79aa48..7ad0db65a6fa 100644 --- a/drivers/mtd/nand/denali_pci.c +++ b/drivers/mtd/nand/denali_pci.c @@ -124,3 +124,7 @@ static struct pci_driver denali_pci_driver = { }; module_pci_driver(denali_pci_driver); + +MODULE_DESCRIPTION("PCI driver for Denali NAND controller"); +MODULE_AUTHOR("Intel Corporation and its suppliers"); +MODULE_LICENSE("GPL v2");
This change resolves a new compile-time warning when built as a loadable module: WARNING: modpost: missing MODULE_LICENSE() in drivers/mtd/nand/denali_pci.o see include/linux/module.h for more information This adds the license as "GPL v2", which matches the header of the file. MODULE_DESCRIPTION and MODULE_AUTHOR are also added. Signed-off-by: Jesse Chan <jc@linux.com> --- drivers/mtd/nand/denali_pci.c | 4 ++++ 1 file changed, 4 insertions(+) -- 2.14.1