Problem mit INT0 am ATmega16

Ralf wrote:
> delay.h braucht noch den Systemtakt (#define F_CPU 8000000)
> In einer ISR brauchst du den Interrupt nicht zusätzlich disablen...das
> macht der compiler für dich.
Falsch, das macht die Hardware. Der Compiler hat damit nichts zu tun. 
Ein sei() in einem Interrupt-Handler (auch am Ende) kann sogar Probleme 
verursachen. Schließlich ist die unscheinbare schließende Klammer am 
Ende der ISR in Wirklichkeit viel mehr als nur eine einfache Klammer...

> Deswegen ist auch das Delay in der ISR noch schlimmer als es ohnehin
> schon ist.
Ohne das sei() am Ende wäre es gar nicht so tragisch, da in diesem 
Falle nur ein einziger Interrupt aktiv ist. Durch das Prellen des 
Tasters wird aber vermutlich das Interrupt-Flag während des Delays schon 
wieder gesetzt und noch bevor der Controller am Ende der ISR die 
Register zurückschreiben und aus dem Handler rausspringen kann, wird der 
Interrupt erneut auseglöst (eben durch das sei()). Das kann durchaus 
knallen...

> Andere ISR werden in diesen 3Sekunden einfach nicht
> aufgerufen.
Andere gibt es in diesem Falle auch nicht.

> Setz doch lieber in der ISR nur ein flag was  du ausserhalb der ISR
> auswertest.
Das sollte man generell machen, auch wenn es in diesem Fall sogar so 
gehen müsste.

Ralf wrote:
> ...welcher Taster?
Sorry, kommt davon, wenn man an mehreren Problemstellungen gleichzeitig 
arbeitet...

Weiß der AVR auch, dass er mit 16 MHz laufen soll? Thema [[AVR 
Fuses]]...

Rush wrote:
> Ich hätte es eben gerne digital ;-) Analog hab ich auch schon dran
> gedacht aber mag ich nicht.
Schon mal an den Mittelweg gedacht? Der ATMega16 hat auch einen 
eingebauten Analog-Komparator. Da bist Du selbst Herr der 
Schaltschwellen und die Wahrscheinlichkeit, dass es zuverlässig klappt, 
ist größer...

ADC != Analog-Komparator
Das sind zwei völlig unterschiedliche Dinge.

@ Rush

> wenn ich den Fototransistor jetzt kurzzeitig  beleuchte und gleich
> wieder abdunkle geht die LED zwar an, bleibt aber anstatt von 3 sekunden
> ca 7 sekunden an, wieso?

Zwei INT0 hintereinander?

@ Ralf

> Deswegen ist auch das Delay in der ISR noch schlimmer als es ohnehin
> schon ist. Andere ISR werden in diesen 3Sekunden einfach nicht
> aufgerufen. Setz doch lieber in der ISR nur ein flag was du ausserhalb
> der ISR auswertest.

Sehe ich genauso. Als Flag kann man die LED selbst nehmen.

1

// MCU ist Atmega16

2

// F_CPU ist im Projekt/Makefile auf 16000000L definiert

3

#include <avr/io.h>

4

#include <util/delay.h>

5

#include <avr/interrupt.h>

6

7

8

ISR(INT0_vect)

9

{

10

  PORTD &= ~(1<<PD4); // active-low LED AN

11

}

12

13

14

void init(void)

15

{

16

  DDRD |= (1<<PD4);

17

  PORTD |= (1<<PD4); // active-low LED AUS

18

  PORTD &= ~(1<<PD2);

19

/*

20

Table 35. Interrupt 0 Sense Control

21

ISC01 ISC00 Description

22

  0     0   The low level of INT0 generates an interrupt request.

23

  0     1   Any logical change on INT0 generates an interrupt request.

24

  1     0   The falling edge of INT0 generates an interrupt request.

25

  1     1   The rising edge of INT0 generates an interrupt request.

26

*/

27

  MCUCR |= (1<<ISC01) | (1<<ISC00); // rising edge of INT0

28

  GICR |= (1<<INT0);

29

  sei();

30

}

31

32

33

void main(void)

34

{

35

  init();

36

37

  while (1)

38

  {

39

    if ( !(PORTD & (1<<PD4)) ) // Ist active-low LED AN?

40

    {

41

      // hier Code für Auslöser einfügen

42

      // ...

43

      _delay_ms(3000);   // 3s Leuchten LED

44

      PORTD |= (1<<PD4); // active-low LED AUS

45

    }

46

  }

47

}