ADC für ATXmega128A1

[Fogtech]

Hallo,

ich ein Fehler in dem Code. Wenn Spannung am ADC fehlt runter, LEDs zeigen das. Aber wenn es steigt, LEDs sind immer aus. Ich nutze CodeVisionAVR und arbeite mit dem Xmega128-A1 Xplained board.

#include <xmega128a1.h> //Xmega-A1-Xplained development board

#define LED PORTE// LED as PORT E.

volatile int Result;

void main()
{
LED.DIR = 0b00001111;  //E0 - E3 - outputs.

PORTA.DIR = 0;                                  // configure PORTA as input
ADCA.CTRLA |= 0x1;                              // enable adc
ADCA.CTRLB = ADC_RESOLUTION_12BIT_gc;           // 12 bit conversion
ADCA.REFCTRL = ADC_REFSEL_VCC_gc | 0x02;        // internal 1V bandgap reference
ADCA.PRESCALER = ADC_PRESCALER_DIV8_gc;         // peripheral clk/8 (2MHz/16=250kHz)
ADCA.CH0.CTRL = ADC_CH_INPUTMODE_SINGLEENDED_gc;// single ended
ADCA.CH0.MUXCTRL = ADC_CH_MUXPOS_PIN0_gc; 	    // PORTA:0 

    while(1)  // cycle
    { 
        ADCA.CH0.CTRL |= ADC_CH_START_bm; // start conversion on channel 0
        while(!ADCA.CH0.INTFLAGS);
        Result = ADCA.CH0RES;
        
        if (Result > 1000) //
        {
            LED.OUTSET = (1 << 0); // output E0 High.
        } //if
        if (Result > 2000) //
        {
            LED.OUTSET = (1 << 1); // output E1 High.
        } //if        
        if (Result > 3000) //
        {
            LED.OUTSET = (1 << 2); // output E2 High.
        } //if        
        if (Result > 4000) //
        {
            LED.OUTSET = (1 << 3); // output E3 High.
        } //if
       }//while(1)
}//main

[Che Guevara]

Hi,

ich programmiere zwar nicht mit C, hätte aber einen Vorschlag.
Als erstes würde ich zu debug-Zwecken ein Delay in die Mainloop einfügen, um zu sehen, was passiert. Außerdem würde ich die If-Abfragen besser eingrenzen, den angenommen dein Wert ist 3100, dann trifft das auf 3 Ifs zu:
if (Result > 1000)
if (Result > 2000)
if (Result > 3000)
Stattdessen könntest du schreiben:
if (Result >= 1000 && Result < 2000)
if (Result >= 2000 && Result < 3000)
usw...

Ein >= ist übrigens besser als ein >, da der µC nur das >= kennt, nicht jedoch das >. Somit sparst du ein paar Takte (auch wenns bei dieser Anwendung egal sein sollte).

Gruß
Chris

[robin]

liegt daran, dass du die LEDs nicht wieder ausschaltest.

Entweder bei jeder if-schleife ein:
else
LED.OUTCLR = ...

oder alles in eine Variable und am ende erst die leds an bzw. aus machen.

[Che Guevara]

Hi,

durch die Zuweisung sollten doch die nicht gebrauchten LEDs wieder ausgeschaltet werden, oder irre ich mich da?
LED.OUTSET = (1 << 1);
bedeutet doch:
PORTE = &B00000010

Gruß
Chris

[Fogtech]

Hi,
LED.OUTSET = (1 << 1); - das ist richtig
Die Sache ist die, das die LEDs lassen sich ausschalten, aber nicht wieder leuchten, wenn aber Result wieder steigt.


Gruß
Max

[Che Guevara]

Hi,

hast du den die Möglichkeit, die Daten irgendwie zu visualisieren, also z.b. über UART an den PC senden? Dann könntest du dir Result ausgeben lassen.

Gruß
Chris