The System Trace Macrocell (STM) is an IP block falling under the
CoreSight umbrella. It's main purpose it so expose stimulus channels
to any system component for the purpose of information logging.
Bindings for this IP block adds a couple of items to the current
mandatory definition for CoreSight components. The driver has been posted
here[1].
[1]. https://lkml.org/lkml/2015/2/25/743
Signed-off-by: Mathieu Poirier <[email protected]>
---
.../devicetree/bindings/arm/coresight.txt | 25 ++++++++++++++++++++++
1 file changed, 25 insertions(+)
diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt
index a3089359aaa6..854127578718 100644
--- a/Documentation/devicetree/bindings/arm/coresight.txt
+++ b/Documentation/devicetree/bindings/arm/coresight.txt
@@ -17,6 +17,7 @@ its hardware characteristcs.
- "arm,coresight-tmc", "arm,primecell";
- "arm,coresight-funnel", "arm,primecell";
- "arm,coresight-etm3x", "arm,primecell";
+ - "arm,coresight-stm", "arm,primecell";
* reg: physical base address and length of the register
set(s) of the component.
@@ -31,6 +32,14 @@ its hardware characteristcs.
layout using the generic DT graph presentation found in
"bindings/graph.txt".
+* Additional required properly for System Trace Macrocells (STM):
+ * reg: along with the physical base address and length of the register
+ set as described above, another entry is required to describe the
+ mapping of the extended stimulus port area.
+
+ * reg-names: the only acceptable values are "stm-base" and
+ "stm-channel-base", each corresponding to the areas defined in "reg".
+
* Required properties for devices that don't show up on the AMBA bus, such as
non-configurable replicators:
@@ -198,3 +207,19 @@ Example:
};
};
};
+
+4. STM
+ stm@20100000 {
+ compatible = "arm,coresight-stm", "arm,primecell";
+ reg = <0 0x20100000 0 0x1000>,
+ <0 0x28000000 0 0x180000>;
+ reg-names = "stm-base", "stm-channel-base";
+
+ clocks = <&soc_smc50mhz>;
+ clock-names = "apb_pclk";
+ port {
+ stm_out_port: endpoint {
+ remote-endpoint = <&main_funnel_in_port2>;
+ };
+ };
+ };
--
1.9.1
On Thu, Feb 26, 2015 at 10:14 AM, Mathieu Poirier
<[email protected]> wrote:
> The System Trace Macrocell (STM) is an IP block falling under the
> CoreSight umbrella. It's main purpose it so expose stimulus channels
> to any system component for the purpose of information logging.
>
> Bindings for this IP block adds a couple of items to the current
> mandatory definition for CoreSight components. The driver has been posted
> here[1].
>
> [1]. https://lkml.org/lkml/2015/2/25/743
>
> Signed-off-by: Mathieu Poirier <[email protected]>
Seems reasonable, but a couple minor comments below. Merge this with the driver:
Acked-by: Rob Herring <[email protected]>
> ---
> .../devicetree/bindings/arm/coresight.txt | 25 ++++++++++++++++++++++
> 1 file changed, 25 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt
> index a3089359aaa6..854127578718 100644
> --- a/Documentation/devicetree/bindings/arm/coresight.txt
> +++ b/Documentation/devicetree/bindings/arm/coresight.txt
> @@ -17,6 +17,7 @@ its hardware characteristcs.
> - "arm,coresight-tmc", "arm,primecell";
> - "arm,coresight-funnel", "arm,primecell";
> - "arm,coresight-etm3x", "arm,primecell";
> + - "arm,coresight-stm", "arm,primecell";
No versions of STM?
> * reg: physical base address and length of the register
> set(s) of the component.
> @@ -31,6 +32,14 @@ its hardware characteristcs.
> layout using the generic DT graph presentation found in
> "bindings/graph.txt".
>
> +* Additional required properly for System Trace Macrocells (STM):
> + * reg: along with the physical base address and length of the register
> + set as described above, another entry is required to describe the
> + mapping of the extended stimulus port area.
> +
> + * reg-names: the only acceptable values are "stm-base" and
> + "stm-channel-base", each corresponding to the areas defined in "reg".
It is not really clear that "extended stimulus port area" corresponds
to "stm-channel-base".
> +
> * Required properties for devices that don't show up on the AMBA bus, such as
> non-configurable replicators:
>
> @@ -198,3 +207,19 @@ Example:
> };
> };
> };
> +
> +4. STM
> + stm@20100000 {
> + compatible = "arm,coresight-stm", "arm,primecell";
> + reg = <0 0x20100000 0 0x1000>,
> + <0 0x28000000 0 0x180000>;
> + reg-names = "stm-base", "stm-channel-base";
> +
> + clocks = <&soc_smc50mhz>;
> + clock-names = "apb_pclk";
> + port {
> + stm_out_port: endpoint {
> + remote-endpoint = <&main_funnel_in_port2>;
> + };
> + };
> + };
> --
> 1.9.1
>
[...]
> +* Additional required properly for System Trace Macrocells (STM):
s/properly/properties/
Mark.