For my xmega16A4 FatFS mp3 player I get that warning for my spiC_wrt() in my main.c file.
In main.c:
init_media(); // warning here
In vs1011.c : init_media() -> sd_cmd() -> spiC_wrt()
Since I include vs1011.h in main.c, why would it give this warning? Why doesn't it know about that function via header file just like it knows about init_media() ? I know that putting a func. declaration or extern would fix it though.
thanks in advance
why would it give this warning?
I did, using tools #2 & #3 in sign.
So when you click on the source files window you can see the file that contains the code?
extern void init_media(void); // declared in C or H file ... init_media(); // no warnings now ...
The Compiler must have already been told the 'signature' of any function before it is used.
It is common to put all the function prototypes that are defined in "module.c" in a header called "module.h".
Note that the 'extern' keyword is optional.
A function that takes no parameters should use the void keyword e.g. function(void). A function that does not return a value should be declared as a void function().
If you omit the function return type, it defaults to int.
If you omit the parameters in a prototype the Compiler cannot check your usage.
C files will compile just fine with the correct declarations.
The Linker will complain if it cannot resolve all the externals. e.g. you forget the C file that defines init_media()
David.
John: Yes
David, that's my point I've done all that so it was a surprise to see the warning. If I do the extern like U, I get the same warning.
Is it spelled correctly? Maybe the declaration and definition are spelled differently.
Correctly spelled.
What function does not it like, spiC_wrt or init_media? Is init_media a macro? Your OP does not make much sense.
Doesn't like spiC_wrt(). Init_media()'s called from main.c and init_media() and spiC_wrt() are defined in vs1011.c. Arrows meant that init. calls functions pointed to.
The exact error message would be nice...
Does it complain about main.c or vs1011.c? I guess about vs1011.c. In vs1011.c, do you define spiC_wrt before you call it?
I found it ! In vs1011.h file I had declared it as spi_wrt(). :oops: I must have made the change months ago, 'cause i know it was in there at 1st. I remember now taking out the 'C' just to min. the function name, but then I thought since the xmega has multiple spi that it would be better to qualify each by its port letter ( and forgot to change it back in *.h ). Since the code worked, I just moved on to other things and only now dived back into it to see about getting rid of warning. I'm Sorry to waste your time.
