mmh wer findet den Fehler?

Gast #1175394
Lesenswert?
• ▲
▼
Hallo zusammen,

nachdem ich heute bereits hin und her probiert habe, wende ich mich nun 
an Euch.

Im Anhang ist der Code für meine Maschinensteuerung. Da ich mich mit dem 
ADW des Atmega32 nicht so gut auskenne, aber gleich ein praktisches 
Problem dafür hatte, habe ich dieses Programm geschrieben.

Zwei Maschinen hängen an den PWM-Ausgängen der Timer0 und Timer2.
Die Pulsweite soll ein Poti bestimmen, das am ADW-Kanal 2 angeschlossen 
ist.
Die Spannungen, die an den Maschinen anstehen, wwerden von 
OP-Verstärkern auf 0-5V gebracht und liegen an den ADW-Kanälen 0 und 1.
Das Programm soll nun so funktionieren, dass es den Unterschied zwischen 
Sollwert (Poti) und der Spannung an der Maschine errechnet und die 
Pulsweite entsprechend neu berechnet.
Ist die MAschine im linkslauf, stehen Spannungen zwischen 0 und 2,47V 
an.
Im Rechtslauf Spannungen zwischen 2,61 und 5V.
Durch den Spannungsbereich, in dem die Maschine (2,47V bis 2,61 V) nicht 
reagiert, weil die Pulsweite zu klein ist, entstehen die Werte 488 und 
535, die auch dazu dienen, die Drehrichtung der Maschine zu ermitteln.

Das Poti ist Teil eines Spannungsteilers und legt an den ADW eine 
Spannung zwischen 2,5 und 5V.

Das Programm gibt Werte erstmal nur an OCR0, berechnet aber alles so, 
als wären beide Maschinen angeschlossen.

Nun zum Problem:
----------------
Wenn ich am Poti drehe, dann passiert nix, obwohl ich zum Ergebnis 
gekommen bin, dass die Wandlungen funktionieren, weil ich mir das 
Ergebnis des Potis auf den PORTC habe anzeigen lassen. Irgendwo ist der 
Wurm bei den Berechnungen. vielleicht verliere ich auch Werte, weil ich 
so viel casten muss, um die Datentypen verträglich zu machen, und mich 
dabei vertan habe.
Vielleicht liegt der Fehler aber auch ganz anderswo.

Könnt Ihr mir helfen?
Angehängte Dateien:
#1175482
Lesenswert?
• ▲
▼
Nein, "setpoint = setpoint -512;" dürfte keine Probleme machen. int ist 
16 Bit lang, d.h. Zahlen von -32768 bis 32767
Aber hier ist ein Problem:
"offset[x] = (offset[x] & 0x7fff);"
Die Zahlen sind im 2er Komplement kodiert, somit liefert das Abschneiden 
des ersten Bits nicht die entsprechende Minus-Zahl. Warum nicht einfach:
"offset[x] = offset[x] * (-1);"

Gruß, Johannes
Gast #1175873
Lesenswert?
• ▲
▼
> Aber hier ist ein Problem:
> "offset[x] = (offset[x] & 0x7fff);"
> Die Zahlen sind im 2er Komplement kodiert, somit liefert das Abschneiden
> des ersten Bits nicht die entsprechende Minus-Zahl. Warum nicht einfach:
> "offset[x] = offset[x] * (-1);"

Geht auch nicht. Das hatte ich zuerst da stehen. Das war jetzt Mut der 
Verzweiflung bei mir! :(
Gast #1176008
Lesenswert?
• ▲
▼
Ich hab den Fehler gefunden.
1
        if (!((OCR0 - offset[0]) < 0))  // Unterlauf abfangen
2
        {
3
          OCR0 = OCR0 - ((unsigned char) offset[0]);
4
                        // Neuen Spannungswert ausgeben
5
        }
6
        else
7
        {
8
          OCR0 = 0;          // kleiner geht nicht
9
        }

So ist es richtig.

Antwort schreiben

Bitte melde dich an, um einen Beitrag zu schreiben.

oder

Mit Google-Account einloggen

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren