AVR Studio: Warnung beim "builden"

[Patrick (Gast)]

Hallo zusammen!!

Ich hab mal ne Frage zu einer Warnmeldung, die ich bekomme, wenn ich 
mein Project im AVR Studio "builde".
Ich bekomm die Meldung:
../can.c:135: warning: passing argument 2 of 'CAN128_TxMsg' from 
incompatible pointer type
Der Code dazu:

extern   void     CAN_Tx(
    CANMsg_t   *msg)      /*FPI Message Object            */  
{
   CAN128_TxMsg(CAN_TxNum, &msg);
  
}

Was heisst das konkret? Kann ich das ignorieren, oder könnten dadurch 
Problem entstehen?

Danke schon mal für die Antworten.
Gruss
Patrick

[Bernhard M. (boregard)]

bist Du sicher, daß Du an CAN128_TxMsg den pointer auf den pointer msg 
übergeben willst, oder müsste es richtig so sein:

CAN128_TxMsg(CAN_TxNum, msg);

denn msg ist ja bereits ein Pointer auf eine Instanz von CANMsg_t...

[Patrick (Gast)]

Wenn ich es so eingebe:

CAN128_TxMsg(CAN_TxNum, msg);

bekomm ich die Fehlermeldung:
../can.c:135: error: incompatible type for argument 2 of 'CAN128_TxMsg'

[Hans-jürgen H. (hjherbert)]

Such den Prototypen von CAN128_TxMsg in deinen Header-Dateien.

[Fred S. (Gast)]

Hallo Patrick,

wie Bernhard und Hans-Jürgen schon geschrieben haben, hast Du hier noch 
nicht klargegelgt, was Du tatsächlich übergeben willst und wie der 
Prototyp Deiner Can128_TxMsg Funktion aussieht. Ich gehe mal davon aus, 
dass Du formell und inhaltlich korrekt vorgegangen bist. Dann könnte ein 
Typecast bei der Übergabe genügen, um die Warnung verschwinden zu 
lassen: Can128_TxMsg(CAN_TcNum, (CANMsg_t*)&msg);

Gruß

Fred

[Patrick (Gast)]

Wenn ich es so eingebe:

  CAN128_TxMsg(CAN_TxNum, *msg);

verschwindet die Meldung.

Der Prototyp ist so angegeben:

extern uint8_t CAN128_TxMsg (uint8_t num, CANMsg_t msg);

[Hans-jürgen H. (hjherbert)]

So passt der Aufruf zu dem was die Funktion will: Die ganze Message als 
Übergabeparameter. Auch wenn dann 8 bis 10 Bytes in den Stack oder in 
viele Register geladen werden müssen.

Hätte ich so nicht programmiert.

[Patrick (Gast)]

>Hätte ich so nicht programmiert.
Wie denn?

[Hans-jürgen H. (hjherbert)]

Das Unterprogramm: Ich hätte es so programmiert, dass es wirklich einen 
Pointer will, und nicht alle Bytes der Message im Stack.

Ich denke das Unterprogramm ist zugekauft. Dann benutze es so wie es ist 
und gib ihm was es will. (sonst musst du alle anderen Aufrufe auch 
ändern)