From 5f3780730bd4add661f8d961e0167bd1a239757c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Xose=20P=C3=A9rez?= <xose.perez@gmail.com>
Date: Sat, 1 Apr 2017 19:59:20 +0200
Subject: [PATCH] Minimize my9291 flickering when firmware boots

---
 code/espurna/light.ino |  2 +-
 code/espurna/relay.ino | 15 ++++++++++-----
 2 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/code/espurna/light.ino b/code/espurna/light.ino
index 861cd455..6596249c 100644
--- a/code/espurna/light.ino
+++ b/code/espurna/light.ino
@@ -166,7 +166,7 @@ void _lightColorSave() {
 
 void _lightColorRestore() {
     String color = getSetting("color", LIGHT_DEFAULT_COLOR);
-    lightColor(color.c_str(), false, true);
+    color_string2array(color.c_str(), _lightColor);
 }
 
 // -----------------------------------------------------------------------------
diff --git a/code/espurna/relay.ino b/code/espurna/relay.ino
index 02a30f9b..d571f97f 100644
--- a/code/espurna/relay.ino
+++ b/code/espurna/relay.ino
@@ -16,10 +16,8 @@ typedef struct {
     unsigned char pin;
     bool reverse;
     unsigned char led;
-
     unsigned int floodWindowStart;
     unsigned char floodWindowChanges;
-
     unsigned int scheduledStatusTime;
     bool scheduledStatus;
     bool scheduledReport;
@@ -159,6 +157,7 @@ bool relayStatus(unsigned char id, bool status, bool report) {
     bool changed = false;
 
     if (relayStatus(id) != status) {
+
         unsigned int floodWindowEnd = _relays[id].floodWindowStart + 1000 * RELAY_FLOOD_WINDOW;
         unsigned int currentTime = millis();
 
@@ -175,11 +174,12 @@ bool relayStatus(unsigned char id, bool status, bool report) {
         _relays[id].scheduledStatus = status;
         _relays[id].scheduledReport = (report ? true : _relays[id].scheduledReport);
 
-        DEBUG_MSG_P(PSTR("[RELAY] scheduled %d => %s in %u ms\n"),
+        DEBUG_MSG_P(PSTR("[RELAY] Scheduled %d => %s in %u ms\n"),
                 id, status ? "ON" : "OFF",
                 (_relays[id].scheduledStatusTime - currentTime));
 
         changed = true;
+
     }
 
     return changed;
@@ -480,6 +480,7 @@ void relaySetup() {
     }
     if (relayMode == RELAY_MODE_SAME) relayRetrieve(false);
     if (relayMode == RELAY_MODE_TOOGLE) relayRetrieve(true);
+    relayLoop();
 
     relaySetupAPI();
     relaySetupMQTT();
@@ -491,11 +492,12 @@ void relaySetup() {
 
 }
 
-void relayLoop(void)
-{
+void relayLoop(void) {
+
     unsigned char id;
 
     for (id = 0; id < _relays.size(); id++) {
+
         unsigned int currentTime = millis();
         bool status = _relays[id].scheduledStatus;
 
@@ -522,6 +524,9 @@ void relayLoop(void)
             #endif
 
             _relays[id].scheduledReport = false;
+
         }
+
     }
+
 }