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 > @@ -79,6 +79,34 @@ specific to ARM. > corresponding interface programming > registers. > > + - CCI PMU node > + > + Node name must be "pmu". > + Parent node must be CCI interconnect node. > + > + A CCI pmu node must contain the following properties: > + > + - compatible > + Usage: required > + Value type: <string> > + Definition: must be set to one of > + "arm,cci-400-pmu" > + "arm,cci-400-pmu,rev0" > + "arm,cci-400-pmu,rev1" Do you really mean only one? Seems like ""arm,cci-400-pmu,rev0", "arm,cci-400-pmu" would be valid. > + > + - reg: > + Usage: required > + Value type: <prop-encoded-array> > + Definition: the base address and size of the > + corresponding interface programming > + registers. > + > + - 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 > + > * CCI interconnect bus masters > > Description: masters in the device tree connected to a CCI port > @@ -163,6 +191,16 @@ Example: > interface-type = "ace"; > reg = <0x5000 0x1000>; > }; > + > + pmu@9000 { > + compatible = "arm,cci-400-pmu,rev0"; > + reg = <0x9000 0x5000>; > + interrupts = <0 101 4>, > + <0 102 4>, > + <0 103 4>, > + <0 104 4>, > + <0 105 4>; > + }; > }; > [ snip ] - k -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- 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