PIC16F873A ADC Setup

[Peter A. (Gast)]

Moin Moin,

kann mir eventuell jemand sagen wo mein Fehler liegt ich bekomme keinen 
Wert in den ADC (AN1) eingelesen. Verwendet wird ein "PIC16F873A" mit 
externem 4MHz Quarz und als Vref. soll VDD/VSS dienen.

void SetupPORT_PINS(void){
   /*Definitionen, PORT Register (0 = AUSGANG / 1 = EINGANG) */  
    //PIN =   76543210      PORT Einstellungen
    TRISA = 0b11111111;     //PORT A
    TRISB = 0b00000001;     //PORT B
    TRISC = 0b00000000;     //PORT C

    /*PIN Definitionen*/
    #define InterruptPin RB0
    #define LED1 RB5
    // und so weiter
}

void Setup_ADC(void){


    ADCON0bits.ADCS0 = 1; 
    ADCON0bits.ADCS1 = 0;

    ADCON0bits.CHS0 = 1;
    ADCON0bits.CHS1 = 0;
    ADCON0bits.CHS2 = 0;

    ADCON1bits.PCFG = 0b00000000;

    ADCON0bits.ADON = 1;
}

int ADC_Read(){

  int l_Ergebniss;

  while (ADCON0bits.GO_nDONE);
  
  l_Ergebniss = (ADRESH<<8)+ADRESL;

  //l_Ergebniss = 500;

  return l_Ergebniss;
}

Die Funktion "SetupPORT_PINS" und "Setup_ADC" werden einmal beim start 
des Controllers aufgerufen. Die Funktion "ADC_Read" Wir in der main 
"While(1)" aufgerufen. Werte Werden auch richtig zurückgegeben aus der 
ADC_read aber im Register "ADRESH" und "ADRESL" ist nichts drin!

Danke für Eure Hilfe =)

[Max H. (hartl192)]

Das ADCON0bits.GO_nDONE musst du auch mal auf '1' setzen um die Wandlung 
zu starten. Ein guter Zeitpunkt wäre am Anfang der ADC_Read().

[Peter A. (Gast)]

int ADC_Read(){

  int l_Ergebniss;

  ADCON0bits.GO_nDONE = 1;

  while (ADCON0bits.GO_nDONE);
  
  l_Ergebniss = (ADRESH<<8)+ADRESL;

  //l_Ergebniss = 500;

  return l_Ergebniss;
}

hat leider nichts bewirkt! :(

Noch irgendwas falsch eingestellt?

Gruß

[Toxic (Gast)]

Peter A. schrieb:
> l_Ergebniss = (ADRESH<<8)+ADRESL;

Schau Dir deine Gleichung nochmal an.

[picalic (Gast)]

Toxic schrieb:
>> l_Ergebniss = (ADRESH<<8)+ADRESL;
>
> Schau Dir deine Gleichung nochmal an.

Und? Welche Erkenntnisse sollte ihm das bringen?

[Toxic (Gast)]

picalic schrieb:
> Und? Welche Erkenntnisse sollte ihm das bringen?

Also "+" in C ist gleich "|"  ?
Wusste ich nicht.

[Max H. (hartl192)]

Toxic schrieb:
> Also "+" in C ist gleich "|"  ?
In diesem Fall ja, a << 8 füllt mit Nullen auf und 0 + b und 0 | b führt 
zum gleichen Ergebnis.

[Little B. (lil-b)]

erster tipp: caste ADRESH auf ein uint16_t (oder ähnlich)

shiftest du einen uint8_t um 8 nach links, bleibt 0x00 übrig!

[Volker S. (vloki)]

Peter A. schrieb:
> kann mir eventuell jemand sagen wo mein Fehler liegt ich bekomme keinen
> Wert in den ADC (AN1) eingelesen.

Heißt das jetzt es kommt immer Null oder der Wert stimmt nicht (ist zu 
klein) ?

Wegen dem "<<" Problem:
- Entweder vor dem shiften auf 16bit casten
- Ergebnis = ADRES; (könnte funktionieren wenn im Header angelegt)
- Ergebnis = ADRESH*256 + ADRESL; (sollte funktionieren)
- Ergebnis = *(short*(&ADRESL)); (???)

[Little B. (lil-b)]

zweiter tipp: du taktest den ADC mit Fosc/8, Fosc darf dann nicht höher 
als 5MHz sein!

[Little B. (lil-b)]

dritter tipp: warte die "Aquisition Time" ab! diese beträgt in der 
Beispielrechnung des Datenblattes knapp 20µs, die du vor dem 
"GO_nDONE=1" warten musst!

Passe diesen Zeitwert für deine Schaltung an und implementiere eine 
wait-funktion!

(datenblatt pic16f873a kapitel 11.1)