ATmega88 - Timer2 OVF mittels Uhrenquarz - Probleme beim Konfigurieren

Hab es jetzt nochmal wie folgt geändert, aber es funktioniert nicht.
Könnte mir vlt. jemand helfen und mir meinen Fehler aufzeigen?!

ASSR   |= (1<<EXCLK);
DDRC   &= ~(1<<PC5);
DDRC   |= (1<<PC4);
PORTC  |= (1<<PC5)|(1<<PC4);
DDRD   &= ~(1<<PD3);
DDRD   |= (1<<PD1);
PORTD  |= (1<<PD3);
EICRA  |= (1<<ISC11);
EIMSK  |= (1<<INT1);
lcd_init();
lcd_clear();
adc_init();
TIMSK2 &=(~(1<<TOIE2)|(1<<OCIE2A)|(1<<OCIE2B));
ASSR   |= (1<<AS2);
TCCR2B |= (1<<CS22)|(1<<CS20);
TCNT2  = 0x00;
TIMSK2 &=(~(1<<TOIE2)|(1<<OCIE2A)|(1<<OCIE2B));
TIMSK2 |= (1<<TOIE2);
sei();

rein aus formalen Gründe (also ohne dass ich ins Datenblatt oder nach 
dem Sinn gefragt habe) kann das hier

1

TIMSK2 &=(~(1<<TOIE2)|(1<<OCIE2A)|(1<<OCIE2B));

nicht richtig sein. Ich vermute du wolltest die ~ vor dir Klammer haben. 
So jeden Falls ist das ein unlogisches Statement. Was soll es bewirken?

1

ASSR   |= (1<<EXCLK);

Was an dem von Krapao geposteten Datenblatt-Zitat war missverständlich?

1

TIMSK2 &=(~(1<<TOIE2)|(1<<OCIE2A)|(1<<OCIE2B));

Wegen falscher Klammerung nahezu unwirksam. Glücklicherweise ja aber eh 
unnötig, oder hast du irgendwann vorher eines der Bits gesetzt?

Wo sind die Punkte d und e aus deinem eigenen Datenblatt-Zitat?

Vielen Dank für eure Antworten.
Klar das invertieren der Interruptenablebits, muss natürlich anders 
aussehen.
Ist mir gestern allein nicht mehr aufgefallen, aber ich bin auch in 
keinsterweise sicher in C Programmierung und erst seit ein paar Wochen 
in Sachen AVRs und Programmierung dabei...sorry.
Der Datenblatttext(siehe post Krapao) zu Bit6(EXCLK) des ASSR bleibt für 
mich dennoch missvertändlich, da zum Einen davon gesprochen wird, dass 
das Bit logisch 1 sein soll im asynchronen Modus um den Uhrquarz 
betreiben zu können, zum Anderen der letzte Satz "Note that the crystal 
Oscillator will only run when this bit is zero." oder ist hiermit der 
Quarz an XTAL gemeint???


ASSR   &= ~(1<<EXCLK) od. ASSR |= (1<<EXCLK);
TIMSK2 &= ~((1<<TOIE2)|(1<<OCIE2A)|(1<<OCIE2B));
ASSR   |= (1<<AS2);
TCCR2B |= (1<<CS22)|(1<<CS20);
TCNT2  = 0x00;
while(ASSR & (1<<TCN2UB));                        //Punkt d
TIMSK2 &= ~((1<<TOIE2)|(1<<OCIE2A)|(1<<OCIE2B));  //Punkt e
TIMSK2 |= (1<<TOIE2);

Guten Tag,

Hier mal ein Code Schnipsel den ich aktuell verwende, der auch 
funktioniert :

1

// Timer2 konfigurieren

2

ASSR   = (1<< AS2);                            // Timer2 asynchron takten

3

long_delay(1000);                               // Einschwingzeit des 32kHz Quarzes

4

TCCR2A = (1<<WGM21) | (1<<WGM20);               // Fast PWM, non inverted      

5

TCCR2A  &= ~(1<<COM2A1) | (1<<COM2A0);

6

TCCR2B = (1<< CS20);                            

7

OCR2A  = 128;                                   // Tastverhältnis 50%

8

while((ASSR & (1<< TCR2BUB)));         // Warte auf das Ende des Zugriffs

9

TIFR2  &= ~(1<<TOV2);                     // Interrupts löschen

10

TIMSK2 |= (1<<TOIE2);                     // Timer overflow Interrupt freischalten   

11

sei();

Ich habe mir über den Vorteiler die Zeit etwas besser einteilbar 
gemacht.

Vielleicht hilft dir das ja schon weiter, habe mich auch mal mit dem 
Thema beschäftigt, aber weis leider nicht mehr genau wo das Problem lag.


Gruß,
Martin

Danke für eure schnelle Unterstützung, werde ich heute noch ausprobieren 
und mal sehen, ob es dann funktioniert.
Anderfalls "nerv" ich nochmal ... ;)

> while(ASSR & (1<<TCN2UB));                        //Punkt d

Ich würde mich nicht darauf verlassen, dass es reicht, sich eines der 
Bits rauszupicken. Ich würde tatsächlich auf alle Bits warten, deren 
zugehörige Register du beschrieben hast (zumal man das ja auch in nur 
einer Schleife machen kann).

> TIMSK2 &= ~((1<<TOIE2)|(1<<OCIE2A)|(1<<OCIE2B));  //Punkt e

Nein, das ist nicht Punkt e. Der sagt "Flags löschen", nicht 
"Enable-Bits löschen".

Und so nebenbei: der Code von Martin macht das auch nicht korrekt.

Hallo Stefan,

auf dein Statement hin muss ich dann doch nochmal fragen.
Hab es jetzt wie folgt umgeschrieben:

ASSR   &= ~(1<<EXCLK);
TIMSK2 &= ~(1<<TOIE2);
ASSR   |= (1<<AS2);
for(w=0,w<=ww,w++)
   {
    _delay_ms(10);
   }
TCCR2B |= (1<<CS22)|(1<<CS20);        // Prescaler 128
TCNT2  = 0x00;
while(ASSR & (1<<TCN2UB));
TIFR2  &= ~(1<<TOV2);
TIMSK2 |= (1<<TOIE2);

1. zu deiner Aussage bezüglich der Wartefunktion:
   Ich laden doch nur TCNT2 mit einem Anfangswert und lasse die andere
   Register des Zählers doch unangetatet bzw. möchte ich den Timer 2
   auch nur zu diesem Zweck und keinem Compare und dergleichem mehr 
nutzen.
   Stellst du dir das dann wie folgt vor?
   while(ASSR &((1<<TCN2UB)|(1<<OCR2AUB)|(OCR2BUB)|
  (1<<TCR2AUB)|(1<<TCR2BUB));

2. Okay der Timer hat mehrere Interrupts, sicher muss man dann alle
   entsprechenden Flags löschen.
   TIFR2&=~((1<<OCF2B)|(1<<OCF2A)|(1<<TOV2))

Michel Schl. schrieb:
> 1. zu deiner Aussage bezüglich der Wartefunktion:
>    Ich laden doch nur TCNT2 mit einem Anfangswert und lasse die andere
>    Register des Zählers doch unangetatet
Und was ist mit TCCR2B?

Michel Schl. schrieb:
> Stellst du dir das dann wie folgt vor?
>    while(ASSR &((1<<TCN2UB)|(1<<OCR2AUB)|(OCR2BUB)|
>   (1<<TCR2AUB)|(1<<TCR2BUB));
Ja, so in etwa.

1

while (ASSR & ((1<<TCN2UB) | (1<<TCR2BUB));

Michel Schl. schrieb:
> 2. Okay der Timer hat mehrere Interrupts, sicher muss man dann alle
>    entsprechenden Flags löschen.
>    TIFR2&=~((1<<OCF2B)|(1<<OCF2A)|(1<<TOV2))
Das löscht aber keines der Flags.

1

TIFR2 = (1<<OCF2B) | (1<<OCF2A) | (1<<TOV2);

Oh ja richtig, dass die Flags mit einer 1 gelöscht werden, habe ich 
mehrfach gelesen. peinlich
Vielen Dank für eure schnelle und hilfreiche Unterstützung.