Frage zu Interrupts bei Atmega32U (Arduino Micro)

[Tom L. (tlin)]

Moin,

ich habe mir mit einem Arduino Micro eine kleine Sensoranzeige gebaut. 
Das Teil hat - neben den diversen Sensoren (DS1820, DHT22, SCT13 und 
LDR) - 2 Buttons. Mit dem einen dieser Buttons kann man durch eine 
Menüstruktur gehen (nur 1 Ebene: einfach eine Seite pro Sensor, recht 
primitiv also).

Das Ding funktioniert soweit so wie ich das haben wollte: 
http://youtu.be/kgwBwX49tQ8 . Das hier ist der aktuelle Source: 
http://www.daemon.de/idisk/Sensorita.ino.txt.

Nun habe ich folgendes Problem: zunächst hatte ich die Tastendrücke 
einfach innerhalb der loop()-Funktion abgefragt. Da jedoch das Auslesen 
der ganzen Sensoren etwas Zeit in Anspruch nimmt, ist es immer mal 
wieder dazu gekommen, dass ein Tastendruck ignoriert wurde. Das habe ich 
gelöst, indem ich die Tastendrücke über Interrupts abfrage (siehe Code). 
Das funktioniert soweit wunderprächtig, führt aber zu einem anderen 
Problem: hin und wieder (schwierig reproduzierbar) erscheint unmittelbar 
nach einem Tastendruck Zeichensalat auf dem LCD. Abgestürzt ist er dann 
nicht, weil er immer noch auf die Buttons reagiert und sich die Anzeige 
auch ändert (von einem Salat zum nächsten). Ich gehe daher davon aus, 
dass das deshalb passiert, weil der Interrupt in dem Moment eine 
laufende Sensorabfrage unterbrochen hat und dadurch irgendwelche 
Variablen leer/falsch befüllt sind (also ein Bufferoverflow ausgelöst 
wird).

Die Frage ist nun: wie kann man sowas lösen? Ich möchte, dass das Gerät 
grundsätzlich auf Tastendrücke ohne grosse Verzögerung reagiert und 
trotzdem alle 5 Sekunden die Sensoren ausgewertet werden ohne, dass der 
Vorgang unterbrochen wird. Ich könnte zwar während der Sensorabfrage 
temporär die Interrupts abschalten, was aber wieder zu Problem Nummer 1 
führt, dass ab und zu Tastendrücke untergehen.

[Tom L. (tlin)]

Keiner eine Idee?

[Karl H. (kbuchegg)]

Taster über Interrupts sind meistens keine gute Idee.

Was du aber tun könntest, um in deinem Programm nicht allzuviel ändern 
zu müssen.

void get_sensors () {
  // fetch all sensor readings
  get_dallas();
  get_lux();
  get_dht();
  get_current();
}

kein Mensch zwingt dich, immer alle Sensoren zur gleichen Zeit 
auszulesen.

Wenn du das zeitlich staffelst, dann dauert das Auslesen lediglich eines 
Sensors nicht so lange. Dafür liest du eben alle halbe Sekunde einen 
anderen der Sensoren aus.

Speziell die hier

void get_dallas () {
  // fetch all dallas DS1820 temperature readings
  sensors.requestTemperatures();
  for (int id=0; id<TempPins; id++) {
    CurrentTemps[id] = sensors.getTempCByIndex(id);
  }
}

werden den Löwenanteil der Zeit beim Sensorauslesen benötigen.
Aber wie gesagt: lies halt nicht alle auf einmal, sondern bei jedem

void loop(void) {
  // record current moments
  LCDMoment = SensorMoment = millis();

  // check if there is enough time gone, to re-check the sensors
  if (SensorMoment - SensorTimer > SensorReadIntervall) {
    get_sensors();

einen nur EINEN anderen. Aber eben jedesmal reihum jeweils den nächsten.