ADC 0 und 1 springen

Sorry habe den Code vergessen

MIST

Hier ist der Ausschnitt der für denn ADC und Die Ausgabe zuständig ist.

#include <avr/io.h>
#include <util/delay.h>
#include <stlib.h>
#include <lcd-routines.h>

unsigned short wertadc0;
unsigned short wertadc1;

...

// Hauptprogramm
void main()
{
...
Start:

    ADC0();
    mV();
    ADC1();
    mA();
    _delay_ms(300);
    goto Start;

while(1)
{
}
return 0;
}

void ADC0()
{
    ADMUX = 0b01000000;
    ADCSRA = 0b10000110;
    ADCSRA |=(1<<ADSC);
    while (!(ADCSRA & ( 1<<ADSC)));
    wertadc0 = ADCW;
}
void ADC1()
{
    ADMUX = 0b01000001;
    ADCSRA = 0b10000110;
    ADCSRA |=(1<<ADSC);
    while (!(ADCSRA & ( 1<<ADSC)));
    wertadc1 = ADCW;
}

void mV()
{
    unsigned char lcdnummer1 = 0x00;
    unsigned char lcdnummer2 = 0x00;
    unsigned char lcdnummer3 = 0x00;
    unsigned char lcdnummer4 = 0x00;

NO1:
    if ( wertadc0 >=1000)
    {
         wertadc0-=1000;
         lcdnummer1 ++;
         goto NO1;
    }
    lcdnummer1 |= 0x30;
NO2:
    if ( wertadc0 >=100)
    {
         wertadc0-=100;
         lcdnummer2 ++;
         goto NO1;
    }
    lcdnummer2 |= 0x30;
NO3:
    if ( wertadc0 >=10)
    {
         wertadc0-=10;
         lcdnummer3 ++;
         goto NO1;
    }
    lcdnummer3 |= 0x30;
NO4:
    if ( wertadc0 >=1)
    {
         wertadc0-=1;
         lcdnummer4 ++;
         goto NO1;
    }
    lcdnummer4 |= 0x30;

    set_cursor(0,2);
    lcd_data(lcdnummer1);
    set_cursor(1,2);
    lcd_data(lcdnummer2);
    set_cursor(2,2);
    lcd_data(lcdnummer3);
    set_cursor(3,2);
    lcd_data(lcdnummer4);
    set_cursor(4,2);
    lcd_string("mV");
}
void mA()
{
    unsigned char lcdnummer1 = 0x00;
    unsigned char lcdnummer2 = 0x00;
    unsigned char lcdnummer3 = 0x00;
    unsigned char lcdnummer4 = 0x00;

NO1:
    if ( wertadc1 >=1000)
    {
         wertadc1-=1000;
         lcdnummer1 ++;
         goto NO1;
    }
    lcdnummer1 |= 0x30;
NO2:
    if ( wertadc1 >=100)
    {
         wertadc1-=100;
         lcdnummer2 ++;
         goto NO1;
    }
    lcdnummer2 |= 0x30;
NO3:
    if ( wertadc1 >=10)
    {
         wertadc1-=10;
         lcdnummer3 ++;
         goto NO1;
    }
    lcdnummer3 |= 0x30;
NO4:
    if ( wertadc1 >=1)
    {
         wertadc1-=1;
         lcdnummer4 ++;
         goto NO1;
    }
    lcdnummer4 |= 0x30;

    set_cursor(8,2);
    lcd_data(lcdnummer1);
    set_cursor(9,2);
    lcd_data(lcdnummer2);
    set_cursor(10,2);
    lcd_data(lcdnummer3);
    set_cursor(11,2);
    lcd_data(lcdnummer4);
    set_cursor(12,2);
    lcd_string("mA");

}

Komisch jetzt springt er nicht mehr sondern vertauscht permanent denn 
Wert.

>Nachdem du den ADC auf einen neuen Kanal stellst, solltest du die erste
>Messung verwerfen. Die ist normalerweise Müll.

Nachdem daß ja immer wieder postuliert wird, und auch im Beispiel-Code 
des Tutorials so steht, kann mir mal jemand sagen, wo genau das im 
Datenblatt steht? Ich finde das einfach nicht.

Oliver

@ Karl Heijnz

dein Code:

void Output( unsigned char Column, unsigned char Row,
             unsigned short Value, const char* Unit )
{
  char Text[20];

  sprintf( "%04u%s", Value, Unit );
  set_cursor( Column, Row );
  lcd_string( Text );
}

void mV()
{
  Output( 0, 2, wertadc0, "mV" );
}

void mA()
{
  Output( 8, 2, wertadc1, "mA" );
}

funktioniert bei mir nicht es wirt auf dem LCD zweimal(
I'schwartzes kästchien schräges X ) ausgegeben.

@ Ralf Schwarz

ich setze doch erst den ADMUX und dann den ADCSRA und starte dann erst 
den ADC. Oder verstehe ich da was falsch?

@Ralf Schwarz
>Klar soweit?

Nö. Ein " ham' wer schon immer so gemacht" ist genauso hilfreich wie 
"viel hilft viel", das einmauern toter Katzen in Brückenfundamente, oder 
"schad' ja nix".

Etwas genauer: Die Anwendung hier sind einzelne Wandlungen, der ADC wird 
jedes mal neu gestartet, VREF wird nicht umgeschaltet. Ein 
Standard-Szenario für ADC-Betrieb ohne free-run.

Wo steht im Danteblatt, daß ein Umschalten des Kanals vor dem Start des 
ADC's zu einem ungültigem Wandlungsergebnius führt?

Oliver

So
jetzt funktioniert alles habe :

while (!(ADCSRA & (1<<ADSC))); gegen while (ADCSRA & (1<<ADSC));
und die Ausgabe von Karl Heinz

Vielen Vielen Vielen Dank

zu dem GOTO Befehl

Ich habe hier im Forum schon oft gelesen das der goto oldschool ist und 
nicht mehr genutzt werden soll. Andere sagen benutzt ihn ( ist einfach 
und funktioniert). Meine Frage ist nun was für Altanatieven gibt es 
denn?

elomt wrote:

> Ich habe hier im Forum schon oft gelesen das der goto oldschool ist und
> nicht mehr genutzt werden soll. Andere sagen benutzt ihn ( ist einfach
> und funktioniert). Meine Frage ist nun was für Altanatieven gibt es
> denn?

Nichts dagegen goto zu benutzen, wo es einen Sinn ergibt. Aber wenn man 
eine Schleife implementiert, duerfte while() fast immer erste Wahl sein. 
Dein Code sieht eher aus wie Markoassembler mit den ganzen "von Hand" 
implementierten Schleifen. Im Uebrigen sind sind deine Spruenge in den 
mA und mV Funktionen hoechst zweifelhaft...

Sorry wenn ich nochmal was schreibe aber ich habe gerade gesehen das 
mein adc wert nicht unter 14 geht wenn ich gnd auf den adc port gebe. 
Weis einer woran das liegen könnte?

die GND's und der ADC liegen auf dem selben GND in der Schaltung

mit einem Multimeter messe ich aber 0,000V am Port dann sollte der ADC 
doch auch 0 anzeigen.

auch 14

Hallo

Ich schreibe erstmal ein neues Programm da mein jetziges mit der ganzen 
LCD Menüführung zu umfangreich ist.

Vielen Dank noch mal für die guten tips.