Attiny2313 Timer1-Interrupt löst nur beim 1. mal aus.

[Matze (Gast)]

Hallo,
Bin grade dabei einen Würfel mit einem Attiny 2313 zu programmieren.
Dabei soll Zuerst ein taster gedrückt, --> Loswürfeln.
Wird der Taster losgelassen, so bleibt der Würfel langsam stehen.
Wenn er steht, binkt das Ergebniss.
Nach einiger zeit soll er sich selbts wieder abschalten.

Diese zeit soll durch einen Timer eingestellt werden.

Beim 1. Würfeln beim anlegen der Spannung würfelt er richtig, danach 
hört er nicht mehr auf zu blinken.
--> ISR Wird nicht mehr aufgerufen.

Nun muss es einen bedeutenden Unterschied zwischen der ersten 
Initialisierung:

  TCCR1A  &= (0<<COM1A0)|(0<<COM1A1);  // CTC Modus
  TCCR1B &= (0<<ICNC1)|(0<<ICES1)|(0<<CS11);  // Prescaler 1024
  TCCR1B |= (1<<WGM12)|(1<<CS10)|(1<<CS12);  
  OCR1A = 0x0020;
  TIMSK |= (1<<OCIE1A);      //Compare Interrupt Enable
       sei();                     //Globale Interrupts ein

Und allen weiteren Initialisierungen geben:

      TCCR1A &= (0<<COM1A0)|(0<<COM1A1);      // CTC Modus
      TCCR1B &= (0<<ICNC1)|(0<<ICES1);    
      TCCR1B |= (1<<WGM12);
      TCNT1=0x0000;                //Zähler rücksetzen 
      OCR1A = 0x0020;
      TIMSK |= (1<<OCIE1A);            //Compare Interrupt Enable
      sei();                    //Globale Interrupts ein
      TCCR1B |= (1<<CS10)|(1<<CS12);        // Timer Starten
      TCCR1B &= (0<<CS11);

Ich finde ihn nicht.
Muss ich eventuell irgendein Bit, das bei jedem das bein Aufruf der ISR 
gesetzt wird immer wieder zurücksetzen? oder Ähnliches?

Grüße
Matze

[Peter II (Gast)]

was soll denn das werden

> TCCR1B &= (0<<CS11);

diese anweisung ist zwar kein Fehler aber ziemlich unsinnig. Es macht

TCCR1B = 0;

wenn das gewollst ist, dann schreibe es auch so.

[Little Helper (Gast)]

Und genau das ist nicht gewollt.
Durch "TCCR1B &= (0<<CS11);" deaktivierst Du den Timer gleich wieder.

[Dietrich L. (dietrichl)]

Matze schrieb:
> Nun muss es einen bedeutenden Unterschied zwischen der ersten
> Initialisierung:
...
> Und allen weiteren Initialisierungen geben:

Wieso gibt es 2 Initialisierungen?

Üblicherweise wird am Anfang vor der Endlosschleife einmal "richtig" 
initialisiert, später ggf. nur einige Parameter geändert. Dann ist auch 
übersichtlicher, was man tut und warum man das tut.

Gruß Dietrich

[Christian (Gast)]

Meistens sind diese Bits eh mit 0 initialisiert.
Ich würde nur die setzen, die wirklich nötig sind.

[Matze (Gast)]

Hab das Problem gelöst, und initialisiere res Register direkt mir 
Hex-Werten.

0x08=Stoppen
0x0d=Starten

Danke für eure Hilfe
Grüße
Matze

[Peter II (Gast)]

Matze schrieb:
> Hab das Problem gelöst, und initialisiere res Register direkt mir
> Hex-Werten.
>
> 0x08=Stoppen
> 0x0d=Starten

das ist doch keine Lösung, lerne lieber wie man bits ordentlich löscht.

http://www.mikrocontroller.net/articles/Bitmanipulation

[Little Helper (Gast)]

Peter II schrieb:
> das ist doch keine Lösung, lerne lieber wie man bits ordentlich löscht.

Das ist ziemlich streng von Dir und so nicht richtig.
Eine Lösung ist, wenn es funktioniert.
Dass es nicht die schickste Lösung ist, stimme ich zu.

Für den ThreadOwner die Zusammenfassung:
Bits setzen:   REGISTER |= (1<<BIT1_NAME) | (1<<BIT2_NAME)
Bits loeschen: REGISTER &= ~((1<<BIT1_NAME) | (1<<BIT2_NAME))