perf cs-etm: Fix bitmap for option
authorSuzuki K Poulose <suzuki.poulose@arm.com>
Wed, 24 Feb 2021 16:48:32 +0000 (09:48 -0700)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 2 Mar 2021 12:47:41 +0000 (09:47 -0300)
commit8c559e8d68630d64d932bada633705f6551427df
tree8669a66e8f9b703343dd65b08a39bb5766f75d01
parent2bb4ccbd95d7fbf58540c8d3d55cbabc8fb95e28
perf cs-etm: Fix bitmap for option

When set option with macros ETM_OPT_CTXTID and ETM_OPT_TS, it wrongly
takes these two values (14 and 28 prespectively) as bit masks, but
actually both are the offset for bits.  But this doesn't lead to further
failure due to the AND logic operation will be always true for
ETM_OPT_CTXTID / ETM_OPT_TS.

This patch uses the BIT() macro for option bits, thus it can request the
correct bitmaps for "contextid" and "timestamp" when calling
cs_etm_set_option().

Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com>
Reviewed-by: Mathieu Poirier <mathieu.poirier@linaro.org>
Reviewed-by: Mike Leach <mike.leach@linaro.org>
Link: https://lore.kernel.org/r/20210213113220.292229-3-leo.yan@linaro.org
Link: https://lore.kernel.org/r/20210224164835.3497311-4-mathieu.poirier@linaro.org
[Extract the change as a separate patch for easier review]
Signed-off-by: Leo Yan <leo.yan@linaro.org>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/arch/arm/util/cs-etm.c