Kriege Timer 1 im Mega2560 nicht zum laufen!

[Mike Litoris (Gast)]

Hi!

Folgendes Problem: Ich brauche auf einem MEGA2560 einen Timerinterrupt 
im Sekundentakt.
Denke ein 16bit timer entsprechend konfiguriert ist da die beste Wahl.

Bis jetzt habe ich immer mit MEGA328 gespielt, habs da auch immer 
hinbekommen, den Timer richtig zu konfigurieren. Leider klappt es dieses 
mal nicht.

Mein Code:

//TIMER STARTEN
  //OC Pins abkoppeln
  TCCR1A |= ((1 << COM1A1));
  TCCR1A |= ((1 << COM1A0));

  //CTC Modus verwenden
  TCCR1A |= ((1 << WGM11));
  TCCR1A |= ((1 << WGM10));
  
  //PRESCALER 1024
  TCCR1B |= ((1 << CS12));
  TCCR1B |= ((1 << CS11));
  TCCR1B |= ((1 << CS10));
  
  //CTC Modus verwenden
  TCCR1B |= ((1 << WGM13));
  TCCR1B |= ((1 << WGM12));

  //Interrupte einschalten
  TIMSK1 |= ((1 << OCIE1A));


  OCR1A = 15624;

  sei();

sowie weiter unten im Code:

ISR(TIMER1_COMPA_vect)
{
  digitalWrite(RoteLEDPin, HIGH);
  delay(10);
  digitalWrite(RoteLEDPin, LOW);
  Serial.print("interrupt");

  
}

Erkennt jemand den Fehler? Ich bin das ganze Kapitel im Datenblat 
durchgegangen, anscheinend hab ich was übersehen?!

Grüße und danke!

[Hubert G. (hubertg)]

Mit WGM10, 11, 12, 13 hast du FastPWM eingestellt.

[spess53 (Gast)]

Hi

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

Das ist External clock source on Tn pin. Clock on rising edge.
Willst du das?

MfG spess

[Mike Litoris (Gast)]

Hi!


Komisch, warum ich das übersehen hatte.

Sieht jetzt so aus:

//TIMER STARTEN
  //OC Pins abkoppeln
  TCCR1A |= ((1 << COM1A1));
  TCCR1A |= ((1 << COM1A0));

  //CTC Modus verwenden
  //TCCR1A |= ((1 << WGM11));
  //TCCR1A |= ((1 << WGM10));
  
  //PRESCALER 1024
  TCCR1B |= ((1 << CS12));
  //TCCR1B |= ((1 << CS11));
  //TCCR1B |= ((1 << CS10));
  
  //CTC Modus verwenden
  //TCCR1B |= ((1 << WGM13));
  TCCR1B |= ((1 << WGM12));

  //Interrupte einschalten
  TIMSK1 |= ((1 << OCIE1A));


  OCR1A = 15624;

  sei();

geht aber immer noch nicht...

[Felix A. (madifaxle)]

Du veroderst immer in der Form "TCCR1A |=". Setzt du das Teil auch im 
Initfall mal auf einen konstanten Wert wie etwa 0 oder greifst du nur 
hier auf das Register zu und verlässt dich auf den Reset des 
Controllers?

[spess53 (Gast)]

Hi

>  TCCR1A |= ((1 << COM1A1));
>  TCCR1A |= ((1 << COM1A0));

Passt nicht zum Kommentar.

>  //CTC Modus verwenden
>  TCCR1B |= ((1 << WGM12));

bei CTC mit OCR1A als Top muss WGM12 und WGM13 gesetzt sein.

MfG Spess

[Thomas E. (thomase)]

Mike Litoris schrieb:
> //OC Pins abkoppeln
>   TCCR1A |= ((1 << COM1A1));
>   TCCR1A |= ((1 << COM1A0));

Was bezweckst du denn damit?

mfg.

[Frederik H. (diveturtle93)]

Hallo

> //Interrupte einschalten
> TIMSK1 |= ((1 << OCIE1A));

Hier hast du ein Falsches Register benutzt.

Es müsste nur TIMSK heißen.

[Mike Litoris (Gast)]

spess53 schrieb:
> bei CTC mit OCR1A als Top muss WGM12 und WGM13 gesetzt sein.

Ich brauch doch Mode 4 und nicht Mode 12? Für Mode 4 nur WGM12.

Felix A. schrieb:
> Du veroderst immer in der Form "TCCR1A |=". Setzt du das Teil auch im
> Initfall mal auf einen konstanten Wert wie etwa 0 oder greifst du nur
> hier auf das Register zu und verlässt dich auf den Reset des
> Controllers?

Der Rest sollte automatisch 0 sein, oder?

Thomas E. schrieb:
> Mike Litoris schrieb:
>> //OC Pins abkoppeln
>>   TCCR1A |= ((1 << COM1A1));
>>   TCCR1A |= ((1 << COM1A0));
>
> Was bezweckst du denn damit?

"These bits control the Output Compare pin (OC0A) behavior. If one or 
both of the COM0A1:0 bits are set, the
OC0A output overrides the normal port functionality of the I/O pin it is 
connected to."
Das brauch ich nicht, deswegen COMA1 und COMA0 auf 0 für "Normal port 
operation, OC0A disconnected."

Danke schonmal!

[Thomas E. (thomase)]

spess53 schrieb:
> bei CTC mit OCR1A als Top muss WGM12 und WGM13 gesetzt sein.

Nee, das ist schon richtig so. 12/13 ist mit ICR1 als Top.

mfg.

[Thomas E. (thomase)]

Frederik H. schrieb:
> Es müsste nur TIMSK heißen.

Nein. Es gibt ein Leben nach dem Atmega8.

Mike Litoris schrieb:
> Das brauch ich nicht, deswegen COMA1 und COMA0 auf 0 für "Normal port
> operation, OC0A disconnected."

Und warum setzt du das dann auf 1?

Hängt da deine LED dran?

mfg.

[Felix A. (madifaxle)]

Normal Port Operation, also kein Zugriff durch den Timer, geht über 
COM1A1=0 und COM1A0=0.

OCR1A wird benutzt, wenn nur WGM12=1.

Im TCCR1B den Takt einstellen (das weißt bislang nur du).

Im TIMSK1 nur OCIE1A=1 setzen.

Dann noch den Interruptvektor setzen und sei(); benutzen, um die zu 
aktivieren.

Wenn du immerzu TCCR1A |= ... benutzt über viele Zeilen, wird es zum 
Einen unübersichtlich und zum Anderen (das hängt davon ab) wurde 
vielleicht woanders auf das Register zugegriffen und es steht irgendwo 
eine 1, die aber stört.

Durch TCCR1A = (1<<...) | (1<<..) | 1<<.....) würde TCCR1A einen 
definierten Wert bekommen.

[Kaj (Gast)]

Timer 1 wird von dem Arduinozeug intern gebraucht! Nimm 'nen anderen 
Timer, dann sehen wir weiter.

[Veit D. (devil-elec)]

Kaj schrieb:
> Timer 1 wird von dem Arduinozeug intern gebraucht! Nimm 'nen
> anderen
> Timer, dann sehen wir weiter.

Ist doch gar nicht wahr. Warum wird immer so ein Unsinn erzählt.
Timer 1 wird nur verwendet wenn man mit den analogWrite Befehl auf Pin 9 
und/oder 10 verwendet. Macht man das nicht, ist Timer 1 vollkommen frei 
verwendbar. Andererseits spricht auch nichts dagegen einen Timer 
mehrfach zuverwenden, mit all seinen ISRs. Muß man sich nur vorher mehr 
Gedanken machen.

@ Mike:  mir fehlt der komplette Code?
Und serial.print hat im ISR Handler auch nichts zu suchen. ISR im ISR. 
Macht man nicht. Führt immer zu Problemen.

[Mike Litoris (Gast)]

Hi!

Ich bin das Datenblatt nochmal durchgegangen und habe nun folgende 
Konfiguration:

  //TIMER STARTEN  
        //CTC Modus und 1024 Vorteiler
  TCCR1B = (1<<WGM12) | (1<<CS10) | (1<< CS12);
  TIMSK1 |= (1<<OCIE1A);


  OCR1A = 15625;

  sei();

Damit sollte der Interrupt einmal pro Sekunde aufgerufen werden:
16000000/(1024*15624) = 1


Dies ist aber nicht der Fall, der Interrupt wird sehr viel öfter 
aufgerufen.
Wo liegt nun der Fehler?

Veit D. schrieb:
> Und serial.print hat im ISR Handler auch nichts zu suchen. ISR im ISR.
> Macht man nicht. Führt immer zu Problemen.

Weiss ich, ist nur zum testen.

Felix A. schrieb:
> Wenn du immerzu TCCR1A |= ... benutzt über viele Zeilen, wird es zum
> Einen unübersichtlich und zum Anderen (das hängt davon ab) wurde
> vielleicht woanders auf das Register zugegriffen und es steht irgendwo
> eine 1, die aber stört.
>
> Durch TCCR1A = (1<<...) | (1<<..) | 1<<.....) würde TCCR1A einen
> definierten Wert bekommen.

Hab versucht das zu beherzigen. Danke!

Grüße!

[Karl H. (kbuchegg)]

Veit D. schrieb:

> Ist doch gar nicht wahr. Warum wird immer so ein Unsinn erzählt.
> Timer 1 wird nur verwendet wenn man mit den analogWrite Befehl

Welcher Timer wird im Arduino benutzt um die Systemuhr zu betreiben?

(Disclaimer: Ich weiss es nicht. Ich frage nur. Dass irgendein Timer 
benutzt wird, ist klar. Denn wie soll sonst millis() funktionieren. Die 
Frage ist: welcher?)

[Mike Litoris (Gast)]

Karl H. schrieb:
> Veit D. schrieb:
>
>> Ist doch gar nicht wahr. Warum wird immer so ein Unsinn erzählt.
>> Timer 1 wird nur verwendet wenn man mit den analogWrite Befehl
>
> Welcher Timer wird im Arduino benutzt um die Systemuhr zu betreiben?
>
> (Disclaimer: Ich weiss es nicht. Ich frage nur. Dass irgendein Timer
> benutzt wird, ist klar. Denn wie soll sonst millis() funktionieren. Die
> Frage ist: welcher?)

Ich verwende die Funktion nicht. soweit ich weiss, wird der Timer dafür 
nur verwendet, wenn man die FUnktion mindestens einmal aufruft?

[Karl H. (kbuchegg)]

Mike Litoris schrieb:

> Ich verwende die Funktion nicht. soweit ich weiss, wird der Timer dafür
> nur verwendet, wenn man die FUnktion mindestens einmal aufruft?

Das glaub ich ehrlich gesagt nicht. Das wäre ziemlich unsinnig bei jedem 
millis() Aufruf erst mal lang und breit zu prüfen, ob der Timer nicht 
schon läuft.

Sorry. Aber Arduino funktioniert nicht nach dem Prinzip "You don't pay 
for what you don't use". Wenn das Ardunion System hochfährt krallt es 
sich ein paar Dinge und setzt sein Basisystem auf. Möglich, das einzelne 
Komponenten erst mit der Benutzung allokiert werden. Aber von 
grundlegenden Sachen kann man davon ausgehen, dass die von Anfang an von 
der Basissoftware ohne Rücksicht auf Verluste allokiert werden. Und 
gerade bei Timern wäre ich da vorsichtig. ALs Arbeitspferde für fast 
alles gibt es kaum ein Programm, welches nicht mindestens 1 Timer in 
Beschlag hat.

Aber: das lässt sich ja rauskriegen, indem man in einem Testprogramm 
welches sonst nichts macht, einfach mal in loop() die 
Timerkonfigurationsregister sich ausgeben lässt. Wenn da alles 0 ist, 
dann ist der Timer unbenutzt.

[Mike Litoris (Gast)]

Ich konnte leider nicht rausfinden, welchen Timer die millis() Funktion 
nutzt.

Ich hab den Interrupt daher mal auf Timer 3 umgezogen. Der wird ja 
komplett identisch angesteuert:

  //TIMER STARTEN
  TCCR3B = (1<<WGM12) | (1<<CS10) | (1<< CS12);
  TIMSK3 |= (1<<OCIE3A);


  OCR3A = 15624;

  sei();

und:

ISR(TIMER3_COMPA_vect){
  
  digitalWrite(RoteLEDPin, HIGH);
  delay(10);
  digitalWrite(RoteLEDPin, LOW);
  Serial.println("interrupt");
}

selbes Phänomen wie mit Timer 1: Interrupt wird ausgelößt, aber viel zu 
schnell...

[Mike Litoris (Gast)]

Hier nochmal der gesamte Code:

//Array zum Speichern aller analogen Eingänge
int AnalogWert[16] = {0};

//Pindefinitionen Ausgänge
int RoteLEDPin = 13;      //Rote LED


void setup() {
  Serial.begin(115200);

  //Alle LED Ausgänge setzen
  pinMode(RoteLEDPin, OUTPUT);

  TimerConfig();
}

void loop() {

}

void TimerConfig()
{
  TCCR5B = 0;
  TCCR5B |= (1<<WGM52) | (1<<CS50) | (1<< CS52);
  
  TIMSK5 = 0;
  TIMSK5 |= (1<<OCIE5A);

  OCR5A = 15624;
  sei();
}


ISR(TIMER5_COMPA_vect){
  
  digitalWrite(RoteLEDPin, HIGH);
  digitalWrite(RoteLEDPin, LOW);
}

Das Signal am Pin 13 hat ungefähr eine Frequenz von 61Hz. Es sollte 1 Hz 
sein.

16MHz, 1024 Vorteiler, 15624 Compare Wert...

Jemand eine Idee?

[Joachim B. (jar)]

Veit D. schrieb:
> Kaj schrieb:
>> Timer 1 wird von dem Arduinozeug intern gebraucht! Nimm 'nen
>> anderen
>> Timer, dann sehen wir weiter.
>
> Ist doch gar nicht wahr. Warum wird immer so ein Unsinn erzählt.
> Timer 1 wird nur verwendet wenn man mit den analogWrite Befehl auf Pin 9
> und/oder 10 verwendet.

hmm, komisch, ich würde dir ja gerne glauben, aber ich habe hier ein 
Programm das auf dem Arduino nano328p auf Timer1 läuft, das selbe (OK 
das gleiche) Programm eben nicht mit Timer1 auf dem mighty m1284p 
Arduino Clone, aber eine kleine Änderung zu Timer3 und es läuft eben.

Ich finde keine Erklärung warum der mighty Arduino Clone nicht mit dem 
Programm auf Timer1 läuft.

[Mike Litoris (Gast)]

Joachim B. schrieb:
> hmm, komisch, ich würde dir ja gerne glauben, aber ich habe hier ein
> Programm das auf dem Arduino nano328p auf Timer1 läuft, das selbe (OK
> das gleiche) Programm eben nicht mit Timer1 auf dem mighty m1284p
> Arduino Clone, aber eine kleine Änderung zu Timer3 und es läuft eben.
>
> Ich finde keine Erklärung warum der mighty Arduino Clone nicht mit dem
> Programm auf Timer1 läuft.

Nagut, ich habs jetzt mit timer 1, 3 und 5 ausprobiert. Immer das selbe 
Ergebnis.

Ich verstehe auch ehrlich gesagt nicht, was noch falsch sein könnte. 
SOllte eigentlich alles passen jetzt?

Grüße!

[Karl H. (kbuchegg)]

Mike Litoris schrieb:

> Ich verstehe auch ehrlich gesagt nicht, was noch falsch sein könnte.
> SOllte eigentlich alles passen jetzt?


Sehe ich auch so. Habs 3 mal nachgerechnet. Aber ein Faktor ~60 und das 
noch dazu zu schnell ist ungewöhnlich. Da klingelt gar nichts.

Moment. Strohhalm greif.
Hast du schon mal probiert, ob dein µC ständig resetet?
Im setup(): Led ein, delay 1 Sekunde, LED aus, delay 1 Sekunde

[Felix A. (madifaxle)]

Hast du mal ins OCR5A geschrieben und eine Sekunde später den Wert aus 
diesem Register gelesen und mal per Uart an ein Terminal gesendet? 
Scheint zwar irgendwie sinnlos, aber vielleicht gibt es da irgendwo ne 
Macke...

[Felix Adam (Gast)]

Oder noch einfacher: gibt die Arduino-IDE ein Listfile (.lss oder sowas) 
aus, in dem man den Quellcode samt Assemblercode sieht? Da könnte sich 
auch ein Blick lohnen.

[Joachim B. (jar)]

Mike Litoris schrieb:
> Nagut, ich habs jetzt mit timer 1, 3 und 5 ausprobiert. Immer das selbe
> Ergebnis.

hast du nachgesehen ob alle Bits in den passenden Register liegen?
vielleicht hilft das ja?

#if defined(__AVR_ATmega328P__)
  #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 LED_ARDUINO      5
  #define LED_ARDUINO_DDR  DDRB
  #define LED_ARDUINO_PORT PORTB
  #define LED_ARDUINO_ON   LED_ARDUINO_PORT|=(1<<LED_ARDUINO)
  #define LED_ARDUINO_OFF  LED_ARDUINO_PORT&=~(1<<LED_ARDUINO)

// arduino nano v3
// D0 (Rx)        PD0
// D1 (Tx)        PD1
// D2 (Int0)      PD2
// D3 (Int1)      PD3
// D4 (XCK/T0)    PD4
// D5 (T1)        PD5
// D6 (AIN0)      PD6
// D7 (AIN1)      PD7
// D8 (ICP1)      PB0
// D9 (OC1A)      PB1 (PWM)
// D10(SS/OC1B)   PB2 (PWM)
// D11(MOSI/OC2)  PB3 (PWM)
// D12(MISO)      PB4
// D13(SCK)       PB5 (LED)

// ANALOG
// A0 (ADC0)      PC0
// A1 (ADC1)      PC1
// A2 (ADC2)      PC2
// A3 (ADC3)      PC3
// A4 (ADC4/SDAglb)  PC4
// A5 (ADC5/SCLgrn)  PC5

//#if defined(__AVR_ATmega328P__)
#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

//                                m1284p
//    mighty ARDUINO          +----\/----+
//                        RST |1  ISP  36| VCC
//              RX0 (D 8) PD0 |2         | PB7 (D 7) PWM/SCK 
 #define LED_ARDUINO                              7
 #define LED_ARDUINO_PORT                         PORTB
 #define LED_ARDUINO_DDR                          DDRB
 #define LED_ARDUINO_ON       LED_ARDUINO_PORT  |= (1<<LED_ARDUINO)
 #define LED_ARDUINO_OFF      LED_ARDUINO_PORT &= ~(1<<LED_ARDUINO)
 
//              TX0 (D 9) PD1 |3         | PB6 (D 6) PWM/MISO 
//        RX1/INT0 (D 10) PD2 |4         | PB5 (D 5) MOSI 
//        TX1/INT1 (D 11) PD3 |5         | PB4 (D 4) PWM/SS
//             PWM (D 12) PD4 |6         | PB3 (D 3) PWM 
//             PWM (D 13) PD5 |7         | PB2 (D 2) INT2 
//             PWM (D 14) PD6 |8         | PB1 (D 1) 
//             PWM (D 15) PD7 |9         | PB0 (D 0) 
//                            |          |
//                            |          |
//          SCLgrn (D 16) PC0 |10      27| PA0 (AI 0 / D24)
//          SDAglb (D 17) PC1 |11      26| PA1 (AI 1 / D25)
//             TCK (D 18) PC2 |12      25| PA2 (AI 2 / D26)
//             TMS (D 19) PC3 |13      24| PA3 (AI 3 / D27)
//             TDO (D 20) PC4 |14      23| PA4 (AI 4 / D28)
//             TDI (D 21) PC5 |15      22| PA5 (AI 5 / D29)
//                 (D 22) PC6 |16      21| PA6 (AI 6 / D30)
//                 (D 23) PC7 |17      20| PA7 (AI 7 / D31)
//                        GND |18      19| AREF
//                            +----------+

//#elif defined(__AVR_ATmega1284P__)
#else
 #error "board_wahl_falsch.h"
#endif // defined(__AVR_ATmega1284P__)

[Dieter F. (Gast)]

Pack den

  TCCR5A = 0;

mal noch mit rein, dann funktioniert es auch.

[Mike Litoris (Gast)]

Dieter F. schrieb:
> Pack den
>
>   TCCR5A = 0;
>
> mal noch mit rein, dann funktioniert es auch.

Oha. Funktioniert.

Erklärung? Klingt nämlich ziemlich nach Vodoo?


Grüße - ich brauch jetzt erstma n Bier :D

[Joachim B. (jar)]

kann man Kriege nicht durch "bekomme" ersetzen?

Kriege sind so negativ

[Veit D. (devil-elec)]

Hallo,

Timer 0 wird für delay, millis und micros verwendet

Timer 1 wird die Servo Lib verwendet, auf Mega Timer 5

tone() nutzt Timer 2

und dann sind wie schon geschrieben bei Verwendung von analogWrite für
verschiedene Pins verschiedene Timer im Einsatz, aber nur dann eben.

Timer 0 sollte man wirklich in Ruhe lassen. Denn fast jeder gute Code
nutzt millis().

Mir ist noch nicht bekannt das es mit analogWrite und Timer Nutzung
Problem gab.

[Dieter F. (Gast)]

Mike Litoris schrieb:
> Erklärung? Klingt nämlich ziemlich nach Vodoo?

Der Versuch einer Erklärung ...

Aus welchen Gründen auch immer (Arduino-Default ?) ist WGM50 beim Start 
gesetzt. Dadurch kommst du in den Mode5 (fast pwm) mit Top-Wert 0x00FF
(passt zu den ca. 61 Hz bei Prescaler 1024).
Bei fast pwm wird das Output Compare Interrupt Flag scheinbar auch 
gesetzt ...

"In non-inverting Compare Output mode,
the Output Compare (OCnx) is cleared on the compare match between TC NTn 
and OCRnx, and set at BOTTOM.
In inverting Compare Output mode output is set on compare match and 
cleared at BOTTOM. "

Müsste man mal debuggen und sich anschauen ... ich hab es halt probiert, 
weil ich keiner Initialisierung außer meiner eigenen vertraue :-)

Haut mich :-)

[kjel (Gast)]

Joachim B. schrieb:
> kann man Kriege nicht durch "bekomme" ersetzen?
>
> Kriege sind so negativ

Nur wenn man so assoziiert.

[Joachim B. (jar)]

kjel schrieb:
>> Kriege sind so negativ
>
> Nur wenn man so assoziiert.

kommt nicht von mir sondern von meinem Deutschlehrer.

Wer kriegt denn schon? Man bekommt höchstens.

[kjel (Gast)]

Joachim B. schrieb:
> kjel schrieb:
>>> Kriege sind so negativ
>>
>> Nur wenn man so assoziiert.
>
> kommt nicht von mir sondern von meinem Deutschlehrer.
>

Und dann will es keiner gewesen sein.

[Joachim B. (jar)]

kjel schrieb:
> Und dann will es keiner gewesen sein.

stimmt und deswegen sollte man "Kriege" vermeiden wo es geht zumal es 
passenderen Ersatz gibt, hier z.B. "Bekomme"