<br><br><div class="gmail_quote">On Fri, Apr 16, 2010 at 4:49 PM, Stefan Reinauer <span dir="ltr"><<a href="mailto:stepan@coresystems.de">stepan@coresystems.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On 4/16/10 2:43 PM, Myles Watson wrote:<br>
>> Modified: trunk/util/vgabios/include/console/console.h<br>
>> ==========================================================================<br>
>> ====<br>
>> --- trunk/util/vgabios/include/console/console.h     Fri Apr 16 03:45:44<br>
>> 2010 (r5450)<br>
>> +++ trunk/util/vgabios/include/console/console.h     Fri Apr 16 13:45:16<br>
>> 2010 (r5451)<br>
>> @@ -1,3 +1,4 @@<br>
>>  #ifndef _CONSOLE_CONSOLE_H<br>
>>  #define _CONSOLE_CONSOLE_H<br>
>> +#define CONFIG_X86EMU_DEBUG 1<br>
>><br>
> If this is a config variable it should be set in Kconfig.  If it isn't, it<br>
> should probably have a different name.<br>
<br>
</div>It's a utility completely independent of coreboot (well, supposed to<br>
be), just sharing some code. So I think it should not depend on the<br>
coreboot Kconfig. But I didn't want to add a separate Kconfig build<br>
system either, so I just hard coded the one variable...<br></blockquote><div> </div><div>Sorry about that.  I should have looked more closely at the path.  Since x86_emu used to be in util ...<br><br>Thanks for straightening me out,<br>
Myles<br></div></div>