ATNEL tech-forum

Re: Drobna poprawka do biblioteki MK_MATRIX_KBD

2018-04-28T12:47:30+01:00

modyfikuj sobie jak chcesz .. jak przestanie ci działać w jakiejś sytuacji to odkomentujesz sobie ten fragment

Statystyki: Napisane przez mirekk36 — 28 kwi 2018, o 12:47

Re: Drobna poprawka do biblioteki MK_MATRIX_KBD

2018-04-28T07:43:22+01:00

Quote:

oj ma ma znaczenie ... jak chcesz to usuń ten fragment ale przetestuj później WSZYSTKIE dostępne konfiguracje

Ja zrobiłem tylko taką modyfikację w kodzie dla klawiatur jednorzędowych:
[syntax=c]#if ENABLE_ON_KEY_LONG_PRESS_EVENT == 1
if( !(key & LONG_PRESS_MASK) && !(key & READED_MASK) ) {
key &= KEY_CODE_MASK;
key |= RELEASED_MASK | READED_MASK;// **** ważne
} else {
lpress = 0;
key = READED_MASK;
}[/syntax]

Zresztą właśnie dla klawiatur matrycowych , ten fragment kodu jest identyczny.
Testowałem to na różne sposoby i wszystko pięknie działa.
A więc utwierdzam się w przekonaniu, że w tym miejscu - mam tu na myśli fragment kodu po instrukcji "else" - zmienna lpress ma być po prostu wyzerowana, bez względu na jej wartość. Musi być także wpisana do zmiennej key wartość READED_MASK. Oczywiście jeśli przytrzymamy przycisk wciśnięty nieco dłużej, to zmienna lpress już wcześniej się wyzeruje ponieważ cały czas inkrementujemy ją w przerwaniu. Ale przy takim zapisie nie ma to już znaczenia i tak wykona się kod po instrukcji "else".
Jeśli jednak po else dodamy jeszcze warunek if:
[syntax=c]#if ENABLE_ON_KEY_LONG_PRESS_EVENT == 1
if( !(key & LONG_PRESS_MASK) && !(key & READED_MASK) ) {
key &= KEY_CODE_MASK;
key |= RELEASED_MASK | READED_MASK;// **** ważne
} else if (lpress ) {
lpress = 0;
key = READED_MASK;
}[/syntax]
no to jeśli lpress wyzeruje się zanim zwolnimy przycisk, do zmiennej key nie zapisze się wartość READED_MASK. W konsekwencji program przestaje reagować na kolejne wciskanie przycisków.

Oczywiście blokowanie inkrementacji zmiennej lpress w przerwaniu po osiągnięciu maksymalnej wartości 255 też rozwiązuje problem.

Statystyki: Napisane przez janusz17 — 28 kwi 2018, o 07:43

Re: Drobna poprawka do biblioteki MK_MATRIX_KBD

2018-04-27T16:10:16+01:00

no tak RELEASED_MASK

------------------------ [ Dodano po: 8 minutach ]

już poprawiłem na górze ...

janusz17 napisał(a):

wydaje mi się, że w tym miejscu wartość zmiennej lpress nie ma już żadnego znaczenia.

oj ma ma znaczenie ... jak chcesz to usuń ten fragment ale przetestuj później WSZYSTKIE dostępne konfiguracje

Statystyki: Napisane przez mirekk36 — 27 kwi 2018, o 16:10

Re: Drobna poprawka do biblioteki MK_MATRIX_KBD

2018-04-27T08:19:38+01:00

Ale ten zapis , to chyba coś za dużo tych READED_MASK?
[syntax=c]key = READED_MASK | READED_MASK; // <------ tę linię TO TRZEBA ROZSZERZYĆ a nie usunąć[/syntax]

Co do usuwania czegokolwiek, to miałem na myśli ten warunek:
[syntax=c]} else if( lpress ) {[/syntax]

wydaje mi się, że w tym miejscu wartość zmiennej lpress nie ma już żadnego znaczenia.

Statystyki: Napisane przez janusz17 — 27 kwi 2018, o 08:19

Re: Drobna poprawka do biblioteki MK_MATRIX_KBD

2018-04-27T07:52:49+01:00

no i sprawa się wyjaśniła - dzięki za wyłapanie małego BUG'a

Trzeba zrobić DWIE małe poprawki w bibliotece:

[syntax=c]#if ENABLE_ON_KEY_LONG_PRESS_EVENT == 1
if( lpress && lpress < 255 ) lpress++; // <------ tę linię TRZEBA POPRAWIĆ[/syntax]

[syntax=c]#if ENABLE_ON_KEY_LONG_PRESS_EVENT == 1
if( !(key & LONG_PRESS_MASK) && !(key & READED_MASK) ) {
key &= KEY_CODE_MASK;
key |= RELEASED_MASK | READED_MASK;
} else if( lpress ) {
lpress = 0;
key = RELEASED_MASK | READED_MASK; // <------ tę linię TO TRZEBA ROZSZERZYĆ a nie usunąć
}[/syntax]

oczywiście każdy fragment powyższego kodu występuje DWA razy w bibliotece i dwa razy trzeba to poprawić

Statystyki: Napisane przez mirekk36 — 27 kwi 2018, o 07:52

Re: Drobna poprawka do biblioteki MK_MATRIX_KBD

2018-04-27T06:34:35+01:00

hmm muszę sprawdzić bo tak z marszu to nie jestem w stanie powiedzieć ... jak sam widzisz sporo jest różnych zależności w tym kodzie

Statystyki: Napisane przez mirekk36 — 27 kwi 2018, o 06:34

Drobna poprawka do biblioteki MK_MATRIX_KBD

2018-04-27T06:29:30+01:00

Testowałem ostatnio bibliotekę MK_MATRIX_KBD z podłączonym jednym przyciskiem.
Moja konfiguracja jest taka:
[syntax=c]//*****************************************************************************************
//-------------------------------- konfiguracja biblioteki ----------------

//--- wybór portu klawiatury ---
#define KBD_PORTPORTC
#define KBD_PINPINC
#define KBD_DIRDDRC

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

#define ROW_KEYS_COUNT1// 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_MASK0b10000000// 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_TIME150// min = 20, default = 50, MAX = 254!

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

#define USE_ASCII_CODES0

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

Okazało się, że przy włączonej opcji ENABLE_ON_KEY_LONG_PRESS_EVENT w pewnych przypadkach następuje zawieszenie obsługi klawiatury.
Jeśli przycisk wciśniemy na czas potrzebny aby zadziałało zdarzenie od długiego wciśnięcia i zaraz po tym zdarzeniu przycisk zwolnimy, to wszystko jest OK.
Jeśli jednak przycisk trzymamy wciśnięty nadal przez dłuższy czas to po zwolnieniu okazuje się, ze już nie ma dalej żadnej reakcji układu ani na krótkie ani na długie wciśnięcie przycisku. Jest to szczególnie widoczne przy ustawionym długim czasie LONG_PRESS_TIME np. powyżej 200.
Po przeanalizowaniu kodu , doszedłem do wniosku, ze przyczyna tkwi w tej części kodu:
[syntax=c]#if ENABLE_ON_KEY_LONG_PRESS_EVENT == 1
if( !(key & LONG_PRESS_MASK) && !(key & READED_MASK) ) {
key &= KEY_CODE_MASK;
key |= RELEASED_MASK | READED_MASK;// **** ważne
} else if( lpress ) { // jeśli licznik lpress zdąży się wyzerować zanim puścimy przycisk, to ten warunek nigdy się nie wykona
lpress = 0;
key = READED_MASK;
}[/syntax]

Ten warunek if(lpress) jest tutaj niepotrzebny. Zresztą w tej części kodu dla obsługi klawiatur matrycowych, tego warunku nie ma.
Jest za to wcześniej w poprzednim warunku if zerowanie licznika lpress i nie bardzo rozumiem w jakim celu. Moim zdaniem wartość tego licznika w tym miejscu jest nieistotna. A może się mylę?

Statystyki: Napisane przez janusz17 — 27 kwi 2018, o 06:29