Testowałem ostatnio bibliotekę MK_MATRIX_KBD z podłączonym jednym przyciskiem.
Moja konfiguracja jest taka:


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:


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ę?

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

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]

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

Trzeba zrobić DWIE małe poprawki w bibliotece:








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

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]

Ale ten zapis , to chyba coś za dużo tych READED_MASK?


Co do usuwania czegokolwiek, to miałem na myśli ten warunek:


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

no tak RELEASED_MASK

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

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



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

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]

Ja zrobiłem tylko taką modyfikację w kodzie dla klawiatur jednorzędowych:


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:

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.

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

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]