Zustände schalten mit Interrupt

[Michi R. (Gast)]

Hallo mikrocontroller.net Gemeinde!

Da ich in Sachen yC-Programmierung noch ziemlich grün hinter den Ohren 
bin und mich bestehende Forumeinträge nicht weitergebracht haben, hier 
meine erste Frage:

Mit meinem Atmega8 will ich 3 LEDs mit einem Taster durchschalten 
können. Sprich bei jedem Tastendruck in den nächsten Zustand schalten. 
Das soll per Interrupt realisiert werden.
Was hab ich bei meinem Code vergessen? Die Ausgänge flackern wild hin 
und her... :(
Vielen Dank für eure Mühe.
Gruß Michi

Hier mein Code:

#define F_CPU 3686000UL
#include <util/delay.h>
#include <avr/io.h>
#include <avr/interrupt.h>

int i=0;

ISR(INT0_vect)
{
  _delay_ms(20);
  i++;
  
}

void eins() 
{
  PORTD=0b10000000;
}

void zwei()
{
  PORTD=0b01000000;
}

void drei()
{
  PORTD=0b00100000;
}

int main(void)
{
  
  DDRD=0b11100000;
        PORTD |= ( 1<<PD2 );  //pull up aktivieren
  
  GICR = ( 1<<INT0 );
  MCUCR = ( 0<<ISC01 | 0<<ISC00 );
  GIMSK = ( 1<< INT0 );

  sei();
  
  
  while(1)
    {
    
    if (i==0) 
    {  
      eins();  
    }
    else if (i==1)
    {
      zwei();
    }
    else if (i==2)
    {
      drei();
    }
    else
      i=0;
  }
}

[Karl H. (kbuchegg)]

Michi R. schrieb:

> Mit meinem Atmega8 will ich 3 LEDs mit einem Taster durchschalten
> können. Sprich bei jedem Tastendruck in den nächsten Zustand schalten.
> Das soll per Interrupt realisiert werden.

Warum?
Taster und externer Interrupt sind eine blöde Idee, auch wenn das immer 
wieder propagiert wird.
Entprellung
die Konmfortroutinen vom PeDa sind alles was man braucht um Tasten 
zuverlässig auszuwerten.

[Karl H. (kbuchegg)]

ISR(INT0_vect)
{
  _delay_ms(20);

genau deswegen ist das eine blöde Idee.
Ein delay_ms in einer ISR ist ein ziemlicher Supergau.

[Karl H. (kbuchegg)]

>

>   MCUCR = ( 0<<ISC01 | 0<<ISC00 );
>

so, so.
Und du denkst also, dass du innerhalb von 20 Millisekunden eine Taste 
drücken und wieder loslassen kannst.

Der Interrupt kommt, solange der Pin auf Low-Pegel ist und auch auf 
Low-Pegel bleibt! Bei dir: solange die Taste gedrückt ist, kommt ein 
Interrupt nach dem anderen.

[Michi R. (Gast)]

Oh! Ich dachte, der Interrupt kommt nur bei fallender Flanke...

[Karl H. (kbuchegg)]

Nicht wenn mein AtMega8 Datenblatt korrekt ist.

[Michi R. (Gast)]

;) So ists besser:

MCUCR = ( 1<<ISC01 | 0<<ISC00 );

Macht aber noch immer nicht, was es soll...
Ich dachte, für diese einfache Anwendung tuts auch ein externer 
Interrupt mit Taster! Bei der "Komfortroutine" blick ich ehrlich gesagt 
nicht wirklich durch...

[Karl H. (kbuchegg)]

Michi R. schrieb:
> ;) So ists besser:
>
> MCUCR = ( 1<<ISC01 | 0<<ISC00 );
>
> Macht aber noch immer nicht, was es soll...
> Ich dachte, für diese einfache Anwendung tuts auch ein externer
> Interrupt mit Taster!

Wie du siehst, ist das eben nicht einfach.

Einfach wäre ein Pollen in der Hauptschleife mit Vergleich ob sich der 
zustand des Input-Pins verändert hat.

> Bei der "Komfortroutine" blick ich ehrlich gesagt
> nicht wirklich durch...

Das macht nichts. Dazu funktionieren sie zu gut.

PS: Welchen Prozessor hast du wirklich? Ein Mega 8 hat kein Register 
namens GIMSK

[Karl H. (kbuchegg)]

Im übrigen

>

> void eins() 
> {
>   PORTD=0b10000000;
> }
>

Gratuliere, du hast dir soeben den notwendigen Pullup Widerstand am Pin 
2 abgeschaltet.
Jetzt weißt du auch, warum man derartige Rundum-Portzuweisungen nicht 
macht.

[Michi R. (Gast)]

Grrr. Bekomm ich doch gleich die volle Breitseite.

Karl Heinz Buchegger schrieb:
> PS: Welchen Prozessor hast du wirklich? Ein Mega 8 hat kein Register
> namens GIMSK

Ich hab nen Atmega8! Komisch, dass er da nicht rummeckert. ;)

Karl Heinz Buchegger schrieb:
> Gratuliere, du hast dir soeben den notwendigen Pullup Widerstand am Pin
> 2 abgeschaltet.

Wie gesagt, bin ziemlicher Anfänger! Umso dankbarer bin ich für deine 
Korrektur.

[Karl H. (kbuchegg)]

Michi R. schrieb:
> Grrr. Bekomm ich doch gleich die volle Breitseite.
>
> Karl Heinz Buchegger schrieb:
>> PS: Welchen Prozessor hast du wirklich? Ein Mega 8 hat kein Register
>> namens GIMSK
>
> Ich hab nen Atmega8! Komisch, dass er da nicht rummeckert. ;)

Ist in der Tat interessant.
Könnte ein Hinweis darauf sein, dass deine Projekteinstellungen nicht 
korrekt sind.
Nichts desto trotz: ein Mega 8 hat laut Datenblatt kein GIMSK Register. 
Atmel wird ja wohl wissen, wie die Dinger heißen

[Michi R. (Gast)]

Wenn auch sehr unprofessionell geschrieben, so läuft das Programm 
tadellos:
Danke für die Vorschläge.

#define F_CPU 3686000UL
#include <util/delay.h>
#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/sleep.h>
int i=0;

ISR(INT0_vect)
{
   _delay_ms(750);
  i++;
}

void eins() 
{
   PORTD &= ~((1<<PD5) | (1<<PD6));
   PORTD |= (1<<PD7);
}

void zwei()
{
  PORTD &= ~((1<<PD7) | (1<<PD5));
  PORTD |= (1<<PD6);
}

void drei()
{
  PORTD &= ~((1<<PD6) | (1<<PD7));
  PORTD |= (1<<PD5);
}

void nichts()
{
  PORTD &= ~((1<<PD6) | (1<<PD7) | (1<<PD5));
}

int main(void)
{
  DDRD=0b11100000;
        PORTD |= ( 1<<PD2 );    //pull up aktivieren
  
  GICR = ( 1<<INT0 );
  MCUCR = ( (1<<SE) | (1<<SM1) | (0<<ISC01) | (0<<ISC00) );

  sei();
  
  
  while(1)
    {
    
    if (i==0) 
    {  
      eins();  
    }
    else if (i==1)
    {
      zwei();
    }
    else if (i==2)
    {
      drei();
    }
    else if (i==3)
    {
      nichts();
      set_sleep_mode(SLEEP_MODE_PWR_DOWN);
      sleep_mode();
    }  
    else if (i>=4)
      i=0;
  }
}

[Karl H. (kbuchegg)]

Michi R. schrieb:
> Wenn auch sehr unprofessionell geschrieben

In der Tat

> ISR(INT0_vect)
> {
>    _delay_ms(750);
>   i++;
> }

sowas ist Murks.

Auf der einen Seite benutzt du einen Interrupt, damit nur ja alles 
möglichst schnell reagiert (was eigentlich völlig sinnlos ist, denn für 
deinen µC bewegst du dich sowieso in Zeitlupe), auf der anderen Seite 
vertrödelst du dann genau hier die Zeit, damit dann ja alles, was der µC 
nebenher zu machen hätte (in einem richtigen Programm) zum Stillstand 
kommt.

So ist das Murks. Offen und ehrlich gesagt.
Einen durchgerosteten Kotflügel am Auto kann man auch mit Klebeband 
reparieren. Hält sogar eine Zeit lang. Murks ist es trotzdem.

[Michi R. (Gast)]

Das weiß ich doch. Ich kanns aber nicht besser. Außerdem ist das ja nur 
der Atmega8 auf dem Übungsboard, und ich will damit lernen.

[Karl H. (kbuchegg)]

Den Link zum Artikel
Entprellung
hab ich dir schon gegeben.

Ich seh natürlich ein, dass die Komfortroutinen in ihrer vollen Pracht 
nicht leicht zu durchschauen sind. Auf der anderen Seite hab ich dort 
für 1 Taster das ganze aufgerollt und eine einfacher zu durchschauende 
Version dafür geschrieben.

[spontan (Gast)]

Es sei Dir gestattet, daß Du es nicht besser kannst (Deine Aussage), daß 
Du aber lernen willst (auch Deine Aussgae) kann hier keiner erkennen. Du 
willst nur die LED leuchten sehen, auch wenn Du den größtmöglichen Murks 
ablieferst.

Lernen und Denken geht aber etwas anders.

[Karl H. (kbuchegg)]

Was ich dir allerdings zu gute halte ist, dass du den Interrupt brauchst 
um den µC damit aus dem Sleep heraus zu holen. Was aber nichts daran 
ändert, dass man Tastenabfragen wegen der Entprellung mittels Polling 
macht.

[Michi R. (Gast)]

> Du willst nur die LED leuchten sehen, auch wenn Du den größtmöglichen Murks
> ablieferst.

Es tut mir wirklich leid, dass mir das Freude bereitet. Wo ist denn das 
Problem, wenn ich mich langsam an die Sache rantasten möchte?

> Lernen und Denken geht aber etwas anders.

Gerne informiere ich dich über jeden einzelnen meiner Lernfortschritte.


kbuchegg, die Entprellung werde ich umgehend implementieren.

> Du aber lernen willst (auch Deine Aussgae) kann hier keiner erkennen.
---> jetzt aber! ;-)

[ich (Gast)]

Michi R. schrieb:
> ---> jetzt aber! ;-)

Geht doch! :-))

[Karl H. (kbuchegg)]

Wobei. Das muss ich heute am Abend direkt mal ausprobieren.
Meiner Meinung nach und ohne das jetzt laufen gelassen zu haben, dürfte 
diese Interrupt Lösung nicht funktionieren.
Das ist ein Level-Interrupt. Sobald die ISR betreten wird, wird das 
zugehörige Interrupt Flag erneut gesetzt, weil ja die Taste noch immer 
gedrückt ist, der Level also immer noch anliegt. Der delay_ms ändert 
daran nichts. Er sorgt mit seinen 750ms lediglich dafür, dass ich als 
Benutzer eine Chance habe, die Taste wieder loszulassen, ohne dass 
gleich alles Amok läuft. Aber die Schaltstufe um lediglich 1 zu erhöhen 
dürfte mir damit nicht gelingen. Mit einmal kurz drauf drücken erhöht 
sich die Stufe um mindestens 2. Das kann ich als Benutzer gar nicht 
verhindern.
Denke ich jetzt mal. Aber wie gesagt: das will ich tatsächlich mal 
ausprobieren, wie das jetzt wirklich ist. Denn da bin ich mir jetzt auch 
unsicher. Ich hab da noch im Hinterkopf, dass die Level Interrupts ein 
bisschen anders funktionieren als die anderen.

[Michi R. (Gast)]

Wie gesagt, bei mir läufts.

Ich dachte, das Flag ist "geschützt", solange die ISR läuft?

[Karl H. (kbuchegg)]

Michi R. schrieb:
> Wie gesagt, bei mir läufts.
>
> Ich dachte, das Flag ist "geschützt", solange die ISR läuft?

Nö.
Keines der Interrupt Request Flags ist in irgendeiner Art und Weise 
geschützt. Sobald die ISR ausgelöst wird, werden sie wieder gelöscht.
Wobei beim INTF0 Flag in der Doku steht

This flag is always cleared when INT0 is configured
as a level interrupt.

Von daher verhält sich das in dem Fall ein wenig anders.


Edit: Du redest vom globalen Interrupt Flag.
Aber der komplette Interrupt Mechanismus ist ein wenig komplexer als nur 
dieses 1 Flag. Es muss ja auch einen Mechanismus geben, der das 
Auftreten eines Ereignisses registriert und festhält, während die CPU 
gerade in einer ISR einen ganz anderen Interrupt bearbeitet. Das 
Auftreten dieses 2-ten Interrupt Ereignisses soll ja deswegen nicht 
verloren gehen.