Global Variablen in Interrupt-Routine

[André (Gast)]

Hallo miteinander,

ich bin kurz vor dem Verzweifeln:

Ich möchte in einem Projekt ganz einfach über einen Timer die globalen 
variablen sec, min, hour hochzählen. Ich habe diese Variablen jeweils 
als

volatile uint8_t deklariert.

Das irrwitzige ist, dass mein code, wenn ich in in SiSy compile und 
brenne, fehlerfrei läuft. Ich möchte aber gern in Zukunft im 
AtmelStudio6 arbeiten. Und hier läuft der code leider nicht fehlerfrei, 
da auf die globalen Vriablen nicht zugeriffen bzw. sie nicht hochgezählt 
werden. Die ISR wird zwar angesprungen und eine LED wird auch im 
Sekundentakt getoggelt, aber die Bedingung sec > 59 wird nie erreicht. 
Und beim build läuft alles ohne Warnungen und Fehlermeldung ab.

Die Interrupt-Routine ist wie folgt umgesetzt:

ISR(TIMER1_COMPA_vect)
{
   sec++;
  PORTB ^= (1<<PB0);
  if (sec > 59)
  {
      min++;
      sec = 0;
      PORTB ^= (1<<PB1);
  }
  if (min > 59)
  {    
    min = 0;
      hour++;
  }
  if (hour>23)
     hour = 0;  
}

Ich habe weitere banale Testprogramme umgesetzt, bei denen 
ausschließlich in der main-loop irgendetwas passiert, und die 
funktionieren einwandfrei. Ich meine das Problem tatsächlich auf die 
Verwendung von globalen Variablen eingrenzen zu können.

Ich bin für jeden Tipp dankbar.
André

Hier noch ein paar Eckdaten:
Ich nutze einen ATMEGA8,
AtmelStudio6 Version 6.1.2730,
gcc 4.7.2

[Michael R. (Firma: Brainit GmbH) (fisa)]

Du möchtest uns sicher den ganzen Code zeigen...

[modstanden (Gast)]

>aber die Bedingung sec > 59 wird nie erreicht.
Hast Du einen Debugger ? Woher weißt Du das ?
Geht es vielleicht, und nur PORTB ^= (1<<PB1); funktioniert nicht ?

[Karl H. (kbuchegg)]

Dann schätze ich mal, dass ein paar vergessene 'volatile' das Problem 
sind.

FAQ: Was hat es mit volatile auf sich

[Wilhelm F. (Gast)]

Karl Heinz schrieb:

> Dann schätze ich mal, dass ein paar vergessene 'volatile' das Problem
> sind.
>
> FAQ: Was hat es mit volatile auf sich

Ob da was weg optimiert wurde, das sollte sich aus den Listings aber 
heraus finden lassen. Dafür sind sie doch da.

[André (Gast)]

Hallo,

ja entschuldigt bitte, dass ich nur die Hälfte des codes gepostet habe.

Nun kommt doch alles anders:
Als ich der Bitte nachkommen wollte, den ganzen code hier einzustellen, 
ist mir aufgefallen, dass der code in SiSy nicht 100% dem in AtmelStudio 
entsprach.

Hier der code, wie ich ihn in SiSy nutze:

#define   F_CPU   3686400
#include <avr\io.h>
#include <avr\interrupt.h>

volatile uint8_t sec, min, hour;

ISR(TIMER1_COMPA_vect)
{
  sec++;
  PORTB ^= (1<<PB0);
  if (sec > 5)
  {
    min++;
    sec = 0;
    PORTB ^= (1<<PB1);
  }
  if (min > 59)
  {
    min = 0;
    hour++;
  }
  if (hour>23)
  hour = 0;
  
}


ISR(ADC_vect)
{
  // Beginn der Ereignisbehandlung
  uint16_t wert=ADC;   // Einlesen des Analogwertes Low+High-Teil
}


void init()
{

  DDRB=0b00000011; // Ports initialisieren
  TCCR1B=0x03;   // Teiler 1/64
  TCCR1B|=0x08;   // Zählen bis Vergleichswert
  OCR1A=57600;   // Vergleichswert speichern
  TIMSK|=0x10;   // Interrupt bei Vergleichswert
  ADMUX=0x40;   // Port, Referenzspannung und Auflösung
  ADCSRA=0xED;   // Modus, Interrupt und Start
  
  sei();
}


int main()
{
  
  init();
  while (1)
  {
    
  }
}

Hier ist noch eine zusätliche, eigentlich unnütze ISR(ADC_vect), da ich 
keinen ADC initialisert habe. Um hier ein Minimalbesipiel zu posten, 
habe ich also diese ISR im Code gelöscht. Das Resultat war, dass nun 
auch das Programm in SiSy nicht mehr läuft. (???)

Wenn ich die Zeilen

ISR(ADC_vect)
{
  // Beginn der Ereignisbehandlung
  uint16_t wert=ADC;   // Einlesen des Analogwertes Low+High-Teil
}

auch in den code im AtmelStudio einfüge, funktioniert das Hochzählen der 
globalen Variablen auch.

Das ist einerseits erfreulich, dass ein und derselbe code in 
unterschiedlichen Entwicklungsumgebungen zum selben Ergebnis führt, ABER 
was hat der funktionierende Programablauf mit der überflüssigen Routine

ISR(ADC_vect)
{
  // Beginn der Ereignisbehandlung
  uint16_t wert=ADC;   // Einlesen des Analogwertes Low+High-Teil
}

zu tun?

Mache ich bei der Interruptbehandlung grundlegend etwas falsch?

Vielen dank für Eure Hilfe und ich entschuldige mich für die gestifftete 
Verwirrung
André

[Karl H. (kbuchegg)]

Gibts du einen Interrupt frei, für den du keinen Handler hast, wird das 
mit einem Programm-Reset bestraft.

Ich  hab keine Ahnung, was da hier

  ADCSRA=0xED;   // Modus, Interrupt und Start

tatsächlich alles freigegeben wird.
Mich interessiert es nämlich nicht mehr, mir die Bitbelegung aus dem 
Datenblatt rauszusuchen. Wenn du nicht in der Lage bist, das so zu 
schreiben, dass man das leicht sehen kann, dann such dir selber raus, ob 
hier ein Interrupt freigegeben wird oder nicht.

[Basti (Gast)]

Da wird also in eine  Interrupt gesprungen dens nicht gibt und du 
fragst, was falsch sein könnte? Lies doch erstmal deine eigenen 
Codekommentare ;)

[Karl H. (kbuchegg)]

Selbiges hier
>  TIMSK|=0x10;   // Interrupt bei Vergleichswert

Wie gesagt:
gibst du einen Interrupt frei, brauchst du einen Handler dafür.
Zuwiderhandlungen werden mit einem Reset bestraft.

[André (Gast)]

Hallo miteinander,

ich war wohl schon betriebsblind und habe bei copy/paste die 
Initialisierung des ADC mitgeschleppt und garnicht mehr auf dem Schirm 
gehabt, Sorry.

Das kommt davon, wenn man versucht seinem Hobby zwischen Tür und Angel 
nachzugehen und eigentlich doch nicht die nötige Ruhe dafür hat.

Aber eins ist sicher: Hier im Forum bekommt man immer die Lösung für 
sein (hausgemachtes) Problem geliefert. Vielen Dank dafür.

Und nochmals Entschuldigung für die Verwirrung.

Gruß
André