ATtiny84: Timer1-Interrupt geht nur 1x

[JR (Gast)]

Hallo zusammen.

Ich bin mit meinem AVR-Latein am Ende:

Folgendes Mini-Programm läuft nicht, bzw. die ISR wird nur 1x 
durchlaufen.
Dabei soll die LED im Sekundentakt getoggelt werden.
Was mache ich falsch?

Liegt es daran, dass bei Nutzung des Registers TCCR1B auch in TCCR1A 
noch etwas gesetzt werden muss?

/*
LED blinken mit timer-interrupt auf attiny84 8Mhz
*/

#include <avr/io.h>
#include <avr/interrupt.h>



ISR( TIM1_COMPB_vect )             
{
   PORTB ^= (1 << 0);    // toggle the LED at PORTB0
}
//-------------------------------------------------------
void timer_init( void )            // Timer 1 init
{
  TCCR1B = (1<<WGM13) | (1<<WGM12) | (1<<CS12) | (0<<CS11) | (1<<CS10);  // CTC ; XTAL / 1024
  OCR1BH = 0x1E;    // set Output compare register to 7812 (== 1 sec)
  OCR1BL = 0x84;
  TIMSK1 = (1<<OCIE1B);  // enable Output Compare B Match Interrupt
}
//-------------------------------------------------------
int main( void )
{
 DDRB = 0x0F;           // make the PORTB pins to outputs
 
 timer_init();
 sei();
 while(1){  //
 }
}

Gruß
JR

[Micha H. (mlh)]

JR schrieb:

>    PORTB ^= (1 << 0);    // toggle the LED at PORTB0

Das geht nicht.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Du kannst aber schreiben

PINB |= (1<<PB0) ;

Dann sollte der Pin toggeln.

[JR (Gast)]

Micha H. schrieb:
>>    PORTB ^= (1 << 0);    // toggle the LED at PORTB0
>
> Das geht nicht.

Ich habe es jetzt probiert mit:
PORTB ^= (1 << PB0);

geht auch nicht :-(

[Thomas E. (thomase)]

JR schrieb:
> Liegt es daran, dass bei Nutzung des Registers TCCR1B auch in TCCR1A
> noch etwas gesetzt werden muss?
Ja.
WGM10 und WGM11.

ISR( TIM1_COMPA_vect )
{
   PORTB ^= (1 << 0);    // toggle the LED at PORTB0
}

void timer_init( void )            // Timer 1 init
{
  TCCR1A = (1 << WGM10) < (1 << WGM11) 
  TCCR1B = (1<<CS12) | (1<<CS10); 
  OCR1A = 7812;
  TIMSK1 = (1<<OCIE1A);  // enable Output Compare B Match Interrupt
}

Micha H. schrieb:
> JR schrieb:
>
>>    PORTB ^= (1 << 0);    // toggle the LED at PORTB0
>
> Das geht nicht.
Interessant. Erzählst du uns auch warum nicht?

mfg.

[JR (Gast)]

Matthias Sch. schrieb:
> PINB |= (1<<PB0) ;

...auch das geht nicht 8((

liegt die Ursache evtl. doch in den Register-Bits?

[Micha H. (mlh)]

Micha H. schrieb:

>>    PORTB ^= (1 << 0);    // toggle the LED at PORTB0
>
> Das geht nicht.

Disregard. Das geht schon, ich habe da nur mit einem Auge hingeschaut, 
nämlich mit dem falschen.

[Thomas E. (thomase)]

Mist! Vertan!

Vergiss das:
> void timer_init( void )            // Timer 1 init
> {
>   TCCR1A = (1 << WGM10) < (1 << WGM11)
>   TCCR1B = (1<<CS12) | (1<<CS10);
>   OCR1A = 7812;
>   TIMSK1 = (1<<OCIE1A);  // enable Output Compare B Match Interrupt
> }
> [/c]
>

So:

void timer_init( void )            // Timer 1 init
{
   TCCR1B = (1<<CS12) | (1<<CS10) | (1 << WGM12);
   OCR1A = 7812;
   TIMSK1 = (1<<OCIE1A);  // enable Output Compare B Match Interrupt
 }

mfg.

[Achim K. (aks)]

Hast Du den Reset (PB3) ausgeschaltet (fuse)?
Wenn nicht, dann macht DDRB = 0x0F einen Reset.

[JR (Gast)]

Thomas Eckmann schrieb:

> void timer_init( void )            // Timer 1 init
> {
>    TCCR1B = (1<<CS12) | (1<<CS10) | (1 << WGM12);
>    OCR1A = 7812;
>    TIMSK1 = (1<<OCIE1A);  // enable Output Compare B Match Interrupt
>  }

Danke!

Jetzt blinkt sie...

Ich werde wohl noch mal das datenblatt lesen müssen.

Gruß
JR

[Thomas E. (thomase)]

Achim K. schrieb:
> Hast Du den Reset (PB3) ausgeschaltet (fuse)?
> Wenn nicht, dann macht DDRB = 0x0F einen Reset.
Ja ne, is klar.

mfg.

[JR (Gast)]

Achim K. schrieb:
> Hast Du den Reset (PB3) ausgeschaltet (fuse)?
> Wenn nicht, dann macht DDRB = 0x0F einen Reset.

Den RESET-PIN (PB3) hab ich nicht weggefused.
Die LED brennt aber auch nicht.
Der ATtiny macht auch keinen RESET.

Ich sollte wohl lieber nur PB0...PB2 zum Ausgang machen...
Danke!

JR

[Thomas E. (thomase)]

JR schrieb:
> Achim K. schrieb:
>> Hast Du den Reset (PB3) ausgeschaltet (fuse)?
>> Wenn nicht, dann macht DDRB = 0x0F einen Reset.
>
> Den RESET-PIN (PB3) hab ich nicht weggefused.
> Die LED brennt aber auch nicht.
> Der ATtiny macht auch keinen RESET.
>
> Ich sollte wohl lieber nur PB0...PB2 zum Ausgang machen...
> Danke!
>
> JR

Hör da nicht drauf. Das ist dummes Zeug. Vor allen Dingen lass die 
Finger von der Resetfuse!

mfg.

[Achim K. (aks)]

Da PORTB nach einem Reset 0 ist, schaltet DDRB = 0x0F PB3 auf 0 und löst 
einen Reset aus, außer der Reset ist per Fuse (Reset Fuse oder DWEN) 
abgeschaltet.

[Thomas E. (thomase)]

Achim K. schrieb:
> Da PORTB nach einem Reset 0 ist, schaltet DDRB = 0x0F PB3 auf 0 und löst
> einen Reset aus, außer der Reset ist per Fuse (Reset Fuse oder DWEN)
> abgeschaltet.
Ich schrieb ja schon "is klar".

Wenn der Pin Reset ist, kannst du intern mit dem machen, was du willst. 
Da passiert gar nichts.

mfg.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Achim K. schrieb:
> Da PORTB nach einem Reset 0 ist, schaltet DDRB = 0x0F PB3 auf 0 und löst
> einen Reset aus, außer der Reset ist per Fuse (Reset Fuse oder DWEN)
> abgeschaltet.

Nö. Ich zitiere mal aus dem Datenblatt, Seite 65:
> Pullup is activated and output driver and digital input are deactivated
> when the pin is used as the RESET pin.

Ausgangstreiber und Eingang ist also gesperrt, wenn der Pin seine 
Funktion als RESET wahrnimmt. Du kannst also z.B. auch nicht den Zustand 
des RESET Pins als PINB3 abfragen.

[Achim K. (aks)]

Sch... OK ihr habt recht.