ATNEL tech-forum 2015-04-18T14:26:14+01:00 https://forum.atnel.pl/feed.php?f=23&t=2875&mode 2015-04-18T14:26:14+01:00 2015-04-18T14:26:14+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=126846#p126846 <![CDATA[Re: Prosty miernik częstotliwości]]> Mam nadzieję że jeszcze ktoś tu zagląda i może mi coś podpowie.

Sprawdzałem wszystko na 1000 sposobów i problem leży w przerwaniu z Tim1. Odkryłem to kiedy wrzuciłem do instrukcji przerwania PORTD ^= (1<<1); i _delay_ms żeby sprawdzić za pomocą diody czy to przerwanie się wykonuje no i niestety nie.

Pacjent Atmega 88P 16 MHz
Do sprawdzenia podaję syg z PWM z pinu atmegi albo z zewnętrznego źródła, np czujnik światła generujący impuls prostokątny.

Czuję że błąd jest banalny, a ja już się zakręciłem w poszukiwaniu tego.

[syntax=c]#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include "lcd/hd44780.h"

volatile uint16_t ilosc_impulsow;
volatile uint16_t przepelnienia_zegara;
volatile uint8_t flaga_zegara;
// zmienne do obliczeń
volatile int czestotliwosc;
volatile int reszta;

void timer_init();

ISR(TIMER1_OVF_vect)
{
ilosc_impulsow++;
PORTD ^= (1<<1);
_delay_ms(200);
}

ISR(TIMER0_OVF_vect)
{
przepelnienia_zegara++;
if (przepelnienia_zegara ==61) // odmierzanie jednej sekundy
{
PORTD ^= (1<<0);
flaga_zegara = 1;
przepelnienia_zegara = 0;
}
}

int main(void)
{
PORTB |= (1<<0);
DDRD |= (1<<3)|(1<<0)|(1<<1);

sei();// włączamy globalny wektor przerwań


timer_init();
lcd_init();// ustawianie lcd
lcd_cls();
lcd_locate(0,0);
lcd_str("FREQ F[Hz]");


while(1)
{
if(flaga_zegara == 1)
{

TCNT1 = reszta;
TCNT1 = 0;
czestotliwosc = ilosc_impulsow * 65536;//liczba złapanych impulsów * pojemnosc timera1
czestotliwosc = czestotliwosc + reszta;// reszta to wartośc licznika T1 w chwili zrobienia
//pomiaru

//wyswietlanie wyniku
lcd_str(" ");
lcd_locate(1,0);
lcd_int(czestotliwosc);
lcd_locate(1,7);
lcd_int(ilosc_impulsow);

flaga_zegara = 0;//zerowanie flagi zegara
ilosc_impulsow = 0;
//PORTD ^= (1<<1);
}

}

}

void timer_init(void)
{//Timer do zegara
TCCR0B |= (1<<CS00)|(1<<CS02);//Prescaler 1024 Timer0 czyli zegar
TIMSK0 |= (1<<TOIE0);//Timer overflow interrupt eneble

// licznik
TCCR1B |= (1<<CS10)|(1<<CS12)|(1<<CS11)|(1<<ICES1);//wyzwalany z zewnątrz rissing edge Timer 1 licznik impulsow
TIMSK1 |= (1<<TOIE1);//Timer/Counter1, Overflow Interrupt Enable


//Timer do PWM
TCCR2A |= (1<<WGM21)|(1<<WGM20)|(1<<COM2B1)|(1<<COM2B0);
TCCR2B |= (1<<CS22); //Prescaler 64
OCR2B = 50;
}[/syntax]

Czekam na krytykę i podpowiedzi.
Pozdrawiam serdecznie.

Statystyki: Napisane przez mk94 — 18 kwi 2015, o 14:26

]]> 2013-05-02T10:12:25+01:00 2013-05-02T10:12:25+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=35865#p35865 <![CDATA[Re: Prosty miernik częstotliwości]]> matisek1233 napisał(a):

karte do klocka
:lol:

Przez głośniki aktywne ustawione na maxa- z wyjścia na słuchawki poprzez rezystor 470R i dwie diody jedna do masy druga do Vcc.

Przy testowaniu kwarców generator był zrealizowany na bramce NAND 4093 plus pozostałe jako bufor- przy zasilaniu takim jak procesor "po drodze" już nic nie wstawiałem.


Co ciekawe, przy kwarcu 4,000 wynik na wyświetlaczu to 3,998 czyli błąd procentowy na poziomie 0,05% :)

Statystyki: Napisane przez janeczqu — 2 maja 2013, o 10:12

]]> 2013-05-02T08:49:25+01:00 2013-05-02T08:49:25+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=35859#p35859 <![CDATA[Re: Prosty miernik częstotliwości]]> Statystyki: Napisane przez matisek1233 — 2 maja 2013, o 08:49

]]> 2013-04-17T18:53:13+01:00 2013-04-17T18:53:13+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=34556#p34556 <![CDATA[Re: Prosty miernik częstotliwości]]> * main1.c
*
* Created on: 2013-04-16
* Autor: janeczqu
*v3
*lcd 2x16
*/
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include "lcd44780.h"

volatile uint16_t licznik ; //liczba przepełnień Timera1
volatile uint16_t reszta ;//reszta impulsów przed przepełnieniem Timera1 =1;
volatile uint32_t czestotliwosc ;//mierzona czestotliwosc
volatile uint8_t przepelnienie =61; //liczba przerwań Timera0 w ciągu 1s
volatile uint8_t sekunda_flaga;

volatile uint16_t tysiace;
volatile uint16_t jednosci;
volatile uint8_t zakres;

int main(void)
{
DDRB &= ~(1<<PB1);
PORTB |= (1<<PB1); //PB1 wejście podciagnięte do plusa

//Timer0 zegar
TCCR0 |= (1<<CS02)|(1<<CS00); // preskaler = 1024
TIMSK |= (1<<TOIE0) | (1<<TOIE1); //przerwanie na przepełnienie

//Timer1 licznik
TCCR1B |= (1<< CS12) |(1<< CS11); //zrodło zewnętrzne - zbocze opadające
TIMSK |= (1<<TOIE1); //przepełnienie

lcd_init(); //inicjacja
lcd_cls(); //czyszczeine
lcd_locate(0,0);
lcd_str("Miernik");
lcd_locate(1,2);
lcd_str("czestotliwosci"); //miejsce na reklame ;)
_delay_ms(1000);
lcd_locate (0,0);
lcd_str("F-meter");
lcd_locate (1,0);
lcd_str("F= ");

sei(); //wlaczenie przerwan

while(1)
{
if (sekunda_flaga==1)
{
sekunda_flaga=0;
reszta = TCNT1; //pobieranie wartości z licznika
TCNT1=0; //zerowanie licznika
czestotliwosc = licznik * 65536; //liczba przepełnień Timera1* pojemność Timera1
czestotliwosc = czestotliwosc + reszta; //obliczam częstotliwosc
licznik = 0;
lcd_locate (1,0);
lcd_str(" ");
if (czestotliwosc<=1000)
{
lcd_locate (1,0);
lcd_str("F=");
lcd_int(czestotliwosc);
lcd_str("Hz");
}
else
{
if (czestotliwosc>100000)
{
czestotliwosc=czestotliwosc/1000;
zakres=1;
}
else zakres=0;
tysiace=czestotliwosc/1000;
jednosci=czestotliwosc%1000;

lcd_locate (1,0);
lcd_str("F=");
lcd_int(tysiace);
lcd_str(",");
if (jednosci<100)
lcd_str("0");
if (jednosci<10)
lcd_str("0");
lcd_int(jednosci);
if (zakres==0)
{
lcd_str("kHz");
}
else
{
lcd_str("MHz");
}
}
}

}
}
ISR(TIMER0_OVF_vect) //odmierzanie 1 sek
{
przepelnienie--;
if (przepelnienie == 0)
{
sekunda_flaga=1;
przepelnienie = 61;
}
}
ISR(TIMER1_OVF_vect)
{
licznik++;
}[/syntax]


Po poprawkach.

Statystyki: Napisane przez janeczqu — 17 kwi 2013, o 18:53

]]> 2013-04-16T20:58:15+01:00 2013-04-16T20:58:15+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=34507#p34507 <![CDATA[Re: Prosty miernik częstotliwości]]> gufim napisał(a):

akbyś troche rozszeżył możliwości biblioteki od wyświetlacza i dopisał sobie funkcje wyświetlającą uint32_t było by O.K
to jeszcze nie mój lvl -próbowałem ale na razie odpuszczam, zrobię to w inny sposób, nawet czytelność się zwiększy ;)
Ale to zapewne jutro dopiero ujrzy światło dzienne .

Statystyki: Napisane przez janeczqu — 16 kwi 2013, o 20:58

]]> 2013-04-16T20:43:26+01:00 2013-04-16T20:43:26+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=34505#p34505 <![CDATA[Re: Prosty miernik częstotliwości]]> Statystyki: Napisane przez gufim — 16 kwi 2013, o 20:43

]]> 2013-04-16T20:36:01+01:00 2013-04-16T20:36:01+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=34503#p34503 <![CDATA[Re: Prosty miernik częstotliwości]]> masz rację- tak to jest jak się nie przetestuje wszystkich parametrów...

Dzięki za zwrócenie na to uwagi, popracuję nad tym ;)

Statystyki: Napisane przez janeczqu — 16 kwi 2013, o 20:36

]]> 2013-04-16T19:50:34+01:00 2013-04-16T19:50:34+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=34492#p34492 <![CDATA[Re: Prosty miernik częstotliwości]]> [syntax=c]volatile uint32_t czestotliwosc ; //mierzona czestotliwosc[/syntax]
a wyświetlasz funkcją
[syntax=c]lcd_int(czestotliwosc);[/syntax]
i tak chyba mi sie wydaje że coś tu nie tak jest gdyż uint32_t to troche wiecej niż int chyba że się myle.

Statystyki: Napisane przez gufim — 16 kwi 2013, o 19:50

]]> 2013-04-16T08:40:31+01:00 2013-04-16T08:40:31+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=34428#p34428 <![CDATA[Re: Prosty miernik częstotliwości]]>
Możesz dodać kilka linijek aby program był bardziej uniwersalny. Np. tak:
[syntax=c]// Zakomentuj wyświetlacz którego nie posiadasz:
//#define WYSW_2X16
#define WYSW 4X20

#ifdef WYSW_2X16
... kod obsługujący wyświetlacz 2x16
#else
... kod obsługujący większy wyświetlacz
#endif[/syntax]

Ale to już rozwinięcie programu. Do tego co jest teraz nie mam więcej zastrzeżeń.

Statystyki: Napisane przez Domanoid — 16 kwi 2013, o 08:40

]]> 2013-04-16T08:11:05+01:00 2013-04-16T08:11:05+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=34427#p34427 <![CDATA[Re: Prosty miernik częstotliwości]]> [syntax=c]*
* main1.c
*
* Created on: 2013-04-16
* Autor: janeczqu
*v2
*/
#include <avr/io.h>
#include <avr/interrupt.h>
#include "lcd44780.h"

volatile uint16_t licznik ; //liczba przepełnień Timera1
volatile uint16_t reszta ;//reszta impulsów przed przepełnieniem Timera1 =1;
volatile uint32_t czestotliwosc ;//mierzona czestotliwosc
volatile uint8_t przepelnienie =61; //liczba przerwań Timera0 w ciągu 1s
volatile uint8_t sekunda_flaga;
int main(void)
{
DDRB &= ~(1<<PB1);
PORTB |= (1<<PB1);//PB1 wejście podciagnięte do plusa

//Timer0 zegar
TCCR0 |= (1<<CS02)|(1<<CS00); // preskaler = 1024
TIMSK |= (1<<TOIE0) | (1<<TOIE1); //przerwanie na przepełnienie

//Timer1 licznik
TCCR1B |= (1<< CS12) |(1<< CS11); //zrodło zewnętrzne - zbocze opadające
TIMSK |= (1<<TOIE1); //przepełnienie
sei();//wlaczenie przerwan

lcd_init();//inicjacja
lcd_cls();//czyszczeine
lcd_locate(0,0);
lcd_str("Miernik");//"oprawa graficzna ;)
lcd_locate(1,2);
lcd_str("czestotliwosci");
lcd_locate(2,0);
lcd_str(" janeczqu Atnel ");
lcd_locate(3,0);
lcd_str ("F[Hz]=");
while(1)
{
if (sekunda_flaga==1)
{
reszta = TCNT1; //pobieranie wartości z licznika
TCNT1=0; //zerowanie licznika
czestotliwosc = licznik * 65536; //liczba przepełnień Timera1* pojemność Timera1
czestotliwosc = czestotliwosc + reszta;//obliczam częstotliwosc
licznik = 0;
lcd_locate(3,6);
lcd_str(" ");//wymazywanie wczesniejszego wyniku
lcd_locate(3,6);
lcd_int(czestotliwosc);
sekunda_flaga=0;
}

}
}
ISR(TIMER0_OVF_vect) //odmierzanie 1 sek
{
przepelnienie--;
if (przepelnienie == 0)
{
sekunda_flaga=1;
przepelnienie = 61;
}
}
ISR(TIMER1_OVF_vect)
{
licznik++;
}[/syntax]

To mnie Mirku zabiłeś z tymi włączeniami i wyłączeniami przerwań ;) Nie miałem o tym pojęcia nawet.
Inne rzeczy też poprawiłem- teraz będę walczył z bardziej "ludzkim" wyświetlaniem częstotliwości.

mirekk36 napisał(a):

na zakończenie - to co napisałem, to nie żadna tam jakaś kocia krytyka, tylko takie hinty (podpowiedzi) z mojej strony - mam nadzieję, że w jakimś stopniu przydadzą się, i życzę powodzenia w dalszym kodowaniu

Ależ nawet konstruktywna krytyka jest mile widziana, a uwagi "wklepuję" w elementarz ;)

Statystyki: Napisane przez janeczqu — 16 kwi 2013, o 08:11

]]> 2013-04-16T06:40:57+01:00 2013-04-16T06:40:57+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=34423#p34423 <![CDATA[Re: Prosty miernik częstotliwości]]>
1. wyłączenie przerwań cli(); PODCZAS rozpoczęcia procedury obsługi przerwania jest ZAWSZE robione automatycznie przez procesor AVR - a zatem nie rób tego w kodzie przerwania

2. włączenie przerwań sei(); na zakończenie procedury obsługi przerwania jest robione AUTOMATYCZNIE przez procesor, więc tego też nie rób w kodzie przerwania. Przy czym o ile powyższe cli(); z pkt.1 akurat niczemu nie zaszkodzi - ot po prostu zbędny rozkaz, to już twój sei() pod koniec przerwania może czasem przynieść sieczkę w niektórych programach, płacz i zgrzytanie zębów - ponieważ odpalasz sei() przed epilogiem przerwania. Musisz bowiem pamiętać że makro przerwania ISR() wygląda mniej więcej tak

[syntax=c]ISR( vect ) {
//..... prolog ......

// .... twój kod obsł. przerwania

// .... epilog .......
}[/syntax]

przy czym prolog i epilog składający się z rozkazów push i pop w asemblerze dodaje sam kompilator, a więc widzisz teraz że twoje sei() odpala się przez epilogiem i może narazić np stos procka na pokrębasienie ;)

------------------------ [ Dodano po: 3 minutach ]

kolejna rzecz, zmiennych globalnych nigdy nie inicjalizuje się ZEREM jak to robisz

[syntax=c]volatile uint16_t licznik =0; //liczba przepełnień Timera1
volatile uint16_t reszta =0; //reszta impulsów przed przepełnieniem Timera1 =1;
volatile uint32_t czestotliwosc =0; //mierzona czestotliwosc[/syntax]

zapamiętaj sobie po prostu, że one ZAWSZE są zerowane automatycznie przez kompilator. Czyli oczywiście to że zrobiłeś = 0 przy każdej z nich - nic nie zaszkodzi ale tak na prawdę jest ZUPEŁNIE niepotrzebne. Dopiero gdy potrzebujesz inicjalizować inną wartością niż zero - tak jak ostatnia ze zmiennych globalnych = 61 - wtedy jest OK

------------------------ [ Dodano po: 7 minutach ]

Następna uwaga, po RESECIE procka WSZYSTKIE piny są WEJŚCIAMI ponieważ rejestry DDRx są wyzerowane, a zatem pisanie takiej linii nie jest konieczne:

[syntax=c]DDRB &= ~(1<<PB1);[/syntax]

ale oczywiście to tylko informacyjnie bo nie wpływa to na jakieś złe działanie programu, ot po prostu niepotrzebnie zajęta pamięć FLASH. Chociaż dla porządku jak się zaczyna programować może być - ale ja często wtedy też komentuję sobie taką linię

[syntax=c]//DDRB &= ~(1<<PB1);[/syntax]

------------------------ [ Dodano po: 11 minutach ]

na koniec, ZDECYDOWANIE ale to ZDECYDOWANIE - nigdy nie używaj jakichkolwiek poleceń do obsługi LCD w jakimkolwiek przerwaniu. To mniej więcej tak samo jakbyś użył tam _delay_ms(). Pamiętaj obsługa LCD jest bardzo powolna .... i tak jak podpowiadał ci już kolega wyżej można było po prostu zamiast tego bloku poleceń

[syntax=c]lcd_locate(3,6);
lcd_str(" "); //wymazywanie wczesniejszego wyniku
lcd_locate(3,6);
lcd_int(czestotliwosc);[/syntax]

ustawić po prostu jakąś zmienną globalną (flagę), np:

[syntax=c]display_flag=1;[/syntax]

natomiast w pętli nieskończonej while(1) w funkcji main dać:

[syntax=c]if( display_flag ) {

lcd_locate(3,6);
lcd_str(" "); //wymazywanie wczesniejszego wyniku
lcd_locate(3,6);
lcd_int(czestotliwosc);

display_flag=0;
}[/syntax]

------------------------ [ Dodano po: 13 minutach ]

na zakończenie - to co napisałem, to nie żadna tam jakaś kocia krytyka, tylko takie hinty (podpowiedzi) z mojej strony - mam nadzieję, że w jakimś stopniu przydadzą się, i życzę powodzenia w dalszym kodowaniu ;)

Statystyki: Napisane przez mirekk36 — 16 kwi 2013, o 06:40

]]> 2013-04-15T23:38:59+01:00 2013-04-15T23:38:59+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=34420#p34420 <![CDATA[Re: Prosty miernik częstotliwości]]>
To pierwsza wersja, zapewne będzie ewoluowała. Jak na razie oprócz generatora na PC nie mam jak sprawdzić pomiarów, jutro pewnie jakiś kwarcowy na czas testów zbuduję i zobaczę jak z dokładnością.

Statystyki: Napisane przez janeczqu — 15 kwi 2013, o 23:38

]]> 2013-04-15T20:48:55+01:00 2013-04-15T20:48:55+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=34407#p34407 <![CDATA[Re: Prosty miernik częstotliwości]]>
pomysł fajny, przejrzałem i sam napisałem podobny. To co zmieniłem i mogę się doczepić to wrzucenie wszystkiego w przerwanie, głownie obsługi wyświetlacza. rozumiem że po to wyłączasz przerwania, żeby nic nie zakłóciło wykonywania tego jednego:) tylko mam wtedy wrażenie, może źle myślę, że na czas wyłączenia przerwań do obsługi lcd itp. uC nie liczy kolejnych impulsów i wynik może być nieprawidłowy. Ja osobiście w przerwaniu od timera0 bym tylko ustawiał flagę sprawdzaną w pętli głównej

Statystyki: Napisane przez swierku — 15 kwi 2013, o 20:48

]]> 2013-04-15T12:51:07+01:00 2013-04-15T12:51:07+01:00 https://forum.atnel.pl/viewtopic.php?t=2875&p=34355#p34355 <![CDATA[Prosty miernik częstotliwości]]> To mój pierwszy post w tym dziale wiec proszę o wyrozumiałość :P
Poniżej przedstawiam program napisany wczoraj- pierwsze testy wykazały poprawny pomiar do 20kHz (generator -program na PC wyjście karta dźwiękowa).
Zachęcam do komentowania jak i testowania, program napisany dla M32, 16MHz- reszta w komentarzach :)
[syntax=c]/*
* main1.c
*
* Created on: 2013-04-14 Buszkolandia
* Autor: janeczqu
*
*/
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include "lcd44780.h"

volatile uint16_t licznik =0; //liczba przepełnień Timera1
volatile uint16_t reszta =0;//reszta impulsów przed przepełnieniem Timera1 =1;
volatile uint32_t czestotliwosc =0; //mierzona czestotliwosc
volatile uint8_t przepelnienie =61; //liczba przerwań Timera0 w ciągu 1s

int main(void)
{
DDRB &= ~(1<<PB1); //PB1 wejście podciagnięte do plusa
PORTB |= (1<<PB1);

//Timer0 zegar
TCCR0 |= (1<<CS02)|(1<<CS00); //preskaler = 1024
TIMSK |= (1<<TOIE0) | (1<<TOIE1); //przerwanie na przepełnienie

//Timer1 licznik
TCCR1B |= (1<< CS12) |(1<< CS11); //zrodło zewnętrzne - zbocze opadające
TIMSK |= (1<<TOIE1); //przepełnienie
sei(); //wlaczenie przerwan

lcd_init(); //inicjacja
lcd_cls(); //czyszczeine
lcd_locate(0,0);
lcd_str("Miernik"); //"oprawa graficzna ;)
lcd_locate(1,2);
lcd_str("czestotliwosci");
lcd_locate(2,0);
lcd_str(" janeczqu Atnel ");
lcd_locate(3,0);
lcd_str ("F[Hz]=");
while(1)
{
lcd_locate(3,6);
}
}
ISR(TIMER0_OVF_vect) //odmierzanie 1 sek
{
przepelnienie--;
if (przepelnienie == 0)
{
cli(); //zatrzymanie przerwań na czas obliczen i wyswietlania
reszta = TCNT1; //pobieranie wartości z licznika
TCNT1=0; //zerowanie licznika
czestotliwosc = licznik * 65536; //liczba przepełnień Timera1* pojemność Timera1
czestotliwosc = czestotliwosc + reszta; //obliczanie częstotliwosci
licznik = 0;
przepelnienie = 61;
lcd_locate(3,6);
lcd_str(" "); //wymazywanie wczesniejszego wyniku
lcd_locate(3,6);
lcd_int(czestotliwosc);
sei(); //wlaczenie przerwan
}
}
ISR(TIMER1_OVF_vect)
{
licznik++;
}[/syntax]



Rozjechały się komentarze :/

Statystyki: Napisane przez janeczqu — 15 kwi 2013, o 12:51

]]>