Atmega8 Timer2 CTC Problem

[Asterix (Gast)]

Hallo,

ich möchte die Pins eines Atmega8 mithilfe des Timer2 togglen; PB0 und 
PB1 sollen dabei gegensätzliche Zustände annehmen; sprich um 180° 
verschoben sein.
Mit folgendem Code klappt das leider nicht, und ich finde auf Teufel 
komm' raus den Fehler nicht.
Der uC läuft bei 1Mhz. Frequenz der Pins sollte bei den gegebenen 
Einstellungen knapp 2kHz sein, aber ich messe 0V an beiden Pins.

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


int main(void)
{
  DDRB |= (1<<DDB1) | (1<<DDB0) | (1<<PB2);
  PORTB |= (1<<PB2);   //um zu testen, ob Port C überhaupt "lebt"
  TCCR2 |= (1<<WGM21);  //CTC mode
  OCR2 = 250;    //top value
  TIMSK |= (1<<OCIE2);    //erlaube Overflow-Interrupts
  sei();                  //erlaube globale Interrupts
    while (1) 
    {
    }
}

ISR(TIMER2_COMP_vect)
{
  PORTB ^= (1<<PB0);      //toggle PB0 bei jedem compare match
  if(PORTB & (1<<PB0))    //setze PB1 gegensätzlich zu PB2
  {
    PORTB &= !(1<<PB1);
  }
  else
  {
      PORTB |= (1<<PB1);
  }
}

Ich stehe total auf dem Schlauch. Weiß jemand weiter und kann mir 
helfen?

[Tom (Gast)]

Die Abfrage im if muß mit PINB erfolgen

if (PINB & (1<<PB0))

[Asterix (Gast)]

Danke Tom,

das stimmt natürlich! Wird gleich geändert... In dem Fall sollte aber 
Pin B0 trotzdem funktionieren. Gerade habe ich gesehen, dass ich Depp 
keine "Clock Source" gesetzt habe. Weil ich keinen Prescaler brauche, 
dachte ich, das sei obsolet, sehe jetzt aber, dass dann der Timer gar 
nicht läuft. Beides wird nachgebessert.

[Asterix (Gast)]

...und das war es. Sry, dass ich hier die Pferde wild gemach habe, für 
so einen Kleinsch***, aber ich habe echt lange daran gesessen.

[Tom (Gast)]

Ist mir auch schon passiert, daher ists mir gleich aufgefallen. Viel 
Erfolg noch.

[Lötlackl *. (pappnase)]

Ich habe starke Zweifel, daß dieser Ausdruck das tut, was es tun soll.

PORTB &= !(1<<PB1);

Meiner bescheidenen Meinung nach sollte es eher heißen.

PORTB &= ~(1<<PB1);

AVR-Studio Disassembling dazu:

PORTB &= !(1<< PB1);
IN        R20,0x02       In from I/O location    // keine Ahnung, was der Quatsch hier bewirken soll, das wissen nur die Compiler-Bauer
OUT       0x02,R17       Out to I/O location     // hier wird es interessant

PORTB &= ~(1 << PB1);}
CBI       0x02,1         Clear bit in I/O register

Feuer frei für die verblödeten Forentrolle, die schon ungeduldig in 
ihren Startlöchern lauern.