* [PATCH] media: mtk-vcodec: add missing MODULE_LICENSE/DESCRIPTION
@ 2017-11-20 6:26 Jesse Chan
2017-11-20 6:39 ` [PATCH v2] " Jesse Chan
2017-11-22 7:41 ` [PATCH] " kbuild test robot
0 siblings, 2 replies; 5+ messages in thread
From: Jesse Chan @ 2017-11-20 6:26 UTC (permalink / raw)
Cc: Jesse Chan, Tiffany Lin, Andrew-CT Chen, Mauro Carvalho Chehab,
Matthias Brugger, linux-media, linux-arm-kernel, linux-mediatek,
linux-kernel
Signed-off-by: Jesse Chan <jc@linux.com>
---
drivers/media/platform/mtk-vcodec/mtk_vcodec_intr.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/media/platform/mtk-vcodec/mtk_vcodec_intr.c b/drivers/media/platform/mtk-vcodec/mtk_vcodec_intr.c
index 113b2097f061..d8212d759067 100644
--- a/drivers/media/platform/mtk-vcodec/mtk_vcodec_intr.c
+++ b/drivers/media/platform/mtk-vcodec/mtk_vcodec_intr.c
@@ -51,3 +51,6 @@ int mtk_vcodec_wait_for_done_ctx(struct mtk_vcodec_ctx *ctx, int command,
return status;
}
EXPORT_SYMBOL(mtk_vcodec_wait_for_done_ctx);
+
+MODULE_LICENSE("GPL v2");
+MODULE_DESCRIPTION("Mediatek video codec driver");
--
2.14.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v2] media: mtk-vcodec: add missing MODULE_LICENSE/DESCRIPTION
2017-11-20 6:26 [PATCH] media: mtk-vcodec: add missing MODULE_LICENSE/DESCRIPTION Jesse Chan
@ 2017-11-20 6:39 ` Jesse Chan
2017-11-20 7:47 ` [PATCH v3] " Jesse Chan
2017-11-22 7:41 ` [PATCH] " kbuild test robot
1 sibling, 1 reply; 5+ messages in thread
From: Jesse Chan @ 2017-11-20 6:39 UTC (permalink / raw)
Cc: Jesse Chan, Tiffany Lin, Andrew-CT Chen, Mauro Carvalho Chehab,
Matthias Brugger, linux-media, linux-arm-kernel, linux-mediatek,
linux-kernel
Signed-off-by: Jesse Chan <jc@linux.com>
---
Change in v2:
- fix a mistake
drivers/media/platform/mtk-vcodec/mtk_vcodec_util.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/media/platform/mtk-vcodec/mtk_vcodec_util.c b/drivers/media/platform/mtk-vcodec/mtk_vcodec_util.c
index 46768c056193..0c28d0b995cc 100644
--- a/drivers/media/platform/mtk-vcodec/mtk_vcodec_util.c
+++ b/drivers/media/platform/mtk-vcodec/mtk_vcodec_util.c
@@ -115,3 +115,6 @@ struct mtk_vcodec_ctx *mtk_vcodec_get_curr_ctx(struct mtk_vcodec_dev *dev)
return ctx;
}
EXPORT_SYMBOL(mtk_vcodec_get_curr_ctx);
+
+MODULE_LICENSE("GPL v2");
+MODULE_DESCRIPTION("Mediatek video codec driver");
--
2.14.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v3] media: mtk-vcodec: add missing MODULE_LICENSE/DESCRIPTION
2017-11-20 6:39 ` [PATCH v2] " Jesse Chan
@ 2017-11-20 7:47 ` Jesse Chan
0 siblings, 0 replies; 5+ messages in thread
From: Jesse Chan @ 2017-11-20 7:47 UTC (permalink / raw)
Cc: Jesse Chan, Tiffany Lin, Andrew-CT Chen, Mauro Carvalho Chehab,
Matthias Brugger, linux-media, linux-arm-kernel, linux-mediatek,
linux-kernel
This change resolves a new compile-time warning
when built as a loadable module:
WARNING: modpost: missing MODULE_LICENSE() in drivers/media/platform/mtk-vcodec/mtk-vcodec-common.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 is also added.
Signed-off-by: Jesse Chan <jc@linux.com>
---
drivers/media/platform/mtk-vcodec/mtk_vcodec_util.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/media/platform/mtk-vcodec/mtk_vcodec_util.c b/drivers/media/platform/mtk-vcodec/mtk_vcodec_util.c
index 46768c056193..0c28d0b995cc 100644
--- a/drivers/media/platform/mtk-vcodec/mtk_vcodec_util.c
+++ b/drivers/media/platform/mtk-vcodec/mtk_vcodec_util.c
@@ -115,3 +115,6 @@ struct mtk_vcodec_ctx *mtk_vcodec_get_curr_ctx(struct mtk_vcodec_dev *dev)
return ctx;
}
EXPORT_SYMBOL(mtk_vcodec_get_curr_ctx);
+
+MODULE_LICENSE("GPL v2");
+MODULE_DESCRIPTION("Mediatek video codec driver");
--
2.14.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] media: mtk-vcodec: add missing MODULE_LICENSE/DESCRIPTION
2017-11-20 6:26 [PATCH] media: mtk-vcodec: add missing MODULE_LICENSE/DESCRIPTION Jesse Chan
2017-11-20 6:39 ` [PATCH v2] " Jesse Chan
@ 2017-11-22 7:41 ` kbuild test robot
2017-11-22 7:51 ` Randy Dunlap
1 sibling, 1 reply; 5+ messages in thread
From: kbuild test robot @ 2017-11-22 7:41 UTC (permalink / raw)
To: Jesse Chan
Cc: kbuild-all, Jesse Chan, Tiffany Lin, Andrew-CT Chen,
Mauro Carvalho Chehab, Matthias Brugger, linux-media,
linux-arm-kernel, linux-mediatek, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1467 bytes --]
Hi Jesse,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linuxtv-media/master]
[also build test ERROR on v4.14 next-20171121]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Jesse-Chan/media-mtk-vcodec-add-missing-MODULE_LICENSE-DESCRIPTION/20171122-124620
base: git://linuxtv.org/media_tree.git master
config: xtensa-allmodconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 4.9.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=xtensa
All errors (new ones prefixed by >>):
>> drivers/media/platform/mtk-vcodec/mtk_vcodec_intr.c:55:16: error: expected declaration specifiers or '...' before string constant
MODULE_LICENSE("GPL v2");
^
drivers/media/platform/mtk-vcodec/mtk_vcodec_intr.c:56:20: error: expected declaration specifiers or '...' before string constant
MODULE_DESCRIPTION("Mediatek video codec driver");
^
vim +55 drivers/media/platform/mtk-vcodec/mtk_vcodec_intr.c
54
> 55 MODULE_LICENSE("GPL v2");
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 51526 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] media: mtk-vcodec: add missing MODULE_LICENSE/DESCRIPTION
2017-11-22 7:41 ` [PATCH] " kbuild test robot
@ 2017-11-22 7:51 ` Randy Dunlap
0 siblings, 0 replies; 5+ messages in thread
From: Randy Dunlap @ 2017-11-22 7:51 UTC (permalink / raw)
To: kbuild test robot, Jesse Chan
Cc: kbuild-all, Tiffany Lin, Andrew-CT Chen, Mauro Carvalho Chehab,
Matthias Brugger, linux-media, linux-arm-kernel, linux-mediatek,
linux-kernel
On 11/21/17 23:41, kbuild test robot wrote:
> Hi Jesse,
>
> Thank you for the patch! Yet something to improve:
missing
#include <linux/module.h>
Jesse, did you build all of these driver changes?
> [auto build test ERROR on linuxtv-media/master]
> [also build test ERROR on v4.14 next-20171121]
> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
>
> url: https://github.com/0day-ci/linux/commits/Jesse-Chan/media-mtk-vcodec-add-missing-MODULE_LICENSE-DESCRIPTION/20171122-124620
> base: git://linuxtv.org/media_tree.git master
> config: xtensa-allmodconfig (attached as .config)
> compiler: xtensa-linux-gcc (GCC) 4.9.0
> reproduce:
> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # save the attached .config to linux build tree
> make.cross ARCH=xtensa
>
> All errors (new ones prefixed by >>):
>
>>> drivers/media/platform/mtk-vcodec/mtk_vcodec_intr.c:55:16: error: expected declaration specifiers or '...' before string constant
> MODULE_LICENSE("GPL v2");
> ^
> drivers/media/platform/mtk-vcodec/mtk_vcodec_intr.c:56:20: error: expected declaration specifiers or '...' before string constant
> MODULE_DESCRIPTION("Mediatek video codec driver");
> ^
>
> vim +55 drivers/media/platform/mtk-vcodec/mtk_vcodec_intr.c
>
> 54
> > 55 MODULE_LICENSE("GPL v2");
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
>
--
~Randy
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-11-22 7:52 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-20 6:26 [PATCH] media: mtk-vcodec: add missing MODULE_LICENSE/DESCRIPTION Jesse Chan
2017-11-20 6:39 ` [PATCH v2] " Jesse Chan
2017-11-20 7:47 ` [PATCH v3] " Jesse Chan
2017-11-22 7:41 ` [PATCH] " kbuild test robot
2017-11-22 7:51 ` Randy Dunlap
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).