Перейти к содержанию

Приём Комманд Через Uart


Рекомендуемые сообщения

Здравствуйте. Помогите кто чем может :) Вобщем есть список комманд, которые передёт программа контроллеру по интерфейсу RS-232.

Комманды разной длины. есть 4 байта. есть 6 байт и есть 8 байт. С проверкой комманд, CRC рзобрался, а вот как определить какой длины пришла команда никак не могу :( На пример делаю проверку по индесу буфера, если индекс = 4 - короткая, если индекс = 6 - длинная, постоянно срабатывает фрагмент индекс = 4....

Кусок кода выкладывать не буду, заранее уверен что он не правильный, если кто поможет буду очень благодарен, а то голову уже сломал :))))

Ссылка на комментарий
Поделиться на другие сайты

А физически они чем отличаются? Может ли короткая команда быть началом длинной? Например, команду 0x0055AAFF0055AAFF невозможно отличить от двух подряд идущих команд 0x0055AAFF. Если между командами гарантируется интервал много больший, чем между байтами команды, можно попытаться реализовать таймаут и таким образом определять конец команды.

Ругался на отсутствие форматирования исходного кода (включая отсутствие осмысленных комментариев и наличие неубранного после конфигуратора мусора) не менее 15 раз.

Часть моих наработок.

Ссылка на комментарий
Поделиться на другие сайты

В принципе я пришёл к той же мысли, сейчас пытаюсь её организовать.

Спасибо за ответ, сейчас напишу формат комманд, может что подскажешь

Команда начинаетс заголовком 0x02 следующий байт определяет длину, если 0x01 - 4 байта, если 0x02 - 5 байт, если 0x03 - 6 байт

Чет не работает... Вроде всё правильно...

switch(rx_buffer[0]){ //Проверяем заголовок команды
case 0x02: //если первый байт 0x02
delay_ms(3); //делаю паузу чтобы принять ост данные
#asm("cli") //выкл прерывания, дыбы обработать принятую команду
switch(rx_buffer[1]){ //Опред длину команды
case 0x01:
// Команда = 4 байта
break;

case 0x02:
// Команда = 5 байт
break;

case 0x03:
// Команда = 6 байт
break;
}
for(i=0;i<8;i++){rx_buffer[i] = 0;}
rx_wr_index = 0;
#asm("sei")
break;

default:
delay_ms(3);
#asm("cli")
for(i=0;i<8;i++){rx_buffer[i] = 0;}
rx_wr_index = 0;
#asm("sei")
break;  
}

Всё работает норм... вроде разобрался... :)

Изменено пользователем Gmail2014
Ссылка на комментарий
Поделиться на другие сайты

Сравнительное тестирование аккумуляторов EVE Energy и Samsung типоразмера 18650

Инженеры КОМПЭЛ провели сравнительное тестирование аккумуляторов EVE и Samsung популярного для бытовых и индустриальных применений типоразмера 18650. 

Для теста были выбраны аккумуляторы литий-никельмарганцевой системы: по два образца одного наименования каждого производителя – и протестированы на двух значениях тока разряда: 0,5 А и 2,5 А. Испытания проводились в нормальных условиях на электронной нагрузке EBD-USB от ZKEtech, а зарядка осуществлялась от лабораторного источника питания в режиме CC+CV в соответствии с рекомендациями в даташите на определенную модель. Подробнее>>

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

Новый аккумулятор EVE серии PLM для GSM-трекеров, работающих в жёстких условиях (до -40°С)

Компания EVE выпустила новый аккумулятор серии PLM, сочетающий в себе высокую безопасность, длительный срок службы, широкий температурный диапазон и высокую токоотдачу даже при отрицательной температуре. 

Эти аккумуляторы поддерживают заряд при температуре от -40/-20°С (сниженным значением тока), безопасны (не воспламеняются и не взрываются) при механическом повреждении (протыкание и сдавливание), устойчивы к вибрации. Они могут применяться как для автотранспорта (трекеры, маячки, сигнализация), так и для промышленных устройств мониторинга, IoT-устройств. Подробнее параметры и результаты тестов новой серии PLM по ссылке.

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

А где код? Приведенный кусок без начала и конца на него не тянет. Может проще так?

char command[8];					  //собственно команда
void receive_command(){		 //процедура приема команды
do{										    //ждем пока не придет 0x02
 command[0]=uart_read();	  //используя явное чтение из UART а не шаманские задержки
while(command!=0x02);
command[1]=uart_read();    //считываем длину. Почему-то в первом посте она была 4;6;8 а во втором - 4;5;6. Видимо формируется она своей программой, а тогда проще задать полную длину команды (4;6;8)
for(unsigned char i=2;i<command[1];i++)command[i]=uart_read(); //в цикле считываем остальные части команды
}

На самом деле это неправильный код, он останавливает программу до приема всего сообщения. Лучше сделать на прерываниях, но для проверки сойдет.

Зачем тайм-аут? Задать длину команд фиксированной, равной длине самой длинной.

Если команду формирует чья-то посторонняя программа - лезть в протокол невозможно, придется подстраиваться под то что есть. Из 1 поста не ясно, своя это программа или чужая.

Ругался на отсутствие форматирования исходного кода (включая отсутствие осмысленных комментариев и наличие неубранного после конфигуратора мусора) не менее 15 раз.

Часть моих наработок.

Ссылка на комментарий
Поделиться на другие сайты

Литиевые батарейки и аккумуляторы от мирового лидера  EVE в Компэл

Компания Компэл, официальный дистрибьютор EVE Energy, бренда №1 по производству химических источников тока (ХИТ) в мире, предлагает продукцию EVE как со склада, так и под заказ. Компания EVE широко известна в странах Европы, Америки и Юго-Восточной Азии уже более 20 лет. Недавно EVE была объявлена поставщиком новых аккумуляторных элементов круглого формата для электрических моделей «нового класса» компании BMW.

Продукция EVE предназначена для самого широкого спектра применений – от бытового до промышленного. Подробнее>>

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

В том то и беда что приходится под чужую дудку плясать...

Хотел на прерываниях, чё-то весь мозг сломал...

В приложении файл с рабочим кодом, если кто подскажет как грамотней написать? Чтобы быстродействие увеличить...

switch(rx_buffer[0]){
			    case 0x00:
			    break;

			    case 0x02:
				    delay_ms(3);
				    #asm("cli")
				    switch(rx_buffer[1]){
									    case 0x01:
										    for(i=0;i<4;i++){putchar(rx_buffer[i]);delay_ms(1);}
									    break;

									    case 0x02:
										    for(i=0;i<5;i++){putchar(rx_buffer[i]);delay_ms(1);}
									    break;

									    case 0x03:
										    for(i=0;i<6;i++){putchar(rx_buffer[i]);delay_ms(1);}
									    break;
									    }
				    for(i=0;i<8;i++){rx_buffer[i] = 0;}
				    rx_wr_index = 0;		  
				    #asm("sei")
			    break;

			    default:					 
				    delay_ms(3);
				    #asm("cli")
				    for(i=0;i<8;i++){rx_buffer[i] = 0;}
				    #asm("sei")
				    rx_wr_index = 0;
			    break;	 
			    }

Ссылка на комментарий
Поделиться на другие сайты

убрать delay_ms() и заменить на честное ожидание флага приема

выложить наконец код

Ругался на отсутствие форматирования исходного кода (включая отсутствие осмысленных комментариев и наличие неубранного после конфигуратора мусора) не менее 15 раз.

Часть моих наработок.

Ссылка на комментарий
Поделиться на другие сайты

убрать delay_ms() и заменить на честное ожидание флага приема

выложить наконец код

Если я правильно понял то проверять флаг RXCIE ?

***** или компилятор дурак или я давно не кодил :)

if(rx_buffer[2] == 0x35){putsf("OK");} //Это условие работает

А вот это условие не работает... Может я напутал чего? :)

if((rx_buffer[2] == 0x35) && (rx_buffer[3] == 0xFF)){putsf("Ver");}

Изменено пользователем Gmail2014
Ссылка на комментарий
Поделиться на другие сайты

должно работать. скорее всего в rx_buffer[3] не FF.

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

Ссылка на комментарий
Поделиться на другие сайты

Если я правильно понял то проверять флаг RXCIE ?

Не угадали. Правильнее всего это делать в прерывании по приему байта, но можно и по флагу. Только не RXCIE а RXC, то есть проверять пришел ли байт, а не разрешено ли прерывание. Изменено пользователем COKPOWEHEU

Ругался на отсутствие форматирования исходного кода (включая отсутствие осмысленных комментариев и наличие неубранного после конфигуратора мусора) не менее 15 раз.

Часть моих наработок.

Ссылка на комментарий
Поделиться на другие сайты

Получается 3 типа комманд (длинной 4,6,8 байт):

02 01 FF FF FF FF KS

02 02 FF FF FF FF FF FF KS

02 03 FF FF FF FF FF FF FF FF KS

02 - заголовок

01,02,03 - длина

FF...FF - данные

KS - контрольная сумма

#define BUFFER_SIZE 200			 // размер приемного буффера
unsigned char buffer[bUFFER_SIZE];	 // приемный буффер
volatile unsigned char counter;		 // счетчик
unsigned char otvet_length = 0;

// Прерывание по приему байта
interrupt [uSART_RXC] void usart_rx_isr(void)
{
char status,data;
status=UCSRA;
data=UDR;
if ((status & (FRAMING_ERROR | PARITY_ERROR | DATA_OVERRUN))==0)
{
buffer[counter] = data;		 // по приему каждого байта записываем его в буффер
if (++counter == BUFFER_SIZE) {		 // в counter считаем количество принятых байт
 counter = 0;
 }
}
}

void main(void)
{
...
while (1)
 {
if (counter>1) {
otvet_length = buffer[1];	 // длинна комманды 2-й байт в запросе (или 1-й в буффере, т.к. отсчет с 0)

// определяемся какой реальной длинне ответа
// соответствует принятая длинна
// 0х01 - 4 байта
//0х02 - 5 байт
// и т.д.

if (counter > otvet_length) {		 // ждем пока примется вся посылка
...
// считаем контрольную сумму //
...
// работаем с данными
}
counter = 0;	 // сбрасываем counter чтобы вновь не попасть под условие counter > otvet_length.
	 // и ждем новую комманду
   }
 }
}

Ссылка на комментарий
Поделиться на другие сайты

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

P.S. mr_smit на счёт описания комманд вы всё правильно расписали, стандартный вид. Ваш пример я буду завтра разбирать, если воможно скиньте ваши контакты в личку, хотел бы пообщаться. :)

if(rx_wr_index){

delay_ms(5); //Увеличил с 3 до 5 и всё заработало норм

#asm("cli")

if(rx_buffer[0] == 0x02){

switch(rx_buffer[1]){

case 0x01:

break;

case 0x02:

break;

case 0x03:

break;

}

}

#asm("sei")

rx_wr_index = 0;

}

Если я правильно понял то проверять флаг RXCIE ?

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

Спасибо за замечание, открыл настолюную книгу Евстифеева и посмотрел описание регистров UART, понял что прокосячил :) Давненько не кодил, подзабыл :)

Ссылка на комментарий
Поделиться на другие сайты

delay_ms(5); //Увеличил с 3 до 5 и всё заработало норм

facepalm.jpg

Ругался на отсутствие форматирования исходного кода (включая отсутствие осмысленных комментариев и наличие неубранного после конфигуратора мусора) не менее 15 раз.

Часть моих наработок.

Ссылка на комментарий
Поделиться на другие сайты

delay_ms(5); //Увеличил с 3 до 5 и всё заработало норм

facepalm.jpg

:dance2: Мне просто надо рабочий проект показать, тестовый так сказать, конечно я всё переделаю :crazy:

Ссылка на комментарий
Поделиться на другие сайты

Вот эта конструкция работает, мне кажется это самое лучшее решение, как вы думаете? Скрин в приложении.

if(!bittst1(UCSRA,7)){
#asm("cli")
if(rx_wr_index){
for(i=0;i<rx_wr_index;i++){putchar(rx_buffer[i]);}
rx_wr_index = 0;
}
#asm("sei")
}

P.S. Как код вставлять чтобы он нормально отображался?

post-177901-0-58303300-1395030277_thumb.png

Изменено пользователем Gmail2014
Ссылка на комментарий
Поделиться на другие сайты

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

Я делал так: буферизация по 8 байт (минимальное круглое число, которого мне хватило), циклические буферы на прием и передачу, без проверок на переполнение (протокол связи довольно медленный, переполнения не возникает, да и не опасно оно там).

uart.txt

pinmacro.txt

Ругался на отсутствие форматирования исходного кода (включая отсутствие осмысленных комментариев и наличие неубранного после конфигуратора мусора) не менее 15 раз.

Часть моих наработок.

Ссылка на комментарий
Поделиться на другие сайты

  • 4 недели спустя...

Честно говоря, ваши исходники кошмар какой-то.

А по теме - обработку принятой информации по УАПП проще всего организовать методом конечных автоматов.

Ниже пример протокола подобного вашему. Структура протокола: два синхробайта "0х55 0хАА", код команды, кол-во байт данных, байты данных, контрольная сумма.

bool checkByte(uint8_t byte) {
 uint8_t cnt = this->cnt_;
 buf[cnt] = byte;
 switch (cnt) {
 case 0:
  if (byte == 0x55)
   cnt++;
  break;
 case 1:
  cnt = (byte == 0xAA) ? 2 : 0;
  break;
 case 2:
  cnt++;
  break;
 case 3:
  if (byte < (size_ - 5)) {
   cnt++;
   maxLen_ = byte + 5;
  } else
   cnt = 0;
  break;
 default:
  cnt++;
  if (cnt >= maxLen_) {
   stat_ = PRTS_STATUS_READ_OK;
  }
  break;
 }
 this->cnt_ = cnt;
 return cnt;
}

Изменено пользователем BerZerKku
Ссылка на комментарий
Поделиться на другие сайты

Присоединяйтесь к обсуждению

Вы публикуете как гость. Если у вас есть аккаунт, авторизуйтесь, чтобы опубликовать от имени своего аккаунта.
Примечание: Ваш пост будет проверен модератором, прежде чем станет видимым.

Гость
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Ответить в этой теме...

×   Вставлено с форматированием.   Восстановить форматирование

  Разрешено использовать не более 75 эмодзи.

×   Ваша ссылка была автоматически встроена.   Отображать как обычную ссылку

×   Ваш предыдущий контент был восстановлен.   Очистить редактор

×   Вы не можете вставлять изображения напрямую. Загружайте или вставляйте изображения по ссылке.

Загрузка...
  • Последние посетители   0 пользователей онлайн

    • Ни одного зарегистрированного пользователя не просматривает данную страницу
×
×
  • Создать...