[PATCH v2 11/13] ARM: only include mach/irqs.h for !SPARSE_IRQ
Rob Herring
robherring2 at gmail.com
Fri Jan 20 16:23:01 EST 2012
On 01/20/2012 03:11 PM, Nicolas Pitre wrote:
> On Fri, 20 Jan 2012, Rob Herring wrote:
>
>> From: Rob Herring <rob.herring at calxeda.com>
>>
>> Make mach/irqs.h optional for SPARSE_IRQ. With this change irqs.h can be
>> removed by converting platforms over to sparse irq.
>>
>> This intentionally breaks platforms that enable SPARSE_IRQ.
>
> I don't get what you mean here. The above seems contradictory.
>
You're right. The intro explains things more clearly. This breaks
platforms (at boot time) that don't select SPARSE_IRQ, but let users
enable it in their config. I don't understand why sparse irq is a user
visible config option. We could move HAVE_SPARSE_IRQ down to each
platform that selects SPARSE_IRQ and prevent enabling, but I think
allowing it to break is good encouragement for others to fix those
platforms. I'm open to other ideas.
Rob
More information about the linux-arm-kernel
mailing list