* [PATCH -next] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
@ 2020-07-14 14:24 ` Wei Yongjun
0 siblings, 0 replies; 8+ messages in thread
From: Wei Yongjun @ 2020-07-14 14:24 UTC (permalink / raw)
To: Hulk Robot, Rafael J. Wysocki, Daniel Lezcano, Michael Ellerman
Cc: Wei Yongjun, linux-pm, linuxppc-dev
The sparse tool complains as follows:
drivers/cpuidle/cpuidle-pseries.c:25:23: warning:
symbol 'pseries_idle_driver' was not declared. Should it be static?
'pseries_idle_driver' is not used outside of this file, so marks
it static.
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
---
drivers/cpuidle/cpuidle-pseries.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/cpuidle/cpuidle-pseries.c b/drivers/cpuidle/cpuidle-pseries.c
index 6513ef2af66a..3e058ad2bb51 100644
--- a/drivers/cpuidle/cpuidle-pseries.c
+++ b/drivers/cpuidle/cpuidle-pseries.c
@@ -22,7 +22,7 @@
#include <asm/idle.h>
#include <asm/plpar_wrappers.h>
-struct cpuidle_driver pseries_idle_driver = {
+static struct cpuidle_driver pseries_idle_driver = {
.name = "pseries_idle",
.owner = THIS_MODULE,
};
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH -next] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
@ 2020-07-14 14:24 ` Wei Yongjun
0 siblings, 0 replies; 8+ messages in thread
From: Wei Yongjun @ 2020-07-14 14:24 UTC (permalink / raw)
To: Hulk Robot, Rafael J. Wysocki, Daniel Lezcano, Michael Ellerman
Cc: linuxppc-dev, Wei Yongjun, linux-pm
The sparse tool complains as follows:
drivers/cpuidle/cpuidle-pseries.c:25:23: warning:
symbol 'pseries_idle_driver' was not declared. Should it be static?
'pseries_idle_driver' is not used outside of this file, so marks
it static.
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
---
drivers/cpuidle/cpuidle-pseries.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/cpuidle/cpuidle-pseries.c b/drivers/cpuidle/cpuidle-pseries.c
index 6513ef2af66a..3e058ad2bb51 100644
--- a/drivers/cpuidle/cpuidle-pseries.c
+++ b/drivers/cpuidle/cpuidle-pseries.c
@@ -22,7 +22,7 @@
#include <asm/idle.h>
#include <asm/plpar_wrappers.h>
-struct cpuidle_driver pseries_idle_driver = {
+static struct cpuidle_driver pseries_idle_driver = {
.name = "pseries_idle",
.owner = THIS_MODULE,
};
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH -next] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
2020-07-14 14:24 ` Wei Yongjun
@ 2020-07-15 15:31 ` Rafael J. Wysocki
-1 siblings, 0 replies; 8+ messages in thread
From: Rafael J. Wysocki @ 2020-07-15 15:31 UTC (permalink / raw)
To: Wei Yongjun
Cc: Hulk Robot, Rafael J. Wysocki, Daniel Lezcano, Michael Ellerman,
Linux PM, linuxppc-dev
On Tue, Jul 14, 2020 at 4:14 PM Wei Yongjun <weiyongjun1@huawei.com> wrote:
>
> The sparse tool complains as follows:
>
> drivers/cpuidle/cpuidle-pseries.c:25:23: warning:
> symbol 'pseries_idle_driver' was not declared. Should it be static?
>
> 'pseries_idle_driver' is not used outside of this file, so marks
> it static.
>
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
> ---
> drivers/cpuidle/cpuidle-pseries.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/cpuidle/cpuidle-pseries.c b/drivers/cpuidle/cpuidle-pseries.c
> index 6513ef2af66a..3e058ad2bb51 100644
> --- a/drivers/cpuidle/cpuidle-pseries.c
> +++ b/drivers/cpuidle/cpuidle-pseries.c
> @@ -22,7 +22,7 @@
> #include <asm/idle.h>
> #include <asm/plpar_wrappers.h>
>
> -struct cpuidle_driver pseries_idle_driver = {
> +static struct cpuidle_driver pseries_idle_driver = {
> .name = "pseries_idle",
> .owner = THIS_MODULE,
> };
Applied as 5.9 material, thanks!
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
@ 2020-07-15 15:31 ` Rafael J. Wysocki
0 siblings, 0 replies; 8+ messages in thread
From: Rafael J. Wysocki @ 2020-07-15 15:31 UTC (permalink / raw)
To: Wei Yongjun
Cc: Linux PM, Daniel Lezcano, Rafael J. Wysocki, Hulk Robot, linuxppc-dev
On Tue, Jul 14, 2020 at 4:14 PM Wei Yongjun <weiyongjun1@huawei.com> wrote:
>
> The sparse tool complains as follows:
>
> drivers/cpuidle/cpuidle-pseries.c:25:23: warning:
> symbol 'pseries_idle_driver' was not declared. Should it be static?
>
> 'pseries_idle_driver' is not used outside of this file, so marks
> it static.
>
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
> ---
> drivers/cpuidle/cpuidle-pseries.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/cpuidle/cpuidle-pseries.c b/drivers/cpuidle/cpuidle-pseries.c
> index 6513ef2af66a..3e058ad2bb51 100644
> --- a/drivers/cpuidle/cpuidle-pseries.c
> +++ b/drivers/cpuidle/cpuidle-pseries.c
> @@ -22,7 +22,7 @@
> #include <asm/idle.h>
> #include <asm/plpar_wrappers.h>
>
> -struct cpuidle_driver pseries_idle_driver = {
> +static struct cpuidle_driver pseries_idle_driver = {
> .name = "pseries_idle",
> .owner = THIS_MODULE,
> };
Applied as 5.9 material, thanks!
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
2020-07-14 14:24 ` Wei Yongjun
@ 2020-07-16 12:56 ` Michael Ellerman
-1 siblings, 0 replies; 8+ messages in thread
From: Michael Ellerman @ 2020-07-16 12:56 UTC (permalink / raw)
To: Hulk Robot, Wei Yongjun, Daniel Lezcano, Rafael J. Wysocki,
Michael Ellerman
Cc: linux-pm, linuxppc-dev
On Tue, 14 Jul 2020 22:24:24 +0800, Wei Yongjun wrote:
> The sparse tool complains as follows:
>
> drivers/cpuidle/cpuidle-pseries.c:25:23: warning:
> symbol 'pseries_idle_driver' was not declared. Should it be static?
>
> 'pseries_idle_driver' is not used outside of this file, so marks
> it static.
Applied to powerpc/next.
[1/1] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
https://git.kernel.org/powerpc/c/92fe8483b1660feaa602d8be6ca7efe95ae4789b
cheers
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
@ 2020-07-16 12:56 ` Michael Ellerman
0 siblings, 0 replies; 8+ messages in thread
From: Michael Ellerman @ 2020-07-16 12:56 UTC (permalink / raw)
To: Hulk Robot, Wei Yongjun, Daniel Lezcano, Rafael J. Wysocki,
Michael Ellerman
Cc: linuxppc-dev, linux-pm
On Tue, 14 Jul 2020 22:24:24 +0800, Wei Yongjun wrote:
> The sparse tool complains as follows:
>
> drivers/cpuidle/cpuidle-pseries.c:25:23: warning:
> symbol 'pseries_idle_driver' was not declared. Should it be static?
>
> 'pseries_idle_driver' is not used outside of this file, so marks
> it static.
Applied to powerpc/next.
[1/1] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
https://git.kernel.org/powerpc/c/92fe8483b1660feaa602d8be6ca7efe95ae4789b
cheers
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
2020-07-16 12:56 ` Michael Ellerman
@ 2020-07-17 4:41 ` Daniel Lezcano
-1 siblings, 0 replies; 8+ messages in thread
From: Daniel Lezcano @ 2020-07-17 4:41 UTC (permalink / raw)
To: Michael Ellerman, Hulk Robot, Wei Yongjun, Rafael J. Wysocki,
Michael Ellerman
Cc: linux-pm, linuxppc-dev
On 16/07/2020 14:56, Michael Ellerman wrote:
> On Tue, 14 Jul 2020 22:24:24 +0800, Wei Yongjun wrote:
>> The sparse tool complains as follows:
>>
>> drivers/cpuidle/cpuidle-pseries.c:25:23: warning:
>> symbol 'pseries_idle_driver' was not declared. Should it be static?
>>
>> 'pseries_idle_driver' is not used outside of this file, so marks
>> it static.
>
> Applied to powerpc/next.
>
> [1/1] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
> https://git.kernel.org/powerpc/c/92fe8483b1660feaa602d8be6ca7efe95ae4789b
Rafael already picked the patch.
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH -next] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
@ 2020-07-17 4:41 ` Daniel Lezcano
0 siblings, 0 replies; 8+ messages in thread
From: Daniel Lezcano @ 2020-07-17 4:41 UTC (permalink / raw)
To: Michael Ellerman, Hulk Robot, Wei Yongjun, Rafael J. Wysocki,
Michael Ellerman
Cc: linuxppc-dev, linux-pm
On 16/07/2020 14:56, Michael Ellerman wrote:
> On Tue, 14 Jul 2020 22:24:24 +0800, Wei Yongjun wrote:
>> The sparse tool complains as follows:
>>
>> drivers/cpuidle/cpuidle-pseries.c:25:23: warning:
>> symbol 'pseries_idle_driver' was not declared. Should it be static?
>>
>> 'pseries_idle_driver' is not used outside of this file, so marks
>> it static.
>
> Applied to powerpc/next.
>
> [1/1] cpuidle/pseries: Make symbol 'pseries_idle_driver' static
> https://git.kernel.org/powerpc/c/92fe8483b1660feaa602d8be6ca7efe95ae4789b
Rafael already picked the patch.
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2020-07-17 4:43 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-14 14:24 [PATCH -next] cpuidle/pseries: Make symbol 'pseries_idle_driver' static Wei Yongjun
2020-07-14 14:24 ` Wei Yongjun
2020-07-15 15:31 ` Rafael J. Wysocki
2020-07-15 15:31 ` Rafael J. Wysocki
2020-07-16 12:56 ` Michael Ellerman
2020-07-16 12:56 ` Michael Ellerman
2020-07-17 4:41 ` Daniel Lezcano
2020-07-17 4:41 ` Daniel Lezcano
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.