ATMEGA169 Timer1 Overflow Interrupt richtig deklarieren

[Martin A. (airmartl)]

Sorry für die banale Frage, aber ist dass so richtig wenn ich Timer1 mit 
Prescaler 8 für den OVERFLOW Interrupt deklarieren möchte?

int main(void)
{
TCNT1 = 0;    // set timer1 to 0
TCCR1B |= (1<<CS11);  // select prescaler of 8 and start the timer
TIFR1 = 0xFF;    // clear interrupt-flags Timer1
TIMSK1|= (1<<TOIE1);   // enable Timer1 overflow interrupt
while(1)                    
    {
    //Programm...
    }
    return (1);

}
ISR (TIMER1_OVF_vect)  
{
i++; //mach etwas
}

[Karl H. (kbuchegg)]

Wenn du schon so fragst, dann existiert ein Problem, für das du keine 
Lösung hast.

Ich lehne mich mal aus dem Fenster.
Hast du i volatile gemacht?

[Holger L. (max5v)]

http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial#Programmieren_mit_Interrupts

Hast du denn die #include <avr/interrupt.h> eingebunden und die 
Interrupts mit sei(); freigegeben ?

[Martin A. (airmartl)]

Hallo Holger, ok hier noch etwas ausführlicher, passt das so (siehe 
unten)?

Hallo Karl, heißt das wenn ich eine zuweisung in der ISR mache, müssen 
alle beteiligten Variablen Volatile sein? Wie ist wenn ich nur in der 
ISR z.B. PortB = 0x01; schreibe? Funktioniert das?

Vielen Dank.

volatile long i;
#include <avr/interrupt.h>

int main(void)
{
TCNT1 = 0;    // set timer1 to 0
TCCR1B |= (1<<CS11);  // select prescaler of 8 and start the timer
TIFR1 = 0xFF;    // clear interrupt-flags Timer1
TIMSK1|= (1<<TOIE1);   // enable Timer1 overflow interrupt
sei();
while(1)                    
    {
    //Programm...
    }
    return (1);

}
ISR (TIMER1_OVF_vect)  
{
i++; //mach etwas
}

[Holger L. (max5v)]

Martin Airmartl schrieb:
> volatile long i;

"long" schluckt natürlich einiges an Speicher, ein anderer Datentyp wäre 
vermutlich besser.
Da der Timer an und für sich nur hochzählt ( nicht kleiner als Null 
wird) wäre außerdem ein unsigned geeignet.

Es scheint alles soweit zu passen, spricht denn irgendwas dagegen es 
auszuprobieren ?

[Martin A. (airmartl)]

Holger L. schrieb:
> "long" schluckt natürlich einiges an Speicher, ein anderer Datentyp wäre
> vermutlich besser.
> Da der Timer an und für sich nur hochzählt ( nicht kleiner als Null
> wird) wäre außerdem ein unsigned geeignet.

Hallo Holger, das einzige was dageben spricht es auszuprobieren ist, 
dass ich erst um 17Uhr Feierabend habe :)

Mit unsigned meinst du einen unsigned integer oder? Der hat dann 32bit 
oder?

Hier die neue daklaration:
volatile unsigned int i;

[Holger L. (max5v)]

Ja das war so gemeint.
Außer dem int gibt es noch den uint16_t ( 0 bis 65.535) und den uint8_t 
( 0 bis 255).
Es kommt ganz darauf an was du mit dem Timer anstellen möchtest.

Martin Airmartl schrieb:
> das einzige was dageben spricht es auszuprobieren ist,
> dass ich erst um 17Uhr Feierabend habe :)

Dafür hat der liebe Gott die Smd -Technik erfunden, solche Boards kann 
man unbemerkt in der Hosentasche mit auf die Arbeit nehmen ;)

[Martin A. (airmartl)]

Holger L. schrieb:
> Dafür hat der liebe Gott die Smd -Technik erfunden, solche Boards kann
> man unbemerkt in der Hosentasche mit auf die Arbeit nehmen ;)

Achso, ja wenn das der Wille vom lieben Gott wahr, dann werd ich sie 
morgen gleich mal mitnehmen :)

Da ich den 16bit Timer voll ausnutzen will benötige ich auch eine 16bit 
Variable. "uint16_t" ist nur eine andere Schreibweise oder

"uint16_t" ist das gleiche wie "unsigned int" oder?

Gruß

[Holger L. (max5v)]

Martin Airmartl schrieb:
> Holger L. schrieb:
>> Dafür hat der liebe Gott die Smd -Technik erfunden, solche Boards kann
>> man unbemerkt in der Hosentasche mit auf die Arbeit nehmen ;)
>
> Achso, ja wenn das der Wille vom lieben Gott wahr, dann werd ich sie
> morgen gleich mal mitnehmen :)

Aber laß dich nicht erwischen, manchmal schaut der Typ von ganz oben zu 
:)

Martin Airmartl schrieb:
> Da ich den 16bit Timer voll ausnutzen will benötige ich auch eine 16bit
> Variable.
Nicht zwingend.
Wenn du Sekunden, Minuten und Stunden zählen möchtest genügen 8 Bit 
Variablen.

//Pseudocode
volatile uint8_t sekunde++;
IF (sekunde == 60)
   volatile uint8_t minute++;
   IF (minute == 60) 
     volatile uint8_t Stunde++;

Die Zeitbasis für die Sekunde müßtest du natürlich aus der 
Timerinterruptzeit errechnen.

Martin Airmartl schrieb:
> "uint16_t" ist nur eine andere Schreibweise oder
> "uint16_t" ist das gleiche wie "unsigned int" oder?

Also uint16_t sollte ein 16 Bit großer unsigned Integer sein.
Aber um darüber etwas zu erzählen bin ich zu wenig Profi.

[chris (Gast)]

bei Variablen größer als 8bit musst du beachten, dass auf diese im 
Hauptprogramm atomar zugegriffen werden muss, indem du den Interrupt 
sperrst vor dem Lesen / Schreiben. Ansonsten gibts irgendwann mal 
merkwürdige Fehler ;)