[5/5] ARC: perf: remove useless ifdefs

Message ID 20181205170609.18690-6-Eugeniy.Paltsev@synopsys.com
State New
Headers show
Series
  • introduce Kernel PMU events support
Related show

Commit Message

Eugeniy Paltsev Dec. 5, 2018, 5:06 p.m.
ifdef around 'arc_pmu_match' structure declaration is useless
as we refer to 'arc_pmu_match' in several places which aren't
guarded with ifdef.

Nevertheless 'ARC' option selects 'OF' unconditionally so we
can simply get rid of this ifdef.

Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>
---
 arch/arc/kernel/perf_event.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Vineet Gupta Dec. 5, 2018, 5:14 p.m. | #1
On 12/5/18 9:06 AM, Eugeniy Paltsev wrote:
> ifdef around 'arc_pmu_match' structure declaration is useless
> as we refer to 'arc_pmu_match' in several places which aren't
> guarded with ifdef.
>
> Nevertheless 'ARC' option selects 'OF' unconditionally so we
> can simply get rid of this ifdef.
>
> Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>


Maybe squash this into trivial cleanups 1/5 ?

Acked-by: Vineet Gupta <vgupta@synopsys.com>

Thx,
-Vineet

Patch

diff --git a/arch/arc/kernel/perf_event.c b/arch/arc/kernel/perf_event.c
index b6207090aaa7..7786c1f005fb 100644
--- a/arch/arc/kernel/perf_event.c
+++ b/arch/arc/kernel/perf_event.c
@@ -643,14 +643,12 @@  static int arc_pmu_device_probe(struct platform_device *pdev)
 	return perf_pmu_register(&arc_pmu->pmu, "arc_pmu", PERF_TYPE_RAW);
 }
 
-#ifdef CONFIG_OF
 static const struct of_device_id arc_pmu_match[] = {
 	{ .compatible = "snps,arc700-pct" },
 	{ .compatible = "snps,archs-pct" },
 	{},
 };
 MODULE_DEVICE_TABLE(of, arc_pmu_match);
-#endif
 
 static struct platform_driver arc_pmu_driver = {
 	.driver	= {