gd32: standardize hwClock naming & error handling

Cleanup and small fixes in GD32 HAL and EMAC code: add missing <cstddef> include and remove duplicate, rename HwClock instance to hw_clock for consistent naming, add CONFIG_CLIB_USE_NULL branch to skip console initialization, and switch EMAC error reporting to network::Error (include network_private.h and remove redundant console::Error declaration). These changes centralize error handling and tidy up includes/initialization.

Author: vanvught

lib-hal/src/gd32/hal_init.cpp

@@ -23,6 +23,8 @@
  * THE SOFTWARE.
  */
 
+#include <cstddef>
+
 #if !defined(_TIME_STAMP_DAY_)
 #define _TIME_STAMP_DAY_ 0
 #endif
@@ -33,7 +35,6 @@
 #define _TIME_STAMP_YEAR_ (2026 - 1900)
 #endif
 
-#include <cstddef>
 #if defined(DEBUG_HAL)
 #undef NDEBUG
 #endif
@@ -97,7 +98,7 @@ void Timer7Config();
 
 #if !defined(DISABLE_RTC)
 #include "hwclock.h"
-static HwClock hwClock;
+static HwClock hw_clock;
 #endif
 
 extern unsigned char _sdmx;      // NOLINT
@@ -122,6 +123,7 @@ void Init() {
 
 #if defined(CONFIG_CLIB_USE_UART0)
     uart0::Init();
+#elif defined(CONFIG_CLIB_USE_NULL)
 #else
     console::Init();
 #endif

lib-network/src/emac/gd32/emac.cpp

@@ -40,10 +40,7 @@
 #endif
 #include "firmware/debug/debug_debug.h"
 #include "gd32.h" // IWYU pragma: keep
-
-namespace console {
-void Error(const char*);
-}
+#include "../src/core/network_private.h"
 
 extern void EnetGpioConfig();
 extern enet_descriptors_struct txdesc_tab[ENET_TXBUF_NUM];
@@ -86,7 +83,7 @@ void __attribute__((cold)) Config() {
     enet_software_reset(ENETx);
 
     if (!emac::phy::Config(PHY_ADDRESS)) {
-        console::Error("emac::phy::Config(PHY_ADDRESS)\n");
+        network::Error(__func__, "emac::phy::Config(PHY_ADDRESS)");
     }
 
     DEBUG_EXIT();