2024-06-03 14:37:08

by Jeff Johnson

Subject: [PATCH] ACPI: add missing MODULE_DESCRIPTION() macros

make allmodconfig && make W=1 C=1 reports:
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/acpi/acpi_tad.o
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/acpi/platform_profile.o

Add the missing invocations of the MODULE_DESCRIPTION() macro.

Signed-off-by: Jeff Johnson <[email protected]>
---
drivers/acpi/acpi_tad.c | 1 +
drivers/acpi/platform_profile.c | 1 +
2 files changed, 2 insertions(+)

diff --git a/drivers/acpi/acpi_tad.c b/drivers/acpi/acpi_tad.c
index 1d670dbe4d1d..b831cb8e53dc 100644
--- a/drivers/acpi/acpi_tad.c
+++ b/drivers/acpi/acpi_tad.c
@@ -27,6 +27,7 @@
#include <linux/pm_runtime.h>
#include <linux/suspend.h>

+MODULE_DESCRIPTION("ACPI Time and Alarm (TAD) Device Driver");
MODULE_LICENSE("GPL v2");
MODULE_AUTHOR("Rafael J. Wysocki");

diff --git a/drivers/acpi/platform_profile.c b/drivers/acpi/platform_profile.c
index 4a9704730224..d2f7fd7743a1 100644
--- a/drivers/acpi/platform_profile.c
+++ b/drivers/acpi/platform_profile.c
@@ -217,4 +217,5 @@ int platform_profile_remove(void)
EXPORT_SYMBOL_GPL(platform_profile_remove);

MODULE_AUTHOR("Mark Pearson <[email protected]>");
+MODULE_DESCRIPTION("ACPI platform profile sysfs interface");
MODULE_LICENSE("GPL");

---
base-commit: a693b9c95abd4947c2d06e05733de5d470ab6586
change-id: 20240603-md-drivers-acpi-misc-8c26065d84f1

2024-06-13 19:07:55

by Rafael J. Wysocki

Subject: Re: [PATCH] ACPI: add missing MODULE_DESCRIPTION() macros

On Mon, Jun 3, 2024 at 4:29 PM Jeff Johnson <[email protected]> wrote:
>
> make allmodconfig && make W=1 C=1 reports:
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/acpi/acpi_tad.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/acpi/platform_profile.o
>
> Add the missing invocations of the MODULE_DESCRIPTION() macro.
>
> Signed-off-by: Jeff Johnson <[email protected]>
> ---
> drivers/acpi/acpi_tad.c | 1 +
> drivers/acpi/platform_profile.c | 1 +
> 2 files changed, 2 insertions(+)
>
> diff --git a/drivers/acpi/acpi_tad.c b/drivers/acpi/acpi_tad.c
> index 1d670dbe4d1d..b831cb8e53dc 100644
> --- a/drivers/acpi/acpi_tad.c
> +++ b/drivers/acpi/acpi_tad.c
> @@ -27,6 +27,7 @@
> #include <linux/pm_runtime.h>
> #include <linux/suspend.h>
>
> +MODULE_DESCRIPTION("ACPI Time and Alarm (TAD) Device Driver");
> MODULE_LICENSE("GPL v2");
> MODULE_AUTHOR("Rafael J. Wysocki");
>
> diff --git a/drivers/acpi/platform_profile.c b/drivers/acpi/platform_profile.c
> index 4a9704730224..d2f7fd7743a1 100644
> --- a/drivers/acpi/platform_profile.c
> +++ b/drivers/acpi/platform_profile.c
> @@ -217,4 +217,5 @@ int platform_profile_remove(void)
> EXPORT_SYMBOL_GPL(platform_profile_remove);
>
> MODULE_AUTHOR("Mark Pearson <[email protected]>");
> +MODULE_DESCRIPTION("ACPI platform profile sysfs interface");
> MODULE_LICENSE("GPL");
>
> ---

Applied as 6.11 material, thanks!