1. ADC Programm

[Thomas (Gast)]

Hallo zusammen,

ich habe in den letzten Tagen mein erstes C-Porgramm zur ADC-Auswertung 
geschrieben. Ich möchte über ein 10k Poti, Spannung am OC0 messen.

Ausgeben möchte ich den jeweiligen Wert über eine LED-Leiste über inkl. 
10 LED's.

Die Led's sind an PA0-PA7 und PA0-PA1.
Mein AtMega32 läuft über den internen Quarz mit 1MHz. Sonst habe ich nur 
den üblichen Grundaufbau am Mikrocontroller. Hardware sollte richtig 
verdrahten sein, da sich die LED's über ein anderes Programm ansprechen 
lassen. Weiter kann ich mit einem Multimeter einen veränderlichen 
Widerstand am Poti messen.
Es kann also nur noch an meinem Programm liegen, bzw. die Konfiguration 
dazu. Als Ref-Spannung nutze ich die 5V am AVCC.

Hier ist mein Programm:

#include <avr/io.h>
#include <inttypes.h>


volatile uint16_t voltage;      //Variable für Spannung an ADC 0-5V 5/1024 0,0048V pro Bit -> 4,8mV pro Bit, umgerechnet in Prozent

uint16_t ADC_Read_Avg( uint8_t channel, uint8_t nsamples );
uint16_t ADC_Read( uint8_t channel );

DDRA = 0xff;
DDRB = 0x03;

int main(void)
{
    while(1)
    {
    //Auslesen des ADC
  voltage = ADC_Read_Avg(0, 4);  // Kanal 0, Mittelwert aus 4 Messungen
  
  
  //Ansteuern der LED's nach Spannungshöhe
  display_voltage(PORTA,PA0,100);
  display_voltage(PORTA,PA1,200);
  display_voltage(PORTA,PA2,300);
  display_voltage(PORTA,PA3,400);
  display_voltage(PORTA,PA4,500);
  display_voltage(PORTA,PA5,600);
  display_voltage(PORTA,PA6,700);
  display_voltage(PORTA,PA7,800);
  display_voltage(PORTB,PB0,900);
  display_voltage(PORTB,PB1,980);
    
  }
}


void ADC_Init(void) {
 
  // die Versorgungsspannung AVcc als Refernz wählen:
  ADMUX = (1<<REFS0);    
 
  // Bit ADFR ("free running") in ADCSRA steht beim Einschalten
  // schon auf 0, also single conversion
  ADCSRA = (1<<ADPS1) | (1<<ADPS0);     // Frequenzvorteiler
  ADCSRA |= (1<<ADEN);                  // ADC aktivieren
 
  ADCSRA |= (1<<ADSC);                  // eine ADC-Wandlung 
  while (ADCSRA & (1<<ADSC) ) {         // auf Abschluss der Konvertierung warten
  }
  (void) ADCW;
}

uint16_t ADC_Read_Avg( uint8_t channel, uint8_t nsamples )
{
  uint32_t sum = 0;
  
  for (uint8_t i = 0; i < nsamples; ++i ) {
    sum += ADC_Read( channel );
  }
  
  return (uint16_t)( sum / nsamples );
}

uint16_t ADC_Read( uint8_t channel )
{
  // Kanal waehlen, ohne andere Bits zu beeinflußen
  ADMUX = (ADMUX & ~(0x1F)) | (channel & 0x1F);
  ADCSRA |= (1<<ADSC);            // eine Wandlung "single conversion"
  while (ADCSRA & (1<<ADSC) ) {   // auf Abschluss der Konvertierung warten
  }
  return ADCW;                    // ADC auslesen und zurückgeben
}


void display_voltage(volatile uint8_t *Port, uint8_t Pin, uint8_t wert)
{  
  if (voltage >= wert){
    *Port |= (1<<Pin);      //Ausgang einschalten
  }
  else{
    *Port &= ~(1<<Pin);      //Ausgang ausschalten
  }
}

Kann mir vielleicht jemand sagen, wo mein Problem liegen kann?
Ich finde es einfach nicht. Hab schon mehrmals drüber geschaut.

Danke.

Gruß Thomas

[rava (Gast)]

ich kenne ja deine Fehlermeldung nicht, aber schreibt doch mal die 
Funktionen in der Reihenfolge, in der sie benutzt werden:


ADC_Init und display_voltage sollten vor die main()-routine.
Außerdem rufst du ADC_Init gar nicht auf.

[Karl H. (kbuchegg)]

rava schrieb:

> Außerdem rufst du ADC_Init gar nicht auf.

Erstens das und zweitens wird es auch eine gute Idee sein, die 
Ausgangspins an denen die LED hängen auch auf Ausgang zu schalten.

[N2 (Gast)]

Hi,

ADC = Analog Digital Converter
Die Kanäle sind von ADC0 bis ADC7 (PA0 bis PA7).

Was Du mit dem Poti am OC0 (PB3) = Output Compare oder AIN0 (gehört zum 
AC = Analog COMPARATOR) machst, ist dem ADC ziemlich egal.

Du kannst den AC zwar für eine AD-Wandlung missbrauchen, aber dann wohl 
kaum mit Deiner ADC_Read...

Gruß N2

[Karl H. (kbuchegg)]

Karl Heinz schrieb:
> rava schrieb:
>
>> Außerdem rufst du ADC_Init gar nicht auf.
>
> Erstens das und zweitens wird es auch eine gute Idee sein, die
> Ausgangspins an denen die LED hängen auch auf Ausgang zu schalten.

Heck.

DDRA = 0xff;
DDRB = 0x03;

int main(void)
{
...

das compiliert doch gar nicht!

[Mirki (Gast)]

Lass Dir das hier mal nochmal durch den Kopf gehen:

    //Auslesen des ADC
  voltage = ADC_Read_Avg(0, 4);  // Kanal 0, Mittelwert aus 4 Messungen


  //Ansteuern der LED's nach Spannungshöhe
  display_voltage(PORTA,PA0,100);
  display_voltage(PORTA,PA1,200);
  display_voltage(PORTA,PA2,300);
  display_voltage(PORTA,PA3,400);
  display_voltage(PORTA,PA4,500);
  display_voltage(PORTA,PA5,600);
  display_voltage(PORTA,PA6,700);
  display_voltage(PORTA,PA7,800);
  display_voltage(PORTB,PB0,900);
  display_voltage(PORTB,PB1,980);


void display_voltage(volatile uint8_t *Port, uint8_t Pin, uint8_t wert)
{
  if (voltage >= wert){
    *Port |= (1<<Pin);      //Ausgang einschalten
  }
  else{
    *Port &= ~(1<<Pin);      //Ausgang ausschalten
  }
}

[rava (Gast)]

das finde ich, sieht noch fast okay aus.

uint8_t wert

 dürfte halt eigentlich nur zwischen 0 und 255 liegen.

[Thomas E. (thomase)]

rava schrieb:
> das finde ich, sieht noch fast okay aus.

Aber nur fast. Die Leds liegen an PortA und der Adc auch. Krasses 
Multiplexing, sozusagen.

mfg.

[Thomas (Gast)]

Hallo zusammen,

danke für all die Hinweise.
Es klappt nun. Hab die Hardware geändert, Programm neu strukturiert 
sowie ADC_init() rein gepackt.

Läuft bestens. :) Danke

Gruß Thomas