Timer und UART vertragen sich nicht

Hat noch niemand dies Problem gehabt?
Johannes

Ohne halbwegs kompletten Quelltext wird Dir niemand helfen können. Bau
am besten mal ein lauffähiges Minimalbeispiel, um den Fehler zu
reproduzieren, oft findet man die Ursache dann auch selbst.

Eine Vermutung: Die Timer-ISR braucht zu lange, z.B. durch
Warteschleifen innerhalb der ISR, so daß die UART-Interrupts nicht
schnell genug bedient werden.

Was sind denn die Symptome für die fehlerhafte Übertragung? Rührt sich
gar nix, oder kommt nur Datenmüll an (z.B. auch Framing-Error), oder
gehen einzelne Zeichen verloren? Verwendest Du Ringpuffer für
eingehende und ausgehende Daten?

MfG Olaf

Hallo Olaf,

die ISR fuer den Timerueberlauf hab ich noch gar nicht eingebunden.
In meinem "Minimalbeispiel" hab ich einfach nur den Timer
initialisiert und gestartet wie im Quelltext oben. Das hat zur Folge,
dass der UART nicht mehr geht.
Inzwischen hab ich mit einema nderen Timer (16bit) gearbeitet und wenn
der läuft, egal ob mit oder ohne seine ISR, funzt der UART.

Ich poste gerne auch den ganzen Quellcode (bitte bescheid sagen!); aber
ehrlich: ich glaube kaum, dass sich jemand die Arbeit macht fuer mich in
meinem Code Fehler zu suchen...

mg,
Johannes

"die ISR fuer den Timerueberlauf hab ich noch gar nicht eingebunden."


Jo, dann gehts natürlich ab in den Wald.

Bevor man einen Interrupt freigibt, muß ein Handler am entsprechenden
Interruptvektor existieren !!!

Für diese Regel gibt es keine Ausnahme !!!


Peter

Aha!

ich nehm alles wieder zurück. Das hab ich nicht gewusst.
Dachte, der Timer laeuft dann vor sich hin...

hmm...Compilerspezialist bin ich nicht, aber sowas koennte der ja auch
merken, oder? ich nehm die Schuld aber ganz auf meinen Puckl,stell mich
5min in eine Ecke und schäme mich dreimal ganz laut!

es tut jetzt vorläufig wie ich will!

mg,
Johannes

Einen Peter Dannegger im Forum zu haben ist Luxus. Ist nicht das
erstemal gewesen, dass dein Tip mich aufgeklaert hat. Danke,
mg,
Johannes

ich auch.
Man sollte die Liste vielleicht noch verlängern (ohne Wertung in Bezug
auf die Reihenfolge:
Rufus, HanneS, Jörg Wunsch, OldBug, CrazyHorse.
Das sind die, die mir jetzt gerade eingefallen sind.

wer sind denn die anderen 4???
:-)

Könnt ihr mir vielleicht sagen wann ich denn extern Interrupt auf 1
setzen soll und wann und wie???

#pragma small code symbols debug
#include "bremslicht.h"

unsigned int aktuellerWert; /* speichert die aktuelle negative
Beschleunigung 0..100 */
unsigned int leuchtendeLEDs; /* speichert die Anzahl anleuchtenden LEDS
*/
float LEDratio; /* hier wird errechnet, wie viel  % eine LED
"repräsentiert" */

unsigned int timer0_wert;

/*

  INT0 Verlauf über die Zeit (Input vom Accelerometer, PWM moduliert)

  ...----><---T1---><--------T2-----------><--T1--><----...
           _______                         _______
          |        |                       |        |
          |        |                       |        |
   _______|        |_______________________|        |_____________

                   |<--Interrupt                   |<--Interrupt

          |<--timer0 fängt an hier         |<--timer0 fängt an hier
              hochzuzählen, weil               hochzuzählen, weil
              GATE0=1 und INT0 high            GATE0=1 und INT0 high

   -----------------------------Zeit------------------------------->

*/

void irq_timer0(void) interrupt 1
{

    timer0_wert = 256*TH0 + TL0;




    TL0 = 0x00;
    TH0 = 0x00;
    TF0 = 0;
    TF1 = 0;



    TMOD = 0x19;//0001=vom Tim1 M0 auf1, 1001=Auslösen desTim0,dann
16Bit Zähler laufen lassen




}


void LEDsInit()
{
    register int i;
    for (i = 0; i < ANZAHL_LEDS; i++)
    {
        LEDs[i] = 0;    // schalte LED aus
    }
    MAX7300SetzeLEDs(LEDs);
}

void AccMeterInit() {

    TR0 = 0;

    TMOD = 0x19;

    TH0 = 0x00;
    TL0 = 0x00;

    ET0 = 1;
    EA = 1;

    IT0=1;// TCON = TCON | 0x11;
    TR0=1;

}



void Initialisieren()
{
    MAX7300Init();
    LEDsInit();
    aktuellerWert = 0;
    leuchtendeLEDs = 0;
    LEDratio = (float) 100 / ANZAHL_LEDS;

    timer0_wert = 0;
    AccMeterInit();
}


void HoleWert() {
    if (timer0_wert > 0 ) {

        aktuellerWert = (int) ((timer0_wert / TIMER1_MAX) * 100);

        if (aktuellerWert < 50) {
            aktuellerWert *= 2;
        }
        else {
            //positive Beschleunigung interessiert uns nicht
            aktuellerWert = 0;
        }
    }
}

void BerechneWert()
{
    leuchtendeLEDs = (unsigned int) aktuellerWert / LEDratio;
}

void AusgabeWert()
{
    register int i;
    if (leuchtendeLEDs > 0) {
        P1_6 = 1;
        P1_7 = 1;
    }
    else {
        P1_6 = 0;
        P1_7 = 0;
    }
    for (i = 0; i < leuchtendeLEDs; i++)
    {
        LEDs[i] = 1;    //lass LED leuchten
    }

    for (i = leuchtendeLEDs; i < ANZAHL_LEDS; i++)
    {
        LEDs[i] = 0;    // schalte LED aus
    }
    MAX7300SetzeLEDs(LEDs);
}

void main()
{
    Initialisieren();
    for (;;) //forever
    {
        HoleWert();
        BerechneWert();
        AusgabeWert();
    }
}

bitte i need help

Das musst du jetzt noch 7mal schreiben. Ab 10 Wiederholungen setzt sich
sofort einer hin und macht dir das.

Und längeren Code immer als Amhang, sonst scrollt man sich ja dämlich.


Peter