Re: [PATCH v2 2/8] irqchip: Supply new driver for STi based devices

[Arnd Bergmann]

On Thursday 27 November 2014 09:02:55 Lee Jones wrote:
> On Tue, 25 Nov 2014, Arnd Bergmann wrote:
> 
> > On Tuesday 25 November 2014 16:24:59 Lee Jones wrote:
> > 
> > > diff --git a/drivers/irqchip/Kconfig b/drivers/irqchip/Kconfig
> > > index b21f12f..e502f15 100644
> > > --- a/drivers/irqchip/Kconfig
> > > +++ b/drivers/irqchip/Kconfig
> > > @@ -93,6 +93,13 @@ config RENESAS_IRQC
> > >     bool
> > >     select IRQ_DOMAIN
> > >  
> > > +config ST_IRQCHIP
> > > +   bool
> > > +   select REGMAP
> > > +   select MFD_SYSCON
> > > +   help
> > > +     Enables SysCfg Controlled IRQs on STi based platforms.
> > > +
> > 
> > I'm confused by the purpose of this code. It's apparently a driver
> > in drivers/irqchip, the Kconfig symbol contains the string IRQCHIP,
> > yet it doesn't actually register an irq_chip.
> > 
> > Also, the name is a bit too generic, ST has lots of different irqchips,
> > and this apparently isn't even one of them 
> 
> Hmm... now you're going to ask me to remember who I had the
> conversation with that alluded to this as the best location for this
> driver.  Unfortunately, I cannot.  Can you think of a better place to
> put it then?

I suspect it's in the right place but should actually be an irqchip
driver. I'm having trouble understanding what this code actually does,
can you you explain the functionality in more detail so we can figure
out what to do with it?

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