Hi Rob,
On 14/08/13 23:38, Rob Herring wrote:
On 08/14/2013 04:03 PM, Kumar Gala wrote:
On Jul 23, 2013, at 4:19 AM, Punit Agrawal wrote:
The CCI PMU can profile bus transactions at the master and slave
interfaces of the CCI. The PMU can be used to observe an aggregated view
of the bus traffic between the various components connected to the CCI.
Extend the existing CCI driver to support the PMU by registering a perf
backend for it.
Document the device tree binding to describe the CCI PMU.
Cc: Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>
Cc: Nicolas Pitre <nico@xxxxxxxxxx>
Cc: Dave Martin <dave.martin@xxxxxxxxxx>
Cc: Rob Herring <rob.herring@xxxxxxxxxxx>
Cc: Will Deacon <will.deacon@xxxxxxx>
Signed-off-by: Punit Agrawal <punit.agrawal@xxxxxxx>
Reviewed-by: Will Deacon <will.deacon@xxxxxxx>
---
Documentation/devicetree/bindings/arm/cci.txt | 38 ++
drivers/bus/arm-cci.c | 642 +++++++++++++++++++++++++
2 files changed, 680 insertions(+)
diff --git a/Documentation/devicetree/bindings/arm/cci.txt b/Documentation/devicetree/bindings/arm/cci.txt
index 92d36e2..5bc95e5 100644
--- a/Documentation/devicetree/bindings/arm/cci.txt
+++ b/Documentation/devicetree/bindings/arm/cci.txt
[snip]
+
+ - interrupts:
+ Usage: required
+ Value type: <prop-encoded-array>
+ Definition: comma-separated list of unique PMU
+ interrupts
What is the list of interrupts related to, should there be an associated interrupts-names
No, interrupt-names is optional, but you are correct that what function
each interrupt is for must be defined.
I'll update the bindings documentation to describe the function of
interrupts.
Thanks for the comments.
Punit
Rob
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
