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? -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html