XMEGA & TWI -> Lese Problem

[Jan H. (janiiix3)]

Moin Moin,

habt ihr ne Idee, wieso ich vom Slave nichts empfange? Senden klappt..
Ist was an der Lese Routine falsch?

Es handelt sich um einen I2C Touch Controller (QT2120 von ATMEL)..
Habe einen Pin als GPIO konfiguriert und dort ne LED angeschlossen. 
Diese kann ich an und aus schalten. Die ID lesen von dem Chip schlägt 
jedoch fehl.

uint8_t twi_read(TWI_t *twi, uint8_t slave_addr, uint8_t reg)
{
  uint8_t ret=0;
  
//   twi->MASTER.ADDR = (slave_addr<<1);
//   while(!(twi->MASTER.STATUS&TWI_MASTER_WIF_bm));
//  
//   twi->MASTER.ADDR = reg;
//   while(!(twi->MASTER.STATUS&TWI_MASTER_WIF_bm));
//  
//  
//   twi->MASTER.ADDR = ((slave_addr<<1)|0x01);
//   while(!(twi->MASTER.STATUS & TWI_MASTER_RIF_bm));
//  
//   ret = twi->MASTER.DATA;
//  
//   twi->MASTER.CTRLC = TWI_MASTER_ACKACT_bm | TWI_MASTER_CMD_STOP_gc;


  twi->MASTER.ADDR = slave_addr<<1 & ~0x01;
  while (!(twi->MASTER.STATUS & TWI_MASTER_WIF_bm) || (twi->MASTER.STATUS & TWI_SLAVE_RXACK_bm));
  twi->MASTER.DATA = reg;
  while (!(twi->MASTER.STATUS & TWI_MASTER_WIF_bm));

  twi->MASTER.ADDR = slave_addr | 0x01;
  while (!(twi->MASTER.STATUS & TWI_MASTER_RIF_bm));
  ret = twi->MASTER.DATA;
  TWIC.MASTER.CTRLC = TWI_MASTER_CMD_STOP_gc;

  return ret;
}

[Bastian W. (jackfrost)]

Laut DB will der QT2120 nach senden der Slave und Mem. Adresse einen 
Stop und dann einen Start, und nicht einen weiteren Start.

Gruß JackFrost

[jim_quakenbush (Gast)]

Bastian W. schrieb:
> und nicht einen weiteren Start.

Nicht ganz:

6.   The host must then send a STOP and a START condition followed by 
the
     slave address again but this time accompanied by the READ bit.


Note:

     Alternatively, instead of step 6, a repeated START can be sent so 
the
     host does not need to relinquish control of the bus.

[Bastian W. (jackfrost)]

jim_quakenbush schrieb:
> Bastian W. schrieb:
>> und nicht einen weiteren Start.
>
> Nicht ganz:
>
> 6.   The host must then send a STOP and a START condition followed by
> the
>      slave address again but this time accompanied by the READ bit.
>
>
> Note:
>
>      Alternatively, instead of step 6, a repeated START can be sent so
> the
>      host does not need to relinquish control of the bus.


Ok die Anmerkung hab ich überlesen.
@TO

Kannst du nach dem lesen wieder schreiben , oder verlierst du danach die 
Abitration ?

Ich hab die TWI Register nicht im Kopf, hast du Automatik für ACK/NACK 
beim lesen an ? Hast du
Bei dem Byte das du gelesen hast ein NACK vor dem Stop gesendet ?

Gruß JackFrost

Hast du einen LA um zu schauen ob das alles richtig gesendet wird ?

[Fabian D. (Gast)]

Bastian W. schrieb:
> Kannst du nach dem lesen wieder schreiben , oder verlierst du danach die
> Abitration ?

Ich toggle die LED an einem der GPIOS hin und her. Und danach lese ich. 
Das Schreiben klappt.

Bastian W. schrieb:
> ch hab die TWI Register nicht im Kopf, hast du Automatik für ACK/NACK
> beim lesen an ? Hast du Bei dem Byte das du gelesen hast ein NACK vor
> dem Stop gesendet ?

Ich habe oben im auskommentierten Teil, zum Schluss einen NACK gesendet. 
Hat leider auch nichts gebracht.

[jim_quakenbush (Gast)]

Hast Du Pullups an SDA, SCL, CHANGE und RESET?

[Bastian W. (jackfrost)]

Fabian D. schrieb:
> Ich toggle die LED an einem der GPIOS hin und her. Und danach lese ich.
> Das Schreiben klappt.

Das geht also auch dauerhaft, also nach dem ersten Zyklus lesen geht das 
schreiben wieder ohne Probleme ?

Mit der Funktion lese ich bei einem Atxmega E Daten aus einem FRAM mit 
aus.

Page_adress ist die Slave ID und Adress ist die Addresse der 
Speicherzelle.

Der Block mit dem PortC ist noch nicht dynamisch, da ich da noch keine 
Arbeit reingesteckt hatte. Btw. das ist von einem C++ Projekt. Wenn du 
also nicht TWIC nutzt musst du da noch den Port ändern.

void twi_read(TWI_t *twiname, uint8_t *readData,uint8_t page_adress, uint8_t Adress, uint8_t bytes)
{
  twiname->MASTER.CTRLC &= ~((1<<TWI_MASTER_ACKACT_bp));
  twiname->MASTER.ADDR = page_adress;
  while (!(twiname->MASTER.STATUS & TWI_MASTER_WIF_bm));
  if(twiname->MASTER.STATUS & (1<<TWI_MASTER_ARBLOST_bp))
  {
    twiname->MASTER.CTRLA = 0;
    PORTC.DIRSET = PIN1_bm;
    for(uint8_t i=0;i<9;i++)
    {
      PORTC.OUTSET = PIN1_bm;
      _delay_us(20);
      PORTC.OUTCLR = PIN1_bm;
      _delay_us(20);
    }
    PORTC.DIRCLR = PIN1_bm;
    twiname->MASTER.CTRLA = TWI_MASTER_ENABLE_bm;
    twiname->MASTER.STATUS = TWI_MASTER_ARBLOST_bm | TWI_MASTER_BUSERR_bm;
    twiname->MASTER.STATUS = TWI_MASTER_BUSSTATE_IDLE_gc;
    twiname->MASTER.CTRLC = TWI_MASTER_CMD_STOP_gc;
    twiname->MASTER.ADDR = page_adress;
    while (!(twiname->MASTER.STATUS & TWI_MASTER_WIF_bm));
  }
  twiname->MASTER.DATA = Adress;
  while (!(twiname->MASTER.STATUS & TWI_MASTER_WIF_bm));
  page_adress |= 0x01;
  twiname->MASTER.ADDR = page_adress;
  for(volatile uint8_t j=0 ;j<bytes ;j++){
    
    while(!(twiname->MASTER.STATUS&TWI_MASTER_RIF_bm));
    if(j == (bytes-1))
    {
      twiname->MASTER.CTRLC = (1<<TWI_MASTER_ACKACT_bp) | TWI_MASTER_CMD_STOP_gc;
    }
    readData[j] = twiname->MASTER.DATA;
  }
}

Gruß JackFrost

[Jan H. (janiiix3)]

@JackFrost
Wieso aktiviert man in der Lesefunktion ständig das Interface? Das ist 
doch total unnötig?!...

[Bastian W. (jackfrost)]

Das ist nur der Fall wenn die Abitration nicht da ist. Wenn z.b. Ein 
Reset beim lesen kam, dann ist der Slave noch im Lesemodus und man kann 
keinen Start mehr senden.

Dann wir das TWI Modul deaktiviert und es werden 9 Pulse über SCL 
gesendet.
SDA bleibt dabei High. Damit kommt mit dem letzten Bit wieder ein NACK 
und der Slave legt keine Daten mehr auf SDA.

Gruß JackFrost