Renamed RiscVDebugSpec to RiscVDebug

This commit is contained in:
Nav
2025-02-02 13:32:25 +00:00
parent a5e027b944
commit f3cd55e53f
33 changed files with 65 additions and 65 deletions

View File

@@ -51,6 +51,6 @@ target_sources(
${CMAKE_CURRENT_SOURCE_DIR}/Wch/WchLinkE/WchLinkE.cpp
# RISC-V Debug Translator implementation
${CMAKE_CURRENT_SOURCE_DIR}/Protocols/RiscVDebugSpec/DebugTranslatorConfig.cpp
${CMAKE_CURRENT_SOURCE_DIR}/Protocols/RiscVDebugSpec/DebugTranslator.cpp
${CMAKE_CURRENT_SOURCE_DIR}/Protocols/RiscVDebug/DebugTranslatorConfig.cpp
${CMAKE_CURRENT_SOURCE_DIR}/Protocols/RiscVDebug/DebugTranslator.cpp
)

View File

@@ -2,7 +2,7 @@
#include <cstdint>
namespace DebugToolDrivers::Protocols::RiscVDebugSpec
namespace DebugToolDrivers::Protocols::RiscVDebug
{
using RegisterValue = std::uint32_t;
using RegisterNumber = std::uint16_t;

View File

@@ -2,7 +2,7 @@
#include <cstdint>
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule
namespace DebugToolDrivers::Protocols::RiscVDebug::DebugModule
{
using RegisterAddress = std::uint8_t;
using RegisterValue = std::uint32_t;

View File

@@ -2,9 +2,9 @@
#include <cstdint>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/DebugModule/DebugModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/DebugModule/DebugModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::DebugModule::Registers
{
struct AbstractCommandAutoExecuteRegister
{

View File

@@ -3,9 +3,9 @@
#include <cstdint>
#include <cassert>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/DebugModule/DebugModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/DebugModule/DebugModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::DebugModule::Registers
{
struct AbstractCommandRegister
{

View File

@@ -2,9 +2,9 @@
#include <cstdint>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/DebugModule/DebugModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/DebugModule/DebugModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::DebugModule::Registers
{
struct AbstractControlStatusRegister
{

View File

@@ -3,9 +3,9 @@
#include <cstdint>
#include <cassert>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/DebugModule/DebugModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/DebugModule/DebugModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::DebugModule::Registers
{
struct ControlRegister
{

View File

@@ -2,9 +2,9 @@
#include <cstdint>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/Common.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/Common.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::DebugModule::Registers
{
struct MemoryAccessControlField
{

View File

@@ -2,9 +2,9 @@
#include <cstdint>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/Common.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/Common.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::DebugModule::Registers
{
struct RegisterAccessControlField
{

View File

@@ -2,9 +2,9 @@
#include <cstdint>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/DebugModule/DebugModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/DebugModule/DebugModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::DebugModule::Registers
{
enum class RegisterAddress: RegisterAddress
{

View File

@@ -2,9 +2,9 @@
#include <cstdint>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/DebugModule/DebugModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/DebugModule/DebugModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::DebugModule::Registers
{
struct StatusRegister
{

View File

@@ -12,7 +12,7 @@
#include "src/Targets/TargetMemory.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec
namespace DebugToolDrivers::Protocols::RiscVDebug
{
struct DebugModuleDescriptor
{

View File

@@ -37,7 +37,7 @@
#include "src/Logger/Logger.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec
namespace DebugToolDrivers::Protocols::RiscVDebug
{
using Registers::DebugControlStatusRegister;

View File

@@ -39,7 +39,7 @@
#include "src/Helpers/Expected.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec
namespace DebugToolDrivers::Protocols::RiscVDebug
{
/**
* Implementation of a RISC-V debug translator

View File

@@ -5,7 +5,7 @@
#include "src/Logger/Logger.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec
namespace DebugToolDrivers::Protocols::RiscVDebug
{
DebugTranslatorConfig::DebugTranslatorConfig(const YAML::Node& configNode) {
if (configNode["target_response_timeout"]) {

View File

@@ -6,7 +6,7 @@
#include "DebugModule/DebugModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec
namespace DebugToolDrivers::Protocols::RiscVDebug
{
struct DebugTranslatorConfig
{

View File

@@ -7,7 +7,7 @@
#include "DebugModule/DebugModule.hpp"
#include "DebugModule/Registers/RegisterAddresses.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec
namespace DebugToolDrivers::Protocols::RiscVDebug
{
/**
* Provides access to the RISC-V target's debug transport module (DTM).

View File

@@ -1,8 +1,8 @@
#pragma once
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/Common.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/Common.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::Registers
{
enum class CpuRegisterNumber: RegisterNumber
{

View File

@@ -2,9 +2,9 @@
#include <cstdint>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/Common.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/Common.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::Registers
{
struct DebugControlStatusRegister
{

View File

@@ -3,9 +3,9 @@
#include <cstdint>
#include <optional>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/TriggerModule/TriggerModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/TriggerModule/TriggerModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::TriggerModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::TriggerModule::Registers
{
struct MatchControl
{

View File

@@ -3,9 +3,9 @@
#include <cstdint>
#include <optional>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/TriggerModule/TriggerModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/TriggerModule/TriggerModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::TriggerModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::TriggerModule::Registers
{
struct TriggerData1
{

View File

@@ -4,9 +4,9 @@
#include <set>
#include <array>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/TriggerModule/TriggerModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/TriggerModule/TriggerModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::TriggerModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::TriggerModule::Registers
{
struct TriggerInfo
{

View File

@@ -2,9 +2,9 @@
#include <cstdint>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/TriggerModule/TriggerModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/TriggerModule/TriggerModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::TriggerModule::Registers
namespace DebugToolDrivers::Protocols::RiscVDebug::TriggerModule::Registers
{
/**
* TODO: Given the single, full width bit field, is this struct really necessary? Review.

View File

@@ -5,7 +5,7 @@
#include "TriggerModule.hpp"
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::TriggerModule
namespace DebugToolDrivers::Protocols::RiscVDebug::TriggerModule
{
struct TriggerDescriptor
{

View File

@@ -2,7 +2,7 @@
#include <cstdint>
namespace DebugToolDrivers::Protocols::RiscVDebugSpec::TriggerModule
namespace DebugToolDrivers::Protocols::RiscVDebug::TriggerModule
{
using RegisterValue = std::uint32_t;
using TriggerIndex = std::uint32_t;

View File

@@ -6,7 +6,7 @@
#include "src/DebugToolDrivers/Wch/Protocols/WchLink/Commands/Command.hpp"
#include "src/DebugToolDrivers/Wch/Protocols/WchLink/Responses/DebugModuleInterfaceOperationResponse.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/DebugModule/DebugModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/DebugModule/DebugModule.hpp"
namespace DebugToolDrivers::Wch::Protocols::WchLink::Commands
{
@@ -16,9 +16,9 @@ namespace DebugToolDrivers::Wch::Protocols::WchLink::Commands
using ExpectedResponseType = Responses::DebugModuleInterfaceOperationResponse;
DebugModuleInterfaceOperation(
DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::DmiOperation operation,
DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::RegisterAddress address,
std::optional<DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::RegisterValue> value = std::nullopt
DebugToolDrivers::Protocols::RiscVDebug::DebugModule::DmiOperation operation,
DebugToolDrivers::Protocols::RiscVDebug::DebugModule::RegisterAddress address,
std::optional<DebugToolDrivers::Protocols::RiscVDebug::DebugModule::RegisterValue> value = std::nullopt
)
: Command(0x08)
{

View File

@@ -2,7 +2,7 @@
#include <vector>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/DebugModule/DebugModule.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/DebugModule/DebugModule.hpp"
#include "src/Services/StringService.hpp"
#include "src/TargetController/Exceptions/DeviceCommunicationFailure.hpp"
@@ -12,12 +12,12 @@ namespace DebugToolDrivers::Wch::Protocols::WchLink::Responses
class DebugModuleInterfaceOperationResponse
{
public:
DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::DmiOperationStatus operationStatus;
DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::RegisterAddress address;
DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::RegisterValue value;
DebugToolDrivers::Protocols::RiscVDebug::DebugModule::DmiOperationStatus operationStatus;
DebugToolDrivers::Protocols::RiscVDebug::DebugModule::RegisterAddress address;
DebugToolDrivers::Protocols::RiscVDebug::DebugModule::RegisterValue value;
explicit DebugModuleInterfaceOperationResponse(const std::vector<unsigned char>& payload) {
using DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::DmiOperationStatus;
using DebugToolDrivers::Protocols::RiscVDebug::DebugModule::DmiOperationStatus;
if (payload.size() != 6) {
throw Exceptions::DeviceCommunicationFailure{
@@ -38,7 +38,7 @@ namespace DebugToolDrivers::Wch::Protocols::WchLink::Responses
this->operationStatus = static_cast<DmiOperationStatus>(status);
this->address = payload[0];
this->value = static_cast<DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::RegisterValue>(
this->value = static_cast<DebugToolDrivers::Protocols::RiscVDebug::DebugModule::RegisterValue>(
(payload[1] << 24) | (payload[2] << 16) | (payload[3] << 8) | (payload[4])
);
}

View File

@@ -25,7 +25,7 @@
namespace DebugToolDrivers::Wch::Protocols::WchLink
{
using namespace ::DebugToolDrivers::Protocols::RiscVDebugSpec;
using namespace ::DebugToolDrivers::Protocols::RiscVDebug;
using namespace Exceptions;
using DebugModule::DmiOperation;

View File

@@ -7,7 +7,7 @@
#include <span>
#include <utility>
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/DebugTransportModuleInterface.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/DebugTransportModuleInterface.hpp"
#include "src/DebugToolDrivers/Usb/UsbInterface.hpp"
#include "src/DebugToolDrivers/Usb/UsbDevice.hpp"
@@ -27,7 +27,7 @@ namespace DebugToolDrivers::Wch::Protocols::WchLink
* Implementation of the WCH-Link protocol, which provides an implementation of a RISC-V DTM interface.
*/
class WchLinkInterface
: public ::DebugToolDrivers::Protocols::RiscVDebugSpec::DebugTransportModuleInterface
: public ::DebugToolDrivers::Protocols::RiscVDebug::DebugTransportModuleInterface
{
public:
static constexpr auto MAX_PARTIAL_BLOCK_WRITE_SIZE = Targets::TargetMemorySize{64};
@@ -37,12 +37,12 @@ namespace DebugToolDrivers::Wch::Protocols::WchLink
DeviceInfo getDeviceInfo();
void setClockSpeed(WchLinkTargetClockSpeed speed, WchTargetId targetId);
::DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::RegisterValue readDebugModuleRegister(
::DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::RegisterAddress address
::DebugToolDrivers::Protocols::RiscVDebug::DebugModule::RegisterValue readDebugModuleRegister(
::DebugToolDrivers::Protocols::RiscVDebug::DebugModule::RegisterAddress address
) override;
void writeDebugModuleRegister(
::DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::RegisterAddress address,
::DebugToolDrivers::Protocols::RiscVDebugSpec::DebugModule::RegisterValue value
::DebugToolDrivers::Protocols::RiscVDebug::DebugModule::RegisterAddress address,
::DebugToolDrivers::Protocols::RiscVDebug::DebugModule::RegisterValue value
) override;
void writeFlashPartialBlock(Targets::TargetMemoryAddress startAddress, Targets::TargetMemoryBufferSpan buffer);

View File

@@ -54,7 +54,7 @@ namespace DebugToolDrivers::Wch
, targetDescriptionFile(targetDescriptionFile)
, wchLinkInterface(wchLinkInterface)
, riscVTranslator(
::DebugToolDrivers::Protocols::RiscVDebugSpec::DebugTranslator{
::DebugToolDrivers::Protocols::RiscVDebug::DebugTranslator{
this->wchLinkInterface,
this->toolConfig.riscVDebugTranslatorConfig,
this->targetDescriptionFile,
@@ -389,7 +389,7 @@ namespace DebugToolDrivers::Wch
Targets::TargetMemoryBufferSpan buffer
) {
using Services::AlignmentService;
using namespace ::DebugToolDrivers::Protocols::RiscVDebugSpec;
using namespace ::DebugToolDrivers::Protocols::RiscVDebug;
if (buffer.empty()) {
return;
@@ -527,7 +527,7 @@ namespace DebugToolDrivers::Wch
) {
using Services::AlignmentService;
using Services::StringService;
using namespace ::DebugToolDrivers::Protocols::RiscVDebugSpec;
using namespace ::DebugToolDrivers::Protocols::RiscVDebug;
if (buffer.empty()) {
return;

View File

@@ -9,7 +9,7 @@
#include "Protocols/WchLink/WchLinkInterface.hpp"
#include "WchLinkToolConfig.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/DebugTranslator.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/DebugTranslator.hpp"
#include "src/Targets/TargetMemory.hpp"
#include "src/Targets/TargetAddressSpaceDescriptor.hpp"
@@ -89,7 +89,7 @@ namespace DebugToolDrivers::Wch
const Targets::RiscV::TargetDescriptionFile& targetDescriptionFile;
Protocols::WchLink::WchLinkInterface& wchLinkInterface;
DebugToolDrivers::Protocols::RiscVDebugSpec::DebugTranslator riscVTranslator;
DebugToolDrivers::Protocols::RiscVDebug::DebugTranslator riscVTranslator;
DeviceInfo toolInfo;

View File

@@ -12,7 +12,7 @@ namespace DebugToolDrivers::Wch
}
if (toolNode["riscv_debug_translator"]) {
this->riscVDebugTranslatorConfig = ::DebugToolDrivers::Protocols::RiscVDebugSpec::DebugTranslatorConfig{
this->riscVDebugTranslatorConfig = ::DebugToolDrivers::Protocols::RiscVDebug::DebugTranslatorConfig{
toolNode["riscv_debug_translator"]
};
}

View File

@@ -3,7 +3,7 @@
#include <yaml-cpp/yaml.h>
#include "src/ProjectConfig.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebugSpec/DebugTranslatorConfig.hpp"
#include "src/DebugToolDrivers/Protocols/RiscVDebug/DebugTranslatorConfig.hpp"
namespace DebugToolDrivers::Wch
{
@@ -13,7 +13,7 @@ namespace DebugToolDrivers::Wch
struct WchLinkToolConfig: public DebugToolConfig
{
bool exitIapMode = true;
::DebugToolDrivers::Protocols::RiscVDebugSpec::DebugTranslatorConfig riscVDebugTranslatorConfig = {};
::DebugToolDrivers::Protocols::RiscVDebug::DebugTranslatorConfig riscVDebugTranslatorConfig = {};
explicit WchLinkToolConfig(const DebugToolConfig& toolConfig);
};