Problem mit Tasterentprellung

[Newbie (Gast)]

Hallöchen,

ich benutze Peter's Tasterentprellung...
http://www.mikrocontroller.net/articles/Entprellung#Komfortroutine_.28C_f.C3.BCr_AVR.29
... und habe ein Problem mit der Funktion "get_key_short(1<<KEY1)".

Ich habe auf KEY1 die Repeat-Funktion der Entprellung und diese funzt 
auch einwandfrei - bei Loslassen des Tasters wird aber jedesmal auch ein 
"get_key_short"-Ereignis ausgelöst.

Ich möchte mir gern einen Dimmertaster bauen, wobei ich aber unbedingt 
auch genau diese "short"-Funktion brauche, um die LEDs mit einem kurzen 
Hub an-/ausschalten zu können.

Hat jemand von euch vielleicht einen Rat? (alle anderen Routinen des 
Codes funktionieren auch miteinander, nur diese eine nicht)

Viele Grüße

[Newbie (Gast)]

EDIT:

Mit der zusätzlichen Variable bON bin ich nun zumindest soweit, dass das 
Ausschalten per kurzem Druck funktioniert:

if(get_key_short(1<<KEY1))
{
if(bON == 1)
{
if(!(get_key_rpt(1<<KEY1)))
nSavedPWM = OCR1A;
OCR1A = 0;
bON = 0;
}
else
{
OCR1A = nSavedPWM;
bON = 1;
}
}

if(get_key_rpt(1<<KEY1))
{
if(nCount > 1)
{
OCR1A += nSteps;
if(OCR1A >= 1023)
{
OCR1A = 0;
}
}
nSavedPWM = OCR1A;
bON = 0;
nCount++;
}
if(PINC & (1<<PINC0))
nCount=0;

Niemand einen Rat?

LG

[Peter D. (peda)]

Beitrag "Re: Universelle Tastenabfrage"

Peter

[Newbie (Gast)]

Hallo Peter,

danke für die schnelle Antwort.;-)

Ich komm' aber leider nicht weiter. WIE genau muß ich denn...
"get_key_short() mit get_key_long_r() und get_key_rpt_l()" miteinander 
verbinden?
Die Grafik von 'Jonas' hab' ich verstanden, aber die Verknüfung der 
Funktionen leider nicht.:(

Viele Grüße

[Newbie (Gast)]

Moin moin,;-)

ich hab' jetzt mal wie folgt versucht, aber ich bekomm es nicht hin, 
'bON' bei kurzem Tastendruck zu toggeln, sodass man neben der 
Dimmfunktion über '...rpt' auch ein-/ausschalten kann:

  while(1)
  {

    if(get_key_short(1<<KEY1))
    {
      if(bON == 1)
      {
        nSavedPWM = OCR1A;
        OCR1A = 0;
        bON = 0;
      }
      else
      {
        OCR1A = nSavedPWM;
        bON = 1;
      }
    }

    if(get_key_long_r(1<<KEY1) || get_key_rpt_l(1<<KEY1))
    {
      if(nCount > 1)
      {
        OCR1A += nSteps;
        if(OCR1A >= 1023)
          OCR1A = 0;
        nSavedPWM = OCR1A;
      }

      //bON = 0;
      nCount++;
    }
    if(PINC & (1<<PINC0))
      nCount=0;



  }
  return 0;

WAS mach ich denn nur falsch???

Viele Grüße

[Newbie (Gast)]

Hey liebe Gemeinde^^,

bin langsam echt am Verzweifeln. Ich bastle jetzt seit seit über 30 
Stunden durchweg an dieser Taster-Entprellung und bekomm es einfach ned 
hin.
Die Tasterentprellung habe ich 1:1 von Peter übernommen, aber die 
Funktion 'Short+Repeat' funktioniert einfach NICHT wirklich. Manchmal 
geht's, aber meistens eher NICHT.

Mit 'Short+Repeat' bekomm ich nach jedem 'Repeat' auch ein 
'Short'-Signal und mit 'Short+LongR+RepeatL'(s. vorheriger Beitrag) 
kommt das 'Short'-Signal nur ab und an mal...?! WAS ist denn da nur los? 
Ist Peter's Code nicht voll funktionsfähig???

Dies ist mein restlicher Code:

#ifndef F_CPU
#define F_CPU 16000000
#endif
/*******************************************************************************************************/
// Einbindung von Standardbibliotheken...
#include <stdint.h>        // Datentypen (standardisiert)
#include <avr/io.h>        // Registernamen
#include <avr/interrupt.h>
#include <stdio.h>         // Rechenoperationen, etc.
#include <math.h>
//#include <string.h>        // Zeichenketten-Funktionen
// ... und externen Dateien
#include "buttons.h"
/*******************************************************************************************************/
#define RELAIS_ON PIND7
#define RELAIS_OFF PINB0
/*******************************************************************************************************/
volatile uint8_t sreg_local;
// zum Deaktivieren der ISRs(Interrupt Service Routine) auf globaler Ebene
void ISR_0() // ISRs deaktivieren
{
    sreg_local = SREG; // Erstellung einer lokalen Sicherungskopie von SREG
    cli();
}
// zum Aktivieren der ISRs auf globaler Ebene
// (Es werden ausschließlich die ISRs aktiviert, die auch vor dem Abschalten schon in Benutzung waren!)
void ISR_1() // ISRs aktivieren
{
    SREG = sreg_local; // Sicherungskopie von SREG laden
}
/*******************************************************************************************************/
// Steuerung der ISR(Interrupt Service Routine)

uint16_t iCount = 0;
uint16_t nVal = 960;
ISR(TIMER1_OVF_vect)
{




  /*iCount++;
  if(iCount>=nVal)
  {
    OCR1A = 5;
    if(iCount>=(nVal*2))
    {
      OCR1A = 50;
      iCount=0;
    }
  }*/
}
/*******************************************************************************************************/
int main (void)
{
  // Konfiguration der I/Os
  // Taster an Pin23:
  DDRC &= ~(1<<PC0); // Pin PC0 als Eingang
  PORTC |= (1<<PC0); // Aktivierung des internen Pull-Up an PC0

  DDRB = 0xFF;
  DDRD = 0xFF;
    /***************************************************************************************************/
  // Konfiguration des Timers
  // Mode:Fast PWM;TOP:OCR1A;Update:BOTTOM;TOV1:TOP;Prescaler:1(FRQ@16MHz = 16MHz)
  TCCR1A = (1<<COM1A1)|(1<<WGM11);
  TCCR1B = (1<<WGM13)|(1<<WGM12)|(1<<CS10);
    OCR1A = 0;          // PWM-Wert
  ICR1 = 1023;        // TOP-Wert des Zählers
  TIMSK1 |= 1<<TOIE1; // Aktivierung des Interrupts(Timer1 Overflow Interrupt Enable)

  TCCR0B = (1<<CS02)|(1<<CS00);      // divide by 1024
  TIMSK0 |= 1<<TOIE0;        // enable timer interrupt
  sei();
  /***************************************************************************************************/
  int16_t nSavedPWM = 0;
  int16_t nCount = 0;
  int16_t bON = 0;

  while(1)
  {
    
    if(get_key_short(1<<KEY1))
    {
      if(bON == 0)
      {
        OCR1A = nSavedPWM;
        bON = 1;
      }
      else
      {
        nSavedPWM = OCR1A;
        OCR1A = 0;
        bON = 0;
      }
      
    }

    if(get_key_long_r(1<<KEY1) || get_key_rpt_l(1<<KEY1))
    {
      if(nCount > 1)
      {
        OCR1A += 50;
        if(OCR1A >= 1023)
          OCR1A = 0;
        nSavedPWM = OCR1A;
      }
      
      bON = 0;
      nCount++;
    }
    if(PINC & (1<<PINC0))
      nCount=0;



  }
  return 0;
}

In Peter's Tasterentprellung habe ich nur einen Teiler für TCNT0, von 
'F_CPU' > 'F_CPU/16', eingefügt, um auf die originale 1MHz-Frequenz zu 
kommen.

PLEASE HELP ME.;-)

Liebe Grüße

[Newbie (Gast)]

EDIT:

Hab's nun auch ohne Hilfe gelöst. Danke trotzdem.;-)
Meine Hilsvariable war nur an der falschen Stelle.

Dimmer mit EIN-/AUS-Schaltfunktion:

if(get_key_short(1<<KEY1))
    {
      if(bON == 0)
      {
        OCR1A = nSavedPWM;
        bON = 1;
      }
      else
      {
        nSavedPWM = OCR1A;
        OCR1A = 0;
        bON = 0;
      }

    }

    if(get_key_rpt(1<<KEY1))
    {
      if(nCount > 1)
      {
        OCR1A = OCR1A + 50;
        if(OCR1A >= 1023)
          OCR1A = 0;

        bON = 0;
        nSavedPWM = OCR1A;
      }


      nCount++;
    }
    if(PINC & (1<<PINC0))
      nCount=0;


LG

[Peter D. (peda)]

Newbie schrieb:
> Mit 'Short+Repeat' bekomm ich nach jedem 'Repeat' auch ein
> 'Short'-Signal und mit 'Short+LongR+RepeatL'(s. vorheriger Beitrag)
> kommt das 'Short'-Signal nur ab und an mal...?! WAS ist denn da nur los?
> Ist Peter's Code nicht voll funktionsfähig???

Er funktioniert.
Aber Du mußt den Link schon lesen und Dich danach richten.
Es steht alles drin.
Und wenn da steht:
> - get_key_short() mit get_key_long_r() und get_key_rpt_l()
dann ist das auch so gemeint. Du darfst keine Funktion weglassen!
Du mußt sie ja nicht auswerten, nur aufrufen reicht.
Dein Würg-Around mit der Hilfsvariable ist dann völlig unnötig.

Überlege mal selber, wie die arme CPU erkennen soll, was Du willst. Sie 
kann doch nicht in Deinen Kopf sehen.


Peter

[Newbie (Gast)]

Hi Peter,

ich hab' noch ein wenig Probleme mit Bitmanipulation und Datentypen, was 
wohl mein größtes Problem darstellt.
So läuft das Programm z.B. auch erst, seit ich meine (Hilfs-)Variablen 
von uint8_t auf int16_t umdeklariert habe.

Wie du oben sehen kannst, habe ich ja deine drei Funktionen aufgerufen - 
UND Probleme bekommen.
Es geht ja jetzt endlich (auch, wenn ich es anders gelöst habe). Falls 
deine anfängliche Variante wirklich funktionieren sollte und ich bei der 
Umsetzung nur Fehler gemacht habe, wäre ich sehr glücklich, wenn du mir 
es "richtig" aufzeigen könntest.^^

Die Hilfsvariable brauche ich UNBEDINGT, da ich sonst NICHT toggeln 
kann.

Liebe Grüße