Externer Interrupt - ATmega328P - Assembler

[Martin (Gast)]

Hallo Zusammen,
ich beschäftige mich gerade mit dem Microcontroller "ATmega328P" und 
versuche in Assembler über die Externen Interrupts eine LED anzusteuern.

Ziel soll es sein, durch einen Interrupt INT0 oder INT1 die LED 
auszuschalten.

Als Entwicklungsboard benutze ich das Arduino Uno und als 
Entwicklungsumgeben das Atmel AVR Studio 7.0 mit AVRDude zum Flashen.

Hier mal mein Code:

.org 0x0000
  rjmp main
.org 0x0002
    rjmp int0_handler    ; IRQ0 Handler
.org 0x0004
    rjmp int1_handler    ; IRQ1 Handler

main:
  ; Setup interrupt INT0 and INT1
  ldi r16, (1<<ISC11) | (1<<ISC10)  ; INT0 and INT1 - falling edge 
  sts EICRA, r16            ; SET EICRA
  ldi r17, (1 << INT0) | (1 << INT1)  ; Enable INT0/INT1 interrupt
  sts EIMSK, r17            ; Set EIMSK
  ldi r16, 0x00            ; Set all ports of DDRD as input
    out DDRD, r16            ; Set DDRD

  sei                  ; Interrupt enable

  ; Setup LED Output
  ldi r16, 0xFF            ; Set all Ports of DDRB as output
    out DDRB, r16            ; Set DDRB
  ldi r16, 0xFF            ; Set all LEDs on PORTB high
    out PORTB, r16            ; Write PORTB
 
start:            ; Endless loop
    rjmp start

int0_handler:
  ldi r16, 0x00      ; Load R16 with 0
    out PORTB, r16          ; Write R16 out to PortB - All Outputs of low
  reti

int1_handler:
  ldi r16, 0x00      ; Load R16 with 0
    out PORTB, r16          ; Write R16 out to PortB - All Outputs of low
  reti

Meine LED an Port DDRB geht an, jedoch kann ich beide Taster an den 
Externen Interrupts drücken und die LED geht nicht mehr aus.

Zum Test ob die Schalter Funktionieren habe ich sie so einmal 
ausgelesen, dann konnte ich die LEDs an bzw ausschalten.

Könnt ihr mir helfen und sagen was ich übersehen oder falsch mache?

Gruß Martin

[input (Gast)]

; Set all ports of DDRD as input

[Marc V. (Firma: Vescomp) (logarithmus)]

Martin schrieb:
> Könnt ihr mir helfen und sagen was ich übersehen oder falsch mache?

> int0_handler:
>   ldi r16, 0x00      ; Load R16 with 0
>     out PORTB, r16          ; Write R16 out to PortB - All Outputs of 
> low
>   reti
> 
> int1_handler:
>   ldi r16, 0x00      ; Load R16 with 0
>     out PORTB, r16          ; Write R16 out to PortB - All Outputs of 
> low
>   reti

 PortB bleibt auf 0, LEDs sind immer an.
 Probiere es mal mit EOR.

   ser    r17
   in    r16, PORTB
   eor    r16, r17
   out    PORTB, r16

[c-hater (Gast)]

Martin schrieb:

>  ldi r16, (1<<ISC11) | (1<<ISC10)  ; INT0 and INT1 - falling edge

Nö. Um das zu erreichen, was der Kommentar meint, was hier passieren 
würde, hätte man eine andere Konstante benutzen müssen...

Was übrigens gleich beweist, dass du nicht wirklich beide Tasten 
getestet haben kannst, denn bei einer hätte es tastsächlich 
funktionieren müssen (und tut es tatsächlich auch).

Es sei denn, zu dem obigen Programmierfehler kommen noch andere Fehler, 
z.B. in der Beherrschung der Toolchain...

[Martin (Gast)]

Hallo,

ser    r17           ; r17 mit 0xff fuellen
in    r16, PORTB     ; PORTB auslesen
eor    r16, r17      ; Exclusiv Oder mit r17 und r16
out    PORTB, r16    ; r16 zurueckschreiben

habe gerade mal versucht deinen Code zu verstehen.
Habe es mal in die InterruptRoutine eingebaut, hat leider nichts 
geholfen.

Ich habe auch das Gefühl, dass ich nicht mal in die Routine komme und 
irgendetwas bei der Initialisierung falsch gemacht habe.
Weis leider nur nicht was...

Martin

[c-hater (Gast)]

Marc V. schrieb:

>  PortB bleibt auf 0, LEDs sind immer an.

Nö, die LEDs werden in main eingeschaltet.

>  ; Setup LED Output
>  ldi r16, 0xFF            ; Set all Ports of DDRB as output
>    out DDRB, r16            ; Set DDRB
>  ldi r16, 0xFF            ; Set all LEDs on PORTB high
>    out PORTB, r16            ; Write PORTB

Und sollen offensichtlich durch Betätigung eines der beiden Taster (egal 
welches) *aus*-geschaltet werden.

Nur leider ist der "Sense" für beide falsch konfiguriert. Die eine ist 
für LowLevel sensualisiert, die andere für steigende Flanke. Also keine 
von beiden für die eigentlich gewünschte fallende Flanke.

Nichtsdestotrotz müßte natürlich der versehentlich für LowLevel 
konfigurierte nach der fallenden Flanke trotzdem reagieren. Der 
Low-Pegel muss halt nur lange genug anliegen, was bei manueller 
Betätigung fast unvermeidbar sein dürfte...

Und der andere müßte eigentlich auch (spätestens beim Loslassen) 
reagieren.

Da dürften neben dem kleinen Programmfehler noch andere Fehler lauern. 
Vermutlich bei der Beherrschung der Toolchain...

[Martin (Gast)]

Hi c-hater,
was genau meinst du mit der Beherrschung der Toolchain?
Ja da habe ich wohl einen Fehler mit der Einstellung der Fallenden 
Flanke gemacht. Aber eigentlich sollte der Interrupt trotzdem reagieren.

Gruß Martin

[Marc V. (Firma: Vescomp) (logarithmus)]

c-hater schrieb:
> Nur leider ist der "Sense" für beide falsch konfiguriert. Die eine ist
> für LowLevel sensualisiert, die andere für steigende Flanke. Also keine
> von beiden für die eigentlich gewünschte fallende Flanke.

 Ja.
 Richtig wäre es:

  ldi r16, (1<<ISC11) | (1<<ISC01)   ; INT0 and INT1 - falling edge 
  sts EICRA, r16                     ; SET EICRA

 Trotzdem hätte es zumindest bei einem funktioniren sollen.

Martin schrieb:
> Meine LED an Port DDRB geht an, jedoch kann ich beide Taster an den
> Externen Interrupts drücken und die LED geht nicht mehr aus.

 LEDs werden ublicherweise andersherum angeschlossen, ANODE uber 2K2 an
 VCC, Kathode an PORT.

 Taster an Masse und das andere Ende ist wo ?

[c-hater (Gast)]

Martin schrieb:

> Ja da habe ich wohl einen Fehler mit der Einstellung der Fallenden
> Flanke gemacht. Aber eigentlich sollte der Interrupt trotzdem reagieren.

Das kommt auf die Schaltung an. Auch da kann man viel falsch machen. 
Also: Schaltplan posten.

Übrigens: was passiert eigentlich, wenn du die eine "aktive" Zeile in 
beiden ISRs auskommentierst? LED an oder aus?

Und was passiert, wenn du dazu auch noch die zweite dieser beiden Zeilen 
auskommentierst?

>  ldi r16, 0xFF            ; Set all LEDs on PORTB high
>    out PORTB, r16            ; Write PORTB

LED an oder aus?

[Martin (Gast)]

Hallo,

c-hater schrieb:
> was passiert eigentlich, wenn du die eine "aktive" Zeile in
> beiden ISRs auskommentierst?

Leds sind immer noch an

c-hater schrieb:
> Und was passiert, wenn du dazu auch noch die zweite dieser beiden Zeilen
> auskommentierst?

Dann ist die LED aus.

Zum Schaltplan:
Led ist die LED direkt am Port 13 des Arduinos welche auf dem Board 
eingelötet ist.

Meine Taster sind an 5V und INT0 und INT1 angeschlossen. Damit sie nicht 
in der Luft "hängen" habe ich sie mit mit einem 10K Widerstand auf GND 
gezogen.

Gruß Martin

[Marc V. (Firma: Vescomp) (logarithmus)]

Martin schrieb:
> Meine Taster sind an 5V und INT0 und INT1 angeschlossen. Damit sie nicht
> in der Luft "hängen" habe ich sie mit mit einem 10K Widerstand auf GND
> gezogen.

 LOL.
 Umdrehen.

[Martin (Gast)]

Hallo,
Umdrehen hat auch nichts gebracht...
Hat jemand noch eine Idee?
Kann es auch etwas mit dem Botloader des Arduinos zu tun haben? Dass die 
Interrupt Einsprungadressen andere sind?

Martin

[Marc V. (Firma: Vescomp) (logarithmus)]

Martin schrieb:
> Kann es auch etwas mit dem Botloader des Arduinos zu tun haben? Dass die
> Interrupt Einsprungadressen andere sind?
 Nein, aber beim m328 springst du in der InterruptTabelle normalerweise
 nicht mit rjmp, sondern mit jmp.

> Umdrehen hat auch nichts gebracht...
 Das kann nicht sein.

> Hat jemand noch eine Idee?
 Ja.
 Das was du geschrieben hast ist nicht das, was du gemacht hast.

[c-hater (Gast)]

Martin schrieb:

> Meine Taster sind an 5V und INT0 und INT1 angeschlossen. Damit sie nicht
> in der Luft "hängen" habe ich sie mit mit einem 10K Widerstand auf GND
> gezogen.

Das hieße, die fallende Flanke tritt auf, wenn man die Taster losläßt, 
nicht, wenn man sie drückt. Also sinnvollerweise entweder die 
Tasterschaltung "umdrehen" oder die INTs für steigende Flanke 
"sensibilisieren". Tasterschaltung umdrehen hätte den Vorteil, dass man 
die internen Pullups nutzen kann und damit zwei Widerstände spart.

Ansonsten: Nach deinen Beobachtungen ist die Ansteuerung der LEDs 
korrekt. Bleibt also die Ansteuerung der Tasten. An welchen Eingängen 
hängen die denn physisch? (Pin-Nummer des ATMega?)

Den Mega328P gibt es in vier Gehäusevarianten, die nicht weniger als 
drei verschiedene Zuordnungen der INT-Pins zu den Pin-Nummern haben:

Package INT0-Pin INT1-Pin
TQFP32  32       1
MLF32   32       1
MLF28   28       1
PDIP28  4        5

[spess53 (Gast)]

Hi

>  ldi r17, (1 << INT0) | (1 << INT1)  ; Enable INT0/INT1 interrupt
>  sts EIMSK, r17            ; Set EIMSK

EIMSK liegt auf IO-Adresse 0x3D. Da muss mit in/out zugegriffen werden. 
Mit STS greifst du ins Leere.

MfG Spess

[spess53 (Gast)]

Hi

>EIMSK liegt auf IO-Adresse 0x3D.

Gemeint war natürlich 0x1D.

MfG Spess

[Marc V. (Firma: Vescomp) (logarithmus)]

spess53 schrieb:
> EIMSK liegt auf IO-Adresse 0x3D. Da muss mit in/out zugegriffen werden.
> Mit STS greifst du ins Leere.

 LOL.
 Man kann mit allem rechnen, nur nicht damit, dass die Fragesteller die
 entsprechenden Datasheets zuerst lesen.
 Wozu auch ?
 Irgendeiner wird es schon tun...

[Martin (Gast)]

Hallo,

spess53 schrieb:
> EIMSK liegt auf IO-Adresse 0x3D. Da muss mit in/out zugegriffen werden.
> Mit STS greifst du ins Leere.

Ja das war das Problem, vielen dank.