PIC16F88 - Taster-Eingabe wird nicht erkannt

[wottis (Gast)]

Hallo miteinander!

Ich möchte mich gerne mit dem PIC16F88 vertraut machen und dessen 
Programmierung in C und ASM.
In einem ersten Laborversuch hab ich einen Lottozahlengenerator mit 
Flowcode erstellen müssen mit dem passenden Schaltplan (im Anhang). Das 
funktioniert auch alles soweit.
Jedoch soll das Ganze auch in ASM u. C geschrieben werden.
Was ich also zur Zeit versuche ist das bei Betätigen des Tasters SEG1 
(obere 7-Segment-Anzeige im Schaltplan) aus geht und SEG2 an. Nun ist 
aber das Problem, dass die Änderung an Port A2 nicht erkannt wird. 
Weiterhin ist bei Start des Programms button = 0 anstelle von 1, weil 
dieser ja egtl an einem Pullup hängt.
Die Verbindung mit dem Taster auf meinem PIC-Demo Board stimmt auch bei 
Betätigung zieht er die Spannung an A2 auf 0V aber es wird vom Programm 
nicht erkannt.

Meine Frage ist also wieso ist button = 0 zu Beginn des Progammstarts 
und warum wird die Änderung an Port A2 nicht erkannt.
Habe schon viel gelesen und versucht und finde einfach nicht mein 
Fehler. Die Segmente an ausschalten ist ja kein Problem aber der Taster 
;(.

Das Programm und der Schaltplan befinden sich im Anhang.


MFG

wottis

/* Präprozessor */
/* ############ */
#include "htc.h"
#include "pic.h"
#include "stdlib.h"
#include "delay.c"
#define SEG1 RA1
#define SEG2 RA0
#define Taster RA2

/* Konfiguration des PCIs
 *
 * WDTE_OFF                 - Watchdogtimer off
 * PWRTE_ON                 - Power-on Timer on
 * FOSC_INTOSCIO            - Interner Taktgeber 8MHz
 * MCLRE_OFF                - Master Clear off
 */

__CONFIG(WDTE_OFF & PWRTE_ON & FOSC_INTOSCIO & MCLRE_OFF & LVP_OFF);


/* Unterfunktionen deklarieren */
/* ########################### */
void init();                            // initialisiere SFR's
void check();                           // überprüft RA2 und speichert Status inn button

unsigned char button;

int main(void)
{
    init();
    while(1)                            // Mainloop
    {
        check();

        while(button == 1)              // wenn Taster nicht betätigt wurde
        {
            SEG1 = 0;                   // SEG1 soll leuchten
            RA0 = 1;
            PORTB = 0b10010001;
            check();                    // prüfe RA2
        }
        while(button == 0)              // wennn Taster betätigt wurde
        {
            SEG2 = 0;                   // SEG2 soll leuchten
            SEG1 = 1;
            PORTB = 0x10;
            check();                    // prüfe RA2
        }
    }

    return 0;
}


void init()
{
    OSCCON = 118;                 // 8MHz interner Takt
    TRISB = 0x00;                 // PortB auf Ausgang
    TRISA = 0b00000100;           // A2 als Eingang und Rest Ausgang
    PORTB = 0x00;                 // clear PortB
    PORTA = 0x00;                 // clear PortA
    SEG1 = 1;                     // Segment 1 u. 2 aus
    SEG2 = 1;
}

void check()
{
    button = RA2;
    DelayMs(100);
}

[frozen eye (Gast)]

Den Port RA kennt Dein Compiler vermutlich nicht.
Versuch mal PORTA2 oder PORTA.2

In ASM wäre es PORTA, 2

[wottis (Gast)]

Die Ports kennt er schon, weil die hab ich aus dem entsprechenden 
Header-File und für RA0 u. RA1 funktioniert es ja auch.
Was noch eine Variante ist PORTAbits.RA2 aber das bringt auch nichts.

[wottis (Gast)]

OK nach dem ich das Debbuging mit dem PICKit3 zum laufen gebracht habe 
und nochmal die PORT Funktionen angeschaut habe konnte ich das Problem 
beheben.

PortA war noch auf Analog I/O  eingestellt nach dem clearen des ANSEL 
Registers funktioniert es nun.

[tt2t (Gast)]

Der PIC16F88 ist der Nachfolger des PIC-Urahns 16F84 und hat selbst 
schon wieder einige Jahre auf dem Buckel. Der Nachfolger zum 16F88 ist 
der 16F1847, der ist pin- und funktionskompatibel, hat mehr und 
schnellere Hardware, hat XLP, ist für "C" optimiert  und kostet auch 
noch wesentlich weniger.

[wottis (Gast)]

Danke für den Hinweis tt2t.
Muss mich aber bisschen mit dem beschäftigen, weil wir an der Hochschule 
damit arbeiten.

[tt4u (Gast)]

Man sollte auch noch die Komparatoren abschalten:

     movlw   b'00000111'
     movwf   CMCON           ; Comparators off