Re: [PATCH 1/2] coresight: replicator: Add Qualcomm CoreSight Replicator driver

2015-05-07 Thread Paul Bolle
On Thu, 2015-05-07 at 17:46 +0300, Ivan T. Ivanov wrote: > Thank you. I am going to fallow approach taken by Mathieu in [1]. > Is this OK with you? I'm OK with anything that is coherent (ie, anything better than built-in only code cross-dressing as modular code). I think Mathieu's approach clears

Re: [PATCH 1/2] coresight: replicator: Add Qualcomm CoreSight Replicator driver

2015-05-07 Thread Ivan T. Ivanov
On Thu, 2015-04-30 at 22:53 +0200, Paul Bolle wrote: > On Wed, 2015-04-29 at 15:19 +0300, Ivan T. Ivanov wrote: > > > +config CORESIGHT_QCOM_REPLICATOR > > + bool "Qualcomm CoreSight Replicator driver" > > + help > > + This enables support for CoreSight link and

Re: [PATCH 1/2] coresight: replicator: Add Qualcomm CoreSight Replicator driver

2015-04-30 Thread Paul Bolle
On Wed, 2015-04-29 at 15:19 +0300, Ivan T. Ivanov wrote: > +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 d

Re: [PATCH 1/2] coresight: replicator: Add Qualcomm CoreSight Replicator driver

2015-04-30 Thread Mathieu Poirier
On 30 April 2015 at 01:21, Ivan T. Ivanov wrote: > > On Wed, 2015-04-29 at 10:28 -0600, Mathieu Poirier wrote: >> On 29 April 2015 at 06:19, Ivan T. Ivanov iva...@linaro.org> wrote: > > > >> > - "arm,coresight-etm4x", "arm,primecell"; >> > + - "qcom,coresight-replica

Re: [PATCH 1/2] coresight: replicator: Add Qualcomm CoreSight Replicator driver

2015-04-30 Thread Ivan T. Ivanov
On Wed, 2015-04-29 at 10:28 -0600, Mathieu Poirier wrote: > On 29 April 2015 at 06:19, Ivan T. Ivanov iva...@linaro.org> wrote: > > - "arm,coresight-etm4x", "arm,primecell"; > > + - "qcom,coresight-replicator", "arm,primecell"; > > Is there some sort of versioning

Re: [PATCH 1/2] coresight: replicator: Add Qualcomm CoreSight Replicator driver

2015-04-29 Thread Mathieu Poirier
On 29 April 2015 at 06:19, Ivan T. Ivanov wrote: > From: Pratik Patel Thanks for crediting the original author. > > This driver manages Qualcomm CoreSight Replicator device, which > resides on the AMBA bus. Replicator has been made programmable to > allow software to turn of the replicator bran