АЦП в avr328p

Добрый день!

Помогите разобраться с получением результатов преобразования АЦП.

Написал вот такую программу в качестве примера:

Код (C++):

void ADC_init()
{
  ADMUX = (1 << REFS0); // V питания
  ADMUX |= (1 << ADIE);
  ADCSRA = (1 << ADEN) // Включаем ADC
  |(1 << ADPS2)|(1 << ADPS1)|(1 << ADPS0);
}
unsigned int ADC_convert()
{
  ADCSRA |= (1 << ADSC);
  while(!(ADCSRA >> 4)&1); // Ждем результатов
  unsigned int result_ADC = (ADCL|ADCH << 8);
  return result_ADC;
}
int main()
{
  ADC_init();
  Serial.begin(9600);
  while(1)
  {
    unsigned int result_ADC = ADC_convert();
    Serial.println(result_ADC);
  }
}

Эта программа должна получить показания АЦП с A0. К A0 подключен переменный резистор.
Проблема в том, что монитор порта постоянно показывает 74 или 73, в чем проблема?

 

Потенциометр подключен к А0. Я написал программу на Arduino для проверки, там результаты правильные.

 

да

 

Могу, дайте пару минут

 

Что этим хотели сказать?

 

не знаю, повлияет или нет - но я бы в процедуре adc_convert в while добавил скобки в условие (не уверен. что оно правильно вычисляется), а в чтении результата явно привел бы ADCH к типу uint_16

 

ADCSRA |= (1<<ADSC);
while(ADCSRA & (1<<ADSC));
return ADC;

 

я раньше так же пробовал, тоже самое

 

Не надо ничего выдумывать. Пиши так

Код (C++):

unsigned int result_ADC = ADC;

Результат лучше в прерывании снимать, а не тупо ждать.

 

Да, если убрать эту строчку, то все работает

 

Вот мои функции работы с АЦП (для 24 тиньки) в режиме 8 бит

Код (C++):

inline void ADC_On(void)
{
    ADCSRA =    (1<<ADEN) |                // Разрешаем работу ADC
        (1<<ADIE) |                // Разрешаем прерывание
        (1<<ADATE) |                // Старт преобразования вручную
        (0<<ADPS2) | (1<<ADPS1) | (1<<ADPS0);        // Прескалер = 8
}

inline void ADC_Start_Conversion (void)
{
     ADCSRA =    (1<<ADEN) |                // Разрешаем работу ADC
        (1<<ADIE) |                // Разрешаем прерывание
        (1<<ADATE) |                // Старт преобразования вручную
        (0<<ADPS2) | (1<<ADPS1) | (1<<ADPS0) |        // Прескалер = 8
        (1<<ADSC);                // Старт конвертации
}

inline void ADC_Off(void)
{
    ADCSRA =    (1<<ADEN) |                // Разрешаем работу ADC
        (0<<ADIE) |                // Запрещаем прерывание
        (1<<ADATE) |                // Старт преобразования вручную
        (0<<ADPS2) | (1<<ADPS1) | (1<<ADPS0) |        // Прескалер = 8
        (0<<ADSC);
}

inline void ADC_Init (void)
{
    DIDR0 =    (1<<ADC1D) | (1<<ADC2D) | (1<<ADC3D);
   
//    ADMUX =    AREF;          
    ADCSRB =    (1<<BIN) | (1<<ADLAR) | (0<<ADTS1) | (0<<ADTS0);    // Двухполярный режим работы
}

inline uint8_t ADC_Get_Data (void)
{
     return ADCH;
}

 

Всем большое спасибо за ответы

 

И вдогонку

Код (C++):

#define AREF_internal        (1<<REFS1) | (0<<REFS0)            // AREF 1.1v
#define AREF_external        (0<<REFS1) | (1<<REFS0)            // AREF external (PA0)
           
#define ADC_Change_in(Analog_input)    ADMUX = AREF_external | (Analog_input) //

 

Вот эта проверка:

не правильная.
Проще сделать так:

Код (C++):

while(ADCSRA & (1 << ADSC)); // Ждем результатов

 

И данные читать лучше так:

Код (C++):

unsigned int result_ADC = ADC; // вместо (ADCL|ADCH << 8);

компилятор сам разберётся в какой последовательности читать регистры ADCH и ADCL и как их "упаковывать" в одно число.

 

А если с помощью прерывания сделать то что я просил, это как лучше сделать?

 

Код (C++):

volatile uint16_t result_ADC; //в секции глобальных переменных

ISR(ADC_vect)
{
result_ADC = ADC;
}

Но лучше через состояние IDLE организовать.