Tidying

src/DebugToolDrivers/Microchip/Protocols/EDBG/AVR/EdbgAvr8Interface.cpp

@@ -96,7 +96,7 @@ namespace DebugToolDrivers::Microchip::Protocols::Edbg::Avr
         const Targets::Microchip::Avr8::Avr8TargetConfig& targetConfig
     )
         : edbgInterface(edbgInterface)
-        , session(EdbgAvr8Session(targetDescriptionFile, targetConfig))
+        , session(EdbgAvr8Session{targetDescriptionFile, targetConfig})
     {}
 
     void EdbgAvr8Interface::init() {

src/Targets/RiscV/Wch/WchRiscV.cpp

@@ -52,6 +52,12 @@ namespace Targets::RiscV::Wch
             )
         )
     {
+        if (this->targetConfig.physicalInterface != TargetPhysicalInterface::SDI) {
+            throw Exceptions::InvalidConfig{
+                "Invalid physical interface selection - WCH RISC-V targets only support the SDI interface"
+            };
+        }
+
         if (
             this->targetConfig.programSegmentKey.has_value()
             && *(this->targetConfig.programSegmentKey) != this->bootProgramSegmentDescriptor.key

src/Targets/TargetDescriptionFiles/TargetDescriptionFiles.cmake

@@ -264,6 +264,8 @@ list(
         ${CMAKE_CURRENT_SOURCE_DIR}/src/Targets/TargetDescriptionFiles/AVR8/MEGA/ATMEGA3290PA.xml
         ${CMAKE_CURRENT_SOURCE_DIR}/src/Targets/TargetDescriptionFiles/AVR8/MEGA/ATMEGA328PB.xml
         ${CMAKE_CURRENT_SOURCE_DIR}/src/Targets/TargetDescriptionFiles/AVR8/MEGA/ATMEGA809.xml
+        ${CMAKE_CURRENT_SOURCE_DIR}/src/Targets/TargetDescriptionFiles/RISC-V/WCH/CH32X035.xml
+        ${CMAKE_CURRENT_SOURCE_DIR}/src/Targets/TargetDescriptionFiles/RISC-V/WCH/CH32V003.xml
 )
 
 # Because the TDF_FILES_LIST is only used to specify dependencies for some custom commands, there is nothing enforcing

src/Targets/TargetMemoryCache.hpp

@@ -16,6 +16,8 @@ namespace Targets
         /**
          * Fetches data from the cache.
          *
+         * TODO: Change return type to TargetMemoryBufferSpan
+         *
          * @param startAddress
          * @param bytes
          *