Re: [PATCH 2/2] ARM: vexpress: Enable selection of SPARSE_IRQ

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

On Wed, 2012-06-13 at 14:23 -0500, Rob Herring wrote:
> On 06/13/2012 12:13 PM, Jon Medhurst (Tixy) wrote:
> > From: Jon Medhurst <tixy@xxxxxxxxxx>
> > 
> > This is required to add vexpress to multiplatform kernels containing
> > other boards which require SPARSE_IRQ.
> > 
> > Signed-off-by: Jon Medhurst <tixy@xxxxxxxxxx>
> > ---
> >  arch/arm/Kconfig |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> > index 84449dd..bdede09 100644
> > --- a/arch/arm/Kconfig
> > +++ b/arch/arm/Kconfig
> > @@ -311,6 +311,7 @@ config ARCH_VEXPRESS
> >  	select HAVE_CLK
> >  	select HAVE_PATA_PLATFORM
> >  	select ICST
> > +	select MAY_HAVE_SPARSE_IRQ
> 
> Why not always select SPARSE_IRQ and remove mach/irqs.h?

I did consider that but my conservative side won out. If there are no
real downsides to sparse irqs that people would object to then I'm happy
to unconditionally enable it. (I'll post a v2 patch that does this.)

-- 
Tixy


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel


[Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [PDAs]     [Linux]     [Linux MIPS]     [Yosemite Campsites]     [Photos]

Add to Google Follow linuxarm on Twitter