diff options
| author | Jiucheng Xu <jiucheng.xu@amlogic.com> | 2023-02-09 12:54:01 +0100 |
|---|---|---|
| committer | Will Deacon <will@kernel.org> | 2023-03-27 16:12:05 +0200 |
| commit | c61e5720f23273269cc67ffb2908cf9831c8ca9d (patch) | |
| tree | 8aa8bf946e53cf770416ddf1179bf3ff94c49721 | |
| parent | drivers/perf: Use devm_platform_get_and_ioremap_resource() (diff) | |
| download | linux-c61e5720f23273269cc67ffb2908cf9831c8ca9d.tar.xz linux-c61e5720f23273269cc67ffb2908cf9831c8ca9d.zip | |
perf/amlogic: Fix config1/config2 parsing issue
The 3th argument of for_each_set_bit is incorrect, fix them.
Fixes: 2016e2113d35 ("perf/amlogic: Add support for Amlogic meson G12 SoC DDR PMU driver")
Signed-off-by: Jiucheng Xu <jiucheng.xu@amlogic.com>
Link: https://lore.kernel.org/r/20230209115403.521868-1-jiucheng.xu@amlogic.com
Signed-off-by: Will Deacon <will@kernel.org>
| -rw-r--r-- | drivers/perf/amlogic/meson_ddr_pmu_core.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/perf/amlogic/meson_ddr_pmu_core.c b/drivers/perf/amlogic/meson_ddr_pmu_core.c index b84346dbac2c..0b24dee1ed3c 100644 --- a/drivers/perf/amlogic/meson_ddr_pmu_core.c +++ b/drivers/perf/amlogic/meson_ddr_pmu_core.c @@ -156,10 +156,14 @@ static int meson_ddr_perf_event_add(struct perf_event *event, int flags) u64 config2 = event->attr.config2; int i; - for_each_set_bit(i, (const unsigned long *)&config1, sizeof(config1)) + for_each_set_bit(i, + (const unsigned long *)&config1, + BITS_PER_TYPE(config1)) meson_ddr_set_axi_filter(event, i); - for_each_set_bit(i, (const unsigned long *)&config2, sizeof(config2)) + for_each_set_bit(i, + (const unsigned long *)&config2, + BITS_PER_TYPE(config2)) meson_ddr_set_axi_filter(event, i + 64); if (flags & PERF_EF_START) |