DS18S20 Bytes auslesen

[Paul S. (paulszymanski)]

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:

bool ow_ReadBit(void);

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

Um zu sehen ob es funktioniert hat dient folgende Funktion:

void ow_binToBinChar(unsigned int bin) { //unsinnigen namen bitte ignorieren
  
  for (int i = 0; i<8; i++) {
    
    if ((bin & 1) == 1) {
      sendchar('1');
    } else if ((bin & 1) == 0){
      sendchar('0');
    }
    bin >>= 1;
  } 
}


// in der hauptfunktion steht:
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

[Uwe (de0508)]

Hallo,

ich verweise auf dies hier:

    for (int b = 0; b<7; b++) {
      buffer <<= 1;       //bitfolge nach links schieben
    }

Klar? - nein dann schau mal hier:

unsigned char ow_ReadByte(void) {
  unsigned char buffer = 0;

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

Diese Funktion würde ich etwas kürzer schreiben:

void ow_binToBinChar(unsigned char bin) {
  
  for (unsigned char i = 0; i<8; i++) {
    sendchar((bin & 1) ? '1' : '0');
    bin >>= 1;
  }

}

[Paul S. (paulszymanski)]

Oh nein -.- Bin ich blind. kopf gegen die wand

Okay so funktioniert es natürlich auch.

Danke für den Hinweis! Danke auch für die kurzschreibweise!

Sorry für diesen Fail,
LG
Paul