<br><br><div class="gmail_quote">On Sat, Oct 17, 2009 at 8:52 AM, Uwe Hermann <span dir="ltr"><<a href="mailto:uwe@hermann-uwe.de" target="_blank">uwe@hermann-uwe.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><div></div><div><br>
> Signed-off-by: Myles Watson <<a href="mailto:mylesgw@gmail.com" target="_blank">mylesgw@gmail.com</a>><br>
<br>
</div></div>Sounds good. If it survives abuild and kbuildall this is<br>
<br>
Acked-by: Uwe Hermann <<a href="mailto:uwe@hermann-uwe.de" target="_blank">uwe@hermann-uwe.de</a>><br></blockquote><div>Rev  4809.<br></div><div> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">


> Index: cbv2/src/arch/i386/boot/coreboot_table.c<br>
> ===================================================================<br>
> --- cbv2.orig/src/arch/i386/boot/coreboot_table.c<br>
> +++ cbv2/src/arch/i386/boot/coreboot_table.c<br>
> @@ -99,7 +99,7 @@ static struct lb_memory *lb_memory(struc<br>
><br>
>  static struct lb_serial *lb_serial(struct lb_header *header)<br>
>  {<br>
> -#if defined(CONFIG_TTYS0_BASE)<br>
> +#if CONFIG_CONSOLE_SERIAL8250<br>
<br>
Does this affect newconfig? Is CONFIG_CONSOLE_SERIAL8250 properly<br>
defined and set in newconfig, or only in kconfig?<br></blockquote><div>CONFIG_CONSOLE_SERIAL8250 defaults to 0 and is always exported.<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;">

<br>
> Index: cbv2/src/southbridge/via/vt8237r/vt8237r_lpc.c<br>
> ===================================================================<br>
> --- cbv2.orig/src/southbridge/via/vt8237r/vt8237r_lpc.c<br>
> +++ cbv2/src/southbridge/via/vt8237r/vt8237r_lpc.c<br>
<br>
<br>
> @@ -499,7 +499,7 @@ static void vt8237_common_init(struct de<br>
>       /* Delay transaction control */<br>
>       pci_write_config8(dev, 0x43, 0xb);<br>
><br>
> -#ifdef CONFIG_EPIA_VT8237R_INIT<br>
> +#if !CONFIG_EPIA_VT8237R_INIT<br>
<br>
This looks incorrect, the "!" should not be there.<br></blockquote><div>Good catch!<br><br>Thanks,<br>Myles <br></div></div><br>