Только что закончил небольшшой заказ в 10 устройств на ATtiny461. В каждом устройстве необходимо было мерить как униполярные, так
и дифф сигналы. Так как прослеживалась тенденция к повторным заказам, решил приложить определенные усилия к написанию программы
калибровки внутреннего ADC. Попытки сделать это чуть не сорвали сроки проекта - оказалось, что ADC ATtiny461 имеют существенные
разбросы по параметрам от экз к экз и др. тонкости:
1. Обычное униполярное измерение в 4 экземплярах из 10 показало ошибку в 20 LSB при напряжениях на входе около 2.2 вольт при
опоре 2.56 вольт. Вроде требования даташита выполнены - для униполярных сигналов все напряжения в доп пределах. Измерения низких
напряжений (в районе 100 mV) были практически идеальны. Насколько я понимаю, ADC в этих 4 экз. имеют слишком большой INL. Как с
этим бороться просто не представляю. А если бы устройств было не 10, а 100? Что, делать стенд для отбраковки кристаллов?
2. ADC в AVR по скорости работы и так аутсайдер, но определенные тонкости выявились и при переключениях каналов. Особенно когда
есть и униполярные и дифф. (у меня было по 2 сигнала каждого типа). Засада в том, что униполярные мерялись стабильно, но дифф
колбасило конкретно, сэмплы отличались почти в два раза. Сначала подумал, что PCB неправильно разведена, но вроде не первый раз
развожу, решил искать причину дальше. По-моему, из AVR аппликух была в свое время выдрана некая функция AdcStable():

C ней дифф сигналы стали измеряться нормально, но косяк проявился в том, что компилятор IAR v5.50 при оптимизации по скорости
сделал так, что цикл в этой функции становился вечным и я не смог разобраться в чем дело. При оптимизации по коду все работало
нормально. Однако это наблюдение покоя так и не дало, поэтому данную функцию пришлось выкинуть. В даташите на ATmega324 написано,
что при измер дифф сигналов схеме нужно время чтобы прошли переходные процессы переключения и это время указано - 125us. Как
только поставил паузу 200us после смены мультиплексора (измерения шли группой, но single conversions) - сразу все начало измеряться нормально. Но горький осадок остался: ADC и так не быстр, скорость выборок падает с увеличением кол-ва сигналов, дак еще и переходные задерживают... Что же делать? Кто как выкручивается?
Мне особенно интересно, как все это хозяйство калибруется в серию? Это же все серьезный риск получить приключений на собственное то самое место...

А на каком именно цикле он зацикливается?
Может сделать
?
А если на этом виснет
то это немного странно, но тоже проверять объявления CHKBIT ADCSRA ADSC добавить volatile куда надо(хотя какраз таки тут он должен быть уже по дефолту)...

А ещё может быть при оптимизации по скорости у вас вообще ADC не инициализируется, а цикл ожидания не может дождаться результатов.

нет, тут точно не виснет. Инициализация ADC происходит в начале программы один раз, кроме того с оптимизацией по коду все работает нормально.

да, наверно стоит попробовать... хотя в аппнотах вроде не было объявлений volatile. Хорошо, что проблема решилась простой задержкой 200us после переключения мультиплексора. Хотя сейчас я задумался, а не лучше ли вместо задержки просто запускать холостое преобразование? В том же даташите на atmega324 проскочила фраза, что при переключении дифф каналов это время необходимо в частности для инициализации схемы компенсации смещения на входах... Но все же ясно, что время холостого преобразования зависит от частоты тактирования ADC, и на высоких частотах одного холостого будет снова не хватать... Потому считаю, что метод введения задержки все же лучше.
Разобрался частично с калибровкой. Большая часть ошибки при N=900...1023 состояла в том, что я опрометчиво посчитал опору как 2560mV (типовое значение), а по факту оказалось 2500mV. Измеренное смещение ADC составило +1LSB, и после всего этого стало ясно что и gain error укладывается в +2LSB. В общем, тяжело, но разобрался. Если бы тока в AVR делали опору поточнее, было бы совсем здорово, а то не от чего оттолкнуться...

вообще для поточнее и побыстрее ставится внешний АЦП )

когда все в одном кристалле ессно будут вылазить разные неудобства, но все они решаемы в той или иной мере, как вы и сами нам рассказали )

такого говнокода не могло быть в AVR аппликухах. Программистов такого уровня в Atmel не держат.


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

Это Вы ерунду "сморозили"...
при смене на дифф. каналах нужна просто пауза на 125мс:
Special care should be taken when changing differential channels. Once a differential channel
has been selected, the gain stage may take as much as 125 μs to stabilize to the new value.
Thus conversions should not be started within the first 125 μs after selecting a new differential
channel. Alternatively, conversion results obtained within this period should be discarded.

Высокого же вы мнения о программистах атмеля. Однако этот код именно из их аппноты ( avr450 )

Да, высокого, вот оригинал по ссылке:


функция написана и отформатирована читаемо по крайней мере.


Отнюдь не ерунду, а то самое "alternatively" из ДШ.
Только не уточнил что АЦП не разгонять выше 8KSPS (125 мкс --> 8Khz).
Вот например как оно делается у меня для дифф каналов:


Куда приятнее в прерывании от АЦП просто забирать 100% верный результат, чем натыкивать delay_us() в основном цикле программы убивая при этом ее портируемость и способность к расширению.

То есть, по-вашему, отформатированный говнокод перестаёт быть говнокодом?

По поводу осмысленности кода - целиком согласен с defunct. Даже индийские программеры уже вроде бы запомнили, что нельзя читать неинициализированные автоматические переменные.
Измеряться правильно стало потому, что в функции делается минимум два измерения. Максимальное число измерений в функции зависит от начального содержимого signed int V[4].
Ещё правильнее то, что Вы эту функцию выкинули.

Дык других вариантов и нет - после переключения канала/режима измерения нужно немного подождать. Как ждать - через delay_us(), по таймеру или скипать одно измерение после переключения - решать Вам.

У АЦП АВР очень больное место одно - точность встроенной опоры. имхо, если исключить прочие неразумности, ничего кроме опоры калибровать не требуется. Правда, это тянет за собой деление (зачастую ненужное где-либо ещё в проге). Еще одно "но" - входные сигналы не должны превышать _минимальное_ даташитное Vref.
Естественный вариант - использовать внешнюю опору или внешний(-ие) АЦП.

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

Вы не ответили на мой вопрос. Вы считаете, что если говнокод отформатировать, то он перестанет быть говнокодом?
Или даже спрошу ещё более конкретно: считаете ли вы приведённый вами кусок кода от атмеля - говнокодом?

Я считаю, что любой нормальный код можно превратить в говнокод одним лишь форматированием. Например просто - разместить всё в одной строке.


Не считаю, потому что код оформлен читаемо - алгоритм читается, править/отладить легко, потенциальный баг видно.
Другой вопрос - дубовая (не расширяемая) реализация алгоритма - это сделано специально, т.к. основная задача любого наглядного пособия - донести суть/принцип работы, а не чтобы его можно было взять и бездумно вставить в свой проект.

В первом же варианте код не читается, править/отлаживать нереально, и баги не видно - это и делает его говнокодом.

Такой способ конечно возможен, но у него есть и недостатки.
- Вы почти в 2 раза уменьшаете итоговый sample rate АЦП
- окончание Sаmple & Hold(для "правильного" преобразования) заметно сдвигается во времени
от начала этого пребразования(это надо учитывать при необходимости)
delay_us() на 125мкс в основном цикле это уже какой-то ненормальный вариант,
таймеры еще никто не отменял...

1. На одну четверть (макс АЦП sample rate - 15kHz, а 125us полюбому надо пропустить).
2. Относительно предыдущего "правильного" преобразования для того же канала там будет всегда одинаковый интервал определяемый sample таймером плюс +/- jitter вносимый более приоритетными обработчиками прерываний.