Re: [PATCH v2 1/7] ARM: davinci, intc: Add OF support for TI interrupt controller
From: Sergei Shtylyov <hidden>
Date: 2012-02-22 12:23:00
Also in:
linux-arm-kernel
Hello. On 22-02-2012 10:50, Heiko Schocher wrote:
Add a function to initialize the davinci interrupt controller (INTC) using a device tree node.
cp_intc is not DaVinci interrupt controller, it's only used in OMAP-L1x
SoCs. cp_intc stands for Common Platform Interrupt Controller. I don't see
where you are initializiing the DaVinci interrupt controller itself. You
should understand that there are two disctinct types of interrupt controllers
used in the SoCs supported by arch/arm/mach-davinci/.
Signed-off-by: Heiko Schocher<redacted> Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Cc: Grant Likely<grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> Cc: Sekhar Nori<redacted> Cc: Wolfgang Denk<redacted>
[...]
quoted hunk
diff --git a/Documentation/devicetree/bindings/arm/davinci/intc.txt b/Documentation/devicetree/bindings/arm/davinci/intc.txt new file mode 100644 index 0000000..dac2f69 --- /dev/null +++ b/Documentation/devicetree/bindings/arm/davinci/intc.txt@@ -0,0 +1,26 @@ +* TI Davinci Interrupt Controller + +davinci are using a TI interrupt controller that can support several +configurable number of interrupts. + +Main node required properties: + +- compatible : should be: + "ti,davinci-intc"
The right "compatible" prop would be "ti,cp_intc" in this case.
WBR, Sergei