summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorUwe Hermann <uwe@hermann-uwe.de>2008-03-20 20:46:44 +0000
committerUwe Hermann <uwe@hermann-uwe.de>2008-03-20 20:46:44 +0000
commit14a3feb0686b9c97034de828844f52c75ccc42d1 (patch)
treea3e4a54ab6796cc60b045a8f03bf23ff5a000d7e
parent35845a2acb869ea028c98c8c2b912ae3c61053e2 (diff)
downloadcoreboot-14a3feb0686b9c97034de828844f52c75ccc42d1.tar.xz
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 <uwe@hermann-uwe.de> Acked-by: Uwe Hermann <uwe@hermann-uwe.de> git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3182 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
-rw-r--r--payloads/libpayload/Makefile2
-rw-r--r--payloads/libpayload/curses/colors.c2
-rw-r--r--payloads/libpayload/curses/keyboard.c1
-rw-r--r--payloads/libpayload/curses/tinycurses.c7
-rw-r--r--payloads/libpayload/drivers/vga.c2
-rw-r--r--payloads/libpayload/i386/coreboot.c2
-rw-r--r--payloads/libpayload/i386/main.c2
-rw-r--r--payloads/libpayload/i386/timer.c2
-rw-r--r--payloads/libpayload/include/libpayload.h2
-rw-r--r--payloads/libpayload/sample/Makefile2
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 <arch/types.h>
+#include <libpayload.h>
/*
* 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