AT Mega 32: Countdown und LCD

[Michel S. (11michi11)]

Hallo,

ich suche nach einer Anregung oder Lösungsansatz zu folgendem "Problem".
Ein Bestandteil meines derzeitiges "Projektes" ist es eine 
Countdown-Zeit vorzuwählen, diese auf einem LCD auszugeben und 
anschließend mit ISR runterzählen zu lassen.
Ich habe es auch soweit realisiert, dass ich mttels Poti an ADC7(PA7) 
eine Zeit zwischen 0 und 15 Minuten vorwählen kann und diese auf dem LCD 
ausgebe.
Der Start des Countdown wird dem Controller dann mittels Tasterdruck 
signalisiert, in Folge dessen Timer 2 initialisiert wird und die Zeit 
runtergezählt wird.
Die ermittelte Countdown-Zeit wird aus dem ADC Wert errechnet und 
mittels
dtostrf gewandelt und an mein LCD (4Bit Modus ohne R/W) nach der LCD 
Routine aus dem hiesigen Tutorial( lcd_string(...); ) übergeben.
Timer 2 initialisiere ich wie folgt:

void timer_an(void)
{
TCCR2|=(1<<CS22);      // Vorteiler 64
TCNT2 = 0x00;
TIMSK|=(1<<TOIE2);
}
ISR des Timer 2
ISR (TIMER2_OVF_vect)
{
ovf++;
TCNT2=0;
}
mein Systemtakt ist 7,3728MHz bei 450 OVFs sollte 1 Sekunde vergangen 
sein?!
Im Hauptprogramm übergebe ich den Inhalt von ovf an eine andere Variable 
und verringere nach einer Sekunde in einer if Schleife den Wert der 
Variablen für die Sekundenanzeige um 1 und wandel ihn erneut mit 
dtostrf, um den vermeindlich aktualisierten Wert erneut auszugeben.
Leider tut sich auf meinem Display rein gar nichts.
Habe Momentan keinen Ansatz wo der Fehler stecken könnte. Muss ich 
womöglich den Wert der im string aufs Display ausgegeben wurde, erst 
mittels atol oder ähnlichem zurückwandeln, um ihn dann verändern zu 
können oder stimmt was mit dem Timer nicht?

Vielen dank im Voraus für eure Antworten.

[Cyblord -. (cyblord)]

Michel Schl. schrieb:
> Muss ich
> womöglich den Wert der im string aufs Display ausgegeben wurde, erst
> mittels atol oder ähnlichem zurückwandeln, um ihn dann verändern zu
> können oder stimmt was mit dem Timer nicht?

Nein du musst nichts zurückverwandeln. Neuen Wert in String und aufs LCD 
ausgeben reicht.

Wo wird dein Wert an das LCD ausgegeben? Ist deine Zählvariable 
volatile?

Aber ohne den ganzen Code wirds nix.

gruß cyblord

[Michel S. (11michi11)]

Hallo ...
meine Zählvariable in der ISR ovf ist volatile.
Hab mir "spaßenshalber" mal in einer neuen Zeile des LCDs die Variable 
ausgeben lassen, an die ich volatile unsigned int ovf im Hauptprogramm 
übergebe bzw. in die ich sie kopieren diese ist unsigened int tc.
Diese wird munter hochgezählt.
Nur scheint das Programm meine if schleifen zu ignorieren und setzt sie 
nicht zurück bzw. minimiert die LCD Anzeige nicht?!

Hab den Code mal kopiert, nur weiß ich nicht ob es verständlich ist, da 
es nur ein Ausschnitt ist.

[Michel S. (11michi11)]

Ist sicher auch einiges überflüssig...sitz nun schon ein paar Stunden 
dabei und habe immer munter alles geändert so wie es mir gerade in den 
Sinn kam.

[Karl H. (kbuchegg)]

Michel Schl. schrieb:
> Ist sicher auch einiges überflüssig...

mit Sicherheit.
Aus deinem Code kann ich wirklich nicht viel machen, selten etwwas 
derart kompliziertes gesehen.

Allerdings frage ich mich, wozu du für Zeiten ein dtostrf brauchst?


Was genau umfasst eigentlich dein komplettes Projekt. Eine Countdown-Uhr 
mit LCD benötigt vieleicht in Summe eine oder eineinhalb 
Bildschirmseiten und da steckt der meiste Code in der ISR, die die Uhr 
runterzählt.

[Cyblord -. (cyblord)]

Michel Schl. schrieb:
> if schleifen

Hopfen und Malz verloren für den der sowas schreibt.

Ausserdem ist das doch nicht dein ganzer Code, da fehlt doch einiges. 
Und poste ihn als .c damit man den auch gescheit angucken kann.

Von einem strukturierten Programmierstil hast du wohl noch nie was 
gehört? Gerade bei komplexeren Programmen musst du die Funktionalität 
sinnvoll in Funktionen auslagern, diese Funktion sinnvoll benennen und 
auch die Variablennamen "sprechend" machen. Ein Spaghetticode ist das, 
herrje...

gruß cyblord

[Krapao (Gast)]

> mein Systemtakt ist 7,3728MHz bei 450 OVFs sollte 1 Sekunde
> vergangen sein?!
> ovf++

Dann sollte

1/ ovf eine globale Variable sein, die z.B. so definiert ist

volatile uint16_t ovf;

2/ bei der Abfrage der Variable ovf im Hauptprogramm sollten Lese- und 
Schreibzugriffe atomar gemacht werden (Artikel Interrupt). In deinem 
Code fehlt das z.B. an diesen Stellen:

  tc=ovf;  // nicht atomarer Zugriff auf ovf!  
  if(tc==450)
  {
    ovf=0; // nicht atomarer Zugriff auf ovf!

[Karl H. (kbuchegg)]

Krapao schrieb:
>> mein Systemtakt ist 7,3728MHz bei 450 OVFs sollte 1 Sekunde
>> vergangen sein?!
>> ovf++
>
> Dann sollte
>
> 1/ ovf eine globale Variable sein, die z.B. so definiert ist
>

> volatile uint16_t ovf;
>

>
> 2/ bei der Abfrage der Variable ovf im Hauptprogramm sollten Lese- und
> Schreibzugriffe atomar gemacht werden (Artikel Interrupt). In deinem
> Code fehlt das z.B. an diesen Stellen:
>

>   tc=ovf;  // nicht atomarer Zugriff auf ovf!
>   if(tc==450)
>   {
>     ovf=0; // nicht atomarer Zugriff auf ovf!
>

grundsätzlich richtig.
Es schadet aber auch nichts, wenn er die komplette Uhrenlogik in die ISR 
verlagert. Auf die Art zählt die Countdownuhr auch dann noch im 
Hintergrund richtig weiter, wenn auf dem LCD gerade irgendwelche 
Benutzerinteraktionen laufen.


So wie das aussieht, will er ja einfach nur ein paar Pins einschalten, 
den Countdown runterzählen und nach der Vorgabezeit sollen sich die Pins 
wieder ausschalten. Das Runterzählen und Ausschalten kann ruhig alles 
geschlossen in der ISR passieren (nicht jedoch den LCD update!), die 
paar Takte die dafür notwendig sind, hat man mit Leichtigkeit.

[Krapao (Gast)]

> Leider tut sich auf meinem Display rein gar nichts.

Den sei() nach dem Aufruf von timer_an() hast du schon im Code, oder?

Aber selbst ohne den solltest du auf dem Display was sehen und wenn es 
nur Nullen bei der Ausgabe von tc sind oder der Vorwahlwert für s und m.

Ich erkenne im Code das Starten des Countdowns nicht und denke der 
Countdown wird bei jedem Durchlauf der while Schleife bearbeitet.

dtostrf() ist Overkill, wie andere auch schon schrieben.

  while ( mode == 3 ) {
    PORTB &=~ (1<<PB4); // Startbit Sprachausgabe zurücksetzen

    // einmalige Textausgabe als Modus Funktionsbeschreibung
    if ( riegel == 255 ) {  
      PORTB &= ~(1<<PB3); // Heizung aus
      PORTD |= (1<<PD7);  // Hintergrundbeleuchtung LCD an
      lcd_clear();
      lcd_setcursor(1,1);
      lcd_string("Ziehzeit Countdown");
      lcd_setcursor(0,2);
      lcd_string("Vorwahl durch Drehen");
      lcd_setcursor(2,4);
      lcd_string("Start via Taster");
      riegel = 254;
    }

    // Startwert des Countdowns festlegen via Poti
    if ( riegel == 254 ) {
      sollwert = adc_soll(); // Stellung des Potis erfassen
      // ADC Wert/(ADCmax.{1023}/Zeitmax.{900s=15min)
      uhr = sollwert/umrechnung_zeit; 
      minuten = uhr / 60;  
      min = (int) minuten;
      sekunden = ((uhr / 60) - min) * 60;
      sek = (int) sekunden;
      m = min;
      s = sek;
      zeit_anzeigen(min, sek);
      // wie ändert sich der Wert für 
      // riegel von 254 auf einen anderen Wert?
    }  

    // Countdown starten via Taster?
    if ( riegel == 253 ) {
      timer_an();
      lcd_clear();
      lcd_setcursor(2,1);
      lcd_string("Countdown l");
      lcd_data(0xE1);
      lcd_string("uft");
      riegel = 252;
    }

// ANFANG COUNTDOWN CODE
// Der Countdown läuft von ANFANG bis ENDE bei jedem Wert für riegel!
// IMHO sollte er jedoch nur nach Tasterbestätigung laufen,
// d.h. eventuell nur wenn riegel == 252 ist!

    // Countdown Variablen m und s runterzählen
    ATOMIC_BLOCK(FORCEON) {
      tc = ovf;            // für Debugzwecke s.u.
      if ( ovf == 450 ) {  // 1s vergangen?
        ovf = 0;           // Timer2 counter reset
        if ( s != 0 ) {
          s -= 1;
        } else {
          s = 59;
          m -= 1;
        }
      }
    }

    // Countdown Variablen m und s ausgeben
    // [                ]
    // [                ]
    // [        mm:ss   ]
    // [                ]
    zeit_anzeigen(m, s);

    // Debugausgabe tc?
    // [                ]
    // [                ]
    // [        mm:ss   ]
    // [tttttt          ]
    PORTD ^= (1<<PD6); // 1s Puls-LED?
    { 
      char s[7];
      lcd_setcursor(0, 4);
      itoa(tc, s, 10); // #include <stdlib.h> erforderlich
                       // ggf. führende Nullen implementieren
      lcd_string(s);
    }

    // Countdown Alarm auslösen?
    if ( (min == 0) && (sek == 0) ) { // => falsche Variablen?
      mode = 4;
    }
    cnt = 0; 
// ENDE COUNTDOWN CODE
  } //Ende Mode 3

Mit einer Hilfsfunktion

void zeit_anzeigen(uint8_t minuten, uint8_t sekunden)
{
  char s[6];
  s[0] = ((minuten % 100) / 10) + '0';
  s[1] = (minuten % 10) + '0';
  s[2] = ':';
  s[3] = ((sekunden % 100) / 10) + '0';
  s[4] = (sekunden % 10) + '0';
  s[5] = 0;
  lcd_setcursor(8, 3);
  lcd_string(s);  
  // hier ggf. Blinken des : implementieren
}

[Michel S. (11michi11)]

Hallo,
vielen Dank für euren zahlreichen und hilfreichen Antworten.
Hatte es derweil bereits selber zum Laufen gebracht, nach eurer Anregung 
hab ich allerdings es doch vorgezogen die Minimierung der Zeitvariablen 
innerhalb der OVF ISR vorzunehmen.
Zu meiner "Verteidigung" bzw. in Bezug zu eurer sicher angebrachten 
Kritik bezüglich meines "Programierstils", sofern man es so nennen darf 
... möchte ich sagen, dass ich bis Dezember letzten Jahres weder von 
Mikrocontrollern, noch von irgendeiner Programmiersprache Ahnung hatte.
Habe dann kurz mit Assembler begonnen und bin seit ca.Februar bei C 
gelandet.
Daher bin ich noch ein vollkommener Neuling auf diesem Gebiet und lerne 
noch bzw. probiere viel aus und mache es erstmal so wie ich es mir in 
meiner einfachen Logik denke ... ;)