[PATCH] ARM: integrator: convert to sparse irqs

[Linus Walleij]

On Mon, Feb 27, 2012 at 4:42 AM, Rob Herring <robherring2 at gmail.com> wrote:
> On 02/26/2012 03:45 AM, Linus Walleij wrote:
>> On Fri, Feb 24, 2012 at 8:15 PM, Jean-Christophe PLAGNIOL-VILLARD
>> <plagnioj at jcrosoft.com> wrote:
>>> On 18:32 Fri 24 Feb     , Linus Walleij wrote:
>>
>>>> -#define NR_IRQS                         47
>>>> +#define NR_IRQS_INTEGRATOR_AP                34
>>>> +#define NR_IRQS_INTEGRATOR_CP                47
>>>
>>> why do you need to specified both?
>>
>> They are two totally different machines and should never have had that
>> common roof of NR_IRQS = max(AP,CP), it's just not elegant.
>>
>> Maybe I'm a bit pedantic but I want to set the .nr_irqs for each machine so that
>> it corresponds exactly to what it actually has, instead of using a roof value.
>> I think it improves readability and understanding.
>
> Really, you want to get rid of the value altogether. Each irq_chip
> should allocate its irq_descs and then you don't need to set .nr_irqs.

OK can you point to a machine that does it the right way and
I'll fix.

Or do I need the irq domains lib to go in first to do that in a
good/non-awkward way?

Yours,
Linus Walleij