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

[SlyD]

Der Roboter müsste ja dazu in die Zukunft sehen?!

  nein ich meinte DU legst fest ob Dein Slave das braucht oder nicht!
Im Programmcode.

aber rep = 0 stehen, also ein Stop eingeleitet werden?

Ne ist invertiert.
Nenn die variable no_rep dann isses logischer  

no_rep muss in ALLEN transmit / read funktionen oben auf 1 gesetzt werden.
Dann kopier Dir die eine die Du für den Sensor brauchst und änder die um, setzt es da auf 0 und setzt es in task_I2CTWI wieder auf 1.

MfG,
SlyD

[teamohnename]

Hi,
sorry, aber entweder ich bin zu blöd dafür oder es ist falsch...
Was meinst Du mit transmit/read Funktionen? Die I2C Read etc. aus der Library oder meine selbst geschriebenen, die die einzelnen Sensoren abfragen? Ersteres wäre ja sinnlos...
Es ändert sich am Oszilloskop nur etwas, wenn ich direkt in der ISR TWSTO komplett ändere, dass generell kein STop mehr gemacht wird, dann gibt es aber auch wieder einen Error.
Habe ich es denn wenigstens richtig verstanden, dass ich in die task_i2c() no_rep auf 1 setzen soll, vor jede eigene Funktion auch no_rep auf 1 setze und dieses dann in der Routine zum Abfragen des Sensors auf 0 setze, dass das also dann so aussieht:

//no_rep wird in der ISR vor TWSTO eingetragen, in task_i2c() am Anfang auf 1 gesetzt und in dieser einen Funktion auf 0 gesetzt
void getIR(void)
{
  no_rep_start = 0;
  I2CTWI_transmitByte(0x00,0x07);
  I2CTWI_readBytes(0x00, sensorBuf, 3);
}

Vielen Dank und
Viele Grüße
teamohnename

[SlyD]

In den I2CTWI_transmit Funktionen in der Lib - direkt bevor TWCR gesetzt wird - muss das auf 1 gesetzt werden.
In den anderen Funktionen könnte man es auch setzten - geht aber glaub ich auch ohne.
no_rep sollte übrigens ein uint_8t sein.


In der task_I2CTWI sollte dann in der if(TWI_operation)
no_rep auf 1 gesetzt werden....
Im einfachsten fall einfach immer ohne unterscheidung welcher Zustand.

readRegisters kannst Du dann übrigens nicht dafür verwenden - das müsste auch noch passend modifiziert werden mit zusätzlichem Zustand in der task_I2CTWI... aber egal. Probier erstmal das.


I2CTWI_transmitByteREPEATED_START(0x00,0x07); <<<---- Deine Eigene Funktion
I2CTWI_readBytes(0x00, sensorBuf, 3);

müsste dann aber klappen.

MfG,
SlyD

[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!