Tastendruck wird nicht erkannt

[Markus N. (k3hlim)]

Hallo Leute,

ich bin zur Zeit mit µCs am experimentieren und noch ziemlich am Anfang. 
Zur Tastenentprellung habe ich den Code von 
'http://rn-wissen.de/wiki/index.php/Taster-Abfrage_in_C'; verwendet und 
soweit es mein Kentnissstand vermag angepasst.

Die Taster schalten in Abhängigkeit vom Zustand (an/aus) und evtl. noch 
einer weiteren Bedingung Relais über Transitoren.

Wenn ich die Taster über if-Abfragen und _delay-Verzögerung abfrage, 
werden die Ausgänge auch geschaltet, nur mit dem angegebenen Code 
funktioniert nicht alles, soll heißen, case 2 reagiert, der Rest jedoch 
nicht.

Hardware habe ich schon überprüft und scheint in Ordnung zu sein.

#define HAUPTPROGRAMM

#include "Deklaration.h"
#include "taster.h"


//  Tasterabfrage

/* 
 * Die Taster-Ports sind hartcodiert:
 * #0 --> PortC.2   (Taster Beleuchtung Radio)
 * #1 --> PortC.3   (Taster Beleuchtung Endstufe)
 * #2 --> PortC.4   (Taster Trennrelais)
 * #3 --> PortC.5   (Taster Anlage)
 */

/* Bei 8MHz Grundtakt läuft Timer0 alle 32µs über.
 * Um auf rund 10ms zu kommen, rufen wir get_taster nur
 * jedes 313. mal auf. */
SIGNAL (SIG_OVERFLOW0)
{
    static unsigned int count_ovl0;
    unsigned int ovl0 = count_ovl0+1;

    if (ovl0 >= 313)
    {
        get_taster (0, PINC & (1<<PC2));
        get_taster (1, PINC & (1<<PC3));
        get_taster (2, PINC & (1<<PC4));
        get_taster (3, PINC & (1<<PC5));
        
        ovl0 = 0;
    }

    count_ovl0 = ovl0;
}

void ioinit()
{
    /* Taster sind Input (default nach RESET) */
    /* Bei LOW-aktiven an den Tastern die PullUps aktivieren, bei HIGH-aktiven nicht */
    #if TASTER_LEVEL
        ;
    #else
        PORTC |= 1 << PC2;
        PORTC |= 1 << PC3;
        PORTC |= 1 << PC4;
        PORTC |= 1 << PC5;
    #endif

    /* Timer0 ohne Prescaler starten */
    TCCR0 |= (0<<CS02) | (0<<CS01) | (1<<CS00);
 
    /* Timer0-Overflow-Interrupt aktivieren */
    TIMSK |= (1<<TOIE0);
}

int main (void) {            // Beginn des Programms

 DDRB   |=  (1<<PB0) | (1<<PB1) | (1<<PB2) | (1<<PB3) | (1<<PB4) | (1<<PB5); 
 PORTB  |=  (0<<PB0) | (0<<PB1) | (0<<PB2) | (0<<PB3) | (0<<PB4) | (0<<PB5); 

 DDRC   |=  (0<<PC0) | (0<<PC1) | (0<<PC2) | (0<<PC3) | (0<<PC4) | (0<<PC5);
 PORTC  |=  (1<<PC0) | (1<<PC1) | (1<<PC2) | (1<<PC3) | (1<<PC4) | (1<<PC5); 

 DDRD   |=  (0<<PD0) | (0<<PD1) | (0<<PD2);
 PORTD  |=  (1<<PD0) | (1<<PD1) | (1<<PD2);

// Interrupts freigeben
 
    sei();
 
// Variablen initialisieren

    St.Anlage  =  0;
    St.Licht   =  0;


    ioinit();
 
    /* Taster konfigurieren (#define NUM_TASTER 4 in taster.h) */
    tasten[0].mode = TM_SHORT;
    tasten[1].mode = TM_SHORT;
    tasten[2].mode = TM_SHORT;
    tasten[3].mode = TM_SHORT;


// Endlose Hauptschleife
 
    while (1)
     {

    signed char tast = taster;


    switch (tast)
     {

       default:

       case NO_TASTER: break;

       case 0:
        {
          if (St.Anlage == 1)
           {
             if (St.Licht == 0)
              {
                PORTB  |=  (1<<PB5);
                St.Licht = 1;
              }
             else
              {
                PORTB  &= ~(1<<PB5);
                St.Licht = 0;
              }
           }
          
          else
           {
             PORTB  &= ~(1<<PB5);
             St.Licht = 0;
           }
        }
        break;

       case 1:
        {
        }
        break;

       case 2:
        {
          if (St.Relais == 0)
           {
             if (bit_is_clear (PIND,0))
              {
                PORTB  |=  (1<<PB4);
                St.Relais  =  1;
              }
           }
        }
        break;

       case 3:
        {
          if (St.Anlage == 0)
           {
             PORTB  |=  (1<<PB0);
             St.Anlage = 1;
           }
          else
           {
             PORTB  &=  ~(1<<PB0);
             St.Anlage = 0;
           }
        }
        break;
     }

     if (tast != NO_TASTER)
         taster = NO_TASTER;


// Steuerung Trennrelais
      
    if (St.Relais == 0)
     {
       if (bit_is_clear (PIND,1))
        {
          _delay_ms (1000);
          if (bit_is_clear (PIND,1))
           {
             PORTB  |=  (1<<PB4);
             St.Relais  =  1;
           }
        }
     }

    if (St.Relais == 1)
     {
       if (bit_is_set (PIND,1) && bit_is_set (PIND,0))
        {
          _delay_ms (100);
          if (bit_is_set (PIND,1) && bit_is_set (PIND,0))
           {
             PORTB  &= ~(1<<PB4);
             St.Relais  =  0;
           }
        }
     }


// Abfrage Lichtschalter

     if (bit_is_clear (PIND,2))
      {
        _delay_ms (100);
        if (bit_is_clear (PIND,2))
         {
           St.Licht  =  1;
         }
      }

     if (bit_is_set (PIND,2))
      {
        _delay_ms (100);
        if (bit_is_set (PIND,2))
         {
           St.Licht  =  0;
         }
      }   
     

     }                                              // Ende Hauptschleife

}                                                   // Ende main()

[Mark R. (stevestrong)]

Bitte die Funktionalität der zwei Variabeln "St.Relais" und "St.Anlage" 
prüfen.

[Markus N. (k3hlim)]

Die Variablen funktionieren ja, wenn ich anstatt der Tastenentprellung 
die Entprellung nur über if-Abfrage und _delay mache, ebenfalls mit den 
Variablen als Status ob gerade aktiv oder inakiv, werden die 
entsprechenden Ausgänge ja geschaltet.

Also als Beispiel für case3:

if (bit_is_clear (PINC,5))
 {
  _delay_ms (10);
  if (bit_is_clear (PINC,5))
   {
    if (St.Anlage == 0)
     {
      PORTB  |=  (1<<PB0);
      St.Anlage = 1;
     }
    else
     {
      PORTB  &= ~(1<<PB0);
      ST.Anlage = 0;
     }
   }
 }

Wenn ich das so mache, wird PB0 geschaltet, allerdings würde ich die 
Tasterabfrage gerne mit dem anderen Code realisieren und vor allem 
verstehen, warum eine Abfrage funktioniert und die andere nicht.