Próbuję napisać program do skanowania klawiatury matrycowej 4x3. Korzystam z Atmega8A taktowana wewnętrzym oscylatorem 8MHz. Program ma zmienić stan pinu do którego podłączona jest dioda LEDx po naciśnięciu przycisku o numerze x.
Problem polega na tym że działa mi tylko pierwsza kolumna klawiatury, proszę o pomoc, wskazówkę, trop.
Oto mój kod:

Niech kolega popatrzy na ten sposób obsługi

_________________
sig off ;(

Dziękuje za poradę ale wolałbym na początek bez przerwań/liczników itp.
Mógłby ktoś zerknąć okiem na kod, naprowadzić lub wskazać blędy?

Bez przerwań też się da Można ten multipleks zrobić w pętli for. W sumie nie tylko

_________________
sig off ;(

Wydaje mi się, że błąd masz w tej części

Bo najpierw piny kolumn mają ustawiony stan wysoki. Ale po przekręceniu się pierwszej pętli for na tych pinach jest już stan niski, który już wciąż się na nich utrzymuje. Poza tym podczas odczytu wierszy stan niski powinien być tylko na jednej kolumnie.

Dlatego spróbuj może na początku pierwszej pętli for ustawiać na pinach kolumn stan wysoki po to żeby potem po ustawieniu na danej kolumnie stanu niskiego, ten stan był tylko na tej jednej kolumnie.

Wprowadziłem poprawkę, jednak nadal działa tylko pierwsza kolumna, czyli funkcja zwraca mi tylko 1,4 lub 7

Przerobiłem troszkę ten program i jakoś działa.

Jednak ja bym zrobił skanowanie klawiatury w przerwaniu od jakiegoś timera co kilkanaście lub kilkadziesiąt ms.


A tak poza tym to w ten sposób nie zeruje się bitów tylko w ten

I po co do funkcji skan_klawiatury przekazujesz tyle argumentów skoro wszystkie argumenty to stałe dosłowne i w każdym wywołaniu tej funkcji są takie same. W tym przypadku można by zrobić tak i też będzie działać

Chyba, że kolega ma jakiś inny plan co do tej funkcji i tylu argumentów.

Tam wystarczy przerzucić "return" w funkcji skan_klawiatury przed ostatni nawias klamrowy i działa, przynajmniej u mnie.

Ale nie do końca bo cały czas mruga dioda pod PD7.