<br><br><div class="gmail_quote">On Thu, Nov 5, 2009 at 10:57 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">Myles Watson wrote:<br>
> +#ifdef CONFIG_COREBOOT_V2<br>
> +             dev = dev_find_device(X86_DX, X86_CX, dev);<br>
><br>
> This looks wrong.  I know it really isn't part of the patch, but isn't<br>
> CONFIG_COREBOOT_V2 always defined?  I guess it isn't in v3...<br>
</div>It should be always defined in v2, and never in v2... Not sure if things<br>
changed meanwhile, but that part of the code is not new, just copied..<br></blockquote><div>That's what I meant by "not part of the patch"<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;">

We could make v2 more similar to v3<br>
<br>
svn mv src/include/console/* src/include<br>
<br>
and<br>
<br>
s/dev_find_device/dev_find_pci_device/<br>
<br>
if you feel like we should drop the define.<br>
<br>
(Or we just drop the define since x86emu is no longer shared with v3)<br></blockquote><div>Either way is fine.<br><br>Thanks,<br>Myles<br></div></div>