Wieso funktioniert mein Programm nicht? (Led ein-/aus -schalten)

Leie Jonas Gast 23.12.2018 18:31 #5669560

Lesenswert?

•

Ich möchte eine Led mittels Taster ein bzw. ausschalten.
Diese Fehler sind zu beobachten: Die Led leuchtet nicht, auch nicht nach 
den Knopfdruck. An der Delay5ms Funktion liegt es nicht!

.equ XTAL = 16000000 ;16 mHz
.equ BTN  = 2 ;Pinnumber of PORT_D
.equ LED  = 3 ;Pinnumber of PORT_D
.def temp1 = r16
.def temp2 = r17
  ;Init stackpointer
  ldi temp1, HIGH(RAMEND)
  out SPH,   temp1
  ldi temp1, LOW(RAMEND)
  out SPL,   temp1
  ;Set led pin to output
  ldi temp1, (1 << LED)
  out DDRD, temp1
  in    temp1, PIND
  sbrc  temp1, (1 << BTN) ;Check if button is pressed
  rcall led_toggle
  rjmp  loop
led_toggle:
  ldi   temp2, (1 << LED)  
  eor   temp1, temp2 ;xor the led pin
  out   PORTD, temp1
  ldi   temp2, 0x64  ;Delay 5ms * 100
led_toggle_0:
  dec   temp2
  rcall delay5ms
  brcc  led_toggle_0

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden

Tipp Gast 23.12.2018 18:46 #5669582

Lesenswert?

•

▲
▼

Warum asm? Schreibs in C.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Felix Gast 23.12.2018 18:49 #5669586

Lesenswert?

•

▲
▼

Kein Pullup? Im Programm wird der interne Pullup nicht genutzt.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

M.K. B. (mkbit) 23.12.2018 19:04 #5669607

Lesenswert?

•

▲
▼

Es kann an der falschen Beschaltung oder Programmierung des Tasters oder 
der LED liegen. Versuch doch erstmal, ob die LED ohne Taster direkt 
leuchten kann.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

S. Landolt Gast 23.12.2018 19:10 #5669618

Lesenswert?

•

▲
▼

> An der Delay5ms Funktion liegt es nicht!

Ich würde trotzdem gerne sehen, dass in ihr tatsächlich SREG 
gesichert&wiederhergestellt wird.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

S. Landolt Gast 23.12.2018 19:17 #5669623

Lesenswert?

•

▲
▼

PS:
Die Befehlsreihenfolge nach §led_toggle_0 ist nämlich absolut unüblich; 
und ich vergaß: dec setzt ohnehin das Carry nicht.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

S. Landolt Gast 23.12.2018 19:28 #5669637

Lesenswert?

•

▲
▼

PPS:
>   sbrc  temp1, (1 << BTN) ;Check if button is pressed
ist auch falsch:
  sbrc  temp1, BTN ;Check if button is pressed

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Sascha W. (sascha-w) 23.12.2018 19:29 #5669639

Lesenswert?

•

▲
▼

S. Landolt schrieb:
> PS:
> Die Befehlsreihenfolge nach §led_toggle_0 ist nämlich absolut unüblich;
> und ich vergaß: dec setzt ohnehin das Carry nicht.
Und mit Sicherheit wird Delay5ms die Flags auch ändern

@ Leie Jonas
also wenn dann so ...

led_toggle_0:
  rcall delay5ms
  dec   temp2
  brne  led_toggle_0


Sascha

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ben B. (stromkraft) (Firma: Funkenflug Industries) 23.12.2018 20:54 #5669743

Lesenswert?

•

▲
▼

> Warum asm? Schreibs in C.
Warum nutzt Du C? Schreib doch alles in Assembler.
Oder noch besser: Bau es in Hardware!

Edit:

Schalte den Taster vom AVR-Pin nach GND und aktive den Pullup:
- Bit im DDR-Register auf Eingang
- Bit PORT-Register auf 1.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Leie Jonas Gast 23.12.2018 21:07 #5669752

Lesenswert?

•

▲
▼

S. Landolt schrieb:
> PPS:
>>   sbrc  temp1, (1 << BTN) ;Check if button is pressed
> ist auch falsch:
>   sbrc  temp1, BTN ;Check if button is pressed

Habe dein Beitrag leider zu schnell überflogen.
Mir ist es erst hinterher beim mehrfachem Debuggen aufgefallen, dass man 
keine Bitmaske übergibt.
Trotzdem danke, dass war halt das einzige Problem.

Ben B. schrieb:
> Oder noch besser: Bau es in Hardware!

hahaha ich kann nicht mehr, immer diese Trolls

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Leie Jonas Gast 23.12.2018 21:11 #5669755

Lesenswert?

•

▲
▼

Nachtrag:
Auch wenn das Programm jetzt fehlerfrei ist, so richtig scheint das dann 
doch nicht zu funktionieren.
Etwa bei jedem 3 Drücken der Taste schaltet die Led viel zu schnell.
Ich habe es auch mit längeren Delays probiert aber, hilft nur teilweise.
Egal wie kurz ich auf den Taster drücke, es ändert nichts an dem 
Problem.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ben B. (stromkraft) (Firma: Funkenflug Industries) 23.12.2018 21:14 #5669758

Lesenswert?

•

▲
▼

Kannst Du bitte das Programm nochmal posten wie Du es jetzt hast?

Ist der Fehler immer periodisch, also immer beim 3. mal aktivieren?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Leie Jonas Gast 23.12.2018 21:36 #5669775

Lesenswert?

•

▲
▼

Ben B. schrieb:
> Kannst Du bitte das Programm nochmal posten wie Du es jetzt hast?
>
> Ist der Fehler immer periodisch, also immer beim 3. mal aktivieren?

Ja gerne.
https://pastebin.com/yJTW9LSx

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ben B. (stromkraft) (Firma: Funkenflug Industries) 23.12.2018 21:44 #5669784

Lesenswert?

•

▲
▼

Zeile 32 und 33 vertauschen

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Heinz R. Gast 23.12.2018 22:03 #5669815

Lesenswert?

•

▲
▼

S. Landolt schrieb:
> An der Delay5ms Funktion liegt es nicht!

>>Ich würde trotzdem gerne sehen, dass in ihr tatsächlich SREG
gesichert&wiederhergestellt wird.

Denk auch darüber nochmal nach.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ben B. (stromkraft) (Firma: Funkenflug Industries) 23.12.2018 23:09 #5669881

Lesenswert?

•

▲
▼

In den meisten Funktionen macht es keinen Sinn, SREG zu sichern. Man 
führt eigentlich keine Funktionen zwischen Vergleich und Sprungbefehl 
oder wo immer man die Flags braucht aus.

Wo das aber zwingend erforderlich ist, sind Interruptfunktionen. Die 
können zu jeder Zeit aufgerufen werden ohne daß man weiß wo, also auch 
mitten in Vergleichen oder Rechen-/BitShift-Operationen mit Carry-Flag. 
Das führt dann zu sehr komischen Fehlern.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Leie Jonas Gast 23.12.2018 23:24 #5669905

Lesenswert?

•

▲
▼

Ben B. schrieb:
> Zeile 32 und 33 vertauschen

Thats it
Die SREG abzusichern, darauf wäre ich nie gekommen.
Zum Glück bin ich diesem Fehler heute begegnet.

Heinz R. schrieb:
> S. Landolt schrieb:
>> An der Delay5ms Funktion liegt es nicht!
>
>>>Ich würde trotzdem gerne sehen, dass in ihr tatsächlich SREG
> gesichert&wiederhergestellt wird.
>
> Denk auch darüber nochmal nach.

Als ich SREG las, dachte ich mir erstmal nichts dabei, weil ich es für 
einen normalen GP-Register gehalten habe. Danke dass du mich darauf 
aufmerksam gemacht hast

Ben B. schrieb:
> Wo das aber zwingend erforderlich ist, sind Interruptfunktionen.

Werde ich mir merken

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ben B. (stromkraft) (Firma: Funkenflug Industries) 24.12.2018 00:07 #5669958

Lesenswert?

•

▲
▼

So schwer ist's aber nicht

push r16
in r16, SREG
push r16
[...]
pop r16
out SREG,r16
pop r16
ret oder reti

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Tipp Gast 24.12.2018 00:09 #5669961

Lesenswert?

•

▲
▼

Ben B. schrieb:
> Warum nutzt Du C? Schreib doch alles in Assembler.
> Oder noch besser: Bau es in Hardware!

Fährst du wie fred Feuerstein Auto? Die Erde dreht sich weiter, auch 
wenn du es nicht merkst!

Antwort Zitieren Bearbeiten Löschen Beitrag melden

J. T. (chaoskind) 24.12.2018 22:21 #5670688

Lesenswert?

•

▲
▼

Tipp schrieb:
> Ben B. schrieb:
>> Warum nutzt Du C? Schreib doch alles in Assembler.
>> Oder noch besser: Bau es in Hardware!
>
> Fährst du wie fred Feuerstein Auto? Die Erde dreht sich weiter, auch
> wenn du es nicht merkst

Wer asm kann, kennt seinen Controller vernünftig ;-).

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Wieso funktioniert mein Programm nicht? (Led ein-/aus -schalten)

Antwort schreiben