ATMEGA128 externer Interrupt resettet

[AndreasP (Gast)]

Hallo zusammen,
mir ist bewusst, dass ein ATMEGA128 nicht unbedingt der optimale uC ist 
um mit dem uC programmieren zu beginnen. Da ich aber ein 
funktionstüchtiges Testboard vorliegen habe versuch ichs damit trotzdem. 
Leider scheine ich schon an einem externen Interrupt zu scheitern.
Zunächst zum Prog.:
An einem funktionstüchtigem Testboard mit einem ATMEGA128 soll an PortC 
eine RGB LED ansteuert werden. Nach Programmstart soll eine Farbe 
leuchten. Nach einem externen Interrupt an PORTD0 (INT0) der bei einer 
fallenden Flanke ausgelöst werden soll, soll sich die Farbe ändern.

Mein Code:

#ifndef F_CPU
#warning "F_CPU war noch nicht definiert, wird nun mit 16000000UL definiert"
#define F_CPU 16000000UL     /* Quarz mit 16 Mhz  */
#endif
#include <avr/io.h>
#include <avr/interrupt.h>

/* ********************************* Variablen und Konstanten definieren************************** */
    volatile uint8_t aaa;
    uint8_t rot    = 0b11111110;
    uint8_t gelb  = 0b11111100;
    uint8_t gruen  = 0b11111101;
    uint8_t cyan  = 0b11111001;
    uint8_t blau  = 0b11111011;
    uint8_t violett = 0b11111010;
    uint8_t weiss   = 0b11111000;
    uint8_t schwarz = 0b11111111;
    
/* ********************************** Main Prog. ********************************************** */    
        
int main(void)
{  
  DDRC = 0xFF;                  //Ausgangsport für die LED definieren
  PORTC = 0xFF;                  //Ausgangspins alle auf logisch 1 setzten (alle LEDs aus / schwarz)
  
   /* *********************************Initialisierung Interrupt ********************************** */
   
  DDRD = 0x00;                  //Eingangspot für Interrupt definieren
  PORTD = 0x01;                  //Pullup Widerstand setzten für Interrupt an fallender Flanke bei INT0
  
  EICRA |= (1<<ISC01);              //INT0 bei fallender Flanke 
  EIMSK |= (1<<INT0);                //Interrupt an INT0 aktivieren
      
  PORTC = rot;                  //Rotes Licht soll leuten bevor Programm startet.

  sei();                      // Erlaube globale Interrupts
  aaa = 0;                    // Wert einer Variablen zuweisen. Diese Variable soll im Interrupt geändert werden
  
   /* ********************Standardprgramm starten: Hier LED Farben werden im 1sec Takt gewechselt.  ************* */
  while(1)
  {   
    if (aaa == 0)
    {
      PORTC = gelb;
    } 
    else
    {
      PORTC = blau;      
    }    
  }
}

/* ********************************** Interrupts *****************************   */
ISR(INT0_vect)
{
  //cli();
  aaa != aaa;
  //sei();  
}

Nach dem Starten des Programms leuchtet es wie gewollt gelb. Das es 
zwischendrin mal rot war ist natürlich nicht zu sehen. Wenn ich jetzt 
aber einen Interrupt an INT0 auslöse indem ich INT0 extern auf Masse 
ziehe, leuchtet es nicht blau, sondern immer rot, d.h. das der uC nach 
dem sei() ununterbrochen resettet so lange INT0 auf Masse liegt.
Leider habe ich in den anderen Beiträgen keine passenden Beiträge 
gefunden, die mir weitergeholfen haben. Sobald ich einen externen 
Interrupt auslöse wird dieser abgearbeitet und danach das Programm neu 
gestartet. Dies hab ich gestestet indem ich in dierkt in die ISR() eine 
Farbfolge ablaufen lies, was auch geklappt hat. Danach wurde aber nicht 
an die reichtige Stelle zurückgesprungen sondern das Prog. neu 
gestartet.
Was ist falsch? Bitte um Hinweise bzw. Verbesserungen des Codes.

Danke im voraus
Andreas

[Stefan E. (sternst)]

M103C-Fuse deaktivieren.

[AndreasP (Gast)]

Ah super, nachdem ich M103C-Fuse deaktiviert habe und den Unsinn:

ISR(INT0_vect)
{
  //cli();
  aaa != aaa;
  //sei();  
}

durch

ISR(INT0_vect)
{
if (aaa == 0)
  {
    aaa = 1;
  } 
  else
  {
    aaa = 0;
  }    
}

ersetz habe geht alles. Danke für die schnelle Hilfe.

[Karl H. (kbuchegg)]

AndreasP schrieb:
> Ah super, nachdem ich M103C-Fuse deaktiviert habe und den Unsinn:
>

> ISR(INT0_vect)
> {
>   //cli();
>   aaa != aaa;
>   //sei();
> }
>

> durch
>

> ISR(INT0_vect)
> {
> if (aaa == 0)
>   {
>     aaa = 1;
>   }
>   else
>   {
>     aaa = 0;
>   }
> }
>

> ersetz habe geht alles. Danke für die schnelle Hilfe.

Der 'Unsinn' macht genau das gleiche, wie deine Langversion :-)

genauso wäre auch

  aaa = 1 - aaa;

gegangen, obwohl es nicht ganz 100% äquivalent ist.

Es führen eben viele Wege nach Rom.

[Stefan E. (sternst)]

Karl Heinz Buchegger schrieb:
> Der 'Unsinn' macht genau das gleiche, wie deine Langversion :-)

Nö (ist nur ein Vergleich ohne Ergebniszuweisung).

[Karl H. (kbuchegg)]

Stefan Ernst schrieb:
> Karl Heinz Buchegger schrieb:
>> Der 'Unsinn' macht genau das gleiche, wie deine Langversion :-)
>
> Nö (ist nur ein Vergleich ohne Ergebniszuweisung).

Ich verzieh mich schon in mein Eckchen zum schämen.
Da hab ich doch tatsächlich =! gelesen, wo != steht.

[Marc (gierig)]

> Ich verzieh mich schon in mein Eckchen zum schämen.
> Da hab ich doch tatsächlich =! gelesen, wo != steht.

vielleicht einfach zum alter stehen und dem Optiker seines Vertrauens
mal einen Besuch abstatten zur Überprüfung der Optischen Parameter
der eignen Biologie / Hilfemittel...

ist heute schon das zweite mal das ich lese das du dich verliest.

[Thomas E. (thomase)]

Marc D. schrieb:
>> Ich verzieh mich schon in mein Eckchen zum schämen.
>> Da hab ich doch tatsächlich =! gelesen, wo != steht.
>
> vielleicht einfach zum alter stehen und dem Optiker seines Vertrauens
> mal einen Besuch abstatten zur Überprüfung der Optischen Parameter
> der eignen Biologie / Hilfemittel...
>
> ist heute schon das zweite mal das ich lese das du dich verliest.

Und du gehst zum Buchhändler deines Vertrauens und kaufst dir einen 
Duden.

mfg.