Merkwürdiger _delay_ms() reset

[Dirk S. (fusebit)]

Moin,

ich stehe gerade vor einem für mich völlig unverständlichen Problem.
Bei mir verursacht die _delay_ms() Funktion plötzlich einen ständig 
wiederholten Reset am ATmega328.

Codeausschnitt:

lcd_init(LCD_DISP_ON);
  lcd_clrscr();
  lcd_gotoxy(0,0);
  lcd_puts("UI-Logger-Solar");
  //Warten damit der OpenLog initialisiert ist
  _delay_ms(250);
  lcd_gotoxy(15,0);  
  lcd_putc('9');
//  wdt_reset();  
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('8');  
//  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('7');

Das passiert aber nur wenn ich die Schaltung "frisch" an die Versorgung 
anklemme. Sobald ich einmal einen Reset auslöse, oder den Chip neu 
flashe funktioniert alles.
Nehme ich die delays raus geht es auch. Nun wird es besonders schräg:
Setze ich ein delay vor den obigen Bereich resettet das Programm bei dem 
delay. Lasse ich den Abschnitt wie oben, dann läuft der Zähler runter 
(geht bis 0) und dann erfolgt der Reset.
- WDT ist aus. Habe es aber auch mit WDT probiert.
- Der Wert für die Funktion spielt keine Rolle, habe ab 1ms vieles 
probiert.
- Verschieden Werte für die Optimierung habe ich auch probiert

Es hat die ganze Zeit funktioniert und plötzlich nicht mehr! Ich habe 
versucht möglichst genau die letzte Änderung auszukommentieren, aber der 
Reset bleibt. Leider weiß ich nicht genau ab welcher Änderung es 
angefangen hat, weil es beim neu flashen ja nicht auftritt.

[Jörg W. (dl8dtl) (Moderator)]

Dirk S. schrieb:
> WDT ist aus. Habe es aber auch mit WDT probiert.

Bist du dir sicher?

Wenn der Wachhund einmal gebissen hat, muss man ihn explizit beim Booten 
bedienen (WDRF zurücksetzen), ansonsten bleibt er bis zum nächsten 
Powercycle aktiv.

[Karl M. (Gast)]

Hallo,

42, was steht in den Fuse-Bits für aktivierte Hardwareeinheiten?

[Teo D. (teoderix)]

Das Problem scheint wo anders zu liegen. Das ganze klingt schwer nach 
Buffer-Overflow.

[1000V Dc (Gast)]

delay_ms kann nicht mit beliebig großen werten aufgerufen werden...

[A. S. (Gast)]

Wie arbeitet delay?
Wann tritt der reset auf? (Hast du ein oszi/LSA? Dann vorher Pin wackeln
Was passiert nach Zähler=0?
Was meinst Du mit "wenn ich reset auslöse"? Den delay-reset oder per 
Pin? Tritt es nur nach Power-on auf? -> Reset-Beschallung
Welche Funktionen triggern den wdt?
Stromaufnahme/Begrenzung?

[Jörg W. (dl8dtl) (Moderator)]

1000V Dc schrieb:
> delay_ms kann nicht mit beliebig großen werten aufgerufen werden...

Doch, schon seit vielen Jahren.

[Dirk S. (fusebit)]

Das geht ja schnell.

Es tritt nur auf wenn die Versorgung weg war. Nach einem Reset per Pin, 
oder einem read vom programmer ist es weg. Das ist für mich ja der 
zentrale Knackpunkt. Weil ab dem Reset alles sauber läuft...

Der Wert des delay macht keinen Unterschied.

Ich habe nur eine minimale Programmierumgebung mit im Weihnachtsurlaub 
:-)
Oszi etc. sind zu Hause im Keller.

Auch wenn ich den WDT korrekt bediene tritt der Reset beim ersten Aufruf 
von _delay_ms() auf, allerdings nicht wenn die weiteren Aufrufe im 
oberen Code-Ausschnitt da sind.

Stromaufnahme ist okay, ein DMM habe ich dabei.

[Dirk S. (fusebit)]

So, nun habe ich den WDT aktiviert und setze ihn in nicht zurück.

Nach dem flashen startet das Programm, geht sauber in die Hauptschleife 
und nach einiger Zeit löst der Wachhund aus. So soll es sein.

Trenne ich nun die Versorgung, dann läuft der Countdown durch und es 
erfolgt der Reset weit vor dem WDT.

Setzte ich nun wdt_reset() dazwischen ändert sich nichts!

int main(void)
{
     wdt_reset();
     /* Start timed sequence */
     WDTCSR |= (1<<WDCE) | (1<<WDE);
     /* Set new prescaler(time-out) value */
     WDTCSR = (1<<WDE) | (1<<WDP3);
     
    DDRB=0b00111111;
     DDRC=0b00110000;
  DDRD=0b11111110;
   
  SETBIT(PORTB,1);
  SETBIT(PORTB,2);
  
  SPCR = (1<<SPE) | (1<<MSTR);                    //Enable SPI Master
  
  UBRR0H = (unsigned char)(11>>8);                  //Baudrate setzen bei 3,6864 MHz: 23=9600; 11=19200
  UBRR0L = (unsigned char)11;                      //siehe Datenblatt
  UCSR0B |= (1<<RXCIE0) | (1<<RXEN0) | (1<<TXEN0);          //Receiver + Transmitter Enable
  UCSR0C |= (1<<UCSZ01)| (1<<UCSZ00);                  //1 Stop Bit/s, 8 Datenbits
  
  set_sleep_mode(6);                          //6: Standby, 3: Power-save, 2: Power-down, 1: ADC noise reduction, 0: idle
  sleep_enable();  

  //TWI setup
  TWBR = 0b00000000;                          //Taktfrequenz maximal, bei 3,68 MHz -> 230 kHz  
  if((RTC_read(RTCHOUR) & 0b01000000)) RTC_write(RTCHOUR,0b00000000);          //24 Stunden format
  
  /*Für das erte Mal und debugging*/
  //Datum festlegen, hier der
  //RTC_write(RTCYEAR,0b00011001);
  //RTC_write(RTCMTH,BCD(12));
  //RTC_write(RTCDATE,BCD(23));
  //RTC_write(RTCHOUR,BCD(19));
  //RTC_write(RTCMIN,BCD(27));  
  
    
  TCCR2B |= (1<<CS22) | (1<<CS21) | (1<<CS20);            //Timer 1 Prescaler 011 -> 64, 010 -> 8
  TIMSK2 |= (1<<TOIE0);                        //Overflow Interupt Enable
  TCNT2 = TIMER;
  
     
  lcd_init(LCD_DISP_ON);
  lcd_clrscr();
  lcd_gotoxy(0,0);
  lcd_puts("UI-Logger-Solar");
  lcd_gotoxy(0,1);
  lcd_puts("2019");
  //Warten damit der OpenLog initialisiert ist
  _delay_ms(250);
  lcd_gotoxy(15,0);  
  lcd_putc('9');
  wdt_reset();  
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('8');  
  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('7');  
  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('6');  
//  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('5');  
//  wdt_reset();    
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('4');  
//  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('3');  
//  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('2');  
//  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('1');  
//  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('0');
//  wdt_reset();
  _delay_ms(250);        

  lcd_clrscr();    

  date_refresh();   
 
  //Titelzeile
  ausgabe("Datum");
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = tab;
  ausgabe("Zeit");
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = tab;  
  ausgabe("Spannung");
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = tab;
  ausgabe("Strom");
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = cr;
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = lf;    
 
   sei(); 
   
    for(;;)
    {
      if(slow==1)                //Nur einmal pro Sekunde!
      {
        spannung=spannung/10;        //Mittelwert
        strom=strom/10;            //Mittelwert
        if(strom<NULLPUNKT) 
        {
          vorzeichen=0;
          strom=NULLPUNKT-strom;
        }
        if(strom>NULLPUNKT)
        {
          vorzeichen=1;
          strom=strom-NULLPUNKT;        
        }
        if(strom==NULLPUNKT) strom=0;
        strom=(strom*10)/4;          //In mA umrechnen
        
        
        if(sekunde<5) seite02();
        else seite01();
        
        
        //Sekundenwerte im Array speichern
        sek_u[a]=spannung;
        sek_i[a]=strom;
        a++;
        if(a==61) a=1;
  
        spannung=0;
        strom=0;
    
        CLRBIT(PORTB,1);
        _delay_ms(1);
        SETBIT(PORTB,1);
      }
    
    adc_i=adc_volt(0);
    strom=strom+adc_i;
    
    adc_u=((unsigned long)adc_volt(1)*TEILER)/1000;
    spannung=spannung+adc_u;
//    wdt_reset();
    sleep_cpu();
    }
}

[Karl M. (Gast)]

Nochmal,

welche Fuse-Bits sind gesetzt, ist BOD Level eingestellt?
Alle Kondensatoren, i.A. 100nF verbaut?
Alle Vcc, AVcc und (AGnd) Gnd angeschlossen?
Wie weich ist die Versorgungsspannung an Vcc?

[Dirk S. (fusebit)]

Hier die Sicherungen...

[Karl M. (Gast)]

Woher weißt Du das?

set_sleep_mode(6);                          //6: Standby, 3: Power-save, 2: Power-down, 1: ADC noise reduction, 0: idle

Da gibt es in den verschiedenen AVRs Bitnamen, die in Kombination das 
gewünschte bewirken.

Dann noch, ohne den passenden Einsprung?

TIMSK2 |= (1<<TOIE0);

Zum Glück lässt sich das Programm nicht Übersetzen, so dass niemand 
einen Test machen kann!

[Karl M. (Gast)]

Nachfrage,

ist es ein ATmega328 oder ein ATmega328p?
Da gibt es Unterschiede.

[Dirk S. (fusebit)]

Sleep Einstellungen machen keinen Unterschied, hatte ich schon komplett 
auskommentiert.

Der Code ist derweil etwas wild, da ich am rumprobieren bin.

Anbei der Schaltplan.

Und wie gesagt: Es hat die ersten Tage funktioniert, heute habe ich 
weiter gemacht und nun tritt es auf. Da es nur bei einem Kaltstart 
passiert habe ich es erst im Laufe des Tages bemerkt.

Werde es jetzt weiter reduzieren...

[Holger L. (max5v)]

Dirk S. schrieb:
> Trenne ich nun die Versorgung, dann läuft der Countdown durch und es
> erfolgt der Reset weit vor dem WDT.

Eventuell mal die Brownout Detection deaktivieren bzw. herabsetzen.
Es kann unter Umständen sein das das Display wenn es noch nicht 
initialisiert ist deutlich mehr Strom zieht als deine Versorgung 
hergibt?

[Dirk S. (fusebit)]

Karl M. schrieb:
> Nachfrage,
>
> ist es ein ATmega328 oder ein ATmega328p?
> Da gibt es Unterschiede.

ATmega328, AtmelStudio ist mit der Signatur zufrieden...

[Karl M. (Gast)]

Da sind noch Antworten ausstehend,

Und dann wird noch im Register UCSR0B der "RX Complete Interrupt Enable" 
und ich sehe keinen Interrupt Routine dafür.

Warum?

[Karl M. (Gast)]

Dirk S. schrieb:
> Karl M. schrieb:
>> Nachfrage,
>>
>> ist es ein ATmega328 oder ein ATmega328p?
>> Da gibt es Unterschiede.
>
> ATmega328, AtmelStudio ist mit der Signatur zufrieden...

Das meine ich nicht, es gibt zwei Datenblätter!

[Dirk S. (fusebit)]

Mal sehen ob das klappt.

Ich versuche ein mp4-Video hochzuladen auf dem das Verhalten zu sehen 
ist.
Erst die Dauerschleife mit dem ungeklärtem Reset nach dem Countdown, 
dann ein "read" mit dem AtmelICE und es läuft (bis der Wachhund 
anbeisst).

Nach einem Kaltstart hängt es wieder...

[Dirk S. (fusebit)]

Und für Karl:

/*
 * UI-Logger-Solar.c
 *
 * Created: 22.12.2019 16:26:01
 * Author : Dirk
 
 ATmega328 mit 16x2 LCD; LED und drei Tasten
 
 LCD:
 PORTD,2: E
 PORTD,3: RW
 PORTD,4: RS
 PORTB,0: D4
 PORTD,7: D5
 PORTD,6: D6
 PORTD,5: D7
 
 
  - WDT fertig implementieren
  - 5 min Mittelwerte
  - Tasten zum umschalten und Uhr einstellen
 
 */ 


#define BIT(x) (1<<(x))
#define SETBIT(p, b) (p) |=BIT(b)
#define CLRBIT(p, b) (p) &= ~BIT(b)
#define TOGGLE(p, b) (p) ^=BIT(b)
#define TIMER 76       
#define VREF 5000    //mV
#define TEILER 4092    //Spannungsteiler
#define NULLPUNKT 2493  //Nullpunkt Allegro
#define STEIGUNG 400  //mV/A für Allegro
#define cr 0x0D
#define lf 0x0A
#define tab 0x09
#define punkt 0x2E
#define dopu 0x3A
//#define F_CPU 1843200UL     //  In compiler symbols festgelegt


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


unsigned long zwischenwert, adc_u, adc_i,spannung, strom, mittel_u, mittel_i;
unsigned int jahr,  sek_i[60], sek_u[60], minute_i[60], minute_u[60], fuenfmin_u, fuenfmin_i;
char output[8];
unsigned char a,byte_eins, byte_zwei, slow, vorzeichen, flag01, data, dummy;
volatile unsigned char sekunde, minute, stunde, tag, monat;
volatile unsigned char byte00, byte01, byte02;

//TWI Variablen
char SLA_W=0b11010000;        //Adressierung und RTC register write
char SLA_R=0b11010001;        //RTC register read
char RTCSEC=0x00;          //RTC register address
char RTCMIN=0x01;
char RTCHOUR=0x02;
char RTCWKDAY=0x03;
char RTCDATE=0x04;
char RTCMTH=0x05;
char RTCYEAR=0x06;
char CONTROL=0x0E;          //Control register address


void RTC_write(char Address, char Data)
{
  TWCR = (1<<TWINT) | (1<<TWSTA) |(1<<TWEN);      //Start condition
  while (!(TWCR & (1<<TWINT)));
  //if ((TWSR & 0xF8) !=START) ERROR();
  
  //Byte senden (Adressierung)
  TWDR = SLA_W;
  TWCR = (1<<TWINT) | (1<<TWEN);
  while (!(TWCR & (1<<TWINT)));
  //if((TWSR & 0xF8) != MT_SLA_ACK) ERROR();
  
  //Byte senden (Daten)
  TWDR = Address;    //RTCSEC register address
  TWCR = (1<<TWINT) | (1<<TWEN);
  while (!(TWCR & (1<<TWINT)));

  //Byte senden (Daten)
  TWDR = Data;
  TWCR = (1<<TWINT) | (1<<TWEN);
  while (!(TWCR & (1<<TWINT)));
  
  TWCR = (1<<TWINT) | (1<<TWEN) | (1<<TWSTO);    //STOP condition
}

char RTC_read(char Address)    //UNFERTIG!!!
{
  char Data;
  
  //Hiermit wird die TWI nach dem schlafen wieder gestrtet
  TWCR &= ~((1 << TWSTO) | (1 << TWEN));
  TWCR |= (1 << TWEN);



  TWCR = (1<<TWINT) | (1<<TWSTA) |(1<<TWEN);      //Start condition
  while (!(TWCR & (1<<TWINT)));
  //if ((TWSR & 0xF8) !=START) ERROR();
  
  //Byte senden (Adressierung)
  TWDR = SLA_W;
  TWCR = (1<<TWINT) | (1<<TWEN);
  while (!(TWCR & (1<<TWINT)));
  //if((TWSR & 0xF8) != MT_SLA_ACK) ERROR();
  
  //Byte senden
  TWDR = Address;    // register address
  TWCR = (1<<TWINT) | (1<<TWEN);
  while (!(TWCR & (1<<TWINT)));

  TWCR = (1<<TWINT) | (1<<TWSTA) |(1<<TWEN);      //Rep. Start condition
  while (!(TWCR & (1<<TWINT)));
  //if ((TWSR & 0xF8) !=START) ERROR();
  
  //Byte senden
  TWDR = SLA_R;
  TWCR = (1<<TWINT) | (1<<TWEN);
  while (!(TWCR & (1<<TWINT)));
  //if((TWSR & 0xF8) != MT_SLA_ACK) ERROR();
  
  //Byte empfangen (Daten)
  
  TWCR = (1<<TWINT) | (1<<TWEN);
  while (!(TWCR & (1<<TWINT)));
  Data = TWDR;
  
  TWCR = (1<<TWINT) | (1<<TWEN) | (1<<TWSTO);    //STOP condition
  
  return Data;
}

void date_refresh(void)
{
  cli();
  sekunde=RTC_read(RTCSEC);
  //sekunde &= ~((1 << 7));              //ST bit loeschen
  sekunde= (sekunde/16*10) + (sekunde%16);
  minute=RTC_read(RTCMIN);
  minute=(minute/16*10)+(minute%16);
  stunde=RTC_read(RTCHOUR);
  //stunde &= ~((1 <<6));              //AM/PM bit loeschen
  stunde=(stunde/16*10) + (stunde%16);
  tag=RTC_read(RTCDATE);
  tag=(tag/16*10)+(tag%16);
  monat=RTC_read(RTCMTH);
  //monat &= ~((1<<5));                //Lapyear bit loeschen
  monat=(monat/16*10)+(monat%16);
  jahr=RTC_read(RTCYEAR);
  jahr=(jahr/16*10)+(jahr%16);
  jahr=jahr+2000;
  sei();
}

char BCD(char wert)      //Die BCD Codierung fuer die RTC erzeugen
{
  char temp=0;
  temp=((wert/10)<<4);
  temp=temp+wert%10;
  return(temp);
}

unsigned int adc_volt(uint8_t kanal)      //ADC Abfrage Funktion
{
  unsigned long berechnung, speicher=0;
  
  cli();
  byte00=1;                          //Beim MCP3002 für beige Känale gleich
  
  if(kanal==0) byte01=128;
  if(kanal==1) byte01=192;
  
  byte02=0;
  
  for(int n=0;n<8;n++)
  {
    CLRBIT(PORTB,2);                    //Start transmission
    SPDR = byte00;
    while (!(SPSR & (1<<SPIF)));
    SPDR = byte01;
    while (!(SPSR & (1<<SPIF)));
    
    byte_eins=SPDR;
    
    SPDR =byte02  ;
    while (!(SPSR & (1<<SPIF)));
    
    byte_zwei=SPDR;
    
    SETBIT(PORTB,2);                    //Stop transmission
    
    berechnung=(byte_eins<<8)+byte_zwei;          //Wert auslesen
    speicher+=berechnung;
  }

  
  berechnung=((speicher+4)/8)*VREF/4096;                  //Mittelwert der AD-Wandler Durchläufe
  
  sei();
  return(berechnung);
}

//serielle Ausgabe
void ausgabe(char text[16])
{
  
  for (unsigned char k=0;k<15;k++)    //max 16 Stellen
  {
    data=text[k];          //kte-Stelle übernehmen
    if(data==0) break;        //Nur Nutzdaten übertragen
    text[k] = dummy;        //kte-Stelle löschen
    while (!(UCSR0A& (1<<UDRE0)));
    UDR0=data;
  }
  
}

void datensatz(void)
{
  //Datum
  utoa(tag,output,10);
  ausgabe(output);
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = punkt;
  utoa(monat,output,10);
  ausgabe(output);
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = punkt;
  utoa(jahr,output,10);
  ausgabe(output);
  while (!(UCSR0A& (1<<UDRE0)));
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = tab;
  
  //Uhrzeit
  utoa(stunde,output,10);
  ausgabe(output);
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = dopu;
  utoa(minute,output,10);
  if(minute<10)
  {
    while (!(UCSR0A& (1<<UDRE0)));
    UDR0 = '0';  
  }
  ausgabe(output);
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = dopu;
  utoa(sekunde,output,10);
  if(sekunde<10)
  {
    while (!(UCSR0A& (1<<UDRE0)));
    UDR0 = '0';
  }  
  ausgabe(output);
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = tab;
  
  //Spannung
  utoa(mittel_u,output,10);
  ausgabe(output);
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = tab;
  //Strom
  utoa(mittel_i,output,10);
  ausgabe(output);
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = tab;
  
  

  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = cr;
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = lf;
}

SIGNAL(TIMER2_OVF_vect)                          //Handler für Timer Interrupt
{
    slow++;
    TCNT2 = TIMER;
    if(slow==10)
    {
      slow=0;
      sekunde++;
      if(sekunde==60)
      {
        minute++;
        sekunde=0;
        if(minute%3==0) date_refresh();
        for(unsigned char i=1;i<60;i++)
        {
          mittel_u=mittel_u+sek_u[i];
          mittel_i=mittel_i+sek_i[i];  
        }  
        mittel_u=mittel_u/60;
        mittel_i=mittel_i/60;
        minute_u[minute]=mittel_u;
        minute_i[minute]=mittel_i;

        datensatz();


      }
    }
}

void seite01(void)
{
  //Erste Zeile
  lcd_gotoxy(0,0);
  lcd_puts("       ");
  utoa(tag,output,10);
  lcd_gotoxy(0,0);
  if(tag<10) lcd_putc('0');
  lcd_puts(output);
  lcd_putc('.');
  utoa(monat,output,10);
  lcd_gotoxy(3,0);
  if(monat<10) lcd_putc('0');
  lcd_puts(output);
  lcd_putc('.');
  utoa(stunde,output,10);
  lcd_gotoxy(8,0);
  if(stunde<10) lcd_putc('0');
  lcd_puts(output);
  lcd_putc(':');
  utoa(minute,output,10);
  lcd_gotoxy(11,0);
  if(minute<10) lcd_putc('0');
  lcd_puts(output);
  lcd_putc(':');
  utoa(sekunde,output,10);
  lcd_gotoxy(14,0);
  if(sekunde<10) lcd_putc('0');
  lcd_puts(output);  
  
  //Zweite Zeile
  lcd_gotoxy(0,1);
  lcd_puts("       ");
  utoa(spannung,output,10);
  if(spannung<10000) lcd_gotoxy(1,1);
  else lcd_gotoxy(0,1);
  lcd_puts(output);
  lcd_puts("mV");
  lcd_gotoxy(9,1);
  if(vorzeichen==0) lcd_putc('-');          
  else lcd_putc('+');
  utoa(strom,output,10);
  lcd_gotoxy(10,1);
  lcd_puts("     ");
  if(strom<1000) lcd_gotoxy(11,1);
  if(strom<100) lcd_gotoxy(12,1);
  if(strom<10)  lcd_gotoxy(13,1);
  lcd_puts(output);
  lcd_gotoxy(14,1);
  lcd_puts("mA");  
}

void seite02(void)
{
  //Erste Zeile
  lcd_gotoxy(0,0);
  lcd_puts("Seite 2");

  
  utoa(stunde,output,10);
  lcd_gotoxy(8,0);
  if(stunde<10) lcd_putc('0');
  lcd_puts(output);
  lcd_putc(':');
  utoa(minute,output,10);
  lcd_gotoxy(11,0);
  if(minute<10) lcd_putc('0');
  lcd_puts(output);
  lcd_putc(':');
  utoa(sekunde,output,10);
  lcd_gotoxy(14,0);
  if(sekunde<10) lcd_putc('0');
  lcd_puts(output);
  
  //Zweite Zeile
  lcd_gotoxy(0,1);
  lcd_puts("       ");
  utoa(mittel_u,output,10);
  if(mittel_u<10000) lcd_gotoxy(1,1);
  else lcd_gotoxy(0,1);
  lcd_puts(output);
  lcd_puts("mV");
  lcd_gotoxy(9,1);
  if(vorzeichen==0) lcd_putc('-');
  else lcd_putc('+');
  utoa(mittel_i,output,10);
  lcd_gotoxy(10,1);
  lcd_puts("     ");
  if(mittel_i<1000) lcd_gotoxy(11,1);
  if(mittel_i<100) lcd_gotoxy(12,1);
  if(mittel_i<10)  lcd_gotoxy(13,1);
  lcd_puts(output);
  lcd_gotoxy(14,1);
  lcd_puts("mA");
}



int main(void)
{
     wdt_reset();
     /* Start timed sequence */
     WDTCSR |= (1<<WDCE) | (1<<WDE);
     /* Set new prescaler(time-out) value */
     WDTCSR = (1<<WDE) | (1<<WDP3);
     
    DDRB=0b00111111;
     DDRC=0b00110000;
  DDRD=0b11111110;
   
  SETBIT(PORTB,1);
  SETBIT(PORTB,2);
  
  SPCR = (1<<SPE) | (1<<MSTR);                    //Enable SPI Master
  
  UBRR0H = (unsigned char)(11>>8);                  //Baudrate setzen bei 3,6864 MHz: 23=9600; 11=19200
  UBRR0L = (unsigned char)11;                      //siehe Datenblatt
  UCSR0B |= (1<<RXCIE0) | (1<<RXEN0) | (1<<TXEN0);          //Receiver + Transmitter Enable
  UCSR0C |= (1<<UCSZ01)| (1<<UCSZ00);                  //1 Stop Bit/s, 8 Datenbits
  
  //set_sleep_mode(6);                          //6: Standby, 3: Power-save, 2: Power-down, 1: ADC noise reduction, 0: idle
  //sleep_enable();  

  //TWI setup
  TWBR = 0b00000000;                          //Taktfrequenz maximal, bei 3,68 MHz -> 230 kHz  
  if((RTC_read(RTCHOUR) & 0b01000000)) RTC_write(RTCHOUR,0b00000000);          //24 Stunden format
  
  /*Für das erte Mal und debugging*/
  //Datum festlegen, hier der
  //RTC_write(RTCYEAR,0b00011001);
  //RTC_write(RTCMTH,BCD(12));
  //RTC_write(RTCDATE,BCD(23));
  //RTC_write(RTCHOUR,BCD(19));
  //RTC_write(RTCMIN,BCD(27));  
  
    
  TCCR2B |= (1<<CS22) | (1<<CS21) | (1<<CS20);            //Timer 1 Prescaler 011 -> 64, 010 -> 8
  TIMSK2 |= (1<<TOIE0);                        //Overflow Interupt Enable
  TCNT2 = TIMER;
  
     
  lcd_init(LCD_DISP_ON);
  lcd_clrscr();
  lcd_gotoxy(0,0);
  lcd_puts("UI-Logger-Solar");
  lcd_gotoxy(0,1);
  lcd_puts("     2019");
  //Warten damit der OpenLog initialisiert ist
  _delay_ms(250);
  lcd_gotoxy(15,0);  
  lcd_putc('9');
  wdt_reset();  
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('8');  
  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('7');  
  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('6');  
  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('5');  
  wdt_reset();    
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('4');  
  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('3');  
  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('2');  
  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('1');  
  wdt_reset();
  _delay_ms(250);
  lcd_gotoxy(15,0);
  lcd_putc('0');
  wdt_reset();
  _delay_ms(250);        

  lcd_clrscr();    

  date_refresh();   
 
  //Titelzeile
  ausgabe("Datum");
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = tab;
  ausgabe("Zeit");
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = tab;  
  ausgabe("Spannung");
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = tab;
  ausgabe("Strom");
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = cr;
  while (!(UCSR0A& (1<<UDRE0)));
  UDR0 = lf;    
 
   sei(); 
   
    for(;;)
    {
      if(slow==1)                //Nur einmal pro Sekunde!
      {
        spannung=spannung/10;        //Mittelwert
        strom=strom/10;            //Mittelwert
        if(strom<NULLPUNKT) 
        {
          vorzeichen=0;
          strom=NULLPUNKT-strom;
        }
        if(strom>NULLPUNKT)
        {
          vorzeichen=1;
          strom=strom-NULLPUNKT;        
        }
        if(strom==NULLPUNKT) strom=0;
        strom=(strom*10)/4;          //In mA umrechnen
        
        
        if(sekunde<5) seite02();
        else seite01();
        
        
        //Sekundenwerte im Array speichern
        sek_u[a]=spannung;
        sek_i[a]=strom;
        a++;
        if(a==61) a=1;
  
        spannung=0;
        strom=0;
    
        CLRBIT(PORTB,1);
        _delay_ms(1);
        SETBIT(PORTB,1);
      }
    
    adc_i=adc_volt(0);
    strom=strom+adc_i;
    
    adc_u=((unsigned long)adc_volt(1)*TEILER)/1000;
    spannung=spannung+adc_u;
//    wdt_reset();
    //sleep_cpu();
    }
}

[Karl M. (Gast)]

Hallo,

Was passiert ist alles durch die Hardware, das gesamte Programm und 
dem aktivierten WDT per Fuse-Bit erklärbar.

Aber da Du nicht alles offenlegen möchtest und auch immer wieder von 
anderen Softwareversionen und Tests schreibst, weiß niemand - außer Dir 
- genau, was zusammen hängt.

Versuche mal deine Posts aus unserer Sicht (Informationsstand) zu sehen.

Das alles ist Vergleichbar mir eine Buchstabensuppe.

[Karl M. (Gast)]

Hallo,

da fehlt mir noch eine Interrupt Routine, wie oben geschrieben.

Dann knallt es hier in der SIGNAL(TIMER2_OVF_vect):

if(minute%3==0) date_refresh();

Da man in einer ISR schon das globale Interruptflag wieder setzt.

SIGNAL(<handler>) ist doch nicht mehr gültig.

https://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Alte_Quellen

[Thomas E. (thomase)]

Karl M. schrieb:
> Das meine ich nicht, es gibt zwei Datenblätter!

Nein.

ATmega48A/PA/88A/PA/168A/PA/328/P sind in einem Datenblatt.

[Dirk S. (fusebit)]

Hallo Karl,

was Du schreibst mag ja stimmen, aber bis dort kommt das Programm 
überhaupt nicht nach einem Kaltstart und nach einem Reset funktioniert 
es einwandfrei!

Auch wenn es mit Sicherheit alles andere als sauber programmiert ist.

[Karl M. (Gast)]

Und hier knallt es entsprechend auch durch die unterschiedlichen 
Datentypen und einem Aufruf in einer Interrupt Service Routine.

void ausgabe(char text[16]);

[Karl M. (Gast)]

Thomas E. schrieb:
> Karl M. schrieb:
>> Das meine ich nicht, es gibt zwei Datenblätter!
>
> Nein.
>
> ATmega48A/PA/88A/PA/168A/PA/328/P sind in einem Datenblatt.

Hallo,

in der Atmel Zeit war dies wie folgt:
Beitrag "Re: Atmega 328 oder 328P ?"

[Thomas E. (thomase)]

Karl M. schrieb:
> in der Atmel Zeit war dies wie folgt:
> Beitrag "Re: Atmega 328 oder 328P ?"

Was willst du mit diesem uralten Thread aussagen? Dass du lange nicht 
mehr auf einen Kalender geguckt hast?

[Theor (Gast)]

@ Dirk

Hm. Die Beschreibung ist leider wirklich sehr schwer verständlich, 
teilweise unverständlich und unklar. Schwierig da was Definitives zu 
sagen.

Allerdings gibt es diese Reset-Probleme in Zusammenhang mit dem delay 
immer wieder und - wenn ich mich recht entsinne - in den meisten Fällen 
liegt es nicht am delay selbst.
Ich weiß, dass ist schwer zu verstehen, wenn doch verschiedene Varianten 
mit und ohne delay unterschiedliches Verhalten zeigen. Dennoch ist es 
oft wahr. Lies mal ein paar von den erwähnten Threads.
Dummerweise kann man häufig nicht gleich den Finger auf den Punkt legen, 
der ausschlaggebend ist und in Verbindung mit dem delay ein Problem 
darstellt.

Daher die Hinweise auf sonstige Fehler, die scheinbar nichts damit zu 
tun haben, was Du als Ursache vermutest.

Wenn man nicht so recht weiß, dann ist das einer der beiden Erfolg 
versprechenden Ansätze.

1. Probleme beseitigen, auch wenn sie scheinbar abseitig sind - nichts 
mit der eigenen Vermutung zu tun haben. (Fehler sind ohnehin Fehler und 
müssen irgendwann sowieso beseitigt werden).

2. So viel wie möglich und detaillierte Information geben. Hier 
insbesondere Schaltplan, Beschreibung der Stromversorgung, 
Compileroptionen.
Manchmal waren es auch hohe Lasten, wie Relais, welche die 
Stromversorgung beeinflussten und die delays haben nur den zeitlichen 
Ablauf verändert - waren aber nicht selbst verantwortlich.

Schwierig die Sache, aber wenn Du es über Dich bringen kannst, dann 
folge einmal den Hinweisen, auch wenn sie Dir zuerst sinnlos vorkommen. 
Versprechen kann ich natürlich nichts, aber die Erfahrung zeigt, dass 
das in die richtige Richtung führt.

[Sascha W. (sascha-w)]

@Dirk,

gib doch am Anfang mal das MCUSR aus, dann siehst du was den letzten 
Reset ausgelöst hat.

Sascha

[Dirk S. (fusebit)]

Da ist wohl etwas Wahres dran.
Ich habe nun die UART Ausgabe auskommentiert und der Fehler ist weg.
Und das obwohl diese an der Problemstelle im Programm noch nicht 
aufgerufen wurde...

[Stefan F. (Gast)]

Dirk S. schrieb:
> Und das obwohl diese an der Problemstelle im Programm noch nicht
> aufgerufen wurde...

Ist das tatsächlich so, oder glaubst du das nur?

Schalte zur Kontrolle innerhalb der Funktion eine LED an, dann bist du 
sicher.

[Dirk S. (fusebit)]

Sobald ich diesen Bereich aktiviere ist der Fehler aktiv, auch wenn noch 
alle Zugriffe auf die Schnittstelle auskommentiert bleiben:

  UBRR0H = (unsigned char)(11>>8);                  //Baudrate setzen bei 3,6864 MHz: 23=9600; 11=19200
  UBRR0L = (unsigned char)11;                      //siehe Datenblatt
  UCSR0B |= (1<<RXCIE0) | (1<<RXEN0) | (1<<TXEN0);          //Receiver + Transmitter Enable
  UCSR0C |= (1<<UCSZ01)| (1<<UCSZ00);                  //1 Stop Bit/s, 8 Datenbits

[Sascha W. (sascha-w)]

Dirk S. schrieb:
> Da ist wohl etwas Wahres dran.
> Ich habe nun die UART Ausgabe auskommentiert und der Fehler ist weg.
> Und das obwohl diese an der Problemstelle im Programm noch nicht
> aufgerufen wurde...

Wie ist RX extern beschalten? Da du den Pullup nicht aktiviert hast und 
den RX-Int aktiviert, aber keine ISR, hast könnte bei offenem Eingang 
schon mal was triggern.

Sascha

[Dirk S. (fusebit)]

Genau, Sascha!

Ich Trottel!
Das Problem war der OpenLog, oder besser der Typ, der den OpenLog 
benutzt.

Und Karl hatte auch recht...

Der OpenLog initialisiert sich nur nach dem Trennen der Versorgung und 
gibt seinen Status aus. Mit der UART Konfiguration hat er einen 
Interrupt ausgelöst der keine ISR hat...

Euch einen guten Rutsch :-)

[Jörg W. (dl8dtl) (Moderator)]

Dirk S. schrieb:
> Ich Trottel!

Nun, einfach zur Gewohnheit machen, dass du für jedes Interruptbit, was 
du irgendwo in einem Register aktivierst, sofort irgendwo eine 
(zumindest leere) ISR hinschreibst. Dann weißt du später, dass du da 
noch was ausfüllen musst.

Schön, dass ihr's gefunden habt.

[Rolf M. (rmagnus)]

Jörg W. schrieb:
> Dirk S. schrieb:
>> Ich Trottel!
>
> Nun, einfach zur Gewohnheit machen, dass du für jedes Interruptbit, was
> du irgendwo in einem Register aktivierst, sofort irgendwo eine
> (zumindest leere) ISR hinschreibst. Dann weißt du später, dass du da
> noch was ausfüllen musst.

Ich würde es sogar umgekehrt machen: Den Interrupt überhaupt erst 
aktivieren, nachdem ich die dazugehörige ISR geschrieben habe. 
Schließlich habe ich dann erst ein Interesse daran, dass der Interrupt 
auch aktiv ist.

Karl M. schrieb:
> Und hier knallt es entsprechend auch durch die unterschiedlichen
> Datentypen und einem Aufruf in einer Interrupt Service Routine.
> void ausgabe(char text[16]);

ausgabe ist nicht reentrant, da es eine globale Variable beschreibt. 
Außerdem darf man keine Stringliterale übergeben, da es auch in den 
übergebenen String schreibt.
Die fehlende Reentranz kann dazu führen, dass bei gleichzeitiger 
Ausführung aus dem Hauptprogramm und der ISR über das Stringende hinaus 
geschrieben wird und damit andere Speicherinhalte überschrieben werden.