From 14a3feb0686b9c97034de828844f52c75ccc42d1 Mon Sep 17 00:00:00 2001 From: Uwe Hermann Date: Thu, 20 Mar 2008 20:46:44 +0000 Subject: [PATCH] Fix code to allow usage of -Wall in libpayload and the sample (trivial). This even fixes two bugs: - get_cpu_speed() didn't return a value. - The line win->_color - PAIR_NUMBER(0); should actually be win->_color = PAIR_NUMBER(0); Signed-off-by: Uwe Hermann Acked-by: Uwe Hermann git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3182 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1 --- payloads/libpayload/Makefile | 2 +- payloads/libpayload/curses/colors.c | 2 ++ payloads/libpayload/curses/keyboard.c | 1 + payloads/libpayload/curses/tinycurses.c | 7 ++++--- payloads/libpayload/drivers/vga.c | 2 +- payloads/libpayload/i386/coreboot.c | 2 +- payloads/libpayload/i386/main.c | 2 +- payloads/libpayload/i386/timer.c | 2 ++ payloads/libpayload/include/libpayload.h | 2 +- payloads/libpayload/sample/Makefile | 2 +- 10 files changed, 15 insertions(+), 9 deletions(-) diff --git a/payloads/libpayload/Makefile b/payloads/libpayload/Makefile index 8d28c2dcca..aa3f07d639 100644 --- a/payloads/libpayload/Makefile +++ b/payloads/libpayload/Makefile @@ -61,7 +61,7 @@ include $(PLATFORM-y) $(BUILD-y) INCLUDES := -I./include INCLUDES += -I$(shell $(CC) -print-search-dirs | head -n 1 | cut -d' ' -f2)include -CFLAGS := -Werror -Os -fno-stack-protector -nostdinc $(INCLUDES) +CFLAGS := -Wall -Werror -Os -fno-stack-protector -nostdinc $(INCLUDES) libpayload.a: $(TARGETS-y) $(AR) rc $@ $(TARGETS-y) diff --git a/payloads/libpayload/curses/colors.c b/payloads/libpayload/curses/colors.c index 34091498c0..5065f5c825 100644 --- a/payloads/libpayload/curses/colors.c +++ b/payloads/libpayload/curses/colors.c @@ -57,4 +57,6 @@ int pair_content(short index, short *fg, short *bg) *bg = (color_pairs[index] >> 4) & 0xF; *fg = color_pairs[index] & 0xF; + + return 0; } diff --git a/payloads/libpayload/curses/keyboard.c b/payloads/libpayload/curses/keyboard.c index 3ec4d9d7bf..2062ac2ffe 100644 --- a/payloads/libpayload/curses/keyboard.c +++ b/payloads/libpayload/curses/keyboard.c @@ -268,6 +268,7 @@ int wgetch(WINDOW *win) int nodelay(WINDOW *win, NCURSES_BOOL flag) { win->_delay = flag ? 0 : -1; + return 0; } #ifdef CONFIG_VGA_CONSOLE diff --git a/payloads/libpayload/curses/tinycurses.c b/payloads/libpayload/curses/tinycurses.c index 946cdeb18b..f9c0798d05 100644 --- a/payloads/libpayload/curses/tinycurses.c +++ b/payloads/libpayload/curses/tinycurses.c @@ -145,7 +145,7 @@ int delwin(WINDOW *win) } WINDOW *derwin(WINDOW *orig, int num_lines, int num_columns, int begy, int begx) { - WINDOW *win; + WINDOW *win = NULL; int i; int flags = _SUBWIN; @@ -167,6 +167,7 @@ WINDOW *derwin(WINDOW *orig, int num_lines, int num_columns, int begy, int begx) if (orig->_flags & _ISPAD) flags |= _ISPAD; + // FIXME //// if ((win = _nc_makenew(num_lines, num_columns, orig->_begy + begy, //// orig->_begx + begx, flags)) == 0) //// return NULL; @@ -308,8 +309,8 @@ WINDOW *newwin(int num_lines, int num_columns, int begy, int begx) win->_line = &ldat_list[ldat_count++]; - /* FIXME: Is this right? Should the window attributes be normal? */ - win->_color - PAIR_NUMBER(0); + /* FIXME: Is this right? Should the window attributes be normal? */ + win->_color = PAIR_NUMBER(0); win->_attrs = A_NORMAL; for (i = 0; i < num_lines; i++) diff --git a/payloads/libpayload/drivers/vga.c b/payloads/libpayload/drivers/vga.c index fd1dd35abb..1c1315935c 100644 --- a/payloads/libpayload/drivers/vga.c +++ b/payloads/libpayload/drivers/vga.c @@ -184,7 +184,7 @@ void vga_putchar(unsigned int ch) vga_fixup_cursor(); } -int vga_move_cursor(int x, int y) +void vga_move_cursor(int x, int y) { cursorx = x; cursory = y; diff --git a/payloads/libpayload/i386/coreboot.c b/payloads/libpayload/i386/coreboot.c index 0eb76dcf4b..8e4a19a5ee 100644 --- a/payloads/libpayload/i386/coreboot.c +++ b/payloads/libpayload/i386/coreboot.c @@ -83,7 +83,7 @@ static int cb_parse_header(void *addr, int len, struct sysinfo_t *info) for (i = 0; i < len; i += 16, ptr += 16) { header = (struct cb_header *)ptr; - if (!strncmp(header->signature, "LBIO", 4)) + if (!strncmp((const char *)header->signature, "LBIO", 4)) break; } diff --git a/payloads/libpayload/i386/main.c b/payloads/libpayload/i386/main.c index ed5c1b9e9c..c880937e98 100644 --- a/payloads/libpayload/i386/main.c +++ b/payloads/libpayload/i386/main.c @@ -27,7 +27,7 @@ * SUCH DAMAGE. */ -#include +#include /* * This structure seeds the stack. We provide the return address of our main diff --git a/payloads/libpayload/i386/timer.c b/payloads/libpayload/i386/timer.c index a787646073..e11973f2fa 100644 --- a/payloads/libpayload/i386/timer.c +++ b/payloads/libpayload/i386/timer.c @@ -62,6 +62,8 @@ unsigned int get_cpu_speed(void) * Multiply that by the number of measured clocks to get the kHz value. */ cpu_khz = (unsigned int)((end - start) * 1193180U / (1000 * 0xffff)); + + return cpu_khz; } static inline void _delay(unsigned int delta) diff --git a/payloads/libpayload/include/libpayload.h b/payloads/libpayload/include/libpayload.h index 48150089bd..d1ca842b94 100644 --- a/payloads/libpayload/include/libpayload.h +++ b/payloads/libpayload/include/libpayload.h @@ -59,7 +59,7 @@ void vga_fill(uint8_t ch, uint8_t attr); void vga_clear(void); void vga_putc(uint8_t row, uint8_t col, unsigned int c); void vga_putchar(unsigned int ch); -int vga_move_cursor(int x, int y); +void vga_move_cursor(int x, int y); void vga_init(void); /* libc/console.c */ diff --git a/payloads/libpayload/sample/Makefile b/payloads/libpayload/sample/Makefile index 9ab3249e12..1daea5f6ec 100644 --- a/payloads/libpayload/sample/Makefile +++ b/payloads/libpayload/sample/Makefile @@ -37,7 +37,7 @@ INCLUDES += -I$(shell $(CC) $(CROSS_CFLAGS) -print-search-dirs | head -n 1 | cut LIBPAYLOAD = ../libpayload.a LIBGCC := $(shell $(CC) $(CROSS_CFLAGS) -print-libgcc-file-name) -CFLAGS := -Werror -Os -fno-stack-protector -nostdinc $(INCLUDES) +CFLAGS := -Wall -Werror -Os -fno-stack-protector -nostdinc $(INCLUDES) all: hello.elf