PMW mit Atmega 32

[Guest (Gast)]

Moin,
ich möchte 2 PWM´s mit dem Atmega 32 Programmieren, deren Frequenz so 
niedrig ist (2Hz), dass man LED´s sichtbar zum blinken bringen kann. Nur 
leider blinkt keine der 3 angeschlossenen LED´s. Die an PD0 glimmt, die 
an PD4 und PD5 sind aus. Weiß jmd weiter?

#include <avr/io.h>
#include <stdlib.h>
#include <avr/interrupt.h>
#include <stdio.h>
#define F_CPU 8000000

int main(void)
{
DDRD = 0xff;
ICR1 = 4000000;    //8MHz/4M = 2Hz
OCR1A =  2000000;  //Duty-Cycle 0,5  
OCR1B = 1000000;  //Duty-Cycle 0,25
TCCR1A = (1<<WGM11)|(1<<COM1A1)|(1<<COM1B1);
TCCR1B = (1<<WGM12)|(1<<WGM13)|(1<<CS10);
TIMSK = (1<<OCIE1A)|(1<<OCIE1B)|(1<<TOIE1);

sei();                      
while(1)
{
}
}
ISR(TIMER1_OVF_vect)
{
  PORTD ^= (1<<PD0);
}
ISR(TIMER1_COMPA_vect)
{
  PORTD ^= (1<<PD5);      
}
ISR(TIMER1_COMPB_vect)          
{
  PORTD ^= (1<<PD4);          
}

[H.Joachim S. (crazyhorse)]

Deine Zahlen mögen rechnerisch richtig sein, passen aber nicht in 16bit 
:-)

[spess53 (Gast)]

Hi

>Deine Zahlen mögen rechnerisch richtig sein, passen aber nicht in 16bit

Richtig. Nimm einen passenden Prescaler.

MfG Spess

[Guest (Gast)]

Habs angepasst. Jetzt geht´s. Vielen Dank

[Sascha (Gast)]

ICR1 ist 16bit groß, da passen keine 4 Millionen rein. Was nachher 
tatsächlich in dem Register steht weiss ich nicht, aber vermutlich nix 
sinnvolles.
2 Hz ist in Software vermutlich sinnvoller zu machen (ich glaube, das 
ist ein Lernprojekt und nichts zeitkritisches, nicht?) aber mit dem 
Timer1 ginge es so:

- PWM Modus 8 (Datenblatt Seite 109)
- ICR1 auf 65536 (=16bit)
- Prescaler auf 1024 (Seite 110)

Macht dann 8Mhz/1024/65536/2 (wegen dual slope)= 0,059Hz. Das müsste 
langsam genug sein.
Kannst auch Modus 14 nehmen, Fast PWM mit ICR1 als TOP, dann entfällt 
das "/2" in der Gleichung.

[spess53 (Gast)]

Hi

>Macht dann 8Mhz/1024/65536/2 (wegen dual slope)= 0,059Hz.

Was hast du an '2 Hz' nicht verstanden?

MfG Spess

[Guest (Gast)]

Zu früh gefreut, so richtig funktioniert es doch noch nicht. Ich habe 
den Code verändert aber es blinkt nur PD4, warum?

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

int main(void)
{
DDRD = 0xff;
ICR1 = 64000;    
OCR1A =  32000;    
OCR1B = 16000;  
TCCR1A = (1<<WGM11)|(1<<COM1A1)|(1<<COM1B1);
TCCR1B = (1<<WGM12)|(1<<WGM13)|(1<<CS10)|(1<<CS11);
TIMSK = (1<<OCIE1A)|(1<<OCIE1B)|(1<<TOIE1);

sei();                      
while(1)
{
}
}
ISR(TIMER1_OVF_vect)
{
  PORTD ^= (1<<PD0);
}
ISR(TIMER1_COMPA_vect)
{
  PORTD ^= (1<<PD5);    
}
ISR(TIMER1_COMPB_vect)          
{
  PORTD ^= (1<<PD4);  
}

[Guest (Gast)]

Niemand eine Idee?

[spess53 (Gast)]

Hi

>Niemand eine Idee?

Hardware überprüfen.

MfG Spess

[asterix (Gast)]

spess53 schrieb:
> Hi
>
>>Niemand eine Idee?
>
> Hardware überprüfen.
>
> MfG Spess

wäre auch mein erster Tipp. Funktioniert überhaupt deine Hardware? Setz 
einfach die Pins mal selbst auf High, dann wirst du ja sehen, ob die LED 
überhaupt leuchten "kann"

[Guest (Gast)]

D.h. die Software ist soweit korrekt? Die Led´s sind nicht defekt, schon 
getestet. Muss ich überhaupt PORTD ^= (1<<PD4) schreiben? Oder toggelt 
PD4 automatisch, da PD4 dem OC1B entspricht und dieser bei Bottom bzw 
Compare match seinen Status automatisch ändert?

[c-hater (Gast)]

Guest schrieb:

> D.h. die Software ist soweit korrekt?

Nein, nicht in dem Sinne, daß sie tun würde, was du haben willst. Aber 
die LEDs sollten immerhin alle drei blinken Und zwar so:

PD0: 1:1 Tastverhältnis, ~0,98Hz
PD4: ~1:3 Tastverhältnis, ~1,95Hz
PD5: ~1:2 Tastverhältnis, ~1,95Hz

Tun sie das nicht, stimmt irgendwas an der Hardware nicht. Oder das 
Programm, was du gepostet hast ist nicht identisch mit dem Programm, was 
tatsächlich auf dem µC läuft.

Übrigens: die beiden COMPx-ISRs sind völlig nutzlos und verbrauchen nur 
Rechenzeit, ohne irgendwas zu bewirken. Abgesehen von dem sinnlosen 
Rechenzeitverbrauch schaden sie allerdings auch nicht weiter.

Ich würde mal darauf tippen, daß dein Hardware-Werk so dermaßen Scheiße 
ist, daß sich der µC in den Reset verabschiedet, sobald irgendwelche 
LEDs plötzlich Strom verbrauchen. Dann wacht er wieder auf, um erneut 
abzustürzen, sobald die erste LED leuchtet.

Also vermutlich hat da mal wieder so ein Ober-Checker kraft seiner 
Wassersuppe auf Stützkondensatoren und/oder auf LED-Vorwiderstände 
verzichtet, obwohl alle Erwachsenen sagen, daß man sowas tatsächlich 
braucht...

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

c-hater schrieb:
> Abgesehen von dem sinnlosen
> Rechenzeitverbrauch schaden sie allerdings auch nicht weiter.

Da bin ich mir nicht so sicher, da der TE genau an den Pins rumfummelt, 
die die Hardware OC Register selbst bearbeiten.
Das macht der Mega von ganz alleine, nur die Pins auf Output stellen und 
die Polarität der Pins mit dem COM1X Bits wählen - die Compare ISRn 
kannst du dir sparen.
Für dich ist 'Clear on Compare Match' der richtige Modus.

[c-hater (Gast)]

Matthias Sch. schrieb:

> Da bin ich mir nicht so sicher, da der TE genau an den Pins rumfummelt,
> die die Hardware OC Register selbst bearbeiten.

Ist aber so. Schau dir einfach das Prinzipschaltbild der Portbits an 
(das steht verdammt nochmal ebenfalls im Datenblatt). Alternative 
Portfunktionen haben bezüglich PORTx-Ausgaben immer absoluten Vorrang, 
bezüglich DDRx meistens (im Falle der Timer-OC-Ausgänge allerdings 
nicht).

Da er nur an den PORTx rumfummelt, ist es praktisch genauso, als wenn er 
garnix tut.

[Thomas E. (thomase)]

Matthias Sch. schrieb:
> Da bin ich mir nicht so sicher, da der TE genau an den Pins rumfummelt,
> die die Hardware OC Register selbst bearbeiten.

Das macht nichts. Die Pins werden von der PWM übernommen. Aber so ein 
Unsinn gehört da natürlich trotzdem nicht hin.

Guest schrieb:
> Muss ich überhaupt PORTD ^= (1<<PD4) schreiben?

Nein, natürlich nicht. Sinn des PWM-Modes ist, dass der Timer das 
alleine macht.

mfg.