While Schleife Abbruchbedingung wird nicht wie geplant ausgeführt

[Chris T. (chris0086)]

Hallo Leute,
ich habe ein Problem mit einer Whileschleife in der solange gewartet 
wird bis entweder ein timeout stattfindet, ODER Daten empfangen wurden.

Die While Schleife wird aber erst nach dem Timeout verlassen, ich sehe 
den Fehler nicht :(

Der Systic wird jede Sekunde um eins erhöht.
Sobald ein string mit Master_send_command gesendet wird schicke ich dann 
die Antwort über die serielle Schnittstelle und u_r_c ist nichtmehr 0.
Jetzt müsste die While Schleife eigentlich sofort verlassen werden, wird 
sie aber erst nach 3 Sekunden... dann geht erst die LED an

Sieht jemand einen Fehler?

ISR(USART_RXC_vect)
{
  
  
  uint8_t i;
  if(uart_transmit_progress)
  {
    uart_r_array[u_rx_start] = UDR;
    uart_back_read=1;

    //u_rx_start++;
    
  }
  else
  {


    //USART_Transmit(receive_string[i]);
    //USART_Transmit(i);
    i = u_rx_start;
    u_receive_string[i] = UDR;
    //lcd_putc(i);
    // Sendstring: 01 10 01 00 01 04 00 00 30 18
    if (i==5)
        {
          receive_max = u_receive_string[i] +6;
          
          //if((u_receive_string[i] & 0x10 )== 0x10)

        }

    if(i == receive_max)
    {
      u_rx_start=0;
      
      u_r_c = receive_max;
      //USART_Transmit('Z');

    }
    else
    {

      i++;
      u_rx_start = i;
    }
  }
}


ISR(TIMER1_COMPA_vect)
{
  systic++;
  //LED3_C;
}

int main(void)
{


  //LED-Ausgänge konfigurieren
  DDRA |= (1<<DDA7) | (1<<DDA6) | (1<<DDA5);


  //USART initialisieren

  TRANSMIT_CONF; // Transmit Pin als Ausgang
  //USART0
  UBRRH = UBRR_VAL_UART >> 8;
  UBRRL = UBRR_VAL_UART& 0xFF;

  //USART0
  UCSRB |= (1<<TXEN);                // UART TX einschalten
  UCSRB |= (1<<RXEN);
  UCSRB |= (1<<RXCIE);
  UCSRB |= (1<<TXCIE);
  UCSRC |= (1<<URSEL)|(1<<UCSZ0)| (1<<UCSZ1) | (1<<UPM1);    // Asynchron 8N1 even

  
  
  //Timer initialisieren
  
  //Systemtic
  TCCR1B |= (1<<WGM12);
  TCCR1B |= (1<< CS12)|(1<< CS10); //CPU/1024
  TIMSK |= (1<<OCIE1A);      //Compare match
  OCR1A = 8000;          // 1 Sekunde
  
  sei();




  while(1)
  {
    if(u_r_c != 0) //Es wurden Daten empfangen
    {



      uart_putc((uint8_t) u_r_c);
      u_r_c = 0;
      LED3_OFF ;
    }
    if(systic)
    {
      Master_send_command(current_controller,48,0);
      LED3_OFF;
// HIER DIE WHILE SCHLEIFE
      while((systic<=3) && !(u_r_c ) );  
      if(systic >= 3) // Gerät hat nicht geantwortet
      {
        
      }
      if(u_r_c != 0)    //Gerät hat geantwortet
      {
        
        uint16_t acceleration_sensor = 0;
        if(u_receive_string[1] == 48)    //Slaveantwort mit Sensorinformationen  
        {
          if(u_receive_string[6] != 0)
          {
            LED3_ON;
          }
          else              //kein Sensor hat ausgelöst
          {
            
          }
        }
        u_r_c = 0;
      }
      current_controller++;
      systic = 0;
    }
    //Alle controller ausgelesen, jetzt schauen ob ein Panel geschaltet werden muss und das tun
    if(current_controller>SLAVE_CONTROLLER)
    {
      current_controller = 1;
      uint8_t controller;
      uint8_t feld;
      feldgetroffen = 0;          //36 = Feld2 controller 4
      feld = (feldgetroffen>>4);
      controller = (feldgetroffen &= 0x0F);
      //uart_putc(30);
      //uart_putc(feld);
      //uart_putc(controller);
      if(feld != 0)              // es wurde wirklich ein Feld getroffen
      {
        if(controller !=0)          //Slavecontroller einstellen
        {
          Master_send_command(controller,(feld |= (1<<4) ),colour_1);
        }
        else                //Mastercontroller einstellen
        {
          switch_color((colour_1>>16),(colour_1>>8),(colour_1>>0),feld);
        }
        
      }
      //LED3_OFF;
      
    }
    
  }
}

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Chris T. schrieb:
> Sieht jemand einen Fehler?

Das fehlt erstmal die Deklaration deiner Variablen. Ohne die gesehen zu 
haben, vermute ich mal, das es der alte Fehler ist, Flags oder 
Variablen, die in ISRs und dem Hauptprogramm benutzt werden, nicht als 
'volatile' zu erklären.
Du spart übrigens nichts, wenn du statt kryptischer Zeichenketten wie 
'u_r_c' z.B. 'Receive_Complete' oder so schreibst. Lesen tut sichs aber 
besser - tust du bei anderen Variablen ja auch.

[Oliver S. (oliverso)]

Chris T. schrieb:
> u_r_c = 0;

Oliver

[Chris T. (chris0086)]

Danke an die Hilfe, es läuft jetzt. Der Fehler war ganz woanders...

[Stefan F. (Gast)]

> Der Fehler war ganz woanders...

Nett wäre es, die Lösung des Rätsels hier zu veröffentlichen.