Re: [PATCH -next] powerpc/perf: Make symbol 'isa207_pmu_format_attr' static
On Fri, 9 Apr 2021 17:01:19 +0800, Bixuan Cui wrote: > The sparse tool complains as follows: > > arch/powerpc/perf/isa207-common.c:24:18: warning: > symbol 'isa207_pmu_format_attr' was not declared. Should it be static? > > This symbol is not used outside of isa207-common.c, so this > commit marks it static. Applied to powerpc/next. [1/1] powerpc/perf: Make symbol 'isa207_pmu_format_attr' static https://git.kernel.org/powerpc/c/107dadb046178173dea18e0a78ff8ea3cc27c213 cheers
Re: [PATCH -next] powerpc/perf: Make symbol 'isa207_pmu_format_attr' static
On 4/9/21 2:31 PM, Bixuan Cui wrote: > The sparse tool complains as follows: > > arch/powerpc/perf/isa207-common.c:24:18: warning: > symbol 'isa207_pmu_format_attr' was not declared. Should it be static? > > This symbol is not used outside of isa207-common.c, so this > commit marks it static. Patch looks good to me. Reviewed-by: Kajol Jain Thanks, Kajol Jain > > Reported-by: Hulk Robot > Signed-off-by: Bixuan Cui > --- > arch/powerpc/perf/isa207-common.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/powerpc/perf/isa207-common.c > b/arch/powerpc/perf/isa207-common.c > index e4f577da33d8..487f9e914b5c 100644 > --- a/arch/powerpc/perf/isa207-common.c > +++ b/arch/powerpc/perf/isa207-common.c > @@ -21,7 +21,7 @@ PMU_FORMAT_ATTR(thresh_stop,"config:32-35"); > PMU_FORMAT_ATTR(thresh_start,"config:36-39"); > PMU_FORMAT_ATTR(thresh_cmp, "config:40-49"); > > -struct attribute *isa207_pmu_format_attr[] = { > +static struct attribute *isa207_pmu_format_attr[] = { > _attr_event.attr, > _attr_pmcxsel.attr, > _attr_mark.attr, >
[PATCH -next] powerpc/perf: Make symbol 'isa207_pmu_format_attr' static
The sparse tool complains as follows: arch/powerpc/perf/isa207-common.c:24:18: warning: symbol 'isa207_pmu_format_attr' was not declared. Should it be static? This symbol is not used outside of isa207-common.c, so this commit marks it static. Reported-by: Hulk Robot Signed-off-by: Bixuan Cui --- arch/powerpc/perf/isa207-common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/perf/isa207-common.c b/arch/powerpc/perf/isa207-common.c index e4f577da33d8..487f9e914b5c 100644 --- a/arch/powerpc/perf/isa207-common.c +++ b/arch/powerpc/perf/isa207-common.c @@ -21,7 +21,7 @@ PMU_FORMAT_ATTR(thresh_stop, "config:32-35"); PMU_FORMAT_ATTR(thresh_start, "config:36-39"); PMU_FORMAT_ATTR(thresh_cmp,"config:40-49"); -struct attribute *isa207_pmu_format_attr[] = { +static struct attribute *isa207_pmu_format_attr[] = { _attr_event.attr, _attr_pmcxsel.attr, _attr_mark.attr,