RGB-Code Fehleranalyse

[Thomas (Gast)]

Hi,
habe nun einen Code geschrieben um mittels eines Tasters die Farben 
einer RGB-LED umzuschalten. Dabei soll so gut wie möglich der Verlauf 
eines Farbkreises nachgebildet werden. Bei jedem Druck auf den Taster 
soll sich die Farbe um ein Stcük ändern. Das funktioniert ab und zu gut, 
dann ändert sich die Farbe jedoch plötzlich wieder auf einen absolut 
falschen Wert.
Vielleicht findet ja jemand von euch meinen Fehler.

mfg und danke Thomas

#include <asf.h>
#include <avr/io.h>
#include <avr/pgmspace.h>

#define F_CPU 16000000

uint8_t shortPress=5; //50ms debounce Zeit
uint16_t longPress=50; //500ms Wartezeit
volatile uint8_t i=0;
volatile uint8_t colorVariable=0;


const uint8_t RGB_Value[] PROGMEM ={      //RGB Array
  255, 0, 0,
  255, 32, 0,
  255, 64, 0,
  255, 96, 0,
  255, 128, 0,
  255, 159, 0,
  255, 191, 0,
  255, 223, 0,
  255, 255, 0,
  223, 255, 0,
  191, 255, 0,
  159, 255, 0,
  128, 255, 0,
  96, 255, 0,
  64, 255, 0,
  32, 255, 0,
  0, 255, 0,
  0, 255, 32,
  0, 255, 64,
  0, 255, 96,
  0, 255, 128,
  0, 255, 159,
  0, 255, 191,
  0, 255, 223,
  0, 255, 255,
  0, 223, 255,
  0, 191, 255,
  0, 159, 255,
  0, 128, 255,
  0, 96, 255,
  0, 64, 255,
  0, 32, 255,
  0, 0, 255,
  32, 0, 255,
  64, 0, 255,
  96, 0, 255,
  128, 0, 255,
  159, 0, 255,
  191, 0, 255,
  223, 0, 255,
  255, 0, 255,
  255, 0, 223,
  255, 0, 191,
  255, 0, 159,
  255, 0, 128,
  255, 0, 96,
  255, 0, 64,
  255, 0, 32,
  255, 0, 0,
  };

ISR (TIMER2_COMPA_vect){            //ISR Routine - Wird ca alle 10ms aufgerufen
  uint8_t tmp_cnt;
  tmp_cnt = i;
  if((PIND & (1<<PD4))){                  //Ist der Taster beim Aufruf gedrückt wird die Laufvariable erhöht
    tmp_cnt++;}
  if(!(PIND & (1<<PD4))){                  //Ist beim neuen Aufruf der ISR der Tasternicht gedrückt wird abgefragt wie lang er gedrückt war
    if((tmp_cnt>shortPress) && (tmp_cnt<longPress)){  //Wenn der Taster nur kurz gedrückt war
      tmp_cnt=0;
      colorVariable=colorVariable+3;  // Weiterschalten der Farbe
      ~(PINB |= (1<<PB5));}      //Pin Togglen für Feedback, bzw. Kontrolle der Entprell-Routine
    if(tmp_cnt>longPress){
      tmp_cnt=0;
      //langer Druck = true
      }
    tmp_cnt=0;
    }
    i=tmp_cnt;
  }

int main (void)
{
  board_init();
  
  DDRD |= (1<<PD6) | (1<<PD5);
  DDRB |= (1<<PB1) | (1<<PB5);
  DDRD &= ~(1<<PD4);
  
  
  TCCR0A |= (1<<WGM00) | (1<<WGM01) | (1<<COM0A1) |(1<<COM0B1); // fast PWM Mode, None-inverted mode (High at bottom, Low on Match)
  TCCR0B |= (1<<CS00);              //No Prescaler
  
  TCCR1A |= (1<<WGM10) | (1<<COM1A1) | (1<<COM1B1);
  TCCR1B |= (1<<WGM12) | (1<<CS10);        // Timer 1 für Fast PWM-Initialisieren
  
  TCCR2A |= (1<<WGM21);              // Timer 2 CTC Mode
  TCCR2B |= (1<<CS22) | (1<<CS21) | (1<<CS20);  // Clock/1024
  OCR2A = 156;                  // ISR Aufruf alle 10ms
  
  sei();
  
  TIMSK2 |= (1<<OCIE2A);
  
  
  for(;;){
    if(colorVariable==49){
      colorVariable=0;};
    OCR0A = pgm_read_byte (&RGB_Value[colorVariable]);      //Wert für jede Farbe aus RGB Tabelle an Compare register übergeben
    OCR0B = pgm_read_byte (&RGB_Value[colorVariable+1]);
    OCR1A = pgm_read_byte (&RGB_Value[colorVariable+2]);
  }
}

[sebastian (Gast)]

Du zählst die Variable um 3 hoch, prüfst aber so:
> if(colorVariable==49)

49 ist nicht durch 3 teilbar.