[PATCH v2] drivers: CCI: add ARM CCI PMU support

Will Deacon will.deacon at arm.com
Tue Aug 20 11:07:32 EDT 2013


On Fri, Aug 16, 2013 at 06:19:59PM +0100, 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 at arm.com>
> Cc: Nicolas Pitre <nico at linaro.org>
> Cc: Dave Martin <dave.martin at linaro.org>
> Cc: Rob Herring <rob.herring at calxeda.com>
> Cc: Kumar Gala <galak at codeaurora.org>
> Cc: Stephen Warren <swarren at wwwdotorg.org>
> Cc: Will Deacon <will.deacon at arm.com>
> Signed-off-by: Punit Agrawal <punit.agrawal at arm.com>
> ---
> 
> Hi,
> 
> This is the second version of the CCI PMU driver. The changes incorporate review
> comments on device tree bindings and corresponding update to the driver.
> 
> Cheers,
> Punit
> 
> Will, I've dropped your reviewed-by tag due to the changes. Let me know if you are
> ok with the changes and I'll add the tag.

I'm not the right person to review the DT binding, but the actual driver
code is fine. You can either:

  1. Add my reviewed-by once you have an Ack from a DT maintainer

or

  2. Split the patch in two (binding + driver) and add my reviewed-by to the
     driver part

Either way, once you have the DT part sorted out, please send a fresh
version with the relevant tags and stick me on CC.

Cheers,

Will



More information about the linux-arm-kernel mailing list