Microchip/Atmel AVR ATtiny85, Pin-change interrupt (PCINT0) ) und Watchdog als Timer (WDT)

[Lasse Klingeln (Gast)]

Der folgende Code läuft auf einem "Digispark" Klon mit ATtiny85 bei 5V 
extern (USB ist nicht verbunden).
Wenn ich INPUT mit NC2 low ziehe, schaltet OUTPUT für 204 ms ein - nicht 
für 8 Sekunden.
Stimmt etwas mit der Watchdog-Konfiguration nicht, oder woran kann's 
noch liegen?

// Microchip / Atmel AVR ATtinyx5
// 

//#define F_CPU 1000000 // 8MHz/8 CLKDIV8 (default, fuse sets prescaler in CLKPR, can be overwritten)
#define F_CPU 16500000 // 16.5 MHz: PLL overclocked by micronucleus USB bootloader (USB sync.)


// C types
#include <stdint.h>

// AVR-libc
#include <avr/io.h>
#include <avr/sleep.h>
#include <avr/interrupt.h>
#include <avr/wdt.h>
#include <util/delay.h>


// IO
//  #RESET PB5 1   8 VCC
//         PB3 2   7 PB2
//         PB4 3   6 PB1
//         GND 4   5 PB0
#define NC0       (1<<PB0)  // 5 (MOSI/DI/SDA/AIN0/OC0A/#OC1A/AREF/PCINT0)
#define LED       (1<<PB1)  // 6 (MISO/DO/AIN1/OC0B/OC1A/PCINT1)
#define NC2       (1<<PB2)  // 7 (SCK/USCK/SCL/ADC1/T0/INT0/PCINT2)
#define INPUT     (1<<PB3)  // 2 (PCINT3/XTAL1/CLKI/#OC1B/ADC3)
#define OUTPUT    (1<<PB4)  // 3 (PCINT4/XTAL2/CLKO/OC1B/ADC2)
//                          // 1 (PCINT5/#RESET/ADC0/dW)
//                             4 GND
//                             8 VCC


ISR(PCINT0_vect) {
  GIMSK = 0;          // stop PCIE
  PORTB |= (LED | OUTPUT);    // debug
  WDTCR = (1<<WDIE) | WDTO_8S;  // enable WDT
}


ISR(WDT_vect) {
  wdt_disable();      // stop WDT
  PORTB &= ~(LED | OUTPUT); // debug
  GIMSK = (1<<PCIE);    // enable interrupt
}


int main(void) {
   
  MCUSR = 0;  // reset status
  GIFR = 0xFF;  // clear interrupt flags
  
  // low power
  wdt_disable();    // stop WDT
  sleep_bod_disable();  // BOD
  ACSR |= (1<<ACD);    // analog comparator
  PRR |= (1<<PRTIM0) | (1<<PRTIM1) | (1<<PRUSI) | (1<<PRADC);

  // output(s)
  DDRB = (LED | OUTPUT | NC0 | NC2); //
  PORTB = 0;
  
  // hello world
  for (uint8_t n=0; n<6; n++) {
  PORTB ^= LED;
  _delay_ms(200);
  }

  // enable pin change interrupt
  GIFR |= (1<<PCIF);  // the flag can be cleared by writing a logical one to it
  GIMSK = (1<<PCIE);  // 
  PCMSK = (1<<PCINT3);  // 

  // power down
  set_sleep_mode(SLEEP_MODE_PWR_DOWN);
  sleep_enable();
  sei();
  while (1) {
    sleep_cpu();
  //sleep_disable();
  }
}

[Lasse Klingeln (Gast)]

Mit WDTO_4S reduziert sich die Zeit auf 85 ms.
Die Zeit zwischen Flanke und LED sind über 50 ms!

[Einer K. (Gast)]

Lasse Klingeln schrieb:
> WDTCR = (1<<WDIE) | WDTO_8S;  // enable WDT

Warum nutzt du hier nicht das vorgesehene Macro.
An anderen Stellen tust du das doch auch.

[Lasse Klingeln (Gast)]

Arduino Fanboy D. schrieb:
> Warum nutzt du hier nicht das vorgesehene Macro.

Es gibt ein Makro, um den WDT als Timer zu verwenden? Welches?

https://www.nongnu.org/avr-libc/user-manual/group__avr__watchdog.html

[Lasse Klingeln (Gast)]

Gleiches Verhalten mit

  wdt_enable(WDTO_4S);      // enable WDT
  WDTCR = (1<<WDIE);      // WDT for interrupt

[Lasse Klingeln (Gast)]

Oops.. Moment.

[Lasse Klingeln (Gast)]

  wdt_enable(WDTO_4S);      // enable WDT
  WDTCR |= (1<<WDIE);      // WDT for interrupt

8.8 s

  wdt_enable(WDTO_8S);      // enable WDT
  WDTCR |= (1<<WDIE);      // WDT for interrupt

17.5 s

Läuft er doppelt?

[Frank M. (ukw) (Moderator)]

Lasse Klingeln schrieb:
> #define F_CPU 16500000

Wahrscheinlich läuft Dein µC nur mit 8 MHz. Mit obiger Zeile kannst Du 
nicht die Taktfrequenz vorgeben.

Umgekehrt wird ein Schuh daraus: Du musst hier angeben, mit welchem Takt 
Dein AVR tatsächlich läuft.

[Lasse Klingeln (Gast)]

Vor dem Wiedereinschalten des PCINT noch

  GIFR |= (1<<PCIF);

oder über PCMSK ein- & ausschalten.

[Lasse Klingeln (Gast)]

Frank M. schrieb:
> Wahrscheinlich läuft Dein µC nur mit 8 MHz.

Die PLL ist gefust, der Bootloader ("micronucleus") setzt den Takt wohl 
synchron zur USB.
Hat aber eigentlich nichts mit dem WDT zu tun (128kHz RC)?

[Thomas E. (thomase)]

Lasse Klingeln schrieb:
> WDTCR = (1<<WDIE) | WDTO_8S;  // enable WDT

Das geht nicht.

Auf das WDTCR kann nur in einer Timed Sequence zugegriffen werden. Mach 
das wie im Datenblatt beschrieben oder benutze die Makros.

[Lasse Klingeln (Gast)]

Also:
Fehler 1: WDT nicht korrekt konfiguriert (Schreibsequenz per Makro)
Fehler 2: WDT lief doppelt, weil der PCINT nach Ablauf von WDT sofort 
wieder ausgeführt wurde.
50..60 ms "Latenz" zwischen Flanke und Schalten des Ausgangs: PLL 
start-up time

[Lasse Klingeln (Gast)]

Thomas E. schrieb:
> Auf das WDTCR kann nur in einer Timed Sequence zugegriffen werden. Mach
> das wie im Datenblatt beschrieben oder benutze die Makros.

Im Datenblatt steht dazu "keine Einschränkung", wenn WDTON nicht 
aktiviert ist.
Zum Löschen von WDE ist dagegen die Sequenz erforderlich.

Neuer Versuch, anderer Wert:

WDTCR |= (1<<WDIE) | WDTO_2S;

Geht. 8-D

[Lasse Klingeln (Gast)]

Lasse Klingeln schrieb:
> Geht. 8-D

Geht nicht. Irgendwas stimmt hier wohl mit Atmel Studio nicht.

[Einer K. (Gast)]

Lasse Klingeln schrieb:
> Irgendwas stimmt hier wohl mit Atmel Studio nicht.

Vielleicht auch mit der Badehose :-)

Hier eine Variante mit der Arduino IDE.
Ein paar include hinzufügen, dann müsste es auch mit dem Atmel Studio im 
8 Sekunden Takt blinken.

Getestet mit einem 16,5MHz Digistump Klon.
Die Eingebaute LED hängt an PB1

#include <avr/wdt.h>


int main(void) 
{
  wdt_enable(WDTO_8S);  
  DDRB |= _BV(PB1);   // led output
  WDTCR |= 1<<WDIE;    
  sei();
  for(;;){}
}


ISR(WDT_vect) 
{
  WDTCR |= 1<<WDIE;  
  PINB = _BV(PB1); // toggle led
}

[Thomas E. (thomase)]

Lasse Klingeln schrieb:
> Geht nicht.

Habe dir doch gesagt, daß das nicht geht.

Aber du hast ja trotzdem eine Lösung gefunden:

> Irgendwas stimmt hier wohl mit Atmel Studio nicht.

[Lasse Klingeln (Gast)]

Thomas E. schrieb:
> Habe dir doch gesagt, daß das nicht geht.

Wie im vagen Datenblatt beschrieben, geht es leider auch nicht:

    wdt_reset();
    WDTCR |= (1<<WDCE) | (1<<WDE);
    WDTCR = (1<<WDIE) | WDTO_4S;


Es gibt kein Makro für den WDT als Timer.
Bislang funktioniert es bei mir nur so, wie um 11:55 geschrieben, also 
WDT enable Makro und dann umschalten auf Interrupt:

    wdt_enable(WDTO_4S);
    WDTCR |= (1<<WDIE);

Arduino Fanboy D. schrieb:
> Hier eine Variante mit der Arduino IDE.

Danke, so läuft's auch bei mir.

[Einer K. (Gast)]

Lasse Klingeln schrieb:
> Danke, so läuft's auch bei mir.
Ich weiß.

Und: Gerne geschehen.