* [PATCH] media: atomisp: add missing MODULE_DESCRIPTION() macros
@ 2024-06-11 0:34 Jeff Johnson
2024-06-26 16:37 ` Jeff Johnson
0 siblings, 1 reply; 3+ messages in thread
From: Jeff Johnson @ 2024-06-11 0:34 UTC (permalink / raw)
To: Hans de Goede, Mauro Carvalho Chehab, Sakari Ailus, Greg Kroah-Hartman
Cc: linux-media, linux-staging, linux-kernel, kernel-janitors, Jeff Johnson
make allmodconfig && make W=1 C=1 reports:
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/staging/media/atomisp/i2c/atomisp-mt9m114.o
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.o
Add the missing invocations of the MODULE_DESCRIPTION() macro.
Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
---
drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c | 1 +
drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c b/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c
index 7a20d918a9d5..3499353f8ea5 100644
--- a/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c
+++ b/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c
@@ -207,4 +207,5 @@ module_init(init_msrlisthelper);
module_exit(exit_msrlisthelper);
MODULE_AUTHOR("Jukka Kaartinen <jukka.o.kaartinen@intel.com>");
+MODULE_DESCRIPTION("Helper library to load, parse and apply large register lists");
MODULE_LICENSE("GPL");
diff --git a/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c b/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
index 23b1001c2a55..918ea4fa9f6b 100644
--- a/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
+++ b/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
@@ -1614,4 +1614,5 @@ static struct i2c_driver mt9m114_driver = {
module_i2c_driver(mt9m114_driver);
MODULE_AUTHOR("Shuguang Gong <Shuguang.gong@intel.com>");
+MODULE_DESCRIPTION("Aptina mt9m114 sensor support module");
MODULE_LICENSE("GPL");
---
base-commit: 83a7eefedc9b56fe7bfeff13b6c7356688ffa670
change-id: 20240610-md-drivers-staging-media-atomisp-i2c-18a7a4f883eb
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] media: atomisp: add missing MODULE_DESCRIPTION() macros
2024-06-11 0:34 [PATCH] media: atomisp: add missing MODULE_DESCRIPTION() macros Jeff Johnson
@ 2024-06-26 16:37 ` Jeff Johnson
2024-07-02 10:47 ` Hans de Goede
0 siblings, 1 reply; 3+ messages in thread
From: Jeff Johnson @ 2024-06-26 16:37 UTC (permalink / raw)
To: Hans de Goede, Mauro Carvalho Chehab, Sakari Ailus, Greg Kroah-Hartman
Cc: linux-media, linux-staging, linux-kernel, kernel-janitors
On 6/10/2024 5:34 PM, Jeff Johnson wrote:
> make allmodconfig && make W=1 C=1 reports:
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/staging/media/atomisp/i2c/atomisp-mt9m114.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.o
>
> Add the missing invocations of the MODULE_DESCRIPTION() macro.
>
> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
> ---
> drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c | 1 +
> drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c | 1 +
> 2 files changed, 2 insertions(+)
>
> diff --git a/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c b/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c
> index 7a20d918a9d5..3499353f8ea5 100644
> --- a/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c
> +++ b/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c
> @@ -207,4 +207,5 @@ module_init(init_msrlisthelper);
> module_exit(exit_msrlisthelper);
>
> MODULE_AUTHOR("Jukka Kaartinen <jukka.o.kaartinen@intel.com>");
> +MODULE_DESCRIPTION("Helper library to load, parse and apply large register lists");
> MODULE_LICENSE("GPL");
> diff --git a/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c b/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
> index 23b1001c2a55..918ea4fa9f6b 100644
> --- a/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
> +++ b/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
> @@ -1614,4 +1614,5 @@ static struct i2c_driver mt9m114_driver = {
> module_i2c_driver(mt9m114_driver);
>
> MODULE_AUTHOR("Shuguang Gong <Shuguang.gong@intel.com>");
> +MODULE_DESCRIPTION("Aptina mt9m114 sensor support module");
> MODULE_LICENSE("GPL");
>
> ---
> base-commit: 83a7eefedc9b56fe7bfeff13b6c7356688ffa670
> change-id: 20240610-md-drivers-staging-media-atomisp-i2c-18a7a4f883eb
>
Following up to see if anything else is needed from me. Hoping to see this in
linux-next so I can remove it from my tracking spreadsheet :)
/jeff
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] media: atomisp: add missing MODULE_DESCRIPTION() macros
2024-06-26 16:37 ` Jeff Johnson
@ 2024-07-02 10:47 ` Hans de Goede
0 siblings, 0 replies; 3+ messages in thread
From: Hans de Goede @ 2024-07-02 10:47 UTC (permalink / raw)
To: Jeff Johnson, Mauro Carvalho Chehab, Sakari Ailus, Greg Kroah-Hartman
Cc: linux-media, linux-staging, linux-kernel, kernel-janitors
Hi Jeff,
On 6/26/24 6:37 PM, Jeff Johnson wrote:
> On 6/10/2024 5:34 PM, Jeff Johnson wrote:
>> make allmodconfig && make W=1 C=1 reports:
>> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/staging/media/atomisp/i2c/atomisp-mt9m114.o
>> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.o
>>
>> Add the missing invocations of the MODULE_DESCRIPTION() macro.
>>
>> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
>> ---
>> drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c | 1 +
>> drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c | 1 +
>> 2 files changed, 2 insertions(+)
>>
>> diff --git a/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c b/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c
>> index 7a20d918a9d5..3499353f8ea5 100644
>> --- a/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c
>> +++ b/drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c
>> @@ -207,4 +207,5 @@ module_init(init_msrlisthelper);
>> module_exit(exit_msrlisthelper);
>>
>> MODULE_AUTHOR("Jukka Kaartinen <jukka.o.kaartinen@intel.com>");
>> +MODULE_DESCRIPTION("Helper library to load, parse and apply large register lists");
>> MODULE_LICENSE("GPL");
>> diff --git a/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c b/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
>> index 23b1001c2a55..918ea4fa9f6b 100644
>> --- a/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
>> +++ b/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
>> @@ -1614,4 +1614,5 @@ static struct i2c_driver mt9m114_driver = {
>> module_i2c_driver(mt9m114_driver);
>>
>> MODULE_AUTHOR("Shuguang Gong <Shuguang.gong@intel.com>");
>> +MODULE_DESCRIPTION("Aptina mt9m114 sensor support module");
>> MODULE_LICENSE("GPL");
>>
>> ---
>> base-commit: 83a7eefedc9b56fe7bfeff13b6c7356688ffa670
>> change-id: 20240610-md-drivers-staging-media-atomisp-i2c-18a7a4f883eb
>>
>
> Following up to see if anything else is needed from me. Hoping to see this in
> linux-next so I can remove it from my tracking spreadsheet :)
This has already been merged in media-staging (the media's tree for-next)
by Hans Verkuil:
https://git.linuxtv.org/media_stage.git/log/drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c
Regards,
Hans
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-07-02 10:48 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-06-11 0:34 [PATCH] media: atomisp: add missing MODULE_DESCRIPTION() macros Jeff Johnson
2024-06-26 16:37 ` Jeff Johnson
2024-07-02 10:47 ` Hans de Goede
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®