Arduino ignoriert Code

[ich (Gast)]

Hallo,

ich habe folgendes Problem mit einem Arduino UNO, wenn ich in dem Sketch 
(siehe Anhang) noch eine Zeile in die Funktion ISR(TIMER0_COMPA_vect) 
einfüge wird diese nicht ausgeführt.

Weiß jemand warum das so ist und wie man das Lösen kann?

Danke im vorraus

[Einer K. (Gast)]

Einige volatile vergessen
Serielle Ausgaben in der ISR ist eine tödliche Falle.

[leo (Gast)]

ich schrieb:
> Weiß jemand warum das so ist und wie man das Lösen kann?

      row_cnt = 1;
      Serial.println(row_cnt);  //Wird nicht ausgeführt

Die Verwendung von Serial.print ist in ISRs verboten, das haengt 
typischerweise, da es selbst Interrupts verwendet. Besser ein LED 
blinken oder so.

Die Variablen wie row_cnt brauchen ein volatile in der Deklaration.

leo

[ich (Gast)]

Danke für die Hinweise.

[ich (Gast)]

Ich habe mal das Serial.println rausgeworfen und will Pin A0 anstelle an 
und ausschalten. Ich habe bis jetzt damit auch keinen Erfolg.

[ich (Gast)]

ich schrieb:
> Ich habe mal das Serial.println rausgeworfen und will Pin A0 anstelle an
> und ausschalten. Ich habe bis jetzt damit auch keinen Erfolg.

Das digitalWrite(rowClkPin,!digitalRead(rowClkPin)); sollte natürlich in 
die Funktion ISR(TIMER0_COMPA_vect).

[Uff Basse (Gast)]

ich schrieb:
> Ich habe bis jetzt damit auch keinen Erfolg.

Wie stellst du das fest?

[ich (Gast)]

Uff Basse schrieb:
> ich schrieb:
>> Ich habe bis jetzt damit auch keinen Erfolg.
>
> Wie stellst du das fest?

In dem ich mit einem Logic analyzer den Zustand (LOW/HIGH) beobachte.

[Philipp K. (philipp_k59)]

bisschen Pseudo..

boolean isrtrigger=false;

ISR(){
isrtrigger=true;
}

loop(){
if(isrtrigger){
//TU WAS SOLL
isrtrigger=false;
}
}

[leo (Gast)]

ich schrieb:
> und will Pin A0 anstelle an
> und ausschalten.

Wie schnell willst du das sehen?

[ich (Gast)]

Philipp K. schrieb:
> bisschen Pseudo..

Damit habe ich aber auch keinen Erfolg.

[Uff Basse (Gast)]

Philipp K. schrieb:
> bisschen Pseudo..

Fehlt auf jeden Fall ein volatile.

[ich (Gast)]

Uff Basse schrieb:
> Philipp K. schrieb:
>> bisschen Pseudo..
>
> Fehlt auf jeden Fall ein volatile.

Geht auch mit einem volatile nicht.

[Philipp K. (philipp_k59)]

Versuch mal im Setup Vorweg:
TCCR0A =0;
TCCR0B =0;
//TIMSK0 =0; Optional

[Uff Basse (Gast)]

ich schrieb:
> Geht auch mit einem volatile nicht.

Ich sehe gerade, du nutzt unter Arduino Environment den Timer0.

Das kann nicht gehen denn Arduino benutzt den schon.

(Einschränkung: habe mir den Code nicht genau angeschaut,
in Ausnahmefällen mag das gehen. Für den Anfänger ist
das zu vermeiden, es gibt ja noch den Timer 1)

[ich (Gast)]

Danke, jetzt geht es.

[Uff Basse (Gast)]

ich schrieb:
> Danke, jetzt geht es.

... aber die Funktionalität die die Arduino-Umgebung bezüglich
Timer0 voraussetzt ist ausgeschaltet bzw eliminiert.

Das mag noch die eine oder andere Überraschung bringen ...

[Veit D. (devil-elec)]

Hallo,

wenn er im gesamten Sketch Timer 0 gegen Timer 1 getauscht hat, passiert 
gar nichts, rein absolut nichts. Nur seine Timer Initialisierung sollte 
er nochmal überdenken. Mit Register-Veroderung anfangen ist selten gut. 
Hier lautert die echte Falle.

Ich lösche bei der Initialisierung grundsätzlich immer zuerst TCCRnB. 
Damit ist der Timer gestoppt und kann keinen weiteren "Unsinn" machen. 
Auch setze ich die Prescaler erst ganz zum Schluss der Initialisierung. 
Ansonsten würde er zu zeitig loslaufen mitten in der noch 
unvollständigen Konfiguration. Alle   Bits der anderen Register werden 
Zeilenweise verodert, dann kann man weniger Blödsinn machen. Schon rein 
optisch wegen. Außer man möchte jedes Bit kommentieren. Aber selbst da 
fängt man ohne Veroderung an.

TCCR1A = (1<<COM1B1) // Kommentar
       | (1<<WGM11)  // Kommentar
       | (1<<WGM10)  // Kommentar
       ;

Man darf auch nicht vergessen Register zu löschen die man selbst gerade 
nicht benötigt, z.Bsp. TIMSKn.