АЦП и ATmega48, как работать с несколькими каналами Опции

[Дмитрий_Мигачев]

Вообщем не могу понять как работать с несколькими каналами АЦП. Юзаю VMLab, он меня всем удовлетворяет, сейчас пишу тут одну вещь и встал...Дело в том что есть какой то пульт управления, и результаты с рычагов должны оцифровываться 50 раз в секунду, вообщем как я думал, по переполнению таймера счетчика, мы уходим в прерывание, где тама, тама производим необходимую настройку ацп, далее разрешаем прерывание по завершению АЦП и так с двумя каналами. Я думал что бы работать с двумя каналами необходимо завести переменную, к примеру в начеле она равна нулю, после первого АЦП она инкрементируется, и далее производиться обработка второго канала, но компилятор выдает ошибком, вообщем мне кажется что я выбрал не правильный путь, подскажите как сделать правильней
Для понятливости того, что я написал приведу пример своего кода:
*************************
*************************
unsigned char adc_chan, b;
unsigned short kanal1, kanal2;
SIGNAL(SIG_OVERFLOW1){
asm("nop");
TCNT1=0xFFEB;
if (adc_chan==0) {
ADCSRA=_BV(ADEN)+_BV(ADIF)+_BV(ADIE)+_BV(ADPS0);
ADMUX=0; //ADC0
ADCSRA|=_BV(ADSC);
asm("sei");
if (adc_chan==1) {
ADCSRA=_BV(ADEN)+_BV(ADIF)+_BV(ADIE)+_BV(ADPS0);
ADMUX=_BV(MUX2)+_BV(MUX1)+_BV(MUX0);//ADC7
ADCSRA|=_BV(ADSC);
asm("sei");
}
}
SIGNAL(SIG_ADC){
if (adc_chan==0){
b=ADCL;
kanal1=(ADCH<<8)+b;
adc_chan++;
}
if (adc_chan==1){
b=ADCL;
kanal2=(ADCH<<8)+b;
adc_chan=0;
}
}
void idle_init(void) {
SMCR=_BV(SE);
asm("sei");
}
void timer1_init(void) {
TCCR1B=_BV(CS12)+_BV(CS10);//CLK/1024
TCNT1=0xFFEB;
TIMSK1=_BV(TOIE1);
}
// ***********************************************************
// Main program
//
int main(void) {
idle_init();
timer1_init();
adc_chan=0;
while(1) {
asm("sleep");
}

}
но он матюгается на этот код, а как сделать правильней?

[haker_fox]

Первое, что бы я пожелал: оформляйте код в специальных тегах, вот так:

Код

int My_global;
unsigned char adc_chan;
unsigned short kanal1, kanal2;

SIGNAL(SIG_OVERFLOW1)
  {
    TCNT1=0xFFEB;
    switch(adc_chan)
      }  
         case 0:
         ADMUX=0;
         break;

         case 1:
         ADMUX=_BV(MUX2)+_BV(MUX1)+_BV(MUX0);
         break;
       }
    ADCSRA|=_BV(ADSC);
  }


SIGNAL(SIG_ADC)
  {
    switch(adc_chan)
      {
        case 0:
        kanal1=ADC;
        adc_chan++;
        break;

        case 1:
        kanal2=ADC;
        adc_chan=0;
        break;
      }
    ADCSRA&=~_BV(3);
  }


void idle_init(void)
  {
    SMCR=_BV(SE);
  }

void timer1_init(void)
  {
    TCCR1B=_BV(CS12)+_BV(CS10);
    TCNT1=0xFFEB;
    TIMSK1=_BV(TOIE1);
  }

int main(void)
  {
        ADCSRA=_BV(ADEN)+_BV(ADIF)+_BV(ADIE)+_BV(ADPS1)+_BV(ADPS0);  
    idle_init();
    timer1_init();
    adc_chan=0;
    while(1)
      {
        asm("sleep");
      }
  }

Ну примерно бы так я офоромил программу, правда еще надо комментарии дописать. Ну и на всякий случай на симуляторе прогнать: нет ли где ошибок)))

[Дмитрий_Мигачев]

Первое, что бы я пожелал: оформляйте код в специальных тегах, вот так:
...

Ну я понял, что вы изменили, спасибо, но вопрос другой, это оптимальный вариант работы с АЦП (именно в моем случае с двумя каналами), или можно найти другие пути, обязательно наличие следящей переменной adc_chan?

Сообщение отредактировал Дмитрий_Мигачев - May 29 2006, 04:22