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

Hinweis: der ursprüngliche Beitrag ist mehr als 6 Monate alt. Bitte hier nur auf die ursprüngliche Frage antworten, für neue Fragen einen neuen Beitrag erstellen.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.