RE: [EXT] Re: [PATCH v6 1/7] dt-bindings: arm: coresight-tmc: Add "memory-region" property - CoreSight

12 Jan 2024

      Hi Krzysztof,
...
-----Original Message-----
From: Krzysztof Kozlowski krzysztof.kozlowski@linaro.org
Sent: Friday, January 5, 2024 3:24 PM
To: Linu Cherian lcherian@marvell.com; suzuki.poulose@arm.com;
mike.leach@linaro.org; james.clark@arm.com; leo.yan@linaro.org
Cc: linux-arm-kernel@lists.infradead.org; coresight@lists.linaro.org; linux-
kernel@vger.kernel.org; robh+dt@kernel.org;
krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org;
devicetree@vger.kernel.org; Sunil Kovvuri Goutham
sgoutham@marvell.com; George Cherian gcherian@marvell.com; Rob
Herring robh@kernel.org
Subject: [EXT] Re: [PATCH v6 1/7] dt-bindings: arm: coresight-tmc: Add
"memory-region" property
External Email

On 05/01/2024 06:58, Linu Cherian wrote:
...
memory-region 0: Reserved trace buffer memory
TMC ETR: When available, use this reserved memory region for
  trace data capture. Same region is used for trace data
  retention after a panic or watchdog reset.
TMC ETF: When available, use this reserved memory region for
  trace data retention synced from internal SRAM after a panic or
  watchdog reset.
memory-region 1: Reserved meta data memory
TMC ETR, ETF: When available, use this memory for register
  snapshot retention synced from hardware registers after a panic
  or watchdog reset.
Reviewed-by: Rob Herring robh@kernel.org
...
...

memory-region:
items:
 - description: Reserved trace buffer memory for ETR and ETF sinks.

     For ETR, this reserved memory region is used for trace data capture.

     Same region is used for trace data retention as well after a panic

     or watchdog reset.

     This reserved memory region is used as trace buffer or used for

trace
...

     data retention only if specifically selected by the user in sysfs

     interface.

     The default memory usage models for ETR in sysfs/perf modes are

     otherwise unaltered.

     For ETF, this reserved memory region is used by default for

     retention of trace data synced from internal SRAM after a panic

     or watchdog reset.

 - description: Reserved meta data memory. Used for ETR and ETF sinks

     for storing metadata.

If there is going to be new version/resend, add a blank line here. Keep the Rb
tag.
Okay. Will take care in next version.
Linu Cherian.