<br><br><div class="gmail_quote">On Fri, Mar 5, 2010 at 8:04 AM, Myles Watson <span dir="ltr"><<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</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;">
This patch removes most of the rest of the compilation warnings for me.  <br><br>1. Move run_bios prototype to device.h<br>2. Use time.h for get_time()<br></blockquote><div>2b. Move tb_freq into functions.c instead of the time.h <br>
</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">3. Move read_io and write_io to io.c and make them static<br>4. Make a couple of functions static in interrupt.c<br>

5. Refactor a cast from char[] to u64 to get rid of potential alignment problems and a warning <br><br>The only ones left are "unused function" warnings.<br><br>I think we should get rid of that warning, since we conditionally call functions based on debugging and various config variables.  Is there a case where it helps enough to justify all the warnings?<br>

<br>This next part isn't part of the patch, but applying it makes qemu compile with yabel (with and without debugging).<br><br>Index: Makefile<br>===================================================================<br>

--- Makefile    (revision 5186)<br>+++ Makefile    (working copy)<br>@@ -239,7 +239,7 @@<br> CFLAGS = $(INCLUDES) -Os -nostdinc<br> CFLAGS += -nostdlib -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes<br> CFLAGS += -Wwrite-strings -Wredundant-decls -Wno-trigraphs <br>

-CFLAGS += -Wstrict-aliasing -Wshadow <br></blockquote><div> <br>I meant no-unused-function:<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
+CFLAGS += -Wstrict-aliasing -Wshadow -Wno-unused-function<br></blockquote><div> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> ifeq ($(CONFIG_WARNINGS_ARE_ERRORS),y)<br>
 CFLAGS += -Werror<br> endif<br></blockquote><div> </div><div> Signed-off-by: Myles Watson <<a href="mailto:mylesgw@gmail.com" target="_blank">mylesgw@gmail.com</a>><br><br>Thanks,<br>Myles<br></div></div><br>