DS18S20 Bytes auslesen

OP #2347025
Hallo,
ich habe für den DS18S20 eine Funktion geschrieben, die die einzelnen 
Bits ausließt. Diese funktionert auch (wenn ich die 16 mal 
hintereinander in einer Schleife ausführe und jedes mal den Ausgelesenen 
Bit über uart an mein Terminal sende stimmt die Bitfolge mit der 
Temperatur überein).

Nun wollte ich jeweils 8 bit ein eine vorzeichenlose integer Variable 
abspeichern:
1
bool ow_ReadBit(void);
2

3
unsigned int ow_ReadByte(void) {
4
  uint8_t buffer = 0;
5
  for (uint8_t i = 0; i<8; i++) {
6
    buffer |= ow_ReadBit();     //LS Bit speichern 
7
    for (int b = 0; b<7; b++) {
8
      buffer <<= 1;       //bitfolge nach links schieben
9
    }
10
  }
11
  return buffer;
12
}

Um zu sehen ob es funktioniert hat dient folgende Funktion:
1
void ow_binToBinChar(unsigned int bin) { //unsinnigen namen bitte ignorieren
2
  
3
  for (int i = 0; i<8; i++) {
4
    
5
    if ((bin & 1) == 1) {
6
      sendchar('1');
7
    } else if ((bin & 1) == 0){
8
      sendchar('0');
9
    }
10
    bin >>= 1;
11
  } 
12
}
13

14

15
// in der hauptfunktion steht:
16
ow_binToBinChar(ow_ReadByte());

Nun kommt im terminal aber immer 00000001 an. Erwartet habe ich jedoch 
10011000.

Könnt ihr mir einen Hinweis auf meinen Fehler geben? Ich komme da nicht 
drauf :(

LG
Paul
Hallo,

ich verweise auf dies hier:
1
    for (int b = 0; b<7; b++) {
2
      buffer <<= 1;       //bitfolge nach links schieben
3
    }

Klar? - nein dann schau mal hier:
1
unsigned char ow_ReadByte(void) {
2
  unsigned char buffer = 0;
3

4
  for (unsigned char i = 0; i < 8; i++) {
5
    buffer |= ow_ReadBit();     //LS Bit speichern 
6
    buffer <<= 1;       //bitfolge nach links schieben
7
  }
8
  return buffer;
9
}

Diese Funktion würde ich etwas kürzer schreiben:
1
void ow_binToBinChar(unsigned char bin) {
2
  
3
  for (unsigned char i = 0; i<8; i++) {
4
    sendchar((bin & 1) ? '1' : '0');
5
    bin >>= 1;
6
  }
7

8
}

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 15 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.