W ramach ćwiczeń chciałem sobie dołożyć drugą diodę i przycisk do programu Mirka ale koniec końców ta właśnie dioda cały czas jest zasilona ;/ gdzie tkwi błąd?


Faktycznie poprawiam się już ;p

Panie kolego to nie jest twój pierwszy post na forum, już wielokrotnie moderatorzy i ja także przypominaliśmy tobie i pokazywaliśmy jak się umieszcza kod źródłowy na forum w znacznikach [ syntax = c ] ....

jeśli więc nie zaczniesz tego stosować i nie poprawisz tego co wyżej to przykro mi ale będę usuwał takie posty

_________________
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 ]

Za dużo nawiasów miałeś.

udalo mi sie

chciałbym stan diody drugiej można było zmieniać tylko gdy dioda pierwsza świeci.

wydawało mi się że powinno to być coś takiego

W pętli while zamiast
wstaw
W pierwszym przykładzie zamiast LED_PIN1 preprocesor wstawi 0b10000000. Czyli ten warunek zawsze będzie spełniony i nie będzie to miało żadnego związku z diodą LED1.

A tak poza tym to dlaczego wszędzie używasz typu int zamiast wygodnych typów, które są w gcc np. uint8_t?

z przyzwyczajenia wiem że prawidłowo jest uint8_t

Pomogłeś mi dzięki, ale uwaga
if( !(PINC & LED_PIN1) ) dioda 2 da się włączać i wyłączać gdy 1 jest w stanie Low
dlatego wpisałem if( (PINC & LED_PIN1) ) i teraz działa jak chciałem ale dlaczego?

Gdy ten sam program napisze dla diód i przycisków które znajdują się na rejestrach C to dzieją się cuda

czy to ma związek jakiś z odczytem PINC ?

Jeśli chcesz zmieniać stan drugiej diody tylko wtedy gdy pierwsza świeci to powinieneś napisać takjeśli pierwszą diodę zapalasz stanem niskim, albo takjesli zapalasz ją stanem wysokim.
W pierwszym przypadku warunek jest spełniony jeśli na pinie LED1 jest stan niski, a w drugim - jeśli jest stan wysoki.

Na ATB diody włączane są stanem niskim i mi działa pierwszy przykład.


A jakiego procka używasz? Może używasz takiego, który na porcie C ma JTAG. I dlatego część pinów może Ci nie działać.

wszystko działa, może wczoraj ze zmeczenia coś źle zmieniłem w kodzie nie wiem.
Napisałeś że na ATB diody zmieniane są stanem niskim przecież z kodu jaki wkleiłem a Ty poprawiłeś wynika że musi być sterowane plusem ?

Chciałbym dodać do kodu aby po zgaszeniu diody 1 automatycznie zgasła też dioda druga

wymyśliłem coś takiego ale wyskoczył mi żółty błąd

dodałem
include LED_PIN2OFF

A w którym miejscu napisałem, że trzeba coś tam sterować plusem? Kolega chyba nadal jest zmęczony i nieuważnie czyta.


A zamiast tego powinno być

Próbowałem tego to nie działa teraz dioda zachowuje się tak że gdy D1 jest włączona moge właczyć D2 ale tylko na chwile bo ta zaraz gaśnie.

Założyłem sobie że ma to być tak że po wciśnięciu D1 nie mogę włączyć D2 i dodatkowo jesli włączyłem D1 podczas świecenia D2 to D2 gaśnie. Wydaje mi się że ta definicja musi być bardziej rozbudowana ale nie mam pomysłu jak;/

jakieś sugestie?