Interrupt Problem

[Peter (Gast)]

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

#define Takt      7372800
#define UART_Baudrate  9600
#define F_CPU      7372800


#define INT1_vect                       _VECTOR(2)
#define SIG_INTERRUPT1                  _VECTOR(2)


volatile int32_t a;

int32_t b;



ISR(INT1_vect)
{

b=3;
++a;

  if(a == 2)
    {
    PORTD |= (1<<PD5);
    loop_until_bit_is_clear(PIND, PD3);
    a=0;
    }

  PORTD &= ~(1<<PD5);

  sei();
}


int interrupt (void)
{
MCUCR= (1<<ISC11 ) | (1<<ISC10); // Interrupt steigende flanke
GICR= (1<<INT1);
}


int main (void)
{
DDRB = (1<<PB0) | (1<<PB1)| (1<<PB2)| (0<<PB3)| (1<<PB4)| (1<<PB5)| (1<<PB6)| (1<<PB7);
interrupt();
sei();


  while(1)
  {
    
  }
  
}

Hallo zusammen,
ich würde gerne erreichen das bei jedem dritten Interrupt PD5 ein high 
Signal ausgibt solange der Interrupt pin high ist. Leider scheint es so 
das die Variable a immer auf 0 zurückgesetzt wird und der uC immer a=1 
im Interrupt erkennt. An was liegt das ?
Gruß und Danke

[Peter II (Gast)]

Peter schrieb:
> Leider scheint es so
> das die Variable a immer auf 0 zurückgesetzt wird und der uC immer a=1
> im Interrupt erkennt. An was liegt das ?

vermutlich ist der µC nur so schnell das du es nicht siehst. Hast du 
einen Taster dran, wenn ja wird der interrupt wegen dem prellen mehrfach 
aufgerufen.

- a muss nicht volatile wenn sie nur im interupt verwendet wird, so 
sollte am besten static in der funktion liegen

- sei() in der ISR führt höchstes zu fehlern, macht aber bestimmt nicht 
das was du dir erfoffst.

- warten in der ISR ist eine sehr schlechte idee.

[Peter (Gast)]

danke für die Hinweise, dass meiste ist einfach nur aus Verzweiflung 
enstanden. Wenn ich den code so ändere:

  if(a == 1)
    {
    PORTD |= (1<<PD5);
    loop_until_bit_is_clear(PIND, PD3);
    a=0;
    }

seh ich das Signal mit dem oszi bei jedem Interrupt nur wenn die zahl 
höher wie eins ist springt er nicht mehr in die if schleife. Auch wenn 
ich a=0 lösche

[Peter II (Gast)]

Peter schrieb:
> nur wenn die zahl
> höher wie eins ist springt er nicht mehr in die if schleife.

if(a == 1)

warum sollte er das auch machen, wenn du auf 1 abfragst?

[Peter II (Gast)]

es gibt auch einen Simulator - teste es doch einfach dort drin, dann 
sieht du was er macht.

[Peter (Gast)]

Peter II schrieb:
>> nur wenn die zahl
>> höher wie eins ist springt er nicht mehr in die if schleife.
>
> if(a == 1)
>
> warum sollte er das auch machen, wenn du auf 1 abfragst?

Ich mein wenn ich eine andere Zahl dort einsetze.

[Peter II (Gast)]

dann zeig doch bitte richtigen code, wo auch die Frage dazu passt.

[Εrnst B. (ernst)]

Und mach mal in deine main was rein, damit du dauernde Resets erkennen 
kannst.
z.B. einen 1-Sekunden Wait am Anfang, währenddessen eine LED leuchtet.

Und du hast vermutlich dem GCC nicht mitgeteilt, welchen µC du 
verwendest, sonst wären deine INT1_ defines überflüssig.

Schalte die Warnings vom GCC ein, reparier alles bis der nix mehr zu 
meckern findet.

[Peter (Gast)]

Siehe erster Post :). Dort steht if(a == 3) und nichts funktioniert, bei 
if(a == 1) funktioniert es. Und das verstehte ich nicht weil, jede 50 ms 
ein Interrupt ausgelöst wird. Und a eigentlich von 0 bis 3 zählt und 
PD3 high wird und a wieder auf 0 zurückgesetzt. Aber es scheint so wie 
das in jedem sprung in den Interrupt a=0 ist, was ich nicht verstehe.

[Peter II (Gast)]

dann resetet eventuell dein µC wie schon von Εrnst B✶ vermutet.

[Peter (Gast)]

ok ich bastel mal was, danke

[Εrnst B. (ernst)]

Wie gesagt, reparier erstmal deinen Code. das "sei()" in der ISR ist 
z.B. sehr unschön, kann zu stack-overflows führen, und hat keinerlei 
nutzen.
Dann zeig einen Kompilierbaren Code, verrate uns, welcher µC.

[Peter (Gast)]

AVR Studio, Atmega16, Programmer AVRISPmk2

Es gibt keinen Reset eine LED mit einer sekunde wait leuchtet durch

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

#define Takt      7372800
#define UART_Baudrate          9600
#define F_CPU      7372800


#define INT1_vect                       _VECTOR(2)
#define SIG_INTERRUPT1                  _VECTOR(2)


int32_t a;





ISR(INT1_vect)
{

a++;

  if(a == 3)
    {
    PORTD |= (1<<PD5);
    loop_until_bit_is_clear(PIND, PD3);
    a=0;
    }

PORTD &= ~(1<<PD5);


}


int interrupt (void)
{
MCUCR= (1<<ISC11 ) | (1<<ISC10); // Interrupt steigende flanke
GICR= (1<<INT1);
}


int main (void)
{

DDRB = (1<<PB0) | (1<<PB1)| (1<<PB2)| (0<<PB3)| (1<<PB4)| (1<<PB5)| (1<<PB6)| (1<<PB7);
interrupt();
sei();

  while(1)
  {
    
  }
  
}

[Karl H. (kbuchegg)]

Und schmeiss um Himmels willen die beiden

#define INT1_vect                       _VECTOR(2)
#define SIG_INTERRUPT1                  _VECTOR(2)

raus!

Wenn da etwas nicht stimmt, dann willst du eine Fehlermeldung vom 
Compiler und nicht selbst reparieren und Fehler machen.

[Karl H. (kbuchegg)]

>    PORTD |= (1<<PD5);

wie gibst du denn auf einem Portpin etwas aus, der auf Eingang gestellt 
ist?

Und was hängt an deinem Eingangspin PD3? Liegt da ein sauberes Signal 
mit sauberen Flanken an oder muss da mit Prellen gerechnet werden?


> int32_t a;

es gäbe da auch noch int64_t!
Hat ja sonst nichts zu tun, der kleine µC. Um bis 3 zu zählen muss man 
schon ordentlich Bits spendieren.

[Peter (Gast)]

Karl Heinz Buchegger schrieb:
>>    PORTD |= (1<<PD5);
>
> wie gibst du denn auf einem Portpin etwas aus, der auf Eingang gestellt
> ist?
>
>
>
>> int32_t a;
>
> es gäbe da auch noch int64_t!
> Hat ja sonst nichts zu tun, der kleine µC. Um bis 3 zu zählen muss man
> schon ordentlich Bits spendieren.

Ganz ruhig, ich sag doch ich bin verzweifelt :).

> Und was hängt an deinem Eingangspin PD3? Liegt da ein sauberes Signal
> mit sauberen Flanken an oder muss da mit Prellen gerechnet werden?

 Ein schmitt trigger, saubere Steile 5V flanken.

-.-

code update

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

#define Takt      7372800
#define UART_Baudrate  9600
#define F_CPU      7372800

int8_t a;


ISR(INT1_vect)
{

a++;

  if(a == 3)
    {
    PORTD |= (1<<PD5);
    loop_until_bit_is_clear(PIND, PD3);
    a=0;
    }
PORTD &= ~(1<<PD5);
}


int interrupt (void)
{
MCUCR= (1<<ISC11 ) | (1<<ISC10); // Interrupt steigende flanke
GICR= (1<<INT1);
}


int main (void)
{

DDRD = (1<<PD0) | (0<<PD1)| (0<<PD2)| (0<<PD3)| (0<<PD4)| (1<<PD5)| (0<<PD6)| (0<<PD7);
DDRC=0xFF;
DDRB = (1<<PB0) | (1<<PB1)| (1<<PB2)| (0<<PB3)| (1<<PB4)| (1<<PB5)| (1<<PB6)| (1<<PB7);
interrupt();
sei();

  while(1)
  {
    
  }
  
}

[Εrnst B. (ernst)]

Der GCC hat noch ein paar Anmerkungen:

 avr/include/util/delay.h:89:3: Warnung: #warning "F_CPU not defined for 
<util/delay.h>"

 avr/include/avr/signal.h:36:2: Warnung: #warning "This header file is 
obsolete.  Use <avr/interrupt.h>."

Warnung: »F_CPU« redefiniert

In Funktion »interrupt«:
 Warnung: control reaches end of non-void function

[Εrnst B. (ernst)]

Das hier macht der GCC bei mir aus deiner ISR. Schaut korrekt aus.

.global __vector_2
        .type   __vector_2, @function
__vector_2:
        push __zero_reg__
        push r0
        in r0,__SREG__
        push r0
        clr __zero_reg__
        push r24
/* prologue: Signal */
/* frame size = 0 */
/* stack size = 4 */
.L__stack_usage = 4
        lds r24,a
        subi r24,lo8(-(1))
        sts a,r24
        cpi r24,lo8(3)
        breq .L6
        cbi 50-0x20,5
/* epilogue start */
        pop r24
        pop r0
        out __SREG__,r0
        pop r0
        pop __zero_reg__
        reti
.L6:
        sbi 50-0x20,5
.L3:
        sbic 48-0x20,3
        rjmp .L3
        sts a,__zero_reg__
        cbi 50-0x20,5
/* epilogue start */
        pop r24
        pop r0
        out __SREG__,r0
        pop r0
        pop __zero_reg__
        reti
        .size   __vector_2, .-__vector_2

[Peter (Gast)]

Jetzt nicht mehr

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

int16_t a;

ISR(INT1_vect)
{

a++;

  if(a == 2)
    {
    PORTD |= (1<<PD5);
    PORTC |= (1<<PC0);
    loop_until_bit_is_clear(PIND, PD3);
    a=0;
    }

PORTD &= ~(1<<PD5);

}


int interrupt (void)
{
MCUCR= (1<<ISC11 ) | (1<<ISC10); // Interrupt steigende flanke
GICR|= (1<<INT1);

return 0;
}


int main (void)
{

DDRD = (1<<PD0) | (0<<PD1)| (0<<PD2)| (0<<PD3)| (0<<PD4)| (1<<PD5)| (0<<PD6)| (0<<PD7);
DDRC=0xFF;
DDRB = (1<<PB0) | (1<<PB1)| (1<<PB2)| (0<<PB3)| (1<<PB4)| (1<<PB5)| (1<<PB6)| (1<<PB7);
interrupt();
sei();

  while(1)
  {
    
  }
  
}

[Peter (Gast)]

hmm der Controller ist es auch nicht hab ihn grad getauscht. Alles sehr 
merkwürdig

[Εrnst B. (ernst)]

mach mal testweise aus
if(a == 2)

ein
if(a >= 2)

und probiers nochmal.

Aber eigentlich: Der generierte ASM-Code schaut korrekt aus.

[Peter (Gast)]

Leider keine Veränderung. Kann der Uc reseten ohne das man an den 
Augangspins bemerkt?

[Peter II (Gast)]

Peter schrieb:
> Leider keine Veränderung. Kann der Uc reseten ohne das man an den
> Augangspins bemerkt?

kommt auf das Progamm an, das "reseten" geht ja in ein paar µC in der 
Zeit sind die Ausgänge halt Eingänge das kann man aber schlecht von 
aussen feststellen.

Noch mal die Frage, warum startest du nicht einfach mal den simulator?

[Pipeline (Gast)]

Peter schrieb:
> int16_t a;
>
> ISR(INT1_vect)
> {
>
> a++;

mit welchem Wert ist a vorbelegt?

ISR(INT1_vect)
{
   static int16_t a = 0;

   a++;
...

[Pipeline (Gast)]

Peter schrieb:
> if(a == 2)

ach ja, und bei Zählern

   if(a>=2)
...

oder einen rettendes

else

als default Zweig

[Peter (Gast)]

Das Problem war, dass im AVR Studio ein Atmega 32 eingestellt war, aber 
ein Atmega 16 benutzt wurde. Danke für die viele Hilfen

Gruß Peter