ATmega8:не удается переключать каналы АЦП

[sad-angel]

На вход ADC0 и ADC1 микроконтроллера ATmega8 подаються измерительные сигналы. Их нужно обработать и вывести на дисплей. Написал простейшую программу, но она не работает - не происходит переключение каналов АЦП. Программа написана в WinAVR:

Код: Выделить всё

#include <avr/io.h>
#include <avr/interrupt.h>

void ADC_Init(void)
 {
  ADCSRA=((1<<ADEN)|(1<<ADIE)|(1<<ADPS2)|(1<<ADPS1));
  ADMUX=((1<<REFS1)|( 1<<REFS0)|(1<<ADLAR)); //ИОН-2.56в, 8 результат,0-й канал 
 }

//обработчик прерывания АЦП
ISR(ADC_vect)
{
 static unsigned char mode=0;

if(mode==0)
{
 PORTB=ADCH;
 mode=1;
 ADMUX|=1<<MUX0; //Enable ADC1;
}

if(mode==1)
{
 PORTD=ADCH;
 mode=0;
 ADMUX&=~(1<<MUX0);// Enable_ADC0;
}

 ADCSR |= (1<<ADSC); //Start ADC
}

int main(void)
{
 ADC_Init();
 ADCSR |= (1<<ADSC); //Start ADC

 DDRD=0xFF;
 DDRB=0xFF;
 sei();
 while(1)
 {
  asm("nop");
 }
}

[stas00n]

Код: Выделить всё

if(mode==0)
{
 PORTB=ADCH;
 mode=1;
 ADMUX|=1<<MUX0; //Enable ADC1; Тут по-моему забыли скобки - надо (1<<MUX0)
}

if(mode==1)
{
 PORTD=ADCH;
 mode=0;
 ADMUX&=~(1<<MUX0);// Enable_ADC0; 
}

 

[stas00n]

Кстати по этому поводу компилятор должен по идее выдать варнинг. Вы на них смотрите?

[stas00n]

Но дело не в этом. посмотрите на ваш обработчик. Для наглядности исключу все лишнее:

Код: Выделить всё

//обработчик прерывания АЦП
ISR(ADC_vect)
{
 static unsigned char mode=0;
if(mode==0)
{
 mode=1;
}
if(mode==1)
{
 mode=0;
}
}

Доступно?

[sad-angel]

Но дело не в этом. посмотрите на ваш обработчик. Для наглядности исключу все лишнее:

Код: Выделить всё

//обработчик прерывания АЦП
ISR(ADC_vect)
{
 static unsigned char mode=0;
if(mode==0)
{
 mode=1;
}
if(mode==1)
{
 mode=0;
}
}

Доступно?

Спасибо, я все понял! Исправил код:

Код: Выделить всё

ISR(ADC_vect)
{
static unsigned char mode=0;

if(mode==0)
{
PORTB=ADCH;
mode=1;
ADMUX|=1<<MUX0; //Enable ADC1;
}
else {
if(mode==1)
{
PORTD=ADCH;
mode=0;
ADMUX&=~(1<<MUX0);// Enable_ADC0;
}
}
ADCSR |= (1<<ADSC); //Start ADC
}

но потом сделал лучше:

Код: Выделить всё

ISR(ADC_vect)
{
static unsigned char mode=0;
switch(mode)
{ 
case 0:
{
PORTB=ADCH;
mode=1;
ADMUX|=1<<MUX0; //Enable ADC1;
ADCSR |= (1<<ADSC); //Start ADC
} break;

case 1:
{
PORTD=ADCH;
mode=0;
ADMUX&=~(1<<MUX0);// Enable_ADC0;
ADCSR |= (1<<ADSC); //Start ADC
} break; 
}
}

[ARV]

пожалуй, еще лучше было бы так:

Код: Выделить всё

ISR(ADC_vect){
   if(ADMUX & _BV(MUX0))
      PORTD = ADCH;
   else
      PORTB = ADCH;
   ADMUX ^= _BV(MUX0);
   ADCSRA |= _BV(ADSC);
}

[sad-angel]

пожалуй, еще лучше было бы так:

Код: Выделить всё

ISR(ADC_vect){
   if(ADMUX & _BV(MUX0))
      PORTD = ADCH;
   else
      PORTB = ADCH;
   ADMUX ^= _BV(MUX0);
   ADCSRA |= _BV(ADSC);
}

Спасибо!