Merkwürdige Overflows bei PWM-Ausgang

[hochsitzcola (Gast)]

Hallo Zusammen,

ich steure an einem ATTiny85 einen PWM-Ausgang (8-Bit) an. Das 
funktioniert auch soweit. Den Sollwert errechne ich aus einem 
eingelesenen Spannungswert vom AD-Wandler. Funktioniert auch. Wenn ich 
den AD-Wert mal 1000 nehme und hinter wieder durch 1000 teile kommt am 
Ausgang nur noch Mist. Die Variable ist als long definiert, sollte also 
ausreichend groß sein. Ich hab schon mit Typumwandlung rumprobiert, hat 
auch irgendwie Einfluß, auf den Ausgang, aber nicht den gewünschten 
effekt. Ich hab auch schon im Forum gelesen, daß das RAM überlaufen 
könnte, weiß aber nicht ob das der Fall ist. Wer kann helfen?

Gruß
Martin

[Klaus W. (mfgkw)]

Wahrscheinlich sind deine Zwischenergebnisse nicht long, sondern
zu klein.

Mangels Begabung habe ich deinen Quelltext leider nicht so recht
verstanden...

hochsitzcola schrieb:
> Wer kann helfen?

jemand, der deinen Quelltext sieht.
Oder ein Guru. Dem zahlst du Geld, und danach ist dir dein Programm
egal und du bist auch ohne glücklich.

[hochsitzcola (Gast)]

hier der prinzipielle Quellcode

unsigned int vact = 0;
unsigned int aus = 0;
long ziel = 0;
....

vact = ADCW;

ziel = vact*1000;
ziel /=1000;
aus = ziel/4;
OCR0A = aus;



ist natürlich noch mehr drum rum, aber das sind die Schritte bei denen 
wohl irgendwas schief geht....
vact ist der Wert vom ad-Wandler also max. 1024
Der Timer für die PWM ist nur 8 Bit, darum das durch 4 teilen.

Gruß
Martin

[Klaus W. (mfgkw)]

vact ist 2 Byte groß und geht bis etwa 65500.
Wenn der AD-Wert größer als 65 ist, reicht das nicht mehr, weil
in vact*1000 eine unsigned int und eine int multipliziert werden,
das findet in unsigned int statt.
Danach erst wird es an eine long ziel zugewiesen und dafür auf
4 Byte aufgeblasen, aber da sind schon Stellen gelöscht.
ziel = vact*1000ul wäre korrekt.

[Klaus W. (mfgkw)]

Wobei ich nicht verstehe, warum du mit 1000 multiplizierst und
danach gleich wieder teilst.
Aber du wirst deine Gründe haben...

[Stefan B. (stefan)]

Diese Rechnung hat einen Integerüberlauf z.B. wenn ADCW 66 ist (66 * 
1000 ist größer 16-Bit unsigned int).

ziel = vact*1000;

Schreib es so:

ziel = vact * 1000L;

[hochsitzcola (Gast)]

Danke für die prompte Hilfe, jetzt läuft's!
Learning by doing hat ab und zu so seine Tücken, ich hätte wohl im 
C-Kurs besser aufpassen sollen!

Gruß
Martin