linux-kernel - Re: [PATCH] media: atomisp: add missing MODULE

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Wed, 26 Jun 2024 09:37:53 -0700
From: Jeff Johnson <quic_jjohnson@...cinc.com>
To: Hans de Goede <hdegoede@...hat.com>,
        Mauro Carvalho Chehab
	<mchehab@...nel.org>,
        Sakari Ailus <sakari.ailus@...ux.intel.com>,
        Greg
 Kroah-Hartman <gregkh@...uxfoundation.org>
CC: <linux-media@...r.kernel.org>, <linux-staging@...ts.linux.dev>,
        <linux-kernel@...r.kernel.org>, <kernel-janitors@...r.kernel.org>
Subject: Re: [PATCH] media: atomisp: add missing MODULE_DESCRIPTION() macros

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@...cinc.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@...el.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@...el.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