Страница 1 из 1

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

Добавлено: Сб мар 08, 2014 12:36:39
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 часов подряд, можно сделать вывод, что она правильно работает....
Прикрепляю всю программу для интересующихся.

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

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

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

P.P.S.
Код не читал. :)))

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

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

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

P.P.S.
Код не читал. :)))
Да релюхи с диодами стоят. Но я же писал - питание стабильное - там кондер бешенной емкости стоит и стабилизатор напряжения, а перед этим обычный трансформатор (даже не импульсный). Если контроллер бы сбрасывался, то согласно программе он тут же в переменную t2_light занес бы единицу...Я проверял в реальности - МК не сбрасывается и с питанием все ок. Но это касательно напряжения...а вот ток...Дросселя конечно я не ставил)

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

Добавлено: Сб мар 08, 2014 15:55:24
urry
Повторю вопрос. По каким причинам переменная t2_light[0] может стать в 0?
вы видите не весь код, который делает компилятор. В стартовом коде происходит инициализация стека и очистка глобальных переменных, в том числе и вашей переменной, она обнуляется . Кстати, она обязана быть volatile .
Раз сбой происходит раз в несколько часов, логично предсказать, что происходит сброс программы.
Я не вижу где,как вы пишите, "она сразу становится 1".
Для того, чтобы сброс по сторожевому таймеру, например, не "чистил" нужные вам переменные, в линкере указывается (создается ручками) область памяти, свободная от чистки при включении, переменная располагается там.
Как это сделать в кодевижин, представления не имею - читайте хелп, возможно оно там написано.

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

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

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

Добавлено: Сб мар 08, 2014 16:18:18
urry
Прежде чем наезжать на компилятор, необходимо правильно написать код.
Определите переменные, которые меняются как в прерывании, так и в основной программе, с спецификатором volatile.
Запретите прерывание в мейне на тот момент, когда к ним обращаетесь.
Почитайте об атомарном доступе и volatile.

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

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

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

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

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

Добавлено: Сб мар 08, 2014 18:09:07
Volgol_555
Спасибо. Замечания ваши учту в дальнейшем. Отвечу на ваши вопросы:
1) Переменную объявил массивом только для Proteus'a. Тогда при паузе он 100% показывает значение переменной...иначе может не показать...нюансов не знаю...
2) Использовал таймер Т1 из-за удобства и точности. у Т1 разрядов большо - я выбираю 3.96 кГц и прерывание на 3960 штук...Получаю 1 с. При других таймерах так точно я могу считать уже только миллисекунды...а зачем мне потом пересчитывать до секунд, когда я одним таймером Т1 все сделаю как надо)
3) Сброс таймера я делал по привычке и не заморачиваюсь над этим. Но спасибо за совет.
На счет Volatile...можно подробнее про это? Ну подумаешь переменная внезапно изменилась в прерывании...цикл разве не начнет работать уже с новым значением? объясните, пожалуйста, как это происходит на аппаратном уровне. :)

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

Добавлено: Сб мар 08, 2014 18:22:07
urry

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

Добавлено: Сб мар 08, 2014 18:46:52
ИС-пытатель
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;

и т.д.

Естественно, если у Вас переменная потом (например, в начале цикла) инициализируется заново, она поменяет значение. Но в этом цикле, где произошло прерывание, она будет не правильной.

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

Добавлено: Сб мар 08, 2014 18:58:27
Volgol_555
Супер. Спасибо за лекцию! Это все объясняет!!!! :)) Считаю тему закрытой)

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

Добавлено: Вс мар 09, 2014 17:23:56
ИС-пытатель
Только не надо теперь повсеместно пихать volatile. ;)

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