Проблема ADC DMA нарушен порядок значений в буфере

Y.Logvin · 20.03.2019, 14:13

На stm32f105 столкнулся с косяком.
В MX_ADC1_Init инициализируюся каналы в определенном порядке

Цитата:

sConfig.Channel = ADC_CHANNEL_0;
sConfig.Rank = ADC_REGULAR_RANK_1;

sConfig.Channel = ADC_CHANNEL_1;
sConfig.Rank = ADC_REGULAR_RANK_2;
...

Потом запускается АЦП.
HAL_ADC_Start_DMA(&hadc1, (uint32_t*)ADC1ConvertedValues, ADC_DMA_BUFF_SIZE);

И в ADC1ConvertedValues значение из канала 0 может оказаться на N-ной позиции. Порядок следования при этом не нарушается. Причем стабильно от запуска к запуску.
Стоит внести изменения в код(не всегда) - позиция может поплыть.

Кто то с подобной бедой сталкивался?

my504 · 20.03.2019, 14:39

К гадалке не ходи - нет синхронизации старта DMA с селектором каналов.
В результате начало блока плывет по массиву. Примерно тоже самое возникает при приеме пакетов следующих периодически через ДМА УАРТ.
ДМА стартует в момент середины текущего пакета и останавливается в середине следующего...
Еще возможна ошибочная установленная величина счетчика ДМА.

Y.Logvin · 20.03.2019, 18:04

Сообщение от my504

К гадалке не ходи - нет синхронизации старта DMA с селектором каналов.

Оно то так...

В обработчике HAL_ADC_ConvCpltCallback(ADC_HandleTypeDef* hadc)
значение счетчика толком посмотреть не получается hadc-›DMA_Handle-›Instance-›CNDTR.
При каждом следующем останове значение разное(смотрел через промежуточную переменную).

По идее - там всегда должна быть 9ка(использую 9 каналов).

С инициализацией тоже вроде ок - регистр инициализируется значением 9 (в HAL_ADC_Start_DMA).

mike-y-k · 20.03.2019, 18:27

Y.Logvin, тут только анализ кода после компиляции может помочь разобраться.
Листинг кода после препроцессора на входе компилятора и листинг ассемблера.
Попробуйте сделать простой пример на чистом C - все будет нормально работать.
С HAL придётся разбираться со всеми используемыми библиотеками

Y.Logvin · 20.03.2019, 20:09

В общем отчитываюсь - сам дурак.

Моя ошибка была в том, что сначала вызывал
HAL_ADC_Start(&hadc1),
а потом
HAL_ADC_Start_DMA(&hadc1, (uint32_t*)ADC1ConvertedValues, 9)

Нужно наоборот. Т.к. получалось, что АЦП начал перебирать каналы, и DMA цепляется к нему на лету. И там уже как попадет. Странно, что попадало почти всегда в одно место.

P.S. Явно базировался на каком то примере. Кто то мне свинью подложил. Проект начинался давно - хрен найду концы...

mike-y-k · 21.03.2019, 02:27

Y.Logvin, таки анализ кода был верным вектором движения

.
А эта ошибка с порядком инициализации DMA и источника/получателя наверное всех посещала

my504 · 21.03.2019, 05:37

Сообщение от Y.Logvin

Явно базировался на каком то примере.

Это прямое следствие использования HAL.

akegor · 21.03.2019, 06:36

Цитата:

- Хто всравсь?!
- Невiстка!
- Так вона ж теля пасе?!
- Так вiдтiля и несе!

Можно подумать, что без ХАЛа телегу впереди лошади запрячь невозможно!
А возможны перлы и еще смешнее... И не раз это наблюдается.
Не там причины ищете.

Y.Logvin · 21.03.2019, 11:10

Сообщение от my504

Это прямое следствие использования HAL.

У меня есть знакомый, который надрачивает на ассемблер. Но пишет почему то на С++.
HAL это полезная штука - позволяет зачастую не ковырятся в кишках. А это ускоряет разработку.
Да и код с HAL-ом понятней.
Я у себя усугубляю - всю работу с периферией прячу в класс Hal.

Ну а накосячить можно было и без него.
Про USART - из своей практики пример?

my504 · 21.03.2019, 13:03

Сообщение от Y.Logvin

Про USART - из своей практики пример?

Это не пример из практики. Это пример ПРОМЕЖУТОЧНОГО результата из моей практики. Когда запускаешь ДМА с УАРТом, то на выходе ЛОГИЧНО имеешь плывущее начало массива. Когда добавляешь механизм стартового поиска - все становится нормально.
Что касается HAL, то дело не в удобстве-неудобстве. Дело в скрытом в чужом коде функционале, который далеко не всегда понятен тому, кто пользуется HAL-ом. Более того, при ошибках алгоритма, результат рассматривается не непосредственно, а через призму HAL. То есть возникает ложное ощущение отсутствия ошибок в вызываемых функциях.
Точно как в случае с ТС.
Если бы он запускал диаграмму работы через CMSIS, то у него не возникло бы желания сначала стартануть АЦП, а лишь потом DMA.
Кстати, вообще то процесс должен быть В ПРИНЦИПЕ ДРУГОЙ.
Цепочка: семплирующий таймер-АЦП-ДМА. Запуск в обратном порядке. Точнее, АЦП-ДМА как угодно, а таймер запускает весь процесс.