Терморегулятор с DS18b20 и энкодером

[zxz_rus]

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

Общий код:
 

#include <mega8.h>
int NewState,OldState,upState,Temp,downState,count_1; //
unsigned char trig;
float temper;
#asm
   .equ __lcd_port=0x12 ;PORTD
#endasm
#include <lcd.h>
#include <stdio.h>
#include <delay.h>
// 1 Wire Bus functions
#asm
   .equ __w1_port=0x18 ;PORTB
   .equ __w1_bit=2
#endasm
#include <1wire.h>
#include <ds18b20.h>


#pragma warn-
/* eeprom  */
eeprom int temp_eep;
#pragma warn+

interrupt [TIM1_COMPA] void timer1_compa_isr(void)
{


NewState=PINB & 0b00000011;
if(NewState!=OldState)
{
switch(OldState)
    {
    case 2:
        {
        if(NewState == 3) upState++;
        if(NewState == 0) downState++;
        break;
        }
 
    case 0:
        {
        if(NewState == 2) upState++;
        if(NewState == 1) downState++;
        break;
        }
    case 1:
        {
        if(NewState == 0) upState++;
        if(NewState == 3) downState++;
        break;
        }
    case 3:
        {
        if(NewState == 1) upState++;
        if(NewState == 2) downState++;
        break;
        }
    }            
OldState=NewState;

}

 if(trig==1)     //  если флаг активен
   {
      if(OldState==NewState)     // если энкодер в покое то увеличиваем счетчик
       {count_1++;
          if(count_1==5000)     //  если счетчик дотикал до 5 секунд
          {temp_eep=Temp;      // пишем данные в еепром
           count_1=0;         //обнуляем счетчик
           trig=0;           // обнуляем флаг
          }
       
       }   
       
    }  

TCNT1H=0x00;
TCNT1L=0x00;
}

void main(void)
{  
char lcd_buf[17];
char lcd_buf2[17];
// Input/Output Ports initialization
// Port B initialization
// Func7=In Func6=In Func5=In Func4=In Func3=In Func2=In Func1=In Func0=In
// State7=T State6=T State5=T State4=T State3=T State2=T State1=P State0=P
PORTB=0x03;
DDRB=0x00;

// Timer/Counter 1 initialization
// Clock source: System Clock
// Clock value: 1000,000 kHz
// Mode: CTC top=OCR1A
// OC1A output: Discon.
// OC1B output: Discon.
// Noise Canceler: Off
// Input Capture on Falling Edge
// Timer 1 Overflow Interrupt: Off
// Input Capture Interrupt: Off
// Compare A Match Interrupt: On
// Compare B Match Interrupt: Off
TCCR1A=0x00;
TCCR1B=0x0A;
TCNT1H=0x00;
TCNT1L=0x00;
ICR1H=0x00;
ICR1L=0x00;
OCR1AH=0x03;
OCR1AL=0xE8;
OCR1BH=0x00;
OCR1BL=0x00;

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

// Global enable interrupts
#asm("sei")
lcd_init(16);
w1_init();
ds18b20_init(0,-20,50,DS18B20_12BIT_RES);
Temp=temp_eep;  //Выгружаем данные из памяти
while (1)
      {      
      if (upState >= 4)
      {                            
        Temp++;
        upState = 0;
        trig=1;   //   ставим флаг
      }
      if (downState >= 4)
      {                              
        Temp--;
        downState = 0;
        trig=1;   //   ставим флаг
      }                                
      temper=ds18b20_temperature(0);   
      if (temper<=Temp)
            {PINC.1=1;}
            else
            {PINC.1=0;}
      sprintf(lcd_buf,"t=%.1f\xdf""C",temper);     ////0b 99
      sprintf(lcd_buf2,"Temp=%d\xdf""C",Temp);
      lcd_clear();
      lcd_gotoxy(0,0);  
      lcd_puts(lcd_buf);
      lcd_gotoxy(0,1);
      lcd_puts(lcd_buf2);
      
      
      delay_ms(10);
              
      };
}

Проект термометра:

#include <mega8.h>
#include <delay.h>
// 1 Wire Bus functions
#asm
   .equ __w1_port=0x18 ;PORTB
   .equ __w1_bit=2
#endasm
#include <1wire.h>
#include <ds18b20.h>

// Alphanumeric LCD Module functions
#asm
   .equ __lcd_port=0x12 ;PORTD
#endasm
#include <lcd.h>
#include <stdio.h> 

char lcd_buf[17];
void main(void)
{

float temper;  
lcd_init(16);   
w1_init();
ds18b20_init(0,-20,50,DS18B20_12BIT_RES); 
              
while(1)                 
      {
          temper=ds18b20_temperature(0);  
          sprintf(lcd_buf,"t=%.1f\xdfC",temper);    
          lcd_clear();                
          lcd_puts(lcd_buf);
           if (temper<=18)
            {PINC.1=1;}       
          delay_ms(1500);              
      }; 
}

 

При сравнении переменных temper<=temp  ничего не происходит. пробовал Temp заменить числом как во втором примере, тоже ноль реакции. Второй пример работает. Компилятор ошибок не выдает. Пишу в CV AVR. Схема собрана в железе.

[Реклама]

Реклама: ООО ТД Промэлектроника, ИНН: 6659197470, Тел: 8 (800) 1000-321

[COKPOWEHEU]

Форматирование кода ужасно: нет осмысленных комментариев, не убран мусор после автогенератора, кривые отступы, настройки перефирии явно не для людей писались.

Переменную OldState было бы безопаснее объявить внутри прерывания с модификатором static.

Нет модификатора volatile.

Это из того что бросается в глаза.

[Реклама]

20% скидка на весь каталог электронных компонентов в ТМ Электроникс!

Акция "Лето ближе - цены ниже", успей сделать выгодные покупки!

Плюс весь апрель действует скидка 10% по промокоду APREL24 + 15% кэшбэк и бесплатная доставка!

Перейти на страницу акции

Реклама: ООО ТМ ЭЛЕКТРОНИКС, ИНН: 7806548420, info@tmelectronics.ru, +7(812)4094849

[Alex]

11 час назад, zxz_rus сказал:

отклик на энкодер идет слишком долго. То есть пару раз провернешь и только потом переменная увеличивается.

Это говорит лишь о том, что программа где-то тупо висит длительное время. Например, вот тут :

11 час назад, zxz_rus сказал:

temper=ds18b20_temperature(0);

Метод тыка в программировании не годится. Нужно всегда иметь контроль над своей программой, иначе счастья не видать....

[Реклама]

Выбираем схему BMS для корректной работы литий-железофосфатных (LiFePO4) аккумуляторов

 Обязательным условием долгой и стабильной работы Li-FePO4-аккумуляторов, в том числе и производства EVE Energy, является применение специализированных BMS-микросхем. Литий-железофосфатные АКБ отличаются такими характеристиками, как высокая многократность циклов заряда-разряда, безопасность, возможность быстрой зарядки, устойчивость к буферному режиму работы и приемлемая стоимость. Но для этих АКБ, также как и для других, очень важен контроль процесса заряда и разряда, а специализированных микросхем для этого вида аккумуляторов не так много. Инженеры КОМПЭЛ подготовили список имеющихся микросхем и возможных решений от разных производителей. Подробнее>>

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[BARS_]

Все тормозит опрос датчика, т.к. 1-Wire довольно медленный протокол. Опрос надо проводить раз в секунду и только в том случае, если энкодер не вращают.

[Alex]

1 минуту назад, BARS_ сказал:

1-Wire довольно медленный протокол

Ну не на столько, чтобы визуально тормозить работу девайса. Просто нужно правильно подходить к алгоритму программы, а не тяп-ляп делать, как у автора. Тогда будет всё тип-топ...
 

[BARS_]

1 минуту назад, Alex сказал:

визуально тормозить работу девайса

Смотря как часто его долбить. Автор его зачем-то пытается 100 раз в секунду опросить. Плюс неизвестно, насколько кривая у него библиотека. А судя по тому, что температура возвращается во float, она кривая. Да и вообще прошивка написана на тяп ляп. Там, где с головой хватит char, автор лепит целый int. А энкодер вообще надо было на прерывание повесить. В общем мрак...

[Alex]

То что там тяп-ляп - спору нет. К примеру :

12 часа назад, zxz_rus сказал:

if (temper<=Temp)

При том, что одна переменная - float, вторая - int.
Такое сравнение не поддаётся ни какой логике. Т.к. temper будет приведён к int, и будут сравниваться целые значения, зачем тогда вообще заводить float-переменную ?
С таким подходом к программированию, не мудрено, что будут возникать проблемы. Советую ТС'у начать с изучения азов.

А библиотека эта, похоже, что из CVAVR, в которой функция ds18b20_temperature ничего, кроме геморроя, у начинающих не вызывает. Всем известно, что она тупо вешает программу на 1 секунду, при этом наглухо запрещая все прерывания.

[BARS_]

34 минуты назад, Alex сказал:

наглухо запрещая все прерывания

Да, да. Сам столкнулся с этим на заре изучения программирования. Благо довольно быстро распрощался с CV. 

Изменено 26 декабря, 2016 пользователем BARS_

[zxz_rus]

Ну это все конечно хорошо,  а более существенное что нибудь,  кто нибудь посоветует??? 

[BARS_]

11 час назад, Alex сказал:

Всем известно, что она тупо вешает программу на 1 секунду, при этом наглухо запрещая все прерывания.

А тебе этого мало? Полный ответ на твой вопрос

[romuko122]

Такой вариант не пойдет?

[Alex]

36 минут назад, zxz_rus сказал:

а более существенное что нибудь,  кто нибудь посоветует??? 

Самый существенный совет :

12 часа назад, Alex сказал:

Советую ТС'у начать с изучения азов.

 

PS: мне вот интересно, на что Вы расчитывали, начиная методом тыка собирать программу из кусков, не имея ни малейшего представления что делает каждая строка кода ? На чудо ? Или на то, что на форумах за Вас всё сделают ?
Наши подсказки более чем достаточны для устранения проблемы. Но Вы, вместо "спасиб", начинаете неблагодарно оскаливаться в нашу сторону.

[zxz_rus]

21 час назад, Alex сказал:

PS: мне вот интересно, на что Вы расчитывали, начиная методом тыка собирать программу из кусков, не имея ни малейшего представления что делает каждая строка кода ? На чудо ? Или на то, что на форумах за Вас всё сделают ?
Наши подсказки более чем достаточны для устранения проблемы. Но Вы, вместо "спасиб", начинаете неблагодарно оскаливаться в нашу сторону.

Alex, нет вы не правильно меня поняли. Я премного благодарен за помощь. А решение проблемы я так и не нашел, почему контроллер не переключает состояния порта. И отнюдь я не "не имею ни малейшего представления о  том что выполняет каждая строка".

21 час назад, romuko122 сказал:

Такой вариант не пойдет?

Спасибо. дельный совет. Но у меня задача не собрать конкретный девайс, а разобраться в работе кода и спроектировать свое. Вопрос заключался в том, что без энкодера все пашет. А с ним перестает. Порт не реагирует на сравнение. Компилятор не ругается. Прошу прощения если я кого то задел своими вопросами, и ответами. )

Изменено 27 декабря, 2016 пользователем zxz_rus

[BARS_]

33 минуты назад, zxz_rus сказал:

А решение проблемы я так и не нашел

Блин, да сказали же уже:

В 26.12.2016 в 11:35, Alex сказал:

ds18b20_temperature ничего, кроме геморроя, у начинающих не вызывает. Всем известно, что она тупо вешает программу на 1 секунду, при этом наглухо запрещая все прерывания.

Что еще надо для решения проблемы? Может код переписать? Плюс к этому использование sprintf худшее, что можно сделать.

33 минуты назад, zxz_rus сказал:

И отнюдь я не "не имею ни малейшего представления о  том что выполняет каждая строка"

По коду и вопросам этого не скажешь.

Изменено 27 декабря, 2016 пользователем BARS_

[Геннадий]

55 минут назад, zxz_rus сказал:

...А решение проблемы я так и не нашел...

Если будете искать решение, как в школе (писАть свое сочинение, составляя его из кусков чужих), то решения не будет. Будет только хохма достойная интернета.

Сначала надо разобраться как работает "железо" (датчик, МК, LCD, энкодер) как вместе, так и порознь друг от друга. А потом решать псевдопроблему.