ATtiny84A ADC funktioniert nicht

[Hans F. (dani1632)]

Hallo!
Ich verwenden den ATtiny84A und will den ADC benutzen:
http://www.atmel.com/Images/doc8183.pdf

Mein CODE:

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

void init();
volatile uint16_t t1var = 0;


int main(void)
{
    init();
    while(1)
    {

    }
  return 0;
}
void init()
{
  PRR &= ~(1 << PRADC);  //disable Power Reduction for ADC
  ADMUX = (1 << REFS0) | (1 << MUX1);  //externe Referenz und ADC Channel 2 (PA2) aktivieren
  _delay_ms(1);
  ADCSRA = (1 << ADEN) | (1 << ADIE) | (1 << ADPS1) | (1 << ADPS0); // ADC aktivieren, AD INterrupt aktivieren und prescaler 8 -> 125 kHz
  _delay_ms(1);
  ADCSRA = (1 << ADSC);          

  sei();
}

ISR(ADC_vect)
{
         DDRA |= (1 << PA7);
  PORTA |= (1 << PA7);
}

Ich will nach fertiger Wandlung in der ISR ein Port auf High setzen, um 
zu sehen, dass der ADC mit interrupt funktioniert. An dem port hängt 
eine LED zur Anzeige, wenn ich im main die beiden zeilen der ISR 
aufrufe, leuchtet sie. Anscheinend wird jedoch die ISR nie erreicht, 
WARUM???

Vielen Dank!

[Felix P. (fixxl)]

Der Grund ist ganz simpel:

ADCSRA = (1 << ADEN) | (1 << ADIE) | (1 << ADPS1) | (1 << ADPS0);
ADCSRA = (1 << ADSC);

Was steht nach der zweiten Anweisung in ADCSRA? Setz doch vor das zweite 
= mal einen | und probiere es erneut.

[Stefan E. (sternst)]

Daniel F. schrieb:
> WARUM???

Weil du hier

  ADCSRA = (1 << ADSC);

den ADC-Interrupt deaktivierst (sogar den ganzen ADC deaktivierst).

[amateur (Gast)]

Die Atmels sind nicht sonderlich vergesslich. Aus diesem Grunde ist es 
auch unnötig bei jeder Unterbrechung den Befehl:
DDRA |= (1 << PA7);
auszuführen. Einmal während der Initialisierung reicht wohl aus.

Der Befehl:
PORTA |= (1 << PA7);
setzt Dir jedes Mal den Pin A7, wobei es keinen Unterschied gibt, ob 
dies zum hundertsten oder hundertersten Mal geschieht. Auch beim 
hundertzweiten Mal tut sich von außen gesehen nicht neues ;-)

[Hans F. (dani1632)]

:) vielen vielen Dank, was für ein blöder fehler...

@amateur: ist mir klar, sind wirklich nur testzwecke...

DANKE