PWM per UART verändern in C

[Thorsten K. (thorstenk)]

Juuten Tach, erstmal.

Bin neu im Gebiet der µC und arbeite gerade an meinem Praktikumsthema 
und genau da hackt es derzeit.
Ich soll mit einem Atmega8 folgendes realisieren:
PWM (hab ich), die per UART verändert wird in der Pulsbreite (klappt 
nicht)
Timer mit 50 µs (hab ich)
UART, welches gesendet Zeichen zurück schickt (hab ich).

Nur eben die Verarbeitung der Zeichen aus der UART in das entsprechenden 
PWM will nicht so ganz.

Hat da jemand eine Idee?
Fertige Code´s sind zwar schön, helfen mir nicht, will es ja auch 
lernen.

[Werner A. (homebrew)]

Dann solltest du mal zeigen, was du schon hast...

[Thorsten K. (thorstenk)]

Anbei die c-File. WinAVR gibt ein paar Warnmeldungen wegen den 
Funktionsbezeichnungen raus, dass sehe ich aber nich als großes Problem 
an.

[Stefan E. (sternst)]

Thorsten Karrenbrock wrote:
> Anbei die c-File. WinAVR gibt ein paar Warnmeldungen wegen den
> Funktionsbezeichnungen raus, dass sehe ich aber nich als großes Problem
> an.

Als Anfänger sollte man jede Warnung wie eine Fehlermeldung behandeln. 
Beispiel aus deinem Code:

main.c:39: warning: passing argument 1 of 'eeprom_read_byte' from 
incompatible pointer type

volatile int16_t eeFooWord EEMEM = 0x64;
...
  test_byte = eeprom_read_byte(&eeFooWord);

Na, fällt dir da was auf?

[Thorsten K. (thorstenk)]

Das da eindeutiger der Pointertyp nicht stimmt ist mir klar und wie ich 
es korrigieren werde wäre mein nächster schritt, manchmal bringt es 
nicht, wenn man 2 Stunden sich um einen Fehler kümmert, da sollte man 
lieber abschalten und was anderes machen und genau so gehe ich da ran, 
entweder ich versuche knapp 2 Stunden dass mit dem UART hinzubekommen 
oder machen mich an den Bearbeitung der Wanungen.

[Stefan E. (sternst)]

Wenn du für diese Warnung 2 Stunden brauchst, ist das nur ein Grund 
mehr, sich damit zu beschäftigen, denn dadurch bekommt man 
Wissen/Erfahrung.
Und noch was: was ist, wenn du es mit dem UART genau deshalb nicht 
hinbekommst, weil die Warnungen auf ein Problem deuten, was das 
verhindert? Man sollte den vorhandenen Code erstmal fehlerfrei bekommen, 
bevor man neuen Code hinzu schreibt.

[Stefan E. (sternst)]

> Nur eben die Verarbeitung der Zeichen aus der UART in das entsprechenden
> PWM will nicht so ganz.

Ich kann gar keine "Verarbeitung" sehen, du schickst es einfach nur 
zurück.

[Thorsten K. (thorstenk)]

Das ist ja mein Problem wo ich hänge, an der Verarbeitung der gesendeten 
Zeichen.

[Stefan E. (sternst)]

Naja, bei "klappt nicht" und "will nicht so ganz" dachte ich halt, es 
sei schon was da, was aber nicht funktioniert. Eigentlich meintest du 
also "fehlt noch".

Wie kommt denn der neue Wert per UART, als binärer Wert, oder als Text?

[Thorsten K. (thorstenk)]

Per Text, wollte denn dann eigendlich per atoi umwandeln, vergleichen 
und je nach ergebnis des Vergleichs eine Anweisung ausführen lassen.

[Stefan E. (sternst)]

Ja, und woran hapert es genau?

[Timmo H. (masterfx)]

>Per Text, wollte denn dann eigendlich per atoi umwandeln, vergleichen
>und je nach ergebnis des Vergleichs eine Anweisung ausführen lassen.
Siehste dann machs doch einfach. Wenn du die Daten zurückschicken 
kannst, dann kannst du sie doch auch an atoi übergeben, oder etwa nicht?

[Thorsten K. (thorstenk)]

Kleines Erfolgserlebnis:

habe mich in die get_string() geklemmt und lasse mir dann den
über atoi den uart_rx_buffer in eine int umwandeln.
Sobald der int-Wert kleinergleich ICR1 ist, ändert er die Pulsbreite.

void get_string(char *daten)
{
 int x;
 if (uart_rx_flag ==1)
 {
   strcpy(daten, uart_rx_buffer);
   x = atoi (uart_rx_buffer);
   if(x<=ICR1)
   {
     uart_eeprom = x;
   } 
   uart_rx_flag = 0;
 }
}