Проблема в коде CodeVisionAVR. Внутри.

[Vova777]

Не могу понять, в чем дело. Посмотрите, пожалуйста, мой код, у меня проблема - то ESP через раз к сети подключается, то MAC-адрес отправляет через раз. Как я устал, не могу понять, в чем дело - все же нормально, но работает через раз.

Спойлер

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

/*******************************************************
This program was created by the
CodeWizardAVR V3.12 Advanced
Automatic Program Generator
© Copyright 1998-2014 Pavel Haiduc, HP InfoTech s.r.l.
http://www.hpinfotech.com

Project : 
Version : 
Date    : 03.02.2016
Author  : 
Company : 
Comments: 


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

#include <mega16.h>
#include <delay.h>
#include <string.h>
#include <stdlib.h>

// Standard Input/Output functions
#include <stdio.h>

//=====================
// Declare your global variables here

// Числовые данные
int int_data = 0;                    // массив, в который выполняется конвертация команд, если поступают числа

// Счетчики
unsigned char COUNT_cycle = 0;       // счетчик основных циклов
unsigned char MAC_simbol_count = 0;  // перменная для счета символов '"' - для пропуска данных перед приемом MAC-адреса

// Маркеры
unsigned char MARKER_get_mac = 0;    // маркер - был ли запрос IP и MAC адресов
unsigned char START_rec_to_mas = 0;  // маркер - начало записи данных в массив команды
unsigned char COM_answer_ok = 0;     // маркер - был принят первый символ команды "OK"
 
// Массивы и их индексы
unsigned char adr_data[30];          // массив, заполняемый по мере приёма MAC-адреса
unsigned char y = 0;                 // индекс элементов массива adr_data
unsigned char str_data[10];          // массив, заполняемый по мере приёма команд
unsigned char x = 0;                 // индекс элементов массива str_data

// Команды
unsigned char COM_get_mac[]="AT+CIFSR";                                // Команда получить IP и MAC адреса ESP8266
unsigned char COM_send_mac[]="AT+CIPSEND=17";                          // Команда отправки полученного MAC-адреса
unsigned char LINK_type[]="AT+CIPMUX=0";                               // Команда установить режим одиночных подключений                          
unsigned char LINK_conf[]="AT+CIPSTART=\"TCP\",\"192.168.1.38\",8000"; // Команда для настройки TCP-соединения                                                         
unsigned char COM_send_id[]="AT+CIPSEND=10";                           // Команда отправки ID устройства   
unsigned char DATA_send_id[]="ID00000001";                             // ID устройства

unsigned char COM_send_ON[]="AT+CIPSEND=3";                            // Команда отправки ответа на команду ON
unsigned char DATA_send_ON[]=">ON";                                    // Ответ на команду ON
unsigned char COM_send_OFF[]="AT+CIPSEND=4";                           // Команда отправки ответа на команду OFF
unsigned char DATA_send_OFF[]=">OFF";                                  // Ответ на команду OFF
unsigned char COM_send_PWM[]="AT+CIPSEND=0,4";                         // Для отправки полученного числового значения ШИМ

// Тестовые переменные
unsigned char MARKER_send_mac;                                         // маркер - была отправка IP и MAC-адресов
unsigned char MARKER_com_ready;                                        // маркер - готовность принимать команды
                          
//=====================
// External Interrupt 0 service routine
interrupt [EXT_INT0] void ext_int0_isr(void)
{

}

// External Interrupt 1 service routine
interrupt [EXT_INT1] void ext_int1_isr(void)
{

}

#define DATA_REGISTER_EMPTY (1<<UDRE)
#define RX_COMPLETE (1<<RXC)
#define FRAMING_ERROR (1<<FE)
#define PARITY_ERROR (1<<UPE)
#define DATA_OVERRUN (1<<DOR)

// USART Receiver buffer
#define RX_BUFFER_SIZE 8
char rx_buffer[RX_BUFFER_SIZE];

#if RX_BUFFER_SIZE <= 256
unsigned char rx_wr_index=0,rx_rd_index=0;
#else
unsigned int rx_wr_index=0,rx_rd_index=0;
#endif

#if RX_BUFFER_SIZE < 256
unsigned char rx_counter=0;
#else
unsigned int rx_counter=0;
#endif

// This flag is set on USART Receiver buffer overflow
bit rx_buffer_overflow;

// USART Receiver interrupt service routine
interrupt [USART_RXC] void usart_rx_isr(void)
{
char status,data;
status=UCSRA;
data=UDR;

//=====================
//=====================
// Пропуск первых значений перед началом приема MAC-адреса
if ((data == '"')&&(MARKER_get_mac == 1)&&(START_rec_to_mas == 0))  
{          
  MAC_simbol_count++;   
}
//=====================
// Начало приёма MAC-адреса от ESP
if ((data == '"')&&(MAC_simbol_count == 3)&&(MARKER_get_mac == 1)&&(START_rec_to_mas == 0))  
{          
  memset(adr_data, '', sizeof(adr_data));  
  START_rec_to_mas = 1;
  y = 0; 
  return;
}
//=====================
// Заполнение массива принятым MAC-адресом
if ((START_rec_to_mas == 1)&&(data != '"')&&(MARKER_get_mac == 1))
{
  adr_data[y] = data;
  y++;
}
//=====================
// Анализ принятого MAC-адреса
if ((START_rec_to_mas == 1)&&(data == '"')&&(MARKER_get_mac == 1))
{    
  START_rec_to_mas = 0;
  MAC_simbol_count = 0;
  MARKER_send_mac = 0;
  y = 0;          
  return;     
}           

//=====================
//=====================
// Приём команды "ОК"
if ((data == 'O')&&(START_rec_to_mas == 0)) 
{
  COM_answer_ok = 1;
}

if ((data == 'K')&&(START_rec_to_mas == 0)&&(COM_answer_ok == 1))
{          
  PORTB.1=1;
  COM_answer_ok = 0; 
}

if (data == '*') PORTB.1=0;
/*
//=====================
//=====================
// Начало приёма команд от ESP
if ((data == ':')&&(START_rec_to_mas == 0)&&(MARKER_com_ready == 1)) 
{          
  memset(str_data, '', sizeof(str_data));   
  START_rec_to_mas = 1;
  x = 0;  
  return;
}

//=====================
// Заполнение массива принятой командой
if ((START_rec_to_mas == 1)&&(data != 13)&&(MARKER_com_ready == 1))
{
  str_data[x] = data;
  x++;
}

//=====================
// Анализ принятой команды
if ((START_rec_to_mas == 1)&&(data == 13)&&(MARKER_com_ready == 1))  
{  
  START_rec_to_mas = 0;
  x = 0;
    
    if (strncmp (str_data,"ON", 2) == 0) // Условие при получении команды "ON"
    {
       PORTB.0=1;
       OCR1A = 1023;       
       printf("%s\r\n", COM_send_ON);
       delay_ms(10);
       printf("%s\r\n", DATA_send_ON);
       delay_ms(10);
    }   
    if (strncmp (str_data,"OFF", 3) == 0) // Условие при получении команды "OFF"
    {
       PORTB.0=0;
       OCR1A = 0;       
       printf("%s\r\n", COM_send_OFF);
       delay_ms(10);
       printf("%s\r\n", DATA_send_OFF);
       delay_ms(10);
    } 
       
    if ((str_data[0] >= '0')&(str_data[0] <= '9')) // Условие при получении числовых данных
    {          
        int_data = atoi(str_data);
        OCR1A = int_data;
        
        printf("%s\r\n", COM_send_PWM);
        delay_ms(10);
        printf("%s\r\n", str_data);
        delay_ms(10);
        
        memset(str_data, '', sizeof(str_data));
                          
        if (int_data != 0)
        {
          PORTB.0=1;
        }
        else
        {
          PORTB.0=0;
        }
              
    }    
}
*/
//=====================

if ((status & (FRAMING_ERROR | PARITY_ERROR | DATA_OVERRUN))==0)
   {
   rx_buffer[rx_wr_index++]=data;
#if RX_BUFFER_SIZE == 256
   // special case for receiver buffer size=256
   if (++rx_counter == 0) rx_buffer_overflow=1;
#else
   if (rx_wr_index == RX_BUFFER_SIZE) rx_wr_index=0;
   if (++rx_counter == RX_BUFFER_SIZE)
      {
      rx_counter=0;
      rx_buffer_overflow=1;
      }
#endif
   }
}

#ifndef _DEBUG_TERMINAL_IO_
// Get a character from the USART Receiver buffer
#define _ALTERNATE_GETCHAR_
#pragma used+
char getchar(void)
{
char data;
while (rx_counter==0);
data=rx_buffer[rx_rd_index++];
#if RX_BUFFER_SIZE != 256
if (rx_rd_index == RX_BUFFER_SIZE) rx_rd_index=0;
#endif
#asm("cli")
--rx_counter;
#asm("sei")
return data;
}
#pragma used-
#endif

// USART Transmitter buffer
#define TX_BUFFER_SIZE 8
char tx_buffer[TX_BUFFER_SIZE];

#if TX_BUFFER_SIZE <= 256
unsigned char tx_wr_index=0,tx_rd_index=0;
#else
unsigned int tx_wr_index=0,tx_rd_index=0;
#endif

#if TX_BUFFER_SIZE < 256
unsigned char tx_counter=0;
#else
unsigned int tx_counter=0;
#endif

// USART Transmitter interrupt service routine
interrupt [USART_TXC] void usart_tx_isr(void)
{
if (tx_counter)
   {
   --tx_counter;
   UDR=tx_buffer[tx_rd_index++];
#if TX_BUFFER_SIZE != 256
   if (tx_rd_index == TX_BUFFER_SIZE) tx_rd_index=0;
#endif
   }
}

#ifndef _DEBUG_TERMINAL_IO_
// Write a character to the USART Transmitter buffer
#define _ALTERNATE_PUTCHAR_
#pragma used+
void putchar(char c)
{
while (tx_counter == TX_BUFFER_SIZE);
#asm("cli")
if (tx_counter || ((UCSRA & DATA_REGISTER_EMPTY)==0))
   {
   tx_buffer[tx_wr_index++]=c;
#if TX_BUFFER_SIZE != 256
   if (tx_wr_index == TX_BUFFER_SIZE) tx_wr_index=0;
#endif
   ++tx_counter;
   }
else
   UDR=c;
#asm("sei")
}
#pragma used-
#endif

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

// Input/Output Ports initialization
// Port A initialization
// Function: Bit7=In Bit6=In Bit5=In Bit4=In Bit3=In Bit2=In Bit1=In Bit0=In 
DDRA=(0<<DDA7) | (0<<DDA6) | (0<<DDA5) | (0<<DDA4) | (0<<DDA3) | (0<<DDA2) | (0<<DDA1) | (0<<DDA0);
// State: Bit7=T Bit6=T Bit5=T Bit4=T Bit3=T Bit2=T Bit1=T Bit0=T 
PORTA=(0<<PORTA7) | (0<<PORTA6) | (0<<PORTA5) | (0<<PORTA4) | (0<<PORTA3) | (0<<PORTA2) | (0<<PORTA1) | (0<<PORTA0);

// Port B initialization
// Function: Bit7=In Bit6=In Bit5=In Bit4=In Bit3=In Bit2=Out Bit1=Out Bit0=Out 
DDRB=(0<<DDB7) | (0<<DDB6) | (0<<DDB5) | (0<<DDB4) | (0<<DDB3) | (1<<DDB2) | (1<<DDB1) | (1<<DDB0);
// State: Bit7=T Bit6=T Bit5=T Bit4=T Bit3=T Bit2=0 Bit1=T Bit0=0 
PORTB=(0<<PORTB7) | (0<<PORTB6) | (0<<PORTB5) | (0<<PORTB4) | (0<<PORTB3) | (0<<PORTB2) | (0<<PORTB1) | (0<<PORTB0);

// Port C initialization
// Function: Bit7=In Bit6=In Bit5=In Bit4=In Bit3=In Bit2=In Bit1=In Bit0=In 
DDRC=(0<<DDC7) | (0<<DDC6) | (0<<DDC5) | (0<<DDC4) | (0<<DDC3) | (0<<DDC2) | (0<<DDC1) | (0<<DDC0);
// State: Bit7=T Bit6=T Bit5=T Bit4=T Bit3=T Bit2=T Bit1=T Bit0=T 
PORTC=(0<<PORTC7) | (0<<PORTC6) | (0<<PORTC5) | (0<<PORTC4) | (0<<PORTC3) | (0<<PORTC2) | (0<<PORTC1) | (0<<PORTC0);

// Port D initialization
// Function: Bit7=Out Bit6=In Bit5=Out Bit4=Out Bit3=In Bit2=In Bit1=In Bit0=In 
DDRD=(0<<DDD7) | (0<<DDD6) | (1<<DDD5) | (1<<DDD4) | (0<<DDD3) | (0<<DDD2) | (0<<DDD1) | (0<<DDD0);
// State: Bit7=T Bit6=T Bit5=T Bit4=T Bit3=T Bit2=T Bit1=T Bit0=T 
PORTD=(0<<PORTD7) | (0<<PORTD6) | (0<<PORTD5) | (0<<PORTD4) | (0<<PORTD3) | (0<<PORTD2) | (0<<PORTD1) | (0<<PORTD0);

// Timer/Counter 0 initialization
// Clock source: System Clock
// Clock value: Timer 0 Stopped
// Mode: Normal top=0xFF
// OC0 output: Disconnected
TCCR0=(0<<WGM00) | (0<<COM01) | (0<<COM00) | (0<<WGM01) | (0<<CS02) | (0<<CS01) | (0<<CS00);
TCNT0=0x00;
OCR0=0x00;

// Timer/Counter 1 initialization
TCCR1B = 0x00; //stop Timer
TCNT1H = 0xFC; // 11111100  
TCNT1L = 0x01; // 00000001  
OCR1AH = 0x03; // PWM(PD5) = OCR1A / 10.23 
OCR1AL = 0xFF; // 0x03FF это число 1023
OCR1BH = 0x03;
OCR1BL = 0xFF;
ICR1H  = 0x03;
ICR1L  = 0xFF; 
TCCR1A = 10000011;  
TCCR1B = 00001011;  // start Timer
OCR1A  = 0x00;

// Timer/Counter 2 initialization
// Clock source: System Clock
// Clock value: Timer 0 Stopped
// Mode: Normal top=0xFF
// OC2 output: Disconnected
TCCR2=(0<<WGM00) | (0<<COM01) | (0<<COM00) | (0<<WGM01) | (0<<CS02) | (0<<CS01) | (0<<CS00);
TCNT2=0x00;
OCR2=0x00;

// Timer(s)/Counter(s) Interrupt(s) initialization
TIMSK=(0<<OCIE2) | (0<<TOIE2) | (0<<TICIE1) | (0<<OCIE1A) | (0<<OCIE1B) | (0<<TOIE1) | (0<<OCIE0) | (0<<TOIE0);

// External Interrupt(s) initialization
// INT0: Off
// INT0 Mode: Low level
// INT1: Off
// INT1 Mode: Low level
// INT2: Off
GICR|=(0<<INT1) | (0<<INT0) | (0<<INT2);
MCUCR=(0<<ISC11) | (0<<ISC10) | (0<<ISC01) | (0<<ISC00);
MCUCSR=(0<<ISC2);
GIFR=(0<<INTF1) | (0<<INTF0) | (0<<INTF2);

// USART initialization
// Communication Parameters: 8 Data, 1 Stop, No Parity
// USART Receiver: On
// USART Transmitter: On
// USART Mode: Asynchronous
// USART Baud Rate: 9600
UCSRA=(0<<RXC) | (0<<TXC) | (0<<UDRE) | (0<<FE) | (0<<DOR) | (0<<UPE) | (0<<U2X) | (0<<MPCM);
UCSRB=(1<<RXCIE) | (1<<TXCIE) | (0<<UDRIE) | (1<<RXEN) | (1<<TXEN) | (0<<UCSZ2) | (0<<RXB8) | (0<<TXB8);
UCSRC=(1<<URSEL) | (0<<UMSEL) | (0<<UPM1) | (0<<UPM0) | (0<<USBS) | (1<<UCSZ1) | (1<<UCSZ0) | (0<<UCPOL);
UBRRH=0x00;
UBRRL=25;

// Analog Comparator initialization
// Analog Comparator: Off
// The Analog Comparator's positive input is
// connected to the AIN0 pin
// The Analog Comparator's negative input is
// connected to the AIN1 pin
ACSR=(1<<ACD) | (0<<ACBG) | (0<<ACO) | (0<<ACI) | (0<<ACIE) | (0<<ACIC) | (0<<ACIS1) | (0<<ACIS0);
SFIOR=(0<<ACME);

// ADC initialization
// ADC disabled
ADCSRA=(0<<ADEN) | (0<<ADSC) | (0<<ADATE) | (0<<ADIF) | (0<<ADIE) | (0<<ADPS2) | (0<<ADPS1) | (0<<ADPS0);

// SPI initialization
// SPI disabled
SPCR=(0<<SPIE) | (0<<SPE) | (0<<DORD) | (0<<MSTR) | (0<<CPOL) | (0<<CPHA) | (0<<SPR1) | (0<<SPR0);

// TWI initialization
// TWI disabled
TWCR=(0<<TWEA) | (0<<TWSTA) | (0<<TWSTO) | (0<<TWEN) | (0<<TWIE);

// Global enable interrupts
#asm("sei")

//=====================
// Инициализация ESP8266
delay_ms(5000);

printf("%s\r\n", LINK_type);
delay_ms(100); 
printf("%s\r\n", LINK_conf);
delay_ms(100);

delay_ms(1000);

printf("%s\r\n", LINK_type);
delay_ms(100); 
printf("%s\r\n", LINK_conf);
delay_ms(100);

delay_ms(1000);

printf("%s\r\n", LINK_type);
delay_ms(100); 
printf("%s\r\n", LINK_conf);
delay_ms(100);

delay_ms(1000);

printf("%s\r\n", COM_get_mac);    // Запрос IP и MAC адресов
MARKER_get_mac = 1;               // Маркер - запрос IP и MAC адресов был

//=====================
// Основной цикл программы
while (1)
  {      
      PORTB.2 = 1;
      delay_ms(200);
      PORTB.2 = 0;
      delay_ms(100);
      COUNT_cycle++;
      
//=====================      
      if (COUNT_cycle == 10)  // Отправка сообщения статуса
      {                                       
        printf("%s\r\n", COM_send_id);     // Отправка ID
        delay_ms(10);
        printf("%s\r\n", DATA_send_id);
        delay_ms(10);
        
        if (MARKER_send_mac == 0)
        {
          delay_ms(100);
          printf("%s\r\n", COM_send_mac);    // Отправка MAC-адреса
          delay_ms(10);
          printf("%s\r\n", adr_data);
          delay_ms(10);
          MARKER_send_mac = 1;
          
          delay_ms(500);
        }
                
        COUNT_cycle = 0;     
      }
                
  }
}    

[ARV]

вряд ли кто захочет изучать ваш код настолько глубоко, чтобы понять проблему однозначно.
я же, как и ранее, вижу в обработчике прерываний задержки, что потенциально является источником проблем.
кроме того, как и ранее, я вижу какие-то левые куски кода, который непонятно зачем присутствует: вроде бы вы самостоятельно разбираете приходящие данные, в то же время какие-то ошметки кода с буфером приема у вас тоже есть...
наконец, смущает тактовая частота 4 МГц - не маловато ли?

[Vova777]

я же, как и ранее, вижу в обработчике прерываний задержки, что потенциально является источником проблем

В перспективе, я попробую отойти от задержек, но сейчас без них вообще не работает ничего...

я вижу какие-то левые куски кода, который непонятно зачем присутствует: вроде бы вы самостоятельно разбираете приходящие данные, в то же время какие-то ошметки кода с буфером приема у вас тоже есть...

Эти "ошметки" автоматически сгенерировал CodeVisionAVR, я не стал убирать, вдруг нужно для нормальной работы...

наконец, смущает тактовая частота 4 МГц - не маловато ли?

Нет, оборудование отлично работает и так.

Мне нужна Ваша помощь здесь, кроме Вас мне не отвечает никто, не могут понять суть проблемы.

[ARV]

Судя по всему, вы решили сделать какой-то сложный по логике проект, но подходите к нему по-дилетантски.

Я вам который раз говорю: задержки в прерываниях - это плохо, тем более по 100 миллисекунд. Не забывайте, что пока вы сидите в одном прерывании, никакие другие (без особых ухищрений) не обрабатываются. Т.е. вы зависли на 100 мс в прерывании по приему байта, при этом передача по прерываниям не работает у вас, и наоборот. Эти эффекты могут приводить к очень трудноуловимым проблемам. Т.е. я не могу утверждать, что у вас не работает это и друге именно из-за задержек, но они могут влиять, и ни я, никто другой не скажет вам, как именно могут влиять. Просто прикиньте: при скорости USART 9600 бод передача/прием одного байта занимает примерно 1 мс. Т.е. пока там у вас длится задержка, девайс МОГ БЫ ответить вам пакетом из 100 байт! но вы этого никогда не узнаете, ибо прерывания по приему запрещены и все МОЖЕТ быть потеряно (я не утверждаю, т.к. не знаю логики обмена, но потенциально проблема МОЖЕТ иметь место).

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

Третье: вы активно используете в обработчике прерываний "тяжелые" функции типа sprintf - это так же далеко не хороший подход. Такой обработчик прерываний будет много регистров сохранять в стеке, а так же в стеке могут быть сохранены и внутренние переменные этих функций, что в конечном итоге может приводить к налезанию стека на область переменных, а в итоге - совершенно чудное поведение программы, не поддающееся разумному объяснению, вроде вашего "11 раз срабатывает, а потом прекращает".

Боюсь, что ничем, кроме этих общих расплывчатых советов, я вам не помогу:
- избавьтесь от задержек в прерываниях
- откажитесь от тяжелых функций в прерываниях
- очистите код от мусора
- разберитесь с синтаксисом (& против && и т.п. - раньше вы этим сильно грешили)

[Vova777]

- избавьтесь от задержек в прерываниях
- откажитесь от тяжелых функций в прерываниях

Если бы Вы подсказали, как это сделать, я был бы Вам очень признателен... Вот я как думаю... Если я в прерывании буду устанавливать некую переменную в единицу, а в основном цикле буду мониторить ее состояние и если там будет единица, то выполнять условие с функциями printf. Нормально так будет?

[ARV]

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

[Vova777]

Спасибо большое, Ваши ответы помогают...