Прием по USART Опции

[Jenya7]

Камень STM32F303CB.Принимаю данные

Код

void USART2_IRQHandler(void)
{
    if ((USART2->ISR & USART_ISR_RXNE) != RESET)
    {
        uint8_t chr = USART2->RDR;

        if (chr == '\r')  
        {
          usart2_rx_buf[usart2_rx_idx] = '\0';
          usart2_rx_idx = 0;
          usart2_rx_ready = 1;
        }
        else
        {
            if(usart2_rx_idx < RX2_BUFFERSIZE)  //no overflow
            {
                usart2_rx_buf[usart2_rx_idx] = chr;
                usart2_rx_idx++;
            }
            else
                usart2_rx_overflow = 1;
            }
    
    }
}

после рисета первый раз я попадаю в условие USART2->ISR & USART_ISR_RXNE - в регистре ISR я вижу RXNE = 1.
И все - все последующте разы я не захожу в условие так как RXNE = 0.
В документации написано что при приеме данных в USART2->RDR флаг взводиться RXNE = 1 а при чтении из USART2->RDR - флаг очищается RXNE =0.
Что я делаю не так?

Еще такая проблема - я все время захожу в USART2_IRQHandler - даже когда размыкаю провод RX - хотя у меня включено только одно прерывание - по принятию -

Код

USART_ITConfig(USARTx, USART_IT_RXNE, ENABLE);
NVIC_EnableIRQ(USART2_IRQn);

Сообщение отредактировал Jenya7 - Aug 31 2018, 11:58

[x893]

Начните с примеров.

[Jenya7]

Начните с примеров.

скачал примеры для F3Discovery - для USART примеров нет.

Нашел пример для моего камня

Код

void USART1_IRQHandler(void)
{
    if(USART_GetITStatus(USART1, USART_IT_RXNE) != RESET) // data receive
    {
        USART_ClearITPendingBit(USART1, USART_IT_RXNE);
        rx = (unsigned char) USART_ReceiveData(USART1);
        rxbuff[e] = rx;
        e++;
    }

сделал то же самое только для USART2 - та же беда.

Сообщение отредактировал Jenya7 - Aug 31 2018, 11:33

[x893]

скачал примеры для F3Discovery - для USART примеров нет.

Тогда надо сначала освоить поиск в гугле или на github

[Jenya7]

Тогда надо сначала освоить поиск в гугле или на github

мне интересно какая цель ваших ответов? чтоб количество сообщений было больше? за это что то дают? медаль какую?

[x893]

Смысл простой - надо научиться самому искать ответы на простейшие вопросы. А не задавать и потом читать килотонны дисскуссий ни о чём. При том, что в SPL/HAL примеры есть прямо на локальном диске после установки. Мне кажется спрашивать надо о том, что действительно вызывает трудности (нет в гугле или сложные алгоритмы) - например как по данным с аккселерометра отпределить активность (сон, бег, прыжок или падение). Это просто пример, а не вопрос.

А завтра у ТС возникнет FE или OV и опять пойдут вопросы. А когда он сам научится ответы искать, то и времени от проблемы до решения будет меньше тратиться.

[Сергей Борщ]

Еще такая проблема - я все время захожу в USART2_IRQHandler - даже когда размыкаю провод RX - хотя у меня включено только одно прерывание - по принятию -

Чудес не бывает. Либо у вас остался взведенным флаг этого события, либо разрешено прерывание по какому-то еще. Чтобы это узнать, достаточно прочитать регистры настройки и описание их содержимого. Попав в прервание сначала читаем SR, чтобы узнать, какие из флагов выставлены и какие биты отвечают за разрешение прерывания по этим флагам. Потом читаем регистры с этими битами разрешения и выясняем, какое именно событие могло вызвать переход в обработчик прерывания. Потом лезем в исходники "библиотеки" и выясняем, кто и зачем прерывания по этому событию разрешил. Неужели так сложно до этого додуматься самому? И да, без чтения документации у вас ничего не получится даже с самыми лучшими библиотеками, не говоря уже о "библиотеках".

[mcheb]

мне интересно какая цель ваших ответов? чтоб количество сообщений было больше? за это что то дают? медаль какую?

Чтобы не было лишних вопросов

Код

void USART2_IRQHandler(void)
{
    if(USART2->ISR & USART_ISR_RXNE)  // Данные пришли?
    {
        IN_FIFO.buf[IN_FIFO.Put++ % DATABUFSIZE]=USART2->RDR; // кладём в буфер
    }
}
//
void uart2_putc(unsigned char ch)
{
    USART2->TDR=ch;
    while(!(USART2->ISR & USART_ISR_TC));
}
//

[Arlleex]

Чтобы не было лишних вопросов...

Это зря Вы так думаете, что их не будет
И, кстати, Ваш пример не совсем корректен. Если данных будет слишком много, Вы будете их постоянно принимать, т.к. никакого события переполнения входного буфера, видимо, не предусмотрено.

Но как пример, вполне себе...

Сообщение отредактировал Arlleex - Sep 1 2018, 12:16

[Jenya7]

я отладил прием

Код

void USART2_IRQHandler(void)
{
    uint8_t chr;
    uint32_t sr = USART2->ISR;
    if (sr & USART_ISR_RXNE)
    {
        chr = (uint16_t)(USART2->RDR);
      
        if (chr == '\r')  
        {
            usart2_rx_buf[usart2_rx_idx] = '\0';
            usart2_rx_idx = 0;
            usart2_rx_ready = 1;
        }
        else
        {
            if(usart2_rx_idx < RX2_BUFFERSIZE)  //no overflow
            {
                usart2_rx_buf[usart2_rx_idx] = chr;
                usart2_rx_idx++;
            }
            else
              usart2_rx_overflow = 1;
        }
    }
    USART2->ICR = (uint16_t)~USART_FLAG_RXNE;
}

Проблема такая. Мастер посылает

Код

USART_SendBuf(USART2, "MBE\r", 4);

Delay_ms(100);

И слейв принимает исправно. Но если убираю задержку - Delay_ms(100); слейв принимает как попало "МMBE\r" , "MBЕE\r" , "\0BЕE\r". Как можно улучшить прием?

10 мили - Delay_ms(10) - тоже работает нормально. меньше - ломается.

нет Delay_ms(10) - тоже ломается.

Сообщение отредактировал Jenya7 - Sep 7 2018, 08:58

[HardEgor]

Delay_ms(100);[/code]И слейв принимает исправно. Но если убираю задержку - Delay_ms(100); слейв принимает как попало "МMBE\r" , "MBЕE\r" , "\0BЕE\r". Как можно улучшить прием?

Так может у вас мастер(USART_SendBuf) сливает символы? - На последние биты последнего символа накладываются биты следующего символа. Хотя вроде "\r" нормально принимается.... или теряется вся предыдущая строка?
А у вас случайно не RS485 полудуплекс?

[Jenya7]

Так может у вас мастер(USART_SendBuf) сливает символы? - На последние биты последнего символа накладываются биты следующего символа. Хотя вроде "\r" нормально принимается.... или теряется вся предыдущая строка?
А у вас случайно не RS485 полудуплекс?

Я для отладки подключил напрямую пины
MASTER_USART2_TX - SLAVE_USART2_RX
MASTER_USART2_RX - SLAVE_USART2_TX
Ставлю точку останова на стороне мастера - вижу четко уходит строка - никаких накладок. а если ставлю точку останова со стороны слейва, на строке usart2_rx_ready = 1; то вижу накладки.

поставил скоп на MASTER_USART2_TX - выходящий пакет довольно красивый.

Сообщение отредактировал Jenya7 - Sep 7 2018, 09:32

[HardEgor]

Ставлю точку останова на стороне мастера - вижу четко уходит строка - никаких накладок. а если ставлю точку останова со стороны слейва, на строке usart2_rx_ready = 1; то вижу накладки.

На какое событие в передатчике вы ставите точку останова? Transmit data register empty или Transmission complete?

[Jenya7]

На какое событие в передатчике вы ставите точку останова? Transmit data register empty или Transmission complete?

передатчик работает по Transmission complete

Код

void USART_SendBuf(USART_TypeDef *USARTx, uint8_t *data, uint32_t size)
{
  while (size--)
  {
      uint32_t timeout = USART_TIMEOUT;
      // wait until data register is empty
      while( !(USARTx->ISR & USART_FLAG_TC) ) {if(!timeout--) break; }    
      USARTx->TDR = *data++;
  }
}

[HardEgor]

передатчик работает по Transmission complete

Тогда возможные проблемы с передатчиком смотреть осциллографом, а лучше лог.анализатором.
А со стороны приемника, после приема байта, надо еще проверить Overrun error bit, заодно и всё что еще есть по ошибкам в USART.

[Сергей Борщ]

10 мили - Delay_ms(10) - тоже работает нормально. меньше - ломается.

нет Delay_ms(10) - тоже ломается.

Помедитируйте над строчкой "USART2->ICR = (uint16_t)~USART_FLAG_RXNE;" Если вы хотите сбросить флпг приема, то, во-первых не нужен "~", а во-вторых этот флаг уже сбросился, когда вы читали RDR. Если же вы хотели сбросить все остальные флаги - то как-то то странно делать это без проверки, да и передачу такой сброс наверняка поломает.

[jcxz]

Помедитируйте над строчкой "USART2->ICR = (uint16_t)~USART_FLAG_RXNE;" Если вы хотите сбросить флпг приема, то, во-первых не нужен "~", а во-вторых этот флаг уже сбросился, когда вы читали RDR. Если же вы хотели сбросить все остальные флаги - то как-то то странно делать это без проверки, да и передачу такой сброс наверняка поломает.

Он в первую очередь приём поломает.
Самое удивительное, что такие горе-пейсатели как будто друг у друга быдлокод передирают. Только что в соседней теме по аналогичному случаю советовал автору подумать, что будет если между чтением регистра приёма UART и последующим принудительным сбросом флага успеет придти символ.
Казалось бы очевидная ситуация - как можно не видеть такого???
Зато "баги" IAR-а находят чуть не каждый день.

[Jenya7]

Помедитируйте над строчкой "USART2->ICR = (uint16_t)~USART_FLAG_RXNE;" Если вы хотите сбросить флпг приема, то, во-первых не нужен "~", а во-вторых этот флаг уже сбросился, когда вы читали RDR. Если же вы хотели сбросить все остальные флаги - то как-то то странно делать это без проверки, да и передачу такой сброс наверняка поломает.

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

[Arlleex]

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

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

[Сергей Борщ]

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

То есть читать документацию вы не хотите принципиально. Тогда я пас.

[Jenya7]

То есть читать документацию вы не хотите принципиально. Тогда я пас.

я не нашел в документации ответа на проблему. если знаете укажите.

[Сергей Борщ]

если знаете укажите.

Указывал уже. Толку-то?

Чудес не бывает. Либо у вас остался взведенным флаг этого события, либо разрешено прерывание по какому-то еще. Чтобы это узнать, достаточно прочитать регистры настройки и описание их содержимого. Попав в прервание сначала читаем SR, чтобы узнать, какие из флагов выставлены и какие биты отвечают за разрешение прерывания по этим флагам. Потом читаем регистры с этими битами разрешения и выясняем, какое именно событие могло вызвать переход в обработчик прерывания.

[Jenya7]

Указывал уже. Толку-то?

У меня разрешено только одно прерывание RXNEIE = 1 - только оно. И в регистрах при отладке я вижу только его.
Флаг RXNE при чтении регистра USART2->RDR очищается. В регистрах при отладке я это вижу.
Вы можете и дальше умничать и говорить тривиальные вещи. Главное чтоб Вам от этого было хорошо. Я для этого открываю темы. Чтоб люди заходили и объясняли мне какое я гавно и какие они умные. А то я ж до сих пор это не понял.

Сообщение отредактировал Jenya7 - Sep 9 2018, 07:42

[Сергей Борщ]

У меня разрешено только одно прерывание RXNEIE = 1 - только оно. И в регистрах при отладке я вижу только его.

Еще раз повторяю: чудес не бывает. RXNEIE разрешает прерывание не только по RXNE. Читайте документацию внимательно.

Флаг RXNE при чтении регистра USART2->RDR очищается. В регистрах при отладке я это вижу.

Какие еще флаги взведены? Вам самому не стало интересно, что же такого магического вы сбрасываете той сторокой, если RNXE уже был сброшен.

Я для этого открываю темы.

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

[Arlleex]

У меня разрешено только одно прерывание RXNEIE = 1 - только оно. И в регистрах при отладке я вижу только его.
Флаг RXNE при чтении регистра USART2->RDR очищается. В регистрах при отладке я это вижу.

Я что-то не понимаю: зачем просматривать отладчиком периферийные регистры, чувствительные к чтению? Естественно, нормально ничего работать не будет, потому что на точке останова в начале прерывания отладчик считывает регистры USART-модуля, в том числе RDR. При этом очищается RXNE и поэтому дальше по коду RXNE никогда не будет взведен.

1. В обработчике вставьте такой код

Код

unsigned int USART_ISRValue;
    
void USART1_IRQHandler(void)
{
    USART_ISRValue = USART1->ISR;
    
    if(USART_GetITStatus(USART1, USART_IT_RXNE) == SET)
    {
        unsigned char CurrentSymbol = USART_ReceiveData(USART1);
        
        USART_SendData(USART1, CurrentSymbol);
        
        USART_ClearITPendingBit(USART1, USART_IT_RXNE);
    }
    
    return;
}

2. Зайдите в режим отладки.
3. Сбросьте все точки останова.
4. Закройте окно регистров USART в отладчике.
5. Поставьте точку останова на "return" в обработчике прерывания.
6. Откройте любой терминал, и отправьте 1 символ на плату - у Вас сработает точка останова.
7. Зафиксируйте значение USART_ISRValue - какое оно?
8. Теперь откройте окошки регистров USART.
9. Не убирая точки останова, убедитесь, что все работает также.
10. А вот теперь переместите точку останова на строчку

Код

USART_ISRValue = USART1->ISR;

11. Отправьте символ на МК, дождитесь срабатывания точки останова.
12. Сделайте 1 шаг (считайте значение USART_ISRValue). Как видно, оно отличается от значения в прошлых пунктах.

Выводы:

1. Нельзя пользоваться бездумно отладчиком при работе с чувствительной к чтению регистровой моделью периферии.
2. USART2->ICR = (uint16_t)~USART_FLAG_RXNE; - полная параша, убрать это нужно.

Проблема такая. Мастер посылает

Код

USART_SendBuf(USART2, "MBE\r", 4);

Delay_ms(100);

И слейв принимает исправно. Но если убираю задержку - Delay_ms(100); слейв принимает как попало "МMBE\r" , "MBЕE\r" , "\0BЕE\r". Как можно улучшить прием?

10 мили - Delay_ms(10) - тоже работает нормально. меньше - ломается.

нет Delay_ms(10) - тоже ломается.

4 - это количество символов? Ну если это так - то, во-первых, понятно, почему принимается всегда по-разному, а во-вторых, такой код никуда не годится - Вам, как пользователю этой функции нафиг знать не нужно, сколько символов в ней. Она должна отправить строку. А строка в Си уже имеет искусственный ограничитель в конце.
Устали Вы меня...

[Jenya7]

4 - это количество символов? Ну если это так - то, во-первых, понятно, почему принимается всегда по-разному, а во-вторых, такой код никуда не годится - Вам, как пользователю этой функции нафиг знать не нужно, сколько символов в ней. Она должна отправить строку. А строка в Си уже имеет искусственный ограничитель в конце.
Устали Вы меня...

а что плохого в USART_SendBuf(USART2, "MBE\r", 4)? я посылаю 4 символа и почему они должны приниматься по разному?

я посмотрел регистры поглубже и выявил проблему. при быстрой посылке взводится флаг ORE (overrun error), NF (noise flag) и иногда FE (framing error).
вопрос как с этим бороться.

если посылать пинг-понг ORE и FE не взводяться. хотя NF иногда проскакивает.

Сообщение отредактировал Jenya7 - Sep 9 2018, 12:28

[Arlleex]

а что плохого в USART_SendBuf(USART2, "MBE\r", 4)? я посылаю 4 символа и почему они должны приниматься по разному?

А, не туда посмотрел.
Работать будет, но все равно не ясен смысл еще одного аргумента (количества передаваемых символов). Почему не

Код

USART_SendBuf(USART2, "MBE\r");

я посмотрел регистры поглубже и выявил проблему. при быстрой посылке взводится флаг ORE (overrun error), NF (noise flag) и иногда FE (framing error).
вопрос как с этим бороться.

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

[Jenya7]

А, не туда посмотрел.
Работать будет, но все равно не ясен смысл еще одного аргумента (количества передаваемых символов). Почему не

Код

USART_SendBuf(USART2, "MBE\r");

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

у меня есть посылка строки

Код

void USART_SendString(USART_TypeDef *USARTx, const char *string)
{
  while(*string)
  {
    uint32_t timeout = USART_TIMEOUT;
    // wait until data register is empty
    while( !(USARTx->ISR & USART_FLAG_TC) ) {  if(!timeout--) break; }
    USARTx->TDR = *string++; //
  }
}

это я для отладки строку посылаю. но реально я буду посылать не ASCII символы а uint8_t.

частота кора 72 Мега. скорость UART 115200.

кстати мне не понятно почему по флагам ORE и FE генерируется прерывание.

Сообщение отредактировал Jenya7 - Sep 9 2018, 12:55

[Arlleex]

USART_TIMEOUT

Чему равен? Есть уверенность, что отправляющий UART не напихивает в регистр без продыху?

[Jenya7]

Чему равен? Есть уверенность, что отправляющий UART не напихивает в регистр без продыху?

USART_TIMEOUT = 100000. я стал точкой останова на timeout-- максимум 10 - 20 итераций происходит.

убрал { if(!timeout--) break; } вроде нормально флаг отрабатывает. символ ушел флаг взвелся.

Сообщение отредактировал Jenya7 - Sep 9 2018, 13:34

[Сергей Борщ]

кстати мне не понятно почему по флагам ORE и FE генерируется прерывание.

почти на каждый ваш вопрос ответ в документации есть.