<br><br><div class="gmail_quote">On Wed, Dec 10, 2008 at 11:02 AM, Uwe Hermann <span dir="ltr"><<a href="mailto:uwe@hermann-uwe.de">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 class="Wj3C7c">On Wed, Dec 10, 2008 at 09:57:21AM -0700, Myles Watson wrote:<br>
> This patch fixes some warnings in the Kontron build and a little bit of<br>
> whitespace.<br>
><br>
> Signed-off-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>><br>
><br>
> Thanks,<br>
> Myles<br>
<br>
</div></div>> Index: svn/northbridge/intel/i945/raminit.c<br>
> ===================================================================<br>
> --- svn.orig/northbridge/intel/i945/raminit.c<br>
> +++ svn/northbridge/intel/i945/raminit.c<br>
> @@ -1070,7 +1070,7 @@ static void sdram_rcomp_buffer_strength_<br>
>       };<br>
><br>
>       const u8 * strength_multiplier;<br>
> -     const u8* const * slew_group_lookup;<br>
> +     const u8 * slew_group_lookup;<br>
<br>
That should be<br>
<br>
> +     const u8 *slew_group_lookup;</blockquote><div>fixed. <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;"><br>
> -static void set_subsystem(struct device * dev, u16 vendor, u16 device)<br>
> +static void set_subsystem(struct device * dev, unsigned vendor, unsigned device)<br>
<br>
Nope, u16 is the correct one, please don't change in this direction,<br>
rather change more *int* to u8/u16/u32 where appropriate.</blockquote><div>That was Ron's comment too.  Fixed. <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: svn/mainboard/kontron/986lcd-m/stage1.c<br>
> ===================================================================<br>
> --- svn.orig/mainboard/kontron/986lcd-m/stage1.c<br>
> +++ svn/mainboard/kontron/986lcd-m/stage1.c<br>
> @@ -207,7 +207,6 @@ void hardware_stage1(void)<br>
>  {<br>
>       void    early_superio_config_w83627thg(void);<br>
>       void    ich7_enable_lpc(void);<br>
> -     int boot_mode = 0;<br>
<br>
Not used/needed?<br>
</blockquote><div><br>Not used.<br><br>Thanks for the quick review.  Sorry our mails crossed.<br><br>Myles </div></div><br>