On 2021-05-10 21:10, Suzuki K Poulose wrote:
Hi Tao
On 10/05/2021 12:05, Tao Zhang wrote:
We are trying to achieve more types of Coresight source devices. For example, we have a type of coresight source devic named TPDM. In the process of using, sometimes mulitiple TPDMs need to be connected to the different input ports on the same funnel. Meanwhile, these TPDMs also need to output from different ports on the funnel. But, at present the Coresight driver assumes a) Only support Coresight source type ETM, ETR and ETF
Did you mean ETM and STM here ? ETR & ETF are not source types, rather they are SINK.
Yes, I mean ETM and STM here.
b) Funnels only support mulitiple inputs and one output Which doesn't help to add the above feature for our new Coresight source device TPDM. So, in order to accommodate the new device, we develop the following patches.
Where is the TPDM driver ? Could you give us a rough idea of the behavior in terms of the input / output ?
We have plans to upload the TPDM driver in the feature. TPDM is a type of hardware component for debugging and monitoring on Qualcomm targets. The primary use case of the TPDM is to collect data from different data source and send it to a TPDA for packetization, timestamping and funneling. And the output of TPDA is a regular AMBA ATB stream and can be thought of as any other trace source in the system. You can get a general understanding of the TPDM and TPDA driver through the following patch. https://source.codeaurora.org/quic/la/kernel/msm-5.4/commit/?h=LV.AU.0.2.0.r...
a) Add support more types of Coresight source devices.
Which ones ? where is the code ?
In the patch "0001-coresight-add-support-to-enable-more-coresight-paths.patch", we replaced the original path save method with the funcation "coresight_store_path". In the original method, "coresight_enable" only can store the tracer path for ETM and STM. In the function "coresight_store_path", it can store the tracer path for more types of Coresight sources.
b) Add support for multiple output ports on funnel and the output ports could be selected by Corsight source.
Does the "TPDM" require programming to switch these output or are these "static" ?
Is this something that can be avoided by having a "fake" static-replicator in the path ?
e.g, TPDM ________________________________________________ In0 | | -> Out0 In1 | Static-Funnel -> Static-Replicator | -> Out1 In2 | | -> Out2 ________________________________________________
Is this something that can be solved ? Again, please give a brief description of the TPDM device and the driver code in the series to give us a complete picture of what you are trying to do.
Reviewing some changes without having the full picture is not going to be helpful.
Suzuki
Now the link can support multiple out ports, but there are no entries from the link's device tree can figure out from which input port to which output port. This patch provide the entry "source" in device tree, which can associate the input port of the link with the output port. e.g, if we want to achieve the following link connection. ------------------------------------------------------------------------- Source0 -> In0 | | Out0 -> Source1 -> In1 |Funnel 0| Out1 -> Source2 -> In2 | | Out2 -> ------------------------------------------------------------------------- We can configure the "source" entries as the following device tree. Funnel 0 { out-ports { port@0 { reg = <0> Out0: endpoint { remote-endpoint = <... ...>; source = <Source0> }
} port@1 { reg = <1> Out1: endpoint { remote-endpoint = <... ...>; source = <Source1> }
} port@2 { reg = <0> Out2: endpoint { remote-endpoint = <... ...>; source = <Source2> }
} }
in-ports { port@0 { reg = <0> In0: endpoint { remote-endpoint = <... ...>; }
} port@1 { reg = <1> Out1: endpoint { remote-endpoint = <... ...>; }
} port@2 { reg = <0> Out2: endpoint { remote-endpoint = <... ...>; }
} } }
Best, Tao