On Thu, Apr 04, 2019 at 09:47:53AM +0100, Suzuki K Poulose wrote:

Hi

On 04/04/2019 05:47, Mathieu Poirier wrote:

...

Good day to all,

Suzuki and Leo have recently posted patches to unify the way funnel and replicators are supported. Those patches prompted me to have a closer look at the bindings we currently use. At this time we have:

Non configurable replicator: "coresight-replicator" Configurable replicator: "coresight-dynamic-replicator"

Non Configurable funnel: "coresight-static-funnel" (public but not merged yet) Configurable funnel: "coresight-funnel"

This is quite confusing to me and I suggest the following:

Non configurable replicator: "coresight-static-replicator" Configurable replicator: "coresight-dynamic-replicator"

Non Configurable funnel: "coresight-static-funnel" Configurable funnel: "coresight-dynamic-funnel"

Current bindings would be kept around for backward compatibility but marked as obsolete in the coresight DT bindings document. Leo has stepped forward to do the work and Rob Herring (DT maintainer) is on board with the idea. All we need is to settle on a naming convention, hence this email.

I thought about it when I posted the original suggestion. But didn't want to disturb the existing compatibles. But given that changing the "coresight-funnel" doesn't really matter, as long as "primecell" appears in the list of compatibles. And it is good to have it unified across all components. So I am happy with the proposal.

Cool! Thanks for confirmation, Suzuki.

This looks good to me as well, will send patch set soon.

Thanks, Leo Yan