Problema interrupt su linea SPI con pic18f46k22

[spivo]

Ciao a tutti, ho un problemino di software un po' difficile da spiegare.

Questo che allego è parte del codice per un micro pic18f46k22, il codice dovrebbe fare questo:

Aspettare che un raspberry lo interroghi tramite connessione spi (spi2 nel mio caso particolare), se il primo byte è 0x61 riceve 30 byte e invia i 30 che lui si era preparato.
Quando torna al ciclo infinito se i il primo dei trenta era 0x61 e l'ultimo 0x62 aggirona i dati per il prossimo invio.

Codice: Seleziona tutto

void main(void)
{
    ConfigureOscillator();
    ConfigurePort();
    ConfigureSPI1();
    ConfigureMCPD();
   

    // Visualizza accensione
    LEDrosso = 1;
    LEDverde = 1;

    //CONFIGURAZIONE SPI2 SLAVE
    SSP2CON1bits.SSPEN = 1;
    SSP2CON1bits.SSPM = 0b0100;
    SSP2CON1bits.CKP = 0;

    SSP2STATbits.CKE = 1;
    SSP2STATbits.SMP = 0;

    SSP2BUF = 0;

    //CONFIGURAZIONE INTERRUPT
    RCONbits.IPEN = 1;

    PIE3bits.SSP2IE = 1;
    IPR3bits.SSP2IP = 1;
    PIR3bits.SSP2IF = 0;

    //Prepara le variabili prima dell'avvio
    ch_num = 0;
    SistemazioneDati();

    for (i=0;i<100; i++) __delay_ms(10);
    LEDrosso = 0;
    LEDverde = 0;

    //ATTIVA INTERRUPT
    INTCONbits.GIE = 1;


    while(1)
    {


         if (ch_in[0] == 0x61 && ch_in[29] == 0x62){
             ch_num = 0;
             ch_in[0] = 0;
             ch_in[29] = 0;
                // invia dati a mm5450 1
                MM_5450_1();
                // Lampeggio led dopo l'invio dei dati
                LEDverde = ~LEDverde;
                // Sistemazione dati prima del prossimo invio
                SistemazioneDati();
         }


    // Rilevamento MCP23S17 1
    MCPD_1=0; //Indica allo slave che i dati in arrivo
        SSP1BUF=0b01000001; //indirizzo + read
        while(!SSP1IF);
        ch_canc=SSP1BUF;
        SSP1IF=0;
        SSP1BUF=0x12; //Invio GPIOA
        while(!SSP1IF);
        ch_canc=SSP1BUF;
        SSP1IF=0;
        SSP1BUF=0x00;
        while(!SSP1IF);
        ch_out[3]&=SSP1BUF;
        SSP1IF=0;
    MCPD_1=1; //Fine comunicazione
    MCPD_1=0; //Inizio comunicazione
        SSP1BUF=0b01000001; //indirizzo + read
        while(!SSP1IF);
        ch_canc=SSP1BUF;
        SSP1IF=0;
        SSP1BUF=0x13; //Invio GPIOB
        while(!SSP1IF);
        ch_canc=SSP1BUF;
        SSP1IF=0;
        SSP1BUF=0x00;
        while(!SSP1IF);
        ch_out[4]&=SSP1BUF;
        SSP1IF=0;
    MCPD_1=1; //Fine comunicazione

        for (i=0;i<2; i++) __delay_ms(1);
        LEDrosso = ~LEDrosso;
    }
}

void interrupt high_isr(void)
{
    if (PIR3bits.SSP2IF)
    {
        ch_in[ch_num] = SSP2BUF;
        SSP2BUF = ch_out[ch_num];
        ch_num++;
       
        if (ch_in[0] != 0x61 || ch_num > 30) ch_num = 0;

    PIR3bits.SSP2IF = 0; //reset interrupt
    }
}


Mentre aspetta di essere inerrogato da raspberry il PIC continua a leggre un mcp23s17 per monitorare degli ingressi digitali e fa lampeggiare freneticamente il LEDrosso.

Prima che inserissi la parte di codice relativa all'mcp23s17 tutto funzionava bene. (raspberry conta se una connessione da errore e in due settimane di connessioni continue non ha mai dato errori)

Con il codice postato, da un centinaio di errori al giorno.

Ho provato anche ad eliminare il lampeggio del LEDrosso, (che avevo inserito all'inizio per sapere se il PIC era acceso). Se elimino questa parte di codice da una marea di errori.

Spero di essermi spiegato, non so nemmeno come intitolare il post, grazie caio Ivo

[GuidoB]

Secondo me il problema sta in quei

Codice: Seleziona tutto

while(!SSP1IF);

che costringono il processore ad attendere dentro il loop principale.
Mentre attende non può fare altre cose.

Credo che dovresti mettere in una variabile il punto in cui sei arrivato (lo stato).
Non ti devi fermare ad aspettare che SSP1IF diventi vero, ma lo controlli e solo se è vero esegui le azioni da fare in quello stato, e aggiorni lo stato.

In pratica dovresti strutturare la gestione dell'mcp23s17 come una macchina a stati.

[WALTERmwp]

Ciao spivo, non so quanto tu possa ritenere affidabile la diagnostica del lampone, forse lo è e allora la consideriamo tale.

L'I/O expander presumo sia anch'esso collegato via SPI però dei registri interessati (per MSSP1) non si vede alcuna impostazione; mi pare non venga utilizzato l'irq, altrimenti ci sarebbe qualcosa nella relativa routine.
Non si vedono i contenuti delle prime funzioni di inizializzazione e non si vedono le dichiarazioni delle variabili, non vorrei che un array pestasse i "piedi" all'altro.
Insomma, forse mancano un po di informazioni.

Con un po di fantasia, per essere propositivo, provo ad azzardare una ipotesi: se il fault di comunicazione da parte del raspy è basato anche sul contenuto del messaggio, funzione dell'aggiornamento del contenuto dei dati ( ... SistemazioneDati(); ), allora è più che probabile che tu ti possa trovare con una valanga di errori.
E qui la soluzione sarebbe quella indicata da GuidoB.

Saluti

[spivo]

Ciao e grazie a tutti,
Aveva ragione GuidoB il problema doveva essere nei while, ora ho sostituito il codice che leggeva mcp23s17 con questo:

Codice: Seleziona tutto

    if (passo == 0) {
    // Rilevamento MCP23S17 1
    MCPD_1=0; //Indica allo slave che i dati in arrivo
        SSP1BUF=0b01000001; //indirizzo + read
        passo = 1;
    }
    if (passo == 1 && SSP1IF==1) {  //while(!SSP1IF);
        ch_canc=SSP1BUF;
        SSP1IF=0;
        passo = 2;
    }
    if (passo == 2){
        SSP1BUF=0x12; //Invio GPIOA
        passo = 3;
    }
    if (passo == 3 && SSP1IF==1) {  //while(!SSP1IF);
        ch_canc=SSP1BUF;
        SSP1IF=0;
        passo = 4;
    }
    if (passo == 4){
        SSP1BUF=0x00;
        passo = 5;
    }
    if (passo == 5 && SSP1IF==1) {   //while(!SSP1IF);
        ch_out[3]&=SSP1BUF;
        SSP1IF=0;
        passo = 6;
    }
    if (passo == 6) {
    MCPD_1=1; //Fine comunicazione
    MCPD_1=0; //Inizio comunicazione
        SSP1BUF=0b01000001; //indirizzo + read
        passo = 7;
    }
    if (passo == 7 && SSP1IF==1) {  //while(!SSP1IF);
        ch_canc=SSP1BUF;
        SSP1IF=0;
        passo = 8;
    }
    if (passo == 8) {
        SSP1BUF=0x13; //Invio GPIOB
        passo = 9;
    }
    if (passo == 9 && SSP1IF==1) {  //while(!SSP1IF);
        ch_canc=SSP1BUF;
        SSP1IF=0;
        passo = 10;
    }
    if (passo == 10) {
        SSP1BUF=0x00;
        passo = 11;
    }
    if (passo == 11 && SSP1IF==1) {//while(!SSP1IF);
        ch_out[4]&=SSP1BUF;
        SSP1IF=0;
        passo = 12;
    }
    if (passo == 12) {
    MCPD_1=1; //Fine comunicazione
    passo = 0;
    }


ora la comunicazione spi con raspberry funziona benissimo ma non riesco più a leggere il mcp23S17

P.S.
io credevo che gli interrupt interrompessero il programma indipendentemente da cosa stesse eseguendo, in teoria anche nel main c'è un ciclo infinito che comunque viene interrotto.

Grazie ancora a tutti Ciao Ivo

[WALTERmwp]

(...) non riesco più a leggere il mcp23S17 (...)

e certo.
Guarda bene cosa hai scritto ...
Se, per esempio, passo vale 1 e la flag SSP1IF ti fa entrare nello "if" relativo, cosa fai, che valori imposti ?
e appena esci da quello "if" cosa succede ?

Saluti

p.s.
no, ho riportato una considerazione sbagliata, ma la lascio lì, ci "guardo" meglio ...

[spivo]

in teoria se passo == 1 e SSP1IF == 1 (vorrebbe dire che è arrivato il dato la da leggere) allora leggo il dato assegnandolo alla variabile ch_canc poi resetto il flag SSP1IF e avanzo di un passo che diventa 2. Quando esco se passo == 2 allora invio un altro dato e avanzo di un passo ....
perche non funzia?

[WALTERmwp]

Ho messo il p.s.

Saluti

[spivo]

Scusa non avevo letto tutto
Grazie mille Ciao Ivo

[WALTERmwp]

Questo loop

Codice: Seleziona tutto

for (i=0;i<2; i++) __delay_ms(1);
            LEDrosso = ~LEDrosso;


nel codice del Post [4], lo hai conservato e nella medesima posizione del codice del Post [1] ?

Saluti

[spivo]

Scusate il ritardo, ma purtroppo non è un bel periodo.

Ho controllato meglio il codice postato nel post[4] ed è corretto, il problema è che ci sono altre connessioni spi nelle funzioni tipo SistemazioneDati(); ed altre.
Quindi per correggere l'errore devo riscrivere tutto il codice a "passi" un bel lavoraccio.
Grazie a tutti, Ciao Ivo.