Stoppuhr mit TIMER

[Serieousl L. (stephan2013)]

Hallo,
Ich will ein Stoppuhr- Programm schreiben mittels Timer.

Also die Uhr funktioniert schon aber das mit Stoppen und wieder starten 
haut nicht hin.
Wenn ich auf Stop drücke stoppt die Uhr vllt nach 3-4 Sekunden aber auch 
nicht immer -.-
Das Programm is für einen Luminary Mikrocontroller.
Hier der Code:

int main (void) 
{
  int sek = 0;
  int min = 0;
  int stunde=0;
  // int i=0;
  int statusUp = 0;
  int statusDown = 0;


  char ausgabe[50]; 

  SysCtlPeripheralEnable(SYSCTL_PERIPH_GPIOF);   // port f enablen
  SysCtlPeripheralEnable(SYSCTL_PERIPH_GPIOE);   // port e enablen

  GPIOPinTypeGPIOInput(GPIO_PORTE_BASE, GPIO_PIN_0 | GPIO_PIN_1);    //Configures PIN0, PORTF for use as GPIO input --> Up- und Down-Taster
  
  GPIOPadConfigSet(GPIO_PORTF_BASE, GPIO_PIN_1, GPIO_STRENGTH_2MA, GPIO_PIN_TYPE_STD_WPU);  //Aktiviert Pull-Up Widerstand  für Select Taster
  GPIOPadConfigSet(GPIO_PORTE_BASE, GPIO_PIN_0 | GPIO_PIN_1 , GPIO_STRENGTH_2MA, GPIO_PIN_TYPE_STD_WPU);  //Aktiviert Pull-Up Widerstand für Up- und Down-Taster
    

  GPIOPinTypeGPIOInput(GPIO_PORTF_BASE, GPIO_PIN_1);   //Setze den PIN_1(Select-Button) als Input
  GPIOPadConfigSet(GPIO_PORTF_BASE, GPIO_PIN_1, GPIO_STRENGTH_2MA, GPIO_PIN_TYPE_STD_WPU);


  RIT128x96x4Init(1000000); //SPI-Takt

  //Set the clocking
  SysCtlClockSet(SYSCTL_SYSDIV_1 | SYSCTL_USE_OSC | SYSCTL_OSC_MAIN | SYSCTL_XTAL_8MHZ);
  // Peripherie aktivieren (Timermodul 0,GPIOs)
  SysCtlPeripheralEnable(SYSCTL_PERIPH_TIMER0);
  //Timermodul 0 als periodischer 32-Bit Timer konfigurieren
  TimerConfigure(TIMER0_BASE, TIMER_CFG_32_BIT_PER);
  //Timervorladewert für Ablauf nach 1 Sekunde bei 8MHz-Takt fixieren
  TimerLoadSet(TIMER0_BASE, TIMER_A, 8000000)  ;
  //Timer A des Timer0 freigeben
  TimerEnable(TIMER0_BASE, TIMER_A);

  while(1)
  {
    while (TimerIntStatus(TIMER0_BASE,false) != TIMER_TIMA_TIMEOUT);
           //warten, bis Timer A des Timermodul 0 den Wert 0 erreicht hat
    
    TimerIntClear(TIMER0_BASE, TIMER_TIMA_TIMEOUT);     // Flag löschen
    sek++;                           // Sekundenzähler erhöhen
    
    // Stoppen
    if((GPIOPinRead(GPIO_PORTE_BASE, GPIO_PIN_0) == 0) && (statusUp == 0))
    {
      // TimerDisable(TIMER0_BASE, TIMER_A); // Timer stoppen
      TimerDisable(TIMER0_BASE, TIMER_A);
      statusUp = 1;
    }
    else if(GPIOPinRead(GPIO_PORTE_BASE, GPIO_PIN_0)!= 0) // wenn taster losgelassen wird
    {
        statusUp=0;                       
    }

    // wieder starten
    if((GPIOPinRead(GPIO_PORTE_BASE, GPIO_PIN_1) == 0) && (statusDown == 0))
    {
      // TimerDisable(TIMER0_BASE, TIMER_A); // Timer stoppen
    TimerEnable(TIMER0_BASE, TIMER_A);
      statusDown= 1;
    }
    else if(GPIOPinRead(GPIO_PORTE_BASE, GPIO_PIN_0)!= 0) // wenn taster losgelassen wird
    {
        statusDown=0;                       
    }
    
    if(sek == 60)                         // Wenn sekunde = 60 ==> 1 Minute
    {
      sek=0;
      min++;    
    }
    if(min == 60)                         // Wenn minute = 60 ==> 1 Stunde
    {
      min=0;
      stunde++;
    }
    if(stunde == 24)
    {
      sek=0;
      min=0;
      stunde=0;  
    }  
    sprintf(ausgabe, "Zeit : %02i:%02i:%02i",stunde,min,sek);           // Stunde, Minute und Sekunde in String konvertieren
    RIT128x96x4StringDraw(ausgabe, 10, 24, 15);
    
  
  }
}

[Karl H. (kbuchegg)]

> Ich will ein Stoppuhr- Programm schreiben mittels Timer.

Dann solltest du dir mal ansehen, was du mit dem Timer sonst noch so 
machen kannst. Zum Beispiel, wei man da bei einem Overflow eine 
Interrupt Routine drann hängt.

Das hier

    while (TimerIntStatus(TIMER0_BASE,false) != TIMER_TIMA_TIMEOUT);

ist effektiv kein Ersatz dafür. Denn an dieser Stelle wartet dein 
Programm auf den Überlauf des Timers.
Solange dein Programm nicht mehr machen soll, als die Zeit zählen, mag 
das noch angehen. In dem Moment, in dem dein Programm aber nebeher auch 
noch andere Dinge machen soll, wie zb ein paar Taster überwachen, ist so 
etwas nicht tragbar.

Und spätestens, nachdem du hier

    if((GPIOPinRead(GPIO_PORTE_BASE, GPIO_PIN_1) == 0) && (statusDown == 0))
    {
      // TimerDisable(TIMER0_BASE, TIMER_A); // Timer stoppen
    TimerEnable(TIMER0_BASE, TIMER_A);
...

den Timer überhaupt abgeschaltet hast, wird dann die erst genannte 
Schleife bis zum St. Nimmerleinstag warten. Da wird dann kein Timeout 
(ich nehme mal an, dass das ein Overflow ist) mehr kommen. Denn ... der 
Timer läuft ja nicht.


Fazit: so gehts nicht.
Die vernünftigste Lösung ist eine Trennung des Uhrenmechanismus von den 
Tasten. Wenn dein Timer Interrupts kann (wovon ich mal stark ausgehe), 
dann  besteht die Lösung darin, die Uhr als Ganzes (ohne AUsgabe) im 
Interrupt zu machen und die Tasten in der Hauptschleife zu behandeln.

[Serieousl L. (stephan2013)]

zuerst danke für die Antwort
Also mir wurde gesagt in der Schule, dass ich das ohne Interrupts machen 
muss.

Und wie meinen Sie das, dass ich das mit der Tastenabfragung auserhalb 
verwenden soll ?

[Karl H. (kbuchegg)]

Serieousl Lesrely schrieb:
> zuerst danke für die Antwort
> Also mir wurde gesagt in der Schule, dass ich das ohne Interrupts machen
> muss.

Ok.
Dann aber bitte so

  while(1)
  {
    if( TimerIntStatus(TIMER0_BASE,false) == TIMER_TIMA_TIMEOUT)
    {
      .. Timer Überlauf behandeln.
         Uhr weiterzählen und Anzeige auf neuesten Stand bringen
    }

    Tasten behandeln
  }

In der Hauptschleife müssen die Tasten IMMER behandelt werden und nicht 
nur zu jeder vollen Sekunde!

[Serieousl L. (stephan2013)]

Vielen Danke jetzt funktioniert es !
Tut mir leid wenn ich so viel frage :

Was macht diese Zeile genau:

if( TimerIntStatus(TIMER0_BASE,false) == TIMER_TIMA_TIMEOUT)

Und wieso hat das von mir nicht vorher gestimmt
ich muss ja abfragen während er zählt oder?
Wie du es gemacht hast überprüft er die Tasten zum Schluss oder wie ?