I2C Puffer enthält falsche Werte

[Reto (Gast)]

Ich sende von einem Mikrocontroller zum anderen eine Befehl über I2C und 
sende diesen dann per UART weiter.

Das funktioniert eigentlich recht gut, wenn ich bei empfangen.c in der 
while Schlaufe eine Pause von mindestens 1ms einbaue. Ohne diese habe 
ich mit dem ersten Befehl das Problem, dass nicht alle Bytes einen Wert 
enthalten.

Hat jemand eine Idee?

//senden.c
void write_slave1(void)
{
  unsigned char temp = ((0x21^1^0x35^1^1)|0x01);
  i2c_start_wait(0x50 + I2C_WRITE);
  i2c_write(1);  // Startadresse
  i2c_write(0x00);  // Temp auf 0x00 bis alle Daten geschrieben sind
  i2c_write(1);     // Adresse
  i2c_write(0x35);  // Rot
  i2c_write(1);  // Grün
  i2c_write(1);  // Blau
  i2c_write(temp);  // CRC
  i2c_stop();
  i2c_start_wait(0x50 + I2C_WRITE);
  i2c_write(1);  // Startadresse
  i2c_write(0x21);  // Befehl 0x21
  i2c_stop();       // stopt I2C Verbindung
}

int main(void)
{
  i2c_init();                             // initialize I2C library

  //write_slave();
  while(1)
  {
    write_slave1();
    _delay_ms(1000);
    write_slave2();
    _delay_ms(1000);
    write_slave3();
    _delay_ms(1000);

  }
  return 0;
}


//empfangen.c
while(ieCntS < READINS && ieCntN < 252/6)    //Recive max x Instructions or check Recive Buffer one times
{
  delay_ms(1);
  if(rxbuffer[ie] != 0x00)  //Something in receive buffer
  {
    if(((rxbuffer[ie]^rxbuffer[ie+1]^rxbuffer[ie+2]^rxbuffer[ie+3]^rxbuffer[ie+4]) | 0x01) != rxbuffer[ie+5]) //Check CRC
    {
      rxbuffer[ie]=0;   //Clear recive instruction
    }
    else
    {
      for(int i=0; i<SENDBUFFER; i++)    //Search free space in send buffer
      {
        if(!sendPufLED[is][1])
        {
          sendPufLED[is][0] = 0;          //Counter for age
          sendPufLED[is][1] = rxbuffer[ie];    //Instruction
          rxbuffer[ie]=0;    //Clear recive instruction  
          sendPufLED[is][2] = rxbuffer[ie+1];    //Address
          sendPufLED[is][3] = rxbuffer[ie+2];    //Data1
          sendPufLED[is][4] = rxbuffer[ie+3];    //Data2
          sendPufLED[is][5] = rxbuffer[ie+4];    //Data3
          if(is++ == SENDBUFFER)
            is = 1;    //Index send buffer to next row
          ieCntS++;  //Sucessfull read of instruction
          break;
        }
        else
        {
          if(is++ == SENDBUFFER)
            is = 1;    //Index send buffer to next row
          if(i == SENDBUFFER-1)
          {
            uart_puts_p(PSTR("! Send buffer overloadet\n"));
            ieCntN = SENDBUFFER;  //Not possible to recive instruction at the moment, leave
          }
        }
      }
    }
  }
  ie = ie+6;  //Index recive buffer to next instruction
  if(ie==253)
    ie=1;
  ieCntN++;  
}

[Karl H. (kbuchegg)]

Na ja.

Das hier
>   if(rxbuffer[ie] != 0x00)  //Something in receive buffer
ist ja nicht wirklih das Kriterium dafür, das etwas reingekommen ist.

[Reto (Gast)]

Eigentlich schon:

Ich schreibe zuerst das erste Frame Byte auf 0x00 und schreibe dann den 
Rest des Frames. Zulest schreibe ich das erste Byte des Frames.

Somit sollte der das neue Frame erst erkennen, wenn das ganze Frame 
beschrieben ist.