Frage zur TWI Lib des ATXmega

[Malte _. (malte)]

Hallo,
ich verwende die TWI Lib von Atmel um einen I2C EEProm anzusprechen.
Leider bekomme ich bei einem Lesezugriff nach einem Schreibzugriff 
zunächst die vorherigen Daten geliefert. Erst ein zweiter Lesezugriff 
liefert dann die korrekten Daten. Mit einem Ozi habe ich mir den ersten 
Lesezugriff auf dem I2C Bus angesehen. Hier werden bereits die korrekten 
neuen Daten übertragen. Wo liegt der Fehler?
Die TWI Lib gibt es hier:
http://www.atmel.com/dyn/resources/prod_documents/AVR1308.zip

#include <avr/io.h>

#include "i2ceeprom.h"
#include "main.h"
#include "twi_master_driver.h"
#include "rs232.h"

#define TWI_PWRSAVEREG PR_PRPE
#define TWI_PWRSAVEBIT PR_TWI_bm
#define TWI TWIE
#define TWI_IO PORTE
#define TWI_DATA_PIN PIN1CTRL
#define TWI_CLK_PIN PIN0CTRL

//eeprom allows maximum 100kHz at 1.7V
#define TWI_BAUDRATE 80000
#define TWI_BAUDSETTING TWI_BAUD(F_CPU, TWI_BAUDRATE)
//fixed 1010 + A2 = VCC, A1, A0 = GND -> 1010100 -> 0x54
#define TWI_SLAVE_ADDRESS 0x54

TWI_Master_t g_twiMaster;

uint8_t i2ceep_init(void) {
  TWI_PWRSAVEREG &= ~PR_TWI_bm;
  TWI_IO.TWI_DATA_PIN = PORT_OPC_PULLUP_gc;
  TWI_IO.TWI_CLK_PIN = PORT_OPC_PULLUP_gc;
  TWI_MasterInit(&g_twiMaster, &TWI, TWI_MASTER_INTLVL_LO_gc, TWI_BAUDSETTING);
  PMIC.CTRL |= PMIC_LOLVLEN_bm;

  while (TWI_MasterReady(&g_twiMaster) != TWIM_STATUS_READY);
  uint8_t buffer[3];
  buffer[0] = 0x0;
  buffer[1] = 0x0;
  buffer[2] = 0x0F;
  TWI_MasterWrite(&g_twiMaster, TWI_SLAVE_ADDRESS, buffer, 3);
  while (TWI_MasterReady(&g_twiMaster) != TWIM_STATUS_READY);
  buffer[2] = 0;


  TWI_MasterWriteRead(&g_twiMaster, TWI_SLAVE_ADDRESS, buffer, 2, 1);
  while (TWI_MasterReady(&g_twiMaster) != TWIM_STATUS_READY);
  if (g_twiMaster.readData[0] == 0x0F) {
    return 0;
  }
  return 1;
}

uint8_t i2ceep_writebyte(uint16_t address, uint8_t value) {
  uint8_t buffer[3];
  buffer[0] = address>>8;
  buffer[1] = (uint8_t)address;
  buffer[2] = value;
  while (TWI_MasterReady(&g_twiMaster) != TWIM_STATUS_READY);
  TWI_MasterWrite(&g_twiMaster, TWI_SLAVE_ADDRESS, buffer, 3);
  return 0;
}

uint8_t i2ceep_readbyte(uint16_t address) {
  uint8_t buffer[3];
  buffer[0] = address>>8;
  buffer[1] = (uint8_t)address;
  while (TWI_MasterReady(&g_twiMaster) != TWIM_STATUS_READY);
  TWI_MasterWriteRead(&g_twiMaster, TWI_SLAVE_ADDRESS, buffer, 2, 1);
  while (TWI_MasterReady(&g_twiMaster) != TWIM_STATUS_READY);
  rs232_puthex(g_twiMaster.readData[0]); //debug only
  return g_twiMaster.readData[0];
}

void i2ceep_test(uint8_t data) {
  //while (TWI_MasterReady(&g_twiMaster) != TWIM_STATUS_READY);
  TWI_MasterWrite(&g_twiMaster, TWI_SLAVE_ADDRESS, &data, 1);
}

void i2ceep_disable(void) {
  TWI_PWRSAVEREG |= PR_TWI_bm;
}

ISR(TWIE_TWIM_vect) {
  TWI_MasterInterruptHandler(&g_twiMaster);
}

[Malte _. (malte)]

Also einen Fehler in der Lib habe ich gefunden:

#define TWIM_STATUS_READY              0
#define TWIM_STATUS_BUSY               1

bool TWI_MasterReady(TWI_Master_t *twi)
{
  bool twi_status = (twi->status & TWIM_STATUS_READY);
  return twi_status;
}

ist natürlich immer gleich 0. Und somit äquivalent zu TWIM_STATUS_READY. 
D.h. es wurde nie auf den Master gewartet und dementsprechend der 
vorherige Wert aus dem Buffer gelesen.
Die Lib in

bool TWI_MasterReady(TWI_Master_t *twi)
{
  if (twi->status == TWIM_STATUS_READY) {
    return true;
  }
  return false;
}

ändern hilft. Außerdem fehlt noch irgendwo ein Waitstate, da lesen 
direkt folgend auf schreiben keine Daten liefert (außer man wartet 
dazwischen).
Aber so funktioniert das Lesen + Schreiben (einfach noch mal lesen wenn 
es nicht geklappt hat, anstelle des Wartens):

uint8_t i2ceep_init(void) {
  TWI_PWRSAVEREG &= ~PR_TWI_bm;
  TWI_IO.TWI_DATA_PIN = PORT_OPC_PULLUP_gc;
  TWI_IO.TWI_CLK_PIN = PORT_OPC_PULLUP_gc;
  TWI_MasterInit(&g_twiMaster, &TWI, TWI_MASTER_INTLVL_LO_gc, TWI_BAUDSETTING);
  PMIC.CTRL |= PMIC_LOLVLEN_bm;
  i2ceep_writebyte(0, 0xA);
  uint8_t value = i2ceep_readbyte(0);
  if (value == 0x0A) {
    return 0;
  }
  return 1;
}

uint8_t i2ceep_writebyte(uint16_t address, uint8_t value) {
  uint8_t buffer[3];
  buffer[0] = address>>8;
  buffer[1] = (uint8_t)address;
  buffer[2] = value;
  while (!TWI_MasterReady(&g_twiMaster));
  TWI_MasterWrite(&g_twiMaster, TWI_SLAVE_ADDRESS, buffer, 3);
  return 0;
}

uint8_t i2ceep_readbyte(uint16_t address) {
  uint8_t buffer[2];
  buffer[0] = address>>8;
  buffer[1] = (uint8_t)address;
  while (!TWI_MasterReady(&g_twiMaster));
  do {
    TWI_MasterWriteRead(&g_twiMaster, TWI_SLAVE_ADDRESS, buffer, 2, 1);
    while (!TWI_MasterReady(&g_twiMaster));
  } while (g_twiMaster.bytesRead < 1);
  return g_twiMaster.readData[0];
}