PIC18F Fehler im Code finden (C)

[Tim H. (pic_fan)]

Hallo zusammen,

ich suche schon seit ein paar Stunden nach dem fehler im Code.
Was ich rausgefunden habe, dass der Fehler in den unten aufgeführten 
code stecken muss.

Zur Thematik: Ein angeschlossenes Sensor modul am Pic soll die Messwerte 
per UART an den Pc übertragen.
Zum testen nutze ich Putty ein Terminal programm.
Ich habe einen ISR programmiert der beim RX vom UART ausgelöst wird und 
die im code ersichtliche funktionen ausführen soll.

Das bild zeigt was passiert.

In Putty gebe ich eine 1 ein um den interrupt 
auszulösen.(1=temp,2=Feuchtigkeit)

Der interrupt wird ausgelöst und sendet mir das richtige ergebnis + die 
ausgabe die er eignetlich nur machen soll wen weder 1 noch die 2 
gedrückt wurden.

Danach kann ich auch keinen weitern interrupt Auslösen.(z.B. Erst die 1 
für Temp dann die 2 für Feuchtigkeit)

void interrupt SerialRxPinInterrupt()
{
    uint8_t i = 0;
    //Prüft ob der interrup von uart empfang kommt
    if(PIR1bits.RCIF == 1)
    {
        while(DataRdyUSART()){
            u_buffer[i] = ReadUSART(); //wenn ja lies datein
            i++;
        }
        if(u_buffer[0] == '1'){ //1= temp
            Uart_Send_Temp();
        }
        else if(u_buffer[0] == '2'){ //2 = Feuchtigkeit
            Uart_Send_Hum();
        }
        else if(u_buffer[0]== '\0' || u_buffer[0]== '\r' || u_buffer[0]== '\n' ){
            putsUSART("\n");
        }        
        PIR1bits.RCIF = 0; //Löschend des RX interrupt flag bit
    }//end if
}//end interrupt

Danke im Vorraus

Tim

[Max H. (hartl192)]

Wie sieht Uart_Send_Hum bzw. -Temp aus?
Die ISR macht man eigentlich so kurz wie möglich und ruft darin keine 
(langen) Funktionen auf.

[Tim H. (pic_fan)]

Max H. schrieb:
> Wie sieht Uart_Send_Hum bzw. -Temp aus?

UartOpen() findet in der main statt.

void Uart_Send_Temp(void){
    sprintf(buffer,"%.2f", temp()); //Übertragung für 2 nach komma stellen
    putsUSART(buffer);// senden des i2c convertieren wertes über usart übertragen
}

die hum siet genau so aus.

gruß

[Tim H. (pic_fan)]

Fehler Gefudnen er wurde behoben.

Danke Aber
CLOSE

[Alex W. (a20q90)]

Tim Halbach schrieb:
> Fehler Gefudnen er wurde behoben.
>
> Danke Aber
> CLOSE

Und welcher war es jetzt? Wir wollen auch was lernen!

[Tim H. (pic_fan)]

Ich habe ien Konfiguration überschrieben.

C-Code

void setupUart(void){
    //pins auf input setzten, Pic Configuriert sich die pins nach bedarf für USART
    asm("BANKSEL TRISC");
    asm("MOVLW 0xD8"); //Tris ports auf inputstellen RX TX und SDA SCL
    asm("MOVWF TRISC");
//    asm("BSF TRISC,7"); !!!HIER DER FEHLER!! standvorher drine
//    asm("BSF TRISC,6");

    uartConfig = USART_RX_INT_ON & USART_TX_INT_OFF & USART_ASYNCH_MODE & USART_EIGHT_BIT & USART_BRGH_HIGH;
    baud = 103; // Baudrate 9600 = 51 (8Mhz), 103(16mhz)= 9615

    //Interrupt
    RCIF = 0; //RX pin flag auf 0 setzten (reset)
    RCIP = 0; //Keine hohe Priorität für den interrupt
    RCIE = 1; //Einschalten für RX Intereuppt
    PEIE = 1; //Pheripheral interuppt erlauebn (Uart ist ein Pheripharal)
}

void initI2C(void){
//======================= Rausgenommen da hier eine änderung des TRISC stattgefunden hat.
    //    asm("BANKSEL TRISC");
    //    asm("MOVLW 0x18"); // SCL (RC3) und SDA (RC4) Auf inputstellen
    //    asm("MOVWF TRISC");

    asm("BANKSEL SSPCON1");
    asm("MOVLW 0x28");  // Master Mode Clock einstellung (Master mode clock = FOSC/(4 * (SSPADD + 1)))
    asm("MOVWF SSPCON1"); //& SCL &SDA als serialport bestimmen

    asm("BANKSEL SSPSTAT"); //Slew mode dissable
    asm("BSF SSPSTAT,7");

    //Baudrate von I2C einstellen auf 100khz
    asm ("MOVLW 0x27"); // (FOSC/(4*xKhz))-1 = (4Mhz/(4*100Khz))-1 = 9 (8mhz = 19) (16mhz=  0x27)
    asm ("BANKSEL SSPADD");
    asm ("MOVWF SSPADD");

    asm("BANKSEL SSPCON2");// ack in master mode
    asm("bcf SSPCON2,5");
}

ich ahbe in 2 Funktionen eine Änderung am INput des TRISC vorgenommen 
als erste lege ich Die USART config fest wo ich die RX und TX pin auf 
input stelle(R6 & RC7 = 0xC0)

das gleiche muss icha cuuh bei der initialisierung von I2C machen dabi 
habe ich aber andere pin (RC3 & RC4 = 0x18)

Also musste ich RC6, RC7, RC3 und RC4 setzen und das ist = 0xD8

EIn blöder programmier fehler und sehr unübersichtlich das cih alles in 
verschiedenen .c Datein haben.

Gruß

Tim

[Max H. (hartl192)]

Wenn der Fehler in dem Teil des Codes, den du nicht gepostet hast, war, 
können wir natürlich nichts machen...

Btw: Es ist trotzdem nicht sauber in der ISR alle möglichen Funtionen 
wie z.B. sprintf mit float aufzurufen.