AVR ISR bekommt geänderte Variablen nicht mit

[Michael N. (garril)]

Hallo,

ich setze in der main-Funktion immer wieder andere Werte in Variablen.
Diese Werte werden dann beim Overflow des Timers gebraucht.

(Timer baue ich noch um, damit der nicht bis 255 sondern wie benötigt 
bis 50 zählt. So wusste ich halt, dass es geht und habs deshalb 
gelassen)

Hier mein Code:

#include <avr/io.h>
#include <avr/interrupt.h>

#define F_CPU 16000000L                  // Systemtakt in Hz
#define an(port,b) (port) |= (1<<(b))
#define aus(port,b) (port) &= ~(1<<(b))

  // aktueller Counterstand
  int pwm_counter=0;
  int poti_counter=0;

  // Helligkeit von 1 bis 50
  int pb1_soll = 1; //zw. 1 und 50: BLAU
  int pb2_soll = 1; //zw. 1 und 50: GRÜN
  int pb3_soll = 1; //zw. 1 und 50: ROT

int main(void) {

  // Für ADC Werter
  uint16_t result;

  // Ausgänge setzen
  DDRB|=(1<<PB1);
  DDRB|=(1<<PB2);
  DDRB|=(1<<PB3);

  //ADC zum Poti auslesen
  ADMUX = (0<<REFS1) | (1<<REFS0);  // ADC Ref auf Avcc, PC0 gewählt, normale Formatierung
  ADCSRA= (1<<ADEN) | (1<<ADSC) | (0<<ADFR) | (1<<ADPS2) | (1<<ADPS1) | (1<<ADPS0);
  while (ADCSRA & (1<<ADSC) ) {}  // auf Abschluss der Konvertierung warten
  result = ADCW; //Dummy auslesen
  ADCSRA|= (1<<ADSC); //Aktuellen tatsächlichen Wert holen, wird später für erstes Poti verwendet

  // Timer 0 konfigurieren
  TCCR0 = (1<<CS00); // Prescaler 0!!!

  // Overflow Interrupt erlauben
  TIMSK |= (1<<TOIE0);
 
  // Global Interrupts aktivieren
  sei();

  while (1) {
    //folgende Zeilen sind nur zum Debuggen. LED wird nicht heller!
    int pb1_soll=pb1_soll+1;
    int pb2_soll=pb2_soll+1;
    int pb3_soll=pb3_soll+1;

    //Poti an PC0 ->steuert-> PB1
    if (poti_counter==0) {
      while (ADCSRA & (1<<ADSC) ) {}  // auf Abschluss der Konvertierung warten
      result = ADCW; //PC0 auslesen
      pb1_soll=result/20;
      ADMUX = (0<<REFS1) | (1<<REFS0) | (0<<MUX2) | (0<<MUX1) | (1<<MUX0);  // ADC Ref auf Avcc, PC1 gewählt, normale Formatierung
      ADCSRA|= (1<<ADSC); //Aktuellen tatsächlichen Wert holen
      poti_counter++;
    }
    if (poti_counter==1) {
      while (ADCSRA & (1<<ADSC) ) {}  // auf Abschluss der Konvertierung warten
      result = ADCW; //PC1 auslesen
      pb2_soll=result/20;
      ADMUX = (0<<REFS1) | (1<<REFS0) | (0<<MUX2) | (1<<MUX1) | (0<<MUX0);  // ADC Ref auf Avcc, PC1 gewählt, normale Formatierung
      ADCSRA|= (1<<ADSC); //Aktuellen tatsächlichen Wert holen
      poti_counter++;
    }
    if (poti_counter==2) {
      while (ADCSRA & (1<<ADSC) ) {}  // auf Abschluss der Konvertierung warten
      result = ADCW; //PC2 auslesen
      pb3_soll=result/20;
      ADMUX = (0<<REFS1) | (1<<REFS0) | (0<<MUX2) | (0<<MUX1) | (0<<MUX0);  // ADC Ref auf Avcc, PC1 gewählt, normale Formatierung
      ADCSRA|= (1<<ADSC); //Aktuellen tatsächlichen Wert holen
      poti_counter=0;
    }
  }
}

#ifndef TIMER0_OVF_vect
// Für ältere WinAVR Versionen z.B. WinAVR-20071221 
#define TIMER0_OVF_vect TIMER0_OVF0_vect
#endif
// Interrupt
ISR (TIMER0_OVF_vect) {
  if (pb1_soll==pwm_counter) {
    aus(PORTB,PB1);
  }
  if (pb2_soll==pwm_counter) {
    aus(PORTB,PB2);
  }
  if (pb3_soll==pwm_counter) {
    aus(PORTB,PB3);
  }
  if (pwm_counter>=50) {
    pwm_counter=0;
    if (pb1_soll>0) {
      an(PORTB,PB1);
    }
    if (pb2_soll>0) {
      an(PORTB,PB2);
    }
    if (pb3_soll>0) {
      an(PORTB,PB3);
    }
  }
  else {
    pwm_counter++;
  }

}

Die Farben der LEDs (bzw. ist es eine RGB-LED) ändert sich nicht.
Die Werte bleiben immer wie ich sie am Anfang der Variablendeklaration 
von pb1_soll, pb2_soll und pb3_soll eingestellt habe.

Schonmal Danke für eure Hilfe.

[moep (Gast)]

Variablen, die du sowohl in deiner main als auch in ISRs benutzt müssen 
als "volatile" deklariert werden.

[moep (Gast)]

Und:

> int pb1_soll=pb1_soll+1;
> int pb2_soll=pb2_soll+1;
> int pb3_soll=pb3_soll+1;

Die "int"s müssen hier weg

[Glaskugel (Gast)]

volatile
volatile wirkt zusammen mit Typdeklarationen als Modifizierer.

  volatile typ name;

Das Schlüsselwort volatile teilt dem Compiler mit, daß die mit name 
bezeichnete Variable mit dem Datentyp typ durch Ereignisse außerhalb der 
Kontrolle des (normalen) Programmflusses verändert werden kann.
Der Wert der Variablen muß deshalb vor jedem Zugriff neu aus dem 
Hauptspeicher eingelesen werden, d.h. er darf nicht in einem Register 
des Prozessors zwischengespeichert werden.

[leicht geändertes Zitat von 
http://www.imb-jena.de/~gmueller/kurse/c_c++/c_volat.html]

[Michael N. (garril)]

Danke ihr seid meine Helden :)

Es funktioniert.