Watchdog Reset mit Oled Display und Rotary Encoder HILFE

[Peerless R. (peerless_r)]

Servuz,

ich baue gerade einen Ersatz-Controller für eine Kühlbox, da der alte 
(ChinaMüll) Controller abgeraucht ist. Ich verwende einen Wemos D1 Mini, 
ein 128x64 SSD1306 OLED Display und einen Rotary Encoder über I2C.

Das ganze soll relativ simpel sein, man schaltet das Gerät an, es zeigt 
die aktuelle Temperatur auf dem Display an. Man drückt den Rotary 
Encoder Button und ist nun im Einstellungs-Modus, durch Drehen stellt 
man die Ziel-Temperatur ein, durch erneutes Drücken des Buttons wird 
diese bestätigt. Den Berechnungsteil mit Widerstand/Temperatur messen 
und Steuern des Kompressors etc hab ich noch ausgelassen geht mir 
erstmal nur um die Bedienelemente. Soweit funktioniert jetzt alles ABER 
wenn ich den Button drücke, dann einen Wert einstelle und dann einfach 
warte und nichts mache, triggert der Watchdog und macht nen Reset. Das 
selbe wenn ich den Button drücke und dann gar keinen Wert einstelle und 
einfach nur warte. So nach 2-3 Sekunden hängt sich da was auf und ich 
hab keine Ahnung wieso. Bin dann doch eher Elektroniker als 
Programmierer und hoffe mir kann jemand weiterhelfen.
Ich verwende die Library ESPRotary und für das Display die Adafruit 
SSD1306/GFX Libraries.

Hier hab ich mal ein kurzes Video von dem Problem zum besseren 
Verständnis:

https://streamable.com/76rj2y

Und hier natürlich mein Code:

#include <Wire.h>
#include <Adafruit_GFX.h>
#include <Adafruit_SSD1306.h>
#include "ESPRotary.h"

#define ROTARY_PIN1 12
#define ROTARY_PIN2 14
#define CLICKS_PER_STEP 4
#define MIN_POS         -20
#define MAX_POS         20
#define START_POS       0
#define INCREMENT       1
#define SCREEN_WIDTH 128
#define SCREEN_HEIGHT 64
#define OLED_RESET -1
#define LOGO_WIDTH 128
#define LOGO_HEIGHT 64

ESPRotary r;

Adafruit_SSD1306 display(SCREEN_WIDTH, SCREEN_HEIGHT, &Wire, OLED_RESET);

const unsigned char logo_bmp [] PROGMEM = {
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x7f, 0xff, 0xe0, 0x00, 0x00, 0x00, 0x00, 0x00, 0x7c, 0x00, 0x0f, 0xc0, 0x7c, 0x00, 0x00, 0x00, 
  0x7f, 0xff, 0xe0, 0x00, 0x00, 0x00, 0x00, 0x00, 0x7c, 0x00, 0x0f, 0xc0, 0x7c, 0x00, 0x00, 0x00, 
  0x7f, 0xff, 0xc0, 0x00, 0x00, 0x00, 0x00, 0x00, 0xfc, 0x00, 0x0f, 0x80, 0xfc, 0x00, 0x00, 0x00, 
  0x7f, 0xff, 0xff, 0xf0, 0x00, 0x00, 0x00, 0x00, 0xf8, 0x00, 0x1f, 0x80, 0xf8, 0x00, 0x00, 0x00, 
  0x7f, 0xff, 0xff, 0xf0, 0x00, 0x00, 0x00, 0x01, 0xf8, 0x00, 0x1f, 0x01, 0xf8, 0x00, 0x00, 0x00, 
  0x00, 0x0f, 0xff, 0xe0, 0x00, 0x00, 0x00, 0x01, 0xf8, 0x00, 0x00, 0x01, 0xf8, 0x00, 0x00, 0x00, 
  0x00, 0x1f, 0xff, 0xff, 0xf8, 0x0f, 0xff, 0x81, 0xf0, 0xfe, 0x3f, 0x1f, 0xff, 0x07, 0xff, 0xe0, 
  0x00, 0x1f, 0xff, 0xff, 0xf8, 0x3f, 0xff, 0x83, 0xf1, 0xfc, 0x3e, 0x1f, 0xff, 0x1f, 0xff, 0xc0, 
  0x00, 0x1f, 0x07, 0xff, 0xf0, 0x7f, 0xff, 0x83, 0xe3, 0xf8, 0x7e, 0x3f, 0xfe, 0x3f, 0xff, 0xc0, 
  0x00, 0x3f, 0x0f, 0xff, 0xf0, 0x7f, 0xff, 0x03, 0xe7, 0xf0, 0x7e, 0x3f, 0xfe, 0x3f, 0xff, 0xc0, 
  0x00, 0x3e, 0x0f, 0xff, 0xf0, 0xff, 0xff, 0x07, 0xef, 0xe0, 0x7c, 0x07, 0xe0, 0x3f, 0xff, 0x80, 
  0x00, 0x3e, 0x0f, 0x83, 0xe0, 0xf8, 0x3f, 0x07, 0xff, 0xc0, 0xfc, 0x07, 0xc0, 0x7e, 0x1f, 0x80, 
  0x00, 0x7e, 0x1f, 0x87, 0xe0, 0xf8, 0x3e, 0x07, 0xff, 0x80, 0xfc, 0x07, 0xc0, 0x7c, 0x1f, 0x80, 
  0x00, 0x7c, 0x1f, 0x07, 0xc1, 0xf8, 0x7e, 0x0f, 0xff, 0x00, 0xf8, 0x0f, 0xc0, 0xfc, 0x1f, 0x00, 
  0x00, 0xfc, 0x1f, 0x07, 0xc1, 0xf0, 0x7e, 0x0f, 0xbe, 0x01, 0xf8, 0x0f, 0x80, 0xfc, 0x3f, 0x00, 
  0x00, 0xfc, 0x3f, 0x0f, 0xc3, 0xf0, 0x7c, 0x1f, 0xbf, 0x01, 0xf0, 0x1f, 0x80, 0xf8, 0x3f, 0x00, 
  0x00, 0xf8, 0x3e, 0x0f, 0x83, 0xf0, 0xfc, 0x1f, 0x9f, 0x01, 0xf0, 0x1f, 0x81, 0xf8, 0x3e, 0x00, 
  0x01, 0xf8, 0x7e, 0x1f, 0x83, 0xe0, 0xf8, 0x1f, 0x1f, 0x03, 0xf0, 0x1f, 0x01, 0xf8, 0x7e, 0x00, 
  0x01, 0xf0, 0x7c, 0x1f, 0x87, 0xe0, 0xf8, 0x3f, 0x1f, 0x03, 0xe0, 0x3f, 0x01, 0xf0, 0x7c, 0x00, 
  0x01, 0xf0, 0x7c, 0x1f, 0xff, 0xff, 0xff, 0xfe, 0x1f, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xfe, 
  0x03, 0xf0, 0xfc, 0x3f, 0xff, 0xff, 0xff, 0xfe, 0x0f, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xfe, 
  0x03, 0xe0, 0xf8, 0x3f, 0xf7, 0xfd, 0xff, 0x7e, 0x0f, 0xf3, 0xff, 0xbf, 0xfb, 0xfe, 0x7f, 0xfe, 
  0x07, 0xe1, 0xf8, 0x3f, 0xe3, 0xf8, 0xfe, 0x7c, 0x0f, 0xe3, 0xff, 0x1f, 0xf1, 0xfc, 0x7f, 0xfe, 
  0x07, 0xe1, 0xf8, 0x7f, 0xc1, 0xf0, 0x7c, 0x7c, 0x0f, 0xc0, 0xfe, 0x0f, 0xe0, 0xf8, 0x1f, 0xfe, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 
  0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00
};

void setup() {
  Serial.begin(9600);
  delay(50);

  pinMode(13, INPUT);

  r.begin(ROTARY_PIN1, ROTARY_PIN2, CLICKS_PER_STEP, MIN_POS, MAX_POS, START_POS, INCREMENT);
  r.setChangedHandler(rotate);

  if(!display.begin(SSD1306_SWITCHCAPVCC, 0x3C)) {
    Serial.println(F("SSD1306 allocation failed"));
    for(;;);
  }
  
  startup();
  display.setTextSize(4);
  display.setTextColor(SSD1306_WHITE);
}

void loop() {
  showTemp();
  if (digitalRead(13) == LOW) {
    setTemp();
  }
}

void setTemp(void) {
  delay(500);
  display.setTextSize(4);
  display.setTextColor(SSD1306_WHITE);
  display.clearDisplay();
  display.fillRect(0, 60, 128, 10, SSD1306_WHITE);
  display.display();
  do {
      r.loop();
  } while (digitalRead(13) != LOW);
  delay(500);
}

void showTemp(void) {
  display.clearDisplay();
  display.setCursor(0, 10);
  int tempval = analogRead(A0);
  long temp = map (tempval, 0, 1023, -20, 20);
  display.print(temp);
  display.print((char)247);
  display.print("C");
  display.display();
}

void startup(void) {
  display.clearDisplay();
  display.drawBitmap(
    (display.width()  - LOGO_WIDTH ) / 2,
    (display.height() - LOGO_HEIGHT) / 2,
    logo_bmp, LOGO_WIDTH, LOGO_HEIGHT, 1);
  display.display();
  delay(1500);
  String str = "Modell 650";
  display.clearDisplay();
  display.setTextColor(SSD1306_WHITE);
  display.setTextSize(2);
  display.setCursor(128-(str.length())/2, 10);
  display.print(F("Modell 650"));
  display.display();
  delay(1500);
}

void rotate(ESPRotary& r) {
  int pos = r.getPosition();
  Serial.println(pos);
  display.clearDisplay();
  display.fillRect(0, 60, 128, 10, SSD1306_WHITE);
  display.setCursor(0, 10);
  display.print(pos);
  display.print((char)247);
  display.print("C");
  display.display();
}

[Stefan F. (Gast)]

Espressif empfiehlt maximal 20ms, sonst riskierst du einen Ausfall der 
WLAN Schnittstelle. Deine Warteschleife dauert so lange, dass der 
Watchdog einen Neustart auslöst.

Diesen und weitere gemeine Hinweise findest du auf 
http://stefanfrings.de/esp8266/index.html#fallstricke

do 
{
    r.loop();
} 
while (digitalRead(13) != LOW);

Dein Programm ist falsch strukturiert. Eigentlich soll es solche 
Warteschleifen nicht geben. Die loop soll bei jedem Durchlauf nur kurz 
etwas machen und dann die Kontrolle zurück an das Betriebssystem 
übergeben (indem sie sich beendet).

Deine langen delay() Aufrufe sind schon schlecht. An dieser Stelle 
rettet dich allerdings das Arduino Framework, weil es innerhalb der 
delay Funktion aktiv Kontrolle an das Betriebssystem abgibt indem es 
yield() aufruft. Das könntest du in deiner Warteschleife auch tun. 
Allerdings treibt man damit den Verbrauch von Stapelspeicher (der eh 
relativ klein ist) in die Höhe und handelt sich ab und zu mal andere 
unerwartete Seiteneffekte ein.

Lies meinen Artikel über Zustandsautomaten, denn da lernst du, wie du 
deine Aufgabe ganz ohne Warteschleifen umsetzen kannst.
http://stefanfrings.de/multithreading_arduino/index.html

[Peerless R. (peerless_r)]

Hey danke für deine ausführliche Antwort, versteh das Problem jetzt 
auch, da hat wirklich ne Fliege geklemmt. Hab den Code umgeschrieben, 
damit er im main loop bleibt und dort über Variablen checkt ob er sich 
im "Settings Modus" befindet. Jetzt funktioniert alles wunderbar. :)