Einfacher Atmega 328 Timer code ungewollte pausen

Hi Jim!

leider half das auch nicht...

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.

Die einfachste Variante:

1

volatile char t;

2

   ...

3

   while (1)

4

      PORTA = (t & 1) ? 0xff : 0x00;

5

   ...

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.

> 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:

1

// user timers, will count down to zero and then stop

2

volatile uint8_t t1, t2, t3; // t1 for leds, t2 and t3 are free

3

4

ISR (TIMER1_COMPA_vect)

5

{

6

    if (t1) t1--;

7

    if (t2) t2--;

8

    if (t3) t3--;

9

}

10

11

int main() {

12

    // setup

13

    ...

14

    for (;;) {

15

        if (t1 == 0) { // led timer expired?

16

            // turn leds off and on

17

            ...

18

            t1 = 4; // restart led timer

19

        }

20

        // do something else

21

        ...

22

    }

23

}

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

1

#include <avr/io.h>

2

3

int main(void)

4

{

5

  // connect led to pin PC0

6

  DDRB |= (1 << 5);

7

8

  TCCR1B |= (1 << WGM12)|(1 << CS11)|(1 << CS10);

9

  TCNT1 = 0;

10

  OCR1A = 25000;

11

12

  int counter = 0;

13

14

  while(1)

15

  {

16

    counter = 0;

17

    if (TIFR1 & (1 << OCF1A))

18

    {

19

    counter = 1;

20

    for (int i=0; i < 30000; i++);  

21

    }

22

    PORTB ^= (1 << 5);

23

    TIFR1 |= (1 << OCF1A);    

24

  }

25

}

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?

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.

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?

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)