[coreboot] [PATCH] cleanup x86emu without yabel

[Myles Watson]

On Thu, Nov 5, 2009 at 10:57 AM, Stefan Reinauer <stepan at coresystems.de>wrote:

> Myles Watson wrote:
> > +#ifdef CONFIG_COREBOOT_V2
> > +             dev = dev_find_device(X86_DX, X86_CX, dev);
> >
> > This looks wrong.  I know it really isn't part of the patch, but isn't
> > CONFIG_COREBOOT_V2 always defined?  I guess it isn't in v3...
> It should be always defined in v2, and never in v2... Not sure if things
> changed meanwhile, but that part of the code is not new, just copied..
>
That's what I meant by "not part of the patch"


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

Thanks,
Myles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20091105/b94fa110/attachment.html>