DS18S20 auslesen (finde den fehler nicht)

Tobias N. schrieb:
> Hier mal mein C Code.
> http://nopaste.info/b485621093.html

Funktioniert bei dir die Funktion für Anhänge nicht?

Tobias N. schrieb:
> MCU ist der ATMega8 mit 1Mhz internem Takt.

Hast du das auch dem Compiler gesagt?

Tobias N. schrieb:
> F_CPU 1000000
>
> so stehts inner Makefile,
schön und was ist in den Fuses eingestellt? externer Quarz? Welcher 
Wert?

Noch ein Tipp!

Optimierung -OS einstellen sonst stimmt das ganze delay gedöns nicht.

Und noch ein Tipp:

nimm gleich einen DS18B20 !

Ersetz mal die Grundfunktionen mit dem hier.
Formatierung musst halt noch anpassen und sag bescheid ob es nun geht 
oder nicht.

1

unsigned char ow_reset(void)

2

{

3

  uint8_t err=100;

4

  DDRC |= (1<<PC5);                        // define as ouput

5

6

  _delay_us(480); // leave it low for 480us

7

  DDRC &= ~(1<<PC5);

8

9

  _delay_us(60); // wait for presence

10

  err = (PINC & (1<<PC5)) >> PC5; // get presence signal

11

12

  if( err == 0 ){            // short circuit --> err=2

13

    err = 2;

14

    }

15

16

  _delay_us(480); // wait for end of timeslot

17

    return err;

18

} // 0=presence, 1 = no part

19

20

21

void write_bit(char bitval)

22

{

23

  DDRC |= (1<<PC5);                    // define as ouput

24

  _delay_us(8);

25

  if(bitval==1)

26

  {

27

     DDRC &= ~(1<<PC5);                // define as input

28

  }

29

  _delay_us(120);

30

  DDRC &= ~(1<<PC5);                // define as input

31

32

}// Delay provides 16us per loop, plus 24us. Therefore delay(5) = 104us

33

34

35

36

unsigned char read_bit(void)

37

{

38

unsigned char i;

39

  DDRC |= (1<<PC5); // pull DQ low to start timeslot

40

  DDRC &= ~(1<<PC5); // then return high

41

  _delay_us(10);

42

  i = (PINC & (1<<PC5)) >> PC5));

43

  _delay_us(105);

44

  return i;    // return value of DQ line

45

}

46

47

48

unsigned char read_byte(void)

49

{

50

  unsigned char i;

51

  unsigned char value = 0;

52

  for (i=0;i<8;i++)

53

  {

54

     if(read_bit()) value|=0x01<<i; // reads byte in, one byte at a time and then

55

   // shifts it left

56

57

  }

58

    return(value);

59

}

60

61

62

void write_byte(char val)

63

{

64

  unsigned char i;

65

66

  for (i=0; i<8; i++) // writes byte, one bit at a time

67

  {

68

     val &= 0x01; // copy that bit to temp

69

     write_bit(val); // write bit in temp into

70

     val = val >> i; // shifts val right 'i' spaces

71

  }

72

73

}

hier findest du alles zum onwire schön erklärt sogar mit Doku in 
Deutsch.
Beitrag "1-Wire-Bus-Projekt: DS1820  DS18S20  DS2450  DS2408  unter C und 8051"

So noch ein versuch und ersetz mal das hier!
Da du ein Oszi hast kannst du ja mal nachsehen wie die Bits über den Bus 
huschen.
Wichtig! Nach ow_reset(void) solltest den presence signal erkennen
und danach sollten die Daten zusehen sein.

void Read_Temperature(void)
{
char get[10];
char temp_lsb,temp_msb;
int k;
char temp_f,temp_c;
ow_reset();
write_byte(0xCC); //Skip ROM
write_byte(0x44); // Start Conversion

while( read_byte == 0);//_delay_us(104);


ow_reset();
write_byte(0xCC); // Skip ROM
write_byte(0xBE); // Read Scratch Pad
for (k=0;k<9;k++){get[k]=read_byte();}
// printf("n ScratchPAD DATA = 
%X%X%X%X%Xn",get[8],get[7],get[6],get[5],get[4],get[3],get[2],get[1],get 
[0]);
temp_msb = get[1]; // Sign byte + lsbit
temp_lsb = get[0]; // Temp data plus lsb
if (temp_msb <= 0x80){temp_lsb = (temp_lsb/2);} // shift to get whole 
degree
temp_msb = temp_msb & 0x80; // mask all but the sign bit
if (temp_msb >= 0x80) {temp_lsb = (~temp_lsb)+1;} // twos complement
if (temp_msb >= 0x80) {temp_lsb = (temp_lsb/2);}// shift to get whole 
degree
if (temp_msb >= 0x80) {temp_lsb = ((-1)*temp_lsb);} // add sign bit
//printf( "nTempC= %d degrees Cn", (int)temp_lsb ); // print temp. C
temp_c = temp_lsb; // ready for conversion to Fahrenheit
temp_f = (((int)temp_c)* 9)/5 + 32;
//printf( "nTempF= %d degrees Fn", (int)temp_f ); // print temp. F
}

Ups kleiner fehler eingeschlichen

while( read_bit == 0);//_delay_us(104);

sollte es heissen

Hier noch ein Beispiel was man auf dem Oszi sehen müsste wenn der 
Romcode
ausgelesen wird.  Siehe Anhang!

@Tobias

Ja! Hab ich mit Ultrascope aufgezeichnet.
Die Bilder habe ich damals mit Drawplus X2 von Serif bearbeitet.

Wie ich aber die Aufzeichnung dann aber gemacht habe, da muss ich erst
nochmal darüber Nachdenken.
Hab damals auch eine weile gebraucht bis ich mit meinem Rigol zurecht 
kam.

Da ich gerade auch wieder mit dem DS18S20 beschäftige werd ich die
die Aufzeichnungen nocheinmal replizieren.

@Tobias

Habe doch glatt falsche Bilder in der PDF gehabt!
Hier nochmal Überarbeitet.

Zu den Einstellungen am Oszi!
Bei mir Rigol DS1202CA folgendes einstellen:


Trigger Menu

MODUS  --> RAND
Quelle --> CH1  Messtaster
Anstieg --> Steigend
Sweep -->  EINZ   ( Einelmessung )

Timbase 100µs einfach mal am Drehknopf Scale drehen!

Wenn du alles Angeschlossen und Eingestellt hast die RUN/STOP drücken.

Dein Programm sollte so Programmiert sein das Du einen Taster drückst
an deinem Aufbau. Dann sollte ROMCODE eingelesen werden.
Die RUN/STOP Taste wird dann ROT.
Jetzt kannst du mit dem Horizontal Poti die Position nach rechts
oder links abfahren.

Bei Verwendung des Ultrascops Treiber installieren.
Den DSO mit dem PC über USB verbinden und eine Verbindung herstellen.

Im Fenster Upload/ Download ganz unten Virual Panel aufrufen.
Dann sollt im DSO rechts oben rmt stehen. D.h. Du kannst das DSO
nur noch Virtuell Bedienen. Alle Funktionen am DSO sind gesperrt.
Im Vrtual Panel kannst nun schön Bildchen machen und Exportieren.

Im Anhang ist noch ein Testprogramm das aber noch anpassen musst.

DDRC &= ~(1<<PC5);   // 1-wire Leitung auf LOW (Eingang)

fehlt noch in main.

Tobias N. schrieb:
> Ich habe mir jetzt mal den Code im anhnag zusammen gebastelt. Bei dem
> Reset mit den US erhalte ich eine 0, bei dem anderen eine 1, also mir
> wird jetzt im wechsel auf meinem LCD eine 0 und dann wieder eine 1 usw
> angezeigt. Das sagt mir doch jetzt der Reset funktioniert oder?
>
> Dann kann ich mich nämlich weiter daran machen :)

Nein!

Wenn der Sensor eingesteckt ist und der Presence Impuls erkannt wird 
sollte
auf deinem LCD ein Ok erscheinen andernfalls Error.
Zum Test einfach mal raus ziehen dann sollte immer Error erscheinen.

Auserdem würde ich an deiner Stelle in ow_reset den error direkt an 
einem
Port ausgeben wo eine LED angeschlossen ist. Damit hast schonmal eine 
Kontrolle ohne LCD Routinen.

Tobias N. schrieb:
> im wechsel auf meinem LCD eine 0 und dann wieder eine 1 usw
> angezeigt.

Ich verstehe nicht was du damit meinst.

Die diese Aussage von dir bei einem angeschlossenen DS18S20 ist einfach 
falsch.

Bei angeschlossenen DS18S20 sollte eine 0 = OK dauerhaft Angezeigt 
werden
und erst 1 = error wenn derselbe Sensor nicht mehr angeschlossen ist.

Also nichts im wechsel oder sowas! jetzt klar!

Ach ja muss mich noch bei dir Entschuldigen! Die Code-Beispiele von mir
sind ja für AVR. Das du mit PIC Arbeitest hatte ich Total ignoriert.
Bei meinen Routinen wird der Eingang über das Datenrichtungs Register
gesteuert. Ob es sowas im PIC auch gibt entzieht sich meiner Kenntnis.

Die delays in deinem Code scheinen nicht ganz zu stimmen, da muss man
schon ziemlich genau an das Timing halten wie es im Datenblatt steht.

Er hat einen Atmeta 8 und stellt sich auch Megadusselig an.

Wie man mit Internet und Unterstützung das nich zum Laufen bringen kann 
ist mir schleierhaft.

Problemlösungskompetenz tendiert gegen 0.

Maxim schrieb:
>Problemlösungskompetenz tendiert gegen 0.

Alle Achtung! Solche Politikerphrasen schon am frühen Morgen -
da tun mir gleich wieder die Knie weh....

Buna-Pelzer

Kennst du den Unterschied zwischen delay_ms und delay_us?
Auch die Bedeutung der Parameter?

Wie bin ich blos auf PIC gekommen.


Tobias N. schrieb:
> das timing bei
> ow_reset() stimmt also der ds18s20 antwortet

So gesehen JA!

Schmeiss mal diese Code Zeilen
PORTC |= (1<<PC5) und das PORTC &= ~(1<<PC5);  raus

Es reicht völlig mit DDRC zuarbeiten.
PULL-UP Ein- und Auschalterei ist völlig überflüssig.


Und mach das hier endlich in main rein.
 PORTC = 0x00;        // Pull_UP aus
 DDRC &= ~(1<<PC5);   // 1-wire Leitung auf LOW (Eingang)

Ach nochwas!

Wie hast Du deinen DS18S20 angeschlossen? Sowie im Anhang?

Falls Du den 4,7k Widerstand nicht angeschlossen hast funktioniert
die Sache auch nicht.

Das Ruhe Signal auf dem Bus ist ja High-Signal.
Daten werden werden durch herunter ziehen der DQ Leitung übertragen.

Lange Impulse sind Nullen kurze Impulse Einsen und die Zeiten was
ne 0 ist oder 1 findest du im Datenblatt bzw, schön in der oben
Verlinkten Doku in Deutsch. Musst halt durch Arbeiten.

Tobias N. schrieb:
> Hat keiner eine Idee? Ich verzweifel langsam ein wenig.

Anregungen gab es doch genug.
Wie wäre es wenn du mal von deinem Hardware Aufbau ein paar Bilder 
machst
im PNG format.

Irgendwie scheinst alles zu Ignorieren!

@Tobias
Ähh wo genau ist der Sensor angeschlossen? An der blauen Klemme?

Hat Dir niemand systematisches Arbeiten, oder systematische Fehlersuche 
beigebracht?

Das mit der mangelnden Lösungskompetenz finde ich schon gut auf den 
Punkt gebracht!

Tobias N. schrieb:
> der
> DS18S20 wird extern betrieben und zwischen vcc und DQ ist ein 4,7k

Die Aussage hat mich jetzt verwirrt.
Ich hätte da jetzt angenommen, das der Senor Fernab der Platine irgendwo 
ist.

Aber nun gut das Du selber den Fehler gefunden hast.

Tobias N. schrieb:
> er funktioniert jetzt aber nur "volle" grad zahlen.

Ist schon mal die halbe Miete.
Der Sensor macht im Normalbetrieb 0,5°C Schritte.
Mit dem Sensor geht auch 0,1 °C Schritte.

Klaus schrieb:
> Mit dem Sensor geht auch 0,1 °C Schritte.

Wenn dann wohl eher Schritte von
0.5°C, 0.25°C, 0.125°C oder 0.0625°C

> Wirrer Aufbau, wirre Vorgehensweise. Da würd ich auch nix gebacken
> bekommen. Alles schön auf einem Steckbrett wäre wesentlich
> übersichtlicher als alles über 3m Strippen zu verbinden. Aber das kommt
> halt auch davon dass man statt eines nackten µC lieber auf integrierte
> Boards setzt.


wirrer Satzbau, Groß- Kleinschreibung ist auch unbekannt...

> kann der auch sowas wie 23,52Grad usw?

Ich halte mir den Bauch vor Lachen.........

Tobias N. schrieb:
> kann der auch sowas wie 23,52Grad usw? naja muss halt nur rausfinden
> wie.

Ne ne so war es nicht gemeint.

Normal misst er 0, 0,5, 1,0, 1,5 usw.
Höhere Auflösung  0,0,  0,1, 0,2, 0,3 oder von mir aus 27,3 usw.

Ich finde fast in jeder Zeile Verbesserungspotential:

Der gröbste Bug:
void ow_wr_bit(unsigned char bitwert)
{
    // Start Time Slot: DQ auf Low
  OWDDR |= (1<<OWICPIN);   // 1-wire Leitung auf LOW (Ausgang)
  OWPORT &= ~(1<<OWICPIN); // DQ auf Low

  // Bei logŽ1Ž: sofort wieder auf High = nur kurzer Low-Impuls
    if(bitwert)
  {
    OWPORT |= (1<<OWICPIN); // DQ auf High
  }

    _delay_us(105);                   // ca. 105 us warten bis

    OWPORT |= (1<<OWICPIN); // DQ auf High
}

Es fehlt zwei _delay_us(8)! Mindestdauer des L- und H-Pegels.


Und das bitweise Rausschieben geht auch eleganter:
void ow_wr_byte(unsigned char dat)
{
    unsigned char i;
    unsigned char maske = 1;

    // 8 Bits nacheinander raus schieben, LSB zuerst
    for (i=0; i<8; i++)
    {
        if (dat & maske) ow_wr_bit(1);          // log.Ž1Ž senden
        else ow_wr_bit(0);                      // log.Ž0Ž senden
        maske = maske * 2;                      // nächstes Bit 
selektieren
    }
}

//diese Zeile einmal am Anfang
  OWPORT &= ~(1<<OWICPIN); // DQ auf Low   geschaltet wird mit DDR !

void ow_wr_byte(unsigned char data){
  uint8_t mask = 1;
  do{
    OWDDR |= (1<<OWICPIN); // 1-wire Leitung auf LOW (Ausgang)
    _delay_us(8); //Mindestdauer für L-Impuls

    if (data & mask == 0) _delay_us(105); //wenn 0, lange warten
    OWDDR &=~(1<<OWICPIN); // DQ auf High

    _delay_us(xx);//Mindestdauer für H-Impuls
    }
  while ( mask *= 2 ); //mask ist Maske und Bitzähler zugleich
  }

Tut mir leid, Deine Beratungsresistenz ist wirklich außerordentlich.
Du darfts nicht die Daten hart an OWPORT ausgeben, weil sonst zwei 
Ausgänge gegeneinander treiben. Der OW-Bus ist wired-or, d.h. Pins 
dürfen nur nach L gezogen werden, nicht nach H (was Du machst).


> kann der auch sowas wie 23,52Grad usw?
Der Vorgänger DS1820 und andere  konnten das, er hatte eine 
(temperaturabhängige) Auflösung von etwa 0.01°.
Aber Vorsicht, manche ICs sind mit DS1820 beschriftet und sind in 
Wirklichkeit DS18S20 (z.B. Pollin 180014 (2,50)).

Wenn Du das Scratchpad auslesen kannst, hier kompakter Code zum 
Ausgeben:
Beitrag "Re: DS18S20 - extended resolution bei Temperaturen um 0°C"


> kann der auch sowas wie 23,52Grad usw?

fast, der Wert nach 23,50 ist 23,56
Beitrag "Re: DS18S20 - extended resolution bei Temperaturen um 0°C"

>Aber Vorsicht, manche ICs sind mit DS1820 beschriftet und sind in
>Wirklichkeit DS18S20 (z.B. Pollin 180014 (2,50)).

daher sollte man mal den product code auslesen um zu sehen was für ein 
typ das ist...

ds18b20 ist zu den älteren NICHT CODE KOMPATIBEL!!!!!

Auch das steht im Datasheet.

Mach mal nen Checker Lehrgang, Tobi !