Attiny45 Interrupt in c

[basti (Gast)]

Hi,
ich kämpfe gerade mit dem folgenden Code in c für den Attiny45.
Beim Betätigen eines Pull-up-Tasters soll die Led für zwei Sekunden 
leuchten.
Sie leuchtet auch, sowohl beim Drücken als auch bei stark verzögertem 
Loslassen (durch fehlendes Debouncing), aber sie lässt sich zwischen 
Drücken und leuchten gut 1/2 Sekunde Zeit...
Könnt ihr mir meinen Fehler zeigen?

#define F_CPU 8000000UL

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




int warten = 1;


int main(void)
{
  DDRB = (1 << PORTB1) | (0 << PORTB2);
  MCUCR = (1 << ISC01) | (1 << ISC00);
  GIMSK = (1 << INT0);
  
    while (1) 
    {
    sei();
    if (warten == 0)
    {
      cli();
      _delay_ms(2000);
      PORTB &= ~(1 << PORTB1);
    }
    }
}

ISR(INT0_vect)
{
  PORTB |= (1 << PORTB1);
  warten = 0;
}

[Trumpeltier (Gast)]

Wo fragst Du den Taster ab?

[basti (Gast)]

Am PINB2.

[Stefan F. (Gast)]

Mach die Abfrage mal ohne Interrupt, so kannst du relativ leicht einen 
Hardwarefehler ausschließen.

[basti (Gast)]

Das habe ich bereits gemacht, es funktioniert problemlos.

[Uwe D. (monkye)]

Ich bin jetzt kein C Spezi, aber wenn Du während des

 _delay_ms(2000);

 drückst, dann wird nach der ISR doch weiter gewartet, oder??

[Trumpeltier (Gast)]

Dein Taster prellt vermutlich so stark, daß Dein Programm sofort wieder 
in den Interrupt spring.

[Frank (Gast)]

Hallo
ich glaube du musst deine Variable als volatile kennzeichnen.
Ansonsten fällt die Werteänderung der "Optimierung" zum Opfer
und dein Hauptprogramm bekommt von der Änderung nichts mit.

volatile int warten = 1;

[basti (Gast)]

Hm, ich weiß nicht genau was du meinst. Während des _delay_ms sind ja 
alle Interrupts ausgeschaltet, da tut sich also nichts. Wenn ich 
gedrückt halte bis die 2s vorbei sind, bleibt die LED aus, da nur eine 
steigende Flanke erkannt wird.

[basti (Gast)]

Probiere ich mal, ohne das vollständig verstanden zu haben :)

[Mattias (Gast)]

Habe mal im Code ein paar Kommentare in den Code gemacht. Schau es dir 
mal an und überleg dir mal wie du die Variable wann setzt.
basti schrieb:
> int warten = 1; //1. hier wird auf 1 initialisiert, diese variable wird im 
interrupt verändert. Dann muss noch ein volatile davor
>
> int main(void)
> {
>   // initialisieren
>     while (1)
>     {
>     sei(); // interrupts einschalten
>     if (warten == 0)
>     {
>       cli(); // interrupts aus..
>       _delay_ms(2000);//2sek warten .. ok
>       PORTB &= ~(1 << PORTB1); // pin low.. ok
>     }
>     }
> }
>
> ISR(INT0_vect)
> {
>   PORTB |= (1 << PORTB1); // pin high.. ok
>   warten = 0; // auf null setzen.. wann wird diese wieder 1??
> }

Viele Grüße
Mattias

[basti (Gast)]

Danke Mattias, die Kommentare habe ich vergessen.
Ich habe bereits die Variable in "volatile int" geändert, das hat noch 
nicht geholfen. "warten" muss natürlich am Ende der if-Schleife auf 0 
gesetzt werden, das probiere ich mal.

[basti (Gast)]

Ich meine natürlich dass "warten" auf eins gesetzt wird.

[Mattias (Gast)]

basti schrieb:
> Ich meine natürlich dass "warten" auf eins gesetzt wird.

Darauf wollte ich hinaus.

Viele Grüße
Mattias

[basti (Gast)]

Jetzt klappt alles perfekt, klasse :D
Danke für eure Hilfe, ich werd' mich dann mal zum Thema "volatile" 
Variablen schlau machen.

[Trumpeltier (Gast)]

Oops, die LED sollte auch dann sofort leuchten. Ich glaube, die Grippe 
vernebelt mich komplett.
Setze mal Matthias diffizielen Hinweis um, und nenne eine If-Bedingung 
nicht Schleife.

[Peter II (Gast)]

basti schrieb:
> Während des _delay_ms sind ja
> alle Interrupts ausgeschaltet, da tut sich also nichts.

dein ganz bestimmt nicht.

[basti (Gast)]

Kommt nicht wieder vor, und gute Besserung!

[Mattias (Gast)]

Das volatile sagt dem Compiler, dass die CPU die Variable jedes mal vom 
RAM lesen muss bevor er was damit anstellt. Und nicht einen 'alten' Wert 
aus dem Register zum Vergleich nutzt.

Viele Grüße
Mattias

[basti (Gast)]

Ok, das ergibt Sinn, wenn vom Interrupt aus darauf zugegriffen wird.