Refactor pixel types & config; add build flags

Rename and consolidate pixel type/map enums and metadata into a compact, constexpr TypeInfo layout in pixeltype.h (Type->LedType, Map->LedMap, etc.), remove the old pixeltype.cpp, and add helper functions (GetTypeInfo, GetTypeName, GetMapName, GetTxH, GetTypeByName/GetMapByName). Update PixelConfiguration to use the new TypeInfo (led count, protocol, default/max Hz, maps), improve validation logic, and propagate enum/name changes across pixel output, patterns, test pattern, JSON status, and other pixel-related files. Add common::ToValue/FromValue constexpr helpers with static_assert checks. Apply IWYU pragma comments, tighten constexpr/formatting, bump copyright years to 2026 where applicable, and tidy minor warnings (pragmas, spacing). Also add CONFIG_HTTPD_OPTIMIZE_O2 define to several BOARD entries in common Makefile and adjust some remoteconfig/http headers and generated content includes. These changes centralize pixel metadata, simplify runtime logic, and standardize naming/API across the codebase.

Author: vanvught

common/include/common/utils/utils_enum.h

@@ -2,7 +2,7 @@
  * @file utils_enum.h
  *
  */
-/* Copyright (C) 2025 by Arjan van Vught mailto:info@gd32-dmx.org
+/* Copyright (C) 2025-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
@@ -30,19 +30,20 @@
 
 namespace common
 {
-
 /// Converts an enum class value to its underlying integer type.
-template <typename Enum>
-inline auto ToValue(Enum e) noexcept {
+template <typename Enum> constexpr auto ToValue(Enum e) noexcept -> std::underlying_type_t<Enum>
+{
+    static_assert(std::is_enum_v<Enum>);
     return static_cast<std::underlying_type_t<Enum>>(e);
 }
 
 /// Converts an integer value to the corresponding enum class value.
-template <typename Enum>
-inline Enum FromValue(std::underlying_type_t<Enum> value) noexcept {
+template <typename Enum> constexpr Enum FromValue(std::underlying_type_t<Enum> value) noexcept
+{
+    static_assert(std::is_enum_v<Enum>);
     return static_cast<Enum>(value);
 }
 
 } // namespace common
 
-#endif  // COMMON_UTILS_UTILS_ENUM_H_
+#endif // COMMON_UTILS_UTILS_ENUM_H_

common/include/firmware/pixeldmx/show.h

@@ -2,7 +2,7 @@
  * display.h
  *
  */
-/* Copyright (C) 2025 by Arjan van Vught mailto:info@gd32-dmx.org
+/* Copyright (C) 2025-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
@@ -41,8 +41,8 @@ inline void Show(uint32_t line, pixelpatterns::Pattern pattern = pixelpatterns::
     assert(display != nullptr);
 
     display->ClearEndOfLine();
-    display->Printf(line, "%s:%d G%d %s", pixel::GetType(configuration.GetType()), configuration.GetCount(), configuration.GetGroupingCount(),
-                    pixel::GetMap(configuration.GetMap()));
+    display->Printf(line, "%s:%d G%d %s", pixel::GetTypeName(configuration.GetType()), configuration.GetCount(), configuration.GetGroupingCount(),
+                    pixel::GetMapName(configuration.GetMap()));
     display->ClearLine(8); // Status line
 
     if (pattern != pixelpatterns::Pattern::kNone)

common/make/gd32/Board.mk

@@ -22,12 +22,7 @@ endif
 ifeq ($(strip $(BOARD)),BOARD_GD32F207RG)
 	MCU=GD32F207RG
 	DEFINES+=-DCONFIG_STORE_USE_SPI
-endif
-
-ifeq ($(strip $(BOARD)),BOARD_GD32F303CB)
-	MCU=GD32F303CB
-	DEFINES+=-DCONFIG_STORE_USE_ROM
-	DEFINES+=-DNO_EMAC
+	DEFINES+=-DCONFIG_HTTPD_OPTIMIZE_O2
 endif
 
 ifeq ($(strip $(BOARD)),BOARD_GD32F303RC)
@@ -39,18 +34,22 @@ endif
 ifeq ($(strip $(BOARD)),BOARD_GD32F407RE)
 	MCU=GD32F407RE
 	DEFINES+=-DCONFIG_STORE_USE_SPI
+	DEFINES+=-DCONFIG_HTTPD_OPTIMIZE_O2
 endif
 
 ifeq ($(strip $(BOARD)),BOARD_GD32F450VI)
 	MCU=GD32F450VI
+	DEFINES+=-DCONFIG_HTTPD_OPTIMIZE_O2
 endif
 
 ifeq ($(strip $(BOARD)),BOARD_16X4U_PIXEL)
 	MCU=GD32F450VI
+	DEFINES+=-DCONFIG_HTTPD_OPTIMIZE_O2
 endif
 
 ifeq ($(strip $(BOARD)),BOARD_GD32F470VG)
 	MCU=GD32F470VG
+	DEFINES+=-DCONFIG_HTTPD_OPTIMIZE_O2
 endif
 
 ifeq ($(strip $(BOARD)),BOARD_GD32F207C_EVAL)
@@ -71,16 +70,19 @@ endif
 ifeq ($(strip $(BOARD)),BOARD_BW_OPIDMX4)
 	BOARD_DMX=4
 	DEFINES+=-DCONFIG_STORE_USE_SPI
+	DEFINES+=-DCONFIG_HTTPD_OPTIMIZE_O2
 endif
 
 ifeq ($(strip $(BOARD)),BOARD_DMX3)
 	BOARD_DMX=3
 	DEFINES+=-DCONFIG_STORE_USE_SPI
+	DEFINES+=-DCONFIG_HTTPD_OPTIMIZE_O2
 endif
 
 ifeq ($(strip $(BOARD)),BOARD_DMX4)
-	DEFINES+=-DCONFIG_STORE_USE_SPI
 	BOARD_DMX=4
+	DEFINES+=-DCONFIG_STORE_USE_SPI
+	DEFINES+=-DCONFIG_HTTPD_OPTIMIZE_O2
 endif
 
 ifdef BOARD_DMX

lib-network/src/core/tcp.cpp

@@ -46,7 +46,6 @@
 #if defined(DEBUG_NET_TCP)
 #undef NDEBUG
 #endif
-
 #pragma GCC diagnostic push
 #if (__GNUC__ < 10)
 #pragma GCC diagnostic ignored "-Wconversion"

lib-pixel/include/gd32/gpio/pixeloutputmulti.h

@@ -2,7 +2,7 @@
  * @file pixeloutputmulti.h
  *
  */
-/* Copyright (C) 2021-2025 by Arjan van Vught mailto:info@gd32-dmx.org
+/* Copyright (C) 2021-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
@@ -40,8 +40,8 @@
 #include <cstdint>
 
 #include "pixeltype.h"
-#include "gd32/gpio/pixeloutputmulti_config.h"
-#include "gd32.h"
+#include "gd32/gpio/pixeloutputmulti_config.h" // IWYU pragma: keep
+#include "gd32.h"                              // IWYU pragma: keep
 
 class PixelOutputMulti
 {

lib-pixel/include/pixel.h

@@ -80,16 +80,16 @@ inline void SetPixelColour([[maybe_unused]] uint32_t port_index, uint32_t pixel_
 #if defined(PIXELPATTERNS_MULTI)
     switch (PixelConfiguration::Get().GetType())
     {
-        case pixel::Type::WS2801:
+        case pixel::LedType::kWS2801:
             output_type->SetColourWS2801(port_index, pixel_index, kColours.Red(), kColours.Green(), kColours.Blue());
             break;
 
-        case pixel::Type::APA102:
-        case pixel::Type::SK9822:
+        case pixel::LedType::kAPA102:
+        case pixel::LedType::kSK9822:
             output_type->SetPixel4Bytes(port_index, pixel_index, 0xFF, kColours.Red(), kColours.Green(), kColours.Blue());
             break;
 
-        case pixel::Type::P9813:
+        case pixel::LedType::kP9813:
         {
             const auto kRed = kColours.Red();
             const auto kBlue = kColours.Blue();
@@ -98,7 +98,7 @@ inline void SetPixelColour([[maybe_unused]] uint32_t port_index, uint32_t pixel_
             break;
         }
 
-        case pixel::Type::SK6812W:
+        case pixel::LedType::kSK6812W:
             output_type->SetColourRTZ(port_index, pixel_index, kColours.Red(), kColours.Green(), kColours.Blue(), kColours.White());
             break;
 
@@ -110,7 +110,7 @@ inline void SetPixelColour([[maybe_unused]] uint32_t port_index, uint32_t pixel_
     auto& pixel_configuration = PixelConfiguration::Get();
     const auto kType = pixel_configuration.GetType();
 
-    if (kType != pixel::Type::SK6812W)
+    if (kType != pixel::LedType::kSK6812W)
     {
         output_type->SetPixel(pixel_index, kColours.Red(), kColours.Green(), kColours.Blue());
     }