Added write-memory function to DebuggableDevice

azonenberg · azonenberg · commit 4b276046952e · 2018-07-27T16:20:09.000-07:00
diff --git a/ARM7TDMISProcessor.cpp b/ARM7TDMISProcessor.cpp
@@ -351,7 +351,7 @@ void ARM7TDMISProcessor::DumpRegisters()
 		LogNotice("%10s: %08x\n", GetRegisterName(reg), ReadCPURegister(reg));
 }*/
 
-uint32_t ARM7TDMISProcessor::ReadMemory(uint32_t addr)
+uint32_t ARM7TDMISProcessor::ReadMemory(uint32_t /*addr*/)
 {
 	throw JtagExceptionWrapper(
 		"Memory access not implemented for this CPU yet",
@@ -360,6 +360,13 @@ uint32_t ARM7TDMISProcessor::ReadMemory(uint32_t addr)
 	return 0;
 }
 
+void ARM7TDMISProcessor::WriteMemory(uint32_t /*addr*/, uint32_t /*value*/)
+{
+	throw JtagExceptionWrapper(
+		"Memory access not implemented for this CPU yet",
+		"");
+}
+
 /**
 	@brief Halts the CPU and enters debug state
  */
diff --git a/ARM7TDMISProcessor.h b/ARM7TDMISProcessor.h
@@ -146,6 +146,7 @@ class ARM7TDMISProcessor 	: public DebuggableDevice
 	virtual void PrintRegisters();
 
 	virtual uint32_t ReadMemory(uint32_t addr);
+	virtual void WriteMemory(uint32_t addr, uint32_t value);
 
 	/*
 	////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
diff --git a/DebuggableDevice.cpp b/DebuggableDevice.cpp
@@ -2,7 +2,7 @@
 *                                                                                                                      *
 * ANTIKERNEL v0.1                                                                                                      *
 *                                                                                                                      *
-* Copyright (c) 2012-2016 Andrew D. Zonenberg                                                                          *
+* Copyright (c) 2012-2018 Andrew D. Zonenberg                                                                          *
 * All rights reserved.                                                                                                 *
 *                                                                                                                      *
 * Redistribution and use in source and binary forms, with or without modification, are permitted provided that the     *
@@ -59,3 +59,8 @@ uint32_t DebuggableDevice::ReadMemory(uint32_t addr)
 {
 	return m_iface->ReadMemory(addr);
 }
+
+void DebuggableDevice::WriteMemory(uint32_t addr, uint32_t value)
+{
+	m_iface->WriteMemory(addr, value);
+}
diff --git a/DebuggableDevice.h b/DebuggableDevice.h
@@ -2,7 +2,7 @@
 *                                                                                                                      *
 * ANTIKERNEL v0.1                                                                                                      *
 *                                                                                                                      *
-* Copyright (c) 2012-2016 Andrew D. Zonenberg                                                                          *
+* Copyright (c) 2012-2018 Andrew D. Zonenberg                                                                          *
 * All rights reserved.                                                                                                 *
 *                                                                                                                      *
 * Redistribution and use in source and binary forms, with or without modification, are permitted provided that the     *
@@ -55,6 +55,7 @@ class DebuggableDevice
 	virtual std::string GetDescription() =0;
 
 	virtual uint32_t ReadMemory(uint32_t addr);
+	virtual void WriteMemory(uint32_t addr, uint32_t value);
 
 	////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 	// Debug commands
diff --git a/FTDIJtagInterface.cpp b/FTDIJtagInterface.cpp
@@ -432,18 +432,28 @@ void FTDIJtagInterface::SharedCtorInit(uint32_t type, const std::string& layout)
 		m_gpioDirection.push_back(false);
 	}
 
-	//Digilent HS1 and azonenberg's usb-jtag-mini (0403:8028): GPIOL3 is active HIGH output enable
+	//Digilent HS1 and azonenberg's usb-jtag-mini (0403:8028)
 	if(layout == "hs1")
 	{
+		//GPIOL3 is active HIGH output enable
 		SetGpioDirectionDeferred(3, true);
 		SetGpioValueDeferred(3, true);
 	}
 
-	//JTAGKey (or compatible bus blaster etc): GPIOL0 is active LOW output enable
+	//JTAGKey (or compatible bus blaster etc)
 	else if(layout == "jtagkey")
 	{
+		//GPIOL0 is active LOW output enable
 		SetGpioDirectionDeferred(0, true);
 		SetGpioValueDeferred(0, false);
+
+		//GPIOH0 is TRST_N, hold this high for now since we don't support TRST in the API yet
+		SetGpioDirectionDeferred(4, true);
+		SetGpioValueDeferred(4, true);
+
+		//GPIOH2 is TRST_N_OE_N, hold this high so TRST is driven
+		SetGpioDirectionDeferred(6, true);
+		SetGpioValueDeferred(6, true);
 	}
 
 	else

Original file line number	Diff line number	Diff line change
`@@ -432,18 +432,28 @@ void FTDIJtagInterface::SharedCtorInit(uint32_t type, const std::string& layout)`
`432`	`432`	`m_gpioDirection.push_back(false);`
`433`	`433`	`}`
`434`	`434`
`435`		`- //Digilent HS1 and azonenberg's usb-jtag-mini (0403:8028): GPIOL3 is active HIGH output enable`
	`435`	`+ //Digilent HS1 and azonenberg's usb-jtag-mini (0403:8028)`
`436`	`436`	`if(layout == "hs1")`
`437`	`437`	`{`
	`438`	`+ //GPIOL3 is active HIGH output enable`
`438`	`439`	`SetGpioDirectionDeferred(3, true);`
`439`	`440`	`SetGpioValueDeferred(3, true);`
`440`	`441`	`}`
`441`	`442`
`442`		`- //JTAGKey (or compatible bus blaster etc): GPIOL0 is active LOW output enable`
	`443`	`+ //JTAGKey (or compatible bus blaster etc)`
`443`	`444`	`else if(layout == "jtagkey")`
`444`	`445`	`{`
	`446`	`+ //GPIOL0 is active LOW output enable`
`445`	`447`	`SetGpioDirectionDeferred(0, true);`
`446`	`448`	`SetGpioValueDeferred(0, false);`
	`449`	`+`
	`450`	`+ //GPIOH0 is TRST_N, hold this high for now since we don't support TRST in the API yet`
	`451`	`+ SetGpioDirectionDeferred(4, true);`
	`452`	`+ SetGpioValueDeferred(4, true);`
	`453`	`+`
	`454`	`+ //GPIOH2 is TRST_N_OE_N, hold this high so TRST is driven`
	`455`	`+ SetGpioDirectionDeferred(6, true);`
	`456`	`+ SetGpioValueDeferred(6, true);`
`447`	`457`	`}`
`448`	`458`
`449`	`459`	`else`