Передача строки по UART

ARV · Сообщение **ARV** » Пн янв 09, 2017 22:09:13

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

разве не в этом смысл форума? разве дающий советы изначально не позиционирует сам себя, как более знающий?
я встреваю только в том случае, когда вижу попытку сделать хуже, чем было. топикстартер спросил совета, и я молчал, пока не увидел ВРЕДНЫЙ совет. в чем вредность - я обосновал. вы считаете, что из толерантности к чужому мнению, которое вредит другим, я должен промолчать? ок, теперь буду молчать, когда люди демонстрируют полную безграмотность: "АБ" - это два магических числа... да еще и с сарказмом демонстрируют...

а потом вы же сами будете отсылать новичка учить матчасть, когда он будет спрашивать "почему в Си в строке на 1 символ больше, чем в кавычках" - да? это лучше? причем отсылать будете куда как в более резкой форме, чем я.

ладно, теперь можете потереть тему, как модератор.

всё равно моё мнение не интересно тем, кто считает неприемлемым обращать внимание на собственные ляпы...

Реклама

LEX38RUS

В чем может быт л шибка?
Записал я данные принятые по UART в буфер далее по окончанию приёма активирую переменную и проверяю в цикле условие
If (start == 1)
If (strcmp(rx_buffer, buffer) == 0)
PORTC = 0b00000001

Где:
start переменная активируется после окончания приёма данных.
rx_buffer принятые данные
buffer переменная типа char заполненая символами ON тоесть char buffer[] = "ON"
В итоге два этих буфера должы быть одинаковы но на деле ни чего не работает.

Добавлено after 13 minutes 10 seconds:
Пробовал и таким способом

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

define RX_BUFFER   128    //длина нашего буфера, читай макс. длина строки

volatile unsigned char work=0;
volatile unsigned char rx_buffer[RX_BUFFER];
int rx_index;
ISR(USART_RXC_vect)
   {
   unsigned char status,data;
   status=UCSRA;
   data=UDR;
   if ((status&((1<<FE)|(1<<PE)|(1<<DOR)))==0)
      {
      if (data==10) // CR
      {
         work=1;
      }
      else rx_buffer[rx_index]=data;

      if (++rx_index == RX_BUFFER) rx_index=0;
      
      }
   
   }
в майне:
   if (work) 
      {
      work=0;   
                  //тут пробовал так же проверять бесполезно.
                }

Переменная work не принимает значение 1, через терминал посылал данные с окончанием символом CR

Реклама

но на деле ни чего не работает

Кот покажите весь полностью. Тяжело угадать, почему у Вас может не работать.

Добавлено after 14 minutes 17 seconds:

LEX38RUS писал(а):Пробовал и таким способом

А вот это что такое ? :

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

if ((status&((1<<FE)|(1<<PE)|(1<<DOR)))==0)

oleg110592

LEX38RUS писал(а):через терминал посылал данные с окончанием символом CR

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

if (data==10) // CR

В ASCII и Юникоде возврат каретки вроде определён как 13 (или шестнадцатеричное 0D), могу и ошибаться. Лучше обозначить магическим числом '\r'

имхо.

Реклама

LEX38RUS

делал и как 13 бесполезно

Реклама

oleg110592

взял исходник из первоисточника, наверное, покорежил так:

Спойлер

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

#include <avr/io.h>
#define F_CPU 8000000UL //8MHz
#include <util/delay.h>
#include <avr/interrupt.h>

uint8_t receive = 0;
uint8_t rx_data = 0;
volatile uint8_t rx_flag = 0;

void UARTInit(void) {
    UBRRH = 0;
    UBRRL = 51; //baud rate 9600
    UCSRB = (1<<RXEN)|(1<<TXEN)|(1<<RXCIE);
    UCSRC = (1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0); //8 bit, 1 stop bit
}

void UARTSend(uint8_t data) {
    while(!(UCSRA & (1<<UDRE)));
    UDR = data;
}

unsigned char UARTGet() {
    while(!rx_flag);
    rx_flag = 0;
    return rx_data;
}

int main(void) {
    sei();
    UARTInit();
    while(1) {
        receive = UARTGet();
        //receive++;
        //UARTSend(receive);
		UARTSend('O');
		UARTSend('K');
		UARTSend('\r');
    }
}

ISR(USART_RXC_vect) {
    rx_data = UDR;
	if (rx_data=='\r') // CR
	{
		rx_flag = 1;
	}	
}

в протеусе работает:

Реклама

LEX38RUS

Вот еще пример кода:

Спойлер

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

#define F_CPU 8000000
 
#include <avr/io.h>
#include <avr/interrupt.h>
#include <stdio.h>

#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)

#define RX_BUFFER_SIZE 16
char rx_buffer[RX_BUFFER_SIZE];
char buffer[] = "*ON$";

unsigned char rx_wr_index,rx_rd_index,rx_counter;

char rx_buffer_overflow;
char read_enable = 0;
volatile char _out = 0;
 
ISR(USART_RXC_vect )
{
	char status,data;
	status=UCSRA;
	data=UDR;
	if ((status & (FRAMING_ERROR | PARITY_ERROR | DATA_OVERRUN))==0)
	{
		if (data =='*')
		{
			rx_wr_index=0;
			read_enable = 1;
 
		}
 
		if((data == '$')&&(read_enable == 1))
		{
				read_enable = 0;
				_out = 1;				
		}
 
		if (read_enable == 1)
		{
			rx_buffer[rx_wr_index++]=data;
			if (rx_wr_index == RX_BUFFER_SIZE) rx_wr_index=0;
 
			if (++rx_counter == RX_BUFFER_SIZE)
			{
				rx_counter=0;
				rx_buffer_overflow=1;
			}
 
		}
 
	}
}
 
int main(){
 
        DDRC= 0b00000001;

	// USART initialization
	// Communication Parameters: 8 Data, 1 Stop, No Parity
	// USART Receiver: On
	// USART Transmitter: On
	// USART Mode: Asynchronous
	// USART Baud Rate: 9600
	UCSRA=0x00;
	UCSRB=0x98;
	UCSRC=0x86;
	UBRRH=0x00;
	UBRRL=0x33;
 
	sei();

 
	while (1)
	{
 
    	if (_out == 1)
		{	
                      If (strcmp(rx_buffer, buffer) == 0)
                          {
                             PORTC = 0b00000001;
                           }

			_out = 0;
			rx_wr_index= 0;
			rx_counter = 0;			
		}
	}
}

Если опять пытаюсь сравнить буфер

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

If (strcmp(rx_buffer, buffer) == 0)
PORTC = 0b00000001

то та же ни чего не работает.
Вообще не могу вкурить как его сравнить этот буфер принятый.

Здесь алгоритм такой, когда принимаем символ $ то активируем переменную _out = 1, и заканчиваем прием read_enable = 0
Далее уже в цикле сравниваем записанные данные в буфер.
Но ни чего не выходит.

oleg110592

корявенько так накарябал, может дядюшка Мокус поправит, сравнение работает

Спойлер

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

#include <avr/io.h>
#define F_CPU 8000000UL //8MHz
#include <util/delay.h>
#include <avr/interrupt.h>
#include <string.h>

#define RX_BUFFER_SIZE 5
uint8_t received;
char rx_data [5];
volatile uint8_t rx_flag = 0;
volatile uint8_t indexchar = 0;
char buffer[] = "*ON$";
 
void UARTInit(void) {
    UBRRH = 0;
    UBRRL = 51; //baud rate 9600
    UCSRB = (1<<RXEN)|(1<<TXEN)|(1<<RXCIE);
    UCSRC = (1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0); //8 bit, 1 stop bit
}

void UARTSend(uint8_t data) {
    while(!(UCSRA & (1<<UDRE)));
    UDR = data;
}

unsigned char UARTGet() {
    while(!rx_flag);
    rx_flag = 0;
    indexchar = 0;
    rx_data[4] = '\0';
    return 1;
}

int main(void) {
    DDRC = 0b00000001;
    sei();
    UARTInit();
    while(1) {
        received = UARTGet();
        if (strcmp(buffer, rx_data) == 0)
        {
            PORTC = 0b00000001;
        }
    }
}

ISR(USART_RXC_vect) {
    rx_data[indexchar] = UDR;
	if (rx_data[0] !='*')
	{
		indexchar = 0;
		return;
	}
    if ((rx_data[0]=='*') && (rx_data[indexchar]=='$'))
    {
        rx_flag = 1;
		return;
    }
    if(++indexchar >= RX_BUFFER_SIZE)
    {
        indexchar = 0;
    }
}

LEX38RUS

Благодарю. Будем изучать в чем ошибки у меня. Это я слизывал код с инета.

LEX38RUS

Код с поста выше:

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

#include <avr/io.h>
#define F_CPU 8000000UL //8MHz
#include <util/delay.h>
#include <avr/interrupt.h>
#include <string.h>

#define RX_BUFFER_SIZE 6
uint8_t received;
char rx_data [6];
volatile uint8_t rx_flag = 0;
volatile uint8_t indexchar = 0;

void UARTInit(void) {
	UBRRH = 0;
	UBRRL = 51; //baud rate 9600
	UCSRB = (1<<RXEN)|(1<<TXEN)|(1<<RXCIE);
	UCSRC = (1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0); //8 bit, 1 stop bit
}

void UARTSend(uint8_t data) {
	while(!(UCSRA & (1<<UDRE)));
	UDR = data;
}

unsigned char UARTGet() {
	while(!rx_flag);
	rx_flag = 0;
	indexchar = 0;
	rx_data[4] = '\0';
	return 1;
}

int main(void) {
	DDRC = 0b00000001;
	sei();
	UARTInit();
	while(1) {
		received = UARTGet();
		if (strcmp("*ON!", rx_data) == 0)
		{
			PORTC = 0b00000001;
		}
		else
		if (strcmp("*OF!", rx_data) == 0)
		{
			PORTC = 0b00000000;
		}

	}
}

ISR(USART_RXC_vect) {
	rx_data[indexchar] = UDR;
	if (rx_data[0] !='*')
	{
		indexchar = 0;
		return;
	}
	if ((rx_data[0]=='*') && (rx_data[indexchar]=='!'))
	{
		rx_flag = 1;
		return;
	}
	if(++indexchar >= RX_BUFFER_SIZE)
	{
		indexchar = 0;
	}
}

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

oleg110592

криминала в тексте не вижу, может фузы, кварц 8мГц присутствует?

LEX38RUS

Кварц внутренний на 8 стоит с фьюзами все ровно

Добавлено after 1 minute 46 seconds:
Вот скрин фьюзов

oleg110592

хм на скриншоте JTAG - это не atmega8? Птичка это включен или выключен?

LEX38RUS

Атмега 16. Забыл указать. Да включён.

oleg110592

так выключить, SPIEN не задеть!

LEX38RUS

А что изменится при отключение jtag?

oleg110592

глянул док. - вроде для PORTC0 отключение житага не даст ничего - может пробовать перейти на скорость UARTа ниже (1200), никогда меги с использованием UART без кварца не включал.

Z_h_e

oleg110592 писал(а):никогда меги с использованием UART без кварца не включал.

Включал несколько раз на 9600 мегу8 со внутренним RC генератором, а OSCCAL не менял . Проблем не было.

Добавлено after 6 minutes 35 seconds:

LEX38RUS писал(а):но если залить прошивку в кристалл то включить светодиод удается а вот выключить уже нет.

Я данный топик только бегло поглядывал. Хороший метод для тестирования UART, это возвращать на комп, что принял. В любой терминальной программе сразу будет видно.

LEX38RUS

Попробую внешний поставить на 12. Но думаю ни чего не изменится. Отпишусь о результатах. Скорость кстати понижал.
Включение то нормально отробатывает.

Добавлено after 5 minutes 11 seconds:
И добавлю чтоб назад отсылал то что принял. Это я так понимаю, что то вроде эхо?

oleg110592

Спойлер

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

        if (strcmp("*ON!", rx_data) == 0)
        {
            PORTC = 0b00000001;
            UARTSend('O');
            UARTSend('N');
            UARTSend('\r');
        }
        else if (strcmp("*OF!", rx_data) == 0)
        {
            PORTC = 0b00000000;
            UARTSend('O');
            UARTSend('F');
            UARTSend('\r');
        }

проверил протеус, мега16 - все четко