Commit c61e5720 authored by Jiucheng Xu's avatar Jiucheng Xu Committed by Will Deacon
Browse files

perf/amlogic: Fix config1/config2 parsing issue 



The 3th argument of for_each_set_bit is incorrect, fix them.

Fixes: 2016e211 ("perf/amlogic: Add support for Amlogic meson G12 SoC DDR PMU driver")
Signed-off-by: default avatarJiucheng Xu <jiucheng.xu@amlogic.com>
Link: https://lore.kernel.org/r/20230209115403.521868-1-jiucheng.xu@amlogic.com


Signed-off-by: default avatarWill Deacon <will@kernel.org>
parent bc12f344

drivers/perf/amlogic/meson_ddr_pmu_core.c

+6 −2
Original line number Diff line number Diff line
@@ -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)