Bitmaskierung

Gast #2020457
Lesenswert?
• ▲
▼
Register PHY_RSSI:
RX_CRC_VALID | RND_VALUE1 | RND_VALUE0 | RSSI4 | RSSI3 | RSSI2 | RSSI1 | 
RSSI0

Programmzeilen:
SigStr[Empf] = (PHY_RSSI &= 0b00011111);

spätere Wiederabe:

usart_puts(", RSSI=");
utoa(SigStr[Empf],s,10);
usart_puts(s);

Problem:
Die Ausgabe hat Werte die Größer sind als 31...

Stimmt meine Maskierung so nicht? Ich möchte nur die RSSIx auslesen und 
in die volatile SigStr[] schreiben...

Prozessor: Atmega128RFA1

was mache ich falsch?
Gast #2020481
Lesenswert?
• ▲
▼
aaahh.
Guter Hinweis.
Das Register ist ReadOnly.

Nun fehlt mir aber tatsächlich jede Idee, wie ich da vernünftig nur die 
Bits lesen kann, die mich interessieren.
Außer ich nehme dafür eine Zwischenvariable, also

volatile uint8_Zwischensp;

Zwischensp = PHY_RSSI
SigStr[Empf] = (Zwischensp & 0b00011111);


Gibt es eine schönere Lösung?
Moderator Persönliche Seite #2020510
Lesenswert?
• ▲
▼
Großes Fragezeichen schrieb:
> Gibt es eine schönere Lösung?

Oft ist die Zwischenvariable übrigens die "schönere Lösung", weil sie
besser lesbar werden kann.  Der Compiler hält die Werte am Ende
ohnehin nur in Registern (so die Optimierung aktiviert ist).

Allerdings würde ich für Variablen, deren "Lebenszeit" nur 3 Zeilen
beträgt, nicht so einen langen Namen wählen.

Antwort schreiben

Bitte melde dich an, um einen Beitrag zu schreiben.

oder

Mit Google-Account einloggen

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren