<br><br><div class="gmail_quote">On Thu, Dec 18, 2008 at 3:18 AM, Carl-Daniel Hailfinger <span dir="ltr"><<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.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;">
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>
</blockquote><div><br>I think this is a great idea.  Isn't the correct order to fix all the warnings, then make it an error?<br><br>Thanks,<br>Myles <br></div></div><br>