Einfacher Atmega 328 Timer code ungewollte pausen

[fred (Gast)]

Hallo Mikrocontroller-Freunde,

ich versuche einen Atmega328p eine LED jede zweite sekunde eine sekunde 
leuchten zu lassen. Habe ich jetzt hinbekommen mit dem Timer1,
macht der MCU nach 4 mal blinken 10s lang gar nichts...

#include <avr/io.h>
  #include <avr/interrupt.h>

  int t = 0;
  int main(void)
  {
    DDRB = 0b000100000;
    OCR1A = 0x3D08;

    TCCR1B |= (1 << 3);
    // Mode 4, CTC on OCR1A

    TIMSK1 |= (1 << 1);
    //Set interrupt on compare match

    TCCR1B |= 0x05;
    // set prescaler to 1024 and start the timer


    sei();
    // enable interrupts


    while (1)
    {
      while(t==1)
      {
        PORTB = 0xFF;
      }
      PORTB = 0x00;
      t=0;
      
    }
  }

  ISR (TIMER1_COMPA_vect)
  {
    t++;
  }

Könnt ihr mir weiterhelfen? Ich bin am verzweifeln :(
Danke

[Jim M. (turboj)]

volatile int t = 0;

[fred (Gast)]

Hi Jim!

leider half das auch nicht...

[foobar (Gast)]

Die Interaktion zwischen Interruptroutine und Hauptprogramm robust zu 
bekommen, ist nicht trivial. Das volatile nannte Jim schon. Reicht 
leider nicht, du brauchst auch noch nen atomaren Datentyp (z.B. char 
oder uint8_t).

Und dann muß das Protokoll auch noch korrekt sein - deines ist es 
nicht. Stell dir vor, t ist 0 und kurz nach dem Test "while(t==1)" und 
noch vor dem "t=0" kommt der Interrupt, t wird 1 und von main sofort 
wieder auf 0 gesetzt ohne jemals gemerkt zu haben, dass es kurz auf 1 
war.

[foobar (Gast)]

Die einfachste Variante:

volatile char t;
   ...
   while (1)
      PORTA = (t & 1) ? 0xff : 0x00;
   ...

[fred (Gast)]

Hi foobar,

leider verstehe ich die variante nicht wirklich...
und später muss/ will ich die eine LED durch drei ersetzen.
Dann muss jede led 4s nacheinander blinken mit einem zeitabstand von 4s.

[foobar (Gast)]

> leider verstehe ich die variante nicht wirklich

Die läßt den Timer einfach durchlaufen. Je nachdem, ob der gerade oder 
ungerade ist, werden die LEDs ein- oder ausgeschaltet.

Etwas flexibler wäre man z.B. mit so etwas:

// user timers, will count down to zero and then stop
volatile uint8_t t1, t2, t3; // t1 for leds, t2 and t3 are free

ISR (TIMER1_COMPA_vect)
{
    if (t1) t1--;
    if (t2) t2--;
    if (t3) t3--;
}

int main() {
    // setup
    ...
    for (;;) {
        if (t1 == 0) { // led timer expired?
            // turn leds off and on
            ...
            t1 = 4; // restart led timer
        }
        // do something else
        ...
    }
}

[fred (Gast)]

Jetzt habe ich was anderes ausprobiert,
und ich versteh die welt nicht mehr.

#include <avr/io.h>

int main(void)
{
  // connect led to pin PC0
  DDRB |= (1 << 5);
  
  TCCR1B |= (1 << WGM12)|(1 << CS11)|(1 << CS10);
  TCNT1 = 0;
  OCR1A = 25000;
  
  int counter = 0;

  while(1)
  {
    counter = 0;
    if (TIFR1 & (1 << OCF1A))
    {
    counter = 1;
    for (int i=0; i < 30000; i++);  
    }
    PORTB ^= (1 << 5);
    TIFR1 |= (1 << OCF1A);    
  }
}

füge ich bei
[code]
if (TIFR1 & (1 << OCF1A))
    {
    counter = 1;
    for (int i=0; i < 30000; i++);
    }
[\code]

dies ein:
[code]
if (TIFR1 & (1 << OCF1A))
    {
    PORTB ^= (1 << 5);
    }
[\code]

Warum bekomme ich wenn ich die for-schleife im if-vergleich drin habe 
ein schnelleres blinken? und wenn nur der toggelbefehl drin steht die 
gewünschte zeitspanne?

[foobar (Gast)]

Zielloses rumballern auf Registern ist selten sinnvoll (deine "while 
(1)" Loop wird mehrere 100000 mal pro Sekunde durchlaufen...). Deine 
zwei Varianten machen halt was ganz anderes. Btw, die for-busy-loop wird 
eh wegoptimiert.

[Peter D. (peda)]

foobar schrieb:
> Stell dir vor, t ist 0 und kurz nach dem Test "while(t==1)" und
> noch vor dem "t=0" kommt der Interrupt, t wird 1 und von main sofort
> wieder auf 0 gesetzt ohne jemals gemerkt zu haben, dass es kurz auf 1
> war.

So ist es. Das Programm ist logisch falsch.
Ein Interrupt kommt asynchron zum Main und das muß man berücksichtigen.

[fred (Gast)]

Ok, wie kann ich den zweiten code richtigstellen,
das ich anstatt einem getoggelten PortB einen Variable aufrufe und 
inkrementiere und diese variable dann für drei leds zum leuchten 
benutzen kann?

[Peter D. (peda)]

fred schrieb:
> Ok, wie kann ich den zweiten code richtigstellen,

Welchen 2. Code denn?
Hint: Man darf (sollte) verlinken oder zitieren.

Delay mit Interrupt zu vermanschen ist ganz großer Mumpitz. Du mußt Dich 
für eins entscheiden.

fred schrieb:
> dann für drei leds zum leuchten
> benutzen kann?

Beschreibe mal genau, was die 3 LEDs machen sollen.

[Stefan F. (Gast)]

fred schrieb:
>>PORTA = (t & 1) ? 0xff : 0x00;
> leider verstehe ich die variante nicht wirklich...

Das bedeutet in Worten:

Wenn in t das Bit 1 gesetzt ist, dann setze PORTA auf 0xff (alle 
Ausgänge auf High).
Wenn in t das Bit 1 nicht gesetzt ist, dann setze PORTA auf 0x00 (alle 
Ausgänge auf High).

Ziel = (Ausdruck) ? (Wert wenn wahr) : (Wert wenn falsch)