PIC16F886 resetet sich selbstständig beim ADC auslesen

[ThinkPic (Gast)]

Moin Leute,

leider habe ich ein Problem mit meinem PIC. Dieser resetet sich 
automatisch, sobald ich

return ((ADRESH<<8)+ADRESL);

 ausführe. Leider finde ich keine wirkliche Ursache dafür. Kann mir 
jemand helfen?

Vielen Dank.

#pragma config FOSC = XT        // Oscillator Selection bits (XT oscillator)
#pragma config WDTE = OFF       // Watchdog Timer Enable bit (WDT disabled)
#pragma config PWRTE = ON       // Power-up Timer Enable bit (PWRT enabled)
#pragma config BOREN = ON       // Brown-out Reset Enable bit (BOR enabled)
#pragma config LVP = OFF        // Low-Voltage (Single-Supply) 
#pragma config CPD = OFF        // Data EEPROM Memory Code Protection bit 
#pragma config WRT = OFF        // Flash Program Memory Write Enable bits 
#pragma config CP = OFF         // Flash Program Memory Code Protection bit 
 
#define _XTAL_FREQ 8000000  
#include "xc.h"

int result;

void InitPWM(){
    
    CCP1CON = 0b00001100;
    CCP2CON = 0b00111100;
    PSTRCON = 0b00000100;
    CCPR1L = 40;
    CCPR1H = 0x00;
}

void InitTMR2(){
    
    T2CON=0b01111100; 
    PR2=100;            //100 us
    TMR2=0;
    TMR2IF = 0;
    TMR2ON = 1;
}

void InitADC()
{
  ADCON0 = 0x11000000;
  ADCON1 = 0b10000000; 
  ADCON0bits.ADON = 1;
  __delay_ms(10);
}



int ADC_Read(unsigned char channel)
{
  __delay_ms(2); 
  GO_nDONE = 1; 
  while(GO_nDONE); 
  return ((ADRESH<<8)+ADRESL); 
}

int main(void) {
    
    OSCCON = 0x66; //Interner Oszi
    
    int dc = 0;
    int a = 0;
    
    TRISA = PORTA = TRISB = PORTB = TRISC = PORTB =0x00;
    ANSEL = 0x01;
    
    TRISBbits.TRISB1=0;
    PORTBbits.RB1=0;
    
    TRISAbits.TRISA0=1;
    PORTAbits.RA0=1;
    
    TRISCbits.TRISC2=0;
    PORTCbits.RC2=0;
    TRISCbits.TRISC3=0;
    PORTCbits.RC3=0;
    TRISCbits.TRISC4=0;
    PORTCbits.RC4=0;
    
    TRISAbits.TRISA1=0;
    PORTAbits.RA1=0;
    TRISAbits.TRISA2=0;
    PORTAbits.RA2=0;
    TRISAbits.TRISA5=0;
    PORTAbits.RA5=0;
    
    INTCON = 0;
    PIE1   = 0;
    PIE2   = 0;
    
    InitTMR2();
    InitPWM();
    InitADC();
    
    while(1){
        
        PORTCbits.RC4 = 1;
        PORTAbits.RA2 = 1;

        result = ADC_Read(0); //Analog Channel 0
        
        for (dc = 0; dc < 17; dc++) 
        {
            CCPR1L = dc;
            CCPR2L = 17-dc;
            __delay_ms(100);
        }

        for (dc = 16; dc > 0; dc--) 
        {
            CCPR1L = dc;
            CCPR2L = 17-dc;
            __delay_ms(100);
}
        
}

[Volker S. (vloki)]

ThinkPic schrieb:
> leider habe ich ein Problem mit meinem PIC. Dieser resetet sich
> automatisch, sobald ich

return ((ADRESH<<8)+ADRESL);

> ausführe.

Das scheint mir eh irgendwie falsch. Was soll das denn tun?
Hast du schon mal return ADRES; getestet? Könnte funktionieren.

Woher weißt du, dass es genau an dieser Zeile liegt? Debugger?

(ADRESH<<8 gibt vermutlich immer 0, warum das einen Reset auslösen 
solle, weiß ich allerdings auch nicht)

[HyperMario (Gast)]

ThinkPic schrieb:
> int ADC_Read(unsigned char channel)
..
> return ((ADRESH<<8)+ADRESL);

Solche Konstrukte würde ich lieber sein lassen. Der Compiler muss für 
den return Wert vor dem Interrupt Speicher reservieren. Ob es das macht?

global Irgend_nTyp InterrupReturnValue;

void InterruptRoutine(Irgend_nTyp Irgend_nVarname) {

   InterrupReturnValue = RegisterXY

}

Funzt bei mir zuverlässig.

[HyperMario (Gast)]

Da ist ja kein ADC Interrupt. Vergiss es oder probier ob es so läuft.

Sorry.

[ThinkPic (Gast)]

Moin,

ich bin mir nicht sicher, ob es direkt an der Zeile liegt. Im Debugger 
lande ich danach immer direkt wieder an den Anfang der Main.

Leider scheint der ADC aktuell gar nicht zu funktionieren. Ich bekomme 
immer als return 0x0000. Auch wenn ich

result = ((ADRESH<<8)+ADRESL); //Returns Result

 mache.

Ist meine ADC Konfiguration falsch? Möchte VDD als Ref verwenden und A0 
als Input.

[ThinkPic (Gast)]

Volker S. schrieb:
> ThinkPic schrieb:
>> leider habe ich ein Problem mit meinem PIC. Dieser resetet sich
>> automatisch, sobald ichreturn ((ADRESH<<8)+ADRESL);> ausführe.
>
> Das scheint mir eh irgendwie falsch. Was soll das denn tun?
> Hast du schon mal return ADRES; getestet? Könnte funktionieren.
>
> Woher weißt du, dass es genau an dieser Zeile liegt? Debugger?
>
> (ADRESH<<8 gibt vermutlich immer 0, warum das einen Reset auslösen
> solle, weiß ich allerdings auch nicht)

Return ADRES funktioniert nicht. Kennt der Compiler nicht.

[HyperMario (Gast)]

ThinkPic schrieb:
> Return ADRES funktioniert nicht. Kennt der Compiler nicht.

Weil es kein ADRES Register gibt.

Probier  doch mal das Beispiel aus dem Datenblatt.Ist zwar in Assembler, 
lässt sich aber leicht nach C umschreiben.

[Volker S. (vloki)]

HyperMario schrieb:
> Weil es kein ADRES Register gibt.

Oft hat aber jemand dem Compiler gesagt, was es bedeutet.
#define ADRES *(uint_16t*)(&ADRESL) oder so ähnlich

Das geht beim 16F886 leider nicht, weil die Register nicht 
dementsprechend liegen. Bei den neueren funktioniert es vermutlich 
immer.

@ThinkPIC
setz doch mal einen Breakpoint auf den Resetvector, bzw. den 
Interruptvector (im Program Memory View) und schau, ob er vielleicht 
dahin springt.

Oder setz den Breakpoint auf das return, und geh dann im Single Step 
weiter.
(single step auch im Program Memory View)

Wenn das (ADRESH<<8) später immer 0 liefert, dann probier 
((uint16_t)ADRESH)<<8)

[Peter D. (peda)]

ThinkPic schrieb:
> GO_nDONE = 1;
>   while(GO_nDONE);

Vermutlich wird GO_nDONE niemals auf 0 gesetzt, d.h. Endlosschleife und 
irgendwann schlägt der Watchdogreset zu.

[Patrick B. (p51d)]

Peter D. schrieb:
> ThinkPic schrieb:
>> GO_nDONE = 1;
>>   while(GO_nDONE);
>
> Vermutlich wird GO_nDONE niemals auf 0 gesetzt, d.h. Endlosschleife und
> irgendwann schlägt der Watchdogreset zu.

Würde ich auch schätzen.

ThinkPic schrieb:
> int ADC_Read(unsigned char channel)
> {
>   __delay_ms(2);
>   GO_nDONE = 1;
>   while(GO_nDONE);
>   return ((ADRESH<<8)+ADRESL);
> }

Finde ich sowieso etwas speziell. Normalerweise startet man doch eine 
Messung über ein ADC Flag und wartet bis der ADC ein Done Flag setzt. 
Oder das ganze ist im Freilauf. Aber selber irgendwelche Flags 
erfinden...

[Volker S. (vloki)]

Peter D. schrieb:
> Vermutlich wird GO_nDONE niemals auf 0 gesetzt, d.h. Endlosschleife und
> irgendwann schlägt der Watchdogreset zu.

ThinkPic schrieb:
> #pragma config WDTE = OFF       // Watchdog Timer Enable bit (WDT
> disabled)


Wenn der ADC richtig konfiguriert wurde, dann geht GO_nDONE nach 
erfolgter Wandlung wieder auf 0

ThinkPic schrieb:
> ADCON0 = 0x11000000;

Weiß zwar nicht, warum eine falsche Konfiguration einen RESET auslösen 
sollte, aber mach mal trotzdem das x zu einem b ;-)



Patrick B. schrieb:
> Aber selber irgendwelche Flags erfinden...

Wie kommst du jetzt darauf? Noch nie PICs programmiert?



####################################################################
Ich bin weiterhin für Debuggen!

[Lutz (Gast)]

ich kenne den pic zwar nicht aber muss man die Register nicht verodern? 
Wie groß ist ein int?

[Peter D. (peda)]

Lutz schrieb:
> muss man die Register nicht verodern?

Nein.
Links schieben füllt immer mit 0 auf, es ist also egal ob | oder +.