ADC für PIC16F1823 funktioniert nicht

[Oliver K. (ollik)]

Hallo,

ich will für den PIC16F1823 den ADC ausporgrammieren, doch er 
funkioniert einfach nicht! :(

Versuche schon verzweifelt den Fehler zu finden, doch ich weis nicht was 
ich falsch gemacht habe.

Die Read-Funktion gibt mir immer den Wert 0 zurück... egal was ich für 
eine Spannung an dem AD-Pin anlege..

Bitte kann mir jemand helfen. Danke schon mal im Voraus.

Hier ist mein Quellcode:

 
void init_ADC();
unsigned int read_ADC(unsigned char channel);


void init_ADC()
{
    //Select ADC conversion clock, Frc (from a intern Oscillator) 1us - 6us
    ADCON1.ADCS0 = 1;
    ADCON1.ADCS1 = 1;
    ADCON1.ADCS2 = 1;

    //select interne Vref
    ADCON1.ADPREF0 = 1;
    ADCON1.ADPREF1 = 1;

    //select Vref = 2.048V
    FVRCON.ADFVR0 = 0;
    FVRCON.ADFVR1 = 1;

    //adc result format selection --> right justified
    ADCON1.ADFM = 1;
}

unsigned int read_ADC(unsigned char channel)
{
    unsigned int adc_result; //variable for adc-result
    unsigned int temp;

    ADCON0=0x00;

    /*channel selection*/
    switch(channel)
    {
    case 0:
      ADCON0.CHS0 = 0;
      ADCON0.CHS1 = 0;
      ADCON0.CHS2 = 0;
      ADCON0.CHS3 = 0;
      break;
    case 1:
      ADCON0.CHS0 = 1;
      ADCON0.CHS1 = 0;
      ADCON0.CHS2 = 0;
      ADCON0.CHS3 = 0;
      break;
    }

    /*enable ADC-Module*/
    ADCON0.ADON = 1;

    /*start of ad-convertion*/
    ADCON0.GO = 1;

    /*wait until finish of ad-convertion*/
    while(ADCON0.GO);

    /*write adc restult in adc_result*/
    temp = ADRESH <<8;
    adc_result = temp || ADRESL;

    /*disable ADC-Module*/
    ADCON0.ADON = 0;

    return adc_result;
}

void main()
{
     unsigned int voltage = 0;

     TRISC = 0x00;
     PORTC = 0b000000;

     TRISA = 0xFF;
     ANSELA = 0xFF;

     init_ADC();

     while(1){
          voltage = read_ADC(0);

          if(voltage == 0)
               PORTC = 0b000101;
          else if(voltage > 349 && voltage < 700)
          {
               PORTC = 0b000001;
          }
          else if(voltage > 699 && voltage < 1000)
          {
               PORTC = 0b000011;
          }
          else if(voltage > 999)
          {
               PORTC = 0b000111;
          }
          else{
               PORTC = 0b000000;
          }
     }
}

[dsfhg (Gast)]

>adc_result = temp || ADRESL;
Falsch. Logisches statt binärem Oder.

[Oliver K. (ollik)]

Danke für die schnelle Rückmeldung!

Habe es ausgebessert, aber es gibt mir immer noch 0 zurück..

[Tom L. (xedux)]

musst du nicht das DONE-bit abfragen anstatt des GO-bits?

[Oliver K. (ollik)]

aja! mich hat es verwirrt da es kein ADCON0.GODONE bit gibt, sondern es 
heißt ADCON0.GO_NOT_DONE. jz habe ich immer nur GO geschrieben da es mir 
hier keinen Fehler gab.

Jz funktioniert er!! DANKE DANKE DANKE :-) ich war schon am verzweifeln

[Tom L. (xedux)]

:) sehr gut!