вот листинг моей программы , это грубо говоря пищалка против клопов ультро звуковая ,
должна настраиваться переменным резистором подключенным к порту а на нужной частоте.
Но факт в том что программа на каторой компелируется текст начинает глючить , то проходит через прерывание таймера Т1 по три раза , то скачет через строчку и появляется после нескольких
десятков нажатий F11 в разных местах. Самое интересное что эту же программу я заредил в VMLAB
Она и там показала ту же окалесицу. Я с этой бедой борюсь больше 2 лет ( а мне не смешно.)
Голова кругом!!! То-ли менять компилятор (я к этому то долгл приноравливался ) , То ли писать на Ассемблере. Я бы с Удовольствием выложил видео Файл Что эта программа вытворяет , если покажите куда.



#include <avr/io.h>
#include <stdio.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <avr/signal.h>


volatile unsigned char a;

#define INVBIT(x,y) (x^=(1<<y))


ISR(TIMER1_OVF_vect)
{

INVBIT(PORTB,PB1);// управление светодиод
INVBIT(PORTB,PB0);// управление светодиод
a=0;

}








void display(void);

int main(void)
{
TIFR =0;
TIMSK=0x04;// прерывание по перепол т1
TCCR1A=0;
TCCR1B=1;
TCNT1=65535-500;
sei();

SFIOR=0;
ADCSRA=0xe0;

SFIOR=0;
DDRA=0x00;//in
DDRB=0xff;//out
DDRC=0xff;//out
DDRD=0x00;//in

PORTA=0x00;
PORTB=0x01;
PORTC=0x00;
PORTD=0x00;


ADMUX=1;//vibor vhoda

horo:
ADMUX=1;
_delay_ms(1000);
ADCCompare=ADC;
ADCCompare=ADCCompare/3;
ADCCompare=500-ADCCompare;
TCNT1=65535-ADCCompare;

a=1;

horo1:


if (a==0)
{goto horo;}



goto horo1;





}

1. Какой чип?
2. Какая тактовая частота? В даташите написано тактовая на АЦП д.б. 50...200кГц, а у Вас делитель на 2.
3. Зачем постоянно выбираете 1-й канал? Одного раза разве не достаточно?
4. Так наверное будет и понятнее и проще:


5. А разве не надо опрашивать флаг ADIF что бы узнать об окончании конверсии?

Сообщение отредактировал alexeyv - Dec 3 2012, 08:36

Проц Mega16l , Тактовая частота 16 мегагерц .
Пытаюсь настроить колебания на частоту от 16килогерц до 50.
Динамик приходится качать в два раза чаще.
К порту В к выводам 0 и 1 подключены через 4 транзистора динамик и попарно он качается то одним
плечем то другим.
Проблемма в управлении, его нет.
Я создавал приблуды и посерьезнее но проблемма одна программа глючит.

TCNT1 = 65535 - 500 + ADC/3;
Согласно вашей формуле если за место ADC подставить 1000 то TCNT = FE25.

А из-за того что я разложил все действия результат оказался верным FF58.
я его проверял на калькуляторе.

Но вопрос не в этом а в чем писать и чем проверять программу ?
Winavr 2008 12 05
AvrStudio 4.18
Это то вчем я работаю.

Насколькоя понимаю - вы используете таймер в Normal mode. Это - мягко говоря, не лучшее решение: после переполнения и "дрыгания" ногами, Вы выставляете признак обновления счетчика таймера (переменная а), но в ветке обновления счетчика (метка horo) у Вас стоит задержка в целую секунду...
Логичнее использовать CTC mode. Да, и для генерации сигнала, вероятно, лучше использовать вывод ОС1х, предварительно настроив вывод таймера на него.

1. Если FCPU =16МГц, то необходимо поставить делитель АЦП равным 128 (тактовая на АЦП будет 125КГц)

2.

Странно!! А у меня получилось FF58.
Объясните мне разницу между
и


3. Где опрос флага ADIF и его сброс? Ваша проблема понятна - Вы считываете данные с АЦП тогда, когда еще нет точного результата конверсии

4. Для проверки алгоритма поставьте несколько констант вместо считывания с АЦП и проверьте зависимость выхода(частоты) от констант. Если на выходе будет нужная частота, то проблема в считывании показаний с АЦП.
Также, если есть к-либо отладочный порт (USART), можете вывести значения АЦП на ПК и посмотреть как они считываются/изменяются.

5. Если у вас задержки порядка секунды, зачем запускать АЦП во Free Runing? По окончании задержки запустите в одиночном измерении и дождитесь окончании конверсии

Сообщение отредактировал alexeyv - Dec 3 2012, 08:49

я поставил задержку в надежде что появится регулировка от ацп спасибо за подсказку.
с временем я что-то под ошибся .


Господа программисты посмотрите этот клипп и обьясните почему он себя так ведет.

не могу вставить пока , файл 1.5 мегабайта сайт не принимает

Есть столько сайтов , что бы сбросить и дать ссылку

Нашел , скачивайте и посмотрите .
http://files.mail.ru/IF38YS



1 .Зачем мне включать делитель если частотой управляет формула?
2 .Разница лишь в том что если вашу формулу я вставляю в код программы и свой ,я получаю разные результаты.
3. Хорошая мысль Попробую после того как программа заработает.
4. Попробую.
5. Такая задержка это ошибка , я ее уже устранил .

Алексей тоже посмотрите ролик мне интересно ваше мнение.

Формула должна управлять выходной частотой для динамика, а не рабочей частотой от которой работает АЦП - не путайте понятия!!


Не верю. Жаль мне сейчас не на чем проверить. Это всего лишь эквивалентные преобразования.


Это надо сделать ДЛЯ того что бы программа заработала, причем правильно!!

По поводу ролика:
Какой режим оптимизации включен? При включенной оптимизации компилятор как раз выкинет все ваши присваивания и приведет все к виду:
TCNT1 = 65035 + _function_divide(ADC,3);
переменная ADCCompare также выкинется как лишняя.

Замечания по поводу алгоритма
1. Зачем вообще так часто сканировать номинал резистора или он быстро меняется. Достаточно это делать раз в секунду. Причем в Single-mode. Делать это в главном цикле в задержкой в 1 сек и ожиданием окончания конверсии после старта
2. Согласен с Палыч'ем - необходимо использовать тайммер в СТС режиме. В прерывании от него необходимо проверять не изменилось ли значение резистора и только после этого менять период работа таймера. Не изменилось - пусть себе дальше пашет. Использование ОС1х - не обязательно, надо смотреть как у Вас все подлючено и можно ли перебросить выводы.

Переделал вот листинг
#include <avr/io.h>
#include <stdio.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <avr/signal.h>

volatile unsigned int a; // èçìåíÿåìàÿ äëèòåëüíîñòü ìîðãàíèé
volatile unsigned int ADCCompare; //

#define SETBIT(x,y) (x|=(1<<y))
#define CLRBIT(x,y) (x&=~(1<<y))
#define INVBIT(x,y) (x^=(1<<y))
#define CHKBIT(x,y) (x&(1<<y))
#define BITON(x,y) (x&(1<<y))
#define BITOFF(x,y) (!(x&(1<<y)))



ISR(TIMER1_OVF_vect)
{

INVBIT(PORTB,PB1);//
INVBIT(PORTB,PB0);//
a=0;

}

// ((PIND & 0x04) == 0 ) ïðîâåðèòü åñëè 4 áèò = 0
// ((PIND & 0x04) != 0 ) ïðîâåðèòü åñëè 4 áèò = 1
// 7654 3210
// 1000 0000 = 80
// 0100 0000 = 40
// 0010 0000 = 20
// 0001 0000 = 10
// 0000 1000 = 08
// 0000 0100 = 04
// 0000 0010 = 02
// 0000 0001 = 01


int main(void)
{
TIFR =0;
TIMSK=0x04;// ïðåðûâàíèå ïî ïåðåïîë ò1
TCCR1A=0;
TCCR1B=1;
TCNT1=65535-500;
cli();

SFIOR=0;
ADCSRA=0xe7;

SFIOR=0;
DDRA=0x00;//in
DDRB=0xff;//out
DDRC=0xff;//out
DDRD=0x00;//in

PORTA=0x00;
PORTB=0x01;
PORTC=0x00;
PORTD=0x00;


ADMUX=1;//vibor vhoda



horo:

TCNT1=65535-(500-(ADC/3));
a=1;

horo1:


if ( ADIF==1)
{goto horo;}



goto horo1;
}



Оптимизацию сделал на ноль и на один тоже ставил
в листинге на строчке а=1 в программе проподает стрелка при значении CYCLE COUNTER 204 А ЕСЛИ НАЖАТЬ ПАУЗУ ЭТО ЖЕ ЗНАЧЕНИЕ УЖЕ РАВНО 3206010 , ПРИ ЧЕМ НИ КАКИЕ ТОЧКИ ОСТАНОВА РАССТАВЛЕННЫЕ В РАЗНЫХ МЕСТАХ ПРОГРАММЫ И ПРЕРЫВАНИЯ НЕ СРАБАТЫВАЮТ. КАК ТО РАЗ ДАЖЕ ОСТАНОВИ ПРОГРАММУ ГДЕ СТРЕЛА ВЫШЛА ЗА ЛИСТИНГ ПРОГРАММЫ . кАК ОТЛАЖИВАТЬ ПРОГРАММУ ВООБЩЕ НЕ ПОНЯТНО((

Вставьте:

asm("nop");

и на этой строке сделайте точку останова.

Помогает.

(( Попробывал Теже яйца только в профиль.
Я вот все думаю если есть этот Сайт Созданы эти микроконтроллеры есть этот глючный AVR Studio ,
Есть наверное прога в каторой все программируют отлаживают и в тихушку молчат . Это мое мнение.


Неужели в аське писать прогу , просто не понятно как представлять данные выходящие за значение одного байта, как совершать деление и др операции это же вчерашний век . Вот Блин, руки опускаются с такой перспективой.

В чём претензия к AVRStudio?


Есть такая программа, AVRStudio называется.
Есть и другие программы.


Дело всё в вашей программе - что она делать-то должна?
Что странно:

результат АЦП получается 1 раз в секунду, в зависимости от результата устанавливается "период" таймера. Остальное время прерывания переполнения возникают через 65536 циклов.
Может как-то так надо:







?

Где-то примерно так:

Не вдаваясь в тонкости реализации программы, скажу, что студия эмулирует сишный код несколько своеобразно. При ошибках в коде может вообще убрать целые фрагменты кода, которые с точки зрения компилятора выполнять никогда не будут. Для получения полного представления того, что там накомпилилось из вашей программы, отлаживаейте ее в окне дизассемблера.