Problem mit dem Watchdog beim ATtiny44

[H. G. (ledi)]

Hallo!

Ich habe ein Problem mit dem Watchdog bei einem langen Tastendruck.
Bei einem Tastendruck wird die ISR0 ausgelöst und entsprechend 
ausgewertet.

Unter main habe ich folgenden Code:

void main(void)
{   
  SPI_Master_Init();            // Init SPI & MCU Settings
  Init_ADC();                   // Init ADC
  Init_Interrupt ();            // Init Interrupt 0

  wdt_enable(WDTO_4S);          // Enable watch-dog-timer 4 seconds

  MCUCR  |= (1<<SE);            // Sleep enable
  set_sleep_mode(SLEEP_MODE_PWR_DOWN);  // Sleep Mode vorbereiten
    
  while(1)
    {  
    wdt_reset();                // reset watchdog
    ADCSRA   &= !(1<<ADEN);     // ADC deaktivieren
    wdt_disable();
    sleep_mode();
    wdt_enable(WDTO_4S);        // Enable watch-dog-timer 4 seconds
    }
}

Hat wer eine Idee, woran es liegen könnte?

Danke!

[Peter D. (peda)]

Nicht alles zugleich, sondern nacheinander entwickeln:

1.Schritt: die Applikation schreiben und testen.
2.Schritt: falls gewünscht, Sleepmode hinzufügen und testen
3.Schritt: falls gewünscht, Watchdog hinzufügen und testen.


Peter

[avrGerd (Gast)]

Wenn Du Probleme in der ISR hast, dann solltest Du uns den Code schon 
mal zeigen, sonst wird es schwer mit Raten.

Welches Problem hast Du genau? Löst der Watchdog aus?


avrGerd

[H. G. (ledi)]

Bis auf den Watchdog, funktioniert alles wunderbar.

Meine ISR:

ISR(INT0_vect)
{
    MCUCR  &= ~(1<<SE);      // Sleep disable
    ADCSRA   |= 1<<ADEN;        // ADC aktivieren
    wdt_enable(WDTO_4S);      // Enable watch-dog-timer 4 seconds
  
    char ADC_Wert, cube_address;  

    ADCSRA |= (1<<ADSC);            // Start ADC-Converter
      while (ADCSRA & (1<<ADSC));    // wait for ADC-Conversion
    
    //ADC_Wert = ADCH;        // ADCH = ADLAR High-Byte Register f. linksbündig
                    // wenn nur 8 Bit benötigt werden
    ADC_Wert = ADC;  
....
....
....

[Peter D. (peda)]

Heimo G. schrieb:
> ....
> ....
> ....

Ohne Worte.


Peter

[avrGerd (Gast)]

Tja, er mag halt keine Hilfe ...

avrGerd

[Uwe (Gast)]

WOFÜR benutzt du denn den Watchdog ?!?!!

[H. G. (ledi)]

Hier die gesamte ISR:

ISR(INT0_vect)
{
    wdt_enable(WDTO_4S);      // Enable watch-dog-timer 4 seconds
    MCUCR  &= ~(1<<SE);      // Sleep disable
    ADCSRA   |= 1<<ADEN;        // ADC aktivieren
    
    char ADC_Wert, cube_address;  

    ADCSRA |= (1<<ADSC);            // Start ADC-Converter
      while (ADCSRA & (1<<ADSC));    // wait for ADC-Conversion
    
    //ADC_Wert = ADCH;        // ADCH = ADLAR High-Byte Register f. linksbündig
                    // wenn nur 8 Bit benötigt werden
    ADC_Wert = ADC;  

    //Taste = checkbutton(ADC_Wert);// Übergibt ADCL an checkbutton
    checkbutton(ADC_Wert);      // Übergibt ADCL an checkbutton

    _delay_ms(40);


    // Routine um Bit 6 (RX-FIFO Int.-Flag) im Statusregister zu löschen
    Payload(0x07, 0x00);      // Read Status Register
    if (0x40 & USIDR)        // if Bit6 = 1
    {
      Payload(0x27, 0x40);    // Write Data --> clear bit 6 (data ready RX FIFO interrupt)
    }

    // Routine um Bit 4 (Max. number of TX-Retransmitts Int.-Flag) im Statusregister zu löschen
    Payload(0x07, 0x00);      // Read Status Register
    if (0x10 & USIDR)        // if Bit4 = 1 (max. number of retransmission interrupt)
    {
      Payload(0x27, 0x10);    // Write Data --> clear bit 4 (max. RT interrupt)
    }

    Payload(0x07, 0x00);      // Read Status Reg.
    
    if (0x20 & USIDR)        // If ACK received
    {
      Payload(0x27, 0x20);    // Clear ACK (Bit 5)
      
      // if button 11 pressed long and write_counter < 128 write new cube address  
      if ((address_request == 1) && (write_counter < 128))  
      {
        //PORTA |= (1<<PA7);  // LED ON
        //PORTA |= (1<<PA2);  // Oscilloscope test point --> Start Rx-mode

        switch_to_Rx_mode();

        _delay_ms(40);

        CS_low();
        SPI_transfer(0x61);                    // Command read FIFO payload
        SPI_transfer(0x00);                    // 1. dummybyte
        cube_address = USIDR;                  // Read 1. byte
        lsb_address = cube_address;
        eeprom_write_byte(EEPROM_write_address, cube_address);  // Write cube address low byte into EEPROM
        EEPROM_write_address ++;

        SPI_transfer(0x00);                    // 2. dummybyte
        CS_high();
        cube_address = USIDR;                  // Read 2. byte
        eeprom_write_byte(EEPROM_write_address, cube_address);  // Write cube address high byte into EEPROM
        EEPROM_write_address ++;

        eeprom_write_byte(EEPROM_write_address, 0x00);      // Write 0x00 as last byte into EEPROM

        Payload(0xE2, 0x00);                  // clear RX-FIFO
        
        //PORTA &= !(1<<PA2);  // Oscilloscope test point --> Finish Rx-mode

        switch_to_Tx_mode();

        address_request = 0;

        if ((lsb_address != 0x00) && (cube_address != 0x00))
        {
          PORTA &= ~(1<<PA7);  _delay_ms(300);    // LED off
          PORTA |= (1<<PA7);  _delay_ms(100);    // LED on
          PORTA &= ~(1<<PA7);  _delay_ms(300);    // LED off
          PORTA |= (1<<PA7);  _delay_ms(100);    // LED on
          PORTA &= ~(1<<PA7);  _delay_ms(300);    // LED off
          PORTA |= (1<<PA7);  _delay_ms(100);    // LED on
          PORTA &= ~(1<<PA7);  _delay_ms(300);    // LED off
          PORTA |= (1<<PA7);  _delay_ms(100);    // LED on
          PORTA &= ~(1<<PA7);  _delay_ms(300);    // LED off
          PORTA |= (1<<PA7);  _delay_ms(100);    // LED on
          PORTA &= ~(1<<PA7);            // LED off
        }
      }
      
      /*if button 11 pressed long (read new cube address) and EEPROM write address
       is not the last EEPROM write address than read the new cube address*/
      
      if ((address_request == 2) && (EEPROM_read_value != 0x00))
      {
        Cube_high_byte   = eeprom_read_byte(EEPROM_read_address);  // Read value from EEPROM @ address 0x00
        EEPROM_read_address ++;
        Cube_low_byte   = eeprom_read_byte(EEPROM_read_address);  // Read value from EEPROM @ address 0x01
        EEPROM_read_address ++;

        EEPROM_read_value = eeprom_read_byte(EEPROM_read_address);  // Prüfe auf value 0x00

        change_TX_address(Cube_high_byte, Cube_low_byte);

        change_RX0_address(Cube_high_byte, Cube_low_byte);

        address_request = 0;

        PORTA &= ~(1<<PA7);  _delay_ms(100);    // LED off
        PORTA |= (1<<PA7);  _delay_ms(100);    // LED on
        PORTA &= ~(1<<PA7);  _delay_ms(100);    // LED off
        PORTA |= (1<<PA7);  _delay_ms(100);    // LED on
        PORTA &= ~(1<<PA7);  _delay_ms(100);    // LED off
        PORTA |= (1<<PA7);  _delay_ms(100);    // LED on
        PORTA &= ~(1<<PA7);  _delay_ms(100);    // LED off
        PORTA |= (1<<PA7);  _delay_ms(100);    // LED on
        PORTA &= ~(1<<PA7);  _delay_ms(100);    // LED off
        PORTA |= (1<<PA7);  _delay_ms(100);    // LED on
        PORTA &= ~(1<<PA7);            // LED off
      }


      if ((address_request == 2) && (EEPROM_read_value == 0x00))
      {
        EEPROM_read_address = 0;        // Set read address to 0x00
        EEPROM_read_value = 0xFF;        // Set read value to default value
        address_request = 0;

        change_TX_address  (0x36, 0x00);    // Set TX address to default value
        change_RX0_address  (0x36, 0x00);    // Set RX address to default value

        PORTA &= ~(1<<PA7);  _delay_ms(500);    // LED off
        PORTA |= (1<<PA7);  _delay_ms(500);    // LED on
        PORTA &= ~(1<<PA7);  _delay_ms(500);    // LED off
        PORTA |= (1<<PA7);  _delay_ms(500);    // LED on
        PORTA &= ~(1<<PA7);            // LED off
      }
    }
}

[H. G. (ledi)]

Ich habe eine Fernbedienung, mit welcher ich RGB-Leds ansteuere. Für den 
Fall, das sich die Fernbedienung aufhängt, soll sich diese durch einen 
Reset neu starten.

[Peter D. (peda)]

Geh vor an die Tafel und schreibe 100-mal:
"Ich darf in Interrupts _delay_ms() nicht verwenden !!!"


Peter

[spess53 (Gast)]

Hi

>Geh vor an die Tafel und schreibe 100-mal:
>"Ich darf in Interrupts _delay_ms() nicht verwenden !!!"

100-mal ist zu wenig.

MfG Spess

[Frank M. (ukw) (Moderator)]

Peter Dannegger schrieb:
> Geh vor an die Tafel und schreibe 100-mal:
> "Ich darf in Interrupts _delay_ms() nicht verwenden !!!"

Das EEPROM beschreiben ist auch suboptimal. Ich habe das Gefühl, dass 
einiges aus der ISR in die Hauptroutine verlegt werden könnte...
Offenbar ist dem TO nicht ganz bekannt, wofür ISRs da sind.

[H. G. (ledi)]

OK!
Welches Problem entsteht dadurch?

[Peter D. (peda)]

Heimo G. schrieb:
> Ich habe eine Fernbedienung, mit welcher ich RGB-Leds ansteuere. Für den
> Fall, das sich die Fernbedienung aufhängt, soll sich diese durch einen
> Reset neu starten.

Warum soll sie sich denn aufhängen?
Doch nur durch einen Programmfehler.

Versuche mal die obige 3 Schritt Methode.
Du hast nämlich schon eine super Race-Condition drinn durch das alles 
gleichzeitig machen wollen:


Heimo G. schrieb:
> wdt_disable();

Heimo G. schrieb:
> ISR(INT0_vect)
> {
>     wdt_enable(WDTO_4S);      // Enable watch-dog-timer 4 seconds

Heimo G. schrieb:
> sleep_mode();



Peter