Frage zu Soft-PWM

Bene Gast #1102910
Lesenswert?
• ▲
▼ 
Hallo zusammen,

ich habe eben mit dem Soft-PWM Programm aus dem Tutorial rumgespielt. 
Ich möchte die auf 100 Hz definiert PWM-Frequenz erhöhen und erhalte 
dabei immer folgende Fehlermeldung:

../soft_pwm_test.c:129: warning: integer overflow in expression


Ausschnitt aus dem Code:
// Timer 1 Output COMPARE A Interrupt
ISR(TIMER1_COMPA_vect)
{
    static uint8_t pwm_cnt=0;
    uint8_t tmp=0;

    OCR1A += (uint16_t)T_PWM;  <== Zeile 129 <== <== <== <== <==

    if (pwm_setting[0] > pwm_cnt) tmp |= (1<<0);
    if (pwm_setting[1] > pwm_cnt) tmp |= (1<<1);
    if (pwm_setting[2] > pwm_cnt) tmp |= (1<<2);
    if (pwm_setting[3] > pwm_cnt) tmp |= (1<<3);

    PWM_PORT = tmp;                         // PWMs aktualisieren
    if (pwm_cnt==(uint8_t)(PWM_STEPS-1))
        pwm_cnt=0;
    else
        pwm_cnt++;
}

Kann mir jemand einen Tip geben?

Gruß

Bene
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden
Karl H. #1102925
Lesenswert?
• ▲
▼ 
Bene wrote:

> ../soft_pwm_test.c:129: warning: integer overflow in expression

Das bedeutet nichts anderes, als dass in einer konstanten Berechnung der 
Zahlenbereich überschritten wurde.

>     OCR1A += (uint16_t)T_PWM;  <== Zeile 129 <== <== <== <== <==

Aha. Dann sehen wir uns einfach mal an, was T_PWM ist.

#define T_PWM (F_CPU/(F_PWM*PWM_STEPS))

wobei
#define F_CPU 8000000L
#define F_PWM 100
#define PWM_STEPS 256

die einzige Integer-Berechnung die in der T_PWM Formel vorkommt, ist 
F_PWM*PWM_STEPS

Mit deinen gewählten Werten ergibt sich also, dass F_PWM*PWM_STEPS einen 
Wert größer als 32767 ergibt. Und damit ist das ausserhalb des 
int-Bereiches bei 16 Bit. Du könntest zb. den Compiler zwingen die 
Berechnung nicht in int, sondern in long durchzuführen. Dazu genügt es, 
wenn eine der beiden Zahlen eine Long-Zahl ist. Also zb

#define F_PWM 100L
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Karl H. #1102939
Lesenswert?
• ▲
▼ 
Mikes wrote:
> @Karl Heinz
> Und was bringt Dir das, wenn OCR1A doch nur die untersten 8 Bit addiert?

Du irrst.
OCR1A ist ein 16 Bit Register, bestehend aus OCR1AH und OCR1AL


Selbst wenn es ein 8 Bit Register wäre:
Ist uninteressant, der Compiler kann das sowieso nicht prüfen, da der 
neue Wert des Reigsters unter anderem auch vom alten Wert abhängt. Eine 
Warnung wie die oben angegebene wird vom Compiler normalerweise nur dann 
ausgegeben, wenn alle beteilgten Operanden konstant sind. Und bei += 
sind sie das nun mal nicht.

Ausserdem: Auch eine konstante Berechnung, die nur aus int besteht, kann 
durchaus ein Ergebnis liefern, welches in einen uint8_t passt.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Thread beobachten

Antwort schreiben

Bitte melde dich an, um einen Beitrag zu schreiben.

Bestehender Account

oder

Mit Google-Account einloggen

Noch kein Account?

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren