Hardware PWM auf ATmega8

[Eric W. (ceikolon)]

Hallo,
ich bin in dem Gebiet der µCs noch Neueinsteiger und versuche mich zZ an 
einer Hardware PWM. Verwendet wird ein Atmega8 mit 8MHz.
Als Vorlage habe ich diese Seite genommen: 
http://extremeelectronics.co.in/avr-tutorials/pwm-signal-generation-by-using-avr-timers-part-ii/

Ich hab den Code soweit auf mein Atmega8 angepasst

#include <avr/io.h>
#include <util/delay.h>

void InitPWM()
{
   /*
   TCCR0 - Timer Counter Control Register (TIMER0)
   -----------------------------------------------
   BITS DESCRIPTION
   

   NO:   NAME   DESCRIPTION
   --------------------------
   BIT 7 : FOC0   Force Output Compare [Not used in this example]
   BIT 6 : WGM00  Wave form generartion mode [SET to 1]
   BIT 5 : COM01  Compare Output Mode        [SET to 1]
   BIT 4 : COM00  Compare Output Mode        [SET to 0]

   BIT 3 : WGM01  Wave form generation mode [SET to 1]
   BIT 2 : CS02   Clock Select               [SET to 0]
   BIT 1 : CS01   Clock Select               [SET to 0]
   BIT 0 : CS00   Clock Select               [SET to 1]

   The above settings are for
   --------------------------

   Timer Clock = CPU Clock (No Prescalling)
   Mode        = Fast PWM
   PWM Output  = Non Inverted

   */

   //Set OC2 PIN as output. It is  PB3 on ATmega16 ATmega32/Atmega8; PortD6 Kontrolle
   DDRD  =  0b01000000;
   DDRB  =  0b00001000;
   //TCCR0|=(1<<WGM00)|(1<<WGM01)|(1<<COM01)|(1<<CS00);
  TCCR2=01101001;

}

/******************************************************************
Sets the duty cycle of output. 

Arguments
---------
duty: Between 0 - 255

0= 0%

255= 100%

The Function sets the duty cycle of pwm output generated on OC0 PIN
The average voltage on this output pin will be

         duty
 Vout=  ------ x 5v
         255 

This can be used to control the brightness of LED or Speed of Motor.
*********************************************************************/

void SetPWMOutput(uint8_t duty)
{
   OCR2=duty;
}

/******************************************************************** 

Simple Wait Loop

*********************************************************************/

void Wait()
{
 _delay_loop_2(62000);
}

void main()
{
   uint8_t brightness=0;

   //Initialize PWM Channel 0
   InitPWM();

   //Do this forever

   while(1)
   {
      //Now Loop with increasing brightness

      for(brightness=0;brightness<255;brightness++)
      {
         //Now Set The Brighness using PWM

         SetPWMOutput(brightness);
     
     //Kontrolllampe
    PORTD^=0b01000000;
         //Now Wait For Some Time
         Wait();
      }

      //Now Loop with decreasing brightness

      for(brightness=255;brightness>0;brightness--)
      {
         //Now Set The Brighness using PWM

         SetPWMOutput(brightness);

         //Now Wait For Some Time
         Wait();
      }
   }
}

Leider kann ich am PortB3 kein oszillieren messen. Vielleicht könnt ihr 
ja einen Fehler finden...
Achja am PortD6 hängt ne LED, die mir bestätigt das beide For 
Anweisungen durchlaufen werden...

Bin für jede Antwort Dankbar!

[Karl H. (kbuchegg)]

Eric Weiß schrieb:

> Leider kann ich am PortB3 kein oszillieren messen. Vielleicht könnt ihr
> ja einen Fehler finden...

Dein Hauptfehler besteht darin, dass du auf bescheuerter Syntax 
bestehst.

Das hier

  TCCR2=01101001;

tut nicht das, was du denkst das es tun sollte.
01101001
ist keine Binärzahl sondern eine Oktalzahl.

Und darf man erfahren, warum du diese Schreibweise, der einfach zu 
lesenderen, einfacher zu analysierenden und zumindest in dieser Hinsicht 
WESENTLICH weniger fehleranfälligen Schreibweise

   TCCR2 |= (1<<WGM20) | (1<<WGM21) |    // Fast PWM
            (1<<COM21) |                 // Clear when Up, Set when down
            (1<<CS20);                   // Prescaler 1

vorziehst?
Da sieht man auf einen Blick, welche Bits gesetzt werden und kann mit 
den Bitnamen im Datenblatt vergleichen. Schreibst du das binär, muss man 
erst mal die Bits auseinander dröseln, ehe man feststellen kann, was du 
da überhaupt alles einschaltest.
Benutze die Bitnamen aus dem Datenblatt! Dazu wurden die Header Files 
entsprechend gepimpt, damit du das machen kannst!


Hier
   DDRB  =  0b00001000;
ist es zwar prinzipiell richtig, aber von der Schreibweise genau das 
gleiche:
Wenn du Bit 3 (PB3) auf 1 setzen willst, dann schreib das auch so:
   DDRB |= ( 1 << PB3 );

Jetzt kann der Blindenhund eines Blinden auf 15 Meter Entfernung 
erkennen, dass das Bit für PB3 auf 1 gesetzt wird. Kein Mensch muss dazu 
irgendwelche Bits zählen (und sich 3 mal verzählen) oder irgendwelche 
Prefixe für Binärzahlen benutzen (und manchmal darauf vergessen).

[Eric W. (ceikolon)]

Vielen Dank für deine sehr belustigende Antwort ;-).
Grund warum ich es so schreibe steht vermutlich gleich in der ersten 
Zeile meines Beitrages oben.
Ich habe auch erst seit heute deine Schreibweise kennengelernt, werde 
mich aber nun darrauf umstellen (versprochen!).

Die PWM läuft nun auch, Vielen Dank!

[Karl H. (kbuchegg)]

Eric Weiß schrieb:

> Ich habe auch erst seit heute deine Schreibweise kennengelernt,

Tja.
Zitat

   DDRB  =  0b00001000;
   //TCCR0|=(1<<WGM00)|(1<<WGM01)|(1<<COM01)|(1<<CS00);
  TCCR2=01101001;

Man beachte die auskommentierte Zeile