I2C Temperaturwert ausgeben

[TimeOut (Gast)]

Hallo Forum,

ich habe ein Programm bekommen, das per Polling zyklisch einen 
Temperaturwert aufs LCD Display ausgeben soll.
Das ganze passiert über einen IR-Sensor und den I2C Bus.

Das ganze funktioniert soweit, die Temperatur wird auf dem Display 
ausgegeben doch nur ein einziges Mal. Danach muss ich das Programm neu 
starten und er liest wieder einen neuen Wert.

Sprich: Das zyklische Auslesen der Temperatur funktioniert nicht.

Ich denke es hat etwas mit der Übertragung zu tun, (evtl setzt er ein 
Bit nicht richtig zurück?) hänge den Quellcode mal mit an

Danke

Gruß
TimeOut

double Transmission(void)
{

  double grad=0.0;
  vu8 d1=0;
  vu8 d2=0;
  double d1_double,d2_double;
  vu16 daten=0;
  
    
  /* Enable I2C1 ----------------------------------------------------*/
  I2C_Cmd(I2C1, ENABLE);



  /* I2C1 configuration: SMBus Host ------------------------------------------*/
  I2C_InitStructure.I2C_Mode = I2C_Mode_SMBusHost;
  I2C_InitStructure.I2C_DutyCycle = I2C_DutyCycle_2;
  I2C_InitStructure.I2C_OwnAddress1 = I2C1_SLAVE_ADDRESS7;
  I2C_InitStructure.I2C_Ack = I2C_Ack_Enable;
  I2C_InitStructure.I2C_AcknowledgedAddress = I2C_AcknowledgedAddress_7bit;
  I2C_InitStructure.I2C_ClockSpeed = ClockSpeed;
  I2C_Init(I2C1, &I2C_InitStructure);
        

  /* Enable I2C2 ARP */
  I2C_ARPCmd(I2C2, ENABLE);

  /* Enable I2C1 and I2C2 PEC Transmission */
  I2C_CalculatePEC(I2C1, ENABLE);
  I2C_CalculatePEC(I2C2, ENABLE);
  
  /*----- Transmission Phase -----*/
  /* Send I2C1 START condition */
  I2C_GenerateSTART(I2C1, ENABLE);     
  /* Test on I2C1 EV5 and clear it */
  while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_MODE_SELECT)); 
  /* Send Slave address */
  I2C_Send7bitAddress(I2C1, SMBusDefaultHeader, I2C_Direction_Transmitter);
  /* Test on I2C1 EV6 and clear it */
  while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_TRANSMITTER_MODE_SELECTED)); 

  /* Get I2C2 SMBDEFAULT flag status */
  Status = I2C_GetFlagStatus(I2C2, I2C_FLAG_SMBDEFAULT);  //wird evtl. nicht benötigt
   /* Send Command */
  I2C_SendData(I2C1, Command);  
  /* Clear ADDR flag */
  I2C_ClearFlag(I2C2, I2C_FLAG_ADDR);
 
   /* Send I2C1 START condition */
  I2C_GenerateSTART(I2C1, ENABLE); 
  /* Test on I2C1 EV5 and clear it */
  while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_MODE_SELECT)); 
  /* Send Slave address */ 
  I2C_Send7bitAddress(I2C1, SMBusDefaultHeader, I2C_Direction_Receiver); 
  /* Test on I2C1 EV6 and clear it */
  while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_RECEIVER_MODE_SELECTED)); 
  /* Wait for I2C1 received data */
  while(!I2C_GetFlagStatus(I2C1, I2C_FLAG_RXNE)); 
  /* Store received data on I2C1 */
  ReceivedCommand = I2C_ReceiveData(I2C1);     //LSB Byte 
  d1= ReceivedCommand;     //Zwischenspreicherung in vu8
  d1_double=(double)d1;     //Konvertierung in double ; Konvertierung in int funktioniert nicht
  
  /* Wait for I2C1 received data */
  while(!I2C_GetFlagStatus(I2C1, I2C_FLAG_RXNE));   
  /* Store received data on I2C1 */
  ReceivedCommand = I2C_ReceiveData(I2C1);       //MSB Byte
  d2=ReceivedCommand;
  d2_double=(double)d2;
  
  //16Bit unsigned char
  daten =d1;
  daten|=(d2<<8);
  
  //Umrechnung in Grad Celsius
  grad=d2_double*256+d1_double;
  grad/=50.0;
  grad-=273.15;
  
  /* Enable Transfer PEC next for I2C1 */
  I2C_TransmitPEC(I2C1, ENABLE);
  /* Wait for I2C1 received data */
  while(!I2C_GetFlagStatus(I2C1, I2C_FLAG_RXNE));  
  /* Store received PEC on I2C1 */
  PECValue = I2C_ReceiveData(I2C1);
  /* Test on I2C1 EV8 and clear it */
  while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_BYTE_RECEIVED));
  
  /* Send I2C1 STOP Condition */
  I2C_GenerateSTOP(I2C1, ENABLE);
  /* Test on I2C1 EV4 and clear it */
  while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_BYTE_RECEIVED )); 
  /* Clear I2C1 STOPF flag */
  I2C_ClearFlag(I2C1, I2C_FLAG_STOPF);

  return grad;
}

[Jörg S. (joerg-s)]

> Das ganze passiert über einen IR-Sensor und den I2C Bus.
Geht es etwas genauer? Der Sensor ist I2C, ich nehme mal an LM75?
Um welchen Controller Typ geht es eigentlich?

> Das ganze funktioniert soweit, die Temperatur wird auf dem Display
> ausgegeben doch nur ein einziges Mal. Danach muss ich das Programm neu
> starten und er liest wieder einen neuen Wert.
Bleibt das Programm hängen oder läuft es weiter, zeigt aber nichts an?
Und wenn es hängen bleibt, wo?

[TimeOut (Gast)]

Also der Sensor ist ein: MLX90614ESF-BAA
Controller: Cortex M3 (STM32F103ZE)


Soweit ich das sehe bleibt er immer bei:

 /* Test on I2C1 EV5 and clear it */
  while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_MODE_SELECT));

und

 /* Test on I2C1 EV6 and clear it */
  while(!I2C_CheckEvent(I2C1, 
I2C_EVENT_MASTER_TRANSMITTER_MODE_SELECTED));

länger hängen, wenn ich es step-by step durchlaufe.

Allerdings wird der nachfolgende Code trotzdem später abgearbeitet, wenn 
ich ein Breakpoint danach setzte.

[Karl H. (kbuchegg)]

Erst mal feststellen wer der Übeltäter ist:
Der Temperatur-Sensor oder das LCD.

Es ist nie schlau, zuviel Funktionalität in eine Funktion zu packen.
Mach dir 2 Funktionen
die eine, die einen Wert vom LM75 holt
und die andere, die einen Wert an das LCD ausgibt.


Und dann testest du erst mal, ob die Ausgabefunktion ein Problem hat. Zb 
in dem du in einer Schleife (oder was halt dann ausserhalb von 
Transmission steht) immer wieder die Ausgabefunktion mit einem 
Zahlenwert aufrufst, der zb hochgezählt wird. Wenn das klappt, dann 
weißt du, dass es wahrscheinlich nicht das LCD ist. WEnn es nicht 
klappt, dann weißt du, dass du ein Problem mit dem LCD hast.

Funktioniert das LCD, dann nimmst du den Sensor dazu.
In der Schleife wird die FUnktion aufgerufen, die den Sensor ausliest 
und der WErt, den die Funktion liefert wird der AUsgabefunktion 
übergeben (von der du schon weißt, das sie funktioniet - deswegen hast 
du die ja vorher getestet).


Keine 2-Fronten-Kriege anfangen! Daran sind schon ganz andere 
gescheitert. Je feiner und kleiner du eine AUfgabe unterteilen und auch 
testen kannst, umso besser.

[Karl H. (kbuchegg)]

Und wieso musst du eigentlich jedesmal das I2C Subsystem neu 
initialisieren?


Also 3 Funktionen
* I2C Initialisieren
* Wert vom LM75 holen
* Wert am LCD ausgeben

[TimeOut (Gast)]

Hallo Karl Heinz,

vielen Dank für deine Tipps.

Also das LCD Display wird richtig angesteuert, das funktioniert soweit.

Probleme gibt es, wenn ich in der Schleife die Funktion (Transmission) 
aufrufe, die ja den Sensor ausliest

klappt das wieder genau 1x. Beim nächsten Schleifen durchlauf bleibt er 
dann an dieser Funktion stehen und hängt...

[TimeOut (Gast)]

Wenn ich das LCD teste und einen Wert ausgebe, den ich in einer Schleife 
hochzähle, geht das Display immer an und wieder aus und zeigt dann den 
neuen Wert an.

[Jörg S. (joerg-s)]

TimeOut schrieb:
> Allerdings wird der nachfolgende Code trotzdem später abgearbeitet, wenn
> ich ein Breakpoint danach setzte.

>Beim nächsten Schleifen durchlauf bleibt er dann an dieser Funktion
>stehen und hängt
?


Hast du ein (Speicher)Oszi zur Hand um zu schauen was auf dem Bus los 
ist?

[TimeOut (Gast)]

Also auf dem Bus kommt einmal das SDA Signal rein, danach ist Stille.

Mit hängenbleiben meine ich die Stelle:

/* Test on I2C1 EV5 and clear it */
while(!I2C_CheckEvent(I2C1, I2C_EVENT_MASTER_MODE_SELECT));

Habe mich mal im Internet schlau gemacht und das Problem sei wohl die 
fehlerhafte Slaveadressierung in der Library. Dort wird davon 
gesprochen, dass man die Slaveadresse um 1 Bit nach links verschieben 
muss.

Allerdings führt dies nicht zum erhofften Erfolg bei mir.

Also er muss ja einmal durch die komplette "Transmission"-Routine laufen 
sont würde ich keinen Wert auf meinem Display ausgegeben bekommen, beim 
zweiten Mal bleibt er dann in der oben genannten Schleife hängen...

PS:In der main rufe ich die "Transmission()" in einer Endlosschleife auf