Re: [PATCH 02/11] arm: pxa27x: support ICP DAS LP-8x4x

2013-12-11 Thread Arnd Bergmann
On Wednesday 11 December 2013, Sergei Ianovich wrote: > On Wed, 2013-12-11 at 06:11 +0100, Arnd Bergmann wrote: > > It depends: if the driver is for the entire FPGA and does > > the irqchip stuff in addition, it should probably live > > in drivers/mfd. If it's a pure irqchip driver, drivers/irqchip

Re: [PATCH 02/11] arm: pxa27x: support ICP DAS LP-8x4x

2013-12-10 Thread Sergei Ianovich
On Wed, 2013-12-11 at 06:11 +0100, Arnd Bergmann wrote: > It depends: if the driver is for the entire FPGA and does > the irqchip stuff in addition, it should probably live > in drivers/mfd. If it's a pure irqchip driver, drivers/irqchip > is better. You have to be careful in the second case though

Re: [PATCH 02/11] arm: pxa27x: support ICP DAS LP-8x4x

2013-12-10 Thread Arnd Bergmann
On Wednesday 11 December 2013, Sergei Ianovich wrote: > > It probably makes sense to have a single driver file for the > > FPGA device that does this, and only split out the other devices > > from it that consume the irqs. > > Is drivers/irqchip/ the right place this driver? > > I am asking becau

Re: [PATCH 02/11] arm: pxa27x: support ICP DAS LP-8x4x

2013-12-10 Thread Sergei Ianovich
On Tue, 2013-12-10 at 22:57 +0100, Arnd Bergmann wrote: > It should be possible to make it a loadable module, with deferred > probing etc. You wouldn't use IRQCHIP_DECLARE() for this though, > but instead have a platform driver that sets up the irq domain. Thanks for explaning. > It probably make

Re: [PATCH 02/11] arm: pxa27x: support ICP DAS LP-8x4x

2013-12-10 Thread Arnd Bergmann
On Tuesday 10 December 2013, Sergei Ianovich wrote: > On Tue, 2013-12-10 at 13:33 +0100, Linus Walleij wrote: > > On Fri, Dec 6, 2013 at 6:14 PM, Arnd Bergmann wrote: > > > On Friday 06 December 2013, Sergei Ianovich wrote: > > >> On Fri, 2013-12-06 at 01:40 +0100, Arnd Bergmann wrote: > > >> > >

Re: [PATCH 02/11] arm: pxa27x: support ICP DAS LP-8x4x

2013-12-10 Thread Sergei Ianovich
On Tue, 2013-12-10 at 13:33 +0100, Linus Walleij wrote: > On Fri, Dec 6, 2013 at 6:14 PM, Arnd Bergmann wrote: > > On Friday 06 December 2013, Sergei Ianovich wrote: > >> On Fri, 2013-12-06 at 01:40 +0100, Arnd Bergmann wrote: > >> > >> > > + > >> > > +static struct irq_chip lp8x4x_irq_chip = { >

Re: [PATCH 02/11] arm: pxa27x: support ICP DAS LP-8x4x

2013-12-10 Thread Linus Walleij
On Fri, Dec 6, 2013 at 6:14 PM, Arnd Bergmann wrote: > On Friday 06 December 2013, Sergei Ianovich wrote: >> On Fri, 2013-12-06 at 01:40 +0100, Arnd Bergmann wrote: >> >> > > + >> > > +static struct irq_chip lp8x4x_irq_chip = { >> > > + .name = "FPGA", >> > > + .irq_ack

Re: [PATCH 02/11] arm: pxa27x: support ICP DAS LP-8x4x

2013-12-06 Thread Arnd Bergmann
On Friday 06 December 2013, Sergei Ianovich wrote: > On Fri, 2013-12-06 at 01:40 +0100, Arnd Bergmann wrote: > > > Ideally we want shared defconfigs where you enable multiple (or all) pxa > > boards > > at once and end up with a kernel that works on all of them. > > > > For most other platforms

Re: [PATCH 02/11] arm: pxa27x: support ICP DAS LP-8x4x

2013-12-06 Thread Sergei Ianovich
On Fri, 2013-12-06 at 01:40 +0100, Arnd Bergmann wrote: > On Sunday 01 December 2013, Sergei Ianovich wrote: > > > > diff --git a/arch/arm/configs/lp8x4x_defconfig > > b/arch/arm/configs/lp8x4x_defconfig > > new file mode 100644 > > index 000..5cd6d38 > > --- /dev/null > > +++ b/arch/arm/conf

Re: [PATCH 02/11] arm: pxa27x: support ICP DAS LP-8x4x

2013-12-05 Thread Arnd Bergmann
On Sunday 01 December 2013, Sergei Ianovich wrote: > > diff --git a/arch/arm/configs/lp8x4x_defconfig > b/arch/arm/configs/lp8x4x_defconfig > new file mode 100644 > index 000..5cd6d38 > --- /dev/null > +++ b/arch/arm/configs/lp8x4x_defconfig > @@ -0,0 +1,2320 @@ > +# > +# Automatically genera