RFM70 Library - Fehler im Code?

OP #3179672
Hallo,

nach einigen Experimenten mit dem RFM70 Funkmodul, habe ich mich ein 
bisschen genauer mit der Library hier auseinandergesetzt:
http://code.google.com/p/odobot1/source/browse/libraries/rfm70/rfm70.cpp

Ich vermute, dass der folgende Ausschnitt fehlerhaft ist. Kann das 
vielleicht jemand bestätigen?
1
void RFM70::configARD(uint8_t ard) {
2
  if (ard > 0x0f) return;
3
  uint8_t tmp = readRegVal(RFM70_REG_SETUP_RETR);
4

5
  tmp &= ((ard << 4) | 0x0F);
6

7
  writeRegVal(RFM70_CMD_WRITE_REG | RFM70_REG_SETUP_RETR, tmp);
8
}

Angenommen, der Ausgangswert im Register ist 0x00, dann würde die 
Funktion keinerlei Änderung bewirken, da auch der Endwert 0x00 wäre. 
Müsste der Code nicht folgendermaßen aussehen?
1
void RFM70::configARD(uint8_t ard) {
2
  if (ard > 0x0f) return;
3
  uint8_t tmp = readRegVal(RFM70_REG_SETUP_RETR);
4
  
5
  tmp &= 0x0F;
6
  tmp |= (ard << 4);
7

8
  writeRegVal(RFM70_CMD_WRITE_REG | RFM70_REG_SETUP_RETR, tmp);
9
}

Danke schonmal :-)

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 13 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

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.

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.