[coreboot] [PATCH] YABEL compilation warnings
Myles Watson
mylesgw at gmail.com
Fri Mar 5 16:13:39 CET 2010
On Fri, Mar 5, 2010 at 8:04 AM, Myles Watson <mylesgw at gmail.com> wrote:
> This patch removes most of the rest of the compilation warnings for me.
>
> 1. Move run_bios prototype to device.h
> 2. Use time.h for get_time()
>
2b. Move tb_freq into functions.c instead of the time.h
> 3. Move read_io and write_io to io.c and make them static
> 4. Make a couple of functions static in interrupt.c
> 5. Refactor a cast from char[] to u64 to get rid of potential alignment
> problems and a warning
>
> The only ones left are "unused function" warnings.
>
> 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?
>
> This next part isn't part of the patch, but applying it makes qemu compile
> with yabel (with and without debugging).
>
> Index: Makefile
> ===================================================================
> --- Makefile (revision 5186)
> +++ Makefile (working copy)
> @@ -239,7 +239,7 @@
> CFLAGS = $(INCLUDES) -Os -nostdinc
> CFLAGS += -nostdlib -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes
> CFLAGS += -Wwrite-strings -Wredundant-decls -Wno-trigraphs
> -CFLAGS += -Wstrict-aliasing -Wshadow
>
I meant no-unused-function:
> +CFLAGS += -Wstrict-aliasing -Wshadow -Wno-unused-function
>
> ifeq ($(CONFIG_WARNINGS_ARE_ERRORS),y)
> CFLAGS += -Werror
> endif
>
Signed-off-by: Myles Watson <mylesgw at gmail.com>
Thanks,
Myles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100305/9e80a2f6/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: warnings.diff
Type: text/x-patch
Size: 7363 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100305/9e80a2f6/attachment.diff>
More information about the coreboot
mailing list