ATmega 328PB Taster einlesen

[Mans A. (mansmaak)]

Hallo Community,

ich habe folgende Aufgaben bekommen:
--> ich muss einen Timer2 konfigurieren, dieser soll nach 10ms einen 
Interrupt auslösen

--> dieser Interrupt soll zwei tasks steuern eine mit 4s und die andere 
mit 50ms

Task1:
soll eine Sequenz von Leds aktivieren--> diese Aufgabe habe ich 
hingekriegt

Task2:
--> Taster SW2 einlesen
--> Mit diesem Taster soll LED1 im Wechsel mit 25% und 70% der 
Maximalleistung ansteuern
--> Beim ersten Drücken  LED 25%
--> Beim zweiten Drücken  LED 70%
--> Beim dritten Drücken  LED 25%
--> Und so weiter

Bei Task 2 brauche ich Hilfe! Wo ist denn mein Fehler
Das mein aktueller Code:

#define F_CPU 16000000UL
#include <avr/io.h>
#include <avr/interrupt.h>
#include <stdio.h>

#define PERIODE_TASK1_10MS 400  // for 4s task
#define PERIODE_TASK2_10MS 5    // for 50ms task

volatile char gTime1 = 0;
volatile char gTime2 = 0;
volatile char gState = 0;
volatile char gMute = 0;
volatile char gMute1 = 0;
volatile char state = 0;
volatile char gPushStateSW2 = 0;
enum {NO_PUSH, MAYBE, PROBABLY, PUSHED};
enum {LED_0, LED_25, LED_70};





void init_led(void)
{
   DDRB |= (1<<DDB5); // LED 3
   DDRB |= (1<<DDB4); // LED 4
}

void init_LED1(void)
{
  DDRB |= (1<<DDB1); // LED 1
}

void init_timer_10ms(void){
 // Timer 2  
 TCCR2B |= ((1<<CS22) | (1<<CS21) | (1<<CS20)); //Prescaler 1024
 TCCR2A |= (1<<WGM21); // CTC-Mode
 TIMSK2 |= (1<<OCIE2A);
 OCR2A = 155;
 // Timer 1
 TCCR1A |= ((1<<COM1A1) | (1<<COM1B1));//non inverting Mode
 TCCR1A |= (1<<WGM10);// Phase Correct Modus 8-Bit
 TCCR1B |= (1<<CS12);//Prescaler 256
}

void task1 () {
if (gState == 0){
  gState++;
  PORTB |= (1 << PORTB5);
  PORTB |= (1 << PORTB4);       
}
else if (gState == 1){
  gState++;
  PORTB &= ~(1<< PORTB5);
  PORTB &= ~(1 << PORTB4);       
}
else if (gState == 2){
  gState++;
  PORTB |= (1<< PORTB5);
  PORTB &= ~(1 << PORTB4);    
}
else if (gState == 3){
  gState++;
  PORTB |= (1<< PORTB4);
  PORTB &= ~(1 << PORTB5);
  if( gState == 4)
  {
    gState = 0;
  }

}


  
}
char get_button_state_sw2(void){
  // read status of pin
  char statusPinD3 = !(PIND & (1 << PIND3));
  // debounce (entprellen) with help of state machine:
  char sw2Pushed = 0;
  switch(gPushStateSW2){
    case NO_PUSH:
    if(statusPinD3){
      gPushStateSW2 = MAYBE;
      sw2Pushed = 0;
      return sw2Pushed;
      } else{
      gPushStateSW2 = NO_PUSH;
      sw2Pushed = 0;
      return sw2Pushed;
    }
    break;
    case MAYBE:
    if(statusPinD3){
      sw2Pushed = 0;
      gPushStateSW2 = PROBABLY;
      return sw2Pushed;
    }
    else{
      sw2Pushed = 0;
      gPushStateSW2 = NO_PUSH;
      return sw2Pushed;
    }
    break;
    case PROBABLY:
    if(statusPinD3){
      sw2Pushed = 1;
      gPushStateSW2 = PUSHED;
      return sw2Pushed;
    }
    else{
      sw2Pushed = 0;
      gPushStateSW2 = NO_PUSH;
      return sw2Pushed;

    }
    break;
    case PUSHED:
    if(statusPinD3){
      sw2Pushed = 0;
      gPushStateSW2 = PUSHED;
      return sw2Pushed;
      
    }
    else{
      sw2Pushed = 0;
      gPushStateSW2 = NO_PUSH;
      return sw2Pushed;
    }
    break;
  }
  
}


void LED1(){
OCR1A = 0;

while(gMute1>0){
  
  if(gMute==0){
    OCR1A = 64;
  }
  
  else if (gMute !=0){
    OCR1A = 179;
  }
  
}

}
    
  

void task2(){
  if(get_button_state_sw2()){
    gMute1++;
    if(gMute == 0){gMute = 1;}
    else if(gMute == 1){gMute = 0;}
  }
}


  



ISR(TIMER2_COMPA_vect){
  if (gTime1 > 0) {--gTime1;}
  if (gTime2 > 0) {--gTime2;}
}




int main(void) {
  
  // do all initializations
  init_timer_10ms();
  init_led();
  get_button_state_sw2();
  LED1();
  init_LED1();

  sei();
  
  // control task-activation-scheme in endless loop
  // TODO: task2 needs to be added
  while (1){
    if (gTime1 == 0) {
      gTime1 = PERIODE_TASK1_10MS;
      task1();
    }
    if (gTime2 == 0) {
      gTime2 = PERIODE_TASK2_10MS;
      task2();
    }
  }
}

[Peter D. (peda)]

Mans A. schrieb:
> Wo ist denn mein Fehler

Langen Quelltext als Anhang.
So ist es nur schwer zu lesen.
Das get_button_state_sw2 sieht sehr umständlich aus, ich kann daher 
nicht sagen, ob es klappt.
Aber man will nicht den Zustand, sondern die Flanke auswerten. Ich 
schaue daher immer, ob der Tasteneingang unterschiedlich zum 
gespeicherten Zustand ist. Wenn ja, dann zähle ich 4 Durchläufe. Wenn 
dann immer noch unterschiedlich, dann toggle ich den gespeicherten 
Zustand und wenn die gedrückt Flanke, dann setze ich ein Ereignisbit, 
was dann die Mainloop abfragen kann.
Der Einfachheit halber mache ich das für einen ganzen Port parallel 
(vertical Counter), d.h. bis zu 8 Tasten.

[Mans A. (mansmaak)]

Danke! Das hat mir schon weitergeholfen

[Veit D. (devil-elec)]

Hallo,

die grundlegende Idee ist mittels dem 10ms Timer den Taster zu 
entprellen. Die Tasterauswertung muss man sich so bauen das er einem 
nicht retriggerbaren Monoflop entspricht. Also erst wieder auf erneuten 
Tasterdruck reagiert und nicht auf gedrückthalten. Wenn du das hast 
kannst du sauber durch eine Statemachine wandern.

Im 10ms Timer Interrupt baust du noch zwei Zähler rein der Vielfachen 
für 4s und 50ms zählt. Damit baust du dir 2 weitere bool Trigger.

Den Rest würde ich mit enum und switch-case erschlagen.

[Mans A. (mansmaak)]

Danke man!