linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH -next] powerpc/papr_scm: Fix warnings about undeclared variable
@ 2020-09-18  8:59 Wang Wensheng
  2020-09-24 11:46 ` Vaibhav Jain
  2020-10-07  3:21 ` Michael Ellerman
  0 siblings, 2 replies; 3+ messages in thread
From: Wang Wensheng @ 2020-09-18  8:59 UTC (permalink / raw)
  To: linuxppc-dev
  Cc: mpe, benh, paulus, aneesh.kumar, dan.j.williams, vaibhav,
	ira.weiny, santosh, linux-kernel

Build the kernel with 'make C=2':
arch/powerpc/platforms/pseries/papr_scm.c:825:1: warning: symbol
'dev_attr_perf_stats' was not declared. Should it be static?

Signed-off-by: Wang Wensheng <wangwensheng4@huawei.com>
---
 arch/powerpc/platforms/pseries/papr_scm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/platforms/pseries/papr_scm.c b/arch/powerpc/platforms/pseries/papr_scm.c
index 5493bc847bd0..a95aa425e7d4 100644
--- a/arch/powerpc/platforms/pseries/papr_scm.c
+++ b/arch/powerpc/platforms/pseries/papr_scm.c
@@ -823,7 +823,7 @@ static ssize_t perf_stats_show(struct device *dev,
 	kfree(stats);
 	return rc ? rc : (ssize_t)seq_buf_used(&s);
 }
-DEVICE_ATTR_ADMIN_RO(perf_stats);
+static DEVICE_ATTR_ADMIN_RO(perf_stats);
 
 static ssize_t flags_show(struct device *dev,
 			  struct device_attribute *attr, char *buf)
-- 
2.25.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH -next] powerpc/papr_scm: Fix warnings about undeclared variable
  2020-09-18  8:59 [PATCH -next] powerpc/papr_scm: Fix warnings about undeclared variable Wang Wensheng
@ 2020-09-24 11:46 ` Vaibhav Jain
  2020-10-07  3:21 ` Michael Ellerman
  1 sibling, 0 replies; 3+ messages in thread
From: Vaibhav Jain @ 2020-09-24 11:46 UTC (permalink / raw)
  To: Wang Wensheng, linuxppc-dev
  Cc: mpe, benh, paulus, aneesh.kumar, dan.j.williams, ira.weiny,
	santosh, linux-kernel


Thanks for the patch. This looks good to me.

Wang Wensheng <wangwensheng4@huawei.com> writes:

> Build the kernel with 'make C=2':
> arch/powerpc/platforms/pseries/papr_scm.c:825:1: warning: symbol
> 'dev_attr_perf_stats' was not declared. Should it be static?

> Signed-off-by: Wang Wensheng <wangwensheng4@huawei.com>
Reviewed-by: Vaibhav Jain <vaibhav@linux.ibm.com>

-- 
Cheers
~ Vaibhav

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH -next] powerpc/papr_scm: Fix warnings about undeclared variable
  2020-09-18  8:59 [PATCH -next] powerpc/papr_scm: Fix warnings about undeclared variable Wang Wensheng
  2020-09-24 11:46 ` Vaibhav Jain
@ 2020-10-07  3:21 ` Michael Ellerman
  1 sibling, 0 replies; 3+ messages in thread
From: Michael Ellerman @ 2020-10-07  3:21 UTC (permalink / raw)
  To: Wang Wensheng, linuxppc-dev
  Cc: vaibhav, linux-kernel, aneesh.kumar, paulus, santosh, ira.weiny,
	dan.j.williams

On Fri, 18 Sep 2020 08:59:51 +0000, Wang Wensheng wrote:
> Build the kernel with 'make C=2':
> arch/powerpc/platforms/pseries/papr_scm.c:825:1: warning: symbol
> 'dev_attr_perf_stats' was not declared. Should it be static?

Applied to powerpc/next.

[1/1] powerpc/papr_scm: Fix warnings about undeclared variable
      https://git.kernel.org/powerpc/c/4366337490cbe5a35b50e83755be629a502ec7e2

cheers

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-10-07  3:21 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-18  8:59 [PATCH -next] powerpc/papr_scm: Fix warnings about undeclared variable Wang Wensheng
2020-09-24 11:46 ` Vaibhav Jain
2020-10-07  3:21 ` Michael Ellerman

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).