diff options
| author | Rob Herring (Arm) <[email protected]> | 2024-06-26 16:32:26 -0600 |
|---|---|---|
| committer | Will Deacon <[email protected]> | 2024-07-03 14:07:14 +0100 |
| commit | 598c1a2d9f4ba8a04cf92af8e988052179e31199 (patch) | |
| tree | 1585105c9d4fa64876d5eca22ef508e6f9470d01 | |
| parent | 81e15ca3e523a508d62806fe681c1d289361ca16 (diff) | |
perf: arm_pmuv3: Drop unnecessary IS_ENABLED(CONFIG_ARM64) check
The IS_ENABLED(CONFIG_ARM64) check for threshold support is unnecessary.
The purpose is to not enable thresholds on arm32, but if threshold is
non-zero, the check against threshold_max() just above here will have
errored out because threshold_max() is always 0 on arm32.
Signed-off-by: Rob Herring (Arm) <[email protected]>
Acked-by: Mark rutland <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Will Deacon <[email protected]>
| -rw-r--r-- | drivers/perf/arm_pmuv3.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/perf/arm_pmuv3.c b/drivers/perf/arm_pmuv3.c index 02b2ac4cd297..d6248493b5c6 100644 --- a/drivers/perf/arm_pmuv3.c +++ b/drivers/perf/arm_pmuv3.c @@ -1045,7 +1045,7 @@ static int armv8pmu_set_event_filter(struct hw_perf_event *event, return -EINVAL; } - if (IS_ENABLED(CONFIG_ARM64) && th) { + if (th) { config_base |= FIELD_PREP(ARMV8_PMU_EVTYPE_TH, th); config_base |= FIELD_PREP(ARMV8_PMU_EVTYPE_TC, armv8pmu_event_threshold_control(attr)); |