atmega328 timer konfiguration

[Andreas H. (Gast)]

Hallo,

ich bin mal wieder am Verzwifeln mit den Timern vom avr.
Der Controller ist ein atmega328 und ich will mit Timer 1(16bit) ein ppm 
signal mit 50Hz Wiederholfrequenz ausgeben.

Soweit mein code. Ich hoff er ist ausreichend kommentiert

#include <avr/io.h>          

#ifndef F_CPU
#define F_CPU 16000000UL
#endif

uint16_t ppm1, ppm2;

int main (void) {            

  DDRB = 0xFF;
  
    PRR &= ~(1<<PRTIM1); //enable timer/cunter 1
  
     // PB1 and PB2 is now an output

    ICR1 = 0x9C40;
    // set TOP to 40 000 equals 50Hz with prescaler of 8

    OCR1A = 0x9C4; //1250ppm (*2)
    // set PPM1

    OCR1B = 0xDAC; //1750ppm (*2)
    // set PPM2

    TCCR1A |= (1 << COM1A1)|(1 << COM1B1);
    // set none-inverting mode

    TCCR1A |= (1 << WGM11);
    TCCR1B |= (1 << WGM12)|(1 << WGM13);
    // set Fast PWM mode using ICR1 as TOP
    
    TCCR1B |= (1<<CS11);
    // START the timer with 8 prescaler -> increments twice a us

     while(1) {                
    asm volatile ("nop");
     }                         
     return 0;                 
}

Das Problem ist, dass PB1 und 2 (OC1A/B) einfach nicht schalten wollen. 
Wo/Was ist mein Fehler?

Ich hoffe ihr könnt mir helfen.

Danke,
Andreas

[MWS (Gast)]

Würd' mal die Finger vom PRR lassen. Beim Erstbeschreiben eines 
Registers immer die direkte Zuweisung verwenden.

[Oliver (Gast)]

So auf Anhieb sieht das alles gut aus. Das löschen des Bits im 
PRR-Register kannst du dir zwar sparen, da das nach eime Reset sowieso 
gelöscht ist, aber funktionieren sollte es trotzdem.

Also: Falschen Prozessor zum Compilieren eingestellt? Funktionieren die 
Portpins, wenn du die direkt ansteuerst? Taktfrequenz sollte egal sein, 
selbst bei 1MHz müssten die Pins irgendwann schalten, aber lang genug 
gemessen hast du schon, oder?

Oliver

[Stefan E. (sternst)]

The ICR1 Register can only be written when using a Waveform Generation
mode that utilizes the ICR1 Register for defining the counter’s TOP value.
In these cases the Waveform Generation mode (WGM13:0) bits must be set
before the TOP value can be written to the ICR1 Register.

[Andreas H. (Gast)]

Zunächst an alle ein großes Dankeschön für die schnelle Hilfe :)
@MWS: Ok werd ich machen
@Oliver: Ne compiler passt schon
@Stefan Ernst: Danke! Das hab ich wohl überlesen. Ich habs jetzt mal im 
Programm geändert. Nacher werd ichs dann mal flashen und Testen ;)
Meld mich dann nochmal wenns trotzdem nicht Funktionert.

Ansonsten: Nochmals Danke für die Hilfe. Das war mein erster Beitrag in 
diesem Forum und ich bin begeistert über die Hilfsbereitschaft hier :)

Viele Grüße,
Andreas

[Karl H. (kbuchegg)]

Stefan Ernst schrieb:
>

The ICR1 Register can only be written when using a Waveform 
> Generation
> mode that utilizes the ICR1 Register for defining the counter’s TOP 
> value.
> In these cases the Waveform Generation mode (WGM13:0) bits must be set
> before the TOP value can be written to the ICR1 Register.

Oh, das ist aber eine böse Falle. Und so schön versteckt im Datenblatt.

[Karl H. (kbuchegg)]

Andreas H. schrieb:

Etwas konstruktive Kritik


>     ICR1 = 0x9C40;
>     // set TOP to 40 000 equals 50Hz with prescaler of 8

wenn du 40000 meinst, dann schreib auch 40000 und nicht 0x9c40.

Und im übrigen müssen es 39999 sein, da ja der Zählvorgang von 39999 auf 
0 auch 1 Zählvorgang ist

     ICR1 = 39999;

>     OCR1A = 0x9C4; //1250ppm (*2)
>     // set PPM1

Detto: Wenn du 1250 meinst, dann schreib auch 1250. Und wenn du das 
Doppelte davon willst, dann spricht nichts dagegen das so zu schreiben

     OCR1A = 1250 * 2;

Jeder Kommentar, den du einsparen kannst, weil du genau dasselbe direkt 
im Code ausdrücken kannst, ist es Wert dass man ihn durch entsprechenden 
Code ersetzt.

[Ingo (Gast)]

In der Tat sehr böse!!! Hatte noch nie diese Problem, da ich das 
Register immer zum Schluss beschreibe...

[Karl H. (kbuchegg)]

Ingo schrieb:
> In der Tat sehr böse!!! Hatte noch nie diese Problem, da ich das
> Register immer zum Schluss beschreibe...

Geht mir genauso.
Allerdings hätte ich diesen Absatz nie und nimmer bei der Beschreibung 
der Input Capture Edge Select Bits gesucht, sondern beim ICR1H/L 
Register. Nur dort steht nichts davon.

Na ja. Wieder was, was man sich merken muss.

[Andreas H. (Gast)]

@Kalr Heinz: Hab den Code jetzt übersichtlicher umgeschrieben.

Und wenn ich ICR am Ende schreib funktionerts perfekt.

[Ingo (Gast)]

Gibt's eigentlich ne Rubrik "Fallen"? Da gehört das rein!