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 panicor watchdog reset.This reserved memory region is used as trace buffer or used fortrace
data retention only if specifically selected by the user in sysfsinterface.The default memory usage models for ETR in sysfs/perf modes areotherwise unaltered.For ETF, this reserved memory region is used by default forretention of trace data synced from internal SRAM after a panicor watchdog reset.- description: Reserved meta data memory. Used for ETR and ETF sinksfor 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.