Случайный сброс регистра AVR Atmega8A?!

Обсуждаем контроллеры компании Atmel.
Ответить
Потрогал лапой паяльник
Аватара пользователя
Сообщения: 301
Зарегистрирован: Пт сен 23, 2011 13:13:51
Откуда: Минск

Сообщение Volgol_555 »

Всем привет. Написал программку управления светом в ванной и туалете...Почти все прекрасно работает, но обнаружил экспериментальным способом, что у Atmega8 приблизительно раз в день сбрасывается переменная на 0 (иногда 2 раза в день, иногда раз в 2 дня...), хотя по программе это возможно только в начале старта. Watchdog у меня отключен. Вот код программы, чтобы убедиться в правильности моих выводов:

Код: Выделить всё

/*****************************************************
Chip type               : ATmega8
Program type            : Application
AVR Core Clock frequency: 1,000000 MHz
Memory model            : Small
External RAM size       : 0
Data Stack size         : 256
*****************************************************/

#include <mega8.h>
#include <delay.h>

 #define relay1 PIND.5      //дальниый геркон туалета
 #define relay2 PIND.6      //ближний геркон туалета
 #define relay3 PIND.7     //геркон ванной комнаты
 #define relay4 PINB.0     //датчик движения
  #define ON 0
  #define Off 1

  #define light_wc PORTD.2      // освещение туалета
  //#define light_bath PORTD.3      // освещение ванной

unsigned int t1_light, t2_light[1], t1_off;
bit m_bath, light_bath;
bit m_wc;
unsigned int Lw;
int l1, l2;
// Timer1 output compare A interrupt service routine
interrupt [TIM1_COMPA] void timer1_compa_isr(void)
 {
 TCNT1H=0x00;
 TCNT1L=0x00;

  ++l1;
  if (l1==1) PORTC.2=1;
  if (l1==2) {PORTC.2=0; l1=0;}

  if (t1_off>0) ++t1_off;

  if (light_wc==1) ++t1_light;
  else  t1_light=0;
  if (t1_light>1800) light_wc=0;     //выкл. света туалета через 30 мин

  if (t2_light[0]>0) ++t2_light[0];
  if (t2_light[0]>3600) {light_bath=0; t2_light[0]=1;} //выкл. света ванной через 1 час
 }

// Declare your global variables here

void main(void)
{
// Declare your local variables here

// Input/Output Ports initialization
// Port B initialization
// Func7=Out Func6=Out Func5=Out Func4=Out Func3=Out Func2=Out Func1=Out Func0=In
// State7=0 State6=0 State5=0 State4=0 State3=0 State2=0 State1=0 State0=P
PORTB=0x01;
DDRB=0xFE;

// Port C initialization
// Func6=Out Func5=Out Func4=Out Func3=Out Func2=Out Func1=Out Func0=Out
// State6=0 State5=0 State4=0 State3=0 State2=0 State1=0 State0=0
PORTC=0x00;
DDRC=0x7F;

// Port D initialization
// Func7=In Func6=In Func5=In Func4=Out Func3=Out Func2=Out Func1=Out Func0=Out
// State7=P State6=P State5=P State4=0 State3=0 State2=0 State1=0 State0=0
PORTD=0xE0;
DDRD=0x1F;

// Timer/Counter 0 initialization
// Clock source: System Clock
// Clock value: Timer 0 Stopped
TCCR0=0x00;
TCNT0=0x00;

// Timer/Counter 1 initialization
// Clock source: System Clock
// Clock value: 3,906 kHz
// Mode: Normal top=FFFFh
// OC1A output: Discon.
// OC1B output: Discon.
// Noise Canceler: Off
// Input Capture on Falling Edge
// Timer1 Overflow Interrupt: Off
// Input Capture Interrupt: Off
// Compare A Match Interrupt: On
// Compare B Match Interrupt: Off
TCCR1A=0x00;
TCCR1B=0x04;
TCNT1H=0x00;
TCNT1L=0x00;
ICR1H=0x00;
ICR1L=0x00;
OCR1AH=0x0F;
OCR1AL=0x42;
OCR1BH=0x00;
OCR1BL=0x00;

// Timer/Counter 2 initialization
// Clock source: System Clock
// Clock value: Timer2 Stopped
// Mode: Normal top=FFh
// OC2 output: Disconnected
ASSR=0x00;
TCCR2=0x00;
TCNT2=0x00;
OCR2=0x00;

// External Interrupt(s) initialization
// INT0: Off
// INT1: Off
MCUCR=0x00;

// Timer(s)/Counter(s) Interrupt(s) initialization
TIMSK=0x10;

// Analog Comparator initialization
// Analog Comparator: Off
// Analog Comparator Input Capture by Timer/Counter 1: Off
ACSR=0x80;
SFIOR=0x00;

// Watchdog Timer initialization
// Watchdog Timer Prescaler: OSC/2048k
/*#pragma optsize-
WDTCR=0x1F;
WDTCR=0x0F;
#ifdef _OPTIMIZE_SIZE_
#pragma optsize+
#endif */

// Global enable interrupts
#asm("sei")
 m_bath=0;
while (1)
 {     //управление светом туалета - в 2 геркона

   if (relay1==Off || relay2==ON) {delay_ms(100); light_wc=1; t1_off=0; m_wc=0;}
   if (relay1==ON && relay2==Off) {
            if (!m_wc) {t1_off=1; m_wc=1;}
            if (t1_off>2) {light_wc=0;}
                                    }

         //управление светом ванной - в 1 геркон и 1 датчик движения
   if (relay3==Off) { delay_ms(100);
       if (!m_bath) {light_bath=1; t2_light[0]=1; m_bath=1;}   // если дверь открыта работаем от датчика движения с задержкой
        if (relay4==ON) {light_bath=1; t2_light[0]=1;}
        if (relay4==Off && t2_light[0]>8) {light_bath=0; t2_light[0]=1;}

   }
   PORTD.3=light_bath;    PORTD.1=light_bath;

   if (t2_light[0]==0) PORTC.4=1;
   else  PORTC.4=0;

   if (relay3==ON) {          //если дверь закрыта
        if (m_bath) {t2_light[0]=1; m_bath=0;}
        if (t2_light[0]==3 && relay4==ON)  ++Lw;
        if (t2_light[0]==4)  {
          if (Lw>0) {light_bath=1;}
          else {light_bath=0;}
          Lw=0; ++t2_light[0];
         }
   }

       ++l2;
   if (l2<5) PORTC.1=1;
   if (l2>4) PORTC.1=0;
   if (l2>11) l2=0;
   delay_ms(100);

  // #asm("wdr");
      };
}
Как видите watchdog я временно закомментировал. Как видно из программы PORTC.1 выявляет работу цикла while (1), PORTC.2 показывает работу таймера. PORTC.4 показывает, когда у меня проблемная переменная t2_light[0] становится в 0. Т.к. вот при всей работоспособности контроллера t2_light[0] действительно становится в 0. Но если посмотреть программу, то это невозможно...t2_light[0] либо может быть 1, либо больше 1. t2_light[0] я объявил массивом, т.к. в таком случае он заносится в отдельный регистр, который отображается в Proteus. Но это не суть важно. Массив это или нет...результат один и тот же...
Аtmega8 в реальных цепях питается стабильным питанием 4.85В, на входах у нее сидят герконы и один оптрон от датчика движения...Выходами включаем реле через транзисторы. На реле стоят обратные защитные диоды...Я уже не первый десяток раз собираю схемы, поэтому опыт есть!
Повторю вопрос. По каким причинам переменная t2_light[0] может стать в 0? Ошибка CVavr 2.05.03 при компиляции или кристалл бракованный? Чего я не знаю об AVR? Помогите разрешить проблему...а то спать спокойно не могу).
В AVRstudio не вижу смысла проверять программу...т.к. исходя из работоспособности системы как минимум 12 часов подряд, можно сделать вывод, что она правильно работает....
Прикрепляю всю программу для интересующихся.
Вложения
Light_WC_Room_old.rar
Программа в CVavr 2.05 с файлом от Proteus и платой на Sprint Layout 6.0
(299.75 КБ) 146 скачиваний
Самое гениальное - всегда самое простое
Контактная информация:
Реклама
Мучитель микросхем
Аватара пользователя
Сообщения: 472
Зарегистрирован: Ср фев 15, 2012 09:00:34
Откуда: Киев

Сообщение Eugene.Negrobov »

Может питание проседает и МК сбрасывается...
Если есть цифровой осцилл- подруби к линии питания, как можно ближе к ножкам МК, триггер выстави на 4.5В, единичное срабатывание, спадающий фронт. В итоге если будет просадка- осцилл задетектит.
Более дешевый метод- заюзать встроенный в МК "brown out detector".

P.S.
Параллельно релюхам стоят диоды?
Как бы контроллеры не очень любят работать в силовых схемах, где есть хоть какие-то наводки по питанию. Еще могу посоветовать запитать сам МК через LC фильтр.
Чтобы регистр сам сбрасывался- не, не слышал о таком.

P.P.S.
Код не читал. :)))
I'm on a huge rock, flying through space!
Реклама
Потрогал лапой паяльник
Аватара пользователя
Сообщения: 301
Зарегистрирован: Пт сен 23, 2011 13:13:51
Откуда: Минск

Сообщение Volgol_555 »

Eugene.Negrobov писал(а):Может питание проседает и МК сбрасывается...
Если есть цифровой осцилл- подруби к линии питания, как можно ближе к ножкам МК, триггер выстави на 4.5В, единичное срабатывание, спадающий фронт. В итоге если будет просадка- осцилл задетектит.
Более дешевый метод- заюзать встроенный в МК "brown out detector".

P.S.
Параллельно релюхам стоят диоды?
Как бы контроллеры не очень любят работать в силовых схемах, где есть хоть какие-то наводки по питанию. Еще могу посоветовать запитать сам МК через LC фильтр.
Чтобы регистр сам сбрасывался- не, не слышал о таком.

P.P.S.
Код не читал. :)))
Да релюхи с диодами стоят. Но я же писал - питание стабильное - там кондер бешенной емкости стоит и стабилизатор напряжения, а перед этим обычный трансформатор (даже не импульсный). Если контроллер бы сбрасывался, то согласно программе он тут же в переменную t2_light занес бы единицу...Я проверял в реальности - МК не сбрасывается и с питанием все ок. Но это касательно напряжения...а вот ток...Дросселя конечно я не ставил)
Самое гениальное - всегда самое простое
Контактная информация:
Сверлит текстолит когтями
Аватара пользователя
Сообщения: 1262
Зарегистрирован: Пн дек 08, 2008 10:58:48
Откуда: Винница

Сообщение urry »

Повторю вопрос. По каким причинам переменная t2_light[0] может стать в 0?
вы видите не весь код, который делает компилятор. В стартовом коде происходит инициализация стека и очистка глобальных переменных, в том числе и вашей переменной, она обнуляется . Кстати, она обязана быть volatile .
Раз сбой происходит раз в несколько часов, логично предсказать, что происходит сброс программы.
Я не вижу где,как вы пишите, "она сразу становится 1".
Для того, чтобы сброс по сторожевому таймеру, например, не "чистил" нужные вам переменные, в линкере указывается (создается ручками) область памяти, свободная от чистки при включении, переменная располагается там.
Как это сделать в кодевижин, представления не имею - читайте хелп, возможно оно там написано.
Последний раз редактировалось urry Сб мар 08, 2014 16:07:21, всего редактировалось 1 раз.
Контактная информация:
Реклама
Эиком - электронные компоненты и радиодетали
Потрогал лапой паяльник
Аватара пользователя
Сообщения: 301
Зарегистрирован: Пт сен 23, 2011 13:13:51
Откуда: Минск

Сообщение Volgol_555 »

urry писал(а):
Повторю вопрос. По каким причинам переменная t2_light[0] может стать в 0?
вы видите не весь код, который делает компилятор. В стартовом коде происходит инициализация стека и очистка глобальных переменных, в том числе и вашей переменной, она обнуляется . Кстати, она обязана быть volatile .
Раз сбой происходит раз в несколько часов, логично предсказать, что происходит сброс программы.
Я не вижу где,как вы пишете "она сразу становится 1".
Для того, чтобы сброс по сторожевому таймеру, например, не "чистил" нужные вам переменные, в линкере указывается область памяти, свободная от чистки при включении, переменная располагается там.
Как это сделать в кодевижин, представления не имею - читайте хелп, возможно оно там написано.
Если внимательно изучить мой код, то в нем видно, что при любых условиях переменная тут же становится не меньше 1 - эту часть кода уже десятки раз я отработал. Сторожевого таймера у меня нет - он закоменчен! Я не вижу всего реального кода - это верно. Также хочу отметить что сбой происходит неожиданно иногда пару раз за несколько часов, иногда за весь день ни разу (цикличности в этом смысле нет!).
Вот к примеру...я создаю условие когда relay3==Off и при этом relay4==ON при таком условии t2_light[0] всегда должна быть 1 (за искл. момента, когда таймер ее тикнет до 2). Но через какой-то период этого не происходит!!! Мог бы сделать выводы что CVavr неправильно компилит код...но доказать нечем - кстати подобная ситуация при сравнении была в CVavr в прошлой версии, но там оно просто не работало.... А здесь работает не понятно как(
Самое гениальное - всегда самое простое
Контактная информация:
Реклама
Сверлит текстолит когтями
Аватара пользователя
Сообщения: 1262
Зарегистрирован: Пн дек 08, 2008 10:58:48
Откуда: Винница

Сообщение urry »

Прежде чем наезжать на компилятор, необходимо правильно написать код.
Определите переменные, которые меняются как в прерывании, так и в основной программе, с спецификатором volatile.
Запретите прерывание в мейне на тот момент, когда к ним обращаетесь.
Почитайте об атомарном доступе и volatile.
Контактная информация:
Реклама
Вымогатель припоя
Аватара пользователя
Сообщения: 577
Зарегистрирован: Ср июн 19, 2013 08:10:48
Откуда: Москва, СПб, Липецк, Рязань

Сообщение ИС-пытатель »

Добрый день!
У Вас происходит инкремент Вашей переменной в цикле. Скорее всего происходит переполнение (переменная доходит до крайнего значения и обнуляется). попробуйте сделать вместо инкремент такую вставку:

if (Ваша переменная < 0xFFFF) Ваша переменная++;

Далее. не совсем понятно, зачем Вы обозвали свою переменную, как массив. В нем кроме нее ничего не используется. Можно имя попроще.
Далее. тоже мало понятно, зачем вам таймер T1. В большинстве приложений подобных Вашему за глаза хватает Т0 (Он намного проще).
Далее. в обработке Т1 по прерыванию по совпадению Вы сбрасываете счетчик таймера. Это можно сделать автоматически, если установить режим работы СБРОС ПО СОВПАДЕНИЮ. (читайте даташит + Евстифеев справочник по АВР тини + мега)
Далее. Если выкладываете свой код, по максимуму комментируйте хотя бы все ваши переменные (а лучше каждое более менее действие в коде). какая за что отвечает. Сложно догадаться, что Вы имели в виду. А сидеть париться с Вашим кодом за спасибо не очень хочется, тем более, что Вы выложили его как есть, даже не вырезали лишнего.
Далее. Согласен с автором предыдущего сообщения. Если переменная обрабатывается в прерывании (то есть для основного цикла программы она может "вдруг" изменится - объявляйте ее volatile).
Удачи!
Потрогал лапой паяльник
Аватара пользователя
Сообщения: 301
Зарегистрирован: Пт сен 23, 2011 13:13:51
Откуда: Минск

Сообщение Volgol_555 »

Спасибо. Замечания ваши учту в дальнейшем. Отвечу на ваши вопросы:
1) Переменную объявил массивом только для Proteus'a. Тогда при паузе он 100% показывает значение переменной...иначе может не показать...нюансов не знаю...
2) Использовал таймер Т1 из-за удобства и точности. у Т1 разрядов большо - я выбираю 3.96 кГц и прерывание на 3960 штук...Получаю 1 с. При других таймерах так точно я могу считать уже только миллисекунды...а зачем мне потом пересчитывать до секунд, когда я одним таймером Т1 все сделаю как надо)
3) Сброс таймера я делал по привычке и не заморачиваюсь над этим. Но спасибо за совет.
На счет Volatile...можно подробнее про это? Ну подумаешь переменная внезапно изменилась в прерывании...цикл разве не начнет работать уже с новым значением? объясните, пожалуйста, как это происходит на аппаратном уровне. :)
Самое гениальное - всегда самое простое
Контактная информация:
Сверлит текстолит когтями
Аватара пользователя
Сообщения: 1262
Зарегистрирован: Пн дек 08, 2008 10:58:48
Откуда: Винница

Сообщение urry »

Контактная информация:
Вымогатель припоя
Аватара пользователя
Сообщения: 577
Зарегистрирован: Ср июн 19, 2013 08:10:48
Откуда: Москва, СПб, Липецк, Рязань

Сообщение ИС-пытатель »

Volatile не для железа. Он для компилятора.

напишите такой код:

a = 10;
b = a;

и скомпилируйте (действует при включенной оптимизации);

Компилятор вам все это упростит и в выходном коде будет только

a = 10;
b = 10; (оно действительно так);

но если вот сюда влезло прерывание:

a = 10;
<-------прерывание. например, от таймера. В этом прерывании, к примеру, такой код: {a = 5;}
b = a;

то b должно быть тоже 5, а оно 10 (причем всегда). Ведь компилятор упростил код. А вот если вы при объявлении переменной а скажете компилятору, что она может невзначай измениться (в прерывании, например), то он сгенерирует код так, как НУЖНО, а не так, как кажется лучше компилятору.

Объявляется а простым добавлением слова volatile перед переменной или типом:

volatile char a;
char volatile a;
volatile unsigned char a;

и т.д.

Естественно, если у Вас переменная потом (например, в начале цикла) инициализируется заново, она поменяет значение. Но в этом цикле, где произошло прерывание, она будет не правильной.
Потрогал лапой паяльник
Аватара пользователя
Сообщения: 301
Зарегистрирован: Пт сен 23, 2011 13:13:51
Откуда: Минск

Сообщение Volgol_555 »

Супер. Спасибо за лекцию! Это все объясняет!!!! :)) Считаю тему закрытой)
Самое гениальное - всегда самое простое
Контактная информация:
Вымогатель припоя
Аватара пользователя
Сообщения: 577
Зарегистрирован: Ср июн 19, 2013 08:10:48
Откуда: Москва, СПб, Липецк, Рязань

Сообщение ИС-пытатель »

Только не надо теперь повсеместно пихать volatile. ;)

Конечно, ничего плохого не будет. Но оптимизация кода значительно ухудшится (будет больше размер). Действуйте с умом и в разумных пределах! Удачи! :beer:
Ответить

Вернуться в «AVR»