2024-06-02 23:17:28

by Jeff Johnson

[permalink] [raw]
Subject: [PATCH] crypto: xilinx - add missing MODULE_DESCRIPTION() macro

make allmodconfig && make W=1 C=1 reports:
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/crypto/xilinx/zynqmp-aes-gcm.o

Add the missing invocation of the MODULE_DESCRIPTION() macro.

Signed-off-by: Jeff Johnson <[email protected]>
---
drivers/crypto/xilinx/zynqmp-aes-gcm.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/crypto/xilinx/zynqmp-aes-gcm.c b/drivers/crypto/xilinx/zynqmp-aes-gcm.c
index e61405718840..7f0ec6887a39 100644
--- a/drivers/crypto/xilinx/zynqmp-aes-gcm.c
+++ b/drivers/crypto/xilinx/zynqmp-aes-gcm.c
@@ -446,4 +446,5 @@ static struct platform_driver zynqmp_aes_driver = {
};

module_platform_driver(zynqmp_aes_driver);
+MODULE_DESCRIPTION("Xilinx ZynqMP AES Driver");
MODULE_LICENSE("GPL");

---
base-commit: a693b9c95abd4947c2d06e05733de5d470ab6586
change-id: 20240602-md-zynqmp-aes-gcm-c46078273370



2024-06-03 06:53:31

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH] crypto: xilinx - add missing MODULE_DESCRIPTION() macro



On 6/3/24 01:17, Jeff Johnson wrote:
> make allmodconfig && make W=1 C=1 reports:
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/crypto/xilinx/zynqmp-aes-gcm.o
>
> Add the missing invocation of the MODULE_DESCRIPTION() macro.
>
> Signed-off-by: Jeff Johnson <[email protected]>
> ---
> drivers/crypto/xilinx/zynqmp-aes-gcm.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/crypto/xilinx/zynqmp-aes-gcm.c b/drivers/crypto/xilinx/zynqmp-aes-gcm.c
> index e61405718840..7f0ec6887a39 100644
> --- a/drivers/crypto/xilinx/zynqmp-aes-gcm.c
> +++ b/drivers/crypto/xilinx/zynqmp-aes-gcm.c
> @@ -446,4 +446,5 @@ static struct platform_driver zynqmp_aes_driver = {
> };
>
> module_platform_driver(zynqmp_aes_driver);
> +MODULE_DESCRIPTION("Xilinx ZynqMP AES Driver");
> MODULE_LICENSE("GPL");
>
> ---
> base-commit: a693b9c95abd4947c2d06e05733de5d470ab6586
> change-id: 20240602-md-zynqmp-aes-gcm-c46078273370
>

Reviewed-by: Michal Simek <[email protected]>

Thanks,
Michal