ATNEL tech-forum

Re: Lampki choinkowe

2016-01-31T20:34:36+01:00

Dziękuje za pomoc:) Udało się zrobić ze zrozumieniem.

Statystyki: Napisane przez elektronikq — 31 sty 2016, o 20:34

Re: Lampki choinkowe

2016-01-27T00:09:32+01:00

W każdym obiegu pętli while tworzysz dwie zmienne flaga i flaga2. Zmienne te są zmiennymi lokalnymi bo są tworzone wewnątrz funkcji main. Zmienne lokalne tworzone są na stosie i jeśli nie przypiszesz im jakiś konkretnych wartości to wtedy będą miały przypadkowe wartości.

Kompilator ostrzega, że próbujesz użyć zmiennej lokalnej bez jej wcześniejszej inicjalizacji. Podczas tworzenia tych zmiennych przypisz im np. zerowe wartości.

I przenieś je poza while do funkcji main. Bo teraz to niepotrzebnie z każdym obiegiem pętli są wciąz tworzone na nowo.

------------------------ [ Dodano po: 10 minutach ]

Chodzi ci o to, że ten fragment ci nie działa?
[syntax=c]if (timer==5 && flaga==1)
{
PORTC |= LED;
PORTC &= ~LED1;
}[/syntax]
Tak napisałeś program, że nie będzie działał.

No popatrz. Na początku zmienna keylock ma wartość równą zero. Po wciśnięciu klawisza ten warunek zostaje spełniony
[syntax=c]if(!keylock && !(PINC&KEY_DOWN)) //obsługa przycisku[/syntax]

W jego wnętrzu, zmiennej keylock przypisujesz wartość jeden. W każdym następnym obiegu pętli while ten warunek już nie będzie spełniony. Bo wartość keylock jest różna od zera. Więc jak ma się wykonać to co jest we wnętrzu tego warunku.

Statystyki: Napisane przez jacekk232 — 27 sty 2016, o 00:09

Re: Lampki choinkowe

2016-01-27T00:15:18+01:00

Poprawiłem to ale dalej mi to nie działa więc zrobiłem jedną lampke:
[syntax=c]#include
#include
#define KEY_DOWN (1<#define LED (1<#define LED1 (1< volatile uint8_t timer;
uint8_t keylock;
int main (void)
{
DDRC |= LED;
PORTC |= KEY_DOWN|LED;
TCCR1B |= (1< TCCR1B |= (1< OCR1A = 7811; //1 na sekundę
TIMSK |= (1< sei();
while(1)
{
uint8_t flaga;
if(!keylock && !(PINC&KEY_DOWN)) //obsługa przycisku
{
keylock=1;
flaga=1;
PORTC &= ~LED;
if (timer==5 && flaga==1)
{
PORTC |= LED;
PORTC &= ~LED1;
}

}
else if (keylock && (PINC&KEY_DOWN)) keylock++;
}
}
ISR(TIMER1_COMPA_vect)
{
timer++;
if (timer>5)timer=0;
}[/syntax]

Też nie działa. Chyba z timerem coś pomieszałem. Mógłby ktoś zerknąć. W OCR1A wyszło mi 7811,5 ale wstawiłem 7811 czy to przez to?

Statystyki: Napisane przez elektronikq — 27 sty 2016, o 00:04

Re: Lampki choinkowe

2016-01-26T22:58:30+01:00

Faktycznie ;( dziękuje za pomoc.
A taki błąd co oznacza?

Chyba z tymi flagami coś nie tak. Jak już warunek jest spełniony to ja chce je wyzerować na tym polega błąd?

Statystyki: Napisane przez elektronikq — 26 sty 2016, o 22:58

Re: Lampki choinkowe

2016-01-26T22:49:31+01:00

Sprawdź dokładnie kod a szczególnie nawiasy klamrowe. Taki błąd się pojawia jeśli funkcję obsługi przerwania umieści się np. wewnątrz funkcji main a nie poza nią.

W przedstawionym kodzie tak właśnie jest.

Statystyki: Napisane przez jacekk232 — 26 sty 2016, o 22:49

Re: Lampki choinkowe

2016-01-26T22:18:06+01:00

Oscylator wewnętrzny 8MHz.

[syntax=c]#include
#include
#define KEY_DOWN (1< #define LED (1<#define KEY_DOWN (1<#define LED (1<#define LED1 (1<#define LED2 (1< volatile uint8_t timer;
uint8_t keylock;
int main (void)
{
DDRC |= LED;
PORTC |= KEY_DOWN|LED;
TCCR1B |= (1< TCCR1B |= (1< OCR1A = 7811; //1 na sekundę
TIMSK |= (1< sei();
while(1)
{
uint8_t flaga, flaga2;
if(!keylock && !(PINC&KEY_DOWN)) //obsługa przycisku
{
keylock=1;
PORTC &= ~LED;

if (timer>5)
{
PORTC |= LED;
PORTC &= ~LED1;
flaga=1;
}
else if (timer>10 && flaga==1)
{
PORTC |= LED1;
PORTC &= ~LED2;
flaga=0;
flaga2=1;
}
else if (timer>15 && flaga2==1)
{
PORTC |= LED2;
flaga2=0;
}

}
else if (keylock && (PINC&KEY_DOWN)) keylock++;
}

ISR(TIMER1_COMPA_vect)
{
timer++;
if (timer==12)timer=0;
}[/syntax]

I tak zrobiłem, że każda się pali po 10 sek. Ale jak miałem ~32 200 w OCR1A to wyrzucił mi błąd. Ale jak wpisałem 7811 to jest ok, nie wiem dlaczego krzyczał dla 32 200.

Tak jak rozumiem tak zrobiłem, nie posiłkuje się na razie żadnym z programów kolegów bo chciałem zobaczyć czy sam dam radę. Niestety cała linijka od przerwania:[syntax=c]ISR(TIMER1_COMPA_vect)
{
timer++;
if (timer==12)timer=0;
}[/syntax]
podkreślona na czerwono

Statystyki: Napisane przez elektronikq — 26 sty 2016, o 22:18

Re: Lampki choinkowe

2016-01-25T11:26:13+01:00

[syntax=c]if (timer=1&&flaga=1) //??????????
{
PORTC |= LED;
PORTC &= ~LED1;
}

//chyba powinno być tak:

if (((timer == 1) && (flaga == 1)))
{
PORTC |= LED;
PORTC &= ~LED1;
}[/syntax]

Statystyki: Napisane przez Jarecki — 25 sty 2016, o 11:26

Re: Lampki choinkowe

2016-01-25T08:54:09+01:00

Hmm, sciagnij sobie mkavr-calkulator, tam masz zakladke do timerow.

Co do rezonatora, masz 8Mhz, a mówisz ze nie masz..znaczy masz włączony wewnętrzny oscylator na 1MHz?
Jak masz ustawione fuse bity?

Program powinien wygladac tak:

main:
>ustawienie portow i timera
>while
-sprawdzasz przerwanie
-migasz ledka

vector przerwania poza main...

Taki przyklad:
[syntax=c]#include
#include

volatile uint8_t count;

void main()
{
// Prescaler = FCPU/1024
TCCR0|=(1<
//Enable Overflow Interrupt Enable
TIMSK|=(1<
//Initialize Counter
TCNT0=0;

//Initialize our varriable
count=0;

//Port C[3,2,1,0] as out put
DDRC|=0x0F;

//Enable Global Interrupts
sei();

//Infinite loop
while(1);
}

ISR(TIMER0_OVF_vect)
{
//This is the interrupt service routine for TIMER0 OVERFLOW Interrupt.
//CPU automatically call this when TIMER0 overflows.

//Increment our variable
count++;
if(count==61)
{
PORTC=~PORTC; //Invert the Value of PORTC
count=0;
}
}[/syntax]

Statystyki: Napisane przez wchpikus — 25 sty 2016, o 08:54

Re: Lampki choinkowe

2016-01-25T00:33:28+01:00

Atmega16, bez rezonatora. 8000 000Mhz.

[syntax=c]#include
#include
#define KEY_DOWN (1<#define LED (1<volatile uint8_t timer, flaga;
uint8_t keylock;
int main(void)
DDRC |= LED;
PORTC |= KEY_DOWN|LED;
TCCR1B |= (1< TCCR1B |= (1< OCR1A = 31 249; //1 na sekundę
TIMSK |= (1< while(1)
{
if(!keylock && !(PINC&KEY_DOWN)) // tu musisz dać obsługę przycisku
{
keylock=1;
timer=1;
PORTC &= ~LED;

if (timer=1&&flaga=1)
{
PORTC |= LED;
PORTC &= ~LED1;
}

}
}
else if (keylock && (PINC&KEY_DOWN)) keylock++;

ISR(TIMER1_COMPA_vect)
{
if (timer=1)
{
for(timer=1, timer<6, timer++)
flaga=1;
}[/syntax]

W dobrym kierunku idę?

Statystyki: Napisane przez elektronikq — 25 sty 2016, o 00:33

Re: Lampki choinkowe

2016-01-24T23:27:03+01:00

Jeden timer wystarczy.
Jaki uc i jaki rezonator?
Musisz przeliczyc i ustawic timer by miec 1 sek.
W przerwaniu zliczasz i co sekunde zwiekszasz jakas zmienna np sek.
W petli while juz prosto, po sek sprawdzasz i wykonujesz co chcesz.

Statystyki: Napisane przez wchpikus — 24 sty 2016, o 23:27

Lampki choinkowe

2016-01-24T23:20:30+01:00

Chciałbym, zrobić lampki choinkowe. Nie wiem ile timerów musze wykorzystać. Nie bardzo to jeszczee wiem jak rozumieć. Bo timery są w przerwaniach nie mam tak, że ustawiam timer 10 sek np. delay_on i mi zapala za 10 sek i z tym mam problem.
Chce zrobić aby zapalały się 4 lampki. Jedna 5 sekund gaśnie i zapala się druga na 10 gaśnie, 3 na 15 i gaśnie i 4 na 20 i gaśnie.
[syntax=c]#include
#include
#define KEY_DOWN (1<#define LED (1<#define LED1 (1<#define LED2 (1<#define LED3 (1<volatile uint16_t timer;
timer=10; //będziemy odliczać 10sek
uint8_t keylock, flaga;
int main(void)
DDRC |= LED;
PORTC |= KEY_DOWN|LED|LED1|LED2|LED3;
TCCR1B |= (1< TCCR1B |= (1< OCR1A = 30; //1000 na sekundę
TIMSK |= (1< while(1)
{
if(!keylock && !(PINC&KEY_DOWN)) // tu musisz dać obsługę przycisku
{
keylock=1;
PORTC &= ~LED;
flaga=1;
if (flaga=1)
{

}

}
else if (keylock && (PINC&KEY_DOWN)) keylock++;
ISR(TIMER1_COMPA_vect)
{

}[/syntax]

Nie wiem jak to ogarnać:(

Statystyki: Napisane przez elektronikq — 24 sty 2016, o 23:20