Frequenzmessung mit einem Mega32

[Dirk]

Hallo Daniel,

vielleicht kannst du dir da (https://www.roboternetz.de/community...ngen-Messgerät) etwas abgucken?
(Ist zwar für den RP6, aber der hat auch den ATmega32...)

[Kampi]

Danke euch beiden.
Ich habe das gestern auch mal mit der Capture and Compare Unit probiert aber damit hat es leider gar nicht geklappt. Eigentlich dachte ich für sowas einfaches würde das ohne Probleme mit dem Timer klappen....aber naja :/
Der Compiler hat mir gar nichts zu der Variable Counter gesagt (und die Ausgabe per UART klappt auch ohne Probleme) aber trotzdem danke für den Hinweis
Den Link schaue ich mir mal an und gucke mal ob ich da was von entnehmen kann.

[MagicWSmoke]

vielleicht kannst du dir da (https://www.roboternetz.de/community/threads/47088-RP6Control-M32-Impulslängen-Messgerät) etwas abgucken?

Warum setzt Du eigentlich den Wert icrcnt_start.i16h in der ISR laufend zurück, ohne dass Du ihn jemals änderst ? Nicht dass es viel ausmachen würd', aber es ist sinnlos.

 icrcnt_start.i16h = 0;            // Reset upper 16 bits (start value)

Für die einmalige Initialisierung hätte Dir das gereicht:

static icrcounter_t icrcnt_start = {.i32=0};

Der Compiler hat mir gar nichts zu der Variable Counter gesagt (und die Ausgabe per UART klappt auch ohne Probleme) aber trotzdem danke für den Hinweis

../hh10d_meas.c:48: warning: 'Counter' may be used uninitialized in this function

Wenn der Stackspeicher nicht stark benutzt wird, dann ist es möglich dass zufällig für sich diese lokale Variable immer wieder die gleiche Adresse im Stack benutzt wird, dann geht es.
Änderst sich die Stackbenutzung kann dort ein anderer Wert stehen, mit entsprechendem Ergebnis.
Grundsätzlich hat eine lokale Variable als verloren betrachtet zu werden, außer sie wird static deklariert, dann bleibt sie im Speicher erhalten.

[Kampi]

Warum setzt Du eigentlich den Wert icrcnt_start.i16h in der ISR laufend zurück, ohne dass Du ihn jemals änderst ? Nicht dass es viel ausmachen würd', aber es ist sinnlos.

 icrcnt_start.i16h = 0;            // Reset upper 16 bits (start value)

Für die einmalige Initialisierung hätte Dir das gereicht:

static icrcounter_t icrcnt_start = {.i32=0};

Wenn der Stackspeicher nicht stark benutzt wird, dann ist es möglich dass zufällig für sich diese lokale Variable immer wieder die gleiche Adresse im Stack benutzt wird, dann geht es.
Änderst sich die Stackbenutzung kann dort ein anderer Wert stehen, mit entsprechendem Ergebnis.
Grundsätzlich hat eine lokale Variable als verloren betrachtet zu werden, außer sie wird static deklariert, dann bleibt sie im Speicher erhalten.

Ok dank dir für den Hinweis
Ich programmiere nicht so viel und so oft in C, daher sind mir solche Feinheiten nicht ganz klar.

[Kampi]

Hallo Daniel,

vielleicht kannst du dir da (https://www.roboternetz.de/community/threads/47088-RP6Control-M32-Impulslängen-Messgerät) etwas abgucken?
(Ist zwar für den RP6, aber der hat auch den ATmega32...)

Hi,

nochmal kurz ne Frage.....
So richtig steige ich durch dein Programm (leider) nicht durch. Kannst du mir im groben Erklären wie das mit der Input-Capture Unit und deinem Programm funktioniert?
Dank dir!

[MagicWSmoke]

Ich empfand das verlinkte Programm auch ein wenig undurchsichtig, vor allem die Structs und Unions, kann man zwar so machen, muss man aber nicht. Auch ist, obwohl's tatsächlich nicht viel ausmacht, bei jedem Triggern des Input Capture eine Messung möglich. Ein Unterscheiden zwischen erster und zweiter Flanke ist nicht notwendig.
Die normale Vorgehensweise ist so, man nimmt sich 'ne unsigned 16Bit Variable und zieht in der ISR immer den alten Zählerstand vom Neuen ab. Dass dann auch z.B. 200 - 65000 richtig behandelt wird, wird durch Berücksichtigung der Overflows erreicht. Die Overflows werden mit der Zählerbreite multipliziert und zum Ergebnis addiert.
Wichtig ist dabei auf race-conditions aufgrund der Codelaufzeit zu achten, die im Bereich des Zählerüberlaufs vorkommen können, als auch das Ergebnis atomar zu übergeben, damit während der Ausgabe nichts vefälscht wird.