Fix console output newlines and update configs

Append missing trailing newlines to various console::Error/console messages across network, timer, DHCP/IGMP and TCP/UDP code to improve log formatting. Add console::Write declaration and use it in perror, include <cstdint> where needed, and add IWYU pragmas in hal.h. Update copyright years, increase network memory blocks from 8 to 12, raise TCP_MAX_TCBS_ALLOWED from 16 to 32, and normalize HOST_NAME_PREFIX strings (use hyphen). Also simplify ENABLE_USB_HOST conditionals and adjust related includes. Small allocator and error message tweaks included to make diagnostics clearer.

Author: vanvught

lib-clib/src/malloc.cpp

@@ -150,7 +150,7 @@ extern "C"
 
             if (next > block_limit)
             {
-                console::Error("malloc: out of memory");
+                console::Error("malloc: out of memory\n");
 #ifdef DEBUG_HEAP
                 debug_heap();
 #endif

lib-clib/src/perror.cpp

@@ -2,7 +2,7 @@
  * @file perror.cpp
  *
  */
-/* Copyright (C) 2020-2025 by Arjan van Vught mailto:info@gd32-dmx.org
+/* Copyright (C) 2020-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
@@ -31,6 +31,7 @@ namespace console
 void Error(const char*);
 int Putc(int);
 int Puts(const char*);
+void Write(const char*, unsigned int);
 } // namespace console
 
 /*
@@ -107,7 +108,7 @@ extern "C"
         if (s && *s)
         {
             console::Error(s);
-            console::Puts(": ");
+            console::Write(": ", 2);
         }
 
         console::Error(ptr);

lib-hal/console/uart0/console.cpp

@@ -23,6 +23,7 @@
  * THE SOFTWARE.
  */
 
+#include <cstdint>
 #include <cstring>
 
 #include "console.h"

lib-hal/include/gd32/hal.h

@@ -2,7 +2,7 @@
  * @file hal.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
@@ -28,17 +28,13 @@
 
 #include <cstdint>
 
-#include "gd32.h"
+#include "gd32.h" // IWYU pragma: keep
 
-#if defined(ENABLE_USB_HOST) && defined(CONFIG_USB_HOST_MSC)
+#if defined(ENABLE_USB_HOST)
 extern "C"
 {
 #include "usbh_core.h"
-#if defined(GD32H7XX) || defined(GD32F4XX)
-    extern usbh_host usb_host_msc;
-#else
     extern usbh_host usb_host;
-#endif
 }
 #endif
 
@@ -54,10 +50,10 @@ void emac_debug_run();
 #include "task.h"
 #endif
 
-#include "softwaretimers.h"
+#include "softwaretimers.h" // IWYU pragma: keep
 
 #if !defined(DISABLE_RTC)
-#include "hwclock.h"
+#include "hwclock.h" // IWYU pragma: keep
 #endif
 
 #include "hal_panelled.h"
@@ -74,13 +70,9 @@ inline constexpr float kCoreTemperatureMax = +85.0;
 
 inline void Run()
 {
-#if defined(ENABLE_USB_HOST) && defined(CONFIG_USB_HOST_MSC)
-#if defined(GD32H7XX) || defined(GD32F4XX)
-    usbh_core_task(&usb_host_msc);
-#else
+#if defined(ENABLE_USB_HOST)
     usbh_core_task(&usb_host);
 #endif
-#endif
 #if !defined(USE_FREE_RTOS)
     SoftwareTimerRun();
 #endif

lib-hal/superloop/softwaretimers.cpp

@@ -77,7 +77,7 @@ TimerHandle_t SoftwareTimerAdd(uint32_t interval_millis, const TimerCallbackFunc
     if (s_timers_count >= hal::kSoftwareTimersMax)
     {
 #ifndef NDEBUG
-        console::Error("SoftwareTimerAdd: Max timer limit reached");
+        console::Error("SoftwareTimerAdd: Max timer limit reached\n");
 #endif
         return -1;
     }
@@ -134,7 +134,7 @@ bool SoftwareTimerDelete(TimerHandle_t& id)
     }
 
 #ifndef NDEBUG
-    console::Error("SoftwareTimerDelete: Timer not found");
+    console::Error("SoftwareTimerDelete: Timer not found\n");
 #endif
 
     DEBUG_EXIT();
@@ -163,7 +163,7 @@ bool SoftwareTimerChange(TimerHandle_t id, uint32_t interval_millis)
     }
 
 #ifndef NDEBUG
-    console::Error("SoftwareTimerChange: Timer not found");
+    console::Error("SoftwareTimerChange: Timer not found\n");
 #endif
 
     return false;

lib-network/config/net_config.h

@@ -2,7 +2,7 @@
  * @file net_config.h
  *
  */
-/* Copyright (C) 2021-2024 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
@@ -26,16 +26,20 @@
 #ifndef NET_CONFIG_H_
 #define NET_CONFIG_H_
 
+// Valid hostnames (as per RFC 1123) must consist only of ASCII letters (a-z, A-Z), digits (0-9), and hyphens (-). 
+// Labels must be 1-63 characters, with a maximum total length of 253 characters. 
+// They cannot start or end with a hyphen, and should not be all-numeric. 
+
 #if defined(__linux__) || defined (__APPLE__)
 # define UDP_MAX_PORTS_ALLOWED			32
 # define IGMP_MAX_JOINS_ALLOWED			(4 + (8 * 4)) /* 8 outputs x 4 Universes */
-# define TCP_MAX_TCBS_ALLOWED			16
+# define TCP_MAX_TCBS_ALLOWED			32
 # define TCP_MAX_PORTS_ALLOWED			2
 #else
 # define TCP_MAX_PORTS_ALLOWED			1
 # if defined (H3)
 #  if !defined(HOST_NAME_PREFIX)
-#   define HOST_NAME_PREFIX				"allwinner_"
+#   define HOST_NAME_PREFIX				"allwinner-"
 #  endif
 #  define UDP_MAX_PORTS_ALLOWED			16
 #  define IGMP_MAX_JOINS_ALLOWED		(4 + (8 * 4)) /* 8 outputs x 4 Universes */
@@ -46,7 +50,7 @@
  */
 #  define CHECKSUM_BY_HARDWARE
 #  if !defined(HOST_NAME_PREFIX)
-#   define HOST_NAME_PREFIX				"gigadevice_"
+#   define HOST_NAME_PREFIX				"gigadevice-"
 #  endif
 #  if !defined (UDP_MAX_PORTS_ALLOWED)
 #   define UDP_MAX_PORTS_ALLOWED		8

lib-network/src/core/ipv4/dhcp.cpp

@@ -250,7 +250,7 @@ void Inform()
 #ifndef NDEBUG
     if (kHandle < 0)
     {
-        console::Error("DHCP Inform");
+        console::Error("DHCP Inform\n");
         return;
     }
 #endif
@@ -766,7 +766,7 @@ bool Start()
 #ifndef NDEBUG
     if (dhcp->handle < 0)
     {
-        console::Error("DHCP Start");
+        console::Error("DHCP Start\n");
         DEBUG_EXIT();
         return false;
     }

lib-network/src/core/ipv4/igmp.cpp

@@ -378,7 +378,7 @@ void static Join(uint32_t group_address)
     }
 
 #ifndef NDEBUG
-    console::Error("igmp::Join");
+    console::Error("igmp::Join\n");
 #endif
     DEBUG_ENTRY();
 }

lib-network/src/core/network_memory.h

@@ -39,7 +39,7 @@ namespace network::memory
 {
 inline constexpr uint32_t kBlocks =
 #if !defined(CONFIG_NETWORK_MEMORY_BLOCKS)
-    8;
+    12;
 #else
     CONFIG_NETWORK_MEMORY_BLOCKS;
 #endif
@@ -93,7 +93,7 @@ class Allocator
     {
         if (IsFull())
         {
-            console::Error("Full!");
+            console::Error("Allocate:Full!\n");
             return nullptr;
         }
 
@@ -113,7 +113,7 @@ class Allocator
 
         if (IsFull())
         {
-            console::Error("Full!");
+            console::Error("Allocate:Full!\n");
             return UINT16_MAX;
         }
 

lib-network/src/core/tcp.cpp

@@ -1664,7 +1664,7 @@ int32_t Connect(uint32_t remote_ip, uint16_t remote_port, CallbackConnect cb_con
     const auto& netif = netif::global::netif_default;
     if (__builtin_expect((netif.ip.addr == 0), 0))
     {
-        console::Error("Connect: No ip!");
+        console::Error("Connect: No ip!\n");
         return -1;
     }
 
@@ -1673,7 +1673,7 @@ int32_t Connect(uint32_t remote_ip, uint16_t remote_port, CallbackConnect cb_con
     auto* tcb = AllocTcb(remote_port, &out_index);
     if (tcb == nullptr)
     {
-        console::Error("Connect: No TCB!");
+        console::Error("Connect: No TCB!\n");
         return -2;
     }
 
@@ -1714,15 +1714,15 @@ int32_t Close(ConnHandle conn_handle) // graceful FIN
 {
     if (conn_handle >= TCP_MAX_TCBS_ALLOWED)
     {
-        console::Error("Close: Connection handle!");
+        console::Error("Close: Connection handle!\n");
         return -1;
     }
 
     auto* c = &s_tcbs[conn_handle];
 
     if (!c->in_use || c->state == kStateClosed)
     {
-        console::Error("Close: TCB!");
+        console::Error("Close: TCB!\n");
         return -1;
     }
 
@@ -1743,7 +1743,7 @@ int32_t Close(ConnHandle conn_handle) // graceful FIN
     // We only support graceful close from states where FIN makes sense here.
     if (c->state != kStateEstablished && c->state != kStateCloseWait)
     {
-        console::Error("Close: Not graceful!");
+        console::Error("Close: Not graceful!\n");
         return -1;
     }