Add RDM SetFactoryDefaults and ConfigStore reset

Introduce a weak rdm::device::SetFactoryDefaults implementation (new lib-rdm/src/rdm_device.cpp) and expose its declaration in rdmdevice.h. Update RDMDeviceResponder to call rdm::device::SetFactoryDefaults and remove the old configstore forward declaration. Add ConfigStore::Reset() to zero stored data and mark status changed, and use it in RemoteConfig::HandleFactory to reset config and trigger reboot. Misc: switch to network_udp include, minor formatting/whitespace changes, and bump copyright year in remoteconfig.

Author: vanvught

lib-configstore/include/configstore.h

@@ -55,7 +55,16 @@ class ConfigStore : StoreDevice
         kWriting
     };
 
-    [[maybe_unused]] static constexpr char kStateNames[7][16] = {"IDLE", "CHANGED", "CHANGED_WAITING", "ERASING", "ERASED", "ERASED_WAITING", "WRITING"};
+    [[maybe_unused]] static constexpr char kStateNames[7][16] = 
+	{
+		"IDLE", 
+		"CHANGED", 
+		"CHANGED_WAITING", 
+		"ERASING", 
+		"ERASED", 
+		"ERASED_WAITING", 
+		"WRITING"
+	};
 
    public:
     ConfigStore()
@@ -118,6 +127,12 @@ class ConfigStore : StoreDevice
     ConfigStore& operator=(ConfigStore&&) = delete;
 
     ~ConfigStore() = default;
+	
+	void Reset()
+	{
+		memset(s_store, 0, sizeof(s_store));
+		SetStatusChanged();	
+	}
 
     bool Commit() { return Flash(); }
 

lib-dmxnode/include/dmxnode_outputtype.h

@@ -48,8 +48,23 @@ enum class OutputType
     kUndefined
 };
 
-inline constexpr const char* kOutputTypeNames[static_cast<uint32_t>(OutputType::kUndefined)] = {
-    "DMX", "DMX/RDM", "Pixel", "Pixel/DMX", "PWM", "RGB Panel", "Serial", "OSC", "Monitor", "Stepper", "Player", "Art-Net", "Timecode", "None"};
+inline constexpr const char* kOutputTypeNames[static_cast<uint32_t>(OutputType::kUndefined)] = 
+{
+    "DMX", 
+	"DMX/RDM", 
+	"Pixel", 
+	"Pixel/DMX", 
+	"PWM", 
+	"RGB Panel", 
+	"Serial", 
+	"OSC", 
+	"Monitor", 
+	"Stepper", 
+	"Player", 
+	"Art-Net", 
+	"Timecode", 
+	"None"
+};
 
 inline const char* GetOutputType(OutputType type)
 {

lib-rdm/include/rdmdevice.h

@@ -46,6 +46,7 @@ uint32_t BootSoftwareVersionId();
 uint32_t SoftwareVersionId();
 const char* SoftwareVersionLabel(uint32_t& length);
 const char* RootLabel(uint8_t& length);
+void SetFactoryDefaults();
 
 class Device
 {

lib-rdm/include/rdmdeviceresponder.h

@@ -40,11 +40,6 @@
 #include "dmxnode.h"
 #include "dmxnode_outputtype.h"
 
-namespace configstore
-{
-void SetFactoryDefaults();
-} // namespace configstore
-
 class RDMDeviceResponder
 {
     static constexpr char kLanguage[2] = {'e', 'n'};
@@ -192,7 +187,7 @@ class RDMDeviceResponder
         checksum_ = CalculateChecksum();
         is_factory_defaults_ = true;
 
-        configstore::SetFactoryDefaults();
+        rdm::device::SetFactoryDefaults();
 
         DEBUG_EXIT();
     }

lib-rdm/src/rdm_device.cpp

@@ -0,0 +1,35 @@
+/**
+ * @file rdm_device.cpp
+ *
+ */
+/* Copyright (C) 2026 by Arjan van Vught mailto:info@gd32-dmx.org
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+ * THE SOFTWARE.
+ */
+
+#include "rdmdevice.h"
+#include "firmware/debug/debug_debug.h"
+
+namespace rdm::device
+{
+__attribute__((weak)) void SetFactoryDefaults()
+{
+    DEBUG_PUTS("Not implemented.");
+}
+} // namespace rdm::device

lib-remoteconfig/src/remoteconfig.cpp

@@ -2,7 +2,7 @@
  * @file remoteconfig.cpp
  *
  */
-/* Copyright (C) 2019-2025 by Arjan van Vught mailto:info@gd32-dmx.org
+/* Copyright (C) 2019-2026 by Arjan van Vught mailto:info@gd32-dmx.org
  *
  * Permission is hereby granted, free of charge, to any person obtaining a copy
  * of this software and associated documentation files (the "Software"), to deal
@@ -35,7 +35,7 @@
 #include "remoteconfig.h"
 #include "firmwareversion.h"
 #include "hal.h"
-#include "network.h"
+#include "network_udp.h"
 #if !defined(CONFIG_REMOTECONFIG_MINIMUM)
 #include "apps/mdns.h"
 #include "dmxnode_nodetype.h"
@@ -44,7 +44,7 @@
 #include "display.h"
 #include "configstore.h"
 #include "firmware/debug/debug_dump.h"
- #include "firmware/debug/debug_debug.h"
+#include "firmware/debug/debug_debug.h"
 
 namespace remoteconfig::udp
 {
@@ -91,11 +91,24 @@ constexpr struct RemoteConfig::Commands RemoteConfig::kSet[] = {
     {&RemoteConfig::HandleDisplaySet, "display#", 8, true} //
 };
 
-static constexpr char kOutput[static_cast<uint32_t>(remoteconfig::Output::LAST)][12] = {"DMX",     "RDM",    "Monitor", "Pixel",  "TimeCode",  "OSC", "Config",
-                                                                                        "Stepper", "Player", "Art-Net", "Serial", "RGB Panel", "PWM"};
+static constexpr char kOutput[static_cast<uint32_t>(remoteconfig::Output::LAST)][12] = 
+{
+	"DMX",     
+	"RDM",    
+	"Monitor", 
+	"Pixel",  
+	"TimeCode",  
+	"OSC", 
+	"Config",
+	"Stepper", 
+	"Player", 
+	"Art-Net", 
+	"Serial", 
+	"RGB Panel", 
+	"PWM"
+};
 
-RemoteConfig::RemoteConfig(remoteconfig::Output output, uint32_t active_outputs)
-    : output_(output), active_outputs_(active_outputs)
+RemoteConfig::RemoteConfig(remoteconfig::Output output, uint32_t active_outputs) : output_(output), active_outputs_(active_outputs)
 {
     DEBUG_ENTRY();
 
@@ -104,7 +117,7 @@ RemoteConfig::RemoteConfig(remoteconfig::Output output, uint32_t active_outputs)
     assert(s_this == nullptr);
     s_this = this;
 
-     network::iface::CopyMacAddressTo(s_list.mac_address);
+    network::iface::CopyMacAddressTo(s_list.mac_address);
     s_list.output = static_cast<uint8_t>(output);
     s_list.active_outputs = static_cast<uint8_t>(active_outputs);
 
@@ -143,7 +156,6 @@ RemoteConfig::~RemoteConfig()
         delete http_daemon_;
     }
 #endif
-
     network::apps::mdns::ServiceRecordDelete(network::apps::mdns::Services::kConfig);
 #endif
 
@@ -156,31 +168,31 @@ RemoteConfig::~RemoteConfig()
 void RemoteConfig::SetDisplayName(const char* display_name)
 {
     DEBUG_ENTRY();
-    
+
     char array[common::store::remoteconfig::kDisplayNameLength];
 
-	size_t len = strlen(display_name);
+    size_t len = strlen(display_name);
     len = len > (common::store::remoteconfig::kDisplayNameLength - 1) ? common::store::remoteconfig::kDisplayNameLength - 1 : len;
     memcpy(reinterpret_cast<char*>(array), display_name, len);
 
     for (uint32_t i = len; i < common::store::remoteconfig::kDisplayNameLength; i++)
     {
         array[i] = '\0';
     }
-    
-     ConfigStore::Instance().RemoteConfigUpdateArray(&common::store::RemoteConfig::display_name, array, common::store::remoteconfig::kDisplayNameLength);
 
-    DEBUG_EXIT();
-}
+    ConfigStore::Instance().RemoteConfigUpdateArray(&common::store::RemoteConfig::display_name, array, common::store::remoteconfig::kDisplayNameLength);
 
-namespace configstore
-{
-void SetFactoryDefaults();
+    DEBUG_EXIT();
 }
 
 void RemoteConfig::HandleFactory()
 {
-    configstore::SetFactoryDefaults();
+     DEBUG_ENTRY();
+	 
+	 ConfigStore::Instance().Reset();
+	 HandleReboot();
+	 
+	 DEBUG_EXIT();
 }
 
 void RemoteConfig::Input(const uint8_t* buffer, uint32_t size, uint32_t from_ip, [[maybe_unused]] uint16_t from_port)
@@ -301,7 +313,7 @@ void RemoteConfig::HandleList()
 #if !defined(CONFIG_REMOTECONFIG_MINIMUM)
     const auto* const kNodeTypeName = dmxnode::GetNodeType(dmxnode::kNodeType);
 #else
-	const auto* const kNodeTypeName = "Bootloader TFTP";
+    const auto* const kNodeTypeName = "Bootloader TFTP";
 #endif
 
     if (display_name[0] != '\0')