Frage zu RP6 I2C Library: Funktionen der Lib von Peter Fleury?

[teamohnename]

Hallo,
no_rep_start wird jetzt in allen I2C Transmit Funktionen über der TWCR Zuweisung auf 1 gesetzt, außer in einer Funktion I2CTWI_transmitByteREPEATED_START(0x00,0x07);), dort wird es über TWCR auf 0 gesetzt. Auf 1 gesetzt wird es in der task_I2CTWI in if(TWI_operation).
Diese Variable wird dann nur in Zeile 381 der I2CMaster Libs TWSTO zugewiesen:

case TWI_MTX_DATA_ACK:      // Data byte has been transmitted and ACK received
		  if (TWI_bufPos < TWI_msgSize) {
			TWDR = I2CTWI_buf[TWI_bufPos++];
			TWCR = (1<<TWEN)|                                 // TWI Interface enabled
				   (1<<TWIE)|(1<<TWINT)|                      // Enable TWI Interupt and clear the flag to send byte
				   (0<<TWEA)|(0<<TWSTA)|(0<<TWSTO)|           //
				   (0<<TWWC);                                 //  
		  } else {                   // Send STOP after last byte
			TWI_statusReg.lastTransOK = 1;                 // Set status bits to completed successfully. 
			TWCR = (1<<TWEN)|                                 // TWI Interface enabled
				   (0<<TWIE)|(1<<TWINT)|                      // Disable TWI Interrupt and clear the flag
				   (0<<TWEA)|(0<<TWSTA)|(no_rep_start<<TWSTO)|           // Initiate a STOP/REP_START condition.
				   (0<<TWWC);                                //
		  }
		  break;

Sonst niergends!

Das Resultat ist der TWI Error 0x30.
Wenn ich in meiner neuen, angepassten Funktion:

void I2CTWI_transmitByte_RepeatedStart(uint8_t targetAdr, uint8_t data)
{
	while(I2CTWI_isBusy() || TWI_operation != I2CTWI_NO_OPERATION) task_I2CTWI();
	I2CTWI_delay();
	TWI_msgSize = 2;
	I2CTWI_buf[0] = targetAdr;
	I2CTWI_buf[1] = data;
	TWI_statusReg.all = 0;
	no_rep_start = 0;
	TWCR = (1<<TWEN)|(1<<TWIE)|(1<<TWINT)|(0<<TWEA)|(1<<TWSTA)|(0<<TWSTO);
}

Das no_rep_start auf 1 setze, ist alles wie vorher...

Muss eventuell noch irgendwas (ein TWSTO) mit no_rep_start gesetzt oder ergänzt werden?

Danke und
Viele Grüße
teamohnename

[SlyD]

Hallo,

die Bedeutung der Fehlercodes steht übrigens in der Header Datei.
In diesem Fall kam ein NACK nach nem Datentransfer zurück.

> Auf 1 gesetzt wird es in der task_I2CTWI in if(TWI_operation).

wo genau? Muss ganz oben gemacht werden nicht unter den Fallunterscheidungen.
(also wieder bevor TWCR gesetzt wird)

> Muss eventuell noch irgendwas (ein TWSTO) mit no_rep_start gesetzt oder ergänzt werden

Schon möglich. Wie gesagt - ich hab keine Zeit das zu testen das oben gesagte war alles
frei aus dem Kopf ohne nochmal ins Datenblatt zu schauen.


MfG,
SlyD

[teamohnename]

Hi,
wenn es ganz oben in task_I2CTWI steht, ist auch wieder alles wie vorher, also ohne Repeated Start...
Was könnte denn noch geändert werden müssen? Wenn Du keine Zeit hast, ist das auch erstmal nicht so schlimm... Wir werden dann wohl noch etwas experimentieren müssen... Aber wenn Dir noch etwas einfällt; nur her damit!

Vielen, vielen Dank für Deine Hilfe bis jetzt und
Viele Grüße
teamohnename

Hier übrigens nochmal ein Bild vom Scope, wie es war, als das Nack kam:

[Dirk]

Hi,

SlyD hatte gesagt, dass das no_rep = 1 nicht ganz oben in der task_I2CTWI, sondern nach if(TWI_operation) eingefügt werden soll:

void task_I2CTWI(void)
{
 if (!I2CTWI_isBusy()) {
  if (TWI_statusReg.lastTransOK) {
   if(TWI_operation) {
    no_rep = 1;
...

Probier das doch mal!

[teamohnename]

Hi Dirk,
danke für Deine Antwort.
Zuletzt hat SlyD doch gesagt, dass das nach ganz oben muss?!
Wenn ich das unter die Abfragen mache, kommt die Fehlermeldung und das, was ich zuletzt als Bild gepostet habe am Oszilloskop.
Viele Grüße
teamohnename

EDIT:
Anscheinend haben wir den Fehler gefunden, nachdem wir uns nochmal das Datenblatt angeschaut haben. Vorher war das so in der ISR:

TWCR = (1<<TWEN)|                                 // TWI Interface enabled
(0<<TWIE)|(1<<TWINT)|                      // Disable TWI Interrupt and clear the flag
(0<<TWEA)|(0<<TWSTA)|(no_rep_start<<TWSTO)|           // Initiate a STOP condition.
(0<<TWWC);                                //

Dabei würde aber ein Stop durchgeführt werden, wenn no_rep_start 1 ist. Wenn das 0 ist, passiert irgendwas anderes, auf jeden Fall kein Repeated Start, da ein Repeated Start nur durchgeführt wird, wenn das so aussieht:

TWCR = (1<<TWEN)|                                 // TWI Interface enabled
(0<<TWIE)|(1<<TWINT)|                      // Disable TWI Interrupt and clear the flag
(0<<TWEA)|(1<<TWSTA)|(0<<TWSTO)|           // Initiate a REPEATED START condition.
(0<<TWWC);                                //

Jetzt haben wir das so in eine if-Abfrage gepackt:

if(no_rep_start == 1){
  TWCR = (1<<TWEN)|                                 // TWI Interface enabled
  (0<<TWIE)|(1<<TWINT)|                      // Disable TWI Interrupt and clear the flag
  (0<<TWEA)|(0<<TWSTA)|(1<<TWSTO)|           // Initiate a STOP condition.
  (0<<TWWC);                                //
}
else{
  TWCR = (1<<TWEN)|                                 // TWI Interface enabled
  (0<<TWIE)|(1<<TWINT)|                      // Disable TWI Interrupt and clear the flag
  (0<<TWEA)|(1<<TWSTA)|(0<<TWSTO)|           // Initiate a REPEATED START condition.
  (0<<TWWC);                                //
}

Jetzt wird definitiv ein repeated Start gesendet, nachdem die Leseanfrage gesendet wird, bricht der Slave aber mit einem Nack ab (Fehlercode 0x48).
Woran könnte das liegen? Wahrscheinlich ja an der Lesefunktion, wenn man die ausklammert, ist der Fehler da nicht mehr. Was muss man da aber ändern?

EDIT2:
Das Nack liegt daran, dass no_rep_start nicht wieder auf 1 zurückgesetzt wird, nachdem es in der Transmit Funktion auf 0 gesetzt wurde.
Woran liegt das? Vermutlich ja an task_i2c... Aber wo muss das jetzt genau hin?

EDIT3:
So haben wir den test durchgeführt:

void getIR(void)
{
	I2CTWI_transmitByte_RepeatedStart(0x55<<1,0x07);
	while(no_rep_start == 0){} //hier bleibt der Code hängen
	I2CTWI_readRegisters(0x55<<1, 0x07, sensorBuf, 3);
	
// This masks off the error bit of the high byte, then moves it left 8 bits and adds the low byte.
    //mlx90614_l = (((sensorBuf[1] & 0x007F) << 8) + sensorBuf[0]);
}

Wenn wir jetzt aber neben no_rep_start = 1 eine LED anschalten, geht diese an...
Komisch.

[Dirk]

Hi,

müßte beim Rep-Start nicht auch TWIE auf 1 gesetzt werden?

[teamohnename]

Hallo Dirk,
anscheinend nicht, das steht so im Datenblatt:

A REPEATED START condition is generated by writing the following value to TWCR:
TWCR      TWINT    TWEA    TWSTA    TWSTO    TWWC    TWEN       –        TWIE
Value           1            X             1              0             X            1           0          X

Grüße
teamohnename

EDIT:
Ist wohl kein WYSIWYG Editor hier... Nichtmal in der Code Ansicht. Egal, steht im ATmega32 Datenblatt auf S. 184 fast ganz unten.