C167 Interrupt programm

[Birgit Egger (Gast)]

Hi,

ich habe folgendes Programm unten in meinem C167 eingespielt.
Leider funktioniert nur das blinken der LED (Port 2.0) aber nicht das 
einschalten der Led mit dem Taster in der Endlosschleife.

Geht das nicht oder hab ich da was falsch gemacht. Normalerweise müsste 
ja die Endlosschleife ausgeführt werden und wenn ein Interrupt 
daherkommt dann wird die Methode viIsrTmr3 ausgeführt.
Lieg ich da falsch bzw. was mach ich da falsch?

Vielen Dank!!
lg
Birgit

#include <reg167.h>                    /* include 80C167 header file          */

sbit P2_9 = P2^9;                      /* declare portpin to use              */
sbit DP2_9 = DP2^9;                    /* declare direction bit to use        */
sbit P2_10 = P2^10;                      /* declare portpin to use              */
sbit DP2_10 = DP2^10;                    /* declare direction bit to use        */

void Init(void) {
  /// ------------ Timer 3 Control Register ----------
  /// timer 3 works in timer mode
  /// prescaler factor is 512
  /// up/down control bit is reset
  /// external up/down control is disabled
  /// alternate output (toggle T3OUT) function is disabled
  T3CON = 0x0006;
  T3 = 0x5F00; // load timer 3 register
  /// enable timer 3 interrupt
  /// timer 3 interrupt priority level(ILVL) = 1
  /// timer 3 interrupt group level (GLVL) = 0
  T3IC = 0x0044;
  T3R = 1; // set timer 3 run bit
  // Initialize port 2
  DP2 = 0x0001;
  P2 = 0x0000;
  }

  void GT1_viIsrTmr3(void) interrupt T3INT = 0x23 {
  P2 = ~P2;
  }

void main (void)  {

  unsigned char neuerZustand = 0;
    unsigned char alterZustand = 0;

    DP2_9 = 1;                                 // DP 2_9 is an output for LED           
    DP2_10 = 0;                       // DP 2_10 is an input for the KEY 

  Init();
  IEN = 1; // GLOBALLY ENABLE INTERRUPTS

    while (1) {                          
    neuerZustand = !P2_10;            
    if(!alterZustand && neuerZustand) {      // Flanke des Tastendrucks erkennen
        P2_9 = !P2_9;                // LED an- oder ausmachen
      }
    alterZustand=neuerZustand;          // alten/aktuellen Zustand speichern
    }
}

[Birgit Egger (Gast)]

mir würde es auch schon reichen, wenn jemand eine Aussage treffen 
könnte, ob das Programm funktioniert oder ob ich da was grundlegendes 
falsch gemacht habe.
Vielen Dank,
Birgit

[Jörg S. (joerg-s)]

Ich würde erst mal schauen ob Taster und LED grundsätzlich funktionieren

if (P2_10) P2_9 = 1;
else       P2_9 = 0;

Damit solle bei gedrückter Taste die LED leuchten bzw. ausgehen, je 
nachdem wie sie angeschlossen ist.
Wenn das schon nicht geht, wird wohl irgendwas an der Hardware ein 
Problem sein. Wie ist denn der Taster angeschlossen, mit Pull-Up? Wie 
sieht die LED Schaltung aus, mit Transistor gegen Masse geschaltet?

Bei deinem Programm könnte evt. noch das Prellen ein Problem sein. 
möglicherweise funktioniert es deshalb nicht.

[Birgit Egger (Gast)]

Vielen Dank mal für die Antwort,
was ich nicht versteh ist, es funktioniert das Einschalten der LED mit 
dem Taster (ohne Code für den Interrupt) und es funktioniert das 
Interrupt handling ohne den Code für das einschalten der LED mit dem 
Taster aber beides gemeinsam funktioniert nicht - das versteh ich nicht.

Ich hab beim Interrupt "interrupt T3INT = 0x23" hinzugefügt - das ist 
von mir - vielleicht liegt es an dem?

Oder was kann ich da sonst noch machen?

lg
Birgit

[TManiac (Gast)]

Hallo Birgit,

ich glaube du hast ein paar Kleinigkeiten noch nicht so recht 
verstanden.

void GT1_viIsrTmr3(void) interrupt T3INT = 0x23 
{
  P2 = ~P2;
}

Das ist deine Interrupt-Routine. Mit dem "interrupt T3INT" gibst du an 
das dies die für den Timer 3 Overflow Interrupt ist. Das "=0x23" müsste 
doch einen Compiler-Fehler produzieren?

// Initialize port 2
  DP2 = 0x0001;
  P2 = 0x0000;

Damit hast du nur einen einzigen Ausgangs Pin am Port 2. Und zwar den am 
P2_0.

Die Zeilen

DP2_9 = 1;    // DP 2_9 is an output for LED           
DP2_10 = 0;  // DP 2_10 is an input for the KEY

sind recht sinnlos, da du danach in der Init() Funktion die zuvor 
aufgeführten Zeilen ausführst.

Ändere also deine Init()-Routine so ab das dort deine vollständige 
Initialisierung geschieht und schmeiße die anderen Zeile raus (immer 
logischen Code schreiben!!!). Zum Beispiel so:

// Initialize port 2
  DP2 = 0x0201;   // DP2_0 und DP2_9 sind ausgangspins, alle anderen sind eingang
  P2 = 0x0000;

Dein (vielleicht gewolltes) Programm sieht also so aus:
Der Controller invertiert im Hintergrund, gesteuert durch einen 
Interrupt, den gesamten Port 2. Das heißt das auf Grund deiner 
Port-Initialisierung zu beginn nur der Pin 0 des Ports 2 blinkt.

Wird dann der Taster an P2_10 gedrückt, wird der Zustand an P2_9 
invertiert. Wenn alles Gut geht, müsste also danach die LED entgegen der 
anderen Blinken.

Ich selber arbeite mit dem Tasking Compiler anstatt dem Keil, daher mag 
man mir verzeihen wenn ich mich an manchen Stellen geirrt habe.

Gruß,
TManiac

[Birgit Egger (Gast)]

Vielen Dank für die hilfreichen Informationen,

void GT1_viIsrTmr3(void) interrupt T3INT = 0x23 
{
  P2 = ~P2;
}

Das ist deine Interrupt-Routine. Mit dem "interrupt T3INT" gibst du an 
das dies die für den Timer 3 Overflow Interrupt ist. Das "=0x23" müsste 
doch einen Compiler-Fehler produzieren?

wenn ich das = 0x23 weglasse dann bekomm ich einen Fehler, deshalb hab 
ich das dazugeschrieben - das hab ich auch nicht verstanden, was der 
Compiler da nicht mag.
Ich werd das heute Abend mal ausprobieren.
lg und vielen Dank,
Birgit