<br><br><div class="gmail_quote">On Thu, Dec 18, 2008 at 7:52 AM, Jordan Crouse <span dir="ltr"><<a href="mailto:jordan@cosmicpenguin.net">jordan@cosmicpenguin.net</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">Myles Watson wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Thu, Dec 18, 2008 at 3:18 AM, Carl-Daniel Hailfinger <<br>
<a href="mailto:c-d.hailfinger.devel.2006@gmx.net" target="_blank">c-d.hailfinger.devel.2006@gmx.net</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Bao, Zheng found a bug which killed SATA booting on my board.<br>
<br>
This happened because we do not error out on implicit function<br>
declarations. The linker has no way of checking whether the implicitly<br>
assumed function signature is identical to the real signature, so<br>
mismatches can occur and these mismatches are practically impossible to<br>
debug because the code looks completely correct.<br>
<br>
Adding -Werror-implicit-function-declaration to our CFLAGS would solve<br>
this problem nicely, but a lot of files in the tree need to be fixed.<br>
<br>
</blockquote>
<br>
I think this is a great idea.  Isn't the correct order to fix all the<br>
warnings, then make it an error?<br>
</blockquote>
<br></div></div>
Yeah - the unfortunate thing about changes like this is that you end up being responsible for fixing the errors.. :)</blockquote><div><br>Here's my first patch.  It clears up all of them except get_nodes for serengeti.<br>
<br>coreboot/svn/src/cpu/amd/dualcore/dualcore.c:63: warning: implicit declaration of function 'get_nodes'<br><br>The rest were easy.  This one I'm not sure what was supposed to be here.<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>
Carl-Daniel - if you post a list of offending files, we'll all help clear them up.  Dumping the log through grep "implicit declaration of function" should suffice.<font color="#888888"></font></blockquote><div>
<br>If you want to take the get_nodes reference, that would be great.  If this is the way its supposed to be cleaned up, I'll keep going a little more.  I think we should divide it up based on processor type so we don't duplicate work.<br>
<br>Signed-off-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>><br><br>
Thanks,<br>
Myles</div></div><br>