float zu 4x8Bit und zurück

[qwertz (Gast)]

Hallo,

ich messe mit dem ADC die Spannung eines Stromwandlers und berechne 
daraus einen Messwert. Dieser liegt mir als float Zahl vor.

Die float Zahl (vm) möchte ich via CAN-Nachricht verschicken und beim 
Empfänger wieder in eine float Zahl wandeln.

Das mache ich bisher so:

Senden:

uint8_t mm_data[4] = {0};
uint8_t *mm_data_ptr;
  
mm_data_ptr = mm_data;    
memcpy((void*)&mm_data, &vm, sizeof(float));
    
for(uint8_t i=0;i<4;i++)
{
msg.data[i] = *mm_data_ptr;
mm_data_ptr++;
}
can_send_message(&msg);

Empfangen:

float data;
memcpy(&data, can_dump_frame->data, sizeof(float));
printf("Value: %f A\n",data);

Ist es so richtig?
Oder würdet ihr es anders machen?
Bekomme irgendwie nicht das richtige angezeigt beim printf() ....


Grüße

[Johann L. (gjlayde)]

qwertz schrieb:
> Ist es so richtig?

Nur falls auf beiden Seiten float die gleiche Darstellung hat.

In dem Fall brauchst du auch kein mm_data sondern kannst die Adresse des 
float direkt in die Senderoutine (die wohl const void* nimmt) geben.

Ansonsten ist das Forum voll mit dem Thema, es wird gefühlt ca. 1x pro 
Woche durchgekaut.

[qwertz (Gast)]

> Nur falls auf beiden Seiten float die gleiche Darstellung hat.
ATmega88 <--> RaspberryPi (Linux)

> In dem Fall brauchst du auch kein mm_data sondern kannst die Adresse des
> float direkt in die Senderoutine (die wohl const void* nimmt) geben.

Senderoutine?

[Karl H. (kbuchegg)]

qwertz schrieb:

> uint8_t mm_data[4] = {0};
> uint8_t *mm_data_ptr;
>
> mm_data_ptr = mm_data;
> memcpy((void*)&mm_data, &vm, sizeof(float));

Da kopierst du ja schon die Bytes ins Array

> for(uint8_t i=0;i<4;i++)
> {
> msg.data[i] = *mm_data_ptr;
> mm_data_ptr++;
> }

wozu dann nochmal kopieren?

wozu eigentlich die extra Pointer Variable und das extra Array? Wenn in 
msg sowieso schon ein Byte Array enthalten ist, warum lässt du memcpy 
die Bytes nicht gleich dort hinein kopieren?

> Bekomme irgendwie nicht das richtige angezeigt beim printf() ....

Dann lass dir halt mal die Bytes ausgeben und sieh nach, ob die beim 
Sender und beim Empfänger identisch sind.

Wenn sie es sind, dann verwenden die beiden Parteien wohl 
unterschiedliche Floating Point Darstellungen. Muss nicht sein, kann 
aber sein. Der Bytevergleich macht dich sicher. Denn dann weißt du 
zumindest schon mal, dass auf der CAN Übertragung nichts passiert ist 
und der Empfänger genau das zu sehen kriegt, was der Sender auch 
weggeschickt hat.

[holger (Gast)]

>Die float Zahl (vm) möchte ich via CAN-Nachricht verschicken und beim
>Empfänger wieder in eine float Zahl wandeln.

Verschick eine Ganzzahl die deinen Strom in uA darstellt.
Bei einer 32Bit Zahl sollte das für alles ausreichen was du
so an Strom messen kannst.

Mit float hast du nur Ärger am Hals.

[qwertz (Gast)]

> Mit float hast du nur Ärger am Hals.

Okay, nun verpacke ich eine 32Bit zahl in den CAN_Frame. Das ganze wird 
zunächst in einen Ringpuffer geschrieben und später verschickt:

In Puffer:

memcpy((void*)&mm_buf[mm_buf_h], &vm, sizeof(uint32_t)); 
...

Senden:

for(tt; tt != th; tt = (tt + sizeof(uint32_t)) & (MM_BUFSIZE -1))
{
  msg.id=ID;    
  mm_buff_ptr = mm_buff + tt;
    
  for(i=0;i<4;i++)
  {
    msg.data[i] = *mm_buff_ptr;
    mm_buff_ptr++;
  }
  
  can_send_message(&msg);

Empfangen:

memcpy(&dat, can_dump_frame->data, sizeof(long));

printf("Value: %i A\n",dat);

Es erscheint der richtige Wert im Terminal.
Allerdings nur eine 4-Stellige Zahl.
Wo ist der Rest meiner 32 Bit geblieben?

[Karl H. (kbuchegg)]

%i und long passen nicht zueinander.

[qwertz (Gast)]

> %i und long passen nicht zueinander.
Was nutze ich dann für long?
Durch %li ändert sich nichts.


Ich würde es ja eigentlich gern als float Zahl auf dem Terminal 
ausgeben:

long dat;

memcpy(&dat, can_dump_frame->data, sizeof(long));
float data = dat / 1000;
printf("Value: %f A\n",data);


Natürlich funktioniert es so nicht :(
Was mache ich falsch?

[qwertz (Gast)]

okay - es klappt alles.

Danke für die Hilfe!

[holger (Gast)]

>okay - es klappt alles.
>
>Danke für die Hilfe!

Als Gegenleistung könntest du mal die Lösung posten.
Oder der Thread kann gleich komplett gelöscht werden.