Seltsame Probleme die sporadisch auftreten

[Dirk F. (dirkf)]

Hallo,
XC8 compiler PIC18F46Q71.

Ich steuere ein MOSFET jede 10 ms  im Timer Interrupt an.
Es kommt zeitweise zur falschen Ansteuerung im mode=2 (Zeile mit 
//Fehler).
Gemessen mit Oszilloskop.
Habe dann den code geändert  (2 Zeilen ohne Fehler)
Nach der Änderung keine Fehler mehr.
Aber ich verstehe den Grund nicht.

Oder kann es sein, dass in dieser Zeile
mode = slave_in[3] & 0x07;
der Interrupt zuschlägt und dann die Variable mode einen falschen Wert 
bekommt.

volatile unsigned char mode;          // Modus Ansteuerung MOSFET

//  Tabelle MOSFET Taktung------------------
 const char signal[8][12] = {
 {0,0,0,0,0,0,0,0,0,0,0,0},     // Mode 0  = Aus
 {1,1,1,1,1,1,1,1,1,1,1,1},     // Mode 1  = 200% U-Nenn 2xÜbererregung
 {1,0,1,0,1,0,1,0,1,0,1,0},     // Mode 2  = 100% U-Nenn Halbwelle       
 {1,0,0,1,0,0,1,0,0,1,0,0},     // Mode 3  = 75% U-Nenn          
 {1,0,0,0,1,0,0,0,1,0,0,0},     // Mode 4  = 50% U-Nenn  
 {1,0,0,0,0,0,1,0,0,0,0,0},     // Mode 5  = 34% U-Nenn  
 {1,0,0,0,0,0,0,0,0,0,0,0},     // Mode 6  = 17% U-Nenn 
 {1,1,0,0,1,1,0,0,1,1,0,0}      // Mode 7  = 100% U-Nenn  Vollwelle
 };
 
int main(void)
{

while (1)
    { 
    mode = slave_in[3] & 0x07;          // Neuen Wert vom Master
        
    // BYP_SCO1_LAT  = (mode>1);    // Fehler
    if (mode>1) BYP_SCO1_LAT =1;    // kein Fehler
    else        BYP_SCO1_LAT =0;
    }
}


// Im Timer Interrupt jede 10 ms:

        // Nächsten Wert für Takt IGBT aus Array holen
        MCU_SW1_LAT = signal[mode][pulse_pos]; 
        pulse_pos++;                        // Nächster Wert aus Array
        if (pulse_pos == 12) pulse_pos = 0; // Array Ende, dann von Anfang

[Peter D. (peda)]

Nun das eine ist ein Read-Modify-Write auf BYP_SCO1_LAT (nicht atomar),
Das andere sind 2 Zuweisungen ohne vorheriges Lesen (atomar).
RMW-Zugriffe auf IO-Bits können Seiteneffekte auf andere Bits im selben 
Byteregister haben.

[Dirk F. (dirkf)]

Peter D. schrieb:
> auf andere Bits im selben
> Byteregister haben.

Könnte sein. Beide liegen im selben Port:

[Falk B. (falk)]

Dirk F. schrieb:
> Ich steuere ein MOSFET jede 10 ms  im Timer Interrupt an.
> Es kommt zeitweise zur falschen Ansteuerung im mode=2 (Zeile mit
> //Fehler).
> Gemessen mit Oszilloskop.

Wie sieht denn der Fehler aus?

> Habe dann den code geändert  (2 Zeilen ohne Fehler)

Syntaktisch korrekt, aber die Formatierung sollte man nicht so machen. 
Das sit schlecht lesbar und damit fehleranfällig. Wenn, dann so.

if (mode>1) BYP_SCO1_LAT =1; else BYP_SCO1_LAT =0;

oder so

if (mode>1) {
    BYP_SCO1_LAT =1;
} else {
    BYP_SCO1_LAT =0;
}

> Nach der Änderung keine Fehler mehr.
> Aber ich verstehe den Grund nicht.

Ich vermute, daß die Sequenz komplett atomar sein muss, damit sich deine 
Logik nicht verklemmt. Denn mode wird auch in deiner ISR verwendet. Eher 
so.

    cli();  // Interrupts sperren, muss an PIC angepasst werden
    mode = slave_in[3] & 0x07;          // Neuen Wert vom Master
        
    // BYP_SCO1_LAT  = (mode>1);    // Fehler
    if (mode>1) BYP_SCO1_LAT =1;    // kein Fehler
    else        BYP_SCO1_LAT =0;
    sei();  // Interrupts freigeben, muss an PIC angepasst werden

Siehe Interrupt.

[Falk B. (falk)]

Dirk F. schrieb:
>> auf andere Bits im selben
>> Byteregister haben.
>
> Könnte sein. Beide liegen im selben Port:

Da hast du dein Problem.

https://www.mikrocontroller.net/articles/Interrupt#Atomarer_Datenzugriff

[Dirk F. (dirkf)]

Falk B. schrieb:
> Wie sieht denn der Fehler aus?

Normal wird jede zweite Halbwelle eingeschaltet.
Im Fehler fehlt manchmal  eine Halbwelle   oder es werden zwei 
Halbwellen ohne Pause  nacheinander ausgegeben.

[Johann K. (klammerj)]

Hast du schon versucht den mode Update auf den Sequenz Start zu 
Synchronisieren? (Die read-modify-write Probleme sind historisch, als 
die PICs zum lesen und schreiben nur ein Port Register hatten, und der 
externe Portzustand beim RmW in das Latch leckte. Neuere Modelle haben 
separate port und latch Register)

[Dirk F. (dirkf)]

Johann K. schrieb:
> Neuere Modelle haben
> separate port und latch Register)

Der PIC18F46Q71  ist ein neues Modell.
Es geht hier aber nur ums schreiben ins LATCH.