<br><br><div class="gmail_quote">On Sat, Oct 24, 2009 at 5:11 PM, 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="h5">On Fri, Oct 23, 2009 at 05:06:35PM -0600, Myles Watson wrote:<br>
> On Fri, Oct 23, 2009 at 4:56 PM, Uwe Hermann <<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>> wrote:<br>
><br>
> > On Fri, Oct 23, 2009 at 02:43:27PM -0600, Myles Watson wrote:<br>
> > > Ping.  I think I've addressed all of Peter and Uwe's concerns.<br>
> ><br>
> > I Think so, but please repost the updated patch so we can have a look<br>
> > at the final version.<br>
> ><br>
> I don't think the first patches have changed much, but there are two new<br>
> ones that apply on top: peter.diff and uwe.diff.<br>
><br>
> Signed-off-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>><br>
<br>
</div></div>Acked-by: Uwe Hermann <<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>><br></blockquote><div>Rev 4856.<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;">

<div class="im"><br>
<br>
> +config HAVE_LOW_TABLES<br>
> +     bool<br>
> +     default y<br>
> +     help<br>
> +       This Option is unused in the code.  Since two boards try to set it to<br>
> +       'n', they may be broken.  We either need to make the option useful or<br>
> +       get rid of it.  The broken boards are:<br>
> +       asus/m2v-mx_se<br>
> +       supermicro/h8dme<br>
<br>
</div>Please add a "# TODO" comment on top of that for easy grepping.<br></blockquote><div>Done.<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;">

> +config MEM_TRAIN_SEQ<br>
> +     int<br>
> +     default 2<br>
<br>
We should add a MEM_TRAIN_SEQ comment or help text, it's unclear to me what it is<br>
supposed to do. Is it a per-chipset, per-cpu, or per-board option? What<br>
do the values of the variable mean?<br></blockquote><div>All good questions.  It's unclear to me.<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;">

Regardless of that, it seems like it should be a user-visible option in<br>
menuconfig with useful option names for the values of 0, 1, and 2 (which<br>
seem to be the only valid ones).<br></blockquote>The more things we make "user configurable", the more ways a user can break the build/ brick his board.<br><br>Thanks,<br>Myles<br></div><br>