[coreboot] [commit] r5319 - trunk/src/arch/i386/include/arch

[Myles Watson]

On Mon, Mar 29, 2010 at 4:35 PM, Stefan Reinauer <stepan at coresystems.de>wrote:

>  On 3/30/10 12:15 AM, Myles Watson wrote:
>
>  Modified: trunk/src/arch/i386/include/arch/pirq_routing.h
>>
>> ==============================================================================
>> --- trunk/src/arch/i386/include/arch/pirq_routing.h     Mon Mar 29
>> 23:56:26 2010        (r5318)
>> +++ trunk/src/arch/i386/include/arch/pirq_routing.h     Tue Mar 30
>> 00:05:26 2010        (r5319)
>> @@ -18,9 +18,8 @@
>>
>>  #if defined(CONFIG_IRQ_SLOT_COUNT)
>>  #define IRQ_SLOTS_COUNT CONFIG_IRQ_SLOT_COUNT
>> -#elif (__GNUC__ < 3)
>> -#define IRQ_SLOTS_COUNT 1
>>  #else
>> +#warning "No IRQ_SLOT_COUNT in Kconfig."
>>  #define IRQ_SLOTS_COUNT
>>  #endif
>>
>>  I think we should get rid of IRQ_SLOTS_COUNT and move the error into
> Kconfig.  The added warning is very similar to one in pirq_routing.c
>
>
> I like that much better... do you have a patch in mind?
>
Very fair question.  Not yet.  I sent the mail to remind myself.  I'll send
one soon if you don't beat me to it.

Thanks,
Myles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100329/79a0c69f/attachment.html>