ATNEL tech-forum

Re: MK_MATRIX_KBD i repetycja klawisza

2021-07-31T12:01:24+01:00

Gienek_S napisał(a):

Przepraszam bardzo że cię irytują moje wypowiedzi.

Nie nie irytują tylko się zdziwiłem - ale pewnie źle zrozumiałem - więc przepraszam

Gienek_S napisał(a):

tylko to co zauważyłem to funkcje 1. i 2. zwraca tylko raz klawisz. Natomiast funkcja 3.) zwraca klawisz po czasie LONG_PRESS_TIME po czym zwraca klawisz repetowany.
I dlatego żeby uzyskać w jednej funkcji zwrot naciśniętego klawisza zrobiłem taką sumę logiczną tych dwóch zdarzeń w uint8_t GetKey (void)

ok teraz rozumiem po co robisz sobie ten get_key - ale zastanawiam się dlaczego ?

No bo teraz już doszliśmy razem do tego że jednak repetycja działa i tak jak autor to obmyślił czyli tylko dla funkcji nr 3 jak napisałeś.

Spróbuj opisać w czym masz problem ? bo np jeśli w funkcji nr 1 masz coś tam zrobić ale dokładnie to samo chciałbyś żeby się wykonywało podczas repetycji - to przecież nie potrzebne ci ŻADNE getkeye - tylko np, załóżmy że chcę zmieniać stan diody LED za każdym razem tak ?

No to napisz sobie zewnętrzną funkcję np:

[syntax=c]void led_tog( void ) {
PORTC ^= (1<}[/syntax]

i teraz wywołuj że tę funkcję z:

[syntax=c]void key_down( uint8_t key_nr ) {
led_tog();
}[/syntax]

a także z:

[syntax=c]void key_long_press( uint8_t key_nr ) {
led_tog();
}[/syntax]

to jest właściwa droga i właściwe podejście - te funkcje 1-3 są niejako zdarzeniami i wystarczy je oprogramować w dowolny sposób. Czyli nie wstawiaj do nich jakiegoś kodu, szczególnie jeśli ma być powtarzalny tylko włóż ten kod do właściwej nowej funkcji i wywołuj ją z metod na których ci zależy. To troszkę tak jak w programowaniu prawie że obiektowym

Statystyki: Napisane przez mirekk36 — 31 lip 2021, o 12:01

Re: MK_MATRIX_KBD i repetycja klawisza

2021-07-31T08:02:04+01:00

Przepraszam bardzo że cię irytują moje wypowiedzi. Ja tylko opisałem jak u mnie to działa, nie miałem zamiaru Ci nic tłumaczyć jak działa Twoja biblioteka.

Jeszcze raz spróbuję przekazać moje spostrzeżenia.
Zakładam że w pliku "mk_keyboard.h" mamy:
#define ENABLE_ON_KEY_DOWN_EVENT1
#define ENABLE_ON_KEY_UP_EVENT1
#define ENABLE_ON_KEY_LONG_PRESS_EVENT1
#define LONG_PRESS_TIME50// min = 20, default = 50, MAX = 254!
#define LONG_PRESS_AS_AUTO_REPEAT1// 0 - DISABLE AUTO REPEAT, 1 - ENABLE AUTO REPEAT
#define AUTO_REPEAT_RIME15 // min = 1, default = 15, MAX = (LONG_PRESS_TIME-1) !!!

W programie głównym wywołuję tylko funkcję MK_KBD_EVENT(); Czyli wszystko co biblioteka zwraca znajduje się odpowiednio w:
1.) void key_down( uint8_t key_nr )
2.) void key_up( uint8_t key_nr )
3.) void key_long_press( uint8_t key_nr )

tylko to co zauważyłem to funkcje 1. i 2. zwraca tylko raz klawisz. Natomiast funkcja 3.) zwraca klawisz po czasie LONG_PRESS_TIME po czym zwraca klawisz repetowany.
I dlatego żeby uzyskać w jednej funkcji zwrot naciśniętego klawisza zrobiłem taką sumę logiczną tych dwóch zdarzeń w uint8_t GetKey (void)

Statystyki: Napisane przez Gienek_S — 31 lip 2021, o 08:02

Re: MK_MATRIX_KBD i repetycja klawisza

2021-07-30T23:40:38+01:00

Gienek_S napisał(a):

Funkcje biblioteczne key_up i key_down same nie repetują klawisza.

Ale proszę cię (i bez urazy) ale nie opowiadaj bajek bo zastanawiam się czy to ty mi tłumaczysz jak działa moja biblioteka ? czy raczej zapytałeś jak działa ?

Ne wiem w ogóle co ty wymyślasz z tym GetKey ... to jest niestety absurd, dlatego, że jak się ustawi repetycję w bibliotece to funkcja key_long_press() czyli callback jest wywoływany automatycznie zgodnie z interwałem czasu określonym w #define AUTO_REPEAT_RIME, i to do niej wstaw sobie BEEP'a swojego

Statystyki: Napisane przez mirekk36 — 30 lip 2021, o 23:40

Re: MK_MATRIX_KBD i repetycja klawisza

2021-07-30T21:27:08+01:00

repetycja działa jak zmodyfikowałem funkcję "uint8_t GetKey (void)" do postaci:
[syntax=c]uint8_t GetKey (void)
{
char temp = keyL;//keyD or keyU or keyL
char temp2= keyD;

if (temp){
keyL = 0;
return temp;
}
if (temp2){
keyD = 0;
return temp2;
}

return temp;
}[/syntax]
Funkcje biblioteczne key_up i key_down same nie repetują klawisza.

Statystyki: Napisane przez Gienek_S — 30 lip 2021, o 21:27

Re: MK_MATRIX_KBD i repetycja klawisza

2021-07-30T20:41:20+01:00

wiesz co? Właśnie sprawdziłem bibliotekę i działa idealnie

Weź sobie to przetestuj panie kochany najpierw na pliku main.c który dostałeś razem z biblioteką. Działa "od kopa" po pobraniu, i to na tych ustawieniach które pokazałeś z pliku *.h

ja niestety nie mam aż na tyle czasu żeby teraz analizować twój main.c ... więc najpierw jak mówię zacznij od kodu w main.c z pliku biblioteki i jak tam coś nie będzie działało to wtedy napisz co ok?

Statystyki: Napisane przez mirekk36 — 30 lip 2021, o 20:41

Re: MK_MATRIX_KBD i repetycja klawisza

2021-07-30T20:00:14+01:00

obojętnie czy:
#define ENABLE_ON_KEY_LONG_PRESS_EVENT1
czy:
#define ENABLE_ON_KEY_LONG_PRESS_EVENT0

efekt jest taki sam.

Statystyki: Napisane przez Gienek_S — 30 lip 2021, o 20:00

Re: MK_MATRIX_KBD i repetycja klawisza

2021-07-29T21:48:33+01:00

teraz nie pamiętam szczegółów i nie mam jak sprawdzić - ale czy nie próbowałeś wyłączyć zdarzenia

ENABLE_ON_KEY_LONG_PRESS_EVENT

czyli ustawić go na zero? Bo jak ono jest włączone to chyba (ale może się mylę) nie będzie repetycji bo to się troszkę kłóci

sprawdź i daj znać

Statystyki: Napisane przez mirekk36 — 29 lip 2021, o 21:48

MK_MATRIX_KBD i repetycja klawisza

2021-07-29T20:52:19+01:00

Witam.
Testuję tę bibliotekę i nie potrafię uzyskać repetycji klawiszy.
Konfiguracja biblioteki:
[syntax=cpp]//--- wybór portu klawiatury ---
#define KBD_PORTPORTE
#define KBD_PINPINE
#define KBD_DIRDDRE

//--- typ klawiatury ---
#define KBD_TYPEROW_KEYS// KBD_4X3 lub KBD_4X4 lub ROW_KEYS

#define ROW_KEYS_COUNT4// jeśli KBD_TYPE = ROW_KEYS to ustaw ilość klawiszy: 1 - 8

// jeśli zdefiniowana jest maska to przestaje mieć znaczenie ROW_KEYS_COUNT
// wystarczy odkomentować poniższą definicję i ustawić jedynki w miejscach przycisków
#define ROW_KEYS_MASK0b111100// własna maska - jedynki w miejscach pinów z przyciskami

// tylko do celów testowych z klawiaturami jednorzędowymi
//#define ROW_GND_PIN4// ustawienie stanu niskiego GND dla testów prostej jednorzędowej klawiatury

//--- wybór reakcji: wciśnięcie, zwolnienie przycisku albo oba zdarzenia ---
#define ENABLE_ON_KEY_DOWN_EVENT1
#define ENABLE_ON_KEY_UP_EVENT1

#define ENABLE_ON_KEY_LONG_PRESS_EVENT1

#define LONG_PRESS_TIME50// min = 20, default = 50, MAX = 254!

#define LONG_PRESS_AS_AUTO_REPEAT1// 0 - DISABLE AUTO REPEAT, 1 - ENABLE AUTO REPEAT
#define AUTO_REPEAT_RIME15 // min = 1, default = 15, MAX = (LONG_PRESS_TIME-1) !!!

#define USE_ASCII_CODES1

//-------------------------------- koniec konfiguracji biblioteki ----------[/syntax]

Program główny:
[syntax=cpp]#include
#include
#include
#include

#include "LCD/lcd44780.h"

#include "MK_MENU_LIB/mk_menu.h"
#include "MK_MENU_LIB/menu_items.h"
#include "MK_MENU_USER_FILES/mk_menu_display.h"
#include "MK_MENU_USER_FILES/mk_menu_exe.h"

//Tylko dla wersji 3.0
//#include "MK_MENU_LIB/mk_menu_include.h"

#include "MK_MATRIX_KBD/mk_keyboard.h"
#include "makra.h"
#include "bity.h"

/*======================================================
* Definicje BUZ-era i ALarmu
*/
#define PORT_BUZA
#define PIN_BUZ6
#define PORT_ALC
#define PIN_AL2

#define BUZDAJ_BIT(&CONCAT(PORT, PORT_BUZ)).CONCAT(Bit, PIN_BUZ)
#define BEEPBUZ=1; _delay_ms(5); BUZ=0;

#define INIT_BUZCONCAT(DDR, PORT_BUZ)|= 1<#define ALDAJ_BIT(&CONCAT(PORT, PORT_AL)).CONCAT(Bit, PIN_AL)
#define INIT_ALCONCAT(DDR, PORT_AL)|= 1<
uint8_t rep;
uint8_t keyD, keyU, keyL;

// nagłówki własnych funkcji do zdarzeń (callbacków)
//void key_down( uint8_t key_nr, char ascii );
//void key_up( uint8_t key_nr, char ascii );
//void key_long_press( uint8_t key_nr, char ascii );

void key_down( uint8_t key_nr );
void key_up( uint8_t key_nr );
void key_long_press( uint8_t key_nr );
uint8_t GetKey (void);

int main (void)
{
//uint8_t klawisz;

mk_kbd_init();// inicjalizacja biblioteki
// rejestracja własnych funkcji zdarzeń
#if ENABLE_ON_KEY_DOWN_EVENT == 1
register_on_key_down_callback( key_down );
#endif

#if ENABLE_ON_KEY_UP_EVENT == 1
register_on_key_up_callback( key_up );
#endif

#if ENABLE_ON_KEY_LONG_PRESS_EVENT == 1
register_on_key_long_press_callback( key_long_press );
#endif

INIT_BUZ;
BEEP;
INIT_AL;

lcd_init();
lcd_locate(0,0);
lcd_int(1234);
lcd_locate(1,4);
lcd_hex(1234);
lcd_locate(0,5);
lcd_str_P(PSTR("SB-01An"));

/* Timer2 – inicjalizacja przerwania co 10ms */
TCCR2 |= ( 1 << WGM21 ); // tryb pracy CTC
TCCR2 |= ( 1 << CS22 ) | ( 0 << CS21 ) | ( 1 << CS20 ); // preskaler = 1024
OCR2 = (F_CPU / 1024UL / 100UL)-1; // przerwanie porównania co 10ms (100Hz)
TIMSK |= ( 1 << OCIE2 ); // Odblokowanie przerwania CompareMatch

sei();// globalne zezwolenie na przerwania

for (;;){
MK_KBD_EVENT();// zdarzenie biblioteki obsługi klawiatury

if (GetKey()){
BEEP;
}
}
}

// procedura obsługi przerwania
ISR( TIMER2_COMP_vect ) {

isr_kbd_matrix_proc();// wywołanie funkcji przerwania biblioteki klawiatury
}

//moja funkcja odczytu klawiatury na bazie callbacków
uint8_t GetKey (void)
{
char temp = keyD;//keyD or keyU or keyL

if (temp){
keyD = 0;
//BEEP;
}

return temp;
}

// własna funkcja reagująca na key down
void key_down( uint8_t key_nr ) {

keyD = key_nr;//moje przekazanie klawisza z callbacka do GetKey

}

// własna funkcja reagująca na key up
void key_up( uint8_t key_nr ) {

keyU = key_nr;//moje przekazanie klawisza

}

// własna funkcja reagująca na long press
void key_long_press( uint8_t key_nr ) {

static uint8_t lastk;

keyL = key_nr;//moje przekazanie klawisza

if( key_nr != lastk ) rep=0;

lastk = key_nr;
}[/syntax]
BEEP sygnalizuje mi nowe naciśnięcia klawisza. I to działa poprawnie. Ale jak przytrzymam dowolny klawisz to nie jest on repetowany bo powinien mi powtarzać tego BEEP-a.
Co jest nie tak ?
P.S. Procesor ATMega128A

Statystyki: Napisane przez Gienek_S — 29 lip 2021, o 20:52