GD32: Update MFL Platform and Arduino Core

Marlin/src/HAL/GD32_MFL/HAL.h

@@ -105,7 +105,7 @@ extern "C" char* dtostrf(double val, signed char width, unsigned char prec, char
 class MarlinHAL {
 public:
   // Before setup()
-  MarlinHAL() {}
+  MarlinHAL() = default;
 
   // Watchdog
   static void watchdog_init()    IF_DISABLED(USE_WATCHDOG, {});

Marlin/src/HAL/GD32_MFL/MarlinSerial.cpp

@@ -33,8 +33,8 @@
 
 using namespace arduino;
 
-MarlinSerial& MarlinSerial::get_instance(usart::USART_Base Base, pin_size_t rxPin, pin_size_t txPin) {
-  UsartSerial& serial = UsartSerial::get_instance(Base, rxPin, txPin);
+auto MarlinSerial::get_instance(usart::USART_Base Base, pin_size_t rxPin, pin_size_t txPin) -> MarlinSerial& {
+  auto& serial = UsartSerial::get_instance(Base, rxPin, txPin);
   return *reinterpret_cast<MarlinSerial*>(&serial);
 }
 
@@ -61,7 +61,7 @@ MarlinSerial& MarlinSerial::get_instance(usart::USART_Base Base, pin_size_t rxPi
 
   static void emergency_callback() {
     if (!current_serial_instance) return;
-    const uint8_t last_data = current_serial_instance->get_last_data();
+    const auto last_data = current_serial_instance->get_last_data();
     emergency_parser.update(current_serial_instance->emergency_state, last_data);
   }
 

Marlin/src/HAL/GD32_MFL/MarlinSerial.h

@@ -43,7 +43,7 @@
 using namespace arduino;
 
 struct MarlinSerial : public UsartSerial {
-  static MarlinSerial& get_instance(usart::USART_Base Base, pin_size_t rxPin = NO_PIN, pin_size_t txPin = NO_PIN);
+  static auto get_instance(usart::USART_Base Base, pin_size_t rxPin = NO_PIN, pin_size_t txPin = NO_PIN) -> MarlinSerial&;
 
   void begin(unsigned long baudrate, uint16_t config);
   inline void begin(unsigned long baudrate) { begin(baudrate, SERIAL_8N1); }
@@ -57,7 +57,7 @@ struct MarlinSerial : public UsartSerial {
     EmergencyParser::State emergency_state;
 
     // Accessor method to get the last received byte
-    uint8_t get_last_data() { return usart_.get_last_data(); }
+    auto get_last_data() -> uint8_t { return usart_.get_last_data(); }
 
     // Register the emergency callback
     void register_emergency_callback(void (*callback)());

Marlin/src/HAL/GD32_MFL/fastio.h

@@ -21,26 +21,27 @@
  */
 #pragma once
 
-// Fast I/O interfaces for GD32F303RE
+// Fast I/O interfaces for GD32
 
 #include <GPIO.hpp>
+#include <variant.h>
 #include <PinOps.hpp>
 #include <PinOpsMap.hpp>
 
 template<typename T>
 static inline void fast_write_pin_wrapper(pin_size_t IO, T V) {
-  auto port = getPortFromPin(IO);
-  auto pin = getPinInPort(IO);
-  if (static_cast<bool>(V)) gpio::fast_set_pin(port, pin);
-  else                      gpio::fast_clear_pin(port, pin);
+  const PortPinPair& pp = port_pin_map[IO];
+  gpio::fast_write_pin(pp.port, pp.pin, static_cast<bool>(V));
 }
 
-static inline bool fast_read_pin_wrapper(pin_size_t IO) {
-  return gpio::fast_read_pin(getPortFromPin(IO), getPinInPort(IO));
+static inline auto fast_read_pin_wrapper(pin_size_t IO) -> bool {
+  const PortPinPair& pp = port_pin_map[IO];
+  return gpio::fast_read_pin(pp.port, pp.pin);
 }
 
 static inline void fast_toggle_pin_wrapper(pin_size_t IO) {
-  gpio::fast_toggle_pin(getPortFromPin(IO), getPinInPort(IO));
+  const PortPinPair& pp = port_pin_map[IO];
+  gpio::fast_toggle_pin(pp.port, pp.pin);
 }
 
 // ------------------------

Marlin/src/HAL/GD32_MFL/pinsDebug.h

@@ -70,8 +70,9 @@ bool isAnalogPin(const pin_t pin) {
   if (!isValidPin(pin)) return false;
 
   if (getAdcChannel(pin) != adc::ADC_Channel::INVALID) {
-    auto& instance = gpio::GPIO::get_instance(getPortFromPin(pin)).value();
-    return instance.get_pin_mode(getPinInPort(pin)) == gpio::Pin_Mode::ANALOG && !M43_NEVER_TOUCH(pin);
+    const PortPinPair& pp = port_pin_map[pin];
+    auto& instance = gpio::GPIO::get_instance(pp.port).value();
+    return instance.get_pin_mode(pp.pin) == gpio::Pin_Mode::ANALOG && !M43_NEVER_TOUCH(pin);
   }
 
   return false;
@@ -80,8 +81,9 @@ bool isAnalogPin(const pin_t pin) {
 bool getValidPinMode(const pin_t pin) {
   if (!isValidPin(pin)) return false;
 
-  auto& instance = gpio::GPIO::get_instance(getPortFromPin(pin)).value();
-  gpio::Pin_Mode mode = instance.get_pin_mode(getPinInPort(pin));
+  const PortPinPair& pp = port_pin_map[pin];
+  auto& instance = gpio::GPIO::get_instance(pp.port).value();
+  gpio::Pin_Mode mode = instance.get_pin_mode(pp.pin);
 
   return mode != gpio::Pin_Mode::ANALOG && mode != gpio::Pin_Mode::INPUT_FLOATING &&
          mode != gpio::Pin_Mode::INPUT_PULLUP && mode != gpio::Pin_Mode::INPUT_PULLDOWN;