ADC Schwellwert

[Heinz peter P. (poschi8)]

Hallo, ich hab hier mal ein kleines Programm zur Schwellwertüberwachung:

#include <avr/io.h>

void main(void)
{
  DDRD=0xFF;            //PORTD als Ausgang definieren
  unsigned int wert1;



  while(1)
  {
    ADCSRA = 0b10000100;    //ADC Aktivieren und Teilungsfaktor 16
    
    ADMUX = 0b11000001;      
    ADCSRA |= (1<<ADSC);       //interne referenz und ADC1
  while(ADCSRA & (1<<ADSC));     //Warten auf das Ende der Wandlung
    {
      wert1 = ADCW;      //Ergebnis in die Variable Wert schreiben
    }

      if (wert1 == 0x00)    //wenn Wert gleich 0 ist dann ist PD5 aktiv
      {            //sonst ist PD7 aktiv  
        PORTD |= (1<<PD5);
      }
      else
      {
        PORTD |= (1<<PD7);
      }


  }

}

Das Problem ist das wenn sich die Spannung verändert, dann tut sich an 
der LED gar nichts.
Wenn ich zum Beispiel am Anfang ADC1 auf GND habe ist PD5 aktiv, wenn 
ich dann aber auf z.B. 5V hänge passiert gar nichts, bzw es sind beide 
aktiv.
Was habe ich falsch gemacht?!

Mit freundlichen Grüßen,
Poschi

[Heinz peter P. (poschi8)]

Hab das Problem schon gelöst.
Ich musste nur den PORTD wieder zurücksetzten, also nach " ADCSRA |= 
(1<<ADSC); "
gehört noch ein " PORTD = 0x00 "

Mit freundlichen Grüßen,
Poschi

[Helfer (Gast)]

1) Warum benutzt du für erste Experimente nicht den bewährten Code aus 
dem Tutorial

2) Trenne Initialisierung des ADC von der Messroutine

3) "//wenn Wert gleich 0 ist dann ist PD5 aktiv"

Das wurde bereits in einem Thread gestern als schlechte Idee diskutiert.

0 hat dein ADC Wandler in der Praxis u.U. sehr selten. I.d.R. macht es 
mehr Sinn mit Schwellen und Intervalleb zu arbeiten, so wie in dem 
Thread gestern auch beschrieben wurde.

#include <avr/io.h>

void main(void)
{
  unsigned int wert1;

  DDRD = 0xFF; //PORTD als Ausgang definieren

  // ADC Initialisierung
  // (Kryptische Bitwerte aus dem Originalcode sind nicht kontrolliert!)
  ADCSRA = 0b10000100; // ADC enablen und Teilungsfaktor 16
  ADMUX = 0b11000001;  // Interne referenz und ADC1      
  ADCSRA |= (1<<ADSC); // 1. Wandlung starten
  while( (ADCSRA & (1<<ADSC)) ) {} //Warten auf das Ende der Wandlung
  wert1 = ADCW; // 1. Wandlung verwerfen

  while(1) {
    ADCSRA |= (1<<ADSC); // Wandlung starten
    while( (ADCSRA & (1<<ADSC)) ) {} // Warten auf das Ende der Wandlung
    wert1 = ADCW; // Ergebnis in die Variable Wert schreiben

    if( wert1 == 0 ) {            
      //wenn Wert gleich 0 ist dann ist PD5 aktiv
      PORTD |= (1<<PD5);  // PD5 an
      PORTD &= ~(1<<PD7); // PD7 aus
    } else {
      // sonst ist PD7 aktiv  
      PORTD &= ~(1<<PD5); // PD5 aus
      PORTD |= (1<<PD7);  // PD7 an
    }
  }
}

[Tip (Gast)]

Heinz peter Posch schrieb:
> Ich musste nur den PORTD wieder zurücksetzten, also nach " ADCSRA |=
> (1<<ADSC); "
> gehört noch ein " PORTD = 0x00 "

Das hilft zwar in deinem Fall, zerstört aber die Stellung sämtlicher 
anderer Portpins.

Besser wäre

      if (wert1 < SCHWELL)    // wenn Wert unterhalb SCHWELLE
      {
        PORTD &= ~(1<<PD7);   // LED am Pin 7 ausschalten
        PORTD |=  (1<<PD5);   // LED am Pin 5 einschalten
      }
      else
      {
        PORTD &= ~(1<<PD5);   // LED am Pin 5 ausschalten
        PORTD |=  (1<<PD7);   // LED am Pin 7 einschalten
      }

[Heinz peter P. (poschi8)]

Ok!!!
Danke für die Antworten, werd das gleich mal ausprobieren...

Mit freundlichen Grüßen,
Poschi