perf/amlogic: Fix config1/config2 parsing issue
authorJiucheng Xu <jiucheng.xu@amlogic.com>
Thu, 9 Feb 2023 11:54:01 +0000 (19:54 +0800)
committerWill Deacon <will@kernel.org>
Mon, 27 Mar 2023 14:12:05 +0000 (15:12 +0100)
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>
drivers/perf/amlogic/meson_ddr_pmu_core.c

index b84346d..0b24dee 100644 (file)
@@ -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)