16-Bit Timer zählt nicht bis 65535

[Jens K. (mister232)]

Hallo Leute,

ich möchte auf einem ATmega32u4 einen 16-Bit Timer nutzen um an den Pins 
OCR1A und OCR1B jeweils 16-Bit PWM auszugeben. Ich habe den Timer wie 
folgt initialisiert:

DDRB |= (1<<PB5) | (1<<PB6); // PWM-Ports as outputs
TCCR1A |= (1<<WGM11) | (1<<COM1B1) | (1<<COM1A1); // PWM, fast mode, 16 bit, non-inverting
TCCR1B |= (1<<CS11) | (1<<CS10) | (1<<WGM12) | (1<<WGM13); // Prescaler 64
ICR1 |= 0xFFFF; // Set counter border to 16-Bit

Lasse ich den OCR1A-Wert ausgeben und langsam inkrementieren, dann hat 
das PWM-Signal bereits bei 1013 sein Maximum und nicht 65535.

Was mache ich falsch?

[Karl H. (kbuchegg)]

Jens K. schrieb:

> Was mache ich falsch?

Du zeigst nicht das komplette Programm, so dass wir dir nicht helfen 
können, die Stelle in deinem Programm zu finden, wo du dir entweder 
eines der TCCR Register oder das ICR1 zerschiesst.

[Jens K. (mister232)]

Der Rest des Programmes mach eigentlich folgendes

OCR1B = OCR1B + 10;
    
if (OCR1B == 0)
OCR1B = 65535;
      
sprintf(testString,"OCR-Value: %d\n\r", OCR1B);
serPuts(testString);

[Karl H. (kbuchegg)]

Das hier

TCCR1A |= (1<<WGM11) | (1<<COM1B1) | (1<<COM1A1); // PWM, fast mode, 16 bit, non-inverting
TCCR1B |= (1<<CS11) | (1<<CS10) | (1<<WGM12) | (1<<WGM13); // Prescaler 64
ICR1 |= 0xFFFF;

ist übrigens keine so gute Idee. Du willst hier sicherlich nicht, dass 
dir eventuelle 'alte' Bits in den Registern stehen bleiben, sondern du 
willst hier ganz sicher, dass insbesondere die TCCR Register genau die 
Bits, und nur die Bits, gesetzt hat, die du angegeben hast. Dann mach 
da auch eine Zuweisung, damit sich da auf keinen Fall irgendwelche 
"Altlasten" durchmogeln.

TCCR1A = (1<<WGM11) | (1<<COM1B1) | (1<<COM1A1); // PWM, fast mode, 16 bit, non-inverting
TCCR1B = (1<<CS11) | (1<<CS10) | (1<<WGM12) | (1<<WGM13); // Prescaler 64
ICR1 = 0xFFFF;

Die Bits an sich sind korrekt. WGM11, WGM12 und WGM13 sind der "Fast PWM 
Mode mit dem Top Wert aus ICR1". Und du setzt sie auch im korrekten 
Register. Ergo muss das Problem in dem Codeteil liegen, den du nicht 
zeigst, sofern wir davon ausgehen können (und nach einem Reset können 
wir das), das die TCCR Register alle urspünglich den Wert 0 enthielten.

[Jens K. (mister232)]

Jetzt funktioniert es. Hatte tatsächlich an einer Stelle die 
Registerwerte wieder überschrieben :-)