Bit-Schiebeoperationen, mehrere 8-bit-Typen auf einen 32 bit-Typ

Gerhard Gast #1865302
Lesenswert?
• ▲
▼ 
Folgendes Problem:

Ich bekomme ein Array mit 5 Bytes, in denen ein 24-Bit-Wert versteckt 
ist. Und zwar so:

spi_buf[0] = X  X  X  X  X  X  X  X
spi_buf[1] = X  X  X  X  X  X  23 22
spi_buf[1] = 21 20 19 18 17 16 15 14
spi_buf[2] = 13 12 11 10 09 08 07 06
spi_buf[3] = 05 04 03 02 01 00 X  X

Die Zahlen geben dabei die Bit-Positionen an, MSB first...

Jetzt hätte ich gerne eine Funktion, die mir auf einem Atmel AVR mit 
AVR-GCC aus diesem (globalen) Array den 24-bit-Wert in einem uint32_t 
zurückgibt.

Folgendes habe ich probiert:

1
uint32_t get_temp (void)

2
{

3
  return (((spi_buf[1] & 0b00000011) <<  22) |

4


5
          ( spi_buf[2]               <<  14) |

6


7
          ( spi_buf[3]               <<  6)  |

8


9
          ((spi_buf[4] & 0b11111100) >>  2));

10


11
}


Leider funktioniert das nicht. Ich bekomme eine Compiler-Warnung, dass 
ich weiter schiebe, als der Typ breit ist. Außerdem scheinen die Daten 
korrumpiert zu sein.

Was mache ich falsch?
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden
Gerhard Gast #1865313
Lesenswert?
• ▲
▼ 
Jo, läuft... Fehlermeldung ist weg. Ob die Daten jetzt passen, prüfe ich 
gleich mal.

Aber warum? spi_buf ist ein Array aus uint8_t... Wenn der Compiler also 
den Typ der zu schiebenden Variable als Basis nimmt und nicht die Breite 
der fertig geschobenen Variable, dann sollte es bei den anderen Zeilen 
ja auch Probleme geben, oder nicht?!
Antwort Zitieren Bearbeiten Löschen Beitrag melden
AVR-Frickler Gast #1865364
Lesenswert?
• ▲
▼ 
Der Vorschlag von Roland funktioniert weil er erst einen cast nach 
uint32_t macht und dann verschiebt daher ist die Compiler-Warnung weg.

ob das guter Programmierstil sei mal dahin gestellt, ich würde es 
entweder wie folgt machen:

1
uint32_t get_temp (void)

2
{

3
    uint32_t  temp = 0;

4


5
    temp |= ( spi_buf[1] & 0b00000011);

6
    temp << 8;

7
    temp |= spi_buf[2];

8
    temp << 8;

9
    temp |= spi_buf[3];

10
    temp << 6;

11
    temp |= (( spi_buf[4] & 0b11111100) >>  2);

12


13
    return temp;

14
}


oder man macht es mit einem union.

MfG
AVR-Frickler
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Thread beobachten

Antwort schreiben

Hinweis: der ursprüngliche Beitrag ist mehr als 6 Monate alt. Bitte hier nur auf die ursprüngliche Frage antworten, für neue Fragen einen neuen Beitrag erstellen.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Dein Name und deine E-Mail-Adresse

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.