Programmcounter-Sprungbefehl

[Patrick O. (mrjack)]

Hallo zusammen

Für einen Kollegen bin ich eine Anzeige für seinen Töff am entwickeln. 
Dies zeigt den aktuellen Gang, Liter im Tank und diverse weitere 
Eigentschaften an. Die Visualisierung verfolgt über eine 
7-Segmentanzeige (Ganganzeige sowie Tankanzeige) sowie eine 
Balkenanzeige (einfacher Tankpegel)
Bei der Anzeige der Liter habe ich jedoch noch irgeneinen Fehler drin.

Hier ist der Ausschnitt des Codes für die Literermittelung:

ADREAD          BSF  ADCON0,2      ;Tanklevel einlesen
WAITADC          BTFSC  ADCON0,2
    GOTO  WAITADC
    MOVFW  ADRESH
    MOVWF  ADWERT
    BTFSC  PORTA,2                       ;Wenn PortA 2 high ist Liter anzeigen
    GOTO  LITER
    BTFSS  ADWERT,1
    GOTO  LEVEL1
    GOTO  LEVEL2

LITER          BCF  STATUS,0      ;Carry-Bit löschen
    MOVFW  TANKST
    MOVWF  HILFSBYTE      
    CLRF  ADLOOP        ;Loop-Zahl löschen
LOOP          MOVFW  ADWERT        
    ADDWF  HILFSBYTE,w      ;AD-Wandlerwert ermitteln
    BTFSS  STATUS,0
    GOTO  ANZLITER      ;Falls detektiert anzeigen
    MOVLW  b'00000101'      
    ADDWF  ADLOOP,f      ;Loop 5 dazu addieren
    MOVFW  TANKST
    ADDWF  HILFSBYTE,f      ;nächster Tank-Schritt ermitteln
    GOTO  LOOP

;===========================   
;Tankpegel
;=========================== 

LEVEL1          BCF  PORTB,3
    BSF  PORTB,4
    GOTO  MAIN

LEVEL2          BSF  PORTB,3
    BSF  PORTB,4
    GOTO  MAIN

ANZLITER
    CALL  DISPL
    CALL  DELAY1
    MOVFW  ADLOOP
    ADDWF  PCL,f
LITER0
    CALL  DISP2        ;21Liter
    CALL  DELAY1
    CALL  DISP1
    CALL  DELAY1
    GOTO  MAIN

    CALL  DISP2        ;20Liter
    CALL  DELAY1
    CALL  DISP0
    CALL  DELAY1
    GOTO  MAIN

    CALL  DISP1        ;19Liter
    CALL  DELAY1
    CALL  DISP9
    CALL  DELAY1
    GOTO  MAIN

.......(immmer 5 Programmzeilen im Literschritt bis:

    CALL  DISP_        ;0Liter
    CALL  DELAY1
    CALL  DISP0
    CALL  DELAY1
    GOTO  MAIN

Die Literanzeige verändert sich fast nicht, wenn ich das Potentiometer 
am AD-Eingang maximal verstelle. Die einfache Balkenanzeige (Level1, 
Level2), welche immer angezeigt wird, funktioniert einwandfrei.
Irgendetwas bei der Modifikation des Programmcounters geht schief, 
konnte jedoch nicht herausfinden was...

Gruss

[Purzel H. (hacky)]

Schon mal mit dem Simulator probiert. Da kann man singlesteppen.

[17:15 (Gast)]

Siebzehn Zu Fuenfzehn schrieb:
> Da kann man singlesteppen

...oha, sehr schönes Wort, solltest du mal für den neuen Duden 
einreichen

[Teo D. (teoderix)]

Schon überprüft, ob all die Calls, innerhalb ihrer Reichweite liegen?!
Was ist eigentlich mit PCLATCH?
http://www.sprut.de/electronic/pic/grund/adress.htm#pcl


Nett wäre es auch, wenn Du mitteilen würdest, für welchen Prozessor das 
geschrieben wurde!

[Max H. (hartl192)]

Patrick Oppliger schrieb:
> ADDWF  PCL,f
Das kann leicht schiefgehen, da du bis zu 100 Befehle überspringst. PCL 
ist nur 8bit breit und wenn PCL bei der Addition überläuft hast du dann 
eine Sprung zur falschen Adresse. Falls es ein
Enhanced Mid-range PIC ist würde ich den "BRW" Befehl empfehlen.

[Patrick O. (mrjack)]

Ich verwende den PIC16F818. Habe leider im Moment noch eine steinalte 
Software auf meinem alten XP-PC, den ich für basteleien benötige, und 
die hat irgendwelche Probleme im Simulationsmodus.

Mir kam gerade ein anderer Lösungsansatz in den Sinn, bei welchem ich 
weniger grosse Sprungschritte machen muss. Probiere auch mal das ganze 
mit BRW (danke an Max H.!!)
Werde dies mal testen und mich dann wieder melden.

[Peter D. (peda)]

Patrick Oppliger schrieb:
> .......(immmer 5 Programmzeilen im Literschritt bis:
>
>     CALL  DISP_        ;0Liter
>     CALL  DELAY1
>     CALL  DISP0
>     CALL  DELAY1
>     GOTO  MAIN

Sowas derart umständliches nennt man, die Hose mit der Kneifzange 
anziehen.

Warum schaust Du Dir nicht einfach mal an, wie alle anderen 
Programmierer Zahlen anzeigen?

Zuerst zerlegt man die Zahl in Zehner und Einer (z.B. 10 abziehen bis 
Unterlauf). Und dann greift man damit auf eine Tabelle zu, die den 
7-Segment Code für 0..9 beinhaltet (10 * RETLW).
Die beiden Ergebnisbytes kann man, wenns denn unbedingt sein muß, mit 
Delay ausgeben. Timerinterrupt ist natürlich besser, da flackerfrei.

[Patrick O. (mrjack)]

Peter Dannegger schrieb:
> Patrick Oppliger schrieb:
>> .......(immmer 5 Programmzeilen im Literschritt bis:
>>
>>     CALL  DISP_        ;0Liter
>>     CALL  DELAY1
>>     CALL  DISP0
>>     CALL  DELAY1
>>     GOTO  MAIN
>
> Sowas derart umständliches nennt man, die Hose mit der Kneifzange
> anziehen.
>
> Warum schaust Du Dir nicht einfach mal an, wie alle anderen
> Programmierer Zahlen anzeigen?
>
> Zuerst zerlegt man die Zahl in Zehner und Einer (z.B. 10 abziehen bis
> Unterlauf). Und dann greift man damit auf eine Tabelle zu, die den
> 7-Segment Code für 0..9 beinhaltet (10 * RETLW).
> Die beiden Ergebnisbytes kann man, wenns denn unbedingt sein muß, mit
> Delay ausgeben. Timerinterrupt ist natürlich besser, da flackerfrei.

Ich habe meine 7-Segmentanzeige über 2 Ports verteilen müssen, damit die 
Stromversorgung von 100mA pro Port nicht überschritten wird. Für jede 
Anzeige, sei es ein Zeichen oder eine Zahl habe ich ein Unterprogramm 
mit den nötigen Zeilen geschrieben.

Damit ich das Programm simulieren kann, habe ich nun MPLAB v2.10 
installiert, jedoch kommt jedesmal beim Programmbuild ein Fehler: "DOS 
Error: Fail not found". Hatte dies auch schon jemand zuvor?

[Max H. (hartl192)]

Patrick Oppliger schrieb:
> Probiere auch mal das ganze
> mit BRW (danke an Max H.!!)
Das wird nichts, der PIC16F818 ist zu alt und hat den Befehl nicht.

[Peter D. (peda)]

Patrick Oppliger schrieb:
> Ich habe meine 7-Segmentanzeige über 2 Ports verteilen müssen

Ich mache das auch, wenn sich die Platine dadurch leichter routen läßt.
Ich schalte dann eine Routine dazwischen, die das Ausgabebyte auf die 
jeweiligen Pins verteilt (Bitbefehle).