coresight: Fix TRCCONFIGR.QE sysfs interface
authorJames Clark <james.clark@arm.com>
Thu, 20 Jan 2022 11:30:47 +0000 (11:30 +0000)
committerSuzuki K Poulose <suzuki.poulose@arm.com>
Tue, 22 Feb 2022 15:57:00 +0000 (15:57 +0000)
commit91a2f2941df2c9e512aabb3be28ad97adafb7c0f
tree93bfd425456a84005c434cfb264014db0ec47d4d
parent7f4cd33759066c760cbdf303cb4ca0e9f610604d
coresight: Fix TRCCONFIGR.QE sysfs interface

It's impossible to program a valid value for TRCCONFIGR.QE
when TRCIDR0.QSUPP==0b10. In that case the following is true:

  Q element support is implemented, and only supports Q elements without
  instruction counts. TRCCONFIGR.QE can only take the values 0b00 or 0b11.

Currently the low bit of QSUPP is checked to see if the low bit of QE can
be written to, but as you can see when QSUPP==0b10 the low bit is cleared
making it impossible to ever write the only valid value of 0b11 to QE.
0b10 would be written instead, which is a reserved QE value even for all
values of QSUPP.

The fix is to allow writing the low bit of QE for any non zero value of
QSUPP.

This change also ensures that the low bit is always set, even when the
user attempts to only set the high bit.

Signed-off-by: James Clark <james.clark@arm.com>
Reviewed-by: Mike Leach <mike.leach@linaro.org>
Fixes: d8c66962084f ("coresight-etm4x: Controls pertaining to the reset, mode, pe and events")
Cc: stable@vger.kernel.org
Link: https://lore.kernel.org/r/20220120113047.2839622-2-james.clark@arm.com
Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
drivers/hwtracing/coresight/coresight-etm4x-sysfs.c