Date: Thu, 24 Apr 2014 19:47:41 +0200
From: Anders Berg <anders.berg@lsi.com>
To: Linus Walleij <linus.walleij@linaro.org>
CC: Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>,
        Mike Turquette <mturquette@linaro.org>,
        Mark Rutland <mark.rutland@arm.com>,
        Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>,
        David Woodhouse <dwmw2@infradead.org>,
        Russell King <linux@arm.linux.org.uk>,
        "linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 3/6] ARM: dts: Device tree for AXM55xx.
Message-ID: <20140424174741.GA23444@swsaberg01>
References: <cover.1398335771.git.anders.berg@lsi.com>
 <2386bd1367aa44741979461358a72dec89608597.1398335771.git.anders.berg@lsi.com>
 <CACRpkdbU6UashxCuPMDjEB_nbB=mXicGpjJy=HGC6r3qfxCtVQ@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <CACRpkdbU6UashxCuPMDjEB_nbB=mXicGpjJy=HGC6r3qfxCtVQ@mail.gmail.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
Sender: linux-kernel-owner@vger.kernel.org

On Thu, Apr 24, 2014 at 03:24:14PM +0200, Linus Walleij wrote:
> On Thu, Apr 24, 2014 at 12:44 PM, Anders Berg <anders.berg@lsi.com> wrote:
> 
> > Add device tree for the Amarillo validation board with an AXM5516 SoC.
> >
> > Signed-off-by: Anders Berg <anders.berg@lsi.com>
> (...)
> 
> > +                       timer0: timer@2010091000 {
> > +                               compatible = "arm,sp804", "arm,primecell";
> > +                               reg = <0x20 0x10091000 0 0x1000>;
> > +                               interrupts = <GIC_SPI 46 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 46 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 47 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 49 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 51 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 52 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 53 IRQ_TYPE_LEVEL_HIGH>;
> > +                               clocks = <&clk_per>;
> > +                               clock-names = "apb_pclk";
> > +                               status = "okay";
> > +                       };
> > +
> > +                       gpio0: gpio@2010092000 {
> > +                               #gpio-cells = <2>;
> > +                               compatible = "arm,pl061", "arm,primecell";
> > +                               gpio-controller;
> > +                               reg = <0x20 0x10092000 0x00 0x1000>;
> > +                               interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 16 IRQ_TYPE_LEVEL_HIGH>,
> > +                                            <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>;
> > +                               clocks = <&clk_per>;
> > +                               clock-names = "apb_pclk";
> > +                               status = "disabled";
> > +                       };
> 
> One interrupt per CPU core?
> 
> The drivers for these blocks will really just grab the first IRQ and
> then I guess they
> will only be able to execute on CPU0.
> 
> It's definately correct to list all the IRQs here, but how do you envision
> the drivers making use of them in the long run?

It's one interrupt line per input pin (so with the current driver only the first pin
is usable as interrupt source).

/Anders

> 
> Yours,
> Linus Walleij
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/