ext Interrupt mit ATMega644

[Sven B. (svenba)]

Hi!
Ich versuche, ein Touchdisplay per ext Interrupt an PinA4 meines 
ATMega644 abzurufen. Entweder läuft der Interrupt immer wieder durch 
oder das ganze Programm hakt nach ein paar Mal touchen und alles bleibt 
stehen. Die Werte werden so gut wie nie richtig angezeigt.
Leider findet man nirgends richtige Angaben, wie denn nun die PCInts 
außer 0 und 1 richtig angesteuert werden. Hier mal meine Init und ISR, 
ich hoffe, ich kriege ein paar Tipps von euch!!!

; Erst auswählen welcher Int(0 bis 2) mit High.
;   hier: PCInt0 weil Pin für Touch an PA4 = PCInt4 = INT0
        lds    temp1,EIMSK
        sbr    temp1, 0<< INT2 | 0<<INT1 | 1<<INT0
        sts    EIMSK,temp1
;  Flanke H/L, L/H oder Low Pegel einstellen 
        lds    temp1,EICRA
        sbr    temp1,  1<<ISC01 | 0<<ISC00  ; Low Level löst Int aus
        sts    EICRA,temp1      
      
    /*    lds    temp1,EIFR
        sbr    temp1, 1<<INTF2 | 1<<INTF1 | 1<<INTF0
        sts    EIFR,temp1
*/
        
; Welche Pin-Gruppe?
        lds temp1, pcicr
        sbr temp1, 1 << PCIE0
        sts pcicr, temp1
;Welcher Pin in Gruppe genau?
        lds temp1, pcmsk0
        sbr temp1, 1 << pcINT4    ; PA4 als Auslöser
        sts pcmsk0, temp1
        
      ;  sei  ; Alle Interrupts an

Dann kommt das Hauptprogramm, welches im Moment nicht viel macht.
Die ISR sieht so aus:

touch_beruehrt:
                    ; interrupts aus
          push temp1
          in temp1, sreg
          push temp1
          push temp2
          push temp3
          push Discursorl    ; vorherige Werte
          push discursorh
          push disdata1
          push disdata2
          
          in temp1, pina
          andi temp1, 0b00010000
          cpi temp1, 0b00010000
          breq alles_gleich  
    
    ldi     zL, LOW(Touchwert)   ; ZL mit den unteren Bits der Adresse initialisieren
    ldi     zH, HIGH(Touchwert)  ; ZH mit den oberen Bits der Adresse initialisieren          
    ld Discursorl, z+    ; discursorl = temp3 = Zahl
    ld discursorh, z    ; discursorh = temp4 = - oder +

            
          
  
  
          rcall Touch_speichern_x
          rcall Touch_speichern_y
        
          rcall touch_auswerten; jetzt stehen Zahl und +/- in y
          ldi     zL, LOW(Touchwert)   
          ldi     zH, HIGH(Touchwert) 
          
          ld temp1, z+
          cp discursorl, temp1
          breq Zahl_gleich

;Zahl nicht gleich:
          ldi disdata1, 0x0c
          mov disdata2, temp3;discursorl
          rcall sende_daten
          mov disdata2, temp4;discursorh
          rcall sende_daten
          
          ldi     zL, LOW(Touchwert)   
          ldi     zH, HIGH(Touchwert)
          st z+, temp3
          st z, temp4
          
          rcall touch_ruhe
          
          pop disdata2
          pop disdata2
          pop discursorh
          pop  discursorl
          pop temp3
          pop temp2
          pop temp1
          out sreg, temp1
          pop temp1
          
          reti          


Zahl_gleich:
          ld temp1, y
          cp discursorh, temp1
          breq alles_gleich
; nicht gleich::            
          ldi disdata1, 0x0c
          mov disdata2, temp3;discursorl
          rcall sende_daten
          mov disdata2, temp4;discursorh
          rcall sende_daten
          
          ldi     zL, LOW(Touchwert)   
          ldi     zH, HIGH(Touchwert)
          st z+, temp3
          st z, temp4   

alles_gleich:
          rcall touch_ruhe
          pop disdata2
          pop disdata1
          pop discursorh
          pop  discursorl
          pop temp3
          pop temp2
          pop temp1
          out sreg, temp1
          pop temp1
          
          reti

Hilfe!! Stehe ein bißchen auf dem Schlauch!!

[Gast 57 (Gast)]

Ich vermute der Fehler liegt beim Zurücklesen vom Stack,

;Zahl nicht gleich:
          ldi disdata1, 0x0c
          mov disdata2, temp3;discursorl
          rcall sende_daten
          mov disdata2, temp4;discursorh
          rcall sende_daten

          ldi     zL, LOW(Touchwert)
          ldi     zH, HIGH(Touchwert)
          st z+, temp3
          st z, temp4

          rcall touch_ruhe

          pop disdata2; !!!!!!!!!!!!!!!! Beides Mal das gleiche
          pop disdata2; !!!!!!!!!!!!!!!! Register, Tipp-Fehler!
          pop discursorh
          pop  discursorl
          pop temp3
          pop temp2
          pop temp1
          out sreg, temp1
          pop temp1

Schönes Wochenende

[Sven B. (svenba)]

Danke schön, werd´s mal ändern.

Ist die Init der Interrupts denn so richtig??