ATNEL tech-forum 2012-09-29T16:32:10+01:00 https://forum.atnel.pl/feed.php?f=8&t=1607&mode 2012-09-29T16:32:10+01:00 2012-09-29T16:32:10+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16203#p16203 <![CDATA[Re: PRZYCISKI TIMER]]>
Teraz działa elegancko! Dziękuję.
Chyba już ostatnie pytanie w tym temacie...
Chcę żeby dioda mi się zapaliła gdy trzymam wciśnięty przycisk, a nie dopiero gdy puszczę. Jak tutaj skonstruować warunek? Wrzucałem if'a tam gdzie jest sprawdzane, czy stan pinu jest wyzerowany (zwarcie do masy, ale nie chce to działać jak zwykle).

[syntax=c]if( !(PINB & KEY2))
{
_delay_ms(2);
if( !(PINB & KEY2) && !(TCCR1B & ((1<<CS10)|(1<<CS12))))
{
TCNT1=0;
TCCR1B |= (1<<CS10)|(1<<CS12); //TIMER1 ON - PRESKALER 1024
if( TCNT1>7000)
{
OCR2=0;
}
} [/syntax]

Statystyki: Napisane przez Stanley — 29 wrz 2012, o 16:32

]]> 2012-09-29T16:14:35+01:00 2012-09-29T16:14:35+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16200#p16200 <![CDATA[Re: PRZYCISKI TIMER]]> to akurat rzeczywiście debouncing może odgrywać ogromną rolę.

Statystyki: Napisane przez mirekk36 — 29 wrz 2012, o 16:14

]]> 2012-09-29T16:04:40+01:00 2012-09-29T16:04:40+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16197#p16197 <![CDATA[Re: PRZYCISKI TIMER]]> #include <util/delay.h>


#define LED (1<<PB3)
#define KEY2 (1<<PB0)
#define KEY1 (1<<PB1)

int main(void)
{

TCCR2 |= (1<<WGM21)|(1<<WGM20)|(1<<CS20)|(1<<COM21); //fast pwm, prescaler=1, set at bottom

DDRB |= LED;
PORTB |= KEY1|KEY2;
DDRB &= ~(KEY1|KEY2);
OCR2=255; //LED OFF

while(1)
{

if( !(PINB & KEY2) && !(TCCR1B & ((1<<CS10)|(1<<CS12))))
{
TCNT1=0;
TCCR1B |= (1<<CS10)|(1<<CS12); //TIMER1 ON - PRESKALER 1024
}

else if( (PINB & KEY2) && (TCCR1B & ((1<<CS10)|(1<<CS12))))
{
if( TCNT1>7000)
{
OCR2=0;
}
else
{
OCR2=255;
}
TCCR1B &= ~((1<<CS10)|(1<<CS12)); //TIMER OFF
}



}
}
[/syntax]

Dzięki wielkie za dotychczasową pomoc. :)
Działa to niestabilnie i niedokładnie. Niestabilnie bo raz działa, raz nie. Niedokładnie, bo czas przycisku wciśniętego jest źle liczony. 7000 impulsów to jest 0.9 sekundy, a czasem na zwarcie 2 sekundowe nie ma reakcji, czasem na 3 sekundowe zwarcie dioda tylko mignie.
I nie rozumiem drugiego stanu, gdy TCNT1<7000. Zgodnie z warunkiem dioda powinna zgasnąć dopiero po puszczeniu przycisku, a gaśnie przy wciśniętym.

Statystyki: Napisane przez Stanley — 29 wrz 2012, o 16:04

]]> 2012-09-29T12:44:10+01:00 2012-09-29T12:44:10+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16179#p16179 <![CDATA[Re: PRZYCISKI TIMER]]> Ale nie działa ten kod.. :/
Sam spróbuj:
[syntax=c]#include <avr/io.h>
#include <util/delay.h>


#define LED (1<<PB3)
#define KEY2 (1<<PB0)
#define KEY1 (1<<PB1)

int main(void)
{

TCCR2 |= (1<<WGM21)|(1<<WGM20)|(1<<CS20)|(1<<COM21); //fast pwm, prescaler=1, set at bottom

DDRB |= LED;
PORTB |= KEY1|KEY2;
DDRB &= ~(KEY1|KEY2);
OCR2=255; //LED OFF

while(1)
{
if( !(PINB & KEY2) ) {
TCNT1 = 0; // wyzerowanie timera 1
TCCR1B |= (1<<CS10)|(1<<CS12); //TIMER1 ON - PRESKALER 1024
} else if ( TCCR1B & ((1<<CS10)|(1<<CS12)) ) { // sprawdzamy, czy timer tyka...
TCCR1B &= ~( (1<<CS10)|(1<<CS11)|(1<<CS12) ); //TIMER1 OFF
if( TCNT1<15625 ) { OCR2=0;
// akcja dla krótkiego przytrzymania
} else {OCR2=255;
// akcja dla długiego przytrzymania
}
}





}


}

[/syntax]

Działa tylko jedna opcja, jak u mnie działało na długie przyciśnięcie, tak tutaj działa na krótkie przyciśnięcie. :(

Już mi głowa dymi bo co wymyślę to źle, źle i źle. :(

Myślałem żeby nawet po prostu licznik był ciągle włączony ale zerować go podczas wciśnięcia klawisza i sprawdzać ile impulsów już zostało zliczone:

[syntax=c]
if( !(PINB & KEY2) ) {
TCNT1 = 0; // wyzerowanie timera 1
TCCR1B |= (1<<CS10)|(1<<CS12); //TIMER1 ON - PRESKALER 1024
if(TCNT1>15000) OCR2=0; //zwarcie>2s
if(TCNT1<15000) OCR2=255; //zwarcie<2s
}

[/syntax]

Ale też źle i nie wiem dlaczego to nie działa, mnie się wydaje, że to na moją logikę powinno działać.. Nie umiem myśleć w sposób działający dla procesora.. :/

Statystyki: Napisane przez Stanley — 29 wrz 2012, o 12:44

]]> 2012-09-29T12:09:45+01:00 2012-09-29T12:09:45+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16174#p16174 <![CDATA[Re: PRZYCISKI TIMER]]>
Ja się nie wzbraniam przed pomocą i bardzo dobrze zdaję sobie sprawę że kolega totalnie początkuje, dlatego piszę o błędach w warunkach.

Zatem jeszcze raz powiem - owszem potrenować warunki IF można - ale nadal ja będę się upierał że takie zastosowanie timera nawet 8-bitowego to zbrodnia ;) .... (ale to jest oczywiście moje zdanie i nikomu go nie narzucam .... każdy zrobi jak zechce ... ja tylko podpowiadam że tak się nie powinno robić)

Statystyki: Napisane przez mirekk36 — 29 wrz 2012, o 12:09

]]> 2012-09-29T10:34:14+01:00 2012-09-29T10:34:14+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16169#p16169 <![CDATA[Re: PRZYCISKI TIMER]]> Statystyki: Napisane przez mirekk36 — 29 wrz 2012, o 10:34

]]> 2012-09-29T10:27:20+01:00 2012-09-29T10:27:20+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16168#p16168 <![CDATA[Re: PRZYCISKI TIMER]]> Gwyn, coś nie działa ten kod. Przeanalizowałem go:
[syntax=c]else if ( TCCR1B & ((1<<CS10)|(1<<CS12)) )[/syntax]
ten warunek jest fałszywy, spójrz jeszcze na bit od CTC.

Statystyki: Napisane przez Stanley — 29 wrz 2012, o 10:27

]]> 2012-09-29T10:12:37+01:00 2012-09-29T10:12:37+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16165#p16165 <![CDATA[Re: PRZYCISKI TIMER]]>
toż gdybyś sobie poczytał o podstawach timerów programowych to już dawno dawno byś swój cel osiągnął nie marnując timer1 a wykorzystując jakiś timer 8-bitowy jako podstawę czasu do kilku timerów programowych.

wracając jednak do twojej analizy to pomyśl sobie nad tym jak działają warunki IF bo widzę że nie ogarniasz tego niestety :(

toż gdy w pierwszym warunku wyłączysz timer zerując bity preskalera, to on się zakończy i zacznie się sprawdzanie kolejnego warunku a ponieważ klawisz nadal wciśnięty to znowu preksaler ruszy - no nie widzisz tego ? Dlatego mówię że to masło maślane i nie po to żeby cię urazić czy coś - tylko jak najszybciej odwieść cię od takiego pomysłu.

Statystyki: Napisane przez mirekk36 — 29 wrz 2012, o 10:12

]]> 2012-09-29T09:32:41+01:00 2012-09-29T09:32:41+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16156#p16156 <![CDATA[Re: PRZYCISKI TIMER]]>
Wytłumaczę kod po kolei.
[syntax=c]while(1)
{
if( !(PINB & KEY2) )
{
TCCR1B |= (1<<CS10)|(1<<CS12); //TIMER1 ON - PRESKALER 1024
if( !(PINB & KEY2) && TCNT1<15625) OCR2=255; //Jeżeli przycisk jest zwarty do masy KROCEJ niż 2s - LED OFF
if( OCR2==255) TCCR1B &= ~( (1<<CS10)|(1<<CS11)|(1<<CS12) ); //TIMER1 OFF
}



if( !(PINB & KEY2) )
{
TCCR1B |= (1<<CS10)|(1<<CS12); //TIMER1 ON - PRESKALER 1024
if( !(PINB & KEY2) && TCNT1>15625) OCR2=0; //Jeżeli przycisk jest zwarty do masy DLUZEJ niż 2s - LED ON
if( OCR2==0) TCCR1B &= ~( (1<<CS10)|(1<<CS11)|(1<<CS12) ); //TIMER1 OFF
}





}[/syntax]

Timer1 jest wyłączony (nie zlicza impulsów), gdy przycisk KEY2 jest niewciśnięty zgadza się?
W chwili, gdy przycisk KEY2 zostaje wciśnięty, Timer1 zostaje załączony i zaczyna sobie zliczać impulsy zgadza się?
W związku z tym, że Timer1 zlicza impulsy, ja to wykorzystuję w kolejnym warunku i zawczasu obliczyłem, że przy preskalerze 1024 i F_CPU=8MHz otrzymuję ostatecznie sygnał o częstotliwości 78125,5Hz, zgadza się?
Teraz powstaje pytanie ile impulsów musi zliczyć Timer1 aby trwało to 2s, bo taką zwłokę akurat chcę mieć przy długim wciśnięciu klawisza.
2 * 7812,5 = 15625 zgadza się?
I tą wartość wykorzystuję w kolejnym warunku: Jeżeli przycisk KEY2 jest wciśnięty i Timer1 zliczył mniej niż 15625 impulsów, czyli w sumie czas trwania wduszonego przycisku wynosi mniej niż 2 sekundy, to wtedy WYŁĄCZ LED zgadza się?
Ostatni warunek: Jeżeli LED WYŁĄCZONY ---> WYŁĄCZ Timer1 zgadza się?

Druga część w pętli głównej to jest wszystko to samo z dwoma różnicami:
Jeżeli przycisk KEY2 jest wciśnięty i Timer1 zliczył więcej niż 15625 impulsów, czyli w sumie czas trwania wduszonego przycisku wynosi więcej niż 2 sekundy, to wtedy ZAŁĄCZ LED zgadza się?
Ostatni warunek: Jeżeli LED ZAŁĄCZONY ---> WYŁĄCZ Timer1 zgadza się?

Statystyki: Napisane przez Stanley — 29 wrz 2012, o 09:32

]]> 2012-09-29T01:11:05+01:00 2012-09-29T01:11:05+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16146#p16146 <![CDATA[Re: PRZYCISKI TIMER]]>

zwróć tylko uwagę że wszystkie warunki są praktycznie takie same - totalne masło maślane :(

Statystyki: Napisane przez mirekk36 — 29 wrz 2012, o 01:11

]]> 2012-09-29T00:41:34+01:00 2012-09-29T00:41:34+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16145#p16145 <![CDATA[Re: PRZYCISKI TIMER]]>
Ale jeszcze nie chodzi tak jak należy:
[syntax=c]#include <avr/io.h>
#include <util/delay.h>


#define LED (1<<PB3)
#define KEY2 (1<<PB0)
#define KEY1 (1<<PB1)

int main(void)
{

TCCR2 |= (1<<WGM21)|(1<<WGM20)|(1<<CS20)|(1<<COM21); //fast pwm, prescaler=1, set at bottom

TCCR1B |= (1<<WGM12); //TRYB CTC
TCCR1B &= ~( (1<<CS10)|(1<<CS11)|(1<<CS12) ); //TIMER OFF
OCR1A = 65535; //clear at compare match


DDRB |= LED;
PORTB |= KEY1|KEY2;
DDRB &= ~(KEY1|KEY2);
OCR2=255; //LED OFF


while(1)
{
if( !(PINB & KEY2) )
{
TCCR1B |= (1<<CS10)|(1<<CS12); //TIMER1 ON - PRESKALER 1024
if( !(PINB & KEY2) && TCNT1<15625) OCR2=255; //Jeżeli przycisk jest zwarty do masy KROCEJ niż 2s - LED OFF
if( OCR2==255) TCCR1B &= ~( (1<<CS10)|(1<<CS11)|(1<<CS12) ); //TIMER1 OFF
}



if( !(PINB & KEY2) )
{
TCCR1B |= (1<<CS10)|(1<<CS12); //TIMER1 ON - PRESKALER 1024
if( !(PINB & KEY2) && TCNT1>15625) OCR2=0; //Jeżeli przycisk jest zwarty do masy DLUZEJ niż 2s - LED ON
if( OCR2==0) TCCR1B &= ~( (1<<CS10)|(1<<CS11)|(1<<CS12) ); //TIMER1 OFF
}





}


}
[/syntax]

Pomysł nadaje się Twoim zdaniem?
Działa tylko jedna opcja na długie przyciśnięcie klawisza. Na krótkie przyciśnięcie nie działa. Nie rozumiem dlaczego akurat na dłuższe przyciśnięcie działa, a na krótsze już nie skoro warunki są niemalże identyczne tylko ze zmienionymi wartościami impulsów.

Statystyki: Napisane przez Stanley — 29 wrz 2012, o 00:41

]]> 2012-09-29T00:15:13+01:00 2012-09-29T00:15:13+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16144#p16144 <![CDATA[Re: PRZYCISKI TIMER]]>
ale to to raczej jest tylko twoje pobożne życzenie a nie warunek, dokładniej mówiąc to zadziała tak:

jeśli klawisz wciśnięty i rejestr OCR1A (który nawiasem mówiąc nie ma NIC WSPÓLNEGO z tykającym rejestrem TCNT1 timera1) jest mniejszy niż 15625 - a pewnie jest - to warunek zawsze jest spełniony gdy tylko wciśniesz klawisz.

Ale żeby ci zaraz nie przyszedł inny pomysł do głowy np:

if( !(PINB & KEY2) && TCNT1<15625) OCR2=0;

bo tuż po wystartowaniu timera i tak zwykle TCNT1 będzie mniejsze niż 15625 - więc znowu zawsze warunek będzie spełniony gdy tylko klawisz będzie wciśnięty - od razu

nie wiem nawet tzn nie potrafię sobie odwzorować w głowie co ty chcesz tą drugą częścią warunku osiągnąć ??? pomyśl dokładnie jak to działa

Statystyki: Napisane przez mirekk36 — 29 wrz 2012, o 00:15

]]> 2012-09-28T23:11:49+01:00 2012-09-28T23:11:49+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16142#p16142 <![CDATA[Re: PRZYCISKI TIMER]]> Statystyki: Napisane przez Stanley — 28 wrz 2012, o 23:11

]]> 2012-09-28T18:33:37+01:00 2012-09-28T18:33:37+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16116#p16116 <![CDATA[Re: PRZYCISKI TIMER]]> Statystyki: Napisane przez Stanley — 28 wrz 2012, o 18:33

]]> 2012-09-28T17:45:34+01:00 2012-09-28T17:45:34+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16111#p16111 <![CDATA[Re: PRZYCISKI TIMER]]>
post16079.html#p16079

Statystyki: Napisane przez mirekk36 — 28 wrz 2012, o 17:45

]]> 2012-09-28T16:57:52+01:00 2012-09-28T16:57:52+01:00 https://forum.atnel.pl/viewtopic.php?t=1607&p=16106#p16106 <![CDATA[PRZYCISKI TIMER]]> Napisałem wstępnie coś takiego:
[syntax=c]#include <avr/io.h>
#include <util/delay.h>


#define LED (1<<PB3)
#define KEY2 (1<<PB0)
#define KEY1 (1<<PB1)

int main(void)
{

TCCR2 |= (1<<WGM21)|(1<<WGM20)|(1<<CS20)|(1<<COM21); //fast pwm, prescaler=1, set at bottom

TCCR1B |= (1<<WGM12); //TRYB CTC
TCCR1B &= ~( (1<<CS10)|(1<<CS11)|(1<<CS12) ); //TIMER OFF
OCR1A = 65535; //clear at compare match


DDRB |= LED;
PORTB |= KEY1|KEY2;
DDRB &= ~(KEY1|KEY2);
OCR2=255; //LED OFF


while(1)
{
if( !(PINB & KEY2) )
{
TCCR1B |= (1<<CS10)|(1<<CS12); //TIMER ON - PRESKALER 1024
if( !(PINB & KEY2) && OCR1A<15625) OCR2=0; //Jeżeli przycisk jest zwarty do masy krócej niż 2s - LED ON
}

}


}[/syntax]

Dlaczego to nie działa? Coś nie tak robię z Timerem1, ponieważ gdy usunę z ostatniego warunku OCR1A<15625 - to dioda normalnie reaguje na przycisk sterowany przez PWM.

Statystyki: Napisane przez Stanley — 28 wrz 2012, o 16:57

]]>