On 2024/7/18 23:57, Mike Leach wrote:
Hi,
I have detailed comments in the following patches but in summary we should:
- consistently use the term "static trace id" for these devices where
the hardware sets a non-programmable trace ID 2) Simplify the patch set by introducing a new API function int coresight_trace_id_get_system_static_id(int trace_id) This would avoid having to change drivers which use the existing function where no static ID is required.
Thanks for the comments, Mike. I will address your comments in next version.
Thanks Jinlong Mao
Regards
Mike
On Wed, 26 Jun 2024 at 07:07, Mao Jinlong quic_jinlmao@quicinc.com wrote:
Some HW has static trace id which cannot be changed via software programming. For this case, configure the trace id in device tree with "arm,trace-id = <xxx>", and call coresight_trace_id_get_system_id with the trace id value in device probe function. The id will be reserved for the HW all the time if the device is probed.
Changes since V2:
- Change "trace-id" to "arm,trace-id".
- Add trace id flag for getting preferred id or ODD id.
Changes since V1:
- Add argument to coresight_trace_id_get_system_id for preferred id
instead of adding new function coresight_trace_id_reserve_system_id. 2. Add constraint to trace-id in dt-binding file.
Mao Jinlong (3): dt-bindings: arm: Add arm,trace-id for coresight dummy source coresight: Add support to get preferred id for system trace sources coresight: dummy: Add reserve atid support for dummy source
.../sysfs-bus-coresight-devices-dummy-source | 15 +++++ .../arm/arm,coresight-dummy-source.yaml | 6 ++ drivers/hwtracing/coresight/coresight-dummy.c | 59 +++++++++++++++++-- .../hwtracing/coresight/coresight-platform.c | 25 ++++++++ drivers/hwtracing/coresight/coresight-stm.c | 2 +- drivers/hwtracing/coresight/coresight-tpda.c | 2 +- .../hwtracing/coresight/coresight-trace-id.c | 35 +++++++---- .../hwtracing/coresight/coresight-trace-id.h | 11 +++- include/linux/coresight.h | 1 + 9 files changed, 137 insertions(+), 19 deletions(-) create mode 100644 Documentation/ABI/testing/sysfs-bus-coresight-devices-dummy-source
-- 2.41.0
-- Mike Leach Principal Engineer, ARM Ltd. Manchester Design Centre. UK