C: Problem mit Pedas Tastenentprellung - "repeat_next" verändern.

[Frank (Gast)]

Guten morgen!

Ich haba ein Problem mit der Tastenentprellung von Peda. Also insgesamt 
läuft die Geschichte fantastisch! Aber jetzt versuche ich gerade das 
"repeat_next" zu verändern um bei länger gedrückten Tasten die 
Repeat-Funktion zu verschnellern.

Ich habe eine "button_functions.h" und eine "button_functions.c"

button_functions.h

// button_functions.h

#ifndef BUTTON_FUNCTIONS_H
#define BUTTON_FUNCTIONS_H

#include "button_mcu_specific_definitions.h"
#include <stdint.h>



static uint8_t repeat_next = 20;


#define BUTTON_P             ( (get_button_short (BUTTON_3))         \
                               || (get_button_long (BUTTON_3)) )
#define BUTTON_PLUS          ( (get_button_short (BUTTON_1))         \
                               || (get_button_long (BUTTON_1)) )
#define BUTTON_MINUS         ( (get_button_short (BUTTON_2))         \
                               || (get_button_long (BUTTON_2)) )
...
...
void check_buttons( void );

uint8_t get_button_press  (uint8_t button_mask);
uint8_t get_button_repeat (uint8_t button_mask);
uint8_t get_button_short  (uint8_t button_mask);
uint8_t get_button_long   (uint8_t button_mask);
uint8_t get_button_common (uint8_t button_mask);

In dieser.h habe ich repeat_next als static deklariert.

Dazu meine button_functions.c

// button_functions.c

#include "button_functions.h"
#include "button_mcu_specific_definitions.h"
#include "msp430x24x.h"
#include <stdint.h>

extern uint8_t repeat_next;

#define ALL_BUTTONS               ( BUTTON_1 | BUTTON_2 | BUTTON_3 )
#define REPEAT_MASK               ( BUTTON_1 | BUTTON_2 | BUTTON_3 )
#define REPEAT_START              50

// #define BUTTON_TIMEOUT

static uint8_t button_state;              // debounced and inverted key state
static uint8_t button_press;              // key press detect
static uint8_t button_repeat;             // key long press and repeat

void check_buttons( void )
{
  static uint8_t counter0;
  static uint8_t counter1;
  static uint8_t repeat;
  
  uint8_t temp;
    
  temp = button_state ^ ~BUTTON_IN;              // key changed ?
  counter0 = ~( counter0 & temp );               // reset or count ct0
  counter1 = counter0 ^ (counter1 & temp);       // reset or count ct1
  temp &= counter0 & counter1;                   // count until roll over ?
  button_state ^= temp;                          // then toggle debounced state
  button_press |= button_state & temp;           // 0->1: key press detect
  
  if( (button_state & REPEAT_MASK) == 0 )        // check repeat function
  {
    repeat = REPEAT_START;                       // start delay
  }
  
  if( --repeat == 0 )
  {
    repeat = repeat_next;                        // repeat delay
    button_repeat |= button_state & REPEAT_MASK;
  }
}

In dieser .c Datei wird auf die in der .h deklarierte Variable 
repeat_next mittels extern zugegriffen.

Und dann kommt da noch meine Funktion, in der ich diese Repeat-Zeit 
verändern möchte - diese befindet sich in einer anderen .c Datei und 
bindet die button_functions.h mit ein:

...
...

  if( BUTTON_PLUS_REPEAT || BUTTON_PLUS )
  {
    repeat_counter++;
    
    if( (repeat_counter > 10) && (repeat_counter < 20) )
    {
      repeat_next = 10;
    }
    else if( repeat_counter > 20 )
    {
      repeat_counter = 20;
      repeat_next = 5;
    }
...
...
  }

...
...
  if( BUTTON_PLUS_PRESSED || BUTTON_MINUS_PRESSED )
  {
    repeat_button = TRUE;
  }
  else
  {
    repeat_button = FALSE;
    repeat_counter = 20;
    repeat_next = 20;
  }

In dieser .c Datei ist ebenfalls "extern uint8_t repeat_next" 
deklariert.

So, das Problem ist nun, dass wenn das Programm hier vorbei kommt, alles 
wunderbar gezählt wird. "repeat_counter" ist hier natürlich ebenfalls 
static deklariert. Das geht auch alles. repeat_next wird richtig 
verändert, aber jedesmal, wenn durch den 10ms-Interrupt wieder die 
Tastenabfrage aufgerufen wird, ist repeat_next wieder auf 20.

Das schneller zählen geht also nicht.

Was mache ich falsch? Wie relisiert ihr so ein Erhöhen der 
Geschwindigkeit?

[Frank (Gast)]

Mhh...keiner? ;-P


Ich habe mir jetzt ein Funktion geschrieben, die den repeat_next ändert.