Frequenz/timer auswertung atmega8 und ICP

frankparker wrote:
> naja jetzt läuft das Programm ja soweit

Wenn deine Interrupt Routine bei konstanter Frequenz mit immer
unterschiedlichen Werten daherkommt, kann man wohl kaum von funktioniern 
sprechen.


Wie sieht dein Programm jetzt eigentlich aus.

1

;---------------------------------------------------------------------------

2

onTC1:    ;push  temp

3

      ;push  temp2

4

5

      in  temp,ICR1L

6

      in  temp2,ICR1H

7

8

      mov  ZL,temp    ;ZL mit temp(ICR1L) laden

9

      mov  ZH,temp2  ;ZH mit temp2(ICR1H) laden

10

      sub temp,YL    ;temp(ICR1L) minus YL(zuerst 0 dann letzter Messwert)

11

      sbc  temp2,YH  ;temp2(ICR1H) minus YH(zuerst 0 dann letzter Messwert)

12

13

      mov YL,ZL    ;YL mit ZL(temp(ICR1L)) laden

14

        mov YH,ZH    ;YH mit ZH(temp2(ICR1H)) laden

15

16

17

18

        mov  XL,temp

19

        mov  XH,temp2

20

21

      ;pop temp

22

      ;pop temp2 

23

        reti

24

;---------------------------------------------------------------------------

Das wird wohl nicht so gut kommen, wenn deine ISR den Inhalt der 
temp/temp2 Register ohne sichern einfach so verändert. Diese Register 
werden auch an anderen Stellen benutzt!

Was machst du da überhaupt?
Vereinfach das doch mal gründlich. Das ist doch viel zu kompliziert für 
ein simples
   differenz = neuer_wert - alter_wert
   alter_wert = neuer_wert

Bei dir ist XH/XL  die Differenz
            ZH/ZL  der neue Wert
            YH/YL  der alte Wert

(PS: Wenn du den Registern schon Namen gibst, was ja grundsätzlich 
nicht schlecht ist, warum gibst du ihnen dann nicht Namen unter denen 
man sich was vorstellen kann, wie zb AlterWert, Differenz, ...  XH/XL 
ist genauso nichtssagend wie es R18/R19 gewesen wäre)

Die temp/temp2 Register braucht in dieser ISR kein Mensch

Es ist völlig OK, wenn in deiner Hauptschleife die Differenz ständig 
ausgegeben wird. Lass das XH/XL Register dort in Ruhe!
Du kriegst keinen Bonus dafür, wenn dein Prozessor in der Hauptschleife 
sitzt und nur looped.

Und lass die Interrupts nur solange gesperrt, wie es unbedingt notwenig 
ist. In dem Moment, in dem du die Differenz aus den XH/XL register 
geholt hast, können die Interrupts wieder freigegeben werden! Du willst 
ja, dass die Capture Einheit möglichst schnell wieder reagieren kann!

Sobald du den Differenz Wert hast, hast du alle Zeit der Welt für die 
Ausgabe. Es gibt keinen Grund dafür die Interrupts zu sperren. 
Vorausgesetzt natürlich, deine ISR kommt sich nicht registermässig mit 
der Ausgaberoutine in die Haare. Aber die Lösung für dieses Problem kann 
nicht lauten, dass man die Interrupts in der Zeit sperrt. Die Lösung 
muss lauten, die Register entsprechend intelligent zu benutzen bzw. die 
ISR darf keine Register verändern. Insbesondere das SREG Register wird 
gerne vergessen!