On Fri, 12 Apr 2019 at 04:27, Leo Yan <leo.yan@xxxxxxxxxx> wrote:
>
> Since before there have no platforms use static funnel in mainline
> kernel (though maybe some in-house SoC has used it but didn't upstream
> for mainline kernel yet so we don't be aware for it), when enable
> CoreSight DT binding for hikey960, we found the SoC uses the static
> funnel in the link path and but it's not supported in CoreSight funnel
> driver.
>
> When support static funnel it's found the DT binding compatible string
> doesn't follow the same fashion between replicator and funnel; so the
> first two patches are firstly to refine the replicator DT binding and
> correspondingly, change the replicator driver to support new DT binding
> compatible.
>
> Patch 0003 is to update DT documentation to support static funnel (we
> call it as non-configurable funnel in documentation) and unify the
> compatible string with the same fashion for replicator and funnel.
>
> Patch 0004 is to support the static funnel in the CoreSight funnel
> driver.
>
> The consolidated DT binding compatible strings are as below:
>
> Dynamic replicator:
> "arm,coresight-dynamic-replicator", "arm,primecell";
>
> Static replicator:
> "arm,coresight-static-replicator";
> "arm,coresight-replicator"; (obsolete)
>
> Static funnel:
> "arm,coresight-static-funnel";
>
> Dynamic funnel:
> "arm,coresight-dynamic-funnel", "arm,primecell";
> "arm,coresight-funnel", "arm,primecell"; (obsolete)
>
> This patch set has been rebased on CoreSight next branch [1] with
> latest commit d5d246a56af0 ("coresight: Merge the static and dynamic
> replicator drivers") and tested on Hikey960/Hikey with perf commands:
>
> Hikey960:
>
> # perf record -e cs_etm/@ec033000.etr/ --per-thread ./main
> # perf report --tui
>
> Hikey:
>
> # perf record -e cs_etm/@f6404000.etr/ --per-thread uname
> # perf report --tui
>
> [1] https://git.linaro.org/kernel/coresight.git/log/?h=next
>
> == Changes for v5 ==
> * Marked the the obsolete compatible strings to be more abvious
> (Suzuki).
> * Used pr_warn_once() to print out warning if DT binding uses
> the obsolete compatible string (Suzuki).
>
> == Changes for v4 ==
> * Consolidate the DT binding compatible strings for replicator and
> funnel (Mathieu).
> * Changed replicator driver to support new compatible string.
>
>
> Leo Yan (4):
> dt-bindings: arm: coresight: Add new compatible for static replicator
> coresight: replicator: Add new device id for static replicator
> dt-bindings: arm: coresight: Unify funnel DT binding
> coresight: funnel: Support static funnel
>
> .../devicetree/bindings/arm/coresight.txt | 60 +++++++--
> .../hwtracing/coresight/coresight-funnel.c | 116 ++++++++++++++----
> .../coresight/coresight-replicator.c | 6 +-
> 3 files changed, 147 insertions(+), 35 deletions(-)
I have applied this set.
Thanks,
Mathieu
>
> --
> 2.17.1
>
