Timer genaue Zeit und entprellen

>internen vom STK500 8Mhz

Der interne "Quarz" ist ein Oszillator, der mit 3,686 MHz läuft.
Aus dem STK-500-Userguide:
"The frequency of the software generated clock can be set from 0 to 
3.68Mhz. The default value is 3.68Mhz. Section "Oscillator" explains how 
to set the clock frequency from AVR Studio."

Wenn du 8KHz hast, dann sind die controllerintern per RC-Oszillator 
erzeugt, welcher relativ ungenau ist...
Die STK500-Frequenz ist da genauer...

Im Programm ist noch die von PeDa verwendete Quarzfrequenz angegeben.
Ansosnten könnte es funktionieren. Aber waum gibts du die Sekunden per 
DDR aus und nicht per PORT?

Wo liegt denn Dein Problem ?
Man nehme den TimerIRQ, zähle darin die Sekunden hoch und schalte die 
LED im main je nach Sekunde an oder aus:

1

for(;;){

2

    if( second == 60 )

3

      second = 0;

4

    if(second%2)

5

      PORTA &= ~(1<<PINA0); //LED aus

6

    else

7

      PORTA |= (1<<PINA0);  // LED an

8

  }

Also statt nun binär wie im Code acht LEDs gleichzeitig zu schalten, 
wird nur eine getoggelt.
Oder meinst Du etwas anderes ?

Hi Christoph,
>... eine "12". Wenn der String am uC ankomt dann soll die
> Sekunde auf 12 gestellt werden und weiterlaufen. Wie ich einen String
> mit dem uC empfange weiß ich, aber wie setze ich dann die Zeit?

1

int8_t sekunde;

2

sekunde=atoi(string_mit_Sekundenwert);

Gruß

Fred

Hallo Christoph,
> Die Variable Sekunde benötige ich aber als unsigned char und nicht als
> integer. Wie wandle ich das um?

Karl Heinz hat Dir die Antwort schon gegeben. Denk mal nach: "Sekunde" 
kann maximal welchen Wert annehmen? Ja, 60. Wie viele Bits sind dafür in 
einer int8_t Variablen erforderlich? Wird dabei das Vorzeichen-Bit 
gesetzt? Nein? Dann gibt es auch keine Komplikationen bei der Umwandlung 
in unsigned! Falls Du dies nicht nachvollziehen kannst, empfielt sich 
ein Tutorial zu Datentypen.

Gruß

Fred

Hallo Christoph,

Du gibst die Sekunde an "LED_DIR" aus, und das ist DDRA. Wahrscheinlich 
willst Du aber an PORTA o.ä. ausgeben!

Der Befehl

1

2

  DDRA  |= (1 << PA6);

 im Anschluss an

1

  LED_DIR = 0xFF;

bewirkt auch nichts, da schon alle Bits gesetzt sind.

Welche Warnung gibt Dir denn der Compiler?

Gruß

Fred

> ../uhr.c:130: warning: passing arg 1 of `uart_puts' makes pointer from
> integer without a cast

uart_puts erwartet einen Zeiger (auf den Anfang eines Arrays aus char). 
Stattdessen übergibst du ihm einen unsigned char. Das paßt überhaupt 
nicht.

> ../uhr.c:139: warning: implicit declaration of function `atoi'

Dir fehlt der Header <stdlib.h>.

Hallo Christoph,

> Ich habe oben einmal F_CPU definiert und dann XTAL. Darf man das
> überhaupt?
Normalerweise wird F_CPU aus den voreingestellten Parametern (beim 
AVRStudio: "Project"|"Configuration Options") übernommen. Wie es bei 
reiner Verwendung eines "makefile" geht, weiß ich nicht. Welche 
"toolchain" benutzt Du?

Was Johannes zu uart_puts() sagt, stimmt. Wenn Du die Sekunde wieder in 
einen String zurückwandeln möchtest, könntest Du "itoa()" verwenden. 
Oder Du nimmst einfach Deinen "buffer" -- oder setzt Du atoi() auch ein, 
um führende Leerzeichen im übertragenen String zu unterdrücken?

"position" wird nicht so funktionieren, wie Du es Dir wünschst, denn es 
müsste mindestens als "volatile" deklariert sein. Da Du diese Variable 
nur innerhalb der ISR benötigst, wäre es noch besser, "position" 
innerhalb *der* ISR als "static" zu deklarieren und zu 
initialisieren:

1

static uint8_t position=0;

Ich habe nur ganz grob über Deinen Code geschaut, es können durchaus 
noch weitere Probleme drin sein. Versuche mal, die Vorschläge von 
Johannes und mir zu implementieren.

Viele Grüße

Fred

Hallo Christoph,

versuche doch erst einmal, ein Problem (das mit der Binärausgabe) zu 
lösen, ehe Du die serielle Ausgabe probierst. Sonst suchst Du immer 
Fehler an mehreren Stellen gleichzeitig. Das ist ein wichtiges Element 
jeder Fehlereingrenzung. Die serielle Eingabe (siehe Beitrag oben zu 
"position") muss zuerst noch in Ordnung gebracht werden.

Ich empfehle Dir dringlich, Dich mit den Bibliotheksfunktionen vertraut 
zu machen, damit Du Dir Fragen wie die zur Warnung bei Deiner itoa() 
selbst beantworten kannst. Hier eine gute Adresse: 
http://www.nongnu.org/avr-libc/user-manual/group__avr__stdlib.html#g4f6b3dd51c1f8519d5b8fce1dbf7a665
Und das ganze als PDF: 
http://download.savannah.gnu.org/releases/avr-libc/avr-libc-user-manual-1.4.4.pdf

Viele Grüße

Fred

Hi,

1

itoa((int)second, second2, 10);

2

uart_puts(second2);

Was genau hast Du daran nicht hinbekommen? Was ist an der Dokumentation, 
auf die ich Dich im letzten Posting hingewiesen habe, nicht zu 
verstehen?

Lies noch einmal nach, wie man Variable mit "volatile" deklariert.
"position" ist immer noch nicht richtig deklariert (siehe 2 Beiträge von 
mir weiter oben). Ich gebe es auf, Dir hier Tipps zu geben.

Gruß

Fred

Hi Christoph,

> Ich habe auch jetzt die Variable "position" geändert, dass hatte ich
> total vergessen.
OK

> Wie mache ich es denn, das die Sekunde nur einmal zum Rechner geschickt
> wird und nicht viele male hintereinander?
Die Daten sollten (wegen des Flags "habekommando") nur 1x für jeden 
empfangenen String gesendet werden.

Gruß

Fred

Hi,

> Ich möchte das er die Stunden und Minuten in formatierter Uhrzeitform
> sendet. Wie mache ich das?
Dein Programm habe ich mir nicht noch einmal angesehen.
Du stellst eigentlich zwei Fragen. Dazu gleich. Die beste Vorgehensweise 
ist, sich so etwas genau zu überlegen, ehe man Code schreibt. Man 
fragt sich, was wie, wann, unter welchen Umständen, usw. passieren soll 
und welcher Prozess von welchen anderen Prozessen abhängig sein soll. 
Sind diese Fragen beantwortet, kann man ans Kodieren denken.

Erstens fragst Du, wie man die Zeit nur einmal sendet. Dafür schlage ich 
vor, Du detektierst, wann ein neuer Zeitwert (zu testen wäre ja nur der 
Sekundenwert!) vorliegt. Ist das gegeben, sendest Du die Zeit einmal.

Zweitens fragst Du, wie man die Zeit formatieren kann. Vermutlich denkst 
Du an etwas wie HH:MM:SS. Dazu musst Du lernen, wie man Strings 
formatiert. Alternativ kannst Du ja HH schicken, dann den Doppelpunkt, 
dann die Minuten, usw. Die "echten" Befehle zur String-Formatierung 
fressen manchmal viel Speicher (z.B. sprintf(), das mag aber trotzdem 
eine gute Lösung sein!). Da Deine Daten aber in einem festen Format 
(Stunden, Minuten und Sekunden zweistellig) vorliegen, kannst Du auch 
einfacher vorgehen: Du nimmst einen String (z.B.

1

 char Zeitstring[]="  :  :  ";

) und schreibst die Zeitwerte so in diesen String (Vorsicht, fiese 
Falle!):

1

itoa(stundeninteger, Zeitstring, 10);

2

itoa(minuteninteger, &Zeitstring[3], 10);

3

itoa(sekundeninteger,&Zeitstring[6], 10);

Jetzt habe ich Dich absichtlich aufs Glatteis geführt. Hast Du's 
gemerkt? Das geht zwar, aber es gibt einen Haken: itoa() schreibt '\0' 
an das Ende eines jeden Strings, überschreibt Dir also die Doppelpunkte! 
Die musst Du dann erneut setzen.

Wichtige Stringfunktionen, die Du lernen solltest, sind u.a.: strcpy(), 
strcat(), strlen(), strstr().

Das alles nur zur Anregung -- es gibt viele andere mögliche Wege. Die 
Implementierung bleibt "eine Öbung föör den Schööler".

Viel Erfolg,

Fred