Thread (1 message) 1 message, 1 author, 2017-02-21

[PATCH] arm64: dts: juno: update definition for programmable replicator.

From: Sudeep Holla <hidden>
Date: 2017-02-21 15:54:11
Also in: linux-devicetree


On 21/02/17 15:47, Mathieu Poirier wrote:
On 21 February 2017 at 04:02, Sudeep Holla [off-list ref] wrote:
quoted
Hi Mike,

On 17/02/17 19:13, Mike Leach wrote:
quoted
Juno platforms have a programmable replicator splitting the trace output to
TPIU and ETR. Currently this is not being programmed as it is being treated
as a none-programmable replicator - which is the default operational mode
for these devices. The TPIU in the system is enabled by default, and this
combination is causing back-pressure in the trace system resulting in
overflows at the source.

Replaces the existing definition with one that defines the programmable
replicator, using the "qcom,coresight-replicator1x" driver that provides
the correct functionality for CoreSight programmable replicators.
I assume this is just enhancement and not a fix. Since it's too late for
v4.11 (already in merge window now), I will queue this for v4.12
quoted
Signed-off-by: Mike Leach <redacted>
---
 arch/arm64/boot/dts/arm/juno-base.dtsi | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/arch/arm64/boot/dts/arm/juno-base.dtsi b/arch/arm64/boot/dts/arm/juno-base.dtsi
index 9d799d9..6546e23 100644
--- a/arch/arm64/boot/dts/arm/juno-base.dtsi
+++ b/arch/arm64/boot/dts/arm/juno-base.dtsi
@@ -372,12 +372,14 @@
              };
      };

-     coresight-replicator {
-             /*
-              * Non-configurable replicators don't show up on the
-              * AMBA bus.  As such no need to add "arm,primecell".
-              */
-             compatible = "arm,coresight-replicator";
+     coresight-replicator at 20120000 {
+
As stated in a previous post please remove the extra line and the
"coresight-".  Since programmable replicators show up on the AMBA bus
there is no need to differentiate them from other CoreSight devices.
Already done in my local branch ;) along with you tags.

Also I away when merge window closes, so I will be able to send only at
-rc2. ARM SoC team generally prefer to rebase fixes on top of -rc1.

-- 
Regards,
Sudeep
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help