diff --git a/src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/EdbgAvr8Interface.cpp b/src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/EdbgAvr8Interface.cpp index 7ddd9d5f..b2484750 100644 --- a/src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/EdbgAvr8Interface.cpp +++ b/src/DebugToolDrivers/Protocols/CMSIS-DAP/VendorSpecific/EDBG/AVR/EdbgAvr8Interface.cpp @@ -35,6 +35,7 @@ using namespace Bloom::DebugToolDrivers::Protocols::CmsisDap::Edbg::Avr; using namespace Bloom::Exceptions; + using Bloom::Targets::TargetState; using Bloom::Targets::TargetRegister; using Bloom::Targets::TargetRegisters; @@ -327,7 +328,7 @@ std::unique_ptr EdbgAvr8Interface::getAvrEvent() { default: { /* * TODO: This isn't very nice as we're performing an unnecessary copy. Maybe requestAvrEvents should - * return a unique_ptr instead? + * return a unique_ptr instead? */ return std::make_unique(event.value()); } @@ -347,6 +348,7 @@ void EdbgAvr8Interface::activatePhysical(bool applyExternalReset) { // Try again with external reset applied Logger::debug("Failed to activate physical interface on AVR8 target - retrying with external reset applied."); return this->activatePhysical(true); + } else { throw Exception("Activate physical interface command failed"); }