AVR ATmega168 Timer2 PWM Verständniss

[Christian M. (blacky10)]

Hallo zusammen,
Ich versuche schon länger auf einem ATmega168 beim Timer2 Pin OC2A eine 
Fast PWM zu entlocken, allerdings erfolglos. Beim Timer1 hatte ich keine 
Probleme, doch bei  Timer2 habe ich keinen Erfolg. Kann mir einer helfen 
wo ich auf dem Schlauch stehe?

#include <avr/io.h>

int main(void)
{
  
  /* PB3 as output */
  DDRB = (1<<PB3);
  
  /*Asynchronous status register*/
  ASSR = (0<<EXCLK) |    //Enable External clock input
      (0<<AS2);    //Enable Asynchronous Timer 2
  
  /* Timer Counter Control Register A */
  TCCR2B = (0<<FOC2A) |  //Force Output compare A
       (0<<FOC2B) |  //Force Output compare B
       (1<<CS21) |  //Clock select, 32
       (1<<CS20);    //Clock select
       
  /* Timer Counter Control Register B */
  TCCR2A = (1<<COM2A1) |  //Connect OC2A
       (0<<COM2A0) |  //Connect OC2A
       (0<<COM2B1) |  //Connect OC2B
       (0<<COM2B0) |  //Connect OC2B
       (1<<WGM21) |  //Waveform generation mode, Fast PWM
       (1<<WGM20);  //Waveform generation mode
  
  /* Output Compare Register A*/
  OCR2A = 127; //50% cycle
  
  /* Timer Counter Interupt Mask Register */
  TIMSK2 = (0<<OCIE2B) |  //Timer 2 output compare match B interrupt enable
      (0<<OCIE2A) |  //Timer 2 output compare match A interrupt enable
      (0<<TOIE2);    //Timer 2 overflow interrupt enable



    while(1)
    {
    
    }
}

Vielen Dank für eure Hilfe :D
Gruss Christian

[Thomas E. (thomase)]

Christian Müller schrieb:

>   /*Asynchronous status register*/
>   ASSR = (0<<EXCLK) |    //Enable External clock input
>       (0<<AS2);    //Enable Asynchronous Timer 2
Lass das mal ersatzlos weg. Du hast ihm gerade den Clock weggenommen.

mfg.

[holger (Gast)]

>>   ASSR = (0<<EXCLK) |    //Enable External clock input
>>       (0<<AS2);    //Enable Asynchronous Timer 2
>Lass das mal ersatzlos weg. Du hast ihm gerade den Clock weggenommen.

ASSR mit 0 zu beschreiben dürfte das nicht machen.

Setz mal WGM22 in TCCR2B auf 1.

[Christian M. (blacky10)]

Hm das hab ich gemacht, hat leider nix gebracht. Allerdings sollte dies 
meiner Meinung nach sowieso keinen Einfluss gehabt haben, da ich die 
interne Clock benütze. Aber danke für die Idee, hast du weitere?

Ahja was ev noch weiterhilft: Ich habe das Programm im Simulator (AVR 
Studio 5) ausprobiert. Dort Toggelt PORTB.3
Ebenfalls hab ich die Hardware über folgenden Befehl überprüft:

DDRB = (1<<PB3);
PORTB = (1<<PB3);

Der Ausgang geht wie Erwartet auf High. Ein Defekt kann also 
ausgeschlossen werden.

[Christian M. (blacky10)]

Ok ich hab jetzt auch den Tipp von Holger ausprobiert und den folgenden 
Teil geändert:

TCCR2B = (1<<COM2A1) |  //Connect OC2A
       (0<<COM2A0) |  //Connect OC2A
       (0<<COM2B1) |  //Connect OC2B
       (0<<COM2B0) |  //Connect OC2B
       (1<<WGM22)  |  
       (1<<WGM21) |  //Waveform generation mode, Fast PWM
       (1<<WGM20);  //Waveform generation mode

Ich nehme an, dass es sich bei WGM22 um TCCR2B.3 handelt (im Datasheet 
liegt wohl ein Fehler vor, da es das Bit WGM22 TCCR2B nur als Read 
beschreibt).

Jetzt ist der Pin immer HIGH.

[holger (Gast)]

Wieso hast du die beiden jetzt auch in TCCR2B gesetzt?

       (1<<WGM21) |  //Waveform generation mode, Fast PWM
       (1<<WGM20);  //Waveform generation mode

Die gibt es dort gar nicht.

[Christian M. (blacky10)]

Hm da ist mir wohl beim Posten was durcheinander gekommen sry, danke für 
den Hinweis. Um Missverständnisse ausschliessen zu können, hier der 
aktuelle Code:

#include <avr/io.h>

int main(void)
{
  
  /* PB3 as output */
  DDRB = (1<<PB3);
  
//   /*Asynchronous status register*/
//   ASSR = (0<<EXCLK) |    //Enable External clock input
//       (0<<AS2);    //Enable Asynchronous Timer 2
//   
  /* Timer Counter Control Register A */
  TCCR2B = (0<<FOC2A) |  //Force Output compare A
       (0<<FOC2B) |  //Force Output compare B
       (1<<WGM22)  |  
       (1<<CS21) |  //Clock select, 32
       (1<<CS20);    //Clock select
       
  /* Timer Counter Control Register B */
  TCCR2A = (1<<COM2A1) |  //Connect OC2A
       (0<<COM2A0) |  //Connect OC2A
       (0<<COM2B1) |  //Connect OC2B
       (0<<COM2B0) |  //Connect OC2B
       (1<<WGM21) |  //Waveform generation mode, Fast PWM
       (1<<WGM20);  //Waveform generation mode
  
  /* Output Compare Register A*/
  OCR2A = 127; //50% cycle
  
  /* Timer Counter Interupt Mask Register */
  TIMSK2 = (0<<OCIE2B) |  //Timer 2 output compare match B interrupt enable
      (0<<OCIE2A) |  //Timer 2 output compare match A interrupt enable
      (0<<TOIE2);    //Timer 2 overflow interrupt enable



    while(1)
    {
    
    }
}

[holger (Gast)]

>Setz mal WGM22 in TCCR2B auf 1.

Nee, nimm das wieder weg. TOP muss 0xFF sein.
WGM21 und WGM20 auf 1 ist ok.

Dummerweise sehe ich dann aber keinen Fehler mehr in deinem Code.
Müsste gehen.

[Thomas E. (thomase)]

holger schrieb:
>>>   ASSR = (0<<EXCLK) |    //Enable External clock input
>>>       (0<<AS2);    //Enable Asynchronous Timer 2
>>Lass das mal ersatzlos weg. Du hast ihm gerade den Clock weggenommen.
>
> ASSR mit 0 zu beschreiben dürfte das nicht machen.
>
> Setz mal WGM22 in TCCR2B auf 1.

Ja habe ich auch gerade gesehen. Wieder dieses dämliche 0 schieb 
sonstwohin. Erleichtert die Fehlersuche ungemein.

#include <avr/io.h>

int main(void)
{  
  DDRB = (1<<PB3);
  TCCR2B = (1<<CS21)|(1<<CS20);    //Clock select       
  TCCR2A = (1<<COM2A1) | (1<<WGM21) | (1<<WGM20);  //Waveform generation mode
  
  OCR2A = 127; //50% cycle

    while(1)
    {
    
    }
}

> (1<<WGM22)  |

WGM22 war gesetzt. Damit ist TOP OCR2A und Duty ist OCR2B. Nur mit WGM20 
und 21 ist das Mode3 mit TOP = 0xFF und Duty OCR2A. Mit OCR2B und COM2B1 
kannst du den 2.PWM-Kanal schalten.

mfg.

[Christian M. (blacky10)]

Boa vielen Dank, es funktioniert :D
Ich Code eigentlich auch nicht mit (0<<x) jedoch, habe ich es zu 
debuggingzecken so geschrieben. Allerdings sehe ich den Unterschied zu 
meinem allerersten Post nicht (ausser dass der Code natürlich bedeutend 
schlanker ist ;D)

Nochmals vielen Dank, hier sind super hilfsbereite Leute.

[holger (Gast)]

>Boa vielen Dank, es funktioniert :D
>Ich Code eigentlich auch nicht mit (0<<x) jedoch, habe ich es zu
>debuggingzecken so geschrieben. Allerdings sehe ich den Unterschied zu
>meinem allerersten Post nicht (ausser dass der Code natürlich bedeutend
>schlanker ist ;D)

Zeig deinen neuen Code doch mal.

[Christian M. (blacky10)]

Ich verwende jetzt 1:1 den Code von Thomas:

#include <avr/io.h>

int main(void)
{  
  DDRB = (1<<PB3);
  TCCR2B = (1<<CS21)|(1<<CS20);    //Clock select       
  TCCR2A = (1<<COM2A1) | (1<<WGM21) | (1<<WGM20);  //Waveform generation mode
  
  OCR2A = 127; //50% cycle

    while(1)
    {
    
    }
}

[holger (Gast)]

>Ich verwende jetzt 1:1 den Code von Thomas:

Ist identisch zu deinem zuerst geposteten Code.
Da sind nur die funktionslosen Nullschieber rausgekürzt.

Dein Code hätte auch so funktionieren müssen.

[Thomas E. (thomase)]

holger schrieb:
>>Ich verwende jetzt 1:1 den Code von Thomas:
>
> Ist identisch zu deinem zuerst geposteten Code.
> Da sind nur die funktionslosen Nullschieber rausgekürzt.
>
> Dein Code hätte auch so funktionieren müssen.

Ich sehe da auch keinen Unterschied. Dem Compiler ist es auch Wurscht, 
wie das aussieht. Hauptsache er bekommt seine Klammern und Semikolons.

@Christian
Schreib nicht so viele Kommentare. Einmal oben drüber, was der Timer 
machen soll und gut ist. Daß die WGM-Bits für den Modus, die CS-Bits für 
den Prescaler und die COM-Bits für die Ausgänge sind, weiss man, wenn 
man das ein paar Mal gemacht hat, auch so. Und sonst gehört das 
Datenblatt sowieso auf den Tisch oder in 2 Registerkarten in den 
Browser. Eine fürs Pinout, die andere zum Blättern.

mfg.