ATmega644 CTC Timer1 Problem

[prinzmi (Gast)]

Ich wollte einen CTC Timer einstellen, der jede ms in die 
ISR(TIMER1_COMPA_vect) springt.

Das ist der Code:

TCCR1B |= (1<<WGM12) | (1<<CS11) | (1<<CS12); //Prescaler 64
  TCNT1 = 0;
  OCR1A = 249;
  TIMSK1 |= (1 << OCIE1A);

Leider funktioniert es aus mir unerklärlichen Gründen nicht. Es wird die 
ISR nicht aufgerufen.

Jede Hundertstelsekunde klappt aber:

TCCR1B |= (1<<WGM12) | (1<<CS12); //Prescaler 256
  TCNT1 = 0;
  OCR1A = 624;
  TIMSK1 |= (1 << OCIE1A);

Kann mir das jemand erklären. Ich steh voll auf dem Schlauch!

[Uwe (de0508)]

prinzmi,

gibt es für den ATmega644 auch eine Taktfrequenz?

Ich gehe mal von 16MHz aus.

Es liegt natürlich an dieser Zeile, die keinen 1:64 Teiler einstellt.

TCCR1B |= (1<<CS11) | (1<<CS12); //Prescaler 64

das wäre natürlich diese Maske:

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

[Joachim B. (jar)]

prinzmi schrieb:
> Das ist der Code:TCCR1B |= (1<<WGM12) | (1<<CS11) | (1<<CS12);
> //Prescaler 64

wo hast du das her?

Uwe S. schrieb:
> das wäre natürlich diese Maske:TCCR1B |= (1<<CS11) | (1<<CS10);

stimmt natürlich, siehe auch

http://modelleisenbahn-steuern.de/controller/atmega8/12-13-2-tccr1b-atmega8.htm

[prinzmi (Gast)]

Ach du meine Güte!!! Da sitze ich fast eine Stunde über dem Datenblatt 
und dann sehe ich den Wald vor lauter Bäumen nicht.

Uwe S. schrieb:
> das wäre natürlich diese Maske:TCCR1B |= (1<<CS11) | (1<<CS10);

Steht natürlich eindeutig im Datenblatt. Keine Ahnung warum ich das 
immer falsch gelesen habe.
Vielen Dank.

[Kai M. (kai_mauer)]

Joachim B. schrieb:
> stimmt natürlich, siehe auch
>
> http://modelleisenbahn-steuern.de/controller/atmega8/12-13-2-tccr1b-atmega8.htm

Das kann in's Auge gehen, denn dort ist der Atmega8 gemeint, nicht der 
644.
Die Bits sind ja leider nicht in jedem Falle in den gleiche Registern.

[Joachim B. (jar)]

Kai Mauer schrieb:
> Das kann in's Auge gehen, denn dort ist der Atmega8 gemeint,

da habe ich ja mit dem Wechsel vom m328p zum 1284p Glück gehabt, da 
passt es.

Sogar bei den Timer3 liegt alles an "seinem" Platz so das sich die 
#defines leicht ändern liessen.

#if defined(__AVR_ATmega328P__) || defined(M1284P_TIM1)
  #define   TIMSKx              TIMSK1
  #define   OCIExA              OCIE1A
  #define   TIMERx_COMPA_vect   TIMER1_COMPA_vect  // ATmega
  #define   TCCRxA              TCCR1A
  #define   COMxA0              COM1A0
  #define   OCRxA               OCR1A
  #define   TCCRxB              TCCR1B
  #define   WGMx2               WGM12
  #define   CSx0                CS10

  #define ARDUINO_LED      5
  #define DDR_ARDUINO_LED  DDRB
  #define PORT_ARDUINO_LED PORTB
  #define ARDUINO_LED_ON   PORT_ARDUINO_LED|=(1<<ARDUINO_LED)
  #define ARDUINO_LED_OFF  PORT_ARDUINO_LED&=~(1<<ARDUINO_LED)

#elif defined(__AVR_ATmega1284P__)
  #define   TIMSKx              TIMSK3
  #define   OCIExA              OCIE3A
  #define   TIMERx_COMPA_vect   TIMER3_COMPA_vect  // ATmega
  #define   TCCRxA              TCCR3A
  #define   COMxA0              COM3A0
  #define   OCRxA               OCR3A
  #define   TCCRxB              TCCR3B
  #define   WGMx2               WGM32
  #define   CSx0                CS30
  #define ARDUINO_LED      7
  #define DDR_ARDUINO_LED  DDRB
  #define PORT_ARDUINO_LED PORTB
  #define ARDUINO_LED_ON   PORT_ARDUINO_LED|=(1<<ARDUINO_LED)
  #define ARDUINO_LED_OFF  PORT_ARDUINO_LED&=~(1<<ARDUINO_LED)


#endif

Kai Mauer schrieb:
> Die Bits sind ja leider nicht in jedem Falle in den gleiche Registern.

stimmt, man muss immer schauen!

[Thomas E. (thomase)]

Joachim B. schrieb:
> siehe auch
>
> http://modelleisenbahn-steuern.de/controller/atmega8/12-13-2-tccr1b-atmega8.htm

Unsinn. Sondern siehe:

http://www.atmel.com/Images/doc8152.pdf

mfg.

[Joachim B. (jar)]

Thomas Eckmann schrieb:
> Unsinn.

warum? war der erste Sucheintrag und passte

Thomas Eckmann schrieb:
> Sondern siehe:
>
> http://www.atmel.com/Images/doc8152.pdf

klar es gibt mehr als einen Weg, muss ja nicht Rom sein.

Dein Unsinn ist Unsinn :p

[Thomas E. (thomase)]

Joachim B. schrieb:
> warum? war der erste Sucheintrag und passte

Das ist ein Atmega8. Alao passt da gar nichts.

mfg.

[Joachim B. (jar)]

Thomas Eckmann schrieb:
> Das ist ein Atmega8. Alao passt da gar nichts.

müde? sauer? oder bloß unsympatisch?

Joachim B. schrieb:
> TCCR1B |= (1<<CS11) | (1<<CS10);

schaue ich ins 644 Datenblatt
http://www.atmel.com/Images/doc2593.pdf
steht dort auf Seite 127:

ICNC1 ICES1 – WGM13 WGM12 CS12 CS11 CS10 TCCR1B

und auf Seite 128:
Table 14-6.
Clock Select Bit Description
CSn2 CSn1 CSn0 Description
0    0    0    No clock source (Timer/Counter stopped).
0    0    1    clk I/O /1 (No prescaling)
0    1    0    clk I/O /8 (From prescaler)
0    1    1    clk I/O /64 (From prescaler)
1    0    0    clk I/O /256 (From prescaler)
1    0    1    clk I/O /1024 (From prescaler)

Thomas Eckmann schrieb:
> Alao passt da gar nichts.

doch weil viele Megas sich an die Registernamen halten (prüfen muss man 
natürlich ob es für jeden Mega gilt).

also der gewünschte Vorteiler 64 kann mit CS10 und CS11 gesetzt werden, 
nicht anderes war dort beschrieben und das ist RICHTIG,
egal ob es aus dem Atmega8 Datenblatt stammt oder aus dem 644p 
Datenblatt.

Also wer oder was willst du sein?