PIC18F66 ADC per Hand auslesen

[Stefan S. (humus)]

Hallo Zusammen,

kann mir jemand sagen was ich beim Auslesen des ADC von meinem 
PIC18F66K80  falsch mache? Ich verwende MicroC Pro als Compiler. Wenn 
ich die integrierte Lib einsetze, funktioniert es einwandfrei, ich 
möchte aber wissen was passiert und deswegen alles per Hand 
programmieren. Hier mein C-Code:

for (i=0; i<13; i++)
{
adc_rd = 0;
PORTB =pin_mux[i];

ADCON0.ADON = 1;        //Wake up ADC
    ADCON0.CHS0=0;
    ADCON0.CHS1=0;
    ADCON0.CHS2=0;
    ADCON0.CHS3=0;
    ADCON0.CHS4=0;

    ADCON0.GO = 1;          //start 12b ADC conversion
    while(ADCON0.NOT_DONE); //wait until ADC finished

    adc_rd=ADRESH;              //12b ADC result value
    adc_rd=adc_rd<<8;           //bitwweises verschieben nach links
    adc_rd=adc_rd+ADRESL;       //12b ADC result value
ADCON0.ADON = 0;        //set ADC conversion module off
                    
    sprintf(lcd_line[0], "AN0 = %u", adc_rd);                                                                                                                        //The pec_error variable is simply set negative if any PEC errors
    Lcd_Out(1,1, lcd_line[0]);
    
ADCON0.ADON = 1;        //Wake up ADC
adc_rd = 0;
    ADCON0.CHS0=1;
    ADCON0.CHS1=0;
    ADCON0.CHS2=0;
    ADCON0.CHS3=0;
    ADCON0.CHS4=0;

    ADCON0.GO = 1;          //start 12b ADC conversion
    while(ADCON0.NOT_DONE); //wait until ADC finished

    adc_rd=ADRESH;              //12b ADC result value
    adc_rd=adc_rd<<8;           //bitwweises verschieben nach links
    adc_rd=adc_rd+ADRESL;       //12b ADC result value
ADCON0.ADON = 0;        //set ADC conversion module off
 
    sprintf(lcd_line[1], "AN1 = %u", adc_rd);                                                                                                                        //The pec_error variable is simply set negative if any PEC errors
    Lcd_Out(2,1, lcd_line[1]);
    
    sprintf(lcd_line[2], "MUX = %X", pin_mux[i]);                                                                                                                        //The pec_error variable is simply set negative if any PEC errors
    Lcd_Out(3,1, lcd_line[2]);
    Delay_ms(1000);
}

Anstatt von Werten zwischen 0 und 40954 bekomme ich Werte um die 65000 
auf meine LCD angezeigt.

Vielen Dank

[adc_rd (Gast)]

adc_rd nicht in String umgewandelt.

[Volker S. (vloki)]

Stefan S. schrieb:
> Anstatt von Werten zwischen 0 und 40954 bekomme ich Werte um die 65000
> auf meine LCD angezeigt.

Wird irgendwo das Format (Right/Left justified) eingestellt?
Wenn nicht, dann ist die Voreinstellung Left justified!

Den ADC sollte man auch nicht ständig ein und ausschalten.
Den stellt man einmal in der Initialisierung ein und danach schaltet man 
meist nur noch die Kanäle um und startet neue Messungen...

Probier mal ob der Compiler nicht auch das kann:
    ADCON0.CHS = 0; // 1, 2, 3 oder ...
    adc_rd = ADRES; // ohne H/L

[Stefan S. (humus)]

adc_rd schrieb:
> adc_rd nicht in String umgewandelt.

adc_rd ist als unsigned int definiert. Das Display kann doch nur String, 
oder irre ich mich?

Volker S. schrieb:
> Wird irgendwo das Format (Right/Left justified) eingestellt?
> Wenn nicht, dann ist die Voreinstellung Left justified!
Verstehe nicht was du meinst?

> Den ADC sollte man auch nicht ständig ein und ausschalten.
> Den stellt man einmal in der Initialisierung ein und danach schaltet man
> meist nur noch die Kanäle um und startet neue Messungen...
Bringt keine Veränderung.


> Probier mal ob der Compiler nicht auch das kann:
>     ADCON0.CHS = 0; // 1, 2, 3 oder ...
>     adc_rd = ADRES; // ohne H/L
ADRESH/ADRESL ist im Datenblatt mit jeweils 8 bit angegeben.

[Witkatz :. (wit)]

Stefan S. schrieb:
>> Wenn nicht, dann ist die Voreinstellung Left justified!
> Verstehe nicht was du meinst?

s. Datenblatt Kap. "23.2.2 A/D RESULT REGISTERS" erklärt die 
Wertedarstellung sehr anschaulich, sogar mit Bild
http://ww1.microchip.com/downloads/en/DeviceDoc/39977f.pdf#G20.1060972

[Torben K. (tokuhila)]

Um meinen Vorredner zu ergänzen:

a) ist ADRES ein 16bit signed
b) ADCON2.ADFM ist default 0, d.h. left justified
c) Hat der A/D-Wandler im K80 einen Schuss, siehe Errata Item 1

[Volker S. (vloki)]

Stefan S. schrieb:
>> Probier mal ob der Compiler nicht auch das kann:
>>     ADCON0.CHS = 0; // 1, 2, 3 oder ...
>>     adc_rd = ADRES; // ohne H/L
> ADRESH/ADRESL ist im Datenblatt mit jeweils 8 bit angegeben.

Wenn für den Compiler irgendwo definiert ist, dass ADRES ein 16 unsigned 
ist und die gleiche Adresse hat wie ADRESL, dann passt das automatisch. 
(beim xc8 und C18 Compiler ist das so)