Portpinabfrage innerhalb eines Timerinterrupts

[M. G. (ixil96)]

Hallo,

ich suche nach einer Lösung für folgende Aufgabe:
1.) Eine Status LED soll im Sekundentakt blinken. Dieser Sekundentakt 
soll mit einem Timerinterrupt realisiert werden.

2.) Weiteres soll im Sekundentakt ein Taster abgefragt werden. Ist der 
Taster gedrückt, soll sich eine 2. LED für 10 Sekunden einschalten.

3.) Wird der Taster innerhalb dieser 10 Sekunden erneut gedrückt, 
verlängert sich die Einschaltzeit um weitere 10 Sekunden.

Und bei Punkt 3 stehe ich derzeit auf der "Leitung"!
Kann mir da bitte jemand helfen, wie ich das am einfachsten lösen kann?

Hier mein derzeitiger Code:

#include <avr/io.h>
#include <avr/interrupt.h>

#define led1 (1<<PB4)
#define led2 (1<<PB5)

unsigned int timer_overflow_counter = 0;
unsigned int glowtime = 10;
unsigned int glowtime_led_2;
unsigned char taster_flag = 0;

void init_timer_0 (void)
{
  TCCR0B = (1<<CS01) + (1<<CS00);    // Prescaler = 64 -> 8Mhz/64 = 125000 clockspeed
  TIMSK0 = (1<<TOIE0);        // Timer0 overflow interrupt enable
  TCNT0 = 6;              // Timer mit 6 vorladen (256-6 = 250 mal hochzählen bis zum overflow)
}  

int main(void)
{
  DDRB = 0xFF;            // PORTB Richtungsregister = Ausgang
  DDRC = 0xF0;            // PC0...PC3 auf Eingang
  PORTC = 0xF1;            // PC0 = Pull up
  
  init_timer_0();            // Timer 0 initialisieren
  sei();                // Interrupts einschalten
    while(1)
    {
        asm ("NOP");          // Nichts tun
    }
}

ISR(TIMER0_OVF_vect)
{
  if (timer_overflow_counter <= 499)        // Zählvariable 500 mal hochzählen
    timer_overflow_counter ++;
  else                      // Den else-Zweig beim 500 Durchlauf einmal ausführen
  {
    TCNT0 = 6;                  // reload timer0 
    PORTB ^= led1;                // LED toggeln
    timer_overflow_counter = 0;          // Zähler zurücksetzen
    
    if ((PINC &(1<<PINC0)) && (taster_flag == 0))     // Wenn Taster gedrückt wurde...
    {    
      taster_flag = 1;            // Flag für Taster wurde gedrückt setzen
      glowtime_led_2 = glowtime;        // glowtime_led_2 auf Anfangswert 10 setzen
    }
    
    if (taster_flag == 1)            // Taster wurde gedrückt
    {
      PORTB |= led2;              // LED ON
      glowtime_led_2 --;            // glowtime_led_2 pro Sekunde um 1 runter zählen
      
      if (glowtime_led_2 == 0)
      {
        PORTB &= ~led2;            // LED OFF
        glowtime_led_2 = glowtime;      // glowtime_led_2 auf Anfangswert 10 setzen
        taster_flag = 0;          // Taster flag zurücksetzen
      }
    }
  }
}

[User (Gast)]

Du willst doch bei Tastendruck die glowtime um 10s erhöhen, richtig?

Dann addiere doch bei Tastendruck deine Konstante glowtime zu 
glowtime_led_2.
So wie es jetzt ist, wird sie ja immer auf 10 gesetzt.

    if ((PINC &(1<<PINC0)) && (taster_flag == 0))     // Wenn Taster gedrückt wurde...
    {    
      taster_flag = 1;            // Flag für Taster wurde gedrückt setzen
      glowtime_led_2 += glowtime;        // glowtime_led_2 + glowtime
    }

[M. G. (ixil96)]

Hallo,

also eine Lösung habe ich gerade gefunden.
(Bin mal schnell von der "Leitung" runter)

Ich frage den PINC0 unabhängig vom Tastenflag nochmals ab, ob er 
gedrückt wurde und setze die glowtime_led_2 wieder auf den Anfangswert.

Aber ich denke, für das Ganze gibt es sicher eine noch viel einfachere 
Lösung!

if (PINC &(1<<PINC0))
      glowtime_led_2 = glowtime;        // glowtime_led_2 auf Anfangswert 10 setzen

Hat hier jemand einen Vorschlag?

Danke!

[M. G. (ixil96)]

User schrieb:
> Du willst doch bei Tastendruck die glowtime um 10s erhöhen, richtig?
>
> Dann addiere doch bei Tastendruck deine Konstante glowtime zu
> glowtime_led_2.

Ich habe mich hier vielleicht nicht gut ausgedrückt.
Bei einem Tastendruck soll sich die Leuchtzeit der LED auf 10 Sekunden 
zurücksetzen. Also so wie ein nachtriggerbares Monoflop.

[spe (Gast)]

if ((PINC &(1<<PINC0)) && (taster_flag == 0))

wozu hast du die zweite Abfrage überhaupt drin?

if ((PINC &(1<<PINC0)))

reicht doch völlig aus (entspricht auch deiner lösung nur eben nur eine 
Abfrage)

[Fabian O. (xfr)]

1. Warum stellst Du den Timer nicht langsamer ein? In 499 von 500 
Interrupts passiert nichts. Das Vorladen des Counters kannst Du Dir 
sparen, wenn Du den CTC-Modus nimmst.

2. Du fragst den Taster nur alle 10 Sekunden ab. Wenn er 
zwischenzeitlich kurz gedrückt wurde, merkst Du davon nichts.

3. Die Variable taster_flag ist unnötig, die Information steckt schon in 
glowtime_led2.

Eine Alternative wäre, den Taster in der main zu prüfen:

#define GLOW_TIME 10
volatile uint8_t glowtime_led_2 = 0;

int main(void)
{
  // ...
  while (1) {
    if (PINC & (1 << PINC0)) {
      glow_time_led2 = GLOW_TIME;
    }
  }
}

ISR() {
  // ...
  if (glow_time_led2 > 0) {
    glow_time_led2--;
    // LED on
  } else {
    // LED off
  }
}

[M. G. (ixil96)]

spe schrieb:
>

if ((PINC &(1<<PINC0)) && (taster_flag == 0))

>
> wozu hast du die zweite Abfrage überhaupt drin?
>
>

if ((PINC &(1<<PINC0)))

>
> reicht doch völlig aus (entspricht auch deiner lösung nur eben nur eine
> Abfrage)

Ja, stimmt eigentlich. Danke!

ISR(TIMER0_OVF_vect)
{
  if (timer_overflow_counter <= 499)        // Zählvariable 500 mal hochzählen
    timer_overflow_counter ++;
  else                      // Den else-Zweig beim 500 Durchlauf einmal ausführen
  {
    TCNT0 = 6;                  // reload timer0 
    PORTB ^= led1;                // LED toggeln
    timer_overflow_counter = 0;          // Zähler zurücksetzen
    
    if (PINC &(1<<PINC0))             // Wenn Taster gedrückt wurde...
    {    
      taster_flag = 1;            // Flag für Taster wurde gedrückt setzen
      glowtime_led_2 = glowtime;        // glowtime_led_2 auf Anfangswert 10 setzen
    }
    
    if (taster_flag == 1)            // Taster wurde gedrückt
    {
      PORTB |= led2;              // LED ON
      glowtime_led_2 --;            // glowtime_led_2 pro Sekunde um 1 runter zählen
      
      if (glowtime_led_2 == 0)
      {
        PORTB &= ~led2;            // LED OFF
        glowtime_led_2 = glowtime;      // glowtime_led_2 auf Anfangswert 10 setzen
        taster_flag = 0;          // Taster flag zurücksetzen
      }
    }
  }
}