[coreboot] [commit] r5319 - trunk/src/arch/i386/include/arch
Stefan Reinauer
stepan at coresystems.de
Tue Mar 30 00:40:03 CEST 2010
On 3/30/10 12:35 AM, Stefan Reinauer 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?
Hm.. how do we error in Kconfig? Is there a way to do that at all? I
think we didn't use this before...
Should we drop the whole block of code in pirq_routing.h and make it an
#error in pirq_routing.c?
--
coresystems GmbH . Brahmsstr. 16 . D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 . Fax: +49 761 7664613
Email: info at coresystems.de . http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg . HRB 7656
Geschäftsführer: Stefan Reinauer . Ust-IdNr.: DE245674866
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100330/204afbe2/attachment.html>
More information about the coreboot
mailing list