Interrupt Problem am Tiny13 dopelte Dauer ?

[Marcel B. (gigi)]

Hallo,

habe hier ein kleines Programm für den Tiny geschrieben
und bemerke dass der Interrupt bzw die aufgerufene Funktion
10 anstatt 5 Sekunden dauert.

Also alarm() dauert 10 Sekundenm, ABER die funktion
blink() läuft so, wie sie soll.


Auch mit cli(); code sei(); in der if- Schleife ändert sich das 
Verhalten nicht.

#define F_CPU 128000UL


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


volatile uint8_t s;    

int alarm(void);  
int blink(void);  

ISR(INT0_vect)          // ISR - Put "switch" to 1
{
  s=1;
}


int main(void)
{
    DDRB = (1 << PB3);
  MCUCR = (1 << SE) | (1 << SM1) | (0 << SM0) | (0 << ISC00) | (0 << ISC01); // Sleep enabled - PwrDwn // LowLvlInt
  GIMSK = (1 << PCIE); // PCI Enable
  PCMSK = (1 << PCINT4); // PCI PB4
  
  int s = 0;
  
  sei();
  
  while(1)
    {
    
    if(s=1)
    {  cli();
      s=0;
      alarm();
      sei();
    }
    
    blink();
    sleep_mode();
  
    }
}

int alarm(void)
{
  
  s=0;
  PORTB =(0 << PB3);
  _delay_ms(5000);
  PORTB = (1 << PB3);
  
  
  return 0;
  
  
}

int blink (void)
{
  for(uint8_t i=0; i<3; i++)      //LED Signal, everything okay
  {
    PORTB = (0 << PB3);
    _delay_ms(1000);
    PORTB = (1 << PB3);
    _delay_ms(1000);
  }
  
  return 0;
}

[klapp (Gast)]

Marcel B. schrieb:
> Auch mit cli(); code sei(); in der if- Schleife ändert sich das
> Verhalten nicht.

If Schleifen und delays in der ISR macht man nicht.

[Nico W. (nico_w)]

Marcel B. schrieb:
> if(s=1)

Guck nochmal deinen Code an.

[Marcel B. (gigi)]

Hab ich ja auch nicht.

Ich rede von der If-Schleife in der Funktion alarm().

Nico W. schrieb:
> Marcel B. schrieb:
>> if(s=1)
>
> Guck nochmal deinen Code an.

Was ist daran falsch ?

[Werner P. (Gast)]

klapp schrieb:
> Marcel B. schrieb:
>> Auch mit cli(); code sei(); in der if- Schleife ändert sich das
>> Verhalten nicht.
>
> If Schleifen und delays in der ISR macht man nicht.

macht er doch nicht!

Denke es liegt eher an:

if(s=1)

[Nico W. (nico_w)]

Marcel B. schrieb:
> Was ist daran falsch ?

Guck mal ganz genau hin!

[Route_66 H. (route_66)]

klapp schrieb:
> If Schleifen und delays in der ISR macht man nicht.

If schleifen macht man grusätzlich nicht, weil es sowas nicht gibt.

Seine If-Abfrage macht er aber nicht im INT!
Heißt es nicht if(s==1)...?

[klapp (Gast)]

Marcel B. schrieb:
> Ich rede von der If-Schleife in der Funktion alarm().

Es gibt keine If-Schleife! Das Ding heißt zB If Abfrage aber niemals 
Schleife :-)

[Marcel B. (gigi)]

klapp schrieb:
> Marcel B. schrieb:
>> Ich rede von der If-Schleife in der Funktion alarm().
>
> Es gibt keine If-Schleife! Das Ding heißt zB If Abfrage aber niemals
> Schleife :-)

Stimmt!

Dieses == bricht mir noch den Hals! DANKE euch vielmals!

Allerdings blinkt es jetzt nur 1 Sekunde.

[Marcel B. (gigi)]

Und trotz cli() und sei() kann ich das blinken ewig aufrecht erhalten.

 
int alarm(void)
{
  cli();
  s=0;
  PORTB =(0 << PB3);
  _delay_ms(5000);
  PORTB = (1 << PB3);
  sei();
  
  return 0;
  
  
}

[Werner P. (Gast)]

Marcel B. schrieb:
> PORTB =(0 << PB3);
>   _delay_ms(5000);
>   PORTB = (1 << PB3);

Wundert mich eh dass da was blinkt ;-)

Bit setzen:

PORTB |= (1 << PB3);

Bit löschen:

PORTB &= ~(1 << PB3);

[Marcel B. (gigi)]

Werner P. schrieb:
> Marcel B. schrieb:
>> PORTB =(0 << PB3);
>>   _delay_ms(5000);
>>   PORTB = (1 << PB3);
>
> Wundert mich eh dass da was blinkt ;-)
>
> Bit setzen:
> PORTB |= (1 << PB3);
> Bit löschen:
> PORTB &= ~(1 << PB3);


Gut, leuchten, nicht blinken, lach.
Es leuchtet aber keine 5000ms, vielleich 2000.

[HildeK (Gast)]

Werner P. schrieb:
> Wundert mich eh dass da was blinkt ;-)

Doch, geht schon. Es hat eben die Wirkung wie
   PORTB = 0;
und
   PORTB = 8;

Es war zwar anders gemeint, wenn aber der PORTB sonst nicht verwendet 
wird, dann hat es an PB3 dieselbe Wirkung wie das was man üblicherweise 
macht und du auch geschrieben hast:

  PORTB &= ~(1 << PB3);
und
  PORTB |= (1 << PB3);

Beachte: er hatte nicht 'verodert', sondern nur zugewiesen!