* [PATCH] cpufreq: powerpc: add missing MODULE_DESCRIPTION() macros
@ 2024-06-15 6:08 Jeff Johnson
2024-06-29 3:01 ` Jeff Johnson
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Jeff Johnson @ 2024-06-15 6:08 UTC (permalink / raw)
To: Rafael J. Wysocki, Viresh Kumar, Michael Ellerman,
Nicholas Piggin, Christophe Leroy, Naveen N. Rao
Cc: kernel-janitors, linuxppc-dev, linux-kernel, Jeff Johnson,
linux-pm
With ARCH=powerpc, make allmodconfig && make W=1 C=1 reports:
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/ppc-cbe-cpufreq.o
WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/powernv-cpufreq.o
Add the missing invocation of the MODULE_DESCRIPTION() macro to all
files which have a MODULE_LICENSE().
This includes three additional files which, although they did not
produce a warning with the powerpc allmodconfig configuration, may
cause this warning with specific options enabled in the kernel
configuration.
Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
---
Corrections to these descriptions are welcomed. I'm not an expert in
this code so in most cases I've taken these descriptions directly from
code comments, Kconfig descriptions, or git logs. History has shown
that in some cases these are originally wrong due to cut-n-paste
errors, and in other cases the drivers have evolved such that the
original information is no longer accurate.
---
drivers/cpufreq/maple-cpufreq.c | 1 +
drivers/cpufreq/pasemi-cpufreq.c | 1 +
drivers/cpufreq/pmac64-cpufreq.c | 1 +
drivers/cpufreq/powernv-cpufreq.c | 1 +
drivers/cpufreq/ppc_cbe_cpufreq.c | 1 +
5 files changed, 5 insertions(+)
diff --git a/drivers/cpufreq/maple-cpufreq.c b/drivers/cpufreq/maple-cpufreq.c
index f9306410a07f..19ca7f874d28 100644
--- a/drivers/cpufreq/maple-cpufreq.c
+++ b/drivers/cpufreq/maple-cpufreq.c
@@ -238,4 +238,5 @@ static int __init maple_cpufreq_init(void)
module_init(maple_cpufreq_init);
+MODULE_DESCRIPTION("cpufreq driver for Maple 970FX Evaluation Board");
MODULE_LICENSE("GPL");
diff --git a/drivers/cpufreq/pasemi-cpufreq.c b/drivers/cpufreq/pasemi-cpufreq.c
index 039a66bbe1be..92a99f09884a 100644
--- a/drivers/cpufreq/pasemi-cpufreq.c
+++ b/drivers/cpufreq/pasemi-cpufreq.c
@@ -271,5 +271,6 @@ static void __exit pas_cpufreq_exit(void)
module_init(pas_cpufreq_init);
module_exit(pas_cpufreq_exit);
+MODULE_DESCRIPTION("cpufreq driver for PA Semi PWRficient");
MODULE_LICENSE("GPL");
MODULE_AUTHOR("Egor Martovetsky <egor@pasemi.com>, Olof Johansson <olof@lixom.net>");
diff --git a/drivers/cpufreq/pmac64-cpufreq.c b/drivers/cpufreq/pmac64-cpufreq.c
index 2cd2b06849a2..9d3fe36075f8 100644
--- a/drivers/cpufreq/pmac64-cpufreq.c
+++ b/drivers/cpufreq/pmac64-cpufreq.c
@@ -671,4 +671,5 @@ static int __init g5_cpufreq_init(void)
module_init(g5_cpufreq_init);
+MODULE_DESCRIPTION("cpufreq driver for SMU & 970FX based G5 Macs");
MODULE_LICENSE("GPL");
diff --git a/drivers/cpufreq/powernv-cpufreq.c b/drivers/cpufreq/powernv-cpufreq.c
index fddbd1ea1635..e923f717e1d7 100644
--- a/drivers/cpufreq/powernv-cpufreq.c
+++ b/drivers/cpufreq/powernv-cpufreq.c
@@ -1162,5 +1162,6 @@ static void __exit powernv_cpufreq_exit(void)
}
module_exit(powernv_cpufreq_exit);
+MODULE_DESCRIPTION("cpufreq driver for the IBM POWER processors");
MODULE_LICENSE("GPL");
MODULE_AUTHOR("Vaidyanathan Srinivasan <svaidy at linux.vnet.ibm.com>");
diff --git a/drivers/cpufreq/ppc_cbe_cpufreq.c b/drivers/cpufreq/ppc_cbe_cpufreq.c
index 88afc49941b7..72f568d14f30 100644
--- a/drivers/cpufreq/ppc_cbe_cpufreq.c
+++ b/drivers/cpufreq/ppc_cbe_cpufreq.c
@@ -169,5 +169,6 @@ static void __exit cbe_cpufreq_exit(void)
module_init(cbe_cpufreq_init);
module_exit(cbe_cpufreq_exit);
+MODULE_DESCRIPTION("cpufreq driver for Cell BE processors");
MODULE_LICENSE("GPL");
MODULE_AUTHOR("Christian Krafft <krafft@de.ibm.com>");
---
base-commit: 83a7eefedc9b56fe7bfeff13b6c7356688ffa670
change-id: 20240614-md-powerpc-drivers-cpufreq-6d345e48164e
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] cpufreq: powerpc: add missing MODULE_DESCRIPTION() macros
2024-06-15 6:08 [PATCH] cpufreq: powerpc: add missing MODULE_DESCRIPTION() macros Jeff Johnson
@ 2024-06-29 3:01 ` Jeff Johnson
2024-07-01 8:33 ` Viresh Kumar
2024-07-22 7:13 ` Michael Ellerman
2 siblings, 0 replies; 6+ messages in thread
From: Jeff Johnson @ 2024-06-29 3:01 UTC (permalink / raw)
To: Rafael J. Wysocki, Viresh Kumar, Michael Ellerman,
Nicholas Piggin, Christophe Leroy, Naveen N. Rao
Cc: kernel-janitors, linuxppc-dev, linux-kernel, linux-pm
On 6/14/2024 11:08 PM, Jeff Johnson wrote:
> With ARCH=powerpc, make allmodconfig && make W=1 C=1 reports:
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/ppc-cbe-cpufreq.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/powernv-cpufreq.o
>
> Add the missing invocation of the MODULE_DESCRIPTION() macro to all
> files which have a MODULE_LICENSE().
>
> This includes three additional files which, although they did not
> produce a warning with the powerpc allmodconfig configuration, may
> cause this warning with specific options enabled in the kernel
> configuration.
>
> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
> ---
> Corrections to these descriptions are welcomed. I'm not an expert in
> this code so in most cases I've taken these descriptions directly from
> code comments, Kconfig descriptions, or git logs. History has shown
> that in some cases these are originally wrong due to cut-n-paste
> errors, and in other cases the drivers have evolved such that the
> original information is no longer accurate.
> ---
> drivers/cpufreq/maple-cpufreq.c | 1 +
> drivers/cpufreq/pasemi-cpufreq.c | 1 +
> drivers/cpufreq/pmac64-cpufreq.c | 1 +
> drivers/cpufreq/powernv-cpufreq.c | 1 +
> drivers/cpufreq/ppc_cbe_cpufreq.c | 1 +
> 5 files changed, 5 insertions(+)
>
> diff --git a/drivers/cpufreq/maple-cpufreq.c b/drivers/cpufreq/maple-cpufreq.c
> index f9306410a07f..19ca7f874d28 100644
> --- a/drivers/cpufreq/maple-cpufreq.c
> +++ b/drivers/cpufreq/maple-cpufreq.c
> @@ -238,4 +238,5 @@ static int __init maple_cpufreq_init(void)
> module_init(maple_cpufreq_init);
>
>
> +MODULE_DESCRIPTION("cpufreq driver for Maple 970FX Evaluation Board");
> MODULE_LICENSE("GPL");
> diff --git a/drivers/cpufreq/pasemi-cpufreq.c b/drivers/cpufreq/pasemi-cpufreq.c
> index 039a66bbe1be..92a99f09884a 100644
> --- a/drivers/cpufreq/pasemi-cpufreq.c
> +++ b/drivers/cpufreq/pasemi-cpufreq.c
> @@ -271,5 +271,6 @@ static void __exit pas_cpufreq_exit(void)
> module_init(pas_cpufreq_init);
> module_exit(pas_cpufreq_exit);
>
> +MODULE_DESCRIPTION("cpufreq driver for PA Semi PWRficient");
> MODULE_LICENSE("GPL");
> MODULE_AUTHOR("Egor Martovetsky <egor@pasemi.com>, Olof Johansson <olof@lixom.net>");
> diff --git a/drivers/cpufreq/pmac64-cpufreq.c b/drivers/cpufreq/pmac64-cpufreq.c
> index 2cd2b06849a2..9d3fe36075f8 100644
> --- a/drivers/cpufreq/pmac64-cpufreq.c
> +++ b/drivers/cpufreq/pmac64-cpufreq.c
> @@ -671,4 +671,5 @@ static int __init g5_cpufreq_init(void)
> module_init(g5_cpufreq_init);
>
>
> +MODULE_DESCRIPTION("cpufreq driver for SMU & 970FX based G5 Macs");
> MODULE_LICENSE("GPL");
> diff --git a/drivers/cpufreq/powernv-cpufreq.c b/drivers/cpufreq/powernv-cpufreq.c
> index fddbd1ea1635..e923f717e1d7 100644
> --- a/drivers/cpufreq/powernv-cpufreq.c
> +++ b/drivers/cpufreq/powernv-cpufreq.c
> @@ -1162,5 +1162,6 @@ static void __exit powernv_cpufreq_exit(void)
> }
> module_exit(powernv_cpufreq_exit);
>
> +MODULE_DESCRIPTION("cpufreq driver for the IBM POWER processors");
> MODULE_LICENSE("GPL");
> MODULE_AUTHOR("Vaidyanathan Srinivasan <svaidy at linux.vnet.ibm.com>");
> diff --git a/drivers/cpufreq/ppc_cbe_cpufreq.c b/drivers/cpufreq/ppc_cbe_cpufreq.c
> index 88afc49941b7..72f568d14f30 100644
> --- a/drivers/cpufreq/ppc_cbe_cpufreq.c
> +++ b/drivers/cpufreq/ppc_cbe_cpufreq.c
> @@ -169,5 +169,6 @@ static void __exit cbe_cpufreq_exit(void)
> module_init(cbe_cpufreq_init);
> module_exit(cbe_cpufreq_exit);
>
> +MODULE_DESCRIPTION("cpufreq driver for Cell BE processors");
> MODULE_LICENSE("GPL");
> MODULE_AUTHOR("Christian Krafft <krafft@de.ibm.com>");
>
> ---
> base-commit: 83a7eefedc9b56fe7bfeff13b6c7356688ffa670
> change-id: 20240614-md-powerpc-drivers-cpufreq-6d345e48164e
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] 6+ messages in thread
* Re: [PATCH] cpufreq: powerpc: add missing MODULE_DESCRIPTION() macros
2024-06-15 6:08 [PATCH] cpufreq: powerpc: add missing MODULE_DESCRIPTION() macros Jeff Johnson
2024-06-29 3:01 ` Jeff Johnson
@ 2024-07-01 8:33 ` Viresh Kumar
2024-07-15 19:35 ` Jeff Johnson
2024-07-22 7:13 ` Michael Ellerman
2 siblings, 1 reply; 6+ messages in thread
From: Viresh Kumar @ 2024-07-01 8:33 UTC (permalink / raw)
To: Jeff Johnson
Cc: linux-pm, Rafael J. Wysocki, kernel-janitors, linux-kernel,
Christophe Leroy, Nicholas Piggin, Naveen N. Rao, linuxppc-dev
On 14-06-24, 23:08, Jeff Johnson wrote:
> With ARCH=powerpc, make allmodconfig && make W=1 C=1 reports:
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/ppc-cbe-cpufreq.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/powernv-cpufreq.o
>
> Add the missing invocation of the MODULE_DESCRIPTION() macro to all
> files which have a MODULE_LICENSE().
>
> This includes three additional files which, although they did not
> produce a warning with the powerpc allmodconfig configuration, may
> cause this warning with specific options enabled in the kernel
> configuration.
>
> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
--
viresh
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] cpufreq: powerpc: add missing MODULE_DESCRIPTION() macros
2024-07-01 8:33 ` Viresh Kumar
@ 2024-07-15 19:35 ` Jeff Johnson
0 siblings, 0 replies; 6+ messages in thread
From: Jeff Johnson @ 2024-07-15 19:35 UTC (permalink / raw)
To: Viresh Kumar
Cc: linux-pm, Rafael J. Wysocki, kernel-janitors, linux-kernel,
Christophe Leroy, Nicholas Piggin, Naveen N. Rao, linuxppc-dev
On 7/1/2024 1:33 AM, Viresh Kumar wrote:
> On 14-06-24, 23:08, Jeff Johnson wrote:
>> With ARCH=powerpc, make allmodconfig && make W=1 C=1 reports:
>> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/ppc-cbe-cpufreq.o
>> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/powernv-cpufreq.o
>>
>> Add the missing invocation of the MODULE_DESCRIPTION() macro to all
>> files which have a MODULE_LICENSE().
>>
>> This includes three additional files which, although they did not
>> produce a warning with the powerpc allmodconfig configuration, may
>> cause this warning with specific options enabled in the kernel
>> configuration.
>>
>> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
>
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
>
I still don't see this in linux-next. Is anything else needed from me?
Of the almost 300 patches I've submitted to fix these issues tree-wide, this
is one of the 13 remaining. Hopefully this can make it into the 6.11 merge
window. If not, Greg KH has indicated he'll take this as an -rc instead of
waiting for 6.12.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] cpufreq: powerpc: add missing MODULE_DESCRIPTION() macros
2024-06-15 6:08 [PATCH] cpufreq: powerpc: add missing MODULE_DESCRIPTION() macros Jeff Johnson
2024-06-29 3:01 ` Jeff Johnson
2024-07-01 8:33 ` Viresh Kumar
@ 2024-07-22 7:13 ` Michael Ellerman
2024-07-22 15:23 ` Jeff Johnson
2 siblings, 1 reply; 6+ messages in thread
From: Michael Ellerman @ 2024-07-22 7:13 UTC (permalink / raw)
To: Jeff Johnson, Rafael J. Wysocki, Viresh Kumar, Nicholas Piggin,
Christophe Leroy, Naveen N. Rao
Cc: kernel-janitors, linuxppc-dev, linux-kernel, Jeff Johnson,
linux-pm
Jeff Johnson <quic_jjohnson@quicinc.com> writes:
> With ARCH=powerpc, make allmodconfig && make W=1 C=1 reports:
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/ppc-cbe-cpufreq.o
> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/powernv-cpufreq.o
>
> Add the missing invocation of the MODULE_DESCRIPTION() macro to all
> files which have a MODULE_LICENSE().
>
> This includes three additional files which, although they did not
> produce a warning with the powerpc allmodconfig configuration, may
> cause this warning with specific options enabled in the kernel
> configuration.
>
> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
> ---
> Corrections to these descriptions are welcomed. I'm not an expert in
> this code so in most cases I've taken these descriptions directly from
> code comments, Kconfig descriptions, or git logs. History has shown
> that in some cases these are originally wrong due to cut-n-paste
> errors, and in other cases the drivers have evolved such that the
> original information is no longer accurate.
> ---
> drivers/cpufreq/maple-cpufreq.c | 1 +
> drivers/cpufreq/pasemi-cpufreq.c | 1 +
> drivers/cpufreq/pmac64-cpufreq.c | 1 +
> drivers/cpufreq/powernv-cpufreq.c | 1 +
> drivers/cpufreq/ppc_cbe_cpufreq.c | 1 +
> 5 files changed, 5 insertions(+)
>
> diff --git a/drivers/cpufreq/maple-cpufreq.c b/drivers/cpufreq/maple-cpufreq.c
> index f9306410a07f..19ca7f874d28 100644
> --- a/drivers/cpufreq/maple-cpufreq.c
> +++ b/drivers/cpufreq/maple-cpufreq.c
> @@ -238,4 +238,5 @@ static int __init maple_cpufreq_init(void)
> module_init(maple_cpufreq_init);
>
>
> +MODULE_DESCRIPTION("cpufreq driver for Maple 970FX Evaluation Board");
Can you change this one to:
"cpufreq driver for Maple 970FX/970MP boards");
It looks for both those CPUs in probe.
> diff --git a/drivers/cpufreq/powernv-cpufreq.c b/drivers/cpufreq/powernv-cpufreq.c
> index fddbd1ea1635..e923f717e1d7 100644
> --- a/drivers/cpufreq/powernv-cpufreq.c
> +++ b/drivers/cpufreq/powernv-cpufreq.c
> @@ -1162,5 +1162,6 @@ static void __exit powernv_cpufreq_exit(void)
> }
> module_exit(powernv_cpufreq_exit);
>
> +MODULE_DESCRIPTION("cpufreq driver for the IBM POWER processors");
This one's tricky, because it probes based on the device tree, though it
is restricted to CONFIG_POWERNV. It also supports non-IBM CPUs in theory
at least. Maybe something like:
"cpufreq driver for IBM/OpenPOWER powernv systems");
cheers
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] cpufreq: powerpc: add missing MODULE_DESCRIPTION() macros
2024-07-22 7:13 ` Michael Ellerman
@ 2024-07-22 15:23 ` Jeff Johnson
0 siblings, 0 replies; 6+ messages in thread
From: Jeff Johnson @ 2024-07-22 15:23 UTC (permalink / raw)
To: Michael Ellerman, Rafael J. Wysocki, Viresh Kumar,
Nicholas Piggin, Christophe Leroy, Naveen N. Rao
Cc: kernel-janitors, linuxppc-dev, linux-kernel, linux-pm
On 7/22/2024 12:13 AM, Michael Ellerman wrote:
> Jeff Johnson <quic_jjohnson@quicinc.com> writes:
>> With ARCH=powerpc, make allmodconfig && make W=1 C=1 reports:
>> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/ppc-cbe-cpufreq.o
>> WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/cpufreq/powernv-cpufreq.o
>>
>> Add the missing invocation of the MODULE_DESCRIPTION() macro to all
>> files which have a MODULE_LICENSE().
>>
>> This includes three additional files which, although they did not
>> produce a warning with the powerpc allmodconfig configuration, may
>> cause this warning with specific options enabled in the kernel
>> configuration.
>>
>> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
>> ---
>> Corrections to these descriptions are welcomed. I'm not an expert in
>> this code so in most cases I've taken these descriptions directly from
>> code comments, Kconfig descriptions, or git logs. History has shown
>> that in some cases these are originally wrong due to cut-n-paste
>> errors, and in other cases the drivers have evolved such that the
>> original information is no longer accurate.
>> ---
>> drivers/cpufreq/maple-cpufreq.c | 1 +
>> drivers/cpufreq/pasemi-cpufreq.c | 1 +
>> drivers/cpufreq/pmac64-cpufreq.c | 1 +
>> drivers/cpufreq/powernv-cpufreq.c | 1 +
>> drivers/cpufreq/ppc_cbe_cpufreq.c | 1 +
>> 5 files changed, 5 insertions(+)
>>
>> diff --git a/drivers/cpufreq/maple-cpufreq.c b/drivers/cpufreq/maple-cpufreq.c
>> index f9306410a07f..19ca7f874d28 100644
>> --- a/drivers/cpufreq/maple-cpufreq.c
>> +++ b/drivers/cpufreq/maple-cpufreq.c
>> @@ -238,4 +238,5 @@ static int __init maple_cpufreq_init(void)
>> module_init(maple_cpufreq_init);
>>
>>
>> +MODULE_DESCRIPTION("cpufreq driver for Maple 970FX Evaluation Board");
>
> Can you change this one to:
>
> "cpufreq driver for Maple 970FX/970MP boards");
>
> It looks for both those CPUs in probe.
>
>> diff --git a/drivers/cpufreq/powernv-cpufreq.c b/drivers/cpufreq/powernv-cpufreq.c
>> index fddbd1ea1635..e923f717e1d7 100644
>> --- a/drivers/cpufreq/powernv-cpufreq.c
>> +++ b/drivers/cpufreq/powernv-cpufreq.c
>> @@ -1162,5 +1162,6 @@ static void __exit powernv_cpufreq_exit(void)
>> }
>> module_exit(powernv_cpufreq_exit);
>>
>> +MODULE_DESCRIPTION("cpufreq driver for the IBM POWER processors");
>
> This one's tricky, because it probes based on the device tree, though it
> is restricted to CONFIG_POWERNV. It also supports non-IBM CPUs in theory
> at least. Maybe something like:
>
> "cpufreq driver for IBM/OpenPOWER powernv systems");
>
> cheers
Sure, I'll send an update shortly.
/jeff
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-07-22 15:25 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-15 6:08 [PATCH] cpufreq: powerpc: add missing MODULE_DESCRIPTION() macros Jeff Johnson
2024-06-29 3:01 ` Jeff Johnson
2024-07-01 8:33 ` Viresh Kumar
2024-07-15 19:35 ` Jeff Johnson
2024-07-22 7:13 ` Michael Ellerman
2024-07-22 15:23 ` Jeff Johnson
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).