ADC Interrupt mit Freerunning mode

[Dominik V. (nasryn)]

Hallo zusammen ich habe ein kleines Problem und zwar habe egal wie ich 
es drehe und wende, mein Programm funktioniert nicht.
Es ist für das Studium und wir arbeiten mit einem Hallsensor, der den 
Strom als Spannung zurück gibt(0A = 0,5V). 185mV pro A steigt die 
Spannung am Ausgang des Hallsensors.

Mein Problem dabei ist, dass das Programm niemals in die obere If 
Anweisung gelangt.

main
{
   ADMUX = (0 << REFS1)|(0 << REFS0)|(0 << MUX5)|(0 << MUX4)|(0 << MUX3)|(0 << MUX2)|(1 << MUX1)|(0 << MUX0);
   //Free_running Mode
   ADCSRB = (0 << BIN)|(0 << ACME)|(0 << ADLAR)|(0 << ADTS2)|(0 << ADTS1)|(0 << ADTS0);
   //ADC Interrupt aktivieren
   ADCSRA = (1 << ADEN)|(0 << ADSC)|(1 << ADATE)|(0 << ADIF)|(1 << ADIE)|(1 << ADPS2)|(0 << ADPS1)|(1 << ADPS0);

   ADCSRA |= (1 << ADSC);
   sei();
   while(1) ;
}

ISR(ADC-vect)
{
   cli();
   uint16_t current = ADC;
   current = (current - 102) / 37;//Strom = Spannung vom Hallsensor - 0,5V / 185mV
   if(current >= 55)
   {
      //tu was
   }
   else if(current <= 19)
   {
      //lass das
   }
   else
   {
      //Mache Weiter mit der letzten Sache
   }
   sei();
}

[Ralf G. (ralg)]

Dieter V. schrieb:
> ISR(ADC-vect)

Das wird nicht mal compilert!

[Karl M. (Gast)]

Hallo,

richtig ist sicherlich nicht: main { }

Wir kennen nicht deinen µC, deinen µC Takt, deine ADC-Takt, dein Ziel 
der festgelegten ADC Konfiguration.

Prosa: Es soll sein..

So etwas cli(); in einer pseudeo ISR sei(); ist unsinnig wenn es ein 
Atmel AVR µC ust.
Da fehlt Dir, evtl. noch die grundlegenden Infos, wie so eine µC 
arbeitet.

[Karl M. (Gast)]

Ach und noch etwas, wie willst Du deine Aussage herleiten:
//Mein Problem dabei ist, dass das Programm niemals in die obere If
Anweisung gelangt.
Dein Programm hat ja nirgends Datenausgaben, wenn man es überhaupt 
übersetzen könnte!

Ein Debugger hilft Dir hier nicht weiter.

[Dergute W. (derguteweka)]

Moin,

Also erstmal ein Lob an die vorbildliche Initialisierung ;-)
> ADMUX = (0 << REFS1)|(0 << REFS0)|(0 << MUX5)|(0 << MUX4)|(0 << MUX3)|(0 << 
MUX2)|(1 << MUX1)|(0 << MUX0);
usw.

Ansonsten vermute ich mal, dass du deine Integerarithmetik nicht im 
Griff hast: Aus dem ADC vermute ich mal, dass nicht mehr als 10bit 
kommen; also maximal der Wert 1023.

(1023-102)/37 ist aber nur 24; kann also niemals >55 werden.

Gruss
WK

[Draco (Gast)]

Ich tu mich mal:

int main(void)
{
   ADMUX  = (1 << MUX1);
   //ADC Interrupt aktivieren
   ADCSRA = (1 << ADEN) | (1 << ADSC);
   sei();
   while(1) ;
}

ISR(ADC_vect)
{
   uint16_t current = ADC;
   current = (current - 102) / 37;//Strom = Spannung vom Hallsensor - 0,5V / 185mV
   if(current >= 55)
   {
      //tu was
   }
   else if(current <= 19)
   {
      //lass das
   }
   else
   {
      //Mache Weiter mit der letzten Sache
   }
}

Dergute W. schrieb:
> (1023-102)/37 ist aber nur 24; kann also niemals >55 werden.

Rüchtüg!