<br><br><div class="gmail_quote">On Fri, Mar 12, 2010 at 4:00 AM, Knut Kujat <span dir="ltr"><<a href="mailto:knuku@gap.upv.es">knuku@gap.upv.es</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hello,<br>
<br>
this patch fixes the issue where the board wasn't able to start after<br>
getting unplugged. I also added some GPIOs so now the power on led is<br>
working.<br></blockquote><div><br>The spacing doesn't look like it follows these guidelines:<br><a href="http://www.coreboot.org/Development_Guidelines#Coding_Style">http://www.coreboot.org/Development_Guidelines#Coding_Style</a> <br>
<br></div><div> static inline void activate_spd_rom(const struct mem_controller *ctrl)<br> {<br>-    /* nothing to do */<br>+#define SMBUS_SWITCH1 0x70<br>+#define SMBUS_SWITCH2 0x72<br>+//     unsigned device=(ctrl->spd_addr[0])>>8;<br>
It's unclear what this comment is for.  Maybe just drop it?<br><br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Signed-off-by: Knut Kujat <<a href="mailto:knuku@gap.upv.es">knuku@gap.upv.es</a>></blockquote><div>With the formatting and comment addressed: <br></div><div>Acked-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>> <br>
<br>Thanks,<br>Myles<br></div></div>