Erste Schritte mit TWI

[Drazor (Gast)]

Guten Abend,

ich bin gerade dabei einen "Termalscanner" zu bauen, der ein Wärmebild 
von einem Bereich erzeugt. Die Sensorsteuerung ist soweit fertig.
Irgendwie komme ich mit dem Infrarotsensor aber nicht weiter. Ich hoffe, 
ihr könnt mir da weiter helfen. Es handelt sich um den MLX90614.
Vielleicht hat jemand sogar schon mit dem Sensor gearbeitet.
Zwar findet man jede menge Quellcode zu dem Sensor, aber ich kann nichts 
davon 1zu1 adaptieren, da meine Zielplattform kein Arduino ist.
Nutzen kann ich diese library:
https://github.com/ethersex/ethersex/blob/master/hardware/i2c/master/i2c_master.c
https://github.com/ethersex/ethersex/blob/master/hardware/i2c/master/i2c_master.h
https://github.com/ethersex/ethersex/blob/master/hardware/i2c/master/i2c_generic.c
https://github.com/ethersex/ethersex/blob/master/hardware/i2c/master/i2c_generic.h

#define ADDRESS    0x5A

int16_t i2c_MLX90614_read_temperature() 
{
  int16_t data = 0;
  uint8_t data_4lsb, data_4msb;
  
  // Start
  if (!i2c_master_select(ADDRESS, TW_WRITE))  { return 0x00; }
  
  // Wait // ? hier nötig?
  if (i2c_master_transmit_with_ack() != TW_MT_DATA_ACK)  { return 0x00; }
  

  // Write
  TWDR = 0x07;
  if (i2c_master_transmit_with_ack() != TW_MR_DATA_ACK) { return 0x00; }
  

  /* select the slave in read mode */
  TWDR = (chipaddress << 1) | TW_READ;
  
  // Rep Start
  if (i2c_master_start() != TW_REP_START) { return 0x00; }
  if (i2c_master_transmit_with_ack() != TW_MR_DATA_ACK)  { return 0x00; }
  
  data_4lsb = TWDR; // least significant bit

  if (i2c_master_transmit() != TW_MR_DATA_NACK) { return 0x00; }
  
  data_4msb = TWDR; // most significant bit
  if (i2c_master_transmit() != TW_MR_DATA_NACK) { return 0x00; }
  
  data |= data_4msb;
  data |= data << 8;
  data |= data_4lsb;
  
  i2c_master_stop();
  return data;
}

PS: Ich würde mein Projekt auch hier vorstellen, wenn interesse besteht 
:)

[Drazor (Gast)]

Ich habe das Datenblatt auch mal angehängt.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Du solltest dir etwas bessere Fehlercodes ausdenken, so das du auch 
mitkriegst, wo es hakt. Im Moment wirst du wahrscheinlich nur ne 0x00 
zurückkriegen und rätseln, was denn nun schiefläuft. Nützlich ist z.B. 
ein globales Errorbyte, indem du eine Fehlernummer speicherst. Sollte 
deine Routine also Null zurückgeben, lässt du dir die Fehlernummer 
zeigen. Das kann auch für andere Unterroutinen praktisch werden.

Ich kenne den Sensor nicht, könnte mir aber vorstellen, das er 
kalibriert werden muss, bevor es sinnvolle Daten gibt.

[Drazor (Gast)]

Ich habe mir mal dein Tipp zu Herzen genommen.

Zuerst einmal:
Der Sensor wir schonmal gefunden:

i2c detect
detected at: 0x0 (0)
detected at: 0x5a (90)

Ich poste erstmal den ersten Teil des Codes, welcher schon nicht richtig 
arbeitet:

  uint8_t data_4lsb, data_4msb, data_crc;
  int8_t data;

  uint8_t address = 0x5A;

  
  #ifdef DEBUG_I2C
    debug_printf("I2C::MLX90614 start\n");
  #endif
  
  /////////////////////////
  // Start Condition
  /////////////////////////
  if (!i2c_master_select(address, TW_WRITE))
  {
    #ifdef DEBUG_I2C
      debug_printf("I2C::MLX90614 address(%d) not selectable\n", address);
    #endif

    goto end;
  }
  
  if (i2c_master_transmit_with_ack() != TW_MT_SLA_ACK || i2c_master_transmit_with_ack() != TW_MT_DATA_ACK)
  {
    #ifdef DEBUG_I2C
      debug_printf("I2C::MLX90614 address selected, no sla_ack\n");
    #endif

    //goto end;
  }
  
  // Read out
  TWDR = 0x07;
  
  if (i2c_master_transmit_with_ack() != TW_MT_SLA_ACK || i2c_master_transmit_with_ack() != TW_MT_DATA_ACK)
  {
    #ifdef DEBUG_I2C
      debug_printf("I2C::MLX90614 command, no data_ack\n");
    #endif

    goto end;
  }

Als Ausgabe bekomme ich:

D: I2C::MLX90614 start
D: i2c master select adr+mode 0xB4
D: I2C::MLX90614 address selected, no sla_ack
D: I2C::MLX90614 command, no data_ack

Laut Datenblatt sollte ich 2 Acks bekommen. Diese bleiben aber aus.
Wundert euch nicht, warum ich nach SLA und DATA ACK abfrage. Ich wollte 
nur auf Nummer sicher gehen.
Hat da jemand eine idee?

[mgue (Gast)]

Ich nehme mal an, dass du etwas schreiben (TW_WRITE) und nicht lesen 
(TW_READ) willst.

Mach aus jedem

if (i2c_master_transmit_with_ack() != TW_MT_SLA_ACK || i2c_master_transmit_with_ack() != TW_MT_DATA_ACK)

ein

if (i2c_master_transmit_with_ack() != TW_MT_DATA_ACK)

bzw.

ret = i2c_master_transmit_with_ack();
if (ret != TW_MT_DATA_ACK)

TW_MT_SLA_ACK ist hier nicht nötig (siehe weiter unten).

---

if (i2c_master_transmit_with_ack() != TW_MT_SLA_ACK || i2c_master_transmit_with_ack() != TW_MT_DATA_ACK)
  {
    #ifdef DEBUG_I2C
      debug_printf("I2C::MLX90614 address selected, no sla_ack\n");
    #endif

    //goto end;
  }

Dieser Part ist falsch. Es steht zu diesem Zeitpunkt nichts (sinnvolles) 
im TWDR Register.
Auf TW_MT_SLA_ACK wird bereits in i2c_master_select geprüft:

https://github.com/ethersex/ethersex/blob/master/hardware/i2c/master/i2c_master.c#L94

Zur Vollständigkeit: Bei

end:

sollte stehen:

i2c_master_stop();