From mboxrd@z Thu Jan 1 00:00:00 1970 From: mark.rutland@arm.com (Mark Rutland) Date: Tue, 24 Feb 2015 16:19:31 +0000 Subject: question about PMU interrupt in binding In-Reply-To: References: Message-ID: <20150224161931.GT9714@leverpostej> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Feb 24, 2015 at 03:44:27PM +0000, Stuart Yoder wrote: > Stephen/Will, > > Had a question about the definition of "interrupts" in the > PMU binding: > > > commit d84c47837d800c0db1c9f2015a1565504417f812 > > Author: Stephen Boyd > > Date: Fri Feb 7 21:01:24 2014 +0000 > > > > devicetree: bindings: Document Krait performance monitor units (PMU) > > > > Document the Krait PMU compatible string. > > > > Cc: > > Signed-off-by: Stephen Boyd > > Signed-off-by: Will Deacon > > > > diff --git a/Documentation/devicetree/bindings/arm/pmu.txt b/Documentation/devicetree/bindings/arm/pmu.txt > > index 3e1e498..ce73144 100644 > > --- a/Documentation/devicetree/bindings/arm/pmu.txt > > +++ b/Documentation/devicetree/bindings/arm/pmu.txt > > @@ -16,7 +16,14 @@ Required properties: > > "arm,arm11mpcore-pmu" > > "arm,arm1176-pmu" > > "arm,arm1136-pmu" > > -- interrupts : 1 combined interrupt or 1 per core. > > + "qcom,krait-pmu" > > +- interrupts : 1 combined interrupt or 1 per core. If the interrupt is a per-cpu > > + interrupt (PPI) then 1 interrupt should be specified. > > If the interrupt is per-cpu do you mean 1 interrupt _per cpu_ should > be specified? As written it seems to say _only_ "1 interrupt" should be > specificed. The specification as written is correct. If it's a per-cpu interrupt (i.e. a PPI), we'll recognise this fact and handle it appropriately in the backend, on the assumption that if the PMU is connected via a PPI all CPUs share this PPI. Thanks, Mark.