Attiny13a deep sleep - external interrupt

[Nils L. (luene)]

Hallo,

ich habe hier einen Attiny 13A. Er läuft auf einem Steckbrett.
Es sind nur ein Taster und ein 433Mhz Sender angeschlossen, das ganze 
ist an einer Powerbank angeschlossen.

Ich möchte dass wenn der Taster betätigt wird ein 433Mhz-Signal gesendet 
(nennen wird es EIN) wird und wenn der Taster losgelassen wird ein 
anderes 433Mhz Signal (nennen wir es AUS) gesendet wird. In der 
Zwischenzeit soll der Controller in den deep sleep gehen.

Die Funkverbindung klappt.
Wenn ich den Taster betätige wird auch EIN gesendet.
Wenn ich den Taster loslasse wird MEISTENS auch AUS gesendet.
Sporadisch passiert es jedoch, dass beim loslassen erst nochmal EIN 
gesendet wird, und erst danach das für AUS (dazwischen ist 1 Sekunde 
Pause). Der Attiny geht dazwischen nicht in den deep-sleep.

Hat jemand eine Idee woran das liegen könnte?
Ich habe noch nie mit dem Deep sleep gearbeitet, denke dass ich hier 
irgendwas verbockt habe.
Danke schonmal!


Hier ist noch der Code dazu:

/*
 * Velux_Sensor_V1.c
 *
 * Created: 27.01.2021 10:26:57
 * Author : Nils
 */ 

// DEFINES
#define F_CPU 600000    //600 khz

#define DATA_TIME_ON  980
#define DATA_TIME_OFF 350 //ON+OFF == ~1300µs
//Pins
#define DATA_ON   PORTB |=  (1<<PB4)    // PIN B4
#define DATA_OFF PORTB &= ~(1<<PB4)    // PIN B4

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

// FUNCTION DECLARATION
void send_bit0();
void send_bit1();
void send_command(uint32_t code);
void doSleep();

// FUNCTIONS
int main(void)
{
  DDRB |=  (1 << DDB4);  //B4: 433Mhz Sender
  DDRB &= ~(1 << DDB3);  //B3: Taster
  PORTB |= (1 << PB3);  //B3: Pullup

  while(1){
    if ((!(PINB&(1<<PINB3)))){  //EIN
      send_command(0x555);
    }
    else{            //AUS
      send_command(0x552);
    }
    _delay_ms(1000);  //DELAY (DEBUG)
    doSleep();
  }
}

void doSleep(){
  GIMSK |= (1<<PCIE);                     // Enable Pin Change Interrupts
  PCMSK |= (1<<PCINT3);                   // Use PB3 as interrupt pin
  ADCSRA &= ~(1<<ADEN);                   // ADC off


  WDTCR |= (1<<WDP3 )|(0<<WDP2 )|(0<<WDP1)|(1<<WDP0); // 8s
  //WDTCR |= (1<<WDTIE);
  sei();  //INTERRUPT EIN
  set_sleep_mode(SLEEP_MODE_PWR_DOWN);    // replaces above statement
  sleep_mode();
  cli();  //INTERRUPT AUS
  return;
}

void send_command(uint32_t code){        //sendet Code
  for(int cycle = 0; cycle < 8;cycle++){    //8 mal senden
    for (int pos = 24; pos > 0; pos--){    //sendet 24Bit
      if (code & (1 << (pos-1)))        //prüft ob Bit an dieser Stelle 1 oder 0
        send_bit1();  //Sendet 1
      else
        send_bit0();  //Sendet 0
    }
    send_bit0();  //Sendet End-Bit
    _delay_ms(10);  //Pause vor dem nächsten Signal
  }
}

void send_bit1(){
  DATA_ON;
  _delay_us(DATA_TIME_ON);
  DATA_OFF;
  _delay_us(DATA_TIME_OFF);
}

void send_bit0(){
  DATA_ON;
  _delay_us(DATA_TIME_OFF);
  DATA_OFF;
  _delay_us(DATA_TIME_ON);
}

ISR(WDT_vect) {
  return; //LEER
}

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Nils L. schrieb:
> GIMSK |= (1<<PCIE);                     // Enable Pin Change
> Interrupts
>   PCMSK |= (1<<PCINT3);                   // Use PB3 as interrupt pin

Es lohnt sich, hier noch etwaige 'Pending' Interrupts zu löschen, bevor 
der MC schläft.

[Peter D. (peda)]

Nils L. schrieb:
> Hat jemand eine Idee woran das liegen könnte?

Der Taster prellt.

[S. Landolt (Gast)]

Und wo ist die Taster-, d.h. die PCINT-ISR?

[Lutz (Gast)]

Welche ISR? Er liest einfach jede Sekunde den Pin_Status ein...

[Lutz (Gast)]

Ich ziehe die Bemerkung wieder zurück!

[m.n. (Gast)]

Sehr stromsparend auch mit gedrücktem Taster: 
Beitrag "Re: EIN-AUS mit Taster per Interrupt, ATtiny25 o.ä."
Drücken und Loslassen mußt Du entsprechend anpassen.
Verwende einen Taster mit vergoldeten Kontakten oder Reedrelais. Dann 
ist den "Kontaktfreibrennern" ihr Argument genommen ;-)

[Peter D. (peda)]

S. Landolt schrieb:
> Und wo ist die Taster-, d.h. die PCINT-ISR?

Ohne ISR erfolgt das Aufwachen durch Sprung zum Resetvector (0x0000).
Der Interrupt ist ja freigegeben.

Hier ein Beispiel für Sleep mit Statemaschine:
Beitrag "Re: AVR Sleep Mode / Knight Rider"

[kannAllesBesser! (Gast)]

Nils L. schrieb:
> ISR(WDT_vect) {
>   return; //LEER
> }

... das ist auch keine leere ISR und das return ist völlig sinnfrei!
Hier wird fleißig push/pop der Register durchgeführt.

besser
#include interrupt.h
EMPTY_INTERRUPT (WDT_vect)

[Nils L. (luene)]

Peter D. schrieb:
> Der Taster prellt.

Daran habe ich auch schon gedacht. Ich weiß nur nicht, wie ich meinen 
Code anpassen muss, damit der µC so reagiert wie er soll. Der Taster 
soll später durch einen Neigungsschalter ersetzt werden. Also wird es 
nicht zu verhindern sein, dass es prellt.

Matthias S. schrieb:
> Es lohnt sich, hier noch etwaige 'Pending' Interrupts zu löschen, bevor
> der MC schläft.

Wie kann ich diese löschen?
Im Datenblatt habe ich

 GIFR |= (1<<PCIF);

gefunden um Interrupts zurückzusetzen. Kann ich das so in meinen Code 
einfügen, vor dem sleep_mode()?

Oder gehört das an eine andere Stelle?

Peter D. schrieb:
> Ohne ISR erfolgt das Aufwachen durch Sprung zum Resetvector (0x0000).
> Der Interrupt ist ja freigegeben.

Ich habe ja eine ISR. Oder ist das für den externen Interrupt die 
falsche? Löse ich mit meinem Interrupt ungewollt einen Reset aus?

[kannAllesBesser! (Gast)]

Nils L. schrieb:
> Ich habe ja eine ISR.

Lese erstmal das Datenblatt zum Thema Interrupts, nur mit raten kommst 
du nicht weit!

...
EMPTY_INTERRUPT (INT0_vect) //External Interrupt 0
EMPTY_INTERRUPT (PCINT0_vect) //Pin change Interrupt Request0

[S. Landolt (Gast)]

Nils L. schrieb:
> Ich habe ja eine ISR.
Für den Watchdog (der gar nicht aktiviert wird).

> Löse ich mit meinem Interrupt ungewollt einen Reset aus?
Genau. Damit wird dieses Einsekundendelay nicht erreicht, und das 
Tasterprellen tobt sich voll aus.

[Carl D. (jcw2)]

Nils L. schrieb:
> Peter D. schrieb:
>> Der Taster prellt.
>
> Daran habe ich auch schon gedacht. Ich weiß nur nicht, wie ich meinen
> Code anpassen muss, damit der µC so reagiert wie er soll. Der Taster
> soll später durch einen Neigungsschalter ersetzt werden. Also wird es
> nicht zu verhindern sein, dass es prellt.

Dein Code wird beim ersten Zappeln aufwachen, einen zufälligen Wert 
lesen, vermutlich oft den richtigen, aber eben nicht immer. Dann 
verbringt er knapp 900ms mit "Code senden", wobei das auch als 
Entprellung angesehen werden kann. In der Zeit konnte aber ein weiteres 
Zappeln am Pin einen weiteren PCInt0 anfordern, d.h. das zugehörige GIFR 
Bit setzen.

> Matthias S. schrieb:
>> Es lohnt sich, hier noch etwaige 'Pending' Interrupts zu löschen, bevor
>> der MC schläft.

Wenn man den Wunsch hat, daß der wegen Prellen während der Sendezeit 
anstehende Interrupt sich nicht auswirkt, dann sollte man das tun.

> Wie kann ich diese löschen?
> Im Datenblatt habe ich
>

 GIFR |= (1<<PCIF);

> gefunden um Interrupts zurückzusetzen. Kann ich das so in meinen Code
> einfügen, vor dem sleep_mode()?
Ja
> Oder gehört das an eine andere Stelle?
>
> Peter D. schrieb:
>> Ohne ISR erfolgt das Aufwachen durch Sprung zum Resetvector (0x0000).
>> Der Interrupt ist ja freigegeben.
>
> Ich habe ja eine ISR. Oder ist das für den externen Interrupt die
> falsche? Löse ich mit meinem Interrupt ungewollt einen Reset aus?

PCInt0_vect müßte der Richtige sein (oder nur PCInt_vect? das sagt mir 
in der Regel die IDE). Der ist aber eigentlich nur dazu da, den μC 
aufzuwecken und das PCIF Bit in GIFR zu löschen. Letzteres macht die HW 
selbstständig.


Trotz genereller Abneigung dagegen (und Hang zur PeDa-Entprellung) wäre 
hier ein RC-Tiefpaß die simpelste Lösung. Wartezeit zwischen den 
Schaltvorgängen hat man mehr als genug (fast 1s) und das einzige was 
erreicht werden muß, ist daß der Pegel am Pin nach dem ersten Interrupt 
für einige μs konstant bleibt. Und natürlich vor dem erneuten "warten 
auf Schaltvorgang" das PCIF löschen, um zwischenzeitliche Flanken zu 
"überlesen".

Oder eventuell doch mal PeDa-Entprellung ausprobieren und neue 
Erkenntnisse gewinnen.

[Nils L. (luene)]

ok, danke schonmal für euer Feedback!

Ich habe nun einiges angepasst:
- "ISR(WDT_vect)" habe ich durch "EMPTY_INTERRUPT(PCINT0_vect);" ersetzt
- nach dem Aufwachen habe ich ein 10ms Delay eingebaut, das als 
Entprellung gut funktioniert.
- mit "GIFR |= (1<<PCIF);" werden pending Interrupts zurückgesetzt (wenn 
es richtig verstanden habe, ist dies aber überhaupt nicht nötig, da 
"sei()" dieses zurücksetzt?)
- außerdem wird nach dem Senden eines Codes nochmal der Status des 
Eingangspin abgefragt. Wenn sich während des Sendens der Status am Pin 
ändern sollte,
wird nach einem 100ms-Delay der andere Code gesendet. Erst dann geht der 
µc schlafen.

Vielen Dank für eure Tipps!

Hier ist nochmal der überarbeitete Code:

// DEFINES
#define F_CPU 600000    //600 khz

#define DATA_TIME_ON  980
#define DATA_TIME_OFF 350 //ON+OFF == ~1300µs
//Pins
#define DATA_ON   PORTB |=  (1<<PB4)    // PIN B4
#define DATA_OFF PORTB &= ~(1<<PB4)    // PIN B4

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

// FUNCTION DECLARATION
void send_bit0();
void send_bit1();
void send_command(uint32_t code);
void doSleep();

// FUNCTIONS
int main(void)
{
  DDRB |=  (1 << DDB4);  //B4: 433Mhz Sender
  DDRB &= ~(1 << DDB3);  //B3: Taster
  PORTB |= (1 << PB3);  //B3: Pullup
  uint8_t state = 2;

  while(1){
    if (!(PINB&(1<<PINB3))){
      send_command(0x555);  //EIN
      state = 1;
    }
    else{
      send_command(0x552);  //AUS
      state = 0;
    }
    if ((!(PINB&(1<<PINB3))) == state){ //Status unverändert?
      doSleep();
      _delay_ms(10); //delay zum entprellen
    }
    else{
      _delay_ms(100); //Delay zwischen zwei Codes die gesendet werden
    }
  }
}

void doSleep(){
  GIMSK |= (1<<PCIE);                     // Enable Pin Change Interrupts
  PCMSK |= (1<<PCINT3);                   // Use PB3 as interrupt pin
  ADCSRA &= ~(1<<ADEN);                   // ADC off
  GIFR |= (1<<PCIF);  //Interrupts zurücksetzen

  sei();  //INTERRUPT EIN
  set_sleep_mode(SLEEP_MODE_PWR_DOWN);
  sleep_mode();
  cli();  //INTERRUPT AUS
  return;
}

void send_command(uint32_t code){        //sendet Code
  for(int cycle = 0; cycle < 8;cycle++){    //8 mal senden
    for (int pos = 24; pos > 0; pos--){    //sendet 24Bit
      if (code & (1 << (pos-1)))        //prüft ob Bit an dieser Stelle 1 oder 0
        send_bit1();  //Sendet 1
      else
        send_bit0();  //Sendet 0
    }
    send_bit0();  //Sendet End-Bit
    _delay_ms(10);  //Pause vor dem nächsten Signal
  }
}


void send_bit1(){
  DATA_ON;
  _delay_us(DATA_TIME_ON);
  DATA_OFF;
  _delay_us(DATA_TIME_OFF);
}

void send_bit0(){
  DATA_ON;
  _delay_us(DATA_TIME_OFF);
  DATA_OFF;
  _delay_us(DATA_TIME_ON);
}

EMPTY_INTERRUPT(PCINT0_vect);

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Nils L. schrieb:
> Im Datenblatt habe ich GIFR |= (1<<PCIF);
> gefunden um Interrupts zurückzusetzen. Kann ich das so in meinen Code
> einfügen, vor dem sleep_mode()?

Ja, das kannst du.

Nils L. schrieb:
> ist dies aber überhaupt nicht nötig, da
> "sei()" dieses zurücksetzt?

Das wäre mir neu. Wenn schon ein IRQ Flag gesetzt ist, und sei() 
ausgeführt wird, wird sofort dieser Interrupt bearbeitet. sei() setzt 
nichts zurück.