Tasterabfrage xmega8E

[hans (Gast)]

Hallo,

ich versuche gerade mit einem XMEGA8E5 µController zwei Taster im 
Pollingbetrieb abzufragen und jenachdem ob taster1 oder taster 2 
gedrückt ist sollen andere Anweisungen erfüllt werden.

Aber es funktioniert nur eine Abfrage von dem Taster an PIN6 die zweite 
wird nicht ausgeführt...


sieht einer den Fehler im c-code und kann mir bitte weiterhelfen?

 void tasten_abfrage (void)
{
  uint8_t tastenwert_vorher = 1,tastenwert1_vorher = 1,tastenwert_jetzt = 0,tastenwert1_jetzt = 0;
  uint8_t anzahl = 0;
  
  while(1)
  {
    tastenwert_jetzt =  PORTA.IN & PIN6_bm; /*aktueller Tasterwert wird eingelesen*/
    tastenwert1_jetzt =  PORTA.IN & PIN7_bm; /* aktueller Tasterwert wird eingelesen */
    _delay_ms(20);
    
    if (tastenwert_jetzt != tastenwert_vorher) /*gibt es einen Unterschied?*/
    {
      if(tastenwert1_jetzt == 1) /* Tastendruck mitzählen */
      {
        anzahl++;
      }
      
      else 
      {
        if(tastenwert1_jetzt != tastenwert1_vorher)
        {
          if(tastenwert1_jetzt == 0)  
          {
            anzahl = 1;
            anzahl++;
          }
      
      
          switch (anzahl)
          {
        
            case 1: PORTC.OUT = 0x18; /*Taste POL 1 mal gedrückt -> POL an*/
            PORTC.OUTCLR = 0x04; /*UV-Beleuchtung aus*/
            PORTD.OUTCLR = PIN7_bm; /*Status-LED grün an*/
            PORTD.OUTSET = PIN6_bm;  /*Status-LED blau aus*/
            break;
        
        
            case 2: PORTC.OUT = 0x04; /*Taste UV 1 mal gedrückt -> UV an*/
            PORTC.OUTCLR = 0x18; /*POL-Beleuchtung aus*/
            PORTD.OUTCLR = PIN6_bm; /*Status-LED blau an*/
            PORTD.OUTSET = PIN7_bm;  /*Status-LED grün aus*/
            break;
      
        
            default: anzahl = 1;
          }
        }
      }
    }
    
    
    tastenwert_vorher=tastenwert_jetzt; /* aktuellen Tasterwert merken und wieder vergleichen */
  }
}

[Daniel S. (daniel_s49)]

sorry, aber der Code ist echt wirr.

1) Dein äußeres if in der while fragt nur Pin6 ab. Wenn sich der Taster 
nicht ändert, dann kannst du an Pin7 so viel drücken wie du willst.

2) Wenn du Pin7 gedrückt hältst und Pin6 ein paar mal drückst, hast du 
Werte in anzahl, die deutlich größer sind als 2 und damit macht dein 
switch keine LED an.

3) Dein switch kann nur erreicht werden, wenn Pin7 nicht gedrückt 
wird. (ich gehe davon aus, dass 1=gedrückt und 0=nicht gedrückt)

4) tastenwert1_vorher wird nie neu gesetzt und ist damit immer 1

[hans (Gast)]

Daniel S. schrieb:
> sorry, aber der Code ist echt wirr.
>
> 1) Dein äußeres if in der while fragt nur Pin6 ab. Wenn sich der Taster
> nicht ändert, dann kannst du an Pin7 so viel drücken wie du willst.
>
> 2) Wenn du Pin7 gedrückt hältst und Pin6 ein paar mal drückst, hast du
> Werte in anzahl, die deutlich größer sind als 2 und damit macht dein
> switch keine LED an.
>
> 3) Dein switch kann nur erreicht werden, wenn Pin7 nicht gedrückt
> wird. (ich gehe davon aus, dass 1=gedrückt und 0=nicht gedrückt)
>
> 4) tastenwert1_vorher wird nie neu gesetzt und ist damit immer 1

okay ich habe das jetzt anders gelöst funktioniert soweit auch nur habe 
ich noch jetzt ein weiteres Problem:

ich frage die Taster im Timer Interrupt mit der Funktion 
tasten_abfrage(); ab

jetzt habe ich jedoch noch eine weitere Funktion laden_DC(); in der 
while-schleife.

möchte ich jetzt beide funktionen benutzen dann geht nicht wirklich 
was...

kommentiere ich eine der beiden funktionen aus gehen alle fuktionen 
wieder.


Woran kann es liegen? Takte ich den µC zu langsam oder habe ich noch 
einen bock in den Funktionen (wobei die ja einzeln funktionieren)?

#include <avr/io.h>
#include <avr/interrupt.h>
#define F_CPU 8000000UL
#include <util/delay.h>

void clock_init (void)
{
  OSC.CTRL |= OSC_RC8MEN_bm; /*Oszillator auf 8 Mhz einstellen*/
  while(!(OSC.STATUS & OSC_RC8MRDY_bm)); /*Warten bis der Oszillator
  bereit ist*/
  CCP = CCP_IOREG_gc; /*Schützt I/O Register, Interrupts werden
  ignoriert*/
  CLK.CTRL = (CLK.CTRL &~ CLK_SCLKSEL_gm) | CLK_SCLKSEL_RC8M_gc;
  /*aktiviert den internen Oszillator*/
}



void port_init (void)
{
  /*Charge_Voltage*/
  PORTA.DIR &= ~PIN1_bm; /*PORTA.1 als Eingang definiert (0,93V-1,4V)*/
  
  /*Charge_Current*/
  PORTA.DIR &= ~PIN0_bm; /*PORTA.0 als Eingang definiert (1.5V = 1.2A)*/
  
  /*Reedkontakt init*/
  PORTC.DIR &= ~ PIN5_bm; /*PORTC.5 auf Eingang Reedkontakt*/
  PORTC.PIN5CTRL |= PORT_OPC_PULLUP_gc; /*Pull-Up Widerstand am PORTC.5 aktiv (Reedkontakt)*/
  
  /*LEDs init*/
  PORTC.OUT &= ~ PIN2_bm; /*PORTC.2 auf low UV-LEDs->aus*/
  PORTC.OUT &= ~ PIN3_bm; /*PORTC.3 auf high WW-LEDs->aus*/
  PORTC.OUT &= ~ PIN4_bm; /*PORTC.4 auf high KW-LEDs->aus*/
  //PORTC.OUT |= PIN3_bm; /*PORTC.3 auf high WW-LEDs->an 100%*/
  //PORTC.OUT |= PIN4_bm; /*PORTC.4 auf high KW-LEDs->an 100%*/
  PORTC.DIR = 0x1C;      /*PORTC auf Ausgang LEDs*/
  
  /*System ON/OFF*/
  PORTC.DIR |= PIN1_bm; /*PORTC.1 auf Ausgang ON/OFF*/
  PORTC.OUT &= ~PIN1_bm; /*PORTC.1 auf low*/
  
  /*Status LED Init*/
  PORTD.DIR |= PIN6_bm; /*LED Blau*/
  PORTD.OUT |= PIN6_bm;
  
  PORTD.DIR |= PIN7_bm; /*LED Grün*/
  PORTD.OUT |= PIN7_bm;
  
  
  PORTR.DIR |= PIN0_bm; /*LED Rot*/
  PORTR.OUT |= PIN0_bm;
  
  /*Charger Fault*/
  PORTD.DIR &= ~PIN5_bm; /*PORTD.5 als Eingang definiert*/
  
  /*DC-Power OK*/
  PORTD.DIR &= ~PIN4_bm; /*PORTD.4 als Eingang definiert*/
  
  /*Charger ON/OFF*/
  PORTD.DIR |= PIN3_bm;
  PORTD.OUT &= ~ PIN3_bm;
  
  /*Charger Indikator*/
  PORTD.DIR &= ~PIN2_bm; /*PORTD.2 als Eingang definiert*/
  
  /*USB-Power OK*/
  PORTD.DIR &= ~PIN1_bm; /*PORTD.1 als Eingang definiert*/
  
  /*Taster Init*/
  PORTA.DIR &= ~PIN7_bm; /*PORTA7 als Eingang definieren (Taster POL)*/
  PORTA.PIN7CTRL |= PORT_OPC_PULLUP_gc; /*Pull-Up Widerstand am PORTA7 aktiv (Taster POL)*/
  
  PORTA.DIR &= ~PIN6_bm; /*PORTA6 als Eingang definieren (Taster UV) */
  PORTA.PIN6CTRL |= PORT_OPC_PULLUP_gc; /*Pull-Up Widerstand am PORTA6 aktiv (Taster UV)*/
  
  //PORTA.DIR &= ~PIN5_bm; /*PORTA5 als Eingang definieren (Taster Reserved)*/
  //PORTA.PIN5CTRL |= PORT_OPC_PULLDOWN_gc; /*Pull-Up Widerstand am PORTA7 aktiv (Taster3)*/
}


void interrupt_init (void)
{
  PMIC.CTRL |= PMIC_LOLVLEN_bm; /*Interrupt eingeschaltet*/
  sei(); /*Interrupt freischalten*/
}



void timer4_init (void)
{
  TCC4.CTRLA = TC45_CLKSEL_DIV256_gc; /*Vorteiler auf 8 eingestellt*/
  TCC4.CTRLB = TC45_WGMODE_NORMAL_gc; /* Timer auf normal Mode (hochzählend) eingestellt */
  TCC4.PER = 65436; /*Timer vorladen 65436*/
  TCC4.INTCTRLA = TC45_OVFINTLVL_LO_gc; /*Interrupt Modus aktiv*/
  TCC4.INTFLAGS = TC4_OVFIF_bm;
}


void tasten_abfrage (void)
{
  uint8_t tasterwert_vorher = 1, tasterwert_jetzt = 0, anzahl = 0;  
  
  while(1)
  {
    tasterwert_jetzt = PORTA.IN & PIN7_bm; /*aktueller Tasterwert wird eingelesen*/  
    _delay_ms(20);
    
    if(!(PORTA.IN & PIN6_bm)) 
    {
      PORTC.OUT = 0x04; /*Taste UV 1 mal gedrückt -> UV an*/
      PORTC.OUTCLR = 0x18; /*POL-Beleuchtung aus*/
      PORTD.OUTCLR = PIN6_bm; /*Status-LED blau an*/
      PORTD.OUTSET = PIN7_bm;  /*Status-LED grün aus*/   
    }
      
    if(tasterwert_jetzt != tasterwert_vorher)  
    {
      if(tasterwert_jetzt == 0)  
      {
        anzahl++;
      }
      
      switch (anzahl)
      {
        case 1: PORTC.OUT = 0x18; /*Taste POL 1 mal gedrückt -> POL an*/
            PORTC.OUTCLR = 0x04; /*UV-Beleuchtung aus*/
            PORTD.OUTCLR = PIN7_bm; /*Status-LED grün an*/
            PORTD.OUTSET = PIN6_bm;  /*Status-LED blau aus*/             
            break;
            
            
        case 2: PORTC.OUTCLR = 0x18; /*POL-Beleuchtung aus*/
            PORTD.OUTSET = PIN7_bm;  /*Status-LED grün aus*/
            PORTD.OUTCLR = PIN6_bm; /*Status-LED blau an*/            
            break;
            
            default: anzahl = 0;
      }
    }
    tasterwert_vorher=tasterwert_jetzt; /* aktuellen Tasterwert merken und wieder vergleichen */  
  }
}



void laden_DC (void)
{
  
  
  if((PORTC.IN & PIN5_bm) && (!(PORTD.IN & PIN4_bm)) && (!(PORTD.IN & PIN2_bm))) /*Reedkontakt OK (high) & DC-PWR OK (low) & Charge Indicator (low)*/
  {
    PORTC.OUTCLR = 0x1C;
    _delay_ms(300);
    PORTD.OUTTGL = 0x80; /*grüne Status LED toggeln*/
  }
  else
  {
    if(PORTD.IN & PIN2_bm) /*Charge Indicator (high)*/
    {
      PORTC.OUTCLR = 0x1C;
      PORTD.OUT &= ~ PIN5_bm; /*grüne Status LED an*/
    }
  }
}

int main(void)
{
    
  clock_init ();
  port_init();
  timer4_init();
  interrupt_init();  
  
  while(1)
    {
    laden_DC();           
    }
}


ISR(TCC4_OVF_vect)
{
  TCC4.INTFLAGS = TC4_OVFIF_bm;
  tasten_abfrage();
  
}

[Daniel S. (daniel_s49)]

ähm ... du startest in der ISR eine Endlosschleife und wunderst dich, 
warum das Programm nicht funktioniert? Könnte dran liegen, dass der 
Mikrocontroller für immer in der tasten_abfrage() hängt, ohne jemals 
wieder in die lade_DC() zurück zu kommen.
Normalerweise sollte man ISRs so kurz wie möglich halten. Schleifen und 
alles was Zeit kostet, haben da drin nix verloren.

[hans (Gast)]

Daniel S. schrieb:
> ähm ... du startest in der ISR eine Endlosschleife und wunderst
> dich,
> warum das Programm nicht funktioniert? Könnte dran liegen, dass der
> Mikrocontroller für immer in der tasten_abfrage() hängt, ohne jemals
> wieder in die lade_DC() zurück zu kommen.
> Normalerweise sollte man ISRs so kurz wie möglich halten. Schleifen und
> alles was Zeit kostet, haben da drin nix verloren.

du hast recht sehe ich jetzt auch.

Komme aber im Moment nicht auf die Lösung wie müsste ich das dann mit 
der Abfrage gestalten?

Damit beide Funktion gehen?

[Peter D. (peda)]

hans schrieb:
> Komme aber im Moment nicht auf die Lösung wie müsste ich das dann mit
> der Abfrage gestalten?

Beitrag "Universelle Tastenabfrage"

Für Tasten lohnt sich eine eigene Lib. Der Timerinterrupt bereitet die 
Tasten auf und das Main holt sich dann nur die fertigen Events ab.

Das Vermengen des Entprellen mit der Mainloop macht nur Ärger und den 
Code unübersichtlich.