On 30 April 2015 at 01:21, Ivan T. Ivanov <ivan.ivanov@xxxxxxxxxx> wrote: > > On Wed, 2015-04-29 at 10:28 -0600, Mathieu Poirier wrote: >> On 29 April 2015 at 06:19, Ivan T. Ivanov ivanov@xxxxxxxxxx> wrote: > > <snip> > >> > - "arm,coresight-etm4x", "arm,primecell"; >> > + - "qcom,coresight-replicator", "arm,primecell"; >> >> Is there some sort of versioning information we can add like it was >> done for the "coresight-etmXY" bindings? It makes things a lot >> cleaner when a new (and possibly not backward compatible) version gets >> released. >> > > From what I can see, it is version 1. I could add it to compatible > string, but suldn't we use amba_id for this reason. amba_id work well for devices within the same family and compatible up to a certain point. When revisions change backward compatibility is usually not kept and it becomes a nightmare to maintain. > >> > * reg: physical base address and length of the register >> > set(s) of the component. >> > diff --git a/drivers/hwtracing/coresight/Kconfig b/drivers/hwtracing/coresight/Kconfig >> > index 6b331d4..165b681 100644 >> > --- a/drivers/hwtracing/coresight/Kconfig >> > +++ b/drivers/hwtracing/coresight/Kconfig >> > @@ -68,4 +68,13 @@ config CORESIGHT_SOURCE_ETM4X >> > instructions that a processor is executing. This is primarily useful >> > for instruction level tracing. Depending on the implemented version >> > data tracing may also be available. >> > + >> > +config CORESIGHT_QCOM_REPLICATOR >> > + bool "Qualcomm CoreSight Replicator driver" >> > + help >> > + This enables support for CoreSight link and sink driver that are >> > + responsible for transporting and collecting the trace data >> > + respectively. Link and sinks are dynamically aggregated with a trace >> > + entity at run time to form a complete trace path. >> >> The replicator is only a link entity. It is only transporting trace >> data information rather than collecting it. Please review the >> comment. > > True, sorry. copy and paste from CORESIGHT_LINKS_AND_SINKS. > >> Also, can this specific version run on both V7 and V8 >> architecture. If not the proper "depends" should be added. >> > > Same driver/device on both architectures. > > Will fix rest of the comments and will resend. > > Thank you, > Ivan -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
