[coreboot] [PATCH] warning days - m57sli/mcp55
Myles Watson
mylesgw at gmail.com
Mon Apr 12 23:56:49 CEST 2010
> If there are better ways to kill the warnings, please let me know!
> Index: src/mainboard/gigabyte/m57sli/fanctl.c
> ===================================================================
> --- src/mainboard/gigabyte/m57sli/fanctl.c (revision 5411)
> +++ src/mainboard/gigabyte/m57sli/fanctl.c (working copy)
> @@ -71,6 +71,7 @@
> /*
> * Called from superio.c
> */
> +extern void init_ec(uint16_t base);
> void init_ec(uint16_t base)
> {
> int i;
I would ack the patch except for the added extern statements. I thought
extern was for functions and variables that are declared in another file.
I've been fixing the "no previous prototype" warnings in two ways:
1. declare the function static if it is only used in that file
2. Add a prototype in a header that is included in the file where the
function is defined and the file where it's used.
Thanks,
Myles
More information about the coreboot
mailing list