Problem mit dem ADC

[Eric W. (ceikolon)]

Hallo,
ich habe mich heute das erste mal an den ADC gewagt und versuche über 
ein Potentiometer die Geschwindigkeit des Fadings meiner RGB LEDs 
einzustellen.

Bisher leuchten einfach immer nur alle LEDs unabhängig vom verstellen 
des Potis. Vlt weis jmd von euch Rat.
Atmega8 @8Mhz

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

int main()
{
  // OC1A auf Ausgang
  DDRB = (1 << PB1 )|(1 << PB2)|(1 << PB3);
  DDRC=0x00 ; //ADC Eingang
  //
  // Timer 1 einstellen
  //

  TCCR1A = (1<<COM1A1) | (1 << COM1B1)| (1<<WGM10);
  TCCR1B = (1 << CS10);


  //
  // Timer 2 einstellen


  TCCR2 |=  (1 << WGM20) | (1 << CS20) | (1 << COM21);
  TCCR2 |= (1 << CS20);
  
  //Analog Digitalwandler konfigurieren, Refpot=5V
  
  ADMUX |=  (1 << REFS0) | (1 << ADLAR) ;
  ADCSRA|=  (1 << ADSC)| (1 << ADPS1)| (1 << ADPS2)  ; //ADPS1 und ADPS2 so legen, dass ADC Frequenz zwischen 50kHz und 200kHz liegt

  unsigned char counter1 = 255;
  unsigned char counter2 = 0;
  unsigned char counter3 = 0;
  OCR1A = counter1;
  OCR1B = counter2;
  OCR2 = counter3;
  unsigned char a = 1;  //1-->Auschalten
  unsigned char b = 0;  //0-->Anschalten
  unsigned char c = 0;
  
  
  while( 1 )
  {
  unsigned char timer=ADCH; //Lese ADCH aus =timer
  
    if(a==1 & b==0 & c==0)
    {   counter1--;
      counter2++;
    }
    if(a==0 & b==1 & c==0)
    {   counter2--;
      counter3++;
    }
    if(a==0 & b==0 & c==1)
    {   counter1++;
      counter3--;
    }
    
    
    if (counter1==255 )
    {a=1;
    }
    if (counter2==255 )
    {b=1;
    }
    if (counter3==255 )
    {c=1;
    }
    if (counter1==0 )
    {a=0;
    }
    if (counter2==0 )
    {b=0;
    }
    if (counter3==0 )
    {c=0;
    }
    OCR1A = counter1;
    OCR1B = counter2;
    OCR2 = counter3;
    
    for (int i = 0; i < timer; i++) // warte solange wie Timer
    {_delay_ms(1);
    }      
    
  }
}

Bin für jede Hilfe dankbar ;-)

[Marcus P. (Gast)]

Hi,
ich verstehe zwar nicht was du da mit deinen timern machst, aber
1. woher weißt du das die ADC-Wandlung fertig ist, wenn du den Wert 
ließt.
2. Du startest den ADC nur einmal, für genau 1x Messung (und die erste 
Messung sollte sowieso verworfen werden laut Datenblatt), ich denke mal 
der "Free Running Mode" ( ADCSRA|= (1<<ADFR) ) ist das was du suchst.

Gruß
marcus

[Eric W. (ceikolon)]

mit den Timern steuer ich eine Hardware PWM

Marcus P. schrieb:
> 1. woher weißt du das die ADC-Wandlung fertig ist, wenn du den Wert
> ließt.

Das verstehe ich nicht ganz. Kannst mir das genauer erklären?

ADFR habe ich nun gesetzt, muss weiterhin ADSC stehen bleiben?

[Marcus P. (marc2100)]

Also der ADC arbeitet folgender maßen:

0. ADC einstellen (8bit / Frequenz...)
1. ADC wird gestartet (ADCSRA|=(1<<ADSC) )
2. ADC Messung läuft  (ADSC=1 & ADIF=0)
3. ADC Messung fertig
Wenn die Wandlung fertig ist, setzt der ADC Bit4 (ADIF) in ADCSRA, egal 
ob du Interrupts aktiviert hast oder nicht.
Falls der ADC nicht im "Free Running Mode" läuft kann man auch das 
ADSC-Bit abfragen, das ist solange 1, bis die Wandlung beendet ist, da 
der ADC im "Free Running Mode" direkt danach automatisch eine Wandlung 
startet ist das Bit in dem Fall nutzlos, da es immer 1 wäre.

Da du aber eine kontinuierlich Messung benötigst, bleibt dir nur das 
ADIF-Bit, das ist immer 1, wenn ein neues Ergebnis vorhanden ist.
Du könntest also eine Abfrage einbauen die so aussieht

unsigned char adc_wert (void)
{
  if ( ADCSRA & (1<<ADIF) ) //Wenn Bit=1 Wandlung fertig
  {
    ADCSRA|= (1<<ADIF);     //Bit löschen, mit 1 überschreiben
    return ADCH;            //ADC_wert ausgeben
  }
   return 0;                //Musst du dir selbst überlegen, was du 
zurückgeben willst, falls noch kein ergebniss vorliegt
}

Alternativ kannst du natürlich auch solange warten bis es fertig ist, 
aber dann hängt deine Timer-Geschichte auch, wäre also nicht so gut.

while ( !(ADCSRA & (1<<ADIF)) ) //vor dem auslesen einfügen

Aber achtung, auch hier musst du ADIF selbst wieder löschen (mit 1 
überschreiben! )


Gruß
Marcus

[Stefan E. (sternst)]

Wenn du den ADC garnicht einschaltest, kannst du damit natürlich auch 
nichts messen (egal ob Single-Shot oder Free-Running).

[Eric W. (ceikolon)]

Erstmal vielen Dank für die beiden einleuchtenden Beiträge. Jetzt habe 
ich glaub den ADC besser verstanden.
Letzendlich habe ich es so gemacht:

 if ( ADCSRA & (1<<ADIF) ) //Wenn Bit=1 Wandlung fertig
    {
      timer=ADCH;        //Setzer timer
      ADCSRA|=  (1<<ADSC);  //Starte neue Messung
    }

Also kein Free Running, funktioniert auch soweit ganz gut.
Aber ich habe doch noch die ein oder andre Frage offen:
Der ADC wirft nen 10 Bit Wert aus aber bisher lese ich nur einen 8 Bit 
Wert ein (ADCH). Was muss ich tun damit mein Ergebniss feiner wird?

Zweite Frage wäre der ADC seine arbeit neben dem durchgehen des 
Quelltextes tut und somit vlt ein free running besser angebracht wäre, 
indem immer nur der letzte Wert (aktuellste des Potis) in das delay 
eingetragen wird.

Dritte Frage: der ATmega8 besitzt ja 6 ADCs. Kann ich alle gleichzeitig 
laufen lassen? Weil das Problem ist das alle das gleiche Register 
haben...

Sry wenn ich noch etwas unbeholfen bin, aber ich bin nochn Anfänger ( 
wie ihr schon bemerkt habt ;-))

[Karl H. (kbuchegg)]

http://www.mikrocontroller.net/articles/AVR-GCC-Tutorial/Analoge_Ein-_und_Ausgabe#Der_interne_ADC_im_AVR

[Stefan E. (sternst)]

Eric Weiß schrieb:
> funktioniert auch soweit ganz gut.

Na ja, eher zufällig. Du musst das Interrupt-Flag auch wieder löschen.

Eric Weiß schrieb:
> Dritte Frage: der ATmega8 besitzt ja 6 ADCs.

Nein, nur einen. Dieser eine ADC kann mittels eines Multiplexers mit 
einem von 6 Eingängen verbunden werden.

[Spess53 (Gast)]

Hi

>Dritte Frage: der ATmega8 besitzt ja 6 ADCs. Kann ich alle gleichzeitig
>laufen lassen? Weil das Problem ist das alle das gleiche Register
>haben...

Es gibt nur einen ADC mit gemultiplexten Eingängen.

MfG Spess