<br><br><div class="gmail_quote">On Mon, Mar 29, 2010 at 4:40 PM, Stefan Reinauer <span dir="ltr"><<a href="mailto:stepan@coresystems.de">stepan@coresystems.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">



  

<div bgcolor="#ffffff" text="#000000"><div class="im">
On 3/30/10 12:35 AM, Stefan Reinauer wrote:
<blockquote type="cite">
  
On 3/30/10 12:15 AM, Myles Watson wrote:<br>
  <br>
  <blockquote type="cite">
    <div class="gmail_quote">
    <blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Modified:
trunk/src/arch/i386/include/arch/pirq_routing.h<br>
==============================================================================<br>
--- trunk/src/arch/i386/include/arch/pirq_routing.h     Mon Mar 29
23:56:26 2010        (r5318)<br>
+++ trunk/src/arch/i386/include/arch/pirq_routing.h     Tue Mar 30
00:05:26 2010        (r5319)<br>
@@ -18,9 +18,8 @@<br>
      <br>
 #if defined(CONFIG_IRQ_SLOT_COUNT)<br>
 #define IRQ_SLOTS_COUNT CONFIG_IRQ_SLOT_COUNT<br>
-#elif (__GNUC__ < 3)<br>
-#define IRQ_SLOTS_COUNT 1<br>
 #else<br>
+#warning "No IRQ_SLOT_COUNT in Kconfig."<br>
 #define IRQ_SLOTS_COUNT<br>
 #endif<br>
      <font color="#888888"><br>
      </font></blockquote>
    <div>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<br>
    </div>
    </div>
  </blockquote>
  <br>
I like that much better... do you have a patch in mind?<br>
</blockquote>
<br></div>
Hm.. how do we error in Kconfig? Is there a way to do that at all? I
think we didn't use this before...<br></div></blockquote><div>I was thinking about making HAVE_PIRQ_TABLE depend on CONFIG_IRQ_SLOT_COUNT<br><br>Then when someone ported Coreboot to a board they'd set it correctly.  We could tell from abuild results how many boards have HAVE_PIRQ_TABLE set and not CONFIG_IRQ_SLOT_COUNT.<br>
 <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div bgcolor="#ffffff" text="#000000">
Should we drop the whole block of code in pirq_routing.h and make it an
#error in pirq_routing.c?</div></blockquote><div>I guess it depends on how many boards it breaks.<br><br>Thanks,<br>Myles <br></div></div><br>