<br><br><div class="gmail_quote">On Fri, Dec 18, 2009 at 11:43 AM, 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 class="im">On 12/18/09 5:56 PM, Peter Stuge wrote:<br>
> Patrick Georgi wrote:<br>
><br>
>> +++ src/mainboard/kontron/986lcd-m/Kconfig   (working copy)<br>
>> @@ -6,15 +6,22 @@<br>
>>      select NORTHBRIDGE_INTEL_I945<br>
>>      select SOUTHBRIDGE_INTEL_I82801GX<br>
>>      select SUPERIO_WINBOND_W83627THG<br>
>> -    select HAVE_PIRQ_TABLE<br>
>> -    select HAVE_MP_TABLE<br>
>> +    select GENERATE_ACPI_TABLES<br>
>> +    select GENERATE_PIRQ_TABLE<br>
>> +    select GENERATE_MP_TABLE<br>
>><br>
> Hmm? Shouldn't GENERATE_ be set by the user.. ?<br>
><br>
><br>
</div>The user should never have to worry about that stuff.<br>
We should not expect the user to manually enable ACPI and PIRQ before he<br>
has a chance to get a bootable system with coreboot.<br></blockquote><div>As long as the user doesn't touch it, GENERATE_ was set by default as long as the board has HAVE_* set.  What this patch did was make it so that the user can't build without ACPI tables.  I don't think that is what's wanted.  Now to remove the MP_TABLE, the user would have to edit the Kconfig file.<br>
<br>Thanks,<br>Myles <br></div></div><br>