* [PATCH 19/19] dts: bindings: coresight: ETMv4.4 system register access only units [not found] <20200911084119.1080694-1-suzuki.poulose@arm.com> @ 2020-09-11 8:41 ` Suzuki K Poulose 2020-09-18 15:35 ` Mike Leach 0 siblings, 1 reply; 4+ messages in thread From: Suzuki K Poulose @ 2020-09-11 8:41 UTC (permalink / raw) To: linux-arm-kernel Cc: coresight, mathieu.poirier, mike.leach, leo.yan, Anshuman.Khandual, Suzuki K Poulose, devicetree, Rob Herring Document the bindings for ETMv4.4 and later with only system register access. Cc: devicetree@vger.kernel.org Cc: Mathieu Poirier <mathieu.poirier@linaro.org> Cc: Mike Leach <mike.leach@linaro.org> Reviewed-by: Rob Herring <robh@kernel.org> Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com> --- Documentation/devicetree/bindings/arm/coresight.txt | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt index d711676b4a51..cfe47bdda728 100644 --- a/Documentation/devicetree/bindings/arm/coresight.txt +++ b/Documentation/devicetree/bindings/arm/coresight.txt @@ -34,9 +34,13 @@ its hardware characteristcs. Program Flow Trace Macrocell: "arm,coresight-etm3x", "arm,primecell"; - - Embedded Trace Macrocell (version 4.x): + - Embedded Trace Macrocell (version 4.x), with memory mapped access. "arm,coresight-etm4x", "arm,primecell"; + - Embedded Trace Macrocell (version 4.4 and later) with system + register access only. + "arm,coresight-etm-v4.4"; + - Coresight programmable Replicator : "arm,coresight-dynamic-replicator", "arm,primecell"; -- 2.24.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 19/19] dts: bindings: coresight: ETMv4.4 system register access only units 2020-09-11 8:41 ` [PATCH 19/19] dts: bindings: coresight: ETMv4.4 system register access only units Suzuki K Poulose @ 2020-09-18 15:35 ` Mike Leach 2020-09-24 9:48 ` Suzuki K Poulose 0 siblings, 1 reply; 4+ messages in thread From: Mike Leach @ 2020-09-18 15:35 UTC (permalink / raw) To: Suzuki K Poulose Cc: linux-arm-kernel, Coresight ML, Mathieu Poirier, Leo Yan, Anshuman.Khandual, devicetree, Rob Herring On Fri, 11 Sep 2020 at 09:41, Suzuki K Poulose <suzuki.poulose@arm.com> wrote: > > Document the bindings for ETMv4.4 and later with only system register > access. > > Cc: devicetree@vger.kernel.org > Cc: Mathieu Poirier <mathieu.poirier@linaro.org> > Cc: Mike Leach <mike.leach@linaro.org> > Reviewed-by: Rob Herring <robh@kernel.org> > Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com> > --- > Documentation/devicetree/bindings/arm/coresight.txt | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt > index d711676b4a51..cfe47bdda728 100644 > --- a/Documentation/devicetree/bindings/arm/coresight.txt > +++ b/Documentation/devicetree/bindings/arm/coresight.txt > @@ -34,9 +34,13 @@ its hardware characteristcs. > Program Flow Trace Macrocell: > "arm,coresight-etm3x", "arm,primecell"; > > - - Embedded Trace Macrocell (version 4.x): > + - Embedded Trace Macrocell (version 4.x), with memory mapped access. > "arm,coresight-etm4x", "arm,primecell"; > > + - Embedded Trace Macrocell (version 4.4 and later) with system > + register access only. > + "arm,coresight-etm-v4.4"; Any version of ETM can implement register access - including those pre ETM 4.4. Perhaps the new name should simply reflect sys reg access rather than a version. Given that the two compatibility strings should be mutually exclusive for a given device, should the bindings doc (or at least the etm4x component part) be re-written into the .yaml format so that this can be enforced? Regards Mike > + > - Coresight programmable Replicator : > "arm,coresight-dynamic-replicator", "arm,primecell"; > > -- > 2.24.1 > -- Mike Leach Principal Engineer, ARM Ltd. Manchester Design Centre. UK ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 19/19] dts: bindings: coresight: ETMv4.4 system register access only units 2020-09-18 15:35 ` Mike Leach @ 2020-09-24 9:48 ` Suzuki K Poulose 2020-09-24 10:08 ` Mike Leach 0 siblings, 1 reply; 4+ messages in thread From: Suzuki K Poulose @ 2020-09-24 9:48 UTC (permalink / raw) To: mike.leach Cc: linux-arm-kernel, coresight, mathieu.poirier, leo.yan, anshuman.khandual, devicetree, robh On 09/18/2020 04:35 PM, Mike Leach wrote: > On Fri, 11 Sep 2020 at 09:41, Suzuki K Poulose <suzuki.poulose@arm.com> wrote: >> >> Document the bindings for ETMv4.4 and later with only system register >> access. >> >> Cc: devicetree@vger.kernel.org >> Cc: Mathieu Poirier <mathieu.poirier@linaro.org> >> Cc: Mike Leach <mike.leach@linaro.org> >> Reviewed-by: Rob Herring <robh@kernel.org> >> Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com> >> --- >> Documentation/devicetree/bindings/arm/coresight.txt | 6 +++++- >> 1 file changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt >> index d711676b4a51..cfe47bdda728 100644 >> --- a/Documentation/devicetree/bindings/arm/coresight.txt >> +++ b/Documentation/devicetree/bindings/arm/coresight.txt >> @@ -34,9 +34,13 @@ its hardware characteristcs. >> Program Flow Trace Macrocell: >> "arm,coresight-etm3x", "arm,primecell"; >> >> - - Embedded Trace Macrocell (version 4.x): >> + - Embedded Trace Macrocell (version 4.x), with memory mapped access. >> "arm,coresight-etm4x", "arm,primecell"; >> >> + - Embedded Trace Macrocell (version 4.4 and later) with system >> + register access only. >> + "arm,coresight-etm-v4.4"; > > Any version of ETM can implement register access - including those pre > ETM 4.4. Perhaps the new name should simply reflect sys reg access > rather than a version. > You're right. I got it confused with the v8.4 SelfHosted Extensions, which mandates the sysreg access and makes the mem I/O obsolete. How about : "arm,coresight-etm4x-sysreg" ? > Given that the two compatibility strings should be mutually exclusive > for a given device, should the bindings doc (or at least the etm4x > component part) be re-written into the .yaml format so that this can > be enforced? I will take a look, haven't played with the yaml. Thanks for the review ! Suzuki ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 19/19] dts: bindings: coresight: ETMv4.4 system register access only units 2020-09-24 9:48 ` Suzuki K Poulose @ 2020-09-24 10:08 ` Mike Leach 0 siblings, 0 replies; 4+ messages in thread From: Mike Leach @ 2020-09-24 10:08 UTC (permalink / raw) To: Suzuki K Poulose Cc: linux-arm-kernel, Coresight ML, Mathieu Poirier, Leo Yan, anshuman.khandual, devicetree, Rob Herring Hi suzuki, On Thu, 24 Sep 2020 at 10:43, Suzuki K Poulose <suzuki.poulose@arm.com> wrote: > > On 09/18/2020 04:35 PM, Mike Leach wrote: > > On Fri, 11 Sep 2020 at 09:41, Suzuki K Poulose <suzuki.poulose@arm.com> wrote: > >> > >> Document the bindings for ETMv4.4 and later with only system register > >> access. > >> > >> Cc: devicetree@vger.kernel.org > >> Cc: Mathieu Poirier <mathieu.poirier@linaro.org> > >> Cc: Mike Leach <mike.leach@linaro.org> > >> Reviewed-by: Rob Herring <robh@kernel.org> > >> Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com> > >> --- > >> Documentation/devicetree/bindings/arm/coresight.txt | 6 +++++- > >> 1 file changed, 5 insertions(+), 1 deletion(-) > >> > >> diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt > >> index d711676b4a51..cfe47bdda728 100644 > >> --- a/Documentation/devicetree/bindings/arm/coresight.txt > >> +++ b/Documentation/devicetree/bindings/arm/coresight.txt > >> @@ -34,9 +34,13 @@ its hardware characteristcs. > >> Program Flow Trace Macrocell: > >> "arm,coresight-etm3x", "arm,primecell"; > >> > >> - - Embedded Trace Macrocell (version 4.x): > >> + - Embedded Trace Macrocell (version 4.x), with memory mapped access. > >> "arm,coresight-etm4x", "arm,primecell"; > >> > >> + - Embedded Trace Macrocell (version 4.4 and later) with system > >> + register access only. > >> + "arm,coresight-etm-v4.4"; > > > > Any version of ETM can implement register access - including those pre > > ETM 4.4. Perhaps the new name should simply reflect sys reg access > > rather than a version. > > > > You're right. I got it confused with the v8.4 SelfHosted Extensions, which > mandates the sysreg access and makes the mem I/O obsolete. How about : > > "arm,coresight-etm4x-sysreg" ? > > Seems reasonable. Perhaps ensure that the accompanying comment mentions that this is aarch64 access (to cover the unlikely event that some outlier implementation does come along with v8 aarch32 + ETMv4 + sysreg access!) > > Given that the two compatibility strings should be mutually exclusive > > for a given device, should the bindings doc (or at least the etm4x > > component part) be re-written into the .yaml format so that this can > > be enforced? > > I will take a look, haven't played with the yaml. > I used it to describe the CTI bindings as these were brand new. Reasonably straight forwards - there are plenty of examples and the checking tools are pretty good. Regards Mike > Thanks for the review ! > > Suzuki -- Mike Leach Principal Engineer, ARM Ltd. Manchester Design Centre. UK ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-09-24 10:08 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20200911084119.1080694-1-suzuki.poulose@arm.com>
2020-09-11 8:41 ` [PATCH 19/19] dts: bindings: coresight: ETMv4.4 system register access only units Suzuki K Poulose
2020-09-18 15:35 ` Mike Leach
2020-09-24 9:48 ` Suzuki K Poulose
2020-09-24 10:08 ` Mike Leach
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox