From d280e167238b11171353fa4befbece404066ebd8 Mon Sep 17 00:00:00 2001
From: cschwinne <dev.aircoookie@gmail.com>
Date: Sat, 19 Mar 2022 14:21:14 +0100
Subject: [PATCH] Fixed `/json/cfg` unable to set busses (fixes #2589)

---
 CHANGELOG.md      | 5 +++++
 wled00/FX_fcn.cpp | 1 +
 wled00/cfg.cpp    | 4 +++-
 wled00/wled.cpp   | 4 ++--
 wled00/wled.h     | 2 +-
 5 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 977a5d849..399839483 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,6 +2,11 @@
 
 ### Builds after release 0.13.1
 
+#### Build 2203190
+
+-   Fixed `/json/cfg` unable to set busses (#2589)
+-   Fixed Peek with odd LED counts > 255 (#2586)
+
 #### Build 2203160
 
 -   Version bump to v0.13.2-a0 "Toki"
diff --git a/wled00/FX_fcn.cpp b/wled00/FX_fcn.cpp
index 35b6c07d8..a84eeb6f8 100644
--- a/wled00/FX_fcn.cpp
+++ b/wled00/FX_fcn.cpp
@@ -82,6 +82,7 @@ void WS2812FX::finalizeInit(void)
 
   //if busses failed to load, add default (fresh install, FS issue, ...)
   if (busses.getNumBusses() == 0) {
+    DEBUG_PRINTLN(F("No busses, init default"));
     const uint8_t defDataPins[] = {DATA_PINS};
     const uint16_t defCounts[] = {PIXEL_COUNTS};
     const uint8_t defNumBusses = ((sizeof defDataPins) / (sizeof defDataPins[0]));
diff --git a/wled00/cfg.cpp b/wled00/cfg.cpp
index 4fb07fbc8..a7d60dd2a 100644
--- a/wled00/cfg.cpp
+++ b/wled00/cfg.cpp
@@ -116,7 +116,6 @@ bool deserializeConfig(JsonObject doc, bool fromFS) {
       bool reversed = elm["rev"];
       bool refresh = elm["ref"] | false;
       ledType |= refresh << 7; // hack bit 7 to indicate strip requires off refresh
-      s++;
       if (fromFS) {
         BusConfig bc = BusConfig(ledType, pins, start, length, colorOrder, reversed, skipFirst);
         mem += BusManager::memUsage(bc);
@@ -126,6 +125,7 @@ bool deserializeConfig(JsonObject doc, bool fromFS) {
         busConfigs[s] = new BusConfig(ledType, pins, start, length, colorOrder, reversed, skipFirst);
         doInitBusses = true;
       }
+      s++;
     }
     // finalization done in beginStrip()
   }
@@ -455,7 +455,9 @@ bool deserializeConfig(JsonObject doc, bool fromFS) {
   }
 
   if (fromFS) return needsSave;
+  // if from /json/cfg
   doReboot = doc[F("rb")] | doReboot;
+  if (doInitBusses) return false; // no save needed, will do after bus init in wled.cpp loop
   return (doc["sv"] | true);
 }
 
diff --git a/wled00/wled.cpp b/wled00/wled.cpp
index a98116954..942a9666b 100644
--- a/wled00/wled.cpp
+++ b/wled00/wled.cpp
@@ -28,7 +28,7 @@ void WLED::reset()
     yield();        // enough time to send response to client
   }
   applyBri();
-  DEBUG_PRINTLN(F("MODULE RESET"));
+  DEBUG_PRINTLN(F("WLED RESET"));
   ESP.restart();
 }
 
@@ -152,7 +152,7 @@ void WLED::loop()
 
   yield();
 
-  if (doReboot)
+  if (doReboot && !doInitBusses) // if busses have to be inited & saved, wait until next iteration
     reset();
   if (doCloseFile) {
     closeFile();
diff --git a/wled00/wled.h b/wled00/wled.h
index 468c7083c..db0593f9a 100644
--- a/wled00/wled.h
+++ b/wled00/wled.h
@@ -8,7 +8,7 @@
  */
 
 // version code in format yymmddb (b = daily build)
-#define VERSION 2203160
+#define VERSION 2203190
 
 //uncomment this if you have a "my_config.h" file you'd like to use
 //#define WLED_USE_MY_CONFIG