UART к PC на CodeVision

[Fogtech]

Продолжая изучение AVR ATxmega128A1 столкнулся с проблемой при подключении платы с компьютером, а именно отправка слов (string) через UART.
При отправке сообщения "0", я получаю "0x98", что не есть "0".
При отправке сообщения "012345", часть "01234" приходит правильно, но добавляются лишние пакеты. К тому же пропадает предпоследний символ 0x0A.

Красным выделил ошибочные пакеты:

Send_UART("0");
41 98 0A 0D

Send_UART("012345");
41 30 31 32 33 34 4D 43 50 8A 92 9A A2 D6 52 0D

Вот мой скрипт:

//send_uart.h
#include ‹string.h›
#include ‹xmega128a1.h›

void Send_UART(char *data)
{

 char counter = 0x00;
 char lenght = 0x00;            

   lenght = strlen(data); // returns the length of the string 
   
   while (!( USARTC0.STATUS & USART_DREIF_bm));
   USARTC0.DATA = 0x41; // A
   
   while(counter ‹ lenght)
  {
      while (!(USARTC0.STATUS & USART_DREIF_bm));  // ((USARTC0.STATUS & USART_DREIF_bm) == 0);
      USARTC0.DATA = *(data+counter);
      counter++;
  }

   counter = 0x00;
   while (!( USARTC0.STATUS & USART_DREIF_bm));
   USARTC0.DATA = 0x0A; // LF new line
   
   while (!( USARTC0.STATUS & USART_DREIF_bm));
   USARTC0.DATA = 0x0D; //CR corriage return

отправляю через:

 Send_UART("012345");  // Send_UART("0");

Скорость передачи (9600) настроена верно. Ошибку предполагаю в команде strlen(data), так как через нее я хочу определить количество символов, которое "скачет".
Если проблема в этом, как можно ее обойти?

[ForcePoint]

М.б. тут

Send_UART("012345");

надо не просто литерал, а как-то указать, что это именно строка? Т.е. какой-то модификатор, приводящий при вызове Send_UART к аналогии такой примерно записи:

char   test_string[] = "012345";
Send_UART(test_string);

Ещё - в книжке написано, что strlen учитывает ноль, находящийся в конце строки.

Или нет?

[Boba_spb]

while(counter ‹ lenght)
{
while (!(USARTC0.STATUS & USART_DREIF_bm)); // ((USARTC0.STATUS & USART_DREIF_bm) == 0);
USARTC0.DATA = *(data+counter);
counter++;
}

Да сделайте проще - без strlen

while(*data)
{
while (!(USARTC0.STATUS & USART_DREIF_bm)); // ((USARTC0.STATUS & USART_DREIF_bm) == 0);
USARTC0.DATA = *data++;
}

[Fogtech]

М.б. тут

надо не просто литерал, а как-то указать, что это именно строка? Т.е. какой-то модификатор, приводящий при вызове Send_UART к аналогии такой примерно записи:

Код:

char test_string[] = "012345"; Send_UART(test_string);

Исходник брал отсюда. По всей видимости написано для WinAVR.

void Send_UART(char data[])
{
   char Counter = 0x00;
   char lenght = 0x00;            

   lenght = strlen(data);

   while(Counter ‹ lenght)
   {
      while (!(USARTC0.STATUS & USART_DREIF_bm));
      USARTC0.DATA = data[Counter];
      Counter++;
   }

   Counter = 0x00;
   while (!( USARTC0.STATUS & USART_DREIF_bm));
   USARTC0.DATA = 0x0A;
   while (!( USARTC0.STATUS & USART_DREIF_bm));
   USARTC0.DATA = 0x0D;
}

При компиляции CodeVision почему-то ругался на массивы.

function argument #1 of type 'flash unsigned char [11]' is incompatible with required parameter of type 'unsigned char []'

Пришлось код подогнать.

Ещё - в книжке написано, что strlen учитывает ноль, находящийся в конце строки.

Или нет?

Возможно (кстати в какой книге? не помешает самому почитать) только вот 6 лишних символов этим не объяснить.

[Fogtech]

Да сделайте проще - без strlen

while(*data)
{
while (!(USARTC0.STATUS & USART_DREIF_bm)); // ((USARTC0.STATUS & USART_DREIF_bm) == 0);
USARTC0.DATA = *data++;
}

Спасибо за код. Действительно проще написан, но результат увы тот же.

Send_UART("012345")
41 30 31 32 33 34 4D 43 50 8A 92 9A A2 D6 52 0D

Правда теперь strlen как причина отпадает.

[ForcePoint]

Пришлось код подогнать.

В результате, КМК, у Вас получается, что обращаетесь по указателю и пытаетесь применить к нему функцию для работы со строками. Она смотрит по указателю в память, ищет заверщающий строку ноль (я не знаю точно, но, м.б. литерал не содержит его) и возвращает что-то от балды найдя "чужой" ноль.

Возможно (кстати в какой книге? не помешает самому почитать) только вот 6 лишних символов этим не объяснить.

"Турбо С ++, язык и его применение", Цимбал и др. Или Керниган и Ричи.

[Fogtech]

В результате, КМК, у Вас получается, что обращаетесь по указателю и пытаетесь применить к нему функцию для работы со строками. Она смотрит по указателю в память, ищет заверщающий строку ноль (я не знаю точно, но, м.б. литерал не содержит его) и возвращает что-то от балды найдя "чужой" ноль.

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

Книгу увы найти не удалось (.

[Boba_spb]

В С строка заканчиваться нулевым байтом. strlen - дает длину без нуля.
ASM - не ставит ноль.

Надо смотреть что там во flash лежит на месте "01234"

[Fogtech]

Пошел путем меньшего сопротивления, оставив флеш на потом.
Решил использовать команду putsf из библиотеки stdio.h. Работает.

@ForcePoint и @Boba_spb
Спасибо за подсказки!

[Fogtech]

Работая над связью между ПК и Xmega128A1 столкнулся с проблемой, которую никак не могу обойти. А именно: МК отправляет данные только тогда, когда получает сигнал из вне. Для коммуникаций использую следующие функции:

/*...*/
//====================== Write to USARTC0 Transmitter ========================== 
void putchar_usartc0(unsigned char trans[12]) 
{
    while (!(USARTC0.STATUS & USART_DREIF_bm));
        puts(trans);  // Send string through RS232
}

//====================== Receive from USARTC0 ==================================
char getchar_usartc0(void)
{
    unsigned char rec; //

    while (1)
    {
        while (!(USARTC0.STATUS & USART_RXCIF_bm)); 
            rec=USARTC0.DATA;
        if((USARTC0.STATUS & (USART_FERR_bm | USART_PERR_bm | USART_BUFOVF_bm)) == 0)
        return rec;
    }
}
/*...*/

Пробовал изменять статусы (USARTC0.STATUS) и циклы, все напрасно... Прошу подсказать как должна выглядеть функция регулярной отправки. Спасибо.