PIC18 Timer0 auslesen in C

[David M. (mic_dave)]

Hallo zusammen,
ich bin hier schon den ganze Tag am verzweifeln. Und zwar möchte ich mir 
gerne den Wert des Timer0 ausgeben. Ich habe ihn als 16- Bit Timer 
initialisiert.
Nun die Funktionsweise:
Beim Tastendruck soll eine Nachricht an den Rechner über eine RS232- 
Schnittstelle gesendet werden. Diese Nachricht sieht wie folgt aus:

sprintf(text, "(1, %d, %d, %d, %d)", TASTER1, tmr0h, tmr, tmr0l);

(tmr beinhaltet die Anzahl der Überläufe)

Als Ausgabe bekomme ich dann folgendes:
(1, 0, 39, 5312, 0)
Und diese letzte Null steht immer da (tmr0l). Ich weis aber nicht, woran 
das liegt. Er müsste mir da ja eigentlich den Wert tmr0l anzeigen.
Hier mal die Codeschnippsle:

Interrupt:

  unsigned int tmr0l = TMR0L;
  unsigned int tmr0h = TMR0H;
  int_send(tmr, tmr0l, tmr0h);

int_send:

void int_send(unsigned long tmr, unsigned int tmr0l, unsigned int tmr0h){
  if(INTCON3bits.INT2IF){

    INTCON3bits.INT2IE = 0;

    if(INTCON2bits.INTEDG2){
      INTCON2bits.INTEDG2 = 0;
    }
    else{
      INTCON2bits.INTEDG2 = 1;
    }
    INTCON3bits.INT2IF = 0;

    sprintf(text, "(1, %d, %d, %d, %d)", TASTER1, tmr0h, tmr, tmr0l);
    putUsart(text);

    INTCON3bits.INT2IE = 1;
  }

  if(INTCON3bits.INT1IF){

    INTCON3bits.INT1IE = 0;

    if(INTCON2bits.INTEDG1){
      INTCON2bits.INTEDG1 = 0;
    }
    else{
      INTCON2bits.INTEDG1 = 1;
    }
    INTCON3bits.INT1IF = 0;

    tmr0l = TMR0L;
    tmr0h = TMR0H;
    sprintf(text, "(2, %d, %d, %d, %d)", TASTER2, tmr0h, tmr, tmr0l);
    putUsart(text);

    INTCON3bits.INT1IE = 1;
  }

  return;
}

Ich bedanke mich schon vielmals für eure Hilfe.
Falls noch irgendetwas Fehlen sollte, einfach bescheid sagen :)

David

[Bernd R. (Firma: Promaxx.net) (bigwumpus)]

Also,
es ist richtig, daß Du erst TMR0L auslesen mußt, damit TMR0H gebuffert 
wird.

Es ist unnötig innerhalb der ISR den IE auf 0 zu setzen und später 
wieder auf 1 - INTs sind während der ISR quasi sowieso abgeschaltet 
(außer du nutzt Prioritäten).

Innerhalb einer ISR einen Text auszugeben ist quasi Mist.
Eine ISR soll gaaaaanz schnell abgearbeitet werden um dann das lange 
Hauptprogramm weiter zu verfolgen. Das ist dann eine Schleife, die auf 
Zustandsmeldungen der ISRs reagiert.

Trotzdem ist es rätselhaft, warum TMR0L immer 0 ist.

Die Namensgebung finde ich sehr riskant. Ich würde statt tmr0l eher 
tmr0l_ oder ähnliches nehmen, damit nicht irgendwo etwas wegen 
Groß-/Kleinschreibung=egal schief läuft.

[B. L. (b8limer)]

Aber die Anzahl der Überläufe (tmr) bekommst du angezeigt?
Wenn du die Überlaufe angezeigt bekommst dann läuft der Timer zumindest 
schonmal. Wenn nicht, schau warum er nicht läuft.
Man sieht zu wenig von deinem Code - könnte es sein, dass du ausversehen 
den Timerinterrupt nur "scharf" geschalten hast, wenn du einen Taster 
drückst? Dann springt er sofort in die ISR.
Welche Taktquelle hat der Timer? Ist sie richtig konfiguriert?
Ansonsten kann ich mich nur meinem Vorredner anschließen. ISR so kurz 
wie nur irgend möglich. Keine "printf", etc, nichts was rechenintensiv 
wäre. Und riskante Doppelbennenungen vermeiden. Nie irgendwas so nennen 
wie es der Compiler tun könnte. Eigenlich sollte der C-Compiler Groß- 
und Kleinschreibung unterscheiden aber es passieren da u.U auch Fehler.

[David M. (mic_dave)]

Vielen dank erstmal für eure Antworten. Ja das mit der Namesgebeung ist 
mir echt misslungen, wird auch sofort abgeändert. tmr wird korrekt 
angezeigt. Und wie ich die Ausgaben aus dem ISR bekomme, wollte ich mir 
auch noch überlegen, da, wie ihr schon sagtet, der ISR so kurz wie 
möglich gehalten werden sollte. Aber nun nochmal zum eigentlichen 
Problem. Wenn ich meine Ausgabe nun wie folgt gestalte:

    sprintf(text, "(2, %d, %d, ", TASTER1, tmr);
    putUsart(text);
    sprintf(text, "%d, %d)<1>", TMR0H, TMR0L);
    putUsart(text);

d.h. wenn ich die Ausgabe in mehrere Teile aufspalte, dann funktioniert 
das. Es ist also kein Problem am Timer, sondern an dem sprintf, was mir 
ein Rätsel ist.
Hat da irgendjemand eine Idee, woran das liegen könnte?

David