<br><br><div class="gmail_quote">On Tue, Dec 15, 2009 at 11:21 AM, ron minnich <span dir="ltr"><<a href="mailto:rminnich@gmail.com">rminnich@gmail.com</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><div></div><div class="h5">On Tue, Dec 15, 2009 at 9:06 AM, Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> wrote:<br>
><br>
>> yes I can confirm that the Kconfig build method work for me too. As<br>
>> Rudolf said I had to disable VGA ROM as well otherwise it will came up<br>
>> with a exception error. It's necessary to have "Serial port console<br>
>> output" set to yes or it won't compile.<br>
><br>
> I'm not sure it makes sense for Coreboot to have that option.  It only takes<br>
> a few #if defined() statements to make it compile, but I'd rather see the<br>
> option disappear.  Users that want more speed can choose a different console<br>
> verbosity.<br>
><br>
> Comments?<br>
<br>
<br>
</div></div>Assuming all machines now and forever will have serial console ... I<br>
am pretty sure that's an unsafe assumption.<br></blockquote><div>I guess I was looking at it more like this:  All the boards that we support have serial ports.  If we have a serial port it makes sense to enable it.  The option to disable the serial port should disappear because it can be subsumed in the "I have a serial port (CONFIG_SUPERIO_*)" option.  I can see your point, though.<br>
<br>So, to fix the problem of Kconfig having options that break the build, you'd like to use #ifdefs, right?<br><br>Thanks,<br>Myles <br></div></div>