Пишу программу на С для AVR. Cуть - плавный пуск.

Проблемный кусок:



Объявил глобальную переменную. Присвоил ей нулевое значение. При каждом внешнем прерывании переменная инкрементируется. В зависимости от количества прерываний запускается таймер с разной константой.

То есть первых 10 прерываний - константа будет 0x83
Следующий 10 - 0x91 и т.д.

Но на асемблере получилось не так, как планировалось. Программа проверяет каждое условие и всё равно выполняет все подряд операторы. Например в начале, когда n=1, она проверяет не меньше ли оно 10. Оно меньше. Значит выполняются операторы. Потом идёт вторая проверка. Тут условие не выполняется, но всё равно операторы выполняются.

Программа на асемблере:





То что подчеркнул красным проблемные места. Получается R30 всегда либо 1, ли бо 0.


Подскажите, в чём ошибка.

Сообщение отредактировал Владимир_КПИ - Feb 13 2008, 20:21

Проблема в том, что это написано не на "C" . Начните с изучения языка.

Программа писалась в CodeVisionAVR. Языка особо не знаю. Писал по логике.

Возможно проблема вот здесь - "(10<n<21)".

Сообщение отредактировал Владимир_КПИ - Feb 13 2008, 20:47

Хорошая фраза. Можете занести эту фразу себе в подпись.

Жесть! Зело позабавило
Владимир_КПИ, для начала выражение вида if (0<n<11) запишите как if ((n>0)&&(n<11)).
Но вот эту книгу на полку себе поставьте http://www.lib.ru/CTOTOR/kernigan.txt

Спасибо. Исправил. Сейчас проверю работу на практике.

Всё отлично работает. Программа своё отрабатывает. Единственное, что плавный пуск делается за 10 шагов. Нужно сделать чаще, чтобы для зрения не было замерно. Чтобы был именно планым, а не рывками.
Но если добавлю ещё операторов if будет не совсем хорошо - при каждом прерывании делать множество проверок. Нельзя ли как-то оптимизировать код?

Видео

Сообщение отредактировал Владимир_КПИ - Feb 13 2008, 21:23

Да ладно Вам, вполне жизненная конструкция, и кстати очень даже на C

Например так:
unsigned char a;
a = 0;
if (1>a>0) ......
a = 1;
if (1>a>0) ......

Шучу...

http://en.wikipedia.org/wiki/Switch_statement

Дык, чую,что совсем незачем принципиально так строить алгоритм - там что-то системного подхода в упор не видать .
А co switch он такого в обнимку с волшебником натворит .
Ладно пример со switch:

Более эффективный способ проверок, требуется только одно сравнение для каждого поддиапазона.

если хочется ещё быстрее - придётся за это заплатить размером памяти кода.
Например так:


разумеется, нужно позаботиться о случаях когда n меньше 0 (можно объявить как unsigned) и когда n становится больше максимального индекса в массиве table.

Вот это по-моему лучший вариант для моего случая. Объявлю массив со 100 значениями. При каждом прерывании будет инкрементироваться n. За 2 секунды - 100 раз по 1%. Будет и плавнее.



Да, думал над этим. Сейчас что-нибудь придумаю.

unsigned int n=0;

#define tcnt0_max 0xff
#define tcnt0_min 0x83
#define nmax 100

// External Interrupt 0 service routine
interrupt [EXT_INT0] void ext_int0_isr(void)
{
// Place your code here
if(n<nmax){n=n+1;}

TCNT0=tcnt0_min+((tcnt0_max-tcnt0_min)*n)/nmax;
TCCR0=0x03;

}

если nmax>0xff тогда нужно "TCNT0=tcnt0_min+((tcnt0_max-tcnt0_min)*(unsigned long int)n)/nmax;"
операция деления для int (или long int) выполняется не помню сколько, но не более 500 клоков (на 8мГц потратится не более 0,1 мс времени, или 1% полпериода сетевого питания, я бы сделал так. )

Есть массив:

int time[100]={0xFB04, 0xF8F2, .....}

Как из элемента массива отделить байты?

Чтобы потом сделать так:

TCNT1H=0xFB; //TCNT1H=high(time[0])
TCNT1L=0x04; //TCNT1L=low(time[0])

Коментарии условные.

Сообщение отредактировал Владимир_КПИ - Feb 14 2008, 09:58

TCNT1H = (time[n] & 0xFF00) >> 8;
TCNT1L = time[n] & 0xFF;

... и открыть ветку "изучаем С интерактивно"

Сообщение отредактировал Liseev - Feb 14 2008, 10:15

Спасибо